[01:00] blackboxsw: sorry about that. dont know at what point in my history i did that. and then just assumed. [01:00] http://paste.ubuntu.com/25769370/ [01:10] powersj: around ? [01:10] understand whats going on at https://jenkins.ubuntu.com/server/job/cloud-init-ci/ ? [01:11] https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/332446 has 2 failures reported, which i do not think are valid, and also 412 is still running . that seems odd. i figured it posted at end. [01:26] smoser: nacc reported some odd runs at the end of the day. I think something is up with the system or Jenkins master is taking a dump [14:28] 2017-10-19 12:06:38,215 - schema.py[WARNING]: Invalid config:\nntp.pools: {} is not of type \'array\'\nntp.servers: {} is not of type \'array\'\n [14:30] based on this cloud-config: https://github.com/cloud-init/cloud-init/blob/master/tests/cloud_tests/testcases/modules/ntp.yaml [14:31] 2017-10-19 12:06:14,762 - cc_lxd.py[WARNING]: lxd/bridge config must be a dictionary. found a \'\'\n [14:32] cased by this cloud-config: https://github.com/cloud-init/cloud-init/blob/master/tests/cloud_tests/testcases/modules/lxd_dir.yaml [16:32] "If you are using the default CentOS configuration, and if you are not behind a proxy server, fastestmirror is highly recommended." [16:32] well we are behind a proxy... soooo yeah enough of this [16:43] powersj: we can inject a disable fastmirror [16:43] sed -i s/enabled=1/enabled=0/ /etc/yum/pluginconf.d/fastestmirror.conf [16:43] I did that in curtin a while back when we were testing [16:44] I think there's a --disable-plugin to yum [16:44] there is a block where I think we detect if we need to set the proxy [16:44] if that's easier [16:44] well if we only do it when we set the proxy, then it means others can keep using it rather than putting --disable.. everywhere we do a yum [16:44] yeah [16:44] I'll get an MP for that shortly [16:44] * powersj is very fed up with those failures [16:51] https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/332511 [16:51] blackboxsw: rharper ^ disables fastest mirror when using a proxy [16:52] I'll try disabling as well in my centos lxc to validatate here too [16:52] validate even [17:03] powersj: with enabled=0 on both my existing centos6 lxc and a fresh image w/ fastest mirror disabled, I can pull update and install any packages (epel included) without issues with the default mirrors provided. [17:06] we may have to tweak our preferred mirrors at some point if we find problems. but this is a step in the right direction. [17:06] Yeah can try this first :) [17:06] Thanks for checking [17:07] sorry meant to type that here. [17:08] agreed. I've +1'd your branch. ci fell over on it I'm just checking [17:09] Cannot contact torkoal: java.lang.InterruptedException [17:09] bah [17:09] Aborted by Joshua Powers [17:09] hrh [17:09] heh, who does that guy think he is [17:11] yeah... [17:49] smoser: I'm about to merge powersj ci-script fix for no fastestmirror plugin on centos. I know it'll force us to cherrypick into ubuntu/artful your jsonschema branch instead of upstream-snapshot [17:49] https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/332511 [17:49] any concerns smoser ? [17:52] blackboxsw: my concern is that my favorite mirror in japan might not get used [17:52] smoser: ha, and all your click-through ad revenue [17:52] slightly better [17:53] inside "$name" sed -i "s/enabled=1/enabled=0/" "/etc/yum/pluginconf.d/fastestmirror.conf" [17:53] dont need shell there [17:55] blackboxsw: powersj commented there. [17:58] smoser: ah right you mean drop the sh -c [17:58] agreed [17:58] can could do the same with the line above [17:58] yeah, as i said there... milliseconds faster. for a package build and install of dozens of packages... [17:59] the line above uses the shell for redirection [17:59] can't we still pass that w/out sh -c to lxc exec -- "stringcmd >> output" [17:59] * blackboxsw checks [18:00] no. [18:00] there is no shell involved. [18:01] sure enough. smoser correct +1 [18:01] shell is what reads '>>' and opens 'output' for write append and then hands the filehandle to the child process as 1 [18:03] * blackboxsw continues to be pleased w/ hackmd.io for doc writing [18:03] yeah, hackmd.io is the new gist [18:04] they just need to have a 'commit' button [18:04] then powersj can stop making fun of me for using gists and instead make fun of me for using hackio.md [18:05] * powersj shakes head at smoser's "hacks" [18:05] I like that even better [18:27] smoser: https://paste.ubuntu.com/25774147/ [18:27] powersj: no quotes [18:27] there, you're executing a command 'sed -i s/....' [18:28] just like if you typed that (with the quotes) in your shell [18:28] ah ok [18:28] ahhh I see how it is used in other places now [18:29] blackboxsw: updated https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/332511 [18:30] powersj: running it now [18:30] i just want to make sure everyone realizes that i realize how tiny an improvement that is.. [18:30] compared to the vast improvment that powersj made [18:31] I have a whole bag of 'microseconds' for you smoser [18:31] :D [18:31] you got them from not downloading dozens of megabytes from japan [18:31] a penny saved is a penny earned [18:32] and mutiply that by 1000 CI runs [18:32] and you can buy a bag of candy [18:39] mmm, candy [19:06] powersj: drop fastestmirror merged [19:10] blackboxsw: thanks! [19:10] np created a card so we track things that we've actually landed in cloud-init (as I feel it's been too light given recent SRU work) [19:36] https://www.irccloud.com/pastebin/bsxEyHr8/ [19:36] oops sorry about that paste... shame on you irccloud [19:37] powersj: was there a bug associated with this? schema.py[WARNING]: Invalid config:\nntp.pools: {} is not of type \'array\'\nntp.servers: {} is not of type \'array\'\n [19:39] * blackboxsw checks if it is still happening. (I believe it should still happen on master) [19:39] and I'm writing a tiny MP for it [19:42] blackboxsw: I have not put a bug in yet [19:43] powersj: don't worry 'bout it. I'll file one if needed [19:43] I think there's another ntp schema related bug smoser just filed. I can piggy back