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

fix(cluster): ipv6 address in network_interfaces object #7460

Merged
merged 1 commit into from
May 21, 2024

Conversation

juliayakovlev
Copy link
Contributor

@juliayakovlev juliayakovlev commented May 21, 2024

The problem was found during IPv6 test.
IPv6 was converted to full format when parse nodetool status result. This change was represented by #7047

But IPv6 address was not converted when collected info about nodes network interfaces (AWS).

During checking nodes status, we compare between 'nodetool status' output and node address that kept in the network_interfaces object. It fails because the IPv6 address miss leading zeros. As result the test fails with errors 'Failed to find a node in cluster by IP'

Also fix the same problem for peer and gossip output.

Fixes: #7447

Testing

PR pre-checks (self review)

  • I added the relevant backport labels
  • I didn't leave commented-out/debugging code

Reminders

  • Add New configuration option and document them (in sdcm/sct_config.py)
  • Add unit tests to cover my changes (under unit-test/ folder)
  • Update the Readme/doc folder relevant to this change (if needed)

sdcm/cluster.py Show resolved Hide resolved
fruch
fruch previously approved these changes May 21, 2024
Copy link
Contributor

@fruch fruch left a comment

Choose a reason for hiding this comment

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

LGTM

@juliayakovlev juliayakovlev marked this pull request as ready for review May 21, 2024 11:56
The problem was found during IPv6 test.
IPv6 was converted to full format when parse nodetool status result.
This change was represented by  scylladb#7047

But IPv6 address was not converted when collected info about nodes network interfaces (AWS).

During checking nodes status, we compare between 'nodetool status' output and node address
that kept in the network_interfaces object. It fails because the IPv6 address miss leading zeros.
As result the test fails with errors 'Failed to find a node in cluster by IP'

Also fix the same problem for peer and gossip output.

Fixes: scylladb#7447
@juliayakovlev juliayakovlev force-pushed the explode_ipv6_address branch from 67a42eb to 61243f3 Compare May 21, 2024 12:00
@fruch fruch added the backport/none Backport is not required label May 21, 2024
Copy link
Contributor

@fruch fruch left a comment

Choose a reason for hiding this comment

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

LGTM

@fruch fruch merged commit 5636e7f into scylladb:master May 21, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/none Backport is not required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Scylla test with all addresses ipv6 public fails on nodetool status
2 participants