/srv/irclogs.ubuntu.com/2016/11/03/#cloud-init.txt

powersjsmoser: trying to build cloud-init all of a sudden I am getting errors: https://jenkins.ubuntu.com/server/job/cloud-init-integration/17/console12:44
rharperpowersj: didn't you proposed the added coverage report =)12:46
powersjyes lol12:46
powersjso adding "coverage" to test-requirements.txt is the issue12:49
powersjthat is what pypi calls the package and curtin does the same thing12:51
rharperwonder if it just needs to be versioned?13:00
powersjI tried adding "coverage=4.2" and got same error13:04
rharperhrm13:05
rharpercan you repro off of jenkins ?13:05
powersjlet me see. I'll try that after this next session13:05
=== shardy is now known as shardy_lunch
rharperpowersj: I can repro local13:06
powersjrharper: ah! thank you :)13:08
rharperinsane13:08
rharperpowersj: I think we need to add the same package to the requirements.txt and map it to the python-package in archive13:10
rharperpowersj: found a fix13:14
rharperhttp://paste.ubuntu.com/23420677/ powersj13:15
=== smoser changed the topic of #cloud-init to: cloud-init 0.7.8 released 2016-09-12. 0.7.9 open. reviews: https://code.launchpad.net/~cloud-init-dev/cloud-init/+git/cloud-init/+ref/master/+activereviews | fly the W
smoserpowersj, looking13:16
smoserpowersj, hm13:16
smoseroh, yeah. the package build will use it, forgot about that. need to add a build dep13:16
smoserrharper, is right. i'll grab that quick13:17
rharpersmoser: morning!13:17
smoserhey13:18
smoserrharper, coverage was already in test-requirements though13:18
rharpersmoser: yeah, but not in my branch, sorry13:18
smoserhttp://paste.ubuntu.com/23420684/ should work13:19
smoseradding it to "standard named packages"13:19
rharpersmoser: sure, didn't reallize it's standard (which makes sense)13:19
smoseras "standard" is 'coverage' pypy package makes python-coverage and python3-coverage13:19
rharperright13:19
smoserso as this is, we dont need that in the package buidl dependency13:24
smoseras Makefile test does not run coverage.13:24
smoserfor now i think we leave that as it is.13:24
smoserand just add the coverage to the list. it'll get installed in bddeb but not used.13:25
rharpery13:26
powersjrharper: thank you!13:31
smoserpowersj, fixed in trunk now.13:32
powersjsmoser: confirmed working now. Thank you!13:33
=== shardy_lunch is now known as shardy
smoserharlowja, does this  make sense to you: https://code.launchpad.net/~bbaude/cloud-init/+git/cloud-init/+merge/30947815:00
tlonoyhi all, I'm wondering if there is docs presents on how to modify and test cloud-init before submitting code. I've seen this page (https://cloudinit.readthedocs.io/en/latest/topics/hacking.html) but it doesn't goes into great details15:11
tlonoye.g. there doesn't seem to be a test-suite to run15:12
smoserhttps://git.launchpad.net/cloud-init/tree/HACKING.rst15:44
smoserdont know why readthedocs is out of date. i will update it.15:44
smoserie, i just merged that15:44
smosertlonoy, reload https://cloudinit.readthedocs.io/en/latest/topics/hacking.html15:49
harlowjabrb, gotta restart17:38
harlowjawill check out in a few smoser17:38
smoserharlowja, ok.17:41
smoserhad some review commetns for you in a couple mp yesterday to17:41
harlowjayuppers17:42
harlowjasmoser  https://code.launchpad.net/~bbaude/cloud-init/+git/cloud-init/+merge/309478 seems ok to me, i guess such a ordering is needed?17:52
harlowjaweird software be weird, lol17:53
=== rangerpbzzzz is now known as rangerpb
=== rangerpb is now known as rangerpbzzzz

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