Skip to content

Conversation

@vvoland
Copy link
Contributor

@vvoland vvoland commented Apr 17, 2025

Test the rootless install in CI

@vvoland vvoland force-pushed the test-rootless-install branch 2 times, most recently from 9093f9d to 737b898 Compare April 17, 2025 17:31
steps:
- uses: actions/checkout@v3
- name: Rootless install
run: FORCE_ROOTLESS_INSTALL=1 ./rootless-install.sh
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a separate workflow step, because we need to check it outside of container (due to lsmod, iptables checks)

Copy link
Member

Choose a reason for hiding this comment

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

Is it worth adding a comment for that? (i.e. why we need to do separate)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a comment

@vvoland vvoland changed the title gha: Add workflow to test install gha: Add workflow to test rootless install Apr 17, 2025
@vvoland
Copy link
Contributor Author

vvoland commented Apr 17, 2025

Fails as expected because it's before: #487

Will rebase it after a merge and it should become green.

@vvoland vvoland force-pushed the test-rootless-install branch 5 times, most recently from c4094d1 to ad4a61b Compare April 17, 2025 17:50
@vvoland vvoland self-assigned this Apr 17, 2025
@vvoland vvoland marked this pull request as ready for review April 17, 2025 17:52
@vvoland vvoland requested review from robmry and thaJeztah April 17, 2025 17:52
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
@vvoland vvoland force-pushed the test-rootless-install branch from ad4a61b to 5c96993 Compare April 17, 2025 18:05
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@thaJeztah thaJeztah merged commit 16bae30 into docker:master Apr 17, 2025
10 checks passed
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.

3 participants