=== rangerpbzzzz is now known as rangerpb [13:39] Hi ! I was looking to get an OpenStack environment going with CentOS guests with config-drive provided network configuration. It looks like I would need the RHEL version of http://bazaar.launchpad.net/~cloud-init-dev/cloud-init/trunk/revision/1189?start_revid=1189 [13:40] harlowja: ^ I was told you might be able to help ? [14:56] harlowja, ^ [16:32] sup [16:56] harlowja, dmsimard wants you to fix centos for network config [16:56] and config drive :) [16:56] :-P [16:56] ya, damn [16:56] lol [16:56] i knew it, ha [16:56] oh hai [16:56] well, I'm willing to help and all [16:57] friend here mgagne pointed me to an (older?) iteration of that implementation I guess, https://github.com/mgagne/cloud-init-fedora-pkg/blob/epel7/cloud-init-0.7.5-network-info-support.patch [16:58] But the way it was just merged (three weeks ago, talk about timing) looks similar yet different [16:58] harlowja: original version here: https://github.com/racker/cloud-init-fedora-pkg [16:59] cloud-init will be in core OS packages in RHEL 7.3 and CentOS 7.3 this year so it'd be great to have a version cut with that support baked in [17:01] mgagne dmsimard ya, i'd like to have that in also [17:01] do u guys want to work on that, or should i? [17:01] i moved companies recently, so i might have to do some of the legal CCLA crap again smoser [17:01] i gotta aks on that [17:01] *ask [17:02] harlowja: which goes back to my question yesterday asking about a 0.7.7 release... :) [17:02] I can help testing but I'm not knowledgeable enough about those bits to do it myself [17:02] smoser is mr.release ;) [17:02] he's gonna do git soon also [17:02] lol [17:02] i just know it [17:02] i believe [17:02] i'll send smoser a cookie if he does git [17:02] lol [17:03] I would contribute to that cookie fund. [17:04] :) [17:07] harlowja, git will come [17:07] git willi come [17:07] smoser tommorow? [17:07] lol [17:07] today? [17:07] after 16.04 releases (next thursday)... not before. [17:07] 16.04 [17:07] damn it [17:08] another LTS [17:08] i just reinstalled 14.04 [17:08] lol [17:08] if spandhe joins maybe we can convince her to do it. [17:08] she's still yahoo, right ? [17:08] ya [17:09] dmsimard mgagne if u guys want to get a patch working [17:09] https://github.com/mgagne/cloud-init-fedora-pkg/blob/epel7/cloud-init-0.7.5-network-info-support.patch i guess ? [17:09] just maybe a few more tests ? [17:09] cause that patch doesn't seem to have to many [17:10] then maybe by the time those tests happen, smoser will have git [17:10] and we can all have cookies [17:11] i believe! [17:11] lol [17:13] harlowja: let me check if have a more up2date version [17:13] mgagne np [17:13] because we are reading network_info.json now, not vendor_data.json [17:13] right [17:14] that's fine, some network_info.json test cases, that give a sample network_info.json in and check the output would be cool [17:14] *and/or multiple sample inputs ---> check the outputs [17:15] that's scary, I think I fixed it for debian but not rhel [17:17] harlowja: for trusty, for what it's worth: https://gist.github.com/mgagne/46748012efa1ff3389b380a25bedb14d [17:17] cool, ya, tests for these would be nice ;) [17:17] like a bunch of input data files and expected output files [17:18] for debian and ubuntu and redhat and ... [17:18] maybe to far-fetched, butttt [17:18] I might not be the right person to ask to invest time on it atm ^^' [17:18] kk [17:18] dmsimard ? [17:19] don't know, doesn't work with us anymore =) [17:19] delegation not working [17:19] should patch be targeted for v2 or 0.7.x is fine? [17:19] must delegate harder, lol [17:19] still waiting for my clone to arrive from mail [17:20] UPS lose it? [17:20] lol [17:20] unfortunate yes =( they also say we can't ship bio stuff :P [17:20] ah [17:20] good point [17:23] I hear fedex or DHL is looser on the restrictions [17:23] nice nice [17:24] dmsimard let me know if u have time for those kinds of tests [17:24] if not, i can try, but no guarantees on time for that :-P [17:24] or smoser can get git and then we can accept patch, and i can start adding tests [17:24] and cookies [17:31] Out of curiousity, what is the effort involved in moving to git? Is it more than just import-from-bzr-export-to-github? [17:31] larsks we already have that [17:31] but its not in sync with bzr [17:31] larsks https://github.com/openstack/cloud-init is that (export from bzr, import to github) [17:31] Right, I have noted that :). Is the goal to maintain the bzr repo and keep it in sync automatically? [17:31] but bzr has a native mode [17:32] Or why not just perform the manual sync once, then kill the bzr repo and go on our merry way? [17:32] i delegate question to smoser [17:32] :) [17:33] I am using git locally (via git-remote-bzr), which works great for me but confused the heck out of dmsimard yesterday when I forgot and gave him a git commit id from my local repo... [17:34] i think the idea was to keep 0.7.x on bzr (and bzr-git) and anything new using that git above [17:35] That seems clunky, because people are going to continue to make patches against the 0.7.x code for packages, and it would be nice to have it in the same repo as newer stuff for ease for cherry-picking, etc. [17:36] larsks i know, its either that or we talk about killing bzr again :-P [17:37] larsks, it would be git on launchpad. [17:38] smoser: yeah, that's fine. [17:38] but only for 0.7.x [17:38] and then i think i'd ask the openstack-infra guys to kill the 0.7.x branch on https://github.com/openstack/cloud-init version [17:38] right. [17:38] which hopefully they would do [17:38] the other thing that is needed is the revision number thing... [17:39] Which revision number thing? [17:39] Sorry was afk getting food, reading backlog [17:39] harlowja, had something there. but basically i quite often make use of the bzr revision number for things (snapshots) and want some equivalent [17:39] i forget what it is... theres a apython module that gives snapsthos numbers. [17:39] there's a few [17:39] pbr [17:39] a few others [17:40] https://github.com/habnabit/vcversioner (another one) [17:40] and i'm sure a few more [17:41] that wasnt it i dont htink [17:41] Or just use short git commit ids (which are nice because they can be used directly in git commands, etc) [17:41] yeah, but they dont increase. [17:41] ok, I read all that [17:41] True. [17:41] I definitely can't write python tests to save my life, sorry :p [17:42] That would be a great movie. "Do you expect me to write a unit test? No, Mr. Bond; I expect you to die." [17:43] lol [17:49] * smoser writes down not to trusty larsks's movie recommendations [17:49] s/trusty/trust/ [17:49] (fingers type a y if i type trust) [17:49] Yeah, Trusty is old news. [18:10] harlowja: so was that patch a good fit for what landed in trunk, then ? [18:10] minus the debian bits [18:10] * harlowja hasn't looked over to much yet [18:10] lol [18:11] dmsimard i'll get into looking at it more [18:12] no emergencies or anything, I resorted to using dhcp for the time being :( [18:12] k [18:13] Do let me know if you'd like to test anything though, this is something I can help with. [18:13] k [18:13] thx === rangerpb is now known as rangerpbzzzz