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: add notice about --global being experimental on demo startup #63515

Merged
merged 3 commits into from
Apr 13, 2021

Conversation

otan
Copy link
Contributor

@otan otan commented Apr 13, 2021

Refs: #63469

See individual commits for details. Intended for backport for v21.1.1.

@otan otan requested review from knz and ajstorm April 13, 2021 01:01
@otan otan requested a review from a team as a code owner April 13, 2021 01:01
@cockroach-teamcity
Copy link
Member

This change is Reviewable

otan added 3 commits April 13, 2021 13:45
In addition to the release note, reworded the add node (which already
errored beforehand), and adding a clause in RestartNode.

Note there is no code path that allows a restart, since shutdown is
required before a node restart.

Release note (cli change): No longer support `\demo add/shutdown` on
cockroach demo.
Release note (cli change): Added a note when starting up a --global
demo cluster that it is experimental.
Copy link
Contributor

@knz knz left a comment

Choose a reason for hiding this comment

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

Reviewed 3 of 3 files at r1, 2 of 2 files at r2, 1 of 1 files at r3.
Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @ajstorm)

@otan
Copy link
Contributor Author

otan commented Apr 13, 2021

bors r=knz

TFTR!

@craig
Copy link
Contributor

craig bot commented Apr 13, 2021

Build succeeded:

@craig craig bot merged commit c4b7878 into cockroachdb:master Apr 13, 2021
@ajstorm
Copy link
Collaborator

ajstorm commented Apr 13, 2021 via email

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.

4 participants