Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Bug 2034484: Upgrade library-go version #1247

Merged
merged 2 commits into from Jan 27, 2022
Merged

Bug 2034484: Upgrade library-go version #1247

merged 2 commits into from Jan 27, 2022

Conversation

qJkee
Copy link
Contributor

@qJkee qJkee commented Dec 9, 2021 •
edited

Bumping the library-go dependency to latest, this should take advantage of leader election changes for SNO clusters proposed in this library-go PR and performance improvements in this PR.

@qJkee
Copy link
Contributor Author

qJkee commented Dec 9, 2021

/retest

@qJkee qJkee marked this pull request as draft December 9, 2021 17:35
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 9, 2021
@qJkee qJkee marked this pull request as ready for review December 13, 2021 18:37
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 13, 2021
@qJkee qJkee changed the title Upgrade library-go version [WIP] Upgrade library-go version Dec 13, 2021
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 13, 2021
@qJkee qJkee marked this pull request as draft December 13, 2021 19:26
@qJkee qJkee closed this Dec 14, 2021
@qJkee qJkee reopened this Dec 14, 2021
@qJkee qJkee marked this pull request as ready for review December 14, 2021 15:35
@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 14, 2021
@openshift-ci openshift-ci bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 15, 2021
@qJkee
Copy link
Contributor Author

qJkee commented Dec 15, 2021

/retest-required

1 similar comment
@qJkee
Copy link
Contributor Author

qJkee commented Dec 16, 2021

/retest-required

@qJkee
Copy link
Contributor Author

qJkee commented Dec 17, 2021

/retest-required

1 similar comment
@qJkee
Copy link
Contributor Author

qJkee commented Dec 17, 2021

/retest-required

@jerpeter1
Copy link
Member

/retest-required

1 similar comment
@qJkee
Copy link
Contributor Author

qJkee commented Dec 23, 2021

/retest-required

@ggiguash
Copy link

/retest-required

1 similar comment
@qJkee
Copy link
Contributor Author

qJkee commented Dec 28, 2021

/retest-required

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

19 similar comments
@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 26, 2022 •
edited

@qJkee: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-openstack-ovn a8f078f link false /test e2e-openstack-ovn
ci/prow/e2e-gcp-ovn-upgrade a8f078f link false /test e2e-gcp-ovn-upgrade

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.

@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

1 similar comment
@openshift-bot
Copy link
Contributor

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@qJkee qJkee changed the title Bug 2034484: Upgrade library-go version [WIP] Bug 2034484: Upgrade library-go version Jan 27, 2022
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 27, 2022
@qJkee qJkee changed the title [WIP] Bug 2034484: Upgrade library-go version Bug 2034484: Upgrade library-go version Jan 27, 2022
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 27, 2022
@qJkee
Copy link
Contributor Author

qJkee commented Jan 27, 2022

/retest-required

@openshift-merge-robot openshift-merge-robot merged commit 7688798 into openshift:master Jan 27, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 27, 2022

@qJkee: Bugzilla bug 2034484 is in an unrecognized state (ON_QA) and will not be moved to the MODIFIED state.

In response to this:

Bug 2034484: Upgrade library-go version

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JacobTanenbaum JacobTanenbaum Awaiting requested review from JacobTanenbaum

@rcarrillocruz rcarrillocruz Awaiting requested review from rcarrillocruz

Assignees

@squeed squeed

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. bugzilla/severity-high Referenced Bugzilla bug's severity is high for the branch this PR is targeting. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
8 participants
@qJkee @jerpeter1 @ggiguash @eggfoobar @danwinship @squeed @openshift-bot @openshift-merge-robot