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

roachtest: restart-one-node to use explicit addrs #106098

Merged

Conversation

aliher1911
Copy link
Contributor

Test is restarting one node with modified start params. It was not passing advertise-addr option which lead to hostname picked being unaccessible from other cluster nodes. This commit changes test to provide explicit address upon restart.

Fixes #105972

Release note: None

Test is restarting one node with modified start params.
It was not passing advertise-addr option which lead to hostname
picked being unaccessible from other cluster nodes.
This commit changes test to provide explicit address upon restart.

Fixes cockroachdb#105972

Release note: None
@aliher1911 aliher1911 added the backport-23.1.x Flags PRs that need to be backported to 23.1 label Jul 4, 2023
@aliher1911 aliher1911 requested a review from a team July 4, 2023 09:00
@aliher1911 aliher1911 self-assigned this Jul 4, 2023
@aliher1911 aliher1911 requested a review from a team as a code owner July 4, 2023 09:00
@aliher1911 aliher1911 requested review from srosenberg and smg260 and removed request for a team July 4, 2023 09:00
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@aliher1911
Copy link
Contributor Author

TFTR
bors r+

@craig
Copy link
Contributor

craig bot commented Jul 5, 2023

Build succeeded:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-23.1.x Flags PRs that need to be backported to 23.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

roachtest: acceptance/gossip/restart-node-one failed
3 participants