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

796 Fix issues with puppet gem removal of default_env #797

Merged
merged 1 commit into from
Jul 14, 2020
Merged

796 Fix issues with puppet gem removal of default_env #797

merged 1 commit into from
Jul 14, 2020

Conversation

sgnn7
Copy link
Contributor

@sgnn7 sgnn7 commented Jul 14, 2020

Since this is now changed to DEFAULT_ENV, we now create the old
function alias to retain functionality.

Bug: #796
Upstream: https://github.com/puppetlabs/puppet/blob/94df3c1a3992d89b2d7d5db8a70373c135bdd86b/lib/puppet/util.rb

Since this is now changed to `DEFAULT_ENV`, we now create the old
function alias to retain functionality.

Bug: #796
Upstream: https://github.com/puppetlabs/puppet/blob/94df3c1a3992d89b2d7d5db8a70373c135bdd86b/lib/puppet/util.rb
@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) to 85.452% when pulling 73fbbe2 on sgnn7:796-fix-puppet-monkeypatch into 6e35e92 on rodjek:master.

@scotje
Copy link
Collaborator

scotje commented Jul 14, 2020

Tested this locally and it allows pdk test unit to run successfully on my test module.

@scotje scotje merged commit 7df5757 into rodjek:master Jul 14, 2020
@sgnn7 sgnn7 deleted the 796-fix-puppet-monkeypatch branch July 15, 2020 13:57
sgnn7 added a commit to cyberark/conjur-puppet that referenced this pull request Jul 15, 2020
There were upstream issues with broken dependencies that we upstreamed a
fix for and because of this we needed to pin to latest (unreleased)
version of `rspec-puppet` gem. In the process, it seems like we also
discovered that our fork was not required as the tests all pass so it
was likely that we just had a fork to have a pre-release features ready
which are now in mainline.

Upstream bug: rodjek/rspec-puppet#796
Upstream fix: rodjek/rspec-puppet#797
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.

3 participants