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

Fix fatal errors in GxIT and ansible tutorials #4370

Merged
merged 4 commits into from
Oct 3, 2023
Merged

Conversation

tuncK
Copy link
Contributor

@tuncK tuncK commented Oct 3, 2023

I noticed that the IT tutorial (missing config file) and the ansible galaxy tutorial (conda cannot resolve) do not work out of the box anymore. This PR proposes necessary corrections and version updates so that the tutorials work with the current version of Galaxy.

@tuncK tuncK requested review from abretaud and a team as code owners October 3, 2023 08:02
Copy link
Member

@hexylena hexylena left a comment

Choose a reason for hiding this comment

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

Thanks for the conda vesion bump @tuncK! With this, ansible-galaxy works for you, correct? Nothing else needed, just the conda bump?

@hexylena hexylena merged commit 41ca6f2 into galaxyproject:main Oct 3, 2023
@mvdbeek
Copy link
Member

mvdbeek commented Oct 3, 2023

@hexylena
Copy link
Member

hexylena commented Oct 3, 2023

Ah it closes half of it, the other half, yeah ok we should've merged that earlier. THanks @mvdbeek I'd forgotten about that.

@tuncK
Copy link
Contributor Author

tuncK commented Oct 4, 2023

Thanks for the conda vesion bump @tuncK! With this, ansible-galaxy works for you, correct? Nothing else needed, just the conda bump?

Thanks a lot for merging, @hexylena. This indeed should solve the installation error, but not the 502. Though, let me re-visit this latter with a fresh installation, and file a separate issue or PR if still troubling.

@hexylena
Copy link
Member

hexylena commented Oct 4, 2023

Sounds good! Please feel free to also write in the admins matrix channel if you have additional questions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants