=== mgerdts_ is now known as mgerdts | ||
=== gin is now known as ginsul_ | ||
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: ^ | 17:57 |
rharper | smoser: ok | 18:08 |
blackboxsw | just put up an integration test fix for bionic https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/345256 | 18:58 |
=== blackboxsw is now known as blackboxs | ||
=== blackboxs is now known as blackboxsw | ||
blackboxsw | fixes a couple of things I ran into which testing -proposed for the SRU. | 18:59 |
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 | 19:57 |
* 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:02 |
smoser | wow. | 20:14 |
smoser | hm.. | 20:14 |
* blackboxsw is reading man pages at the moment trying to figure out what/why | 20:15 | |
blackboxsw | and there is no diff in the cloud-init-ca-certs.crt file written on bionic vs xenial | 20:18 |
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:23 |
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:24 |
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:25 |
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:34 |
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:36 |
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:37 |
blackboxsw | agreed, we should hire nacc :) | 20:38 |
blackboxsw | #toosoon | 20:38 |
blackboxsw | thanks for that smoser | 20:39 |
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:03 |
paulmey | thanks in advance | 21:04 |
paulmey | Thanks rharper, I'll wait for you or smoser to respond to finalize those checks. | 23:30 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!