This repository has been archived by the owner on Apr 27, 2022. It is now read-only.
Some more automation in the Vagrantfile. #20
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The replace directive is because the relevant patches haven't been merged into obmd itself yet; I'm using that to develop locally. Once the obmd prs are merged we'll remove it. We also probably want to store the httpserver package somewhere more neutral at some point; there's no reason hil-vpn should depend on obmd.
zenhack
force-pushed
the
vagrant-setup
branch
from
8a6ba7a
to
9089b20
Compare
The relevant patches have landed in obmd, so this is no longer necessary.
See the comments.
That does it for calling privop directly, but we still need to get it to work when run via hil-vpnd.
hil-vpnd tries to invoke privop via sudo, which resets the PATH to the defaults. In the centos 7 vagrant image, root does not have /usr/local/bin in its path, so this was failing. We should make these things configurable; see CCI-MOC#19.
zenhack
force-pushed
the
vagrant-setup
branch
from
9089b20
to
57cfad1
Compare
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Just the last 3 commits. With this, you can just:
vagrant up./install.sh...and have everything working, including selinux stuff.
The meat of this is just the last 3 commits; before that is stuff from #17.
See also #19; a lot of this is kindof awkward because some paths are hard-coded and others aren't. It's super weird to that install.sh puts some stuff in /usr/local and other stuff in /usr.