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

rsync is missing #167

Open
nidomiro opened this issue Feb 3, 2023 · 2 comments
Open

rsync is missing #167

nidomiro opened this issue Feb 3, 2023 · 2 comments

Comments

@nidomiro
Copy link

nidomiro commented Feb 3, 2023

My awx instance informed me that it cannot find rsync (for the synchronize-task).
I checked the server and it is still installed.
Next I checked the image of the ExecutionEnvironment and it wasn't found there.

The last image of quay.io/ansible/awx-ee still including rsync is 21.11.0.
With this tag my playbook runs again.

@shanemcd
Copy link
Member

shanemcd commented Feb 3, 2023

We want to avoid adding dependencies directly to awx-ee whenever possible. Collections should declare their own dependencies via the metadata that ansible-builder looks for. Since synchronize is in ansible.posix, we should look to see what's going on there.

I think the problem is here:

https://github.com/ansible-collections/ansible.posix/blob/main/bindep.txt#L4

That should be

rsync [platform:rpm]

@relrod
Copy link
Member

relrod commented Feb 7, 2023

#171 will fix this for the time being, but I agree that going forward this should be fixed in ansible.posix and we should drop this explicit requirement in the future.

softwarefactory-project-zuul bot added a commit to ansible-collections/ansible.posix that referenced this issue Mar 24, 2023
bindep - install rsync on all EL variants

Adds support for installing rsync for centos and related EL variants (AlmaLinux, Rocky, Oracle, etc)
rsync is an extremely common package, available in the base repos of all EL distros.
This is necessary to properly support AWX-EE and other community-built EL Execution Environments.
See: ansible/awx-ee#167

Reviewed-by: Austin Lane <[email protected]>
Reviewed-by: Hideki Saito <[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 a pull request may close this issue.

3 participants