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

CI problem #2257

Open
mariolenz opened this issue Nov 22, 2024 · 2 comments
Open

CI problem #2257

mariolenz opened this issue Nov 22, 2024 · 2 comments

Comments

@mariolenz
Copy link
Collaborator

I don't really understand why the CI fails. It looks like Refresh test_host has been skipped here because hostvars | length >= 1 but now it isn't now. I think that hostvars is empty in both cases, but I don't understand why we have a skipped task at first and then we don't.

I don't think this is because of changing the integration the integration tests to ansible/ansible-zuul-jobs#1887 because my tests worked with this.

BTW I don't really understand those inventory plugins, and I understand their integration tests even less :-(

#2250:

2024-11-15 17:07:30.505895 | controller | TASK [debug] *******************************************************************
2024-11-15 17:07:30.505904 | controller | task path: /home/zuul/.ansible/collections/ansible_collections/community/vmware/tests/integration/targets/inventory_vmware_host_inventory/playbook/build_inventory.yml:17
2024-11-15 17:07:30.505918 | controller | ok: [localhost] => {
2024-11-15 17:07:30.539579 | controller |     "msg": {}
2024-11-15 17:07:30.539606 | controller | }
2024-11-15 17:07:30.539611 | controller |
2024-11-15 17:07:30.539616 | controller | TASK [Refresh test_host] *******************************************************
2024-11-15 17:07:30.539621 | controller | task path: /home/zuul/.ansible/collections/ansible_collections/community/vmware/tests/integration/targets/inventory_vmware_host_inventory/playbook/build_inventory.yml:20
2024-11-15 17:07:30.539629 | controller | skipping: [localhost] => {
2024-11-15 17:07:30.658405 | controller |     "changed": false,
2024-11-15 17:07:30.658442 | controller |     "false_condition": "hostvars | length >= 1",
2024-11-15 17:07:30.658448 | controller |     "skip_reason": "Conditional result was False"
2024-11-15 17:07:30.658452 | controller | }

log (archive.org)

#2255:

2024-11-22 16:06:46.067927 | controller | TASK [debug] *******************************************************************
2024-11-22 16:06:46.067933 | controller | task path: /home/zuul/.ansible/collections/ansible_collections/community/vmware/tests/integration/targets/inventory_vmware_host_inventory/playbook/build_inventory.yml:17
2024-11-22 16:06:46.067947 | controller | ok: [localhost] => {
2024-11-22 16:06:46.153740 | controller |     "msg": {}
2024-11-22 16:06:46.153776 | controller | }
2024-11-22 16:06:46.153784 | controller |
2024-11-22 16:06:46.153791 | controller | TASK [Refresh test_host] *******************************************************
2024-11-22 16:06:46.153798 | controller | task path: /home/zuul/.ansible/collections/ansible_collections/community/vmware/tests/integration/targets/inventory_vmware_host_inventory/playbook/build_inventory.yml:20
2024-11-22 16:06:46.153811 | controller | ok: [localhost] => {
2024-11-22 16:06:46.153818 | controller |     "ansible_facts": {
2024-11-22 16:06:46.153824 | controller |         "test_host": {
2024-11-22 16:06:46.153830 | controller |             "key": "localhost",
2024-11-22 16:06:46.153836 | controller |             "value": {
2024-11-22 16:06:46.153843 | controller |                 "ansible_all_ipv4_addresses": [
2024-11-22 16:06:46.153849 | controller |                     "192.168.0.130",
2024-11-22 16:06:46.153856 | controller |                     "192.168.1.28",
2024-11-22 16:06:46.153862 | controller |                     "38.129.16.175"
2024-11-22 16:06:46.153868 | controller |                 ],

log (archive.org)

@mariolenz
Copy link
Collaborator Author

see also #2256

softwarefactory-project-zuul bot pushed a commit that referenced this issue Nov 23, 2024
SUMMARY
Temporarily disable integration test
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
vmware_host_inventory
ADDITIONAL INFORMATION
#2257
https://forum.ansible.com/t/11073
@mariolenz
Copy link
Collaborator Author

For the record: https://forum.ansible.com/t/11073 suggests ansible/ansible#84339 as a possible reason for this.

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

No branches or pull requests

1 participant