-
Notifications
You must be signed in to change notification settings - Fork 419
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
add a way to set which release of saltstack to use #314
Conversation
SaltStack provides "versioned" repositories, this commit add a way to set which release of salt to use. It adds a pillar "salt:release" which can be set to a specific release (ex: 2016.11). This release is then used to configure properly the repositories URLs for Debian/Ubuntu/RedHat. The default behavior is to point to 'latest', retaining the previous behavior if the "salt:release" pillar is not set.
Please note that this PR is not fully tested. |
@kakwa I will test this tomorrow, I have a few machines to test the different configurations |
I got the following error on Debian based systems: [CRITICAL] Rendering SLS 'base:salt.minion' failed: Jinja syntax error: expected token ',', got 'string' because of the missing trailing comma's in the redhat portion It should be:
|
salt/map.jinja
Outdated
@@ -61,6 +62,8 @@ that differ from whats in defaults.yaml | |||
}, | |||
}, | |||
'RedHat': { | |||
'pkgrepo': 'https://repo.saltstack.com/yum/redhat/$releasever/$basearch/' + salt_release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing trailing comma
salt/map.jinja
Outdated
@@ -61,6 +62,8 @@ that differ from whats in defaults.yaml | |||
}, | |||
}, | |||
'RedHat': { | |||
'pkgrepo': 'https://repo.saltstack.com/yum/redhat/$releasever/$basearch/' + salt_release | |||
'key_url': 'https://repo.saltstack.com/yum/redhat/$releasever/$basearch/' + salt_release + '/SALTSTACK-GPG-KEY.pub' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing trailing comma
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
youps, sorry for that
add missing comas
Thanks @kakwa |
SaltStack provides "versioned" repositories, this commit add a way
to set which release of salt to use.
It adds a pillar "salt:release" which can be set to a specific release
(ex: 2016.11). This release is then used to configure properly the
repositories URLs for Debian/Ubuntu/RedHat.
The default behavior is to point to 'latest', retaining the previous
behavior if the "salt:release" pillar is not set.
Related to #293