-
Notifications
You must be signed in to change notification settings - Fork 42
NETOBSERV-2494: refresh cluster info periodically #2152
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
base: main
Are you sure you want to change the base?
Conversation
- Refactor ClusterInfo to improve how readiness works - Fetch Network resource via ClusterInfo to get CNI - Ignore netpol creation with openshift-sdn - For unknown CNI, allow netpol creation with a warning - Changed default rules: it's now only with ovn that it's enabled by default
- Periodic refresh of cluser info; it requires a mutex - Status update after refresh - Fix issue with static plugin showing "Could not determine if static plugin is supported" because cluster info isn't ready (this error still happens but now it will be retried instead of ignoring the error)
|
@jotak: This pull request references NETOBSERV-2494 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.21.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
OlivierCazade
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Refresh cluster info periodically
plugin is supported" because cluster info isn't ready (this error
still happens but now it will be retried instead of ignoring the error)
Dependencies
n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.