Skip to content

Conversation

@ahrtr
Copy link
Member

@ahrtr ahrtr commented Mar 14, 2025

cc @fuweid @ivanvc @jmhbnz @serathius @siyuanfoundation

Note main & release-3.6 do not have this issue.

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ahrtr, serathius

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

@ahrtr
Copy link
Member Author

ahrtr commented Mar 14, 2025

@serathius thx for the quick review!

@ahrtr ahrtr merged commit 3c65dfa into etcd-io:release-3.5 Mar 14, 2025
25 checks passed
@ahrtr ahrtr deleted the 3.5_add_learner_test_20250314 branch March 14, 2025 13:36
@serathius
Copy link
Member

serathius commented Mar 14, 2025

Oh, Note main & release-3.6 do not have this issue. How?

I'm just surprised.

@serathius
Copy link
Member

serathius commented Mar 14, 2025

Oh, also can you add TestCtlV3PromotingLearner test to all branches if it's not present? No need to block, maybe just create a issue to track it.

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

Development

Successfully merging this pull request may close these issues.

3 participants