Skip to content

Conversation

@gionn
Copy link
Member

@gionn gionn commented Jan 20, 2025

OPSEXP-3019

to be merged after release

@gionn gionn self-assigned this Jan 20, 2025
@gionn gionn force-pushed the OPSEXP-3019-bump-ansible branch from 40468a7 to 720cdb9 Compare January 20, 2025 11:10
@gionn gionn requested review from alxgomz and pmacius January 20, 2025 16:23
@gionn gionn changed the title OPSEXP-3019 Bump python and ansible OPSEXP-3019 Bump python to 3.11 and ansible to 2.17 Jan 20, 2025
@gionn gionn marked this pull request as ready for review January 20, 2025 16:23
pmacius
pmacius previously approved these changes Jan 21, 2025
@gionn gionn marked this pull request as draft January 21, 2025 16:21
@gionn gionn mentioned this pull request Jan 28, 2025
@gionn gionn marked this pull request as ready for review February 4, 2025 14:42
@gionn gionn removed the do-not-merge label Feb 4, 2025
@gionn gionn added the ec2-test Triggers ec2 integrations tests label Feb 7, 2025
@gionn gionn force-pushed the OPSEXP-3019-bump-ansible branch from bd5482b to b43f5be Compare February 7, 2025 14:22
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so I got the idea to remove this since logs were getting spammed with:

WARNING  Another version of 'alfresco.acs' 1.0.0 was found installed in /home/runner/.local/share/virtualenvs/alfresco-ansible-deployment-cebUtTOi/lib/python3.11/dist-packages/ansible_collections, only the first one will be used, 1.0.0 (/home/runner/.ansible/collections/ansible_collections).
WARNING  Another version of 'alfresco.acs' 1.0.0 was found installed in /home/runner/.local/share/virtualenvs/alfresco-ansible-deployment-cebUtTOi/lib/python3.11/site-packages/ansible_collections, only the first one will be used, 1.0.0 (/home/runner/.ansible/collections/ansible_collections).
WARNING  Another version of 'alfresco.acs' 1.0.0 was found installed in /home/runner/.local/share/virtualenvs/alfresco-ansible-deployment-cebUtTOi/lib/python3/dist-packages/ansible_collections, only the first one will be used, 1.0.0 (/home/runner/.ansible/collections/ansible_collections).
WARNING  Another version of 'alfresco.acs' 1.0.0 was found installed in /home/runner/.local/share/virtualenvs/alfresco-ansible-deployment-cebUtTOi/local/lib/python3.11/dist-packages/ansible_collections, only the first one will be used, 1.0.0 (/home/runner/.ansible/collections/ansible_collections).

which was getting worse when molecule was triggering a retry like in https://github.com/Alfresco/alfresco-ansible-deployment/actions/runs/13240172999/job/36956143762#step:3:3708

once I dropped this, then all the single role galaxy metadata was starting to trigger validation issues due to missing namespace, so I decided to revamp all of them which was anyway required.

at this point I am not sure if the root galaxy.yml was just inhibiting all the role galaxy.yml and I think it's a feature (because in that case roles are part of the collection)

I am not sure what to do, but maybe given we are not currently publishing anything we can leave it as it is now.

@gionn gionn requested review from alxgomz and pmacius February 11, 2025 10:21
pmacius
pmacius previously approved these changes Feb 11, 2025
alxgomz
alxgomz previously approved these changes Feb 11, 2025
@gionn gionn dismissed stale reviews from alxgomz and pmacius via 4ffbfd5 February 11, 2025 15:36
@gionn gionn merged commit 19012b1 into master Feb 17, 2025
75 checks passed
@gionn gionn deleted the OPSEXP-3019-bump-ansible branch February 17, 2025 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ec2-test Triggers ec2 integrations tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants