Build a framework for syncing data from outside puppet into a puppet org

RESOLVED FIXED

Status

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: dustin, Assigned: dustin)

Tracking

Details

Attachments

(2 attachments, 2 obsolete attachments)

This will be useful for data from slavealloc (bug 792836) as well as things like synchronizing usernames and keys from LDAP.
Created attachment 823549 [details] [diff] [review]
bug931056.patch

This includes a "fake" extsync, just to show how it works.  I'll either remove that before landing, or remove it in bug 792836.
Attachment #823549 - Flags: review?(bugspam.Callek)
Comment on attachment 823549 [details] [diff] [review]
bug931056.patch

This was causing problems from rsync - see bug 932053.  So I'll have another look.
Attachment #823549 - Flags: review?(bugspam.Callek)
Blocks: 926468
Created attachment 824665 [details] [diff] [review]
bug931056-p1.patch

From the previous patch, I changed this to create temporary files as /etc/hiera/*.tmp, and to exclude those in the rsync-secrets script.
Attachment #823549 - Attachment is obsolete: true
Attachment #824665 - Flags: review?(bugspam.Callek)
Callek, ping
Created attachment 828099 [details] [diff] [review]
bug931056-p1.patch

Sorry, some of the diffs were in a separate commit, and thus omitted from the patch.
Attachment #824665 - Attachment is obsolete: true
Attachment #824665 - Flags: review?(bugspam.Callek)
Attachment #828099 - Flags: review?(bugspam.Callek)
Comment on attachment 828099 [details] [diff] [review]
bug931056-p1.patch

Review of attachment 828099 [details] [diff] [review]:
-----------------------------------------------------------------

::: modules/puppetmaster/manifests/extsync/fake.erb
@@ +1,3 @@
> +echo '---' >> $OUTPUT
> +echo 'fake_param1: <%= param1 %>' >> $OUTPUT
> +echo 'fake_param2: <%= param2 %>' >> $OUTPUT

please remove this file, you have a duplicate in templates/ where it belongs.
Attachment #828099 - Flags: review?(bugspam.Callek) → review+
doc'd at https://wiki.mozilla.org/ReleaseEngineering/PuppetAgain/Extsync
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Blocks: 891859
Created attachment 830361 [details] [diff] [review]
bug931056-p2.patch

I botched things with the last patch, and landed a version without the .tmp stuff included.  And anyway, that was incorrect.  Here's a fix.
Attachment #830361 - Flags: review?(bugspam.Callek)

Updated

5 years ago
Attachment #830361 - Flags: review?(bugspam.Callek) → review+
You need to log in before you can comment on or make changes to this bug.