=== harlowja is now known as harlowja_away | ||
=== harlowja_away is now known as harlowja | ||
=== harlowja is now known as harlowja_away | ||
smoser | harlowja_away, bah to pylint. new version in ubuntu now at 1.X, spews warnings about everything. | 16:12 |
---|---|---|
smoser | :-( | 16:12 |
smoser | http://paste.ubuntu.com/6768706/ | 16:15 |
=== harlowja_away is now known as harlowja | ||
harlowja | smoser arg, stupid pylint | 17:52 |
harlowja | logging-not-lazy | 17:52 |
harlowja | lol | 17:52 |
harlowja | jeez, when did pylint get so crzy | 17:52 |
smoser | harlowja_away, https://code.launchpad.net/~utlemming/cloud-init/vendor_data/+merge/201033 your thoughts there would be appreciated also | 17:53 |
harlowja | sure boss | 17:53 |
harlowja | :) | 17:53 |
smoser | some of those things its seeing are reasonable. | 17:53 |
smoser | but some (unused-argument or unused-variable) | 17:53 |
harlowja | ya | 17:53 |
smoser | we named them with _ to specify that! | 17:53 |
harlowja | nutty | 17:54 |
harlowja | which afaik is convention | 17:54 |
harlowja | k, scott, seems ok to me | 18:25 |
harlowja | smoser harmw ok, tested basic freebsd (basics), seems to be starting fine and finding the distro and all | 18:34 |
harlowja | i'll submit up the rc.d files that sean made | 18:35 |
harlowja | https://code.launchpad.net/~harlowja/cloud-init/freebsd-rc.d/+merge/202155 | 18:42 |
harlowja | harmw ^ | 18:42 |
harmw | compare lines 25,26 with 33 ;) | 18:43 |
harlowja | ya, sean and his '{' | 18:44 |
harlowja | :-P | 18:44 |
harlowja | will fix this for him, lol | 18:44 |
harmw | (lines 50,51 in cloudfinal.. 2 blanc lines, instead of just 1) | 18:45 |
harmw | and more like that as well :p | 18:46 |
harlowja | hmmm, all fixed, lol | 18:46 |
harmw | hm, what could/would cloudinit_override() do anyway? | 18:46 |
harlowja | not sure, let me ask him why | 18:47 |
harmw | ah great :) annoying stuff to bitch about, those whitelines, I know :p | 18:47 |
harlowja | harmw thats just reflecting the same thing in the other sysvinit scripts | 18:48 |
harmw | its the precmd, though I dont know what sourcing that specific file would accomplish | 18:48 |
harmw | ah ok | 18:48 |
harlowja | http://bazaar.launchpad.net/~cloud-init-dev/cloud-init/trunk/view/head:/sysvinit/redhat/cloud-init#L61 | 18:49 |
harlowja | i guess just reflecting that functionality | 18:49 |
harmw | indeed | 18:50 |
harmw | though CLOUDINITARGS is specified in one of the files in L62 or L63 | 18:50 |
harmw | and the freebsd script doesn't use anything that could've been specified in a sysconfig file, hence my question :) | 18:51 |
harlowja | sure sure, understood, maybe for future use then :-P | 18:52 |
harmw | ofc, while that could be an argument to completely drop that part for now as well :p | 18:53 |
harlowja | either or | 18:53 |
harlowja | harmw so i think sean will packge up any missing dependencies and get those shoved up to freebsd ports also | 18:57 |
harmw | cool, I posted the pkg stuff already didn't I | 18:57 |
harlowja | think so, but might just be good to make sure we have a good list | 18:58 |
harlowja | known ones i had to install [configobj, jsonpatch, Cheetah, argparse] | 18:58 |
harlowja | [pyserial, oauth] probably also? | 18:58 |
harlowja | ^ those 2 are datasource usage dependent | 18:59 |
harmw | pkg install python27 py27-yaml py27-requests py27-prettytable py27-cheetah py27-boto dmidecode e2fsprogs gpart sudo | 18:59 |
harmw | and jsonpatch, manually | 18:59 |
harlowja | k | 19:00 |
harmw | though those could've brought in some of the ones you just mentioned aswell :) | 19:00 |
harlowja | np, will make sure sean has a good list | 19:01 |
harmw | great | 19:01 |
smoser | harlowja, you have a thought ? | 19:04 |
smoser | i'm looking to put vendor-data into nocloud | 19:04 |
smoser | that would seem pretty easy | 19:04 |
smoser | but it uses read_seeded | 19:04 |
smoser | which looks for user-data and meta-data | 19:05 |
smoser | both of which are "required" | 19:05 |
smoser | but vendor-data would be optional | 19:05 |
smoser | thats easy peasy for "file://" | 19:05 |
smoser | but not for http:// | 19:05 |
smoser | as we have retries built in. (and at the moment it doens't look like we have support in 'read_file_or_url' for returning None on 404) | 19:06 |
harlowja | u should be able to check the status code of the response right? | 19:07 |
harlowja | http://bazaar.launchpad.net/~cloud-init-dev/cloud-init/trunk/view/head:/cloudinit/url_helper.py#L86 ? | 19:07 |
harlowja | which in the case of a file 'response' is always 200 | 19:07 |
harlowja | how do u want it to work i guess | 19:09 |
harlowja | is the question :-P | 19:09 |
smoser | harlowja, http://paste.ubuntu.com/6769790/ | 19:30 |
smoser | what do yo uthink of that ? | 19:30 |
smoser | none_on not the best of variable names, but a list of status_codes that should return None without retry | 19:30 |
harlowja | hmmm, what about a stop_retry_callback ? | 19:31 |
harlowja | then u can pass that callback the last request, and it can stop the retry if it wants | 19:32 |
harlowja | ? | 19:32 |
harlowja | what do u think smoser ? | 19:34 |
smoser | hm.. | 19:36 |
smoser | so you would just pass the exception | 19:36 |
smoser | the UrlError ? | 19:36 |
smoser | what is check_status ? | 19:36 |
smoser | i think i can acutally just use that. | 19:37 |
harlowja | ya | 19:38 |
harlowja | there u go | 19:38 |
harlowja | seems like a good usage | 19:38 |
smoser | k | 19:38 |
smoser | ok. utlemming i think i'm gonna merge the vendordata stuff now. | 20:08 |
smoser | and then i'll merge some other thigns that i think halrow had suggested | 20:09 |
utlemming | smoser: ack | 20:09 |
smoser | and then i'm going to work to make NoCloud take vendordata | 20:09 |
smoser | utlemming, i think that 'mergedvendoruser' is vestigial | 20:10 |
smoser | right? | 20:10 |
harlowja | like my third baby arm | 20:12 |
harlowja | errr | 20:12 |
harlowja | haha | 20:12 |
smoser | yeah, i've never really understood why you have 3 arms | 20:13 |
smoser | wierd | 20:14 |
harlowja | lol | 20:28 |
harlowja | not my fault at summits that the 3rd baby arm hits u in face, lol | 20:29 |
harlowja | *mind of its own and all | 20:29 |
smoser | harlowja, you there? | 21:31 |
smoser | your added tests/unittests/test_ec2_util.py | 21:31 |
smoser | has 2 test_metadata_fetch_key | 21:31 |
smoser | jeeze harlowja . | 21:47 |
smoser | now i have to fiddle wit hthis biuld-depends thingy httpretty | 21:47 |
harlowja | lol | 21:49 |
harlowja | smoser well its a nice http testing library | 21:49 |
harlowja | it should hopefully already exist | 21:49 |
smoser | yeah. but lots of fallout | 21:49 |
smoser | bddeb now busted | 21:50 |
smoser | (due to version) | 21:50 |
harlowja | hmmm | 21:50 |
smoser | and the version you depend on is > what we have in trusty | 21:50 |
smoser | (but you work on trusty) | 21:50 |
smoser | so i'm dropping that to 0.7.0 | 21:50 |
harlowja | k | 21:50 |
harlowja | i think thats fine | 21:50 |
harlowja | i can alter it if we really need | 21:50 |
harlowja | and drop it | 21:50 |
smoser | ithink th eeasies tthign right now is to drop it as a 'Requires' | 21:53 |
smoser | but i guess i'd like a BuildRequires or somthin and put that in | 21:53 |
smoser | but we have to plumb that into the ./packages | 21:53 |
harlowja | ya, typically test-requires right? | 21:54 |
harlowja | and thats only used for testing | 21:55 |
smoser | is it. | 21:55 |
harlowja | so then it doesn't need to be in buildrequires either | 21:55 |
smoser | ? | 21:55 |
harlowja | ya, from what i've seen at least | 21:55 |
smoser | well in ubunt packaing i'm running the tests | 21:55 |
harlowja | kk | 21:55 |
smoser | so a file 'test-requires' ? | 21:55 |
harlowja | sureee | 21:58 |
harlowja | at least thats how openstack projects do it, maybe there is a different way | 21:59 |
smoser | yeah. thats fine. | 22:02 |
smoser | i'm gonna commit something really quick here to just drop tha tfor now | 22:02 |
smoser | could you try to do a test-requires ? | 22:02 |
smoser | err.. i'll handle that | 22:03 |
smoser | ok. well, just pushed. './tools/bddeb -us -uc' works again | 22:11 |
harlowja | i can | 22:11 |
harlowja | shall i? | 22:11 |
smoser | i added test-requires | 22:11 |
harlowja | kk | 22:11 |
smoser | but tidoesnt get used anywhere really. | 22:11 |
harlowja | can probably add it to the makefile when make test is ran | 22:11 |
smoser | alright. i have to run. | 22:12 |
smoser | have a nice weekend all. | 22:12 |
smoser | thanks for work. | 22:12 |
harlowja | byeeeeeeeee! | 22:12 |
smoser | i'll do more cloud-init work monday i hope | 22:12 |
harlowja | *waves 3rd arm | 22:12 |
smoser | and vendordata support for openstack (through the ec2metadata service) | 22:12 |
harlowja | ya, i gotta work more on my openstackdatasource | 22:13 |
smoser | and then when we get that Openstack datasource (that harlowja) was going to add... | 22:13 |
smoser | :) | 22:13 |
smoser | night night | 22:13 |
smoser | later. | 22:13 |
harlowja | lata | 22:13 |
kwadronaut | morning | 23:20 |
kwadronaut | cloud-init query --name instance_id gives me NotImplementedError: Action 'query' is not currently implemented | 23:21 |
kwadronaut | which makes me a sad panda. using the package from debian backports, 0.7.2 Is this expected, anything i can do t ofix it? | 23:22 |
harlowja | hmmmm, i still think its not fully implemented (unless i forget) | 23:29 |
harlowja | https://code.launchpad.net/~harlowja/cloud-init/query-back-duo | 23:29 |
harlowja | kwadronaut ^ | 23:29 |
harlowja | 'Integrate a slightly cleaner query tool, bring it back to life!' | 23:29 |
harlowja | lol | 23:29 |
kwadronaut | heh | 23:30 |
kwadronaut | it's i think good to have, especially for beginners trying to debug. | 23:31 |
harlowja | agreed | 23:33 |
harlowja | most of the data though is at /var/lib/cloud/ though | 23:33 |
harlowja | http://cloudinit.readthedocs.org/en/latest/topics/dir_layout.html | 23:33 |
harlowja | but not so easy as just using that tool | 23:34 |
kwadronaut | yep | 23:41 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!