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

Add openshift-ovn-kubernetes NS to permitted NS list in admin actions #3998

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tsatam
Copy link
Collaborator

@tsatam tsatam commented Dec 9, 2024

Which issue this PR addresses:

Fixes ARO-13390

What this PR does / why we need it:

Adds the openshift-ovn-kubernetes namespace to the list of OpenShift namespaces permitted for SREs to access using admin API endpoints.

Test plan for issue:

  • Simple unit test was added to ensure this namespace is returned by the list

Is there any documentation that needs to be updated for this PR?

No

How do you know this will function as expected in production?

We will have to see if it works in production or not, we cannot replicate the full SRE admin environment locally without extreme effort.

@tsatam
Copy link
Collaborator Author

tsatam commented Dec 9, 2024

/azp run ci

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@yjst2012 yjst2012 left a comment

Choose a reason for hiding this comment

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

Thanks for putting this together

@tsatam
Copy link
Collaborator Author

tsatam commented Dec 11, 2024

/azp run ci

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kimorris27
Copy link
Contributor

/azp run ci

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kimorris27
Copy link
Contributor

E2E has failed with Timed out waiting for condition during the cluster install twice in a row. I'm going to try it one more time, and then if it fails again we may need to dig deeper.

@kimorris27
Copy link
Contributor

/azp run ci

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

6 participants