netconfig.managed forces base enviroment as default
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 15.7k
- Forks
- 5.6k
- Avg merge
- 2d 44m
- Merged PRs (30d)
- 80
Description
Description of Issue
netconfig.managed forces the use of base as the default for saltenv. this if you aren't looking for it can cause problems if you are planning to use netconfig.managed inside an environment and forget to specify - saltenv: {{saltenv}} in the stanza.
Setup
setup netconfig.managed in a different environment with the template file in that same environment. but don't include a saltenv line in the stanza.
Steps to Reproduce Issue
Use the above setup and run a highstate on the proxy minion with --saltenv=your_enviroment
Watch as the template is tried to be pulled from base and fails.
Versions Report
(Provided by running salt --versions-report. Please also mention any differences in master/minion versions.)
This looks like saltenv was changed from None to base in 2019.2.0 in the code. It should most likely be returned to None.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the netconfig.managed entry point and inspect the saltenv handling, especially the change introduced in 2019.2.0. Reproduce the highstate with --saltenv=your_environment and a template in that environment; done means the template lookup uses the selected environment when no saltenv is specified in the stanza.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- infrastructure
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100