Closed sticky-note closed 3 years ago
Shouldn't be a problem however we have had a report about memory issues when passing full dicts by context:
Something we need to consider in the long run.
@myii I think you are right, about that considering the memory issues. especially on large configurations.
That's exactly why I waited for a review @myii
At which number of vhosts can we consider a large deployment ?
The idea a making a local copy of nginx
map, popping nginx:servers:managed
and passing that copy to each templates seems attractive. Will it be sufficient?
@vutny @myii @aboe76, Seems to consume significantly less RAM on sls rendering with:
{%- set _nginx = nginx.copy() %}
{%- do _nginx.pop('snippets') %}
{%- do _nginx.pop('servers') %}
@myii Ok to merge like that ?
:tada: This PR is included in version 2.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:
PR progress checklist (to be filled in by reviewers)
What type of PR is this?
Primary type
[build]
Changes related to the build system[chore]
Changes to the build process or auxiliary tools and libraries such as documentation generation[ci]
Changes to the continuous integration configuration[feat]
A new feature[fix]
A bug fix[perf]
A code change that improves performance[refactor]
A code change that neither fixes a bug nor adds a feature[revert]
A change used to revert a previous commit[style]
Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)Secondary type
[docs]
Documentation changes[test]
Adding missing or correcting existing testsDoes this PR introduce a
BREAKING CHANGE
?No.
Related issues and/or pull requests
Describe the changes you're proposing
pass
nginx
tosnippets
andserver_config
contextsPillar / config required to test the proposed changes
Debug log showing how the proposed changes work
Documentation checklist
README
(e.g.Available states
).pillar.example
.Testing checklist
state_top
).Additional context