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!