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) pinning specinfra to pass testing #82

Merged
merged 1 commit into from
Jun 23, 2016

Conversation

kevpl
Copy link
Contributor

@kevpl kevpl commented Jun 23, 2016

Specinfra's newer versions (>=2.59.1) inspect classes and use
their name method. Hocon shadows the class.name method, and
breaks as such. There's a separate issue to fix the hocon
problem (puppetlabs/ruby-hocon#75).
specinfra has decided to not use the name method either
(mizzy/specinfra#561).
Once that's merged & released, this should be reverted.

Specinfra's newer versions (>=2.59.1) inspect classes and use
their name method. Hocon shadows the class.name method, and
breaks as such. There's a separate issue to fix the hocon
problem (puppetlabs/ruby-hocon#75).
specinfra has decided to not use the name method either
(mizzy/specinfra#561).
Once that's merged & released, this should be reverted.
@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-rspec_btc-intn-sys/31/

@kevpl
Copy link
Contributor Author

kevpl commented Jun 23, 2016

@tvpartytonight review?

@tvpartytonight tvpartytonight merged commit ab82b01 into voxpupuli:master Jun 23, 2016
@bmjen
Copy link
Member

bmjen commented Jun 24, 2016

@kevpl mizzy/specinfra#560 has merged and addressed the problem. Version 2.59.3 has been released. This can be reverted now.

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