Cover learners with e2e tests

[serathius]

Based on #13837 I expect we are missing some e2e testing.

[bobsongplus]

@serathius would be ok if I work on this?

[serathius]

Sure, to give you some context, most of etcd e2e tests either run 1 or 3 member cluster. Most test scenarios would benefit if we add a case where there is an additional learner in cluster. For examples of test code I would look into code in tests/common.

[bobsongplus]

Working on it

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 21 days if no further activity occurs. Thank you for your contributions.

[serathius]

@TinySong any progress?

[halegreen]

want to work on this issue

[pacoxu]

kubeadm using etcd learner mode to join the control plane node.

• we add some e2e test in kubeadm: add e2e test for etcd learner mode kubernetes/test-infra#28574

[Elbehery]

@jmhbnz can u assign me please ?

[jmhbnz]

Absolutely! Thanks for volunteering 🙏🏻

/assign @Elbehery

[Elbehery]

Hello @serathius

I need more context to resolve this, below are initial ideas

• add IsLearner field to

  etcd/tests/framework/e2e/etcd_process.go

  Line 80 in f37a436

  type EtcdServerProcessConfig struct {

• add AddMemberAsLearner to e2e interface

i am quiet lost, where to start .. also could you inline some specific cases to cover ?

thanks :)

cc @ahrtr @jmhbnz

[serathius]

@Elbehery we would want to add learners to basic e2e tests scenarios that should not care that there is a learner in the cluster. Take a look into https://github.com/etcd-io/etcd/blob/main/tests/common/e2e_test.go on how we automatically generate subtests for different cluster configurations.

[Elbehery]

Thanks @serathius for your response ..

I reviewed the tests yesterday .. So for example, for https://github.com/etcd-io/etcd/blob/main/tests/common/member_test.go, we should add a test that adds a learner member ?

also to create an etcd learner member server process, do i follow the same EtcdServerProcessConfig while adding a boolean for IsLearner ?

wdyt ?

[serathius]

I reviewed the tests yesterday .. So for example, for https://github.com/etcd-io/etcd/blob/main/tests/common/member_test.go, we should add a test that adds a learner member ?

I think there already exists a test for adding a learner member.

etcd/tests/e2e/ctl_v3_member_test.go

Line 46 in 58846bd

func TestCtlV3MemberAddAsLearner(t *testing.T) { testCtl(t, memberAddAsLearnerTest) }

Context on why we have two directories with tests #13637

[serathius]

Any progress @Elbehery ?

[Elbehery]

No .. would you have sometime for a call to discuss it ?

[serathius]

You can reach me on K8s slack https://communityinviter.com/apps/kubernetes/community