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

Download artifacts #86

Closed
wants to merge 5 commits into from
Closed

Conversation

jrosser
Copy link
Contributor

@jrosser jrosser commented Dec 4, 2023

Add utility playbook to download artefacts for all roles as a helper for offline installations

@jrosser jrosser force-pushed the download-artifacts branch 4 times, most recently from 2de06d5 to 2fa72a7 Compare December 4, 2023 13:55
@okozachenko1203 okozachenko1203 self-requested a review December 4, 2023 14:15
@jrosser jrosser force-pushed the download-artifacts branch from 2fa72a7 to a3f18cf Compare December 4, 2023 14:17
@okozachenko1203
Copy link
Member

@jrosser i think this PR is in progress still. Could you convert it to Draft in this case?
Also when you are done, it will be good to add download-artifacts molecule scenario in the github workflows.

@jrosser jrosser marked this pull request as draft December 4, 2023 14:19
@jrosser jrosser force-pushed the download-artifacts branch 3 times, most recently from d92ff22 to 7c5a35c Compare December 5, 2023 14:21
@jrosser jrosser marked this pull request as ready for review January 12, 2024 18:44
Jonathan Rosser added 5 commits June 13, 2024 16:13
All other roles use *_binary_checksum or *_archive_checksum.
The version vars for these tools are used in the defaults file
of their respective role, but is not defined, normally overidden
to the value of 'kubernetes_version' when the role is called.

This patch sets the default value of the installed version to
be the newest one understood by the defaults file.
Usage: ansible-playbook vexxhost.containers.download_artifacts -e target=localhost

This playbook uses the existing download_artifact role and downloads
the artifacts required for all roles.
@jrosser jrosser force-pushed the download-artifacts branch from 7c5a35c to f6e16db Compare June 13, 2024 15:14
@jrosser
Copy link
Contributor Author

jrosser commented Oct 15, 2024

Replaced with a better implementation in #139

@jrosser jrosser closed this Oct 15, 2024
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