-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
salt-ssh not providing jinja file for salt-formula #26585
Comments
You need to specify included/extended jinja templates in your Saltfile with salt-ssh:
extra_filerefs:
- salt://salt/formulas.jinja |
Thanks for that, the Saltfile documentation was pretty lacking but I guess I understand it now. Though this to me seems very much a work-around rather than a fix. Should the compilation of the tarball not find all state dependencies to include them? |
It's not that easy for jinja includes/extends, apparently. |
@TimJones, thanks for the report. |
No problem. Just to add, if adding a Saltfile isn't feasabile for whatever reason (I'm actually doing all this from a runner script so using an instance of
|
This bug is a blocker for any salt-ssh usage for us. Are there nice workarounds that allow us to construct the "extra filerefs" list programmatically? |
Would it be an option to simply include all files in |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue. |
Thank you for updating this issue. It is no longer marked as stale. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue. |
Thank you for updating this issue. It is no longer marked as stale. |
4+ years... |
Yes, to me the If you want to optimize to avoid deploying a huge tar each time:
I tried to look at the way I'll try to attend a salt-ssh metting, there is one tomorrow. |
Hey, any progress? I have this issue, and in very strange way: i have two identical machines with same salt-ssh version (3003.3) and on one machine i have an issue, and on another not. Tried to pass |
Seems to be for me "Fixed" issue by forcing '*'
- common.init
....
- roles.docker-compose-macro And now, it could be used as include: {% import "roles/docker-compose-macro.sls" as compose %}
{{ compose.service('jenkins') }} Now works for me on both machines. Without additional config or options (which did not help anyway). |
Closing this issue due to age and lack of activity. Please test this on version 3006.2 and create a new issue if the problem persists. The new issue template has more information and will allow us to track and reproduce the issue more effectively. Thanks! |
When bootstrapping\setting up a VPS using salt-ssh, I use salt-formula to properly configure the minion after the bootstrap script has done its job.
When running over salt-ssh, the formula fails with the following error:
As you can see, line 15 of the salt-formula/salt/files/minion.d/f_defaults.conf includes the salt/formulas.jinja file, but looking around
/tmp/.root_9e7976__salt/
shows nothing and opening/tmp/.root_9e7976__salt/salt_state.tgz
shows that it never made it into the tar when being compiled.Running the salt-formula unmodified works fine via ´salt new-machine state.sls salt.minion´, and removing the reference to
salt/formulas.jinja
(and all subsequent references toformulas
) allows the formula to run viasalt-ssh new-machine state.sls salt.minion
.The text was updated successfully, but these errors were encountered: