Nginx: Compile dynamic modules

[tangrufus]

Note:

• it doesn't compile or install Nginx
• it assumes dynamic module sources are git repos
• it recompiles if Nginx version or dynamic module repos changed
• it skips when nginx_dynamic_modules is empty

Usage (see #867):

# group_vars/all/main.yml
nginx_dynamic_modules:
  ngx_brotli:
    repo: https://github.com/google/ngx_brotli.git
    objs:
      - ngx_http_brotli_filter_module.so
      - ngx_http_brotli_static_module.so

Related: #863

[partounian]

PPA is already set to this at the top of this file. So I would just remove any of your work for adding adding a deb-src but the rest looks okay.

[tangrufus]

How come you don't get this error?

$ sudo apt-get source nginx
Reading package lists... Done
E: You must put some 'source' URIs in your sources.list

[partounian]

Sorry I didn't actually test these.

[tangrufus]

Seems $ add-apt-repository --enable-source {{ nginx_ppa }} is a better choice

[tangrufus]

Updated:

• .so files should have 644 file permission

[tangrufus]

Will this be consider to be merged, or should I extract it to be a galaxy role?

Thanks!

[swalkinshaw]

Still undecided on this one. But I'm on vacation so not really thinking about this 😉

[tangrufus]

Closing this and continue on #867 because this seems won't be merged without ngx_brotli

[swalkinshaw]

I was actually thinking more about this last night and I'm probably fine with it :)

I'll re-open and review it soon

[tangrufus]

More changes to lure you into it:

• rebased with 5e000e8
• make use of Load modules-enabled config files #859
• don't copy .so files
• removed tags: nginx-dynamic-modules
• allow specific git version

[tangrufus]

These commands works on my machine, not sure why it fails on travis:

$ ansible-playbook --syntax-check -e env=development deploy.yml
$ ansible-playbook --syntax-check -e env=development dev.yml
$ ansible-playbook --syntax-check -e env=development server.yml

[strarsis]

@tangrufus: Can I use this for building the nginx pagespeed extension, too?

[tangrufus]

If you would like to build ngx_pagespeed as a dynamic module instead, use --add-dynamic-module= instead of --add-module=........
Pagespeed Doc

I had trouble setting ngx_pagespeed as a dynamic module last August. It was a ngx_pagespeed issue. But from the latest doc, it seems working now. You can give it a try.

---

This is untested and don't expect it will work without debugging:

# group_vars/all/main.yml
nginx_dynamic_modules:
  ngx_pagespeed:
    repo: https://github.com/apache/incubator-pagespeed-ngx.git
    version: some_commit_hash
    objs:
      - ngx_pagespeed.so

[runofthemill]

this would be great to get merged - would allow Trellis to use https://github.com/masonicboom/ipscrub to reduce the surface area for GPDR issues

[strarsis]

Could https://github.com/ANXS/nginx somehow be re-used for this?

[runofthemill]

@strarsis that looks like it'd require a major refactor, since currently nginx is built & configured without an ansible-galaxy role, so highly unlikely, imo. did you try this PR with TangRufus' snippet to see if it works?