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

scylla-node: use rpc_address instead of listen_address on wait CQL port to be up #342

Merged
merged 1 commit into from
Feb 20, 2024

Conversation

aurelilys
Copy link
Contributor

Hello,

I recently wanted to deploy a scylladb cluster with this playbook, I encountered a small problem when I redefined the scylla_rpc_address varriable to specific IP, the start_one_node task waits for scylla to be up on the wrong address (listen_address) and never succeeds.

@aurelilys aurelilys changed the title Use rpc_address instead of listen_address on wait CQL port to be up scylla_node: Use rpc_address instead of listen_address on wait CQL port to be up Feb 19, 2024
@aurelilys aurelilys changed the title scylla_node: Use rpc_address instead of listen_address on wait CQL port to be up scylla-node: use rpc_address instead of listen_address on wait CQL port to be up Feb 19, 2024
Copy link
Collaborator

@vladzcloudius vladzcloudius left a comment

Choose a reason for hiding this comment

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

The current behavior of the Role is a correct one since we are waiting for the CQL port on the node itself (note the delegate_to statement).

I suggest you open GH issue and describe the issue that you had with all relevant details.

@vladzcloudius
Copy link
Collaborator

vladzcloudius commented Feb 20, 2024

The current behavior of the Role is a correct one since we are waiting for the CQL port on the node itself (note the delegate_to statement).

I suggest you open GH issue and describe the issue that you had with all relevant details.

I'm taking it back - my apologies.
Your patch is good!
Thanks for contributing!

@vladzcloudius vladzcloudius merged commit 193140b into scylladb:master Feb 20, 2024
2 checks passed
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