fabric icon indicating copy to clipboard operation
fabric copied to clipboard

WIP: prepare for etcd version bump.

Open guoger opened this issue 3 years ago • 13 comments

Signed-off-by: Jay Guo [email protected]

guoger avatar Jun 23 '21 16:06 guoger

/ci-run

guoger avatar Jun 24 '21 01:06 guoger

AZP build triggered!

github-actions[bot] avatar Jun 24 '21 01:06 github-actions[bot]

/ci-run

guoger avatar Jul 27 '21 11:07 guoger

/ci-run

denyeart avatar Jul 29 '21 20:07 denyeart

Not sure why the /ci-run trigger isn't working, maybe somebody from GitHub would know @lindluni

denyeart avatar Jul 29 '21 20:07 denyeart

/azp run

denyeart avatar Jul 29 '21 20:07 denyeart

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Jul 29 '21 20:07 azure-pipelines[bot]

@guoger Still working on this?

denyeart avatar Aug 30 '21 20:08 denyeart

@guoger Still working on this?

sorry for late reply and yes i still have some draft code laying around.

missing pieces are:

  • initializing node list when etcdraft instance starts (there's a TODO in code)
  • actually bump etcd/raft lib to sane version
  • add more UTs

guoger avatar Sep 01 '21 06:09 guoger

  • initializing node list when etcdraft instance starts (there's a TODO in code)

@guoger If I understand correctly, as per the current implementation, the consenters are set based on the config block when the raft instance gets initialized. I am trying to understand and relate the need for managing the node list in the node struct. Could you help me here to understand.

Param-S avatar Feb 09 '22 09:02 Param-S

hi @guoger and all,

what's the progress for this pr? :-)

SamYuan1990 avatar Apr 16 '22 13:04 SamYuan1990

hi @guoger and all,

what's the progress for this pr? :-)

will revive it this week, should be able to update PR in couple of days

guoger avatar Jul 03 '22 15:07 guoger

@guoger etcd was updated in https://github.com/hyperledger/fabric/pull/2997 , anything else needed for this?

denyeart avatar Aug 25 '22 14:08 denyeart

@guoger Reminder...

@guoger etcd was updated in #2997 , anything else needed for this?

denyeart avatar Oct 03 '22 16:10 denyeart

No response from contributor, closing.

denyeart avatar Jan 18 '23 14:01 denyeart