Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

containers/ws: Include cockpit-files #21558

Merged
merged 1 commit into from
Jan 29, 2025
Merged

Conversation

martinpitt
Copy link
Member

@martinpitt martinpitt commented Jan 28, 2025

This brings the ws container closer to the Client flatpak. In beiboot mode, Files works everywhere including RHEL 8 [1].

[1] cockpit-project/cockpit-files#917


Tested locally with

podman build -t localhost/ws containers/ws
podman run -d --name cockpit-bastion -p 9999:9090 localhost/ws

and then logging into host.containers.internal (i.e. my laptop) on https://localhost:9999.

cockpit/ws container: Include cockpit-files

The cockpit/ws container now includes cockpit-files. When you log into a remote machine that does not have any cockpit packages installed (package-less mode), the "Files" page is now available.

This brings the ws container closer to the Client flatpak. In beiboot
mode, Files works everywhere including RHEL 8 [1].

[1] cockpit-project/cockpit-files#917
@martinpitt martinpitt added no-test For doc/workflow changes, or experiments which don't need a full CI run, release-note labels Jan 28, 2025
@martinpitt
Copy link
Member Author

FTR, we should clearly add podman and machines as well, I just thought I'd start small.

@martinpitt martinpitt requested a review from jelly January 28, 2025 11:31
@jelly jelly merged commit 91ad819 into cockpit-project:main Jan 29, 2025
34 checks passed
@martinpitt martinpitt deleted the ws-files branch January 29, 2025 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-test For doc/workflow changes, or experiments which don't need a full CI run,
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants