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

Use shared workflows in modules #771

Merged
merged 1 commit into from
Jan 27, 2022

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Dec 22, 2021

Shared workflows mean a lot less changes to modulesync. This leads to easier maintenance.

For now this uses a repository in my own name (https://github.com/ekohl/gha-puppet) but I intend to move it over once I'm ok with it.

Comment on lines -70 to -83
<%- @configs['beaker_fact_matrix'].each do |option, values| -%>
<%= option %>:
<%- values.each do |value| -%>
- "<%= value %>"
<%- end -%>
<%- end -%>
<%- if @configs['excludes'].any? -%>
exclude:
<%- @configs['excludes'].each do |exclude| -%>
<%- exclude.each do |key, value| -%>
<%= key == exclude.first.first ? '-' : ' ' %> <%= key %>: "<%= value %>"
<%- end -%>
<%- end -%>
<%- end -%>
Copy link
Member Author

Choose a reason for hiding this comment

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

TODO: there is no replacement for this yet. I'm thinking about pushing it into puppet_metadata. I started with voxpupuli/puppet_metadata#35.

@ekohl ekohl marked this pull request as ready for review January 11, 2022 21:36
@ekohl ekohl marked this pull request as draft January 11, 2022 21:37
@ekohl
Copy link
Member Author

ekohl commented Jan 11, 2022

Status update:

I'm feeling pretty confident in the CI workflow, both with just unit tests (verified in voxpupuli/puppet-example#15) and with beaker (verified in voxpupuli/puppet-trusted_ca#43).

I haven't started on the release tests, but I'd like to merge voxpupuli/puppet-example#15 and iterate on the release.

@ekohl ekohl force-pushed the shared-workflows branch 2 times, most recently from 6e1c40a to 16fa7e2 Compare January 11, 2022 22:31
@ekohl
Copy link
Member Author

ekohl commented Jan 11, 2022

I haven't started on the release tests, but I'd like to merge voxpupuli/puppet-example#15 and iterate on the release.

After a few small changes, that also works and https://forge.puppet.com/modules/puppet/example/0.2.1 is proof of that. See https://github.com/voxpupuli/puppet-example/actions/runs/1684914941 for the details.

I'd consider gha-puppet almost ready, but IMHO we should decide on voxpupuli/gha-puppet#3 before tagging a 1.0.0 version. voxpupuli/gha-puppet#2 would also be nice, but less important.

@ekohl ekohl marked this pull request as ready for review January 27, 2022 14:32
@ekohl
Copy link
Member Author

ekohl commented Jan 27, 2022

I've tagged version 1.0.0 of gha-puppet and believe this is ready. voxpupuli/puppet-example#20 reflects the last change.

Shared workflows mean a lot less changes to modulesync. This leads to
easier maintenance.
@bastelfreak bastelfreak merged commit 0021bfd into voxpupuli:master Jan 27, 2022
@ekohl ekohl deleted the shared-workflows branch January 27, 2022 16:24
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.

2 participants