Skip to content

run_container try out#2442

Draft
jelly wants to merge 4 commits into
cockpit-project:mainfrom
jelly:admin-s-reduction
Draft

run_container try out#2442
jelly wants to merge 4 commits into
cockpit-project:mainfrom
jelly:admin-s-reduction

Conversation

@jelly
Copy link
Copy Markdown
Member

@jelly jelly commented Feb 16, 2026

No description provided.

@jelly jelly force-pushed the admin-s-reduction branch from e79bb75 to 166fc51 Compare February 16, 2026 12:52
@jelly
Copy link
Copy Markdown
Member Author

jelly commented Feb 16, 2026

The commit message needs to be expanded with some more reasoning and more moved to run_container

jelly added 3 commits March 19, 2026 10:37
Refactor the invocations to use self.execute where possible, in the
future we want to stop using `SSHConnection` as an API.
The test needs a container to be run with some output, this is fast
enough to not require backgrounding.
Checkpoint restore is only supported on privileged containers, so avoid
`self.execute`.
@jelly jelly force-pushed the admin-s-reduction branch from 166fc51 to 96d673b Compare March 19, 2026 14:52
Introduce a new helper for running containers in the background for
tests without requiring the existing `admin_s` session.
@jelly jelly force-pushed the admin-s-reduction branch from 96d673b to 20f7276 Compare March 19, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant