Update config.sls to disable sorting

[lingonl]

Adding sort_keys=False to json() allows nginx.server.config to be sorted as needed for the resulting config file to work with access_log+log_format (issue #102 ) and other parameters (issue #40)

[gravyboat]

Did you mean to close this? I know the current work around is kind of dirty, but does this catch all the edge cases?

[lingonl]

The fix didn't work, sorry. The order was ok for my config but gets shuffled somewhere else and is still unpredictable. I'll work on a fix after the weekend, maybe the same way the postfix formula works (more explicit with the ordering/options).

Groet,

Joost de Mes

On 27 mei 2016, at 20:28, Forrest [email protected] wrote:

Did you mean to close this? I know the current work around is kind of dirty, but does this catch all the edge cases?

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

[ernstae]

I get this error:
nginx: [emerg] "load_module" directive is specified too late in /etc/nginx/modules-enabled/50-mod-http-auth-pam.conf:1

When I add this to my pillar:

nginx:
  ng:
    config:
      include:
        - '/etc/nginx/modules-enabled/*.conf'

The resulting nginx.conf has this line too far below my vhosts definitions, which are attempting to use TCP stream proxying.

I changed my config.sls file to use sort_keys=False on the json() call, but that didn't resolve the issue.