=== zz_natorious is now known as natorious === natorious is now known as zz_natorious === zz_natorious is now known as natorious === natorious is now known as zz_natorious === zz_natorious is now known as natorious === natorious is now known as zz_natorious === rangerpbzzzz is now known as rangerpb === zz_natorious is now known as natorious === natorious is now known as zz_natorious [14:08] claudiupopa, lets just do this here then. [14:08] i owe you some stuff and let me do that now. [14:08] the thing i found yesterday was that trunk was not running 'tox' for me. complaining about windows dlls [14:08] Okay, don't know what's with my sound. [14:09] cloud-init v2 trunk? [14:10] yeah [14:10] i'll get pastbin [14:11] Odd_Bloke, was seeing it too. === zz_natorious is now known as natorious [14:13] Claudiu Popa proposed stackforge/cloud-init: Add the data source base classes and the HTTP OpenStack implementation https://review.openstack.org/188327 [14:14] claudiupopa, http://paste.ubuntu.com/11767934/ [14:14] does it still reproduces? [14:14] yeah. that is trunk on my system right now. [14:14] ImportError: cannot import name 'windll' [14:15] which isnt terribly surprising :) [14:16] Yeah, will take a look, thanks. ;-) [14:16] So..back to bussiness. The openstack stuff should be finally done, after Joshua's final comments. [14:16] I didn't had time to work on other things yet, had some issues with cloudbase-init which needed to be fixed. [14:17] Did you had time to work on the executable part? [14:19] no. because i'm not very productive. [14:19] i will look at your openstack stuff now. [14:25] the crash makes sense. sphinx-autodoc tries to import the modules in order to build the documentation. [14:25] but windll exists only on windows. [14:26] Either we can ignore windows-specific files, but that's hard, since they can be all over the place, or we can write something that does this statically. [14:26] without needing to import the module. [14:28] its not something ew can easily fudge either. [14:29] ie, we can't just make a load_os_specific('windll', 'windows') [14:29] that would just return garbage on linux. because you use stuff from it right away [14:29] windll.kernel32.GetLastError [14:29] Right, that's the case. [14:31] we could proably manage it. [14:31] but if this is just going to be a endless path doesn'st sound like fun [14:32] so doing something statically makes the most sense. [14:32] I'll try to see if I can come up with a sphinx plugin. [14:32] Which operates on ast instead of objects. [14:35] this is a complete hack but it may work as a temporary solution, something like this in kernel32.py: if os.name == 'posix': [14:35] import mock [14:35] sys.modules['ctypes.windll'] = mock.MagicMock() [14:36] I've temporarily done the same in reverse to mock out Linux/posix only python modules to run things on Windows [14:50] claudiupopa, on 'ast' ? [14:50] abstract syntax trees [14:50] will it be an endless path ? [14:51] ie, if we mock those names to None on non-windows, ould that be an endless amount of ongoing work [14:51] yeah, that definitely could be. [14:54] stupid question: do i have to run the brpm scripts on a rhel/centos machine when building cloud-init? [14:55] wasnt sure if there was a way to just run it on my ubuntu dev box as well. figure i just need to make stuff work over in RH land. [15:08] patcable, ? [15:08] the brpm is just a way to get you an rpm. that would hopefully work. [15:08] on ubuntu you can get a deb with bdeb [15:09] smoser: right-- bddeb works great on an ubuntu box, brpm complains about not having some centos packages around. I suspect that I probably need to build the rpms on a centos box, but I wanted ot check [15:11] ah. i understand the question now [15:12] yeah, you need to run that on centos. if you got it to run on ubuntu, patches would probably be accepted, but generally i'd expect to build it on centos. [15:12] that's fine, just wanted to make sure I wasn't missing anything [15:14] thanks! [15:17] we've been hacking to see if we can add support in for handling a user config that has been encrypted, to sort of make a "trusted" cloud init. it's very young right now, though it does work [15:21] patcable, neat. [15:23] yeah. Looking forward to writing about it/proposing patches when we have something that actually works for real :) [15:39] claudiupopa, still there? [15:39] what is _METADATA_NETWORK_KEY = "content_path" [15:46] content_path is an entry from under network_config, with a path for a debian interfaces file. [15:46] http://sprunge.us/KDRX something along these lines [15:54] claudiupopa, ok. i have some very tiny comments. [15:54] sure thing [15:54] i think i'm fine to pull it because i've taken so long, and we can fix them later if you want. [15:55] if it's not too time consuming, I could fix them right away. [15:56] http://paste.ubuntu.com/11768400/ [15:56] claudiupopa, ^ [15:57] metadata_network_key was just confusing to me since its not network specific in any way. its more "referenced_payload_key" or something. [15:57] cool, thanks! [15:59] i did not test that regex. its probably wrong [15:59] i just figure if we're looking for YYYY-MM-DD, we might as well check for that expicitly [15:59] rather than 2339595-2939395995595-12224 [15:59] Yeah, makes sense. :D [15:59] i'd even say you can limit it to a '2' in the first digit. realizing that future humans will laugh histerically at my short sitedness in the year 3000 [16:06] Claudiu Popa proposed stackforge/cloud-init: Add the data source base classes and the HTTP OpenStack implementation https://review.openstack.org/188327 [16:08] smoser: should be fixed. I prefered to leave http_client.CONFLICT as is for now. [16:12] how come ? it just seems wierd to me to have to go elsewhere for that little thing. [16:12] i'm not strongly opposed just curious [16:15] Claudiu Popa proposed stackforge/cloud-init: Add the data source base classes and the HTTP OpenStack implementation https://review.openstack.org/188327 [16:15] yeah, I guess you're right, url_helper should have already what we need. [16:16] claudiupopa, VERSION_REGEX.match(version) [16:16] that will currently match 2015-05-23344555666 [16:16] this is completely nit picky, i do accept :) [16:16] right, I was using .search. [16:17] r'^\d{4}-\d{2}-\d{2}$' [16:17] that does the right thing [16:18] yeah, fixing right now. [16:20] Claudiu Popa proposed stackforge/cloud-init: Add the data source base classes and the HTTP OpenStack implementation https://review.openstack.org/188327 [16:20] woot [16:20] finally. [16:20] :D [16:23] now we need a couple of config modules and that main executable and we're set for the ci. ;-) [16:25] thank you claudiupopa [16:25] Merged stackforge/cloud-init: Add the data source base classes and the HTTP OpenStack implementation https://review.openstack.org/188327 [17:44] so i'm trying to solve [17:44] http://stackoverflow.com/questions/23663070/setup-https-proxy-for-pip-install-command-within-tox-environment [17:44] in a windows friendly way. [17:44] i tried [17:46] tried http://paste.ubuntu.com/11768965/ [17:46] what windows [17:46] *whats windows [17:46] :-P [17:46] but tox 1.6 (in Ubuntu 14.04) doesn't want to substitute {toxinidir} for me. [17:47] any suggestions there? [17:47] remember httpretty and doesn't work so well when http_proxy is set. [17:48] hm.. wonder if maybe we could use no_proxy.. [17:48] smoser: Could you paste the tox.ini you're having problems with? [17:48] http://paste.ubuntu.com/11768965/ [17:54] it works as expected on wily [17:54] basic issue that i was wanting to solve is this: [17:54] http://paste.ubuntu.com/11769000/ [17:54] smoser: What do you see on trusty? [17:55] it tries to execute the string {toxinidir}/tools/foo [17:55] just no substitution [17:55] Try doing "changedir = {toxinidir}" and then just "./tools/..." for the command. [17:55] the paste above shows that tox doesnt run if http_proxy / https_proxy is set, but i need it set when i build in this environment (serverstack has no access to intertubes other than via proxy) [17:56] Right, so you just want the proxy set for the pip call. [17:57] its funny, cause it says "default: {toxinidir}" but that is not the case either on 1.6 [18:11] Odd_Bloke, it doesnt do it. [18:11] smoser: Doesn't do what? [18:11] doc says "when executing the test command". [18:11] it doesnt change the dir. [18:12] cwd at that point is the .tox// [18:13] smoser: Hmm, sounds like we might need to bump the tox version after all. :/ [18:14] or i can just live with having to hack the full path in that case. === natorious is now known as zz_natorious [20:24] Scott Moser proposed stackforge/cloud-init: tox: disable proxies when running nosetests https://review.openstack.org/195294 [20:36] Odd_Bloke, ^ theres a solution there if you want to review [20:39] smoser: Just did. [20:39] Looks good apart from some missing words in a comment. :) === zz_natorious is now known as natorious === natorious is now known as zz_natorious === zz_natorious is now known as natorious