Skip to content

Conversation

@madhuriupadhye
Copy link
Contributor

No description provided.

@madhuriupadhye madhuriupadhye requested a review from thalman May 22, 2025 13:03
@thalman thalman self-assigned this May 22, 2025
user = provider.user("testuser").add()
client.sssd.domain["lookup_family_order"] = "ipv6_only"
ipv6_add = provider.host.conn.run("ip -6 addr show scope global | grep -oE 'inet6 [0-9a-f:]+' | awk '{print $2}'")
print(ipv6_add.stdout)
Copy link
Contributor

Choose a reason for hiding this comment

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

Forgotten print

client.sssd.start()

result = client.tools.id(user.name)
assert result is not None, f"{user.name} was not found, SSSD did not switch to IPv6 family!"
Copy link
Contributor

Choose a reason for hiding this comment

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

You set ipv6_only - this means that the failover code is not switching . This code does not test failover but IPv6 connectivity.

It is unclear to me what you want to achieve/test. If it is variant of the other test I wrote, then you have to use ipv4_first and block ipv4 connectivity. That would trigger failover switch from 4 to 6.

Copy link
Contributor Author

@madhuriupadhye madhuriupadhye May 27, 2025

Choose a reason for hiding this comment

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

I am trying to test different values for lookup_family_order.
Yes, may be not good candidate of failover test, but intend to add to test ipv6_only options.
Will update this test case, but if I wanted to keep this test also, should I move it to other test suite?

Copy link
Contributor

Choose a reason for hiding this comment

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

In that case I do not mind test location. It looks like good candidate for parametrized test.

@madhuriupadhye madhuriupadhye force-pushed the ipv6_only branch 2 times, most recently from fcc5f43 to 6b458ae Compare May 28, 2025 08:30
@danlavu danlavu self-assigned this May 29, 2025
@danlavu danlavu self-requested a review May 29, 2025 12:38
:title: Only attempt to resolve hostnames to IPv6 address
:setup:
1. Create user
2. Set family_order to "ipv6_only"
Copy link
Contributor

Choose a reason for hiding this comment

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

Now we have ipv4_first set, not ipv6_only

client.sssd.start()

result = client.tools.id(user.name)
assert result is not None, f"{user.name} was not found, SSSD did not switch to IPv6 family!"
Copy link
Contributor

Choose a reason for hiding this comment

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

In that case I do not mind test location. It looks like good candidate for parametrized test.

:setup:
1. Create user
2. Set family_order to "ipv6_only"
3. Set IPv6 address in /etc/hosts
Copy link
Contributor

Choose a reason for hiding this comment

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

Now you are setting IPv4 address. It would be good to mention that this address is ureachable (Set unreachable IPv4 address...)

@thalman
Copy link
Contributor

thalman commented Jul 17, 2025

Blocked by SSSD/sssd-test-framework#188

Signed-off-by: Madhuri Upadhye <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants