Hi Andrew, thanks for the review,
Indeed no, I added it to follow upstream instructions, but we can move this to requirements module.

But about `/etc/nginx/conf.d/gitlab.conf` and `/etc/logrotate.d/gitlab` configuration files we do not manage for now the path especially if package is not installed inside catalog.