Add server21 as updateserver

This adds server21 and an updateserver module, which it uses.

[Wladimir Palant, 2013-08-16 13:11:02]

LGTM, only nits below.

http://codereview.adblockplus.org/11439212/diff/1/modules/updateserver/files/...
File modules/updateserver/files/update.adblockplus.org (right):

http://codereview.adblockplus.org/11439212/diff/1/modules/updateserver/files/...
modules/updateserver/files/update.adblockplus.org:24: }
Drop that? We don't have any Mercurial repository here.

http://codereview.adblockplus.org/11439212/diff/1/modules/updateserver/manife...
File modules/updateserver/manifests/init.pp (right):

http://codereview.adblockplus.org/11439212/diff/1/modules/updateserver/manife...
modules/updateserver/manifests/init.pp:12: }
I would rather move this down - this is only relevant for setting up nginx
config and certificates, other files are owned by nginx. The mode parameter
seems completely unused actually, it's defined explicitly everywhere with
update.json being the only exception.

http://codereview.adblockplus.org/11439212/diff/1/modules/updateserver/manife...
modules/updateserver/manifests/init.pp:34: ensure => file,
Maybe use nginx/0644 rather than having root as owner here?

[Felix Dahlke, 2013-08-16 14:07:53]

Uploaded a new patch set (already pushed).

http://codereview.adblockplus.org/11439212/diff/1/modules/updateserver/files/...
File modules/updateserver/files/update.adblockplus.org (right):

http://codereview.adblockplus.org/11439212/diff/1/modules/updateserver/files/...
modules/updateserver/files/update.adblockplus.org:24: }
On 2013/08/16 13:11:02, Wladimir Palant wrote:
> Drop that? We don't have any Mercurial repository here.

Oops. Done.

http://codereview.adblockplus.org/11439212/diff/1/modules/updateserver/manife...
File modules/updateserver/manifests/init.pp (right):

http://codereview.adblockplus.org/11439212/diff/1/modules/updateserver/manife...
modules/updateserver/manifests/init.pp:12: }
On 2013/08/16 13:11:02, Wladimir Palant wrote:
> I would rather move this down - this is only relevant for setting up nginx
> config and certificates, other files are owned by nginx. The mode parameter
> seems completely unused actually, it's defined explicitly everywhere with
> update.json being the only exception.

Keeping all files owned by root, as discussed.

http://codereview.adblockplus.org/11439212/diff/1/modules/updateserver/manife...
modules/updateserver/manifests/init.pp:34: ensure => file,
On 2013/08/16 13:11:02, Wladimir Palant wrote:
> Maybe use nginx/0644 rather than having root as owner here?

As discussed, owner is root for now.

[Wladimir Palant, 2013-08-16 14:29:15]

LGTM