esalberg

Rebased this against current master. Please let me know if I need to make any further changes for it to be approved to merge.

Thanks - even with the timeout on the webhook increased, we're still seeing evidence of overlapping r10k runs, just not quite as frequently. I'm trying to find time to do...

That branch looks great - I'll test it out asap. Since the webhook is a template, I would think it could be possible to have an option to forego the...

Sorry about that - I got pulled off onto some other things, one of which was upgrading to PE 2016.4.2. Could there be an update to r10k in 2016.4.2 (versus...

I implemented the latest r10k module code + fork and was not able to reproduce either the multiple webhook deploys or the "parent directory does not exist" error, which I...

I was referring to this in init.pp: (# Check if user is declaring both classes (# Other classes like r10k::webhook is supported but (# using both classes makes no sense...

I can see your point, but there is also documentation for using the functionality in a single installation here: https://docs.gitlab.com/omnibus/update/README.html#single-deployment I see the value in setting up the process as...

I did not add any spec tests so far, because 1) I wasn't sure how to test getting a class variable specifically (the restart functionality is already spec tested), and...

In case it's useful, this is the functional testing I completed: Functional testing: DONE - restart false global (hiera global yaml) - updates file, does not refresh DONE - restart...

I know my tests are failing - I figured out how to run them locally and have fixed most of the issues. I'll update as soon as I'm done (lots...