[17:54] blackboxsw: did you have changes to https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/345806 that i've not seen / are not pushed there? [17:54] smoser: nope, I'm about to push the annotate schema branch now, just finished unit tests there, then I need to address your comments on os-locall [17:55] should be on that branch in about 10 mins [17:57] ok,. i thought you had said you'd worked on that some. sorry [18:14] smoser: pushed https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/346415 === Guest74322 is now known as mgagne [19:06] blackboxsw: [19:06] $ python3 -m cloudinit.cmd.main devel schema --annotate --config THIS_THING_IS_NOT_A_FILE && echo "that looks good" || echo "fail" [19:06] that looks good [19:07] i just found that when incorrectly using your nicely formated and useful 'to test' bit in Description [19:08] i suspect that part isnt nwe [19:10] +1 not new, but I should fix it in this brnach [19:11] yeah the other parts (nonyaml files for scan or parse errors) used to annotate no error prior to this branch [19:11] I'll add that catch too [19:27] pushed a fix for that annotation. checking review comments [20:27] powersj: fixed your run/container issue [20:29] smoser for shortopts what is r? [20:29] nothing ? [20:30] where did you see that ? [20:30] local short_opts="ahkrsuv" [20:33] powersj: fixed. and fixed the other missing ones there (-n and -p werre missing)