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

Improve release process #42

Merged
merged 2 commits into from
Apr 21, 2021

Conversation

greg-hellings
Copy link
Contributor

Release is now done by creating a release on GitHub
Change testing environments to podman from Docker
Add .cache to gitignore
No need to putz with paths, as molecule and ansible-lint handle this now

@greg-hellings greg-hellings marked this pull request as ready for review April 21, 2021 14:43
Release is now done by creating a release on GitHub
Change testing environments to podman from Docker
Add .cache to gitignore
No need to putz with paths, as molecule and ansible-lint handle this now
passwordless_ssh needs to run systemd
Chrony test seems to need a VM
Copy link
Contributor

@tehsmyers tehsmyers left a comment

Choose a reason for hiding this comment

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

The ceph-ansible ignore over in meta_test is weird, but not worth worrying about.

@greg-hellings
Copy link
Contributor Author

The ceph-ansible ignore over in meta_test is weird, but not worth worrying about.

Ah yes. I need to remove that. It was a way of trying to access ceph-ansible through a collection. But I found out its playbooks had all the options I needed. Once we get a release of this done, it's time to release the Ceph collection. I'll remove it as part of the Ceph release cycle.

@greg-hellings greg-hellings merged commit bed4f0a into oasis-roles:master Apr 21, 2021
@greg-hellings greg-hellings deleted the better_releases branch April 21, 2021 17:18
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