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

cli: remove auto-init with cockroach start without --join #44112

Closed
wants to merge 1 commit into from

Conversation

knz
Copy link
Contributor

@knz knz commented Jan 17, 2020

Fixes #44116.
Informs #24118.

Release note (backward-incompatible change): A CockroachDB node
started with cockroach start without the --join flag no
longer automatically initializes the cluster. The cockrach init
command is now mandatory. The auto-initialization behavior had
been deprecated in version 19.2.

@knz knz requested a review from tbg January 17, 2020 15:06
@knz knz requested a review from a team as a code owner January 17, 2020 15:06
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@knz
Copy link
Contributor Author

knz commented Jan 17, 2020

I'm expecting a bunch of tests to fail, and I'll fix them.

@knz knz force-pushed the 20200119-disable-auto-init branch from fdf4f86 to a231155 Compare January 17, 2020 15:19
@knz knz changed the title cli: deprecate auto-init with cockroach start without --join cli: remove auto-init with cockroach start without --join Jan 17, 2020
@knz knz force-pushed the 20200119-disable-auto-init branch 2 times, most recently from cadc032 to 6695bb7 Compare January 17, 2020 17:18
@knz
Copy link
Contributor Author

knz commented Jan 17, 2020

Some items identified by the CI failures:

  • github.com/cockroachdb/cockroach-go/testserver must be changed to always use init
  • bump the dep in example-orms to pick up the first change
  • change roachprod to use init

Release note (backward-incompatible change): A CockroachDB node
started with `cockroach start` without the `--join` flag no longer
automatically initializes the cluster. The `cockrach init` command is
now mandatory. The auto-initialization behavior had been deprecated in
version 19.2. Users can start `cockroach start-single-node` to start
and auto-initialize a single-node cluster.
@knz knz force-pushed the 20200119-disable-auto-init branch from 6695bb7 to 085ecbf Compare January 17, 2020 17:22
@danhhz
Copy link
Contributor

danhhz commented Jan 30, 2020

Ooo, this might make things a little less messy in #44249. Any guesses on timeline for this merging?

@knz
Copy link
Contributor Author

knz commented Jan 30, 2020 via email

@tbg tbg removed their request for review June 25, 2020 09:50
irfansharif added a commit to irfansharif/cockroach that referenced this pull request Jul 10, 2020
Fixes cockroachdb#44116. Informs cockroachdb#24118.
Reviving cockroachdb#44112.

Release note (backward-incompatible change): A CockroachDB node
started with cockroach start without the --join flag no
longer automatically initializes the cluster. The cockrach init
command is now mandatory. The auto-initialization behavior had
been deprecated in version 19.2.
irfansharif added a commit to irfansharif/cockroach that referenced this pull request Jul 15, 2020
Fixes cockroachdb#44116. Informs cockroachdb#24118. Reviving cockroachdb#44112.

We remove the deprecated behavior of crdb where we previously
auto-initialized the cluster when `cockroach start` was invoked without
a corresponding `--join` flag. We update a few tests along the way that
made use of this behavior by changing them to either explicitly init, or
now lean on `roachprod` changes (from cockroachdb#51329) that transparently manage
initialization.

Release note (backward-incompatible change): A CockroachDB node
started with cockroach start without the --join flag no
longer automatically initializes the cluster. The `cockroach init`
command is now mandatory. The auto-initialization behavior had
been deprecated in version 19.2.

Release note (backward-incompatible change): `cockroach start` without
`--join` is no longer supported. It was deprecated in a previous
release.
irfansharif added a commit to irfansharif/cockroach that referenced this pull request Jul 17, 2020
Fixes cockroachdb#44116. Informs cockroachdb#24118. Reviving cockroachdb#44112.

We remove the deprecated behavior of crdb where we previously
auto-initialized the cluster when `cockroach start` was invoked without
a corresponding `--join` flag. We update a few tests along the way that
made use of this behavior by changing them to either explicitly init, or
now lean on `roachprod` changes (from cockroachdb#51329) that transparently manage
initialization.

Release note (backward-incompatible change): A CockroachDB node
started with cockroach start without the --join flag no
longer automatically initializes the cluster. The `cockroach init`
command is now mandatory. The auto-initialization behavior had
been deprecated in version 19.2.

Release note (backward-incompatible change): `cockroach start` without
`--join` is no longer supported. It was deprecated in a previous
release.
@knz
Copy link
Contributor Author

knz commented Jul 17, 2020

Superseded by #51245

@knz knz closed this Jul 17, 2020
craig bot pushed a commit that referenced this pull request Jul 17, 2020
51245: cli: remove auto-init with `cockroach start` without `--join` r=irfansharif a=irfansharif

Fixes #44116. Informs #24118. Reviving #44112.

Release note (backward-incompatible change): A CockroachDB node
started with cockroach start without the --join flag no
longer automatically initializes the cluster. The cockrach init
command is now mandatory. The auto-initialization behavior had
been deprecated in version 19.2.

Co-authored-by: irfan sharif <[email protected]>
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.

cli: remove auto-init for start without --join.
3 participants