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
auto-bumper: Skip kube-secondary-dns #1474
Conversation
considering this fails, i will check your approach |
Until go-git/go-git#485 is fixed since it blocks the auto bumper when cloning the KSD repository. https://github.com/kubevirt/cluster-network-addons-operator/actions/runs/3629947036/jobs/6122730934 Signed-off-by: Or Shoval <oshoval@redhat.com>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Addressed comments, changed to skip the component instead of trying it |
Bump KubeSecondaryDNS manually, since auto-bumper has problems. See kubevirt#1474 Signed-off-by: Or Shoval <oshoval@redhat.com>
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
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: RamLavi 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 |
Thanks |
Bump KubeSecondaryDNS manually, since auto-bumper has problems. See kubevirt#1474 Signed-off-by: Or Shoval <oshoval@redhat.com>
Is there a presubmit lane that checks auto-bumper compiles ? |
Bump KubeSecondaryDNS manually, since auto-bumper has problems. See kubevirt#1474 Signed-off-by: Or Shoval <oshoval@redhat.com>
Bump KubeSecondaryDNS manually, since auto-bumper has problems. See kubevirt#1474 Signed-off-by: Or Shoval <oshoval@redhat.com>
* ksd: Deploy kubevirt Needed for the e2e tests Signed-off-by: Or Shoval <oshoval@redhat.com> * ksd: Bump manually to v0.0.7 Bump KubeSecondaryDNS manually, since auto-bumper has problems. See #1474 Signed-off-by: Or Shoval <oshoval@redhat.com> * ksd: Fix bump script Since the fields might have empty string, need to use set instead of update. Signed-off-by: Or Shoval <oshoval@redhat.com> * ksd: Run bump script (v0.0.7) Signed-off-by: Or Shoval <oshoval@redhat.com> Signed-off-by: Or Shoval <oshoval@redhat.com>
What this PR does / why we need it:
Skip KubeSecondaryDNS from auto-bumper.
Until go-git/go-git#485 is merged since
it blocks the auto bumper when cloning the KSD repository.
https://github.com/kubevirt/cluster-network-addons-operator/actions/runs/3629947036/jobs/6122730934
See go-git/go-git#495
Special notes for your reviewer:
Release note: