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
[BUG] apache.config.vhosts.standard: mapping error when "enabled: false" #302
Comments
@genaumann Thanks for this report. Bug confirmed locally using Kitchen. Tracked it down to this line:
Please remove the trailing colon which shouldn't be there. That's working at my end. I'll push through a PR in the meantime. |
@genaumann Fix proposed in #303, please confirm there and we'll get that merged. |
## [1.1.3](v1.1.2...v1.1.3) (2021-04-30) ### Bug Fixes * **config/vhosts/standard:** remove erroneous trailing colon ([dedb9e2](dedb9e2)), closes [#302](#302) ### Continuous Integration * **kitchen+gitlab:** adjust matrix to add `3003` [skip ci] ([3c21740](3c21740)) * **vagrant:** add FreeBSD 13.0 [skip ci] ([298fdf4](298fdf4)) * **vagrant:** use pre-salted boxes & conditional local settings [skip ci] ([f354ace](f354ace))
🎉 This issue has been resolved in version 1.1.3 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Your setup
Formula commit hash / release tag
Current commit hash:
3c21740ba52fa5c2b5cf39cddce6a42d13d17988
Versions reports (master & minion)
Pillar / config used
Bug details
Describe the bug
When a site configuration is to be deactivated (
enabled: false
), the state apache.config.vhosts.standard throws the following error:failed: mapping values are not allowed in this context
When I change the value to
enabled: true
, the state will run without any errors.Log at salt-master:
pillar.get: Default () is of type 'str', must be a dict or list to merge. Merge will be skipped.
The text was updated successfully, but these errors were encountered: