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.

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

OCPBUGS-3101: Rebase openshift/etcd 4.11 onto v3.5.6 #170

Merged

Conversation

tjungblu
Copy link

@tjungblu tjungblu commented Dec 7, 2022

No description provided.

Haimantika and others added 30 commits September 19, 2022 15:35
…g-jwt/jwt/v4

Signed-off-by: haimantika mitra <haimantikamitra@gmail.com>
Signed-off-by: Davanum Srinivas <davanum@gmail.com>
Signed-off-by: Hitoshi Mitake <h.mitake@gmail.com>
Signed-off-by: Hitoshi Mitake <h.mitake@gmail.com>
Signed-off-by: Kafuu Chino <KafuuChinoQ@gmail.com>

add test

1

1

1

1

1

1
*: avoid closing a watch with ID 0 incorrectly
Signed-off-by: Manuel Rüger <manuel@rueg.eu>
ubuntu 20.10 is EOL and fails with
E: The repository 'http://security.ubuntu.com/ubuntu groovy-security Release' does not have a Release file.

Signed-off-by: Manuel Rüger <manuel@rueg.eu>
If one of the nodes in the cluster has lost a dns record,
restarting the second node will break it.
This PR makes an attempt to add a comparison without using a resolver,
which allows to protect cluster from dns errors and does not break
the current logic of comparing urls in the URLStringsEqual function.
You can read more in the issue etcd-io#7798

Fixes etcd-io#7798

Signed-off-by: Prasad Chandrasekaran <prasadc@vmware.com>
Co-authored-by: Lili Cosic <cosiclili@gmail.com>
Signed-off-by: Prasad Chandrasekaran <prasadc@vmware.com>
Signed-off-by: Prasad Chandrasekaran <prasadc@vmware.com>
Signed-off-by: Benjamin Wang <wachao@vmware.com>
[3.5] etcdserver: added more debug log for the purgeFile goroutine
[Release-3.5] Makefile: additional logic fix / Update Ubuntu base
…able

The client retries connection without backoff when the server is gone
after the watch stream is established. This results in high CPU usage
in the client process. This change introduces backoff when the stream is
failed and unavailable.

Signed-off-by: Hisanobu Tomari <posco.grubb@gmail.com>
…-3.5

[3.5] client/v3: Add backoff before retry when watch stream returns unavailable
Release-3.5: server/etcdmain: add configurable cipher list to gRPC proxy listener
Signed-off-by: 王霄霄 <1141195807@qq.com>
…PermCache

Signed-off-by: Hitoshi Mitake <h.mitake@gmail.com>
…d-io#14574

Signed-off-by: Oleg Guba <oleg@dropbox.com>
Signed-off-by: Hitoshi Mitake <h.mitake@gmail.com>
[3.5] server: refresh auth info when etcd recovers from a snapshot
…o when recovering from a snapshot

Signed-off-by: Benjamin Wang <wachao@vmware.com>
test: added e2e test case for issue 14571: etcd doesn't load auth info when recovering from a snapshot
Check the client count before creating the ephemeral key, do not
create the key if there are already too many clients. Check the
count after creating the key again, if the total kvs is bigger
than the expected count, then check the rev of the current key,
and take action accordingly based on its rev. If its rev is in
the first ${count}, then it's valid client, otherwise, it should
fail.

Signed-off-by: Benjamin Wang <wachao@vmware.com>
Signed-off-by: jianfei.zhang <jianfei.zhang@daocloud.io>
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 15, 2022
@openshift-ci
Copy link

openshift-ci bot commented Dec 15, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dusk125, Elbehery, tjungblu

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@geliu2016
Copy link

/label cherry-pick-approved

@openshift-ci openshift-ci bot added the cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. label Dec 16, 2022
@tjungblu
Copy link
Author

tjungblu commented Jan 2, 2023

/jira refresh

@openshift-ci-robot
Copy link

@tjungblu: This pull request references Jira Issue OCPBUGS-3101, which is invalid:

  • expected dependent Jira Issue OCPBUGS-3100 to be in one of the following states: VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), but it is POST instead

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

/jira refresh

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.

@tjungblu
Copy link
Author

tjungblu commented Jan 2, 2023

@geliu2016 did you get a chance to test OCPBUGS-3100 already?

@tjungblu
Copy link
Author

tjungblu commented Jan 3, 2023

/jira refresh

@openshift-ci-robot openshift-ci-robot added jira/valid-bug Indicates that a referenced Jira bug is valid 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. and removed jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. labels Jan 3, 2023
@openshift-ci-robot
Copy link

@tjungblu: This pull request references Jira Issue OCPBUGS-3101, which is valid. The bug has been moved to the POST state.

6 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.11.z) matches configured target version for branch (4.11.z)
  • bug is in the state ASSIGNED, which is one of the valid states (NEW, ASSIGNED, POST)
  • dependent bug Jira Issue OCPBUGS-3100 is in the state Verified, which is one of the valid states (VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE))
  • dependent Jira Issue OCPBUGS-3100 targets the "4.12.0" version, which is one of the valid target versions: 4.12.0
  • bug has dependents

In response to this:

/jira refresh

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.

@tjungblu
Copy link
Author

tjungblu commented Jan 3, 2023

/retest-required

5 similar comments
@tjungblu
Copy link
Author

tjungblu commented Jan 3, 2023

/retest-required

@tjungblu
Copy link
Author

tjungblu commented Jan 3, 2023

/retest-required

@tjungblu
Copy link
Author

tjungblu commented Jan 3, 2023

/retest-required

@tjungblu
Copy link
Author

tjungblu commented Jan 3, 2023

/retest-required

@tjungblu
Copy link
Author

tjungblu commented Jan 3, 2023

/retest-required

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 53284bc and 2 for PR HEAD cecbe35 in total

@tjungblu
Copy link
Author

tjungblu commented Jan 4, 2023

/retest-required

2 similar comments
@tjungblu
Copy link
Author

tjungblu commented Jan 4, 2023

/retest-required

@tjungblu
Copy link
Author

tjungblu commented Jan 4, 2023

/retest-required

@openshift-ci
Copy link

openshift-ci bot commented Jan 4, 2023

@tjungblu: all tests passed!

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-merge-robot openshift-merge-robot merged commit c50e9aa into openshift:openshift-4.11 Jan 4, 2023
@openshift-ci-robot
Copy link

@tjungblu: An error was encountered searching for bug OCPBUGS-3101 on the Jira server at https://issues.redhat.com/. No known errors were detected, please see the full error message for details.

Full error message. You do not have the permission to see the specified issue.: request failed. Please analyze the request body for more details. Status code: 401:

Please contact an administrator to resolve this issue, then request a bug refresh with /jira refresh.

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.

@tjungblu tjungblu deleted the openshift-411-etc356 branch January 4, 2023 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet