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

PDK update, move CI to GH Actions #563

Merged
merged 2 commits into from
May 12, 2021

Conversation

genebean
Copy link
Contributor

@genebean genebean commented May 8, 2021

This includes many changes that are a byproduct of updating to the latest version of the PDK. All unit tests pass at this stage but several RuboCop tests still fail.

Tht GH Action was copied from a Vox Pupuli module. Their module testing is pretty slick and utilizes Beaker just as this one has traditionally done so it seems like a good place to start. A couple of lines were commented out as the Gemfile from the PDK doesn't line up exactly with the one used by most Vox Pupuli modules.

RuboCop is disabled for the moment as it is known that those tests do not yet pass.

An example of this branch and the new CI process can be seen in genebean#1

This PR is built atop #573 and includes its change.

@genebean
Copy link
Contributor Author

genebean commented May 9, 2021

@solarkennedy - if you are game, it would be good to enable https://github.com/apps/puppet-community-rangefinder on your modules, or just enable it for your account as it won't hurt anything (that's what I did). I am doing some debugging of CI bits on my fork and you can see its output there: genebean#1

@genebean genebean force-pushed the general_updates branch 11 times, most recently from 2367f7b to e34a2c3 Compare May 11, 2021 14:06
genebean and others added 2 commits May 11, 2021 10:11
This module has long worked on Debian but it had be omitted from the
list of supported OSs. This adds it to the list and also sorts the list
alphabetically.
This includes many changes that are a byproduct of updating to the
latest version of the PDK. All unit tests pass at this stage but
several RuboCop tests still fail.

Tht GH Action was copied from a Vox Pupuli module.
Their module testing is pretty slick and utilizes Beaker just as this
one has traditionally done so it seems like a good place to start. A
couple of lines were commented out as the Gemfile from the PDK doesn't
line up exactly with the one used by most Vox Pupuli modules.

RuboCop is disabled for the moment as it is known that those tests do
not yet pass.
@genebean genebean changed the title [WIP] PDK update + OS and Puppet versions update PDK update, move CI to GH Actions May 11, 2021
@genebean genebean marked this pull request as ready for review May 11, 2021 14:26
@bastelfreak
Copy link
Member

@solarkennedy can you take a look here as well because that's a bigger PR.

@solarkennedy
Copy link
Contributor

Hard to review super well, but it still looks fine to me.
In the readme I noticed the new facts lookup, which is new to me.
I'm glad to see the nodeset stuff go away.

Apparently github actions don't work yet, I guess because of the branch.
I'll merge, and let's see if we can get them green on master.

@solarkennedy solarkennedy merged commit 38946bc into voxpupuli:master May 12, 2021
@genebean
Copy link
Contributor Author

It should be green now. I had the branch running in my fork - results are linked in the top comment. I’m going to start working on the rubocop bits next

@genebean genebean deleted the general_updates branch May 12, 2021 12:19
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.

3 participants