[00:24] <smoser> blackboxsw: nice. i was just about to do that... came in to just go ahead and do an upload
[14:57] <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:58] <smoser> and --dry-run doesn't work so well because it dry-runs everything (even necessary read-only operations)
[18:28] <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:29] <smoser> sounds good.
[18:29] <blackboxsw> I should've changed dryrun when introducing skip bugs, publish, test ... just didn't get around to it
[19:08] <robjo> smoser: https://code.launchpad.net/~rjschwei/cloud-init/+git/cloud-init/+merge/346844  should address parts of lp#1092637
[19:09] <smoser> i think you typod that number
[19:09] <smoser> bug 1772961
[19:09] <smoser> ?
[19:16] <robjo> too many bugs in my Inbox :( , yes cut and paste error
[19:35] <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:39] <smoser> robjo: thanks for looking at that.
[19:40] <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:43] <robjo> sorry still poking around while keeping an eye on the last of the Specter v4 image publishing
[20:24] <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:26]  * blackboxsw is trying to resolve https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1769754