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.

Sign up for GitHub

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

Jump to bottom

[yb-ctl] Fix yb-ctl add_node max server counting #1730

Closed
bmatican opened this issue Jun 20, 2019 · 0 comments · Fixed by yugabyte/yugabyte-installation#11
Closed

[yb-ctl] Fix yb-ctl add_node max server counting #1730

bmatican opened this issue Jun 20, 2019 · 0 comments · Fixed by yugabyte/yugabyte-installation#11
Assignees
@JDNdeveloper

Comments

@bmatican
Copy link
Contributor

Something recently broke add_node, wherein we no longer seem to be waiting for the process to be up and we do a one time count of the max number of TS to wait for, leading us to messages such as:

Waiting for all tablet servers to register: 2/1 (we have 2 processes, but we said we're looking for 1, so we'll never finish...)

cc @mbautin @ndeodhar
yugabyte-ci referenced this issue Jun 21, 2019
Update yugabyte-installation submodule
f9be851 
Summary: New commit includes fix for https://github.com/YugaByte/yugabyte-installation/issues/9

Test Plan: See yugabyte/yugabyte-installation#11

Reviewers: mikhail, bogdan

Reviewed By: bogdan

Differential Revision: https://phabricator.dev.yugabyte.com/D6792
yugabyte-ci referenced this issue Jun 24, 2019
Update yugabyte-installation submodule (#2)
81f1b31 
Summary:
New commit includes fix for https://github.com/YugaByte/yugabyte-installation/issues/9

Dupe of https://phabricator.dev.yugabyte.com/D6792 due to commit getting pulled back in https://phabricator.dev.yugabyte.com/D6795

Test Plan: Jenkins: skip

Reviewers: mikhail, bogdan

Reviewed By: bogdan

Differential Revision: https://phabricator.dev.yugabyte.com/D6803
@mbautin mbautin transferred this issue from yugabyte/yugabyte-installation Jul 8, 2019
@mbautin mbautin changed the title Fix yb-ctl add_node max server counting [yb-ctl] Fix yb-ctl add_node max server counting Jul 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JDNdeveloper JDNdeveloper

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Move filesystem number of servers check to inside call check function
2 participants
@bmatican @JDNdeveloper