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

test: use ipaserver role from collection if it exists #256

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

richm
Copy link
Collaborator

@richm richm commented Feb 3, 2025

No description provided.

@richm
Copy link
Collaborator Author

richm commented Feb 3, 2025

[citest]

@richm
Copy link
Collaborator Author

richm commented Feb 4, 2025

[citest]

@richm
Copy link
Collaborator Author

richm commented Feb 4, 2025

[citest]

@richm
Copy link
Collaborator Author

richm commented Feb 4, 2025

[citest]

@richm
Copy link
Collaborator Author

richm commented Feb 4, 2025

[citest]

@rjeffman
Copy link
Collaborator

rjeffman commented Feb 4, 2025

@richm we plan to migrate all ansible_freeipa releases to a collection in late CY24Q2 in anything but CentOS 8 Stream (which is not supported anymore).

Do you think this change is needed?

On another point of view, a "galaxy role" would be nice to ensure collections are installed (or I'm missing something that already exists and I don't know).

@richm
Copy link
Collaborator Author

richm commented Feb 4, 2025

@richm we plan to migrate all ansible_freeipa releases to a collection in late CY24Q2 in anything but CentOS 8 Stream (which is not supported anymore).

Do you mean CY25?

There are a few problems with the basic ipa test:

  • the legacy role was removed from the latest packages in el9 and el10. So we have to use the FQCN of the role in that case.
  • the upstream CI was changed to use BEAKERLIB - so we have to now use some other way to distinguish if we can use the ansible-freeipa rpm package, or install the role directly from github
  • the ipaserver installation is failing:
2025-02-04T01:55:42Z DEBUG Name ipaserver.test.local resolved to {UnsafeIPAddress('10.31.9.54'), UnsafeIPAddress('fe80::1044:c6ff:fe29:5d99')}
2025-02-04T01:55:42Z WARNING Invalid IP address fe80::1044:c6ff:fe29:5d99 for ipaserver.test.local: cannot use link-local IP address fe80::1044:c6ff:fe29:5d99

Do you think this change is needed?

Something needs to be changed/fixed - not sure exactly what.

On another point of view, a "galaxy role" would be nice to ensure collections are installed (or I'm missing something that already exists and I don't know).

Not sure what you mean?

@richm
Copy link
Collaborator Author

richm commented Feb 4, 2025

[citest]

1 similar comment
@richm
Copy link
Collaborator Author

richm commented Feb 4, 2025

[citest]

@richm
Copy link
Collaborator Author

richm commented Feb 4, 2025

[citest]

@rjeffman rjeffman force-pushed the test-use-ipa-collection branch from ace3ba5 to c1a6948 Compare February 7, 2025 14:03
@rjeffman
Copy link
Collaborator

rjeffman commented Feb 7, 2025

[citest]

@rjeffman rjeffman force-pushed the test-use-ipa-collection branch from c1a6948 to 508771b Compare February 7, 2025 14:19
@rjeffman
Copy link
Collaborator

rjeffman commented Feb 7, 2025

[citest]

Signed-off-by: Rafael Guterres Jeffman <[email protected]>
@rjeffman rjeffman force-pushed the test-use-ipa-collection branch from 508771b to 8b65c74 Compare February 7, 2025 15:04
@rjeffman
Copy link
Collaborator

rjeffman commented Feb 7, 2025

[citest]

1 similar comment
@rjeffman
Copy link
Collaborator

rjeffman commented Feb 7, 2025

[citest]

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