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

(BKR-1049) Updating require to align with oldest supported PE version #1329

Merged
merged 1 commit into from
Mar 14, 2017

Conversation

cdenneen
Copy link
Contributor

PE and puppet-agent currently shipped with 2.1.9
Oldest supported PE release ships with 2.1.8
https://puppet.com/misc/puppet-enterprise-lifecycle
This is to keep beaker aligned with oldest supported PE release and
current puppet-agent vendored ruby.

PE and puppet-agent currently shipped with 2.1.9
Oldest supported PE release ships with 2.1.8
https://puppet.com/misc/puppet-enterprise-lifecycle
This is to keep beaker aligned with oldest supported PE release and
current puppet-agent vendored ruby.
@puppetlabs-jenkins
Copy link
Contributor

Can one of the admins verify this patch?

@cdenneen
Copy link
Contributor Author

cdenneen commented Feb 7, 2017

@tvpartytonight let me know if I'm missing anything in order to move this PR forward.
Thanks

@kevpl
Copy link
Contributor

kevpl commented Mar 2, 2017

@puppetlabs-jenkins test this please

@puppetlabs-jenkins
Copy link
Contributor

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3587/

@cdenneen
Copy link
Contributor Author

cdenneen commented Mar 6, 2017

@kevpl Can you share what this failed on and I'll fix it?

@kevpl
Copy link
Contributor

kevpl commented Mar 9, 2017

sorry @cdenneen, the build fell off our storage, so I don't know why it failed at this point. Let's try again, & hopefully I'll get around to it faster if it's red.

@puppetlabs-jenkins retest this please.

@puppetlabs-jenkins
Copy link
Contributor

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3617/

@cdenneen
Copy link
Contributor Author

cdenneen commented Mar 9, 2017

@kevpl looks like it passed this time. hopefully 1 step closer to merging :)

@kevpl
Copy link
Contributor

kevpl commented Mar 10, 2017

@cdenneen yep, everything looks good here. We do want this change to be documented though, so I've submitted #1352, and think we should block merging this until we're good with that as well.

If you have any questions or comments about that PR, please comment there.

@tvpartytonight tvpartytonight merged commit 24334e7 into voxpupuli:master Mar 14, 2017
@cdenneen
Copy link
Contributor Author

@kevpl @tvpartytonight new release? not sure how often you push new release.

@kevpl
Copy link
Contributor

kevpl commented Mar 23, 2017

@cdenneen it went out yesterday. If you check the beaker wiki, you can always see when we've scheduled our next release, which is typically every 2 weeks.

@cdenneen
Copy link
Contributor Author

@kevpl thanks... now just to get the voxpupuli/beaker-rspec#89 merged

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.

4 participants