Closed tjungblu closed 2 months ago
/lgtm /approve
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Elbehery, tjungblu
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@tjungblu: This pull request explicitly references no jira issue.
/override ci/prow/e2e-aws-ovn-etcd-scaling
@tjungblu: Overrode contexts on behalf of tjungblu: ci/prow/e2e-aws-ovn-etcd-scaling
@tjungblu: 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-metal-ovn-ha-cert-rotation-shutdown | 51c6aef563b3767cfd592617328c8ebde0aec9c6 | link | false | /test e2e-metal-ovn-ha-cert-rotation-shutdown |
ci/prow/e2e-metal-ovn-sno-cert-rotation-shutdown | 51c6aef563b3767cfd592617328c8ebde0aec9c6 | link | false | /test e2e-metal-ovn-sno-cert-rotation-shutdown |
ci/prow/e2e-aws-etcd-recovery | 51c6aef563b3767cfd592617328c8ebde0aec9c6 | link | false | /test e2e-aws-etcd-recovery |
ci/prow/e2e-aws-etcd-certrotation | 51c6aef563b3767cfd592617328c8ebde0aec9c6 | link | false | /test e2e-aws-etcd-certrotation |
Full PR test history. Your PR dashboard.
/cherrypick release-4.16 release-4.15
@Elbehery: #1300 failed to apply on top of branch "release-4.16":
Applying: only update envvar listeners on real changes
Using index info to reconstruct a base tree...
M pkg/etcdenvvar/envvarcontroller.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/etcdenvvar/envvarcontroller.go
CONFLICT (content): Merge conflict in pkg/etcdenvvar/envvarcontroller.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 only update envvar listeners on real changes
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
cherrypicked manually https://github.com/openshift/cluster-etcd-operator/pull/1302
[ART PR BUILD NOTIFIER]
Distgit: cluster-etcd-operator This PR has been included in build cluster-etcd-operator-container-v4.17.0-202407261717.p0.g428a8fd.assembly.stream.el9. All builds following this will include this PR.
While reviewing the changes in the backup PR, I've seen that I've introduced unnecessary listener updates during the lock refactoring two years ago.
This PR ensures we return early when the envvars has not changed.