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

(maint) Allow use of beaker 3.x in acceptance tests #546

Merged
merged 1 commit into from
Feb 3, 2017

Conversation

ScottGarman
Copy link
Contributor

This updates the Gemspec to allow the use of beaker 3.x, which
is needed to include support for new platforms (Ubuntu 16.10
in particular).

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

👍

@puppetcla
Copy link

CLA signed by all contributors.

@MikaelSmith
Copy link
Contributor

If you want this everywhere, you should target it at LTS-1.7. Where does it apply?

@ScottGarman ScottGarman changed the title (maint) Allow use of beaker 3.x in acceptance tests DO NOT MERGE (maint) Allow use of beaker 3.x in acceptance tests Jan 23, 2017
@ScottGarman
Copy link
Contributor Author

Marking this do not merge until some unexpected issues with beaker are sorted out.

@ScottGarman
Copy link
Contributor Author

@MikaelSmith btw, this is needed for Ubuntu 16.10 support, which is targeted for release in 1.9.0. Not sure if we want it everywhere atm.

@MikaelSmith
Copy link
Contributor

Ok, then master seems sufficient.

@ScottGarman ScottGarman changed the title DO NOT MERGE (maint) Allow use of beaker 3.x in acceptance tests (maint) Allow use of beaker 3.x in acceptance tests Jan 23, 2017
@ScottGarman
Copy link
Contributor Author

Ok, the beaker-facter gem has been updated to resolve the issues we were seeing. This PR is now safe to merge.

@ScottGarman
Copy link
Contributor Author

This PR is blocked again until puppetlabs/facter#1495 is addressed.

@ScottGarman ScottGarman changed the title (maint) Allow use of beaker 3.x in acceptance tests DO NOT MERGE (maint) Allow use of beaker 3.x in acceptance tests Jan 24, 2017
@ScottGarman ScottGarman changed the title DO NOT MERGE (maint) Allow use of beaker 3.x in acceptance tests (maint) Allow use of beaker 3.x in acceptance tests Jan 31, 2017
This updates the Gemspec to allow the use of beaker 3.x, which
is needed to include support for new platforms (Ubuntu 16.10
in particular).
@ScottGarman ScottGarman changed the base branch from master to stable February 3, 2017 00:23
@ScottGarman
Copy link
Contributor Author

I've rebased this PR and targeted it at stable for inclusion in puppet-agent 1.9.1.

@MikaelSmith MikaelSmith merged commit bc490cf into puppetlabs:stable Feb 3, 2017
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