Skip to content

test: install ansible collection from tar.gz package#463

Merged
cgwalters merged 1 commit intobootc-dev:mainfrom
henrywang:ansible_offline
Apr 7, 2024
Merged

test: install ansible collection from tar.gz package#463
cgwalters merged 1 commit intobootc-dev:mainfrom
henrywang:ansible_offline

Conversation

@henrywang
Copy link
Collaborator

@henrywang henrywang commented Apr 7, 2024

To avoid galaxy server unstable issue, like 504 gateway timeout, error when getting available versions of collection community.general, download two required collection packages and install them locally. And those two collections are more general and do not have too much update often.

To avoid galaxy server unstable issue, like 504 gateway timeout,
error when getting available versions of collection community.general,
download two required collection packages and install then locally

Signed-off-by: Xiaofeng Wang <henrywangxf@me.com>
- unzip
- how: shell
script: ansible-galaxy collection install community.general ansible.posix
script: ansible-galaxy collection install https://ansible-collection.s3.amazonaws.com/ansible-posix-1.5.4.tar.gz https://ansible-collection.s3.amazonaws.com/community-general-8.5.0.tar.gz
Copy link
Collaborator

Choose a reason for hiding this comment

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

OK, but it seems better to retry this task in a loop; this approach seems to be bypassing their metadata servers; e.g. what if they decided to store things in GCP or quay.io instead?

Copy link
Collaborator Author

@henrywang henrywang Apr 7, 2024

Choose a reason for hiding this comment

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

Sometimes it failed for a long time, like this time(still flaky since Saturday). So retry does not work in this case.
I think saving to quay.io adds complex. Both GCP and S3 work for me.
BTW, I've tried to run ansible inside a container, but I dropped that for two reasons:

  1. it adds a lot of complexity and test code is not easy to understand.
  2. libvirt scenario blocked me.

@cgwalters cgwalters merged commit d22b50d into bootc-dev:main Apr 7, 2024
@henrywang henrywang deleted the ansible_offline branch April 7, 2024 12:41
cgwalters added a commit to cgwalters/bootc that referenced this pull request Nov 5, 2024
…son-1.0.96

build(deps): bump serde_json from 1.0.95 to 1.0.96
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.

2 participants