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

(MODULES-6688) Use travis for CI testing, instead of Jenkins #276

Merged
merged 1 commit into from Feb 27, 2018

Conversation

glennsarti
Copy link
Contributor

@glennsarti glennsarti commented Feb 27, 2018

Previously the module was tested in both Travis and private Jenkins CI systems
however this is no longer required. This commit moves the spec tests that
are tested in Jenkins and moves them to Travis;

  • Only test puppet agent and ruby combinations that are installed in AIO installs
  • Split out validate and lint checking to only happen once per job not for all
    matrix cells
  • Now tests on Puppet latest 3.8, first 4.0, latest 4.7.x, 4.10.x and 5.x gems
  • The Hiera Gem needs to be set on Puppet 4.7.1 as it has incompatibilites with
    later hiera gem versions (4.4.2)
  • Brings the Travis CI config file into a similar format as that used by the PDK
    to make migration easier later.
  • The Travis file needs to remain unmanaged until it supports setting the
    bundler_args setting

@glennsarti glennsarti changed the title {WIP} (maint) Use travis for CI testing, instead of Jenkins (maint) Use travis for CI testing, instead of Jenkins Feb 27, 2018
@glennsarti
Copy link
Contributor Author

glennsarti commented Feb 27, 2018

This is in preparation for removing spec tests in Jenkins. This is the normal practice for Modules team.

Previously the module was tested in both Travis and private Jenkins CI systems
however this is no longer required.  This commit moves the spec tests that
are tested in Jenkins and moves them to Travis;
* Only test puppet agent and ruby combinations that are installed in AIO installs
* Split out validate and lint checking to only happen once per job not for all
  matrix cells
* Now tests on Puppet latest 3.8, first 4.0, latest 4.7.x, 4.10.x and 5.x gems
* The Hiera Gem needs to be set on Puppet 4.7.1 as it has incompatibilites with
  later hiera gem versions (4.4.2)
* Brings the Travis CI config file into a similar format as that used by the PDK
  to make migration easier later.
* The Travis file needs to remain unmanaged until it supports setting the
  bundler_args setting
@glennsarti glennsarti changed the title (maint) Use travis for CI testing, instead of Jenkins (MODULES-6688) Use travis for CI testing, instead of Jenkins Feb 27, 2018
@puppetcla
Copy link

CLA signed by all contributors.

@MikaelSmith MikaelSmith merged commit 174356b into puppetlabs:master Feb 27, 2018
@@ -8,6 +8,40 @@
.travis.yml:
unmanaged: true

# Requires the bundler_args to be supported.
Copy link
Contributor

Choose a reason for hiding this comment

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

Why add this now if it can't be used?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Saving myself (or someone else) future work. That's the only thing missing. PDK adds that by default.

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.

None yet

4 participants