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

Added support for Ubuntu22. #608

Merged
merged 1 commit into from
Oct 27, 2023
Merged

Added support for Ubuntu22. #608

merged 1 commit into from
Oct 27, 2023

Conversation

bonesmoses
Copy link
Contributor

Given that edb-ansible supports Ubuntu20, Ubuntu22 should also be compatible. I didn't physically test every single role, but have worked extensively with install_dbserver, setup_patroni, and setup_etcd, which are all necessary to set up a basic Patroni cluster. None of these were affected simply by adding "Ubuntu22" as a patch, which was necessary for various unrelated tests.

Given that edb-ansible supports Ubuntu20, Ubuntu22 should also be
compatible. I didn't physically test every single role, but have
worked extensively with install_dbserver, setup_patroni, and
setup_etcd, which are all necessary to set up a basic Patroni
cluster. None of these were affected simply by adding "Ubuntu22"
as a patch, which was necessary for various unrelated tests.
@bonesmoses
Copy link
Contributor Author

I took a look at the failed checks, and they were related to not being able to pull edb-ansible from ansible-galaxy, which is... odd.

Copy link
Collaborator

@hannahms hannahms left a comment

Choose a reason for hiding this comment

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

thank you for this contribution!

@hannahms hannahms merged commit 70f1323 into main Oct 27, 2023
30 of 32 checks passed
@hannahms
Copy link
Collaborator

galaxy has a new protocol, this issue has been sporadically popping up, so nothing too alarming

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