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

pdksync - (FM-7392) - Puppet 6 Testing Changes #321

Merged
merged 3 commits into from
Sep 26, 2018

Conversation

pmcmaw
Copy link
Contributor

@pmcmaw pmcmaw commented Sep 24, 2018

(FM-7392) - Puppet 6 Testing Changes
pdk version: 1.7.0

For the spec_helper_acceptance change I have added in some logic to only install augeas_core if the puppet version is 6.0.0, if the version is greater it will hard fail. This will remind the modules team when the newer version of puppet is released, that we should remove this work around.

I have tested this logic on the following versions:
5.0.0 (installs nothing)
6.0.0 (installs augeas_core)
6.0.1 (Hard failure stating that the installation should be removed)

@david22swan
Copy link
Member

@pmcmaw Failing test's

@david22swan
Copy link
Member

Unit Test's now pass. Acceptance test's still failing

@pmcmaw pmcmaw force-pushed the pdksync_FM-7392_pdk_update branch 2 times, most recently from 582247a to f1089e0 Compare September 26, 2018 10:56
@tphoney tphoney merged commit 73cbbbd into master Sep 26, 2018
@tphoney
Copy link
Contributor

tphoney commented Sep 26, 2018

Great work @pmcmaw @david22swan 👍

@pmcmaw pmcmaw added the feature label Oct 3, 2018
@david22swan david22swan deleted the pdksync_FM-7392_pdk_update branch January 29, 2019 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants