| smoser | blackboxsw: nice. i was just about to do that... came in to just go ahead and do an upload | 00:24 |
|---|---|---|
| smoser | blackboxsw: http://paste.ubuntu.com/p/RfPQ3MDVH7/ ? for review-mps ? | 14:57 |
| smoser | i could not figure out how to '--skip' more stuff... | 14:57 |
| smoser | and --dry-run doesn't work so well because it dry-runs everything (even necessary read-only operations) | 14:58 |
| === dhill is now known as Guest96125 | ||
| blackboxsw | smoser, I'm adding an alias, dryrun == skip(bugs,publish,test)... only validate/lint the branch and print errors locally (no remote changes) | 18:28 |
| smoser | sounds good. | 18:29 |
| blackboxsw | I should've changed dryrun when introducing skip bugs, publish, test ... just didn't get around to it | 18:29 |
| robjo | smoser: https://code.launchpad.net/~rjschwei/cloud-init/+git/cloud-init/+merge/346844 should address parts of lp#1092637 | 19:08 |
| smoser | i think you typod that number | 19:09 |
| smoser | bug 1772961 | 19:09 |
| ubot5 | bug 1772961 in cloud-init "suse (packages/suse/cloud-init.spec.in) spec does not build" [Medium,Confirmed] https://launchpad.net/bugs/1772961 | 19:09 |
| smoser | ? | 19:09 |
| robjo | too many bugs in my Inbox :( , yes cut and paste error | 19:16 |
| === rcj is now known as Guest73979 | ||
| === seba is now known as Guest82233 | ||
| === x58 is now known as Guest37083 | ||
| === Guest73979 is now known as rcj | ||
| smoser | robjo: it looks like you still need the 4 changes in https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+ref/fix/1772961-suse-spec-fixes | 19:35 |
| smoser | and then we get to | 19:35 |
| smoser | http://paste.ubuntu.com/p/gZpTmTW3jN/ | 19:35 |
| smoser | robjo: thanks for looking at that. | 19:39 |
| robjo | smoser: my changes were only to address the "missing files" issue, i.e. addition to what you had already done | 19:40 |
| smoser | ah. ok. well, just grab my commits too and take over that branch. | 19:40 |
| smoser | if you'd like i can quickly give you access to an instance that you can use 'run-container' from. | 19:40 |
| robjo | sorry still poking around while keeping an eye on the last of the Specter v4 image publishing | 19:43 |
| === dhill is now known as Guest64332 | ||
| blackboxsw | ok so changes that we landed in cloud-init for salt support freebsd changed the way /etc/salt/pki minion certs are created. Docs (https://docs.saltstack.com/en/2017.7/ref/configuration/minion.html#pki-dir) still claim that the minion certs should live under /etc/salt/pki/minion path, but cloud-init now checks isdir(/etc/salt/pki/minion ) and opts to use just /etc/salt/pki if minion subdir isn't present. This breaks | 20:24 |
| blackboxsw | on ubuntu as those invalid certs in /etc/salt/pki/* are deleted. But, apparently that is intended behavior on FreeBSD. I need to find out why these files are being removed by a 'system salt-minion restart' on bionic++ | 20:24 |
| * blackboxsw is trying to resolve https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1769754 | 20:26 | |
| ubot5 | Ubuntu bug 1769754 in cloud-init (Ubuntu) "salt-minion: public/private keys not preserved in /etc/salt/pki" [Undecided,New] | 20:26 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!