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
Bug 1874928: Pinning the OVN version for 4.5.8 release #248
Bug 1874928: Pinning the OVN version for 4.5.8 release #248
Conversation
@alexanderConstantinescu: No Bugzilla bug is referenced in the title of this pull request. 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. |
As long as there's a Jira card and commitment to revert this PR after 4.5.7 ships, ok with me. |
There is for the commitment to revert. Once openshift/cluster-network-operator#755 merges and has been back-ported, we'll be able to revert this. |
/lgtm |
@dcbw we are tracking it with the urgent bug https://bugzilla.redhat.com/show_bug.cgi?id=1874737 for a 4.5 revert. |
/retest |
From build:
Version pin worked, but might need to be supplied to all packages. |
Signed-off-by: Alexander Constantinescu <aconstan@redhat.com>
6e6e6ad
to
998225f
Compare
@alexanderConstantinescu: This pull request references Bugzilla bug 1874928, which is invalid:
Comment 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. |
/bugzilla refresh |
@knobunc: This pull request references Bugzilla bug 1874928, which is invalid:
Comment 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: alexanderConstantinescu, knobunc 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 |
Why I overrode: There is no 4.6 change for this since we do not want to change the ovn version there. The override was needed so that we could pin the ovn version in 4.5 so that we don't have a schema upgrade yet. We have a 4.6 PR to support schema upgrades properly (openshift/cluster-network-operator#755) with 4.6 and 4.5 tracking bugs. Once that PR lands and is backported to 4.5 we will revert this change. |
Seems to be properly pinned now:
|
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/override ci/prow/e2e-gcp-ovn-upgrade We have test runs of gcp-ovn that passed on clusterbot, and the failures we had have been flakes in the past. |
/override ci/prow/e2e-gcp-ovn-upgrade |
@knobunc: Overrode contexts on behalf of knobunc: ci/prow/e2e-gcp-ovn-upgrade 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. |
@alexanderConstantinescu: All pull requests linked via external trackers have merged: Bugzilla bug 1874928 has been moved to the MODIFIED state. 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. |
Pin the version for 4.5.8 as to avoid problems during the upgrade from 4.5.7 -> 4.5.8
/assign @knobunc @joepvd
- What this PR does and why is it needed
- Special notes for reviewers
- How to verify it
- Description for the changelog