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

Vagrant: Use `salt_call_args` instead of minion_id #493

Closed
wants to merge 1 commit into from

Conversation

@jarondl
Copy link

jarondl commented Sep 27, 2016

The hacky minion_id method to pass file-root and pillar-root to
salt-call does not work since Vagrant 1.8.3.

If Vagrant accepts hashicorp/vagrant#7836 , this PR will use the new
argument. It also sets the hostname using the appropriate option,
superseeding #248.

DO NOT MERGE until Vagrant accepts the PR, and releases a new version.


This change is Reviewable

The hacky `minion_id` method to pass `file-root` and `pillar-root` to
`salt-call` does not work since Vagrant 1.8.3.

If Vagrant accepts hashicorp/vagrant#7836 , this PR will use the new
argument. It also sets the hostname using the appropriate option,
superseeding #248.

DO NOT MERGE until Vagrant accepts the PR, and releases a new version.
@aneeshusa
Copy link
Member

aneeshusa commented Sep 27, 2016

@jarondl Thanks for the PR here and to Vagrant! This has actually been on my todo list for a long time. See my feedback on the Vagrant PR.

@aneeshusa
Copy link
Member

aneeshusa commented Aug 30, 2017

The upstream PR hashicorp/vagrant#7836 was superseded by hashicorp/vagrant#8927, which has been merged, so this should be available when the next version of Vagrant is released.

@jarondl, are you interested in reviving this PR when that happens? It will need to be updated and rebased on top of master, and the docs will need to be updated (the docs/salt.md file in this repo) as welll.

@bors-servo
Copy link
Contributor

bors-servo commented Oct 13, 2017

The latest upstream changes (presumably #729) made this pull request unmergeable. Please resolve the merge conflicts.

@aneeshusa
Copy link
Member

aneeshusa commented Oct 13, 2017

Superseded by #729. Thanks again @jarondl for your initial work here!

@aneeshusa aneeshusa closed this Oct 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.