-
Notifications
You must be signed in to change notification settings - Fork 4.8k
OKD-259: Test OKD Feature set Against OKD and OCP clusters #30693
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
|
@jatinsu: This pull request references OKD-259 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 epic to target the "4.22.0" version, but no target version was set. DetailsIn 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. |
|
Pipeline controller notification For optional jobs, comment This repository is configured in: automatic mode |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: jatinsu The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
| fg, err := fgClient.Get(context.Background(), "cluster", metav1.GetOptions{}) | ||
| o.Expect(err).NotTo(o.HaveOccurred(), "Failed to get cluster FeatureGate") | ||
|
|
||
| // 3. Save original featureset for restoration |
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.
The OKD feature set doesn't allow for a cluster to transition back to default. Should I
- Have a check for when a cluster's original featureset is default so it doesn't fail
- Completely omit this as OKD clusters should have the OKD featureset and wouldn't need to switch back to the original featureset
e4ffb11 to
20c9964
Compare
20c9964 to
45e89b6
Compare
|
Scheduling required tests: Scheduling tests matching the |
|
@jatinsu: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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 kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: 4459b59
New tests seen in this PR at sha: 4459b59
|
This test checks for OKD and OCP clusters and attempts to enable the OKD featureset against them. If it's a OKD cluster, it should be able to enabled, but if it's a OCP cluster it should reject.
Kubenertes PR for context: openshift/kubernetes#2420