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

Rewrite EE test workflows to use ansible-builder 3.0.0; fix EE dependencies #606

Merged
merged 19 commits into from
May 21, 2023

Conversation

felixfontein
Copy link
Contributor

SUMMARY

Also use other base images. ansible-builder 3.0.0 allows arbitrary "RHEL-like" base images, as opposed to ansible-builder < 3.0.0 (which required very specially prepared base images).

ISSUE TYPE
  • Test Pull Request
COMPONENT NAME

EE tests

@felixfontein
Copy link
Contributor Author

CC @Andersson007

@felixfontein
Copy link
Contributor Author

It seems we're hitting ansible/ansible-navigator#1448 when using podman. Let's see whether switching to docker helps... (it did for me locally.)

@nitzmahone
Copy link
Member

Hrm, with a reasonably recent runner version and the default builder 3.0 config, it should have the awx_display bits copied in at runtime and configured dynamically by runner, and the entrypoint script should ensure that the container user always has a writeable homedir (so long as it's running as GID0 inside). If navigator's trying too hard to force both the host user and gid, it's probably not going to end well, since we have no way for the entrypoint to provide a writeable homedir in that case.

use Python 3.9 on RHEL8 / CentOS Stream 8. Manually install cryptography and PyOpenSSL for Python 3.9 as well.
@felixfontein felixfontein changed the title [WIP] Rewrite EE test workflows to use ansible-builder 3.0.0 Rewrite EE test workflows to use ansible-builder 3.0.0 May 20, 2023
@felixfontein felixfontein changed the title Rewrite EE test workflows to use ansible-builder 3.0.0 Rewrite EE test workflows to use ansible-builder 3.0.0; fix EE dependencies May 20, 2023
@felixfontein
Copy link
Contributor Author

ready_for_review

@felixfontein
Copy link
Contributor Author

ansible/ansible-navigator#1539 fixed the podman issue, I've successfully switched back from docker to podman now that that fix is released (ansible-navigator 3.3.1).

@github-actions
Copy link

github-actions bot commented May 21, 2023

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and the docs are now incorporated into main:
https://ansible-collections.github.io/community.crypto/branch/main

@felixfontein felixfontein merged commit c703dd6 into ansible-collections:main May 21, 2023
@felixfontein felixfontein deleted the ee branch May 21, 2023 10:43
@Andersson007
Copy link
Contributor

@felixfontein thanks for pinging! Please do it in related PRs (or maybe even better to put them also to the EE feedback issue to have them in one place)
once I finish with the basic overviews / how-tos, i'll take a look;)

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.

3 participants