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

tests: start candlepin container as privileged on EL 7 #146

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

ptoscano
Copy link
Collaborator

It seems to be needed in EL 7; it should not be a problem, since the tests are usually run on an isolated system.

Ensure to have the right distro facts for the needed check.

It seems to be needed in EL 7; it should not be a problem, since the
tests are usually run on an isolated system.

Ensure to have the right distro facts for the needed check.

Signed-off-by: Pino Toscano <ptoscano@redhat.com>
@ptoscano ptoscano requested a review from richm as a code owner November 22, 2023 07:11
@ptoscano
Copy link
Collaborator Author

[citest]

1 similar comment
@ukulekek
Copy link
Contributor

[citest]

@richm
Copy link
Contributor

richm commented Nov 22, 2023

ok - but note that ci tests will not run on el7 because "7" is not listed as a supported platform at https://github.com/linux-system-roles/rhc/blob/main/meta/main.yml#L13

@ptoscano
Copy link
Collaborator Author

note that ci tests will not run on el7 because "7" is not listed as a supported platform

Yes, that is fine - the actual enablement will come after this.

I sent this because it was already a known bit I needed to change for it; in the meanwhile I tested whether there was anything else needed, and so far it seems not.

@richm richm merged commit e725689 into linux-system-roles:main Nov 27, 2023
23 checks passed
@ptoscano ptoscano deleted the tests-cp-for-el7 branch November 27, 2023 17:52
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.

None yet

3 participants