Skip to content

Add some sleeps on node start up and join to avoid failure#2

Merged
lukebakken merged 2 commits intomasterfrom
feature/bch/slow-down-cluster-join
Jan 17, 2016
Merged

Add some sleeps on node start up and join to avoid failure#2
lukebakken merged 2 commits intomasterfrom
feature/bch/slow-down-cluster-join

Conversation

@hazen
Copy link

@hazen hazen commented Jan 15, 2016

Without these sleeps my VM would constantly fail

@lukebakken
Copy link
Contributor

  • What OS?
  • What were the failures?

@hazen
Copy link
Author

hazen commented Jan 17, 2016

The OS is either CentOS 6.7 or CentOS 7.1

2016-01-17 00:19:22 [info]: Removing old Riak data
2016-01-17 00:19:22 [info]: Configuring riak.conf
2016-01-17 00:19:22 [info]: Starting Riak
Join failed. Try again in a few moments.
Exiting!

@lukebakken
Copy link
Contributor

Thanks, I'll give it a try on my CentOS VM.

@lukebakken
Copy link
Contributor

works-on-my-machine-badge

I even turned down my CentOS 7 VM to one processor and 1GiB ram to slow it down.

I'll make this an option or env var instead.

@lukebakken
Copy link
Contributor

Added feature as the -d SECS option.

lukebakken added a commit that referenced this pull request Jan 17, 2016
Add some sleeps on node start up and join to avoid failure
@lukebakken lukebakken merged commit 8ba18ff into master Jan 17, 2016
@lukebakken lukebakken deleted the feature/bch/slow-down-cluster-join branch January 17, 2016 16:48
@lukebakken
Copy link
Contributor

@javajolt - interesting - I just ran into this when starting up a Riak TS cluster. It has never happened with plain Riak. Do you think it could mean an issue?

@hazen
Copy link
Author

hazen commented Jan 29, 2016

Not sure. Don't know why it would be any slower than regular Riak to start up. I could bring it up in standup today.

@lukebakken
Copy link
Contributor

Were you using Riak TS as well?

@hazen
Copy link
Author

hazen commented Jan 29, 2016

No. For the NodeJS PR I was using 2.1.3 IIRC

@lukebakken
Copy link
Contributor

OK who knows what's up then.

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