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: Enable beibooting and install standard cockpit pages #21042

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

martinpitt
Copy link
Member

@martinpitt martinpitt commented Sep 26, 2024

This provides similar functionality as the Client, so that the ws
bastion container can connect to machines which don't have any Cockpit
packages installed.

Install cockpit-system and cockpit-networkmanager into the container for
now. We'll likely expand this in the future, but let's start small.

https://issues.redhat.com/browse/COCKPIT-954


cockpit/ws container: Connect to servers without installed Cockpit

The quay.io/cockpit/ws container provides Cockpit's web server in environments such as Kubernetes or Fedora CoreOS. It connects to a destination machine via ssh.

Similarly to what the Cockpit Client flatpak supports since release 295, that container can now connect to hosts which don't have any Cockpit packages installed. This version starts a basic set of pages (Overview, Metrics, Terminal, Accounts, and Networking). We will add further pages soon, after vetting them for this use case.

@martinpitt martinpitt added blocked Don't land until something else happens first (see task list) no-test For doc/workflow changes, or experiments which don't need a full CI run, labels Sep 26, 2024
Comment on lines 863 to 864
console.error("login: got unexpected host key prompt, expecting login-data placeholder:", key);
fatal(_("Internal protocol error"));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These 2 added lines are not executed by any test.

@martinpitt martinpitt removed the blocked Don't land until something else happens first (see task list) label Sep 30, 2024
This provides similar functionality as the Client, so that the ws
bastion container can connect to machines which don't have any Cockpit
packages installed.

Install cockpit-system and cockpit-networkmanager into the container
(we'll likely expand this in the future, but let's start small). Take
care to only unpack the RPMs, to avoid installing e.g. NetworkManager
and other big dependencies into the container -- they are useless there.

Update the documentation accordingly. While at it, add an introduction
paragraph what Cockpit is.

https://issues.redhat.com/browse/COCKPIT-954

If you want to also run a web server to log in directly on the CoreOS host, you
can use this container in two modes.
This web server can be deployed as container. It has two modes, which are described below.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"as a container"

Copy link
Member

@mvollmer mvollmer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammo in docs, otherwise looks good to me! thanks!

@martinpitt martinpitt merged commit ab36148 into cockpit-project:main Oct 1, 2024
35 checks passed
@martinpitt martinpitt deleted the ws-beiboot branch October 1, 2024 09:08
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.

3 participants