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

Init from etcd: basic support #590

Merged
merged 5 commits into from
Apr 4, 2024
Merged

Init from etcd: basic support #590

merged 5 commits into from
Apr 4, 2024

Conversation

reshke
Copy link
Collaborator

@reshke reshke commented Apr 2, 2024

change router initialization in clustered setup mechanics.
Router requests current metadata setup from qdb on start. Router sync metadata routine now sync only current coordinator address for router proxy queries.

@reshke reshke force-pushed the initetcd branch 3 times, most recently from 2b51656 to b61736a Compare April 2, 2024 15:53
@reshke reshke changed the title Init etcd basics Init from etcd: basic suppoer Apr 2, 2024
@reshke reshke changed the title Init from etcd: basic suppoer Init from etcd: basic support Apr 2, 2024
@reshke reshke force-pushed the initetcd branch 2 times, most recently from f95c11e to b80edfd Compare April 3, 2024 06:40
Copy link
Collaborator

@Denchick Denchick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to add tests that check whether or not the initialization of metadata for different types of installations actually works.

@reshke reshke force-pushed the initetcd branch 3 times, most recently from ef6fc01 to ad158db Compare April 4, 2024 13:56
examples/coordinator-init.yaml Outdated Show resolved Hide resolved
Copy link
Collaborator

@Denchick Denchick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add comments

@reshke reshke disabled auto-merge April 4, 2024 18:10
@reshke reshke enabled auto-merge (squash) April 4, 2024 18:10
@reshke reshke merged commit 75c910a into master Apr 4, 2024
23 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants