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
SPLAT-1452: Changed vcenter MaxItems to be 3 for TechPreview. #1842
Conversation
@vr4manta: This pull request references SPLAT-1452 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 task to target the "4.16.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. |
Skipping CI for Draft Pull Request. |
Hello @vr4manta! Some important instructions when contributing to openshift/api: |
@vr4manta: This pull request references SPLAT-1452 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 task to target the "4.16.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. |
e26c54f
to
ebbded6
Compare
cbbd567
to
5cbb0c6
Compare
4011c87
to
00387ac
Compare
@@ -0,0 +1,103 @@ | |||
apiVersion: apiextensions.k8s.io/v1 # Hack because controller-gen complains if we don't have this | |||
name: "Shamefully missing" |
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.
Can we make this more appropriate to the feature/types, see how we format this for other tests
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.
This test file confused me. For some reason the tooling said I had to add it when I didn't make a direct change to it. I copied the same test as VSphereControlPlaneMachineSet.yaml. In fact, reviewing all the other tests for controllerconfigs, they are the same copy.
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.
I updated the test name the new convention. We'll need to update the rest of the tests in that directory at a later time.
name: pull-secret | ||
namespace: openshift-config | ||
releaseImage: "" | ||
rootCAData: Y2VydGlmaWNhdGUK |
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.
Nit, missing a new line at the end here
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.
sorry. I copied from other files in the same directory. I'll add the new line.
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
/override ci/prow/verify-crd-schema Pre-existing failures being added to the new FeatureGated version of the file |
@JoelSpeed: Overrode contexts on behalf of JoelSpeed: ci/prow/verify-crd-schema 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 kubernetes/test-infra repository. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jcpowermac, JoelSpeed, rvanderp3, vr4manta The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/override ci/prow/verify-crd-schema Pre-existing failures, see previous |
@JoelSpeed: Overrode contexts on behalf of JoelSpeed: ci/prow/verify-crd-schema 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 kubernetes/test-infra repository. |
@vr4manta: all tests passed! Full PR test history. Your PR dashboard. 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 kubernetes/test-infra repository. I understand the commands that are listed here. |
[ART PR BUILD NOTIFIER] This PR has been included in build ose-cluster-config-api-container-v4.16.0-202405081611.p0.g95e2292.assembly.stream.el9 for distgit ose-cluster-config-api. |
SPLAT-1452
Changes
Dependencies
Blocks