Skip to content

synchronize: use _find_needle instead #433

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

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

Conversation

mnaser
Copy link

@mnaser mnaser commented Mar 21, 2023

SUMMARY

It's not possible to use files from a sub role, this patch fixes this.

Fixes #381

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

synchornize

ADDITIONAL INFORMATION

See #381

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/d98fe75b98a74d8e88bac6026a45b37e

ansible-changelog-fragment FAILURE in 15s
✔️ ansible-test-sanity-docker-devel SUCCESS in 6m 44s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 6m 52s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 9m 33s
✔️ ansible-test-sanity-docker-stable-2.10 SUCCESS in 9m 09s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 9m 21s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 6m 54s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 7m 11s
✔️ ansible-test-units-posix-python39 SUCCESS in 4m 42s
✔️ ansible-test-units-posix-python310 SUCCESS in 5m 47s
✔️ ansible-galaxy-importer SUCCESS in 3m 54s
✔️ build-ansible-collection SUCCESS in 6m 32s

@gtirloni
Copy link

@tima coud you review this PR?

@jillr
Copy link
Contributor

jillr commented May 9, 2024

@mnaser Apologies in the delay getting this reviewed. Would you be willing to add a changelog fragment to this PR?

Copy link
Contributor

@mnaser
Copy link
Author

mnaser commented May 6, 2025

@Akasurde thanks for adding the release note, I'm not sure why jobs are failing, could we help land this?

Copy link
Contributor

@mnaser
Copy link
Author

mnaser commented May 6, 2025

@Akasurde hmm, i see the failures seem to be related to the second time around copy, do you know why/how i can dig into this?

@Akasurde
Copy link
Member

Akasurde commented May 6, 2025

@mnaser I think this -

- name: Test that the file is not copied a second time
test is failing. If you see the logs it is trying to find

looking for "'/root/ansible_collections/ansible/posix/tests/output/.tmp/output_dir/foo.txt'" at "/root/ansible_collections/ansible/posix/tests/output/.tmp/integration/synchronize-pq5aaj8p-ÅÑŚÌβŁÈ/tests/integration/targets/synchronize/files/'/root/ansible_collections/ansible/posix/tests/output/.tmp/output_dir/foo.txt'"

See that the destination is resolved wrong.

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.

Synchronize cannot find directories from sub roles like copy does
4 participants