=== mgerdts_ is now known as mgerdts
=== gin is now known as ginsul_
[17:57] <smoser> rharper: i'm going to land https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/344768 based on your lgtm.
[17:57] <smoser> blackboxsw: ^
[18:08] <rharper> smoser: ok
[18:58] <blackboxsw> just put up an integration test fix for bionic https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/345256
=== blackboxsw is now known as blackboxs
=== blackboxs is now known as blackboxsw
[18:59] <blackboxsw> fixes a couple of things I ran into which testing -proposed for the SRU.
[19:57] <blackboxsw> hrm smoser/raharper, why would update-ca-certificates generate 3 symlinks in /etc/ssl/certs for a given user-data provided trusted certificate in xenial but only 1 in bionic? https://pastebin.ubuntu.com/p/b7DRPygpYr/
[19:57] <blackboxsw> s/1 in bionic/2 in bionic/
[19:57] <blackboxsw> same user-data trusted provided for xenial and bionic
[20:02]  * blackboxsw was thinking of changing the integration test tests/cloud_tests/testcases/modules/ca_cert.(yaml|py) to instead validate that there is a symlink /etc/ssl/cert/cloud-init-ca-certs.pem -> /usr/share/ca-certificates/cloud-init-ca-certs.crt and >= 1 other link pointing to /etc/ssl/cert/cloud-init-ca-certs.pem
[20:02] <blackboxsw> instead of just counting the number of files generated by update-ca-certs
[20:14] <smoser> wow.
[20:14] <smoser> hm..
[20:15]  * blackboxsw is reading man pages at the moment trying to figure out what/why
[20:18] <blackboxsw> and there is no diff in the cloud-init-ca-certs.crt file written on bionic vs xenial
[20:23] <blackboxsw> hrm reminds me of this maybe https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1077020
[20:23] <ubot5> Ubuntu bug 1077020 in cloud-init (Ubuntu Raring) "cloud-init ca-certs leaves a blank line in /etc/ca-certificates.conf" [High,Fix released]
[20:24] <blackboxsw> I'm seeing a blank line in /etc/ca-certificates.conf right now on xenial and bionic
[20:24] <blackboxsw> will see if removing it affects anything
[20:25] <blackboxsw> removing the leading blank line in /etc/ca-certificates.conf has not affect on number of symlinks created in /etc/ssl/certs in bionic or xenial.
[20:34] <smoser> blackboxsw: if i had to guess its this
[20:34] <smoser>  https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=895075
[20:34] <ubot5> Debian bug 895075 in src:ca-certificates "ca-certificates: Please replace 'c_rehash' with 'openssl rehash'" [Normal,Fixed]
[20:36] <blackboxsw> The "openssl rehash" command creates half that many symlinks (one per
[20:36] <blackboxsw> certificate instead of two) because it uses only the newer hash.
[20:36] <blackboxsw> ahhh ok
[20:37] <smoser> git-ubuntu is awesome
[20:37] <blackboxsw> so not a bug, but intended behavior
[20:37] <blackboxsw> that is an excellent find
[20:37] <smoser> http://paste.ubuntu.com/p/DbrQGvMZCD/
[20:38] <blackboxsw> agreed, we should hire nacc :)
[20:38] <blackboxsw> #toosoon
[20:39] <blackboxsw> thanks for that smoser
[21:03] <paulmey> rharper: around?
[21:03] <rharper> paulmey: here
[21:03] <rharper> sorry, I was out last week
[21:03] <rharper> your branch needs a review ?
[21:03] <paulmey> no problem... yes please:   https://code.launchpad.net/~paul-meyer/cloud-init/+git/cloud-init/+merge/344538
[21:04] <paulmey> thanks in advance
[23:30] <paulmey> Thanks rharper, I'll wait for you or smoser to respond to finalize those checks.