minimalcc_migrator was mentioned during the Summit, don't remember by who - so it is "legacy" and only required for moving from very old cloud-init releases? The docs (and source code) are not very clear about its purpose02:12
blackboxswminimal: that comment was me about dropping cc_migrator. it was put into cloud-init in version 0.7.0 in 2012. the migrate the semaphore marker file names from versions prior to 0.7.0. I honestly don't think that we are on an upgrade path from 0.7.0 -> 23.X that we need to worry about anymore. Any VM that is on an upgrade path from 0.7.0 would be on python 2.7 or earlier and will not be able to upgrade past 19.4.02:48
minimalok, that clarifies. So I'll take it out of cloud.cfg on images I create ;-)02:51
blackboxsw19.4 was the last python 2.7 version if I recall. so... I think it highly unlikely any image created in the last decade will still make use of an upgrade path that would warrant rewriting the names of semaphore files.02:58
minimalremoving that will shave fractions of a millisecond off c-i boot times ;-)03:05
minimalblackboxsw: related to upstreaming of patches to c-i from distros that was briefly mentioned in Summet, I noticed that the rendered cloud.cfg for "amazon" doesn't specify any packages (i.e. RPM) management module at all, so I guess AWS are tweaking this in their own packages03:12
blackboxswthink of all the savings across your thousands of VMs :)03:13
blackboxswminimal: I'm guessing the same, I thought I recalled them mentioning that they roll their own cloud.cfg in packaging.03:15
minimalactually must add a comment to this effect to my PR #4272 (i.e. I didn't break it, it was broken already) ;-)03:15
minimalseems strange for the c-i repo's template to have "amazon" templating then if it isn't used03:20
meenalet's see if this passes on Linux! https://github.com/canonical/cloud-init/pull/434813:19
-ubottu:#cloud-init- Pull 4348 in canonical/cloud-init "util.get_proc*: Fix tests on non-Linux systems" [Open]13:19
meenathis now brings us down to 4 failing tests.13:24
meenatime for lunch.13:25
meenahrm… okay, i think i went down the wrong path15:32
meenaand another one: https://github.com/canonical/cloud-init/pull/435122:07
-ubottu:#cloud-init- Pull 4351 in canonical/cloud-init "cc_growpart: fix tests to run on FreeBSD" [Open]22:07
-ubottu:#cloud-init- Pull 4352 in canonical/cloud-init "cmd/clean: fix tests on non-Linux platforms" [Open]22:21
meenaand now we're down to one failure22:21
meenaoh great, this one passes in isolation, but not as part of the whole suite22:33
meenaE       AssertionError: b'hello world' != b'/usr/local/bin/bash: warning: setlocale: [78 chars]orld'                                                                                                                                                                          22:35
meenai got a fix.23:05
meenafix: https://github.com/canonical/cloud-init/pull/435523:23
-ubottu:#cloud-init- Pull 4355 in canonical/cloud-init "subp: Fix spurious test failure on FreeBSD" [Open]23:23
meenajust for giggles, i tried running the test-suite without Bash… 26 failures.23:23
meenai think if we ignore the OpenNebula stuff which actually needs it, everything else could be changed to /bin/sh without anyone noticing anything.23:24
meenadone. I need sleep23:26

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!