Skip to content
This repository has been archived by the owner on May 22, 2018. It is now read-only.

High Sierra #863

Open
wants to merge 32 commits into
base: master
Choose a base branch
from
Open

High Sierra #863

wants to merge 32 commits into from

Conversation

jacobbednarz
Copy link
Member

Tracking branch to merge intermediate branches into whilst we get High Sierra support going.

webflo and others added 4 commits October 25, 2017 20:59
WIP: Hiera Update for Puppet 4
Updates the suite to use the secure environment variables from the UI
and ditch `--token` since we are using password auth.
We shouldn't need to be bundling here as the gems should be stored in
the repository cached already.
Updates Boxen to use a fork of the `librarian` gem since the regex
matching for git environment variables was a bit too loose and
overexposed GitHub credentials.
@jacobbednarz jacobbednarz force-pushed the high-sierra branch 5 times, most recently from 4ce0769 to 0f08582 Compare March 4, 2018 00:46
If for any reason Puppet is already on the PATH, get rid of it to stop
it interferring with commands.

Fixes the following error:

```
Error: Could not create resources for managing Puppet's files and directories in sections [:main, :agent, :ssl]: undefined method `exists?' for nil:NilClass
Error: Could not prepare for execution: Could not create resources for managing Puppet's files and directories in sections [:main, :agent, :ssl]: undefined method `exists?' for nil:NilClass
```
During fresh VM runs, I was getting the following exceptions:

```
Warning: Facter: Error loading fact /opt/boxen/repo/.bundle/ruby/2.3.0/gems/facter-2.5.1/lib/facter/processor.rb: cannot load such file -- cfpropertylist
Error: Could not run: cannot load such file -- cfpropertylist
```

To fix this, I'm explicitly adding it to the Gemfile.
@jacobbednarz jacobbednarz mentioned this pull request Apr 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants