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
Ensure we can specify different CDI CR name. #1383
Conversation
pkg/controller/util.go
Outdated
|
||
return &cr.Spec.Workloads, nil | ||
if len(crList.Items) != 1 { | ||
return nil, fmt.Errorf("Number of CDI CRs != 0") |
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.
!= 1 should be the message
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.
Yup fixed.
Fix bug where the CR name was hard coded to CDI and it failed in HCO. Signed-off-by: Alexander Wels <awels@redhat.com>
9390f78
to
f530583
Compare
/cherrypick release-v1.23 |
@awels: once the present PR merges, I will cherry-pick it on top of release-v1.23 in a new PR and assign it to you. 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 |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: awels 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 |
@awels: #1383 failed to apply on top of branch "release-v1.23":
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. |
Fix bug where the CR name was hard coded to CDI and it failed in HCO. Signed-off-by: Alexander Wels <awels@redhat.com>
Fix bug where the CR name was hard coded to CDI and it failed in HCO. Signed-off-by: Alexander Wels <awels@redhat.com>
Fix bug where the CR name was hard coded to CDI and it failed in HCO.
Signed-off-by: Alexander Wels awels@redhat.com
What this PR does / why we need it:
HCO names the CDI CR a different name than CDI, and we had hard coded a lookup of the CDI CR as cdi, and it failed when deployed with HCO. This PR fixes that issue, and adds some code so we can specify a CI lane that has a different cr name so we can guard against this happening again.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Release note: