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

Updates Beaker tests for Vault #77

Merged
merged 1 commit into from
Jul 10, 2017
Merged

Conversation

petems
Copy link
Contributor

@petems petems commented Jun 13, 2017

  • Updates example spec for beaker (invalid syntax)
  • Changes module install to install archive module
  • Adds new Travis acceptance jobs
  • Updates example in README (invalid syntax)

Copy link
Owner

@jsok jsok left a comment

Choose a reason for hiding this comment

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

Really appreciate this effort 👍👍👍

I'm away on holidays so won't be able to merge for a few weeks, but I'll let you know when I have a chance to review it.

@petems petems force-pushed the fix_beaker_tests branch 2 times, most recently from e214dd9 to 4be5a58 Compare July 1, 2017 13:02
@petems
Copy link
Contributor Author

petems commented Jul 1, 2017

@jsok Build is green now 😄 You got time to review? Cos I've got some other PR's and fixes I want to submit, but I want to wait until the acceptance tests are working before I implement 👍

Copy link
Owner

@jsok jsok left a comment

Choose a reason for hiding this comment

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

Just a few queries. Thanks for getting the build green again!

Gemfile Outdated

RUBY_2_OR_NEWER = RUBY_VERSION >= '2.0.0'
def location_for(place, fake_version = nil)
Copy link
Owner

Choose a reason for hiding this comment

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

What is this for? I can see it used anywhere.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There's some reason I forget, removed! 😓

.travis.yml Outdated
- sudo: required
services: docker
rvm: '2.3.3'
env: PUPPET_INSTALL_VERSION="1.5.2" PUPPET_INSTALL_TYPE=agent BEAKER_set="centos-6-x86_64-docker"
Copy link
Owner

Choose a reason for hiding this comment

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

1.5.2 refers to the version of the puppet-agent? This is fairly old now: https://docs.puppet.com/puppet/4.10/about_agent.html#release-contents-of-puppet-agent-1x

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think because at the time I wrote the original version of these specs, Puppet 4.5 was what was in the LTS release of Puppet Enterprise, I can update it now 😄

* Updates example spec for beaker (invalid syntax)
* Changes module install to install archive module
* Adds new Travis acceptance jobs
* Updates example in README (invalid syntax)
* Updates Gemfile
@jsok
Copy link
Owner

jsok commented Jul 10, 2017

Great will merge now.

@jsok jsok merged commit ad160cc into jsok:master Jul 10, 2017
jsok added a commit that referenced this pull request Jul 10, 2017
Update to summarise #77
@petems petems deleted the fix_beaker_tests branch July 10, 2017 10:27
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.

2 participants