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/kola: add ext.config.networking.nm-ifcfg-rh-plugin test #1994

Merged
merged 1 commit into from
Mar 14, 2023

Conversation

dustymabe
Copy link
Member

mike-nguyen
mike-nguyen previously approved these changes Sep 30, 2022
Copy link
Member

@mike-nguyen mike-nguyen left a comment

Choose a reason for hiding this comment

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

LGTM

@dustymabe
Copy link
Member Author

I'll wait for a review from @travier - so he can comment on what we should do for RHCOS9 and SCOS here.

@travier
Copy link
Member

travier commented Sep 30, 2022

Well, I'm waiting on feedback on the RHCOS issue 😅. Let's wait for next week to see if we have objections.

@travier travier added the hold label Sep 30, 2022
@travier
Copy link
Member

travier commented Mar 13, 2023

Can we add that check for RHCOS 9 too? We'll keep it for now. Thanks

This test checks that the ifcfg-rh NetworkManager plugin is in
the state we expect it to be in. Some more context in:

- coreos#1993
- coreos#1991
- openshift/os#1022
@dustymabe dustymabe merged commit c357ecd into coreos:testing-devel Mar 14, 2023
@dustymabe dustymabe deleted the dusty-kola-test branch March 14, 2023 19:01
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