saltstack-formulas / salt-formula

Yes, Salt can Salt itself!
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
197 stars 423 forks source link

fix(pkgrepo): pkgrepo use cmd.shell #534

Closed aboe76 closed 2 years ago

aboe76 commented 2 years ago

PR progress checklist (to be filled in by reviewers)


What type of PR is this?

Primary type

Secondary type

Does this PR introduce a BREAKING CHANGE?

No.

Related issues and/or pull requests

Describe the changes you're proposing

in pkgrepo/suse/install.sls there was a solution (hack) to grep for baseurl, this was failing because of cmd.run in states can't use "pipes" see https://github.com/saltstack/salt/issues/44226 switch this to cmd.shell.

Pillar / config required to test the proposed changes

Debug log showing how the proposed changes work

Documentation checklist

Testing checklist

Additional context

myii commented 2 years ago

Merged, thanks for the fix, @aboe76.

saltstack-formulas-travis commented 2 years ago

:tada: This PR is included in version 1.10.3 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: