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

Add Ansible-Galaxy release workflow #6

Merged
merged 1 commit into from
Mar 28, 2023

Conversation

sanjaysrikakulam
Copy link
Member

No description provided.

@sanjaysrikakulam
Copy link
Member Author

sanjaysrikakulam commented Feb 13, 2023

I think the release through the CI will create a new gie-proxy role instead of pushing it to the existing 'gie_proxy' role. So, please review this carefully as we do not want to create additional roles. If in case this is going to create a new role, please reject the PR and whoever has access to the usegalaxy_eu's Ansible-Galaxy please push the v0.0.3 to the 'gie_proxy' role

@bgruening
Copy link
Member

bgruening commented Feb 13, 2023

Should we not create a new one, with the proper name - called gx-it-proxy?

@bgruening
Copy link
Member

bgruening commented Feb 13, 2023

ping @natefoo @hexylena

@hexylena
Copy link
Member

I mean from our side we don't want this at all, it'll be part of the main role and gravity's job to install and manage, making some of these older individual roles (also tusd) less useful.

@hexylena
Copy link
Member

re the name, I don't think that's what will happen. the name is taken from the meta.

@sanjaysrikakulam
Copy link
Member Author

re the name, I don't think that's what will happen. the name is taken from the meta.

Oh, yeah! Thank you!

@bgruening
Copy link
Member

I mean from our side we don't want this at all, it'll be part of the main role and gravity's job to install and manage, making some of these older individual roles (also tusd) less useful.

This assumes you have tusd on the headnode, or need gravity on an external node to manage tusd?

@hexylena
Copy link
Member

Yes. (Though I suspect, @natefoo can confirm) it will look a lot like

headnode: run ansible-galaxy, install galaxy
uploads: run ansible-galaxy, no install galaxy, install tusd

@sanjaysrikakulam
Copy link
Member Author

ping @bgruening, @mira-miracoli

@natefoo
Copy link
Collaborator

natefoo commented Mar 8, 2023

Eventually this will be folded into the collection but it's fine continuing as its own thing for now. I don't want to add anything else to the monolithic ansible-galaxy role.

@sanjaysrikakulam sanjaysrikakulam merged commit 29b8915 into usegalaxy-eu:master Mar 28, 2023
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.

None yet

5 participants