Skip to content

(FM-5041) Install PE License for Acceptance Testing #158

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

Merged
merged 1 commit into from
Apr 5, 2016
Merged

(FM-5041) Install PE License for Acceptance Testing #158

merged 1 commit into from
Apr 5, 2016

Conversation

cowofevil
Copy link

To enable installing the module from the module staging forge a PE license
must be present on the master. The test helpers have been updated to add a
method for installing a valid license on a master.

[skip ci]

@phongdly
Copy link
Contributor

phongdly commented Apr 4, 2016

Errors:
puppetlabs-sqlserver/spec/spec_helper_acceptance.rb:38:in <top (required)>': undefined local variable or methodtarget' for main:Object (NameError)

Line 38: https://github.com/cowofevil/puppetlabs-sqlserver/blob/maint/master/FM-5041/install_pe_license/spec/spec_helper_acceptance.rb#L38

@phongdly
Copy link
Contributor

phongdly commented Apr 5, 2016

I passed the license error, but it doesn't know resource type sqlserver::config, might be one of the helper was not called? full logs here: https://gist.github.com/phongdly/ac8f11df660ebb189a8c1bafa5b46fc5

To enable installing the module from the module staging forge a PE license
must be present on the master. The test helpers have been updated to add a
method for installing a valid license on a master. The pre-suite for
install the "puppetlabs-sqlserver" module along with dependencies was
updated to install the modules onto the master then use plug-in sync
to install the modules onto agents.

[skip ci]
@phongdly phongdly merged commit 810c2d8 into puppetlabs:master Apr 5, 2016
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