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

Add GitHub workflows for both specification and acceptance tests #117

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jmciver
Copy link
Contributor

@jmciver jmciver commented Mar 30, 2021

The following commit closely mirrors CI execution principles used in the project's Travis-CI environment. Puppet 7 specification support and acceptance testing has been added. The following acceptance tests pairs have been configured:

centos-7 puppet5
centos-7 puppet6
centos-7 puppet7
centos-8 puppet5
centos-8 puppet6
centos-8 puppet7
debian-8 puppet5
debian-9 puppet6
fedora-30 puppet6
fedora-30 puppet7
fedora-31 puppet6
fedora-31 puppet7
fedora-32 puppet6
fedora-32 puppet7
fedora-33 puppet6
fedora-33 puppet7
ubuntu-1604 puppet5
ubuntu-1604 puppet6
ubuntu-1804 puppet5
ubuntu-1804 puppet6

Three functionality omissions exist:

  • The official CentOS 6 Docker Hub container has a YUM configuration error and thus has been omitted from this commit (todo).
  • RHEL8 does not have an official Docker Hub container and will require further research.
  • Automatic releasing of this module to Puppet Forge has not been implemented in the GitHub actions workflow.

A minor fix has been applied to the fedora-30 node-set specification, which should have been added in commit ID d418180.

Acceptance test configuration for Fedora 30 has been updated from
commit adfd8c7 to use the official supported Puppet agent. This fix
was erroneously called out in d418180 as being completed, but is now
applied.
The following commit closely mirrors CI execution principles used in
the project's Travis-CI environment. Puppet 7 specification support
and acceptance testing has been added. The following acceptance tests
pairs have been configured:

centos-7 puppet5
centos-7 puppet6
centos-7 puppet7
centos-8 puppet5
centos-8 puppet6
centos-8 puppet7
debian-8 puppet5
debian-9 puppet6
fedora-30 puppet6
fedora-30 puppet7
fedora-31 puppet6
fedora-31 puppet7
fedora-32 puppet6
fedora-32 puppet7
fedora-33 puppet6
fedora-33 puppet7
ubuntu-1604 puppet5
ubuntu-1604 puppet6
ubuntu-1804 puppet5
ubuntu-1804 puppet6

Three functionality omissions exist:

* The official CentOS 6 Docker Hub container has a YUM configuration
  error and thus has been omitted from this commit (todo).

* RHEL8 does not have an official Docker Hub container and will require
  further research.

* Automatic releasing of this module to Puppet Forge has not been
  implemented in the GitHub actions workflow.
@jmciver
Copy link
Contributor Author

jmciver commented Mar 30, 2021

The following URL provides an example of the output: https://github.com/jmciver/sgnl05-sssd/actions/runs/699926845

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.

1 participant