| lifeless | JayF: time for multicast, no more local buffering :) | 00:05 |
|---|---|---|
| NobodyCam | JayF: initial (non-working) size http://paste.openstack.org/show/WcmxaveKLblJbOfPtPqN/ | 00:05 |
| devananda | JayF: so that will keep IPA from becoming the default driver we test ironic with | 00:12 |
| devananda | JayF: but it won't stop you guys from running tempest _serially_ | 00:12 |
| jroll | devananda: right, we'll get there | 00:17 |
| jroll | NobodyCam: do not like | 00:17 |
| jroll | NobodyCam: I just built one showing 93mb ramdisk | 00:18 |
| NobodyCam | :) | 00:18 |
| NobodyCam | I have to see what up with the one I build, | 00:19 |
| JayF | DIB IPA images for devstack = awesome. DIB IPA images for production = I'm less convinced :) | 00:19 |
| JayF | although lets get all this stuff working | 00:20 |
| JayF | I want devstack and tests :P <-- jroll get on it | 00:20 |
| * jroll gets on it | 00:21 | |
| * NobodyCam see that is after 5 and steps afk | 00:21 | |
| *** penick has quit IRC | 00:22 | |
| jroll | NobodyCam++ | 00:22 |
| NobodyCam | humm /me may need more cold and flu meds | 00:22 |
| JayF | AMD's bioweapon accidentally infected Ironic developers at Intel HQ, more details at 7 | 00:23 |
| jroll | mikal: I want a rackspace flag that matches your xkcd/devstack tshirt | 00:23 |
| jroll | LOL | 00:23 |
| jroll | NobodyCam: I think I might have this bug too :( | 00:23 |
| *** chuckC has joined #openstack-ironic | 00:42 | |
| Shrews | wow. 112122 still chugging along, eh? | 00:42 |
| jroll | :( | 00:44 |
| jroll | I did watch it, just nothing to do :P | 00:44 |
| Shrews | adam_g: re 109300, yeah, we can do that, but it has a lot more dependencies. i'm fine if you want to go that route, though. | 00:44 |
| Shrews | jroll: lol. thx for watching the pot not boil :) | 00:44 |
| adam_g | Shrews, well, we can have those flags set before the feature flags are merged, they wont break anything | 00:45 |
| Shrews | adam_g: ok. i'll abandon | 00:45 |
| mikal | jroll: that is not a terrible idea | 00:45 |
| mikal | jroll: we've been trying to think of a team flag for a while in fact | 00:45 |
| jroll | mikal: yes pls :) | 00:45 |
| jroll | (I also want a teeth flag, fwiw) | 00:45 |
| JoshNang | /whois mikal | 00:45 |
| JoshNang | lol | 00:46 |
| jroll | stalker! | 00:46 |
| mikal | Heh | 00:46 |
| *** igordcard has quit IRC | 00:47 | |
| jroll | JoshNang: michael still, nova ptl, aussie racker, and grand poobah of blocking ironic patches from reaching nova :P | 00:47 |
| JoshNang | damn leading spaces. but yeah ha i got that. | 00:47 |
| JoshNang | thanks :) | 00:47 |
| *** ellenh has quit IRC | 01:03 | |
| *** mitz has quit IRC | 01:10 | |
| *** mitz has joined #openstack-ironic | 01:11 | |
| *** aswadr has joined #openstack-ironic | 01:28 | |
| Shrews | man, this new check-tempest-dsvm-f20 job is slooooooooooooow | 01:38 |
| *** nosnos has joined #openstack-ironic | 01:42 | |
| *** eguz has joined #openstack-ironic | 01:46 | |
| openstackgerrit | Yongli He proposed a change to openstack/ironic: Rewrite ironic policy to use the new changes of common policy https://review.openstack.org/97731 | 01:46 |
| *** eghobo has quit IRC | 01:50 | |
| Shrews | 112122 is +A'd. should be good by the morning. only took 5hr 25min for the check jobs to run :/ | 01:53 |
| *** rwsu has quit IRC | 01:54 | |
| *** rwsu has joined #openstack-ironic | 02:07 | |
| *** shakamunyi has joined #openstack-ironic | 02:17 | |
| *** Haomeng has quit IRC | 02:34 | |
| *** Haomeng has joined #openstack-ironic | 03:06 | |
| *** pcrews has quit IRC | 03:06 | |
| *** ramineni has joined #openstack-ironic | 03:16 | |
| *** Nisha has joined #openstack-ironic | 03:27 | |
| *** nosnos has quit IRC | 03:39 | |
| *** eghobo has joined #openstack-ironic | 03:40 | |
| *** eghobo has quit IRC | 03:47 | |
| *** rakesh_hs has joined #openstack-ironic | 03:52 | |
| *** eghobo has joined #openstack-ironic | 04:15 | |
| *** nosnos has joined #openstack-ironic | 04:19 | |
| *** pcrews has joined #openstack-ironic | 04:34 | |
| *** jasondotstar has joined #openstack-ironic | 04:38 | |
| *** jasondotstar has quit IRC | 04:43 | |
| *** sabah has joined #openstack-ironic | 04:45 | |
| *** k4n0 has joined #openstack-ironic | 04:47 | |
| *** bmahalakshmi has joined #openstack-ironic | 04:52 | |
| *** Poornima has joined #openstack-ironic | 04:54 | |
| *** jasondotstar has joined #openstack-ironic | 04:56 | |
| *** chenglch has joined #openstack-ironic | 05:06 | |
| *** jasondotstar has quit IRC | 05:19 | |
| *** rakesh_hs has quit IRC | 05:56 | |
| *** bvivek has joined #openstack-ironic | 05:58 | |
| *** mitz has quit IRC | 06:07 | |
| *** mitz has joined #openstack-ironic | 06:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/112210 | 06:10 |
| *** shakamunyi has quit IRC | 06:16 | |
| *** Poornima_ has joined #openstack-ironic | 06:16 | |
| *** Poornima|mtg has joined #openstack-ironic | 06:17 | |
| *** Poornima has quit IRC | 06:18 | |
| *** k4n0 has quit IRC | 06:18 | |
| *** eghobo has quit IRC | 06:19 | |
| *** eghobo has joined #openstack-ironic | 06:20 | |
| *** Poornima_ has quit IRC | 06:21 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 06:27 | |
| *** zigo has quit IRC | 06:28 | |
| *** zigo has joined #openstack-ironic | 06:33 | |
| *** rakesh_hs has joined #openstack-ironic | 06:34 | |
| openstackgerrit | A change was merged to openstack/ironic: Updated from global requirements https://review.openstack.org/111524 | 06:39 |
| *** eguz has joined #openstack-ironic | 07:08 | |
| *** eguz has quit IRC | 07:09 | |
| openstackgerrit | A change was merged to openstack/ironic: Check if boot device is persistent on ipminative https://review.openstack.org/111959 | 07:10 |
| *** eghobo has quit IRC | 07:13 | |
| *** jistr has joined #openstack-ironic | 07:19 | |
| *** f13o_ has joined #openstack-ironic | 07:21 | |
| *** mkerrin has quit IRC | 07:26 | |
| *** mkerrin has joined #openstack-ironic | 07:30 | |
| *** ifarkas has joined #openstack-ironic | 07:41 | |
| *** enikanorov_ has joined #openstack-ironic | 07:56 | |
| openstackgerrit | Rakesh H S proposed a change to openstack/ironic: ipmi double bridging functionality https://review.openstack.org/95775 | 07:57 |
| *** annegentle has quit IRC | 07:58 | |
| *** annegentle has joined #openstack-ironic | 07:58 | |
| *** pcrews has quit IRC | 07:58 | |
| *** keekz has quit IRC | 07:58 | |
| *** Nisha has quit IRC | 07:58 | |
| *** enikanorov__ has quit IRC | 07:59 | |
| *** keekz has joined #openstack-ironic | 07:59 | |
| *** athomas has joined #openstack-ironic | 08:01 | |
| *** f13o_ has quit IRC | 08:02 | |
| openstackgerrit | A change was merged to openstack/ironic: Add charset and engine settings to every table https://review.openstack.org/111402 | 08:10 |
| *** dguerri`afk is now known as dguerri | 08:24 | |
| *** lucasagomes has joined #openstack-ironic | 08:33 | |
| *** MattMan has quit IRC | 08:35 | |
| *** MattMan has joined #openstack-ironic | 08:35 | |
| *** bmahalakshmi has quit IRC | 08:38 | |
| *** bmahalakshmi2 has joined #openstack-ironic | 08:38 | |
| *** annegentle has quit IRC | 08:43 | |
| *** sabah has quit IRC | 08:48 | |
| *** MattMan2 has joined #openstack-ironic | 08:53 | |
| *** MattMan has quit IRC | 08:53 | |
| *** Mikhail_D_wk has quit IRC | 08:53 | |
| *** Mikhail_D_wk has joined #openstack-ironic | 08:53 | |
| *** tteggel_ has quit IRC | 08:53 | |
| *** sbadia has quit IRC | 08:53 | |
| *** Zerpex has quit IRC | 08:53 | |
| *** annegentle has joined #openstack-ironic | 08:53 | |
| *** Zerpex has joined #openstack-ironic | 08:54 | |
| lucasagomes | lifeless, ping re -2, can you please look at the current version of https://review.openstack.org/104738 and remove the -2 if you think it's fine? thanks | 08:54 |
| *** sbadia has joined #openstack-ironic | 08:54 | |
| *** tteggel has joined #openstack-ironic | 08:55 | |
| *** chenglch has quit IRC | 08:57 | |
| *** bmahalakshmi2 has quit IRC | 08:58 | |
| *** igordcard has joined #openstack-ironic | 08:59 | |
| *** Poornima|mtg is now known as Poornima | 09:08 | |
| *** foexle has joined #openstack-ironic | 09:10 | |
| *** rameshg87 has joined #openstack-ironic | 09:15 | |
| *** bmahalakshmi has joined #openstack-ironic | 09:26 | |
| *** bmahalakshmi has quit IRC | 09:30 | |
| *** sabah has joined #openstack-ironic | 09:30 | |
| *** bmahalakshmi has joined #openstack-ironic | 09:31 | |
| *** sunnyzh has joined #openstack-ironic | 09:38 | |
| *** sunnyzh has quit IRC | 09:39 | |
| *** Nisha_ has joined #openstack-ironic | 09:57 | |
| *** Nisha_ has quit IRC | 09:57 | |
| *** Nisha_ has joined #openstack-ironic | 09:57 | |
| *** Nisha_ has quit IRC | 09:58 | |
| *** pelix1 has joined #openstack-ironic | 09:58 | |
| *** Nisha_ has joined #openstack-ironic | 09:58 | |
| *** Nisha has joined #openstack-ironic | 10:01 | |
| *** Nisha_ has quit IRC | 10:02 | |
| *** jistr is now known as jistr|english | 10:19 | |
| openstackgerrit | Imre Farkas proposed a change to openstack/ironic: Add DracDriver and its DracPower module https://review.openstack.org/104850 | 10:23 |
| *** pelix1 is now known as pelix | 10:24 | |
| *** Nisha has quit IRC | 10:26 | |
| *** rakesh_hs has quit IRC | 10:28 | |
| *** rakesh_hs has joined #openstack-ironic | 10:29 | |
| *** enikanorov_ has quit IRC | 10:31 | |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic: Move code to cleanup ImageCache to a common place https://review.openstack.org/110560 | 10:37 |
| *** enikanorov_ has joined #openstack-ironic | 10:39 | |
| *** sabah has quit IRC | 10:44 | |
| *** Alexei_9871 has joined #openstack-ironic | 10:47 | |
| *** annegentle has quit IRC | 10:48 | |
| *** annegentle has joined #openstack-ironic | 10:51 | |
| *** ramineni has quit IRC | 11:03 | |
| *** nosnos has quit IRC | 11:10 | |
| *** lucasagomes is now known as lucas-hungry | 11:36 | |
| openstackgerrit | Imre Farkas proposed a change to openstack/ironic-specs: DRAC vendor passthru for RAID management https://review.openstack.org/107981 | 11:41 |
| *** ifarkas has quit IRC | 11:54 | |
| *** ifarkas has joined #openstack-ironic | 11:57 | |
| *** bmahalakshmi has quit IRC | 12:30 | |
| openstackgerrit | Rakesh H S proposed a change to openstack/ironic: ipmi double bridging functionality https://review.openstack.org/95775 | 12:43 |
| *** Poornima has quit IRC | 12:47 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Fix self.fields on API Port object https://review.openstack.org/112056 | 12:50 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Fix self.fields on API Chassis object https://review.openstack.org/112055 | 12:50 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add create() and destroy() to Chassis object https://review.openstack.org/112290 | 12:50 |
| *** kincl has joined #openstack-ironic | 13:04 | |
| *** Poornima has joined #openstack-ironic | 13:14 | |
| *** jistr|english is now known as jistr | 13:17 | |
| *** jasondotstar has joined #openstack-ironic | 13:34 | |
| *** pcrews has joined #openstack-ironic | 13:38 | |
| *** jasondotstar has quit IRC | 13:40 | |
| openstackgerrit | A change was merged to stackforge/pyghmi: Change to name-only lookups in RAKP https://review.openstack.org/112096 | 13:42 |
| *** jgrimm has joined #openstack-ironic | 13:46 | |
| openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Force non-numeric for compact sensor records https://review.openstack.org/112301 | 13:47 |
| rameshg87 | jroll, hi | 13:47 |
| openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Force non-numeric for compact sensor records https://review.openstack.org/112301 | 13:49 |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/ironic: Use timeutils from one place https://review.openstack.org/112303 | 13:50 |
| *** jasondotstar has joined #openstack-ironic | 13:53 | |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/ironic: Use timeutils from one place https://review.openstack.org/112303 | 13:54 |
| *** rameshg87 has quit IRC | 13:56 | |
| *** rakesh_hs2 has joined #openstack-ironic | 13:57 | |
| *** rakesh_hs has quit IRC | 13:58 | |
| openstackgerrit | A change was merged to stackforge/pyghmi: Handle non-linear and unrecognized linearizations https://review.openstack.org/111731 | 13:58 |
| *** shakamunyi has joined #openstack-ironic | 14:08 | |
| openstackgerrit | A change was merged to openstack/ironic: Use my_ip for neutron URL https://review.openstack.org/104738 | 14:10 |
| openstackgerrit | A change was merged to openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/111629 | 14:10 |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/ironic: Use timeutils from one place https://review.openstack.org/112303 | 14:13 |
| jroll | morning ironic | 14:18 |
| jroll | quiet today | 14:18 |
| lucas-hungry | jroll, yo morning | 14:27 |
| *** lucas-hungry is now known as lucasagomes | 14:28 | |
| * lucasagomes is not hungry anymore | 14:28 | |
| jroll | heya lucasagomes | 14:28 |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/ironic: Use timeutils from one place https://review.openstack.org/112303 | 14:31 |
| NobodyCam | morning Ironic | 14:31 |
| jroll | mornin NobodyCam | 14:32 |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/ironic: Use timeutils from one place https://review.openstack.org/112303 | 14:32 |
| NobodyCam | :) | 14:32 |
| NobodyCam | morning jroll | 14:32 |
| jroll | :) | 14:33 |
| jroll | I think I did catch the portland plague :| | 14:34 |
| NobodyCam | :( dont let it get bad. rest + more rest | 14:35 |
| jroll | indeed | 14:35 |
| NobodyCam | it is a nasty bug, I think even my dogs now have it | 14:35 |
| jroll | I ate all the cold medicine too | 14:36 |
| NobodyCam | :-p | 14:36 |
| NobodyCam | lol that just makes me loopy | 14:36 |
| jroll | I'll be in and out | 14:36 |
| jroll | your dogs?! | 14:36 |
| lucasagomes | morning NobodyCam feeling better? | 14:39 |
| NobodyCam | they are acting sick too. :-p | 14:39 |
| jroll | they're just sad for you :P | 14:39 |
| NobodyCam | :) I hope so | 14:40 |
| lucasagomes | everyone got sick :( | 14:40 |
| NobodyCam | yep :( | 14:41 |
| lucasagomes | hope we not start turining into zombies and all | 14:41 |
| jroll | :( | 14:41 |
| jroll | lol | 14:41 |
| NobodyCam | it prob had somehitng to do with outside being 90 + and inside at intel -10 | 14:41 |
| lucasagomes | lol yeah | 14:42 |
| lucasagomes | it's like going from siberia to sahara in 5 minutes | 14:42 |
| NobodyCam | it really was | 14:42 |
| jroll | hahaha | 14:42 |
| NobodyCam | wounder if the nova folk got hit with it too | 14:43 |
| lucasagomes | I think it was mostly our room | 14:43 |
| lucasagomes | the second room we got | 14:43 |
| NobodyCam | ahh | 14:43 |
| lucasagomes | the first one afair the aircon was grand, but the second gosh | 14:43 |
| jroll | lucasagomes: do you know, what's that redhat disk partitioning thing? | 14:43 |
| NobodyCam | the real price for being so close to the food | 14:43 |
| lucasagomes | jroll, !? no idea... what you mean? like anaconda? | 14:44 |
| jroll | lucasagomes: I think so, thanks! | 14:44 |
| lucasagomes | yeah anaconda is the installer and it does disk partitioning as part of the installation | 14:45 |
| *** Mikhail_D_ltp has quit IRC | 14:45 | |
| NobodyCam | anaconda is the installer no? | 14:45 |
| *** mdorman has joined #openstack-ironic | 14:45 | |
| jroll | oh that's the installer | 14:45 |
| jroll | yep | 14:45 |
| jroll | ha | 14:45 |
| jroll | idk, someone in another channel I'm in asked: "anyone know of a is there a standalone I can feed lvm+disk configuration from a file using some sane format? I know of like debian partman (which I am not sure can be used in standalone)" | 14:46 |
| NobodyCam | brb | 14:46 |
| lucasagomes | jroll, well... yea anaconda is not standalone... but you can describe how to partition the disk in a file when using kickstart | 14:48 |
| lucasagomes | e.g https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/5/html/Installation_Guide/s1-kickstart2-options.html#s2-kickstart2-options-part-examples | 14:48 |
| jroll | nice, and kickstart can do standalone partitioning? | 14:49 |
| lucasagomes | jroll, not that I know :/ | 14:49 |
| jroll | yeah :/ | 14:49 |
| jroll | oh well | 14:50 |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Migration to oslo.utils library https://review.openstack.org/110596 | 14:50 |
| jroll | you'd think out of all of us, we might know something about partitioning disks :P | 14:50 |
| lucasagomes | heh yeah :/ | 14:50 |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Migration to oslo.utils library https://review.openstack.org/110596 | 14:54 |
| jroll | what do y'all think about doing this in juno? :) https://review.openstack.org/#/c/102296/ | 15:01 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add create() and destroy() to Port object https://review.openstack.org/112323 | 15:09 |
| *** killer_prince has quit IRC | 15:12 | |
| * NobodyCam really need to file his expence reports :-p | 15:14 | |
| NobodyCam | s/need/needs/ | 15:14 |
| lucasagomes | jroll, I like that | 15:14 |
| jroll | NobodyCam: thanks for the reminder :P | 15:15 |
| lucasagomes | jroll, I'll +2 but, are we continuing to approve specs at this point? | 15:15 |
| lucasagomes | NobodyCam, ^ | 15:16 |
| jroll | lucasagomes: yeah, I think that cutoff was only for new specs | 15:16 |
| * jroll looks | 15:16 | |
| lucasagomes | ack if so I'm happy in landing that | 15:16 |
| jroll | https://etherpad.openstack.org/p/3sxKH2po1o | 15:16 |
| lucasagomes | don't look like a big chunk of work anyway | 15:16 |
| jroll | sept 4 is the cutoff | 15:16 |
| NobodyCam | yes specs already up can be approved | 15:16 |
| lucasagomes | jroll, but it's going to conflict with iPXE so I really would like to get iPXE merged first :P | 15:16 |
| jroll | lucasagomes: yeah that's fine | 15:17 |
| lucasagomes | jroll, ok approved | 15:17 |
| jroll | oh I need to review that again don't I? | 15:17 |
| jroll | \o/ | 15:17 |
| jroll | thanks | 15:17 |
| lucasagomes | jroll, yup :) I added the http_url config option there | 15:17 |
| lucasagomes | jroll, if you wanna test there's a devstack patch as well you can take a look | 15:17 |
| jroll | oh nice | 15:18 |
| openstackgerrit | A change was merged to openstack/ironic-specs: Support external DHCP providers https://review.openstack.org/102296 | 15:18 |
| NobodyCam | nice | 15:18 |
| lucasagomes | NobodyCam, if you have a time https://review.openstack.org/#/c/90151/ | 15:18 |
| lucasagomes | it has 2 +2 already | 15:19 |
| lucasagomes | and a +1 | 15:19 |
| jroll | hmmm | 15:19 |
| jroll | lucasagomes: https://review.openstack.org/#/c/99318/14..16/ironic/drivers/modules/pxe.py | 15:19 |
| jroll | 1) I don't like the http_url default (since it's missing most of it) | 15:20 |
| NobodyCam | lucasagomes: seems I need to do a quick walkies I will review right after theat | 15:20 |
| NobodyCam | that even | 15:20 |
| lucasagomes | jroll, oh crap... yeah sure | 15:20 |
| jroll | 2) I guess that's it | 15:20 |
| lucasagomes | jroll, maybe we shouldn't have a default right? | 15:20 |
| jroll | well | 15:20 |
| jroll | hmm | 15:21 |
| jroll | maybe http://$my_ip/ | 15:21 |
| jroll | or http://$my_ip:80/ | 15:21 |
| jroll | idk | 15:21 |
| lucasagomes | jroll, that won't be checked until ipxe is enabled | 15:21 |
| lucasagomes | it's hard to come up with a default for that | 15:21 |
| jroll | right | 15:21 |
| jroll | yeah, no default might be fine | 15:21 |
| lucasagomes | yeah maybe http://$myip:80/ but even tho, looks weird | 15:21 |
| lucasagomes | jroll, ack, will fix that | 15:22 |
| jroll | thanks! | 15:22 |
| *** killer_prince has joined #openstack-ironic | 15:22 | |
| jroll | yeah it does look weird, idk | 15:22 |
| *** killer_prince is now known as lazy_prince | 15:22 | |
| romcheg | GheRivero: Ping, are you around? | 15:24 |
| romcheg | Morning NobodyCam, lucasagomes and everyone else! | 15:24 |
| lucasagomes | romcheg, morning buddy! how it's going? | 15:24 |
| jroll | morning romcheg :) | 15:26 |
| romcheg | GheRivero: Ah, I already see that you updated your oslo.utils patch to mention a bug. I was trying to poke you for that. | 15:27 |
| romcheg | Morning jroll! | 15:28 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add iPXE Installation Guide documentation https://review.openstack.org/106809 | 15:28 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add iPXE support for Ironic https://review.openstack.org/99318 | 15:28 |
| romcheg | lucasagomes: Vacation is in one day! How can it be bad? :) | 15:28 |
| *** rakesh_hs2 has quit IRC | 15:29 | |
| GheRivero | romcheg: :) thanks for writting the bug :) | 15:29 |
| *** andreykurilin has joined #openstack-ironic | 15:30 | |
| lucasagomes | romcheg, hah yeah! it can't | 15:30 |
| lucasagomes | romcheg, enjoy it :) going to travel somewhere? | 15:31 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add create() and destroy() to Port object https://review.openstack.org/112323 | 15:31 |
| romcheg | lucasagomes: Yup, a big trip to Spain, Portugal and Holland | 15:31 |
| jroll | lucasagomes: nice, +2 | 15:31 |
| lucasagomes | jroll, o/ cheers | 15:31 |
| lucasagomes | romcheg, ah nice! | 15:31 |
| romcheg | lucasagomes: So I will call you and ask to translate something from Portuguese eventually :-P | 15:32 |
| lucasagomes | romcheg, hah sure, well seriously if needed you can do that no problem | 15:33 |
| romcheg | lucasagomes: Hope I won't need that :) | 15:33 |
| lucasagomes | :D | 15:33 |
| lucasagomes | you also speak spanish so you will be grand | 15:33 |
| lucasagomes | portuguese speakers and spanish speakers can understand each other more or less | 15:33 |
| *** foexle has quit IRC | 15:35 | |
| romcheg | lucasagomes: Ah... my Spanish needs much more practice so it will be much shame and butthurt :) | 15:36 |
| lucasagomes | romcheg, heh | 15:36 |
| romcheg | lucasagomes: I think you can remember your first verbal chat with someone in English :) | 15:37 |
| lucasagomes | romcheg, heh yeah, wasn't great :P | 15:37 |
| lucasagomes | I also need to practice more my spanish, it's very rusty now | 15:37 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 15:43 | |
| *** ifarkas has quit IRC | 15:45 | |
| romcheg | lucasagomes: http://i3.minus.com/ibzcPWyb7fXbJ1.jpg something like this for me but with Spanish | 15:46 |
| lucasagomes | romcheg, hah | 15:46 |
| *** eghobo has joined #openstack-ironic | 15:53 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add create() and destroy() to Port object https://review.openstack.org/112323 | 15:56 |
| JayF | these specs are just a +2+A away from being merged: https://review.openstack.org/#/c/97744/ + https://review.openstack.org/#/c/108445/4 | 16:09 |
| *** romcheg1 has joined #openstack-ironic | 16:09 | |
| *** romcheg1 has left #openstack-ironic | 16:09 | |
| JayF | and given they are on our juno priorities, would be good if we got them in Real Soon Now(tm) | 16:09 |
| *** aswadr has quit IRC | 16:12 | |
| NobodyCam | lucasagomes: 90151 +a'd :) | 16:13 |
| lucasagomes | NobodyCam, yay! | 16:13 |
| lucasagomes | thanks | 16:13 |
| NobodyCam | no no thank you | 16:13 |
| NobodyCam | :) | 16:13 |
| JayF | NobodyCam: don't forget you can +2 +A 97744 as well (you +1'd it) | 16:14 |
| JayF | NobodyCam: together we will take over the world! (or at least write a spec for it) :P | 16:14 |
| NobodyCam | oh ya this is the one we can't reboot | 16:15 |
| NobodyCam | I am fearful that is going to bite us somehow | 16:16 |
| *** Poornima has quit IRC | 16:16 | |
| JayF | NobodyCam: I agree; but I think that means more that we should encourage use of ilo-ipa over ilo-iscsi | 16:17 |
| *** ellenh has joined #openstack-ironic | 16:18 | |
| NobodyCam | JayF: you confident that 10 inutes is a good default? if node is off and say has a 6 minute post time. | 16:18 |
| JayF | It's the default we chose for IPA under similar restrictions iirc | 16:19 |
| jroll | mmmmm | 16:19 |
| jroll | I think we did 15 or 20 | 16:19 |
| JayF | I also think that's something that can be debated more reasonably in the implementation. If that's the only hangup for approving the spec, we should approve the spec and then we can look at the code? | 16:19 |
| Shrews | lucasagomes: let me know if i'm mistaken on 112055 and i'll change my vote | 16:20 |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Migration to oslo.utils library https://review.openstack.org/110596 | 16:20 |
| lucasagomes | Shrews, lemme take a look | 16:20 |
| jroll | JayF, NobodyCam, it's 20 in the glance library or whatever | 16:20 |
| NobodyCam | I think I would like to see 20... JayF I will comment that I I think the default ttl need to be bumped up and that we can do that in the code review part of things | 16:21 |
| lucasagomes | Shrews, so, that's is checking if 'id' is part of the dictionary passed to the create_chassis call | 16:21 |
| JayF | Then I'd be OK with bumping it to 20m, but I think we've beaten this spec up bloody :) | 16:21 |
| NobodyCam | so we can land this thing and start moving forward | 16:21 |
| JoshNang | yeah, we decided 20 when deva brought up he'd seen hardware that took 15 mins to post | 16:21 |
| JayF | in a lot of good ways, actually, but still :) | 16:21 |
| lucasagomes | Shrews, cc_mock.call_args[0][0] will return the dict (I know it looks ugly) | 16:21 |
| lucasagomes | it has to be assertIn | 16:21 |
| lucasagomes | NotIn* | 16:21 |
| Shrews | lucasagomes: ah, ok. i thought it returned a set. i'll change it. thx! | 16:21 |
| *** Mikhail_D_wk has quit IRC | 16:22 | |
| lucasagomes | Shrews, FYI, for nodes: https://github.com/openstack/ironic/blob/master/ironic/tests/api/v1/test_nodes.py#L756 | 16:22 |
| Shrews | lucasagomes: gotcha. thx again | 16:23 |
| lucasagomes | np thank u for the review | 16:23 |
| NobodyCam | anyone have any last words before we land this spec? | 16:23 |
| Shrews | "One small step for man..." | 16:24 |
| Shrews | ??? | 16:24 |
| *** Mikhail_D_ltp has quit IRC | 16:24 | |
| Shrews | probably not what NobodyCam was looking for | 16:25 |
| NobodyCam | lol | 16:25 |
| NobodyCam | but they are in fact words | 16:25 |
| NobodyCam | :) | 16:25 |
| JayF | This is one small step for Ironic, one giant leap for ilokind? | 16:25 |
| NobodyCam | boom | 16:27 |
| *** Alexei_9871 has quit IRC | 16:27 | |
| openstackgerrit | A change was merged to openstack/ironic-specs: iLO Virtual Media iSCSI Deploy Driver https://review.openstack.org/97744 | 16:33 |
| Shrews | woot | 16:33 |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Make DHCP provider pluggable https://review.openstack.org/112351 | 16:36 |
| JayF | NobodyCam: now there's also the dependant spec on ^ that one that is for IPA as well, if we get that one in, I think everything spec-wise except UEFI boot (which is really close as well) we've been wanting for Juno is in | 16:37 |
| JayF | ^ if that isn't true someone should link me to moar specs to review :P | 16:37 |
| *** jistr has quit IRC | 16:37 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 16:39 | |
| ellenh | Hey all, https://review.openstack.org/#/c/112351/ is the patch for the pluggable dhcp spec that just merged — I moved common/neutron.py, but I’m still seeing references to it in ironic/locale/ (it doesn’t break tests though). Am I supposed to edit those files directly, or are they auto-generated somehow? | 16:42 |
| *** Mikhail_D_ltp has quit IRC | 16:45 | |
| lucasagomes | ellenh, no worries about the locale, it will be hanlded by transifex and proposed as a new patch-set by the os bot. E.g https://review.openstack.org/#/c/112210/ | 16:52 |
| openstackgerrit | Ilya Pekelny proposed a change to openstack/ironic: Test migrations with Alembic, using Oslo.db https://review.openstack.org/111984 | 16:52 |
| ellenh | lucasagomes: cool, thanks! | 16:52 |
| lucasagomes | aight folks I will call it a day | 16:59 |
| NobodyCam | JayF: you have a link | 16:59 |
| lucasagomes | have a good night everybody | 16:59 |
| NobodyCam | have a good night lucasagomes | 16:59 |
| lucasagomes | NobodyCam, thanks, hope you get better from the cold/flu | 16:59 |
| NobodyCam | :) I am starting to feel a little better | 17:00 |
| NobodyCam | but thank you | 17:00 |
| lucasagomes | :) | 17:00 |
| *** lucasagomes is now known as lucas-afk | 17:00 | |
| devananda | o/ | 17:00 |
| NobodyCam | good mornign devananda :) | 17:00 |
| *** eguz has joined #openstack-ironic | 17:05 | |
| devananda | found a problem with the ilo virtual media spec | 17:07 |
| devananda | the BP and the spec .rst file names are different | 17:07 |
| NobodyCam | doh | 17:07 |
| devananda | https://blueprints.launchpad.net/ironic/+spec/ironic-ilo-virtualmedia-driver || specs/juno/ironic-ilo-virtualmedia-deploy.rst | 17:07 |
| *** eghobo has quit IRC | 17:08 | |
| devananda | the spec includes a reference to -driver on L6 | 17:08 |
| devananda | so the smaller change would be to just rename the file | 17:08 |
| devananda | wanyen: is this ok with you? ^ If so, I'll do it | 17:08 |
| JayF | I'll look out for that in the future when reviewing specs :x | 17:09 |
| * NobodyCam also makes note to check on that | 17:10 | |
| *** eguz has quit IRC | 17:12 | |
| *** eghobo has joined #openstack-ironic | 17:12 | |
| NobodyCam | JayF: do you have a link to the ipa spec | 17:13 |
| JayF | the ilo-ipa spec? | 17:15 |
| JayF | https://review.openstack.org/#/c/108445 (I added links to the google docs on this one) | 17:16 |
| *** bvivek has quit IRC | 17:16 | |
| JayF | should the spec contain a link back to the blueprint anywhere? | 17:19 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic-specs: Rename ironic-ilo-virtualmedia-deploy -> *-driver https://review.openstack.org/112363 | 17:22 |
| devananda | JayF: yes | 17:22 |
| devananda | JayF: see http://specs.openstack.org/openstack/ironic-specs/specs/template.html | 17:22 |
| devananda | JayF: first line | 17:22 |
| JayF | thanks, I'll bookmark the template | 17:25 |
| NobodyCam | devananda: +2'd 112363 for ya | 17:28 |
| NobodyCam | :-p | 17:28 |
| devananda | thanks | 17:29 |
| JayF | +2+A by me. Thanks Deva, I will look out for that when +2'ing from now on | 17:33 |
| *** linggao has joined #openstack-ironic | 17:38 | |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic-specs: iLO Virtual Media IPA Deploy Driver https://review.openstack.org/108445 | 17:39 |
| JayF | nice, that was quick | 17:40 |
| JayF | ^ he fixed the filename and added a bp link \o/ | 17:40 |
| linggao | folks, do we have any document on how to deploy a bm node with ironic? | 17:41 |
| JayF | What do you mean a 'bm' node? | 17:42 |
| linggao | baremetal node. | 17:42 |
| JayF | baremetal sometimes means nova-baremetal | 17:43 |
| JayF | you just mean provisioning servers as in real servers, not devstack VMs? | 17:44 |
| linggao | right. | 17:44 |
| JayF | okay, I think a doc exists for that generally speaking? | 17:44 |
| JayF | http://docs.openstack.org/developer/ironic/deploy/install-guide.html that looks like it? | 17:45 |
| *** rameshg87 has joined #openstack-ironic | 17:45 | |
| rameshg87 | jroll, hi | 17:45 |
| linggao | with pxe_ipmitool driver, where do you specify pxe_deploy_ramdisk, pxe_deploy_kernel? In the driver_info or in flavor ? | 17:46 |
| JayF | I'm not familiar enough with the pxe driver to give you a certain answer. I think you specify that in driver_info though, but imbw | 17:46 |
| JayF | it certainly seems like a thing that would go in driver_info :) | 17:46 |
| rameshg87 | linggao, you may specify in driver_info | 17:47 |
| *** tatyana has joined #openstack-ironic | 17:47 | |
| linggao | rameshg87, so if it is specified in the flavor, pxe driver will not pick it up? | 17:48 |
| JayF | deploy_ramdisk and kernel shouldn't change between flavors for any ironic driver | 17:50 |
| JayF | it is set per node, in driver_info | 17:50 |
| devananda | linggao: we are moving away from specifying driver-specific things in the nova flavor | 17:50 |
| openstackgerrit | A change was merged to openstack/ironic-specs: Rename ironic-ilo-virtualmedia-deploy -> *-driver https://review.openstack.org/112363 | 17:50 |
| rameshg87 | linggao, looks it will pick up from flavor too | 17:50 |
| rameshg87 | linggao, https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/patcher.py#L102-L105 | 17:50 |
| devananda | linggao: it is still supported for backwards compatibility, but I would encourage you to set those directly on the node.driver_info | 17:50 |
| rameshg87 | linggao, looks like it will update driver_info | 17:51 |
| * devananda notes that we don't actually have a way to indicate this to users | 17:51 | |
| linggao | devananda and rameshg87 thanks for the info. Do we have a document on how to deploy a node wtih ironic? | 17:51 |
| *** tatyana has quit IRC | 17:52 | |
| rameshg87 | devananda, i have a question on this | 17:52 |
| rameshg87 | devananda, https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/patcher.py#L95-L118 | 17:52 |
| rameshg87 | devananda, shouldn't these be part of GenericDriverFields ?? | 17:52 |
| devananda | rameshg87: no - that metadata is related to the particular instance being deployed | 17:54 |
| devananda | rameshg87: i use this litmus test: could this $metadata change when deploying a different instance to this node? yes -> instance_info. no -> driver_info. | 17:55 |
| devananda | linggao: perhaps not :( | 17:56 |
| rameshg87 | devananda, okay | 17:56 |
| devananda | NobodyCam: what happened to the folks who were working on our docs? | 17:57 |
| devananda | I haven't seen anything more on that in a few months, i think | 17:57 |
| Shrews | devananda: when are you on vacation? | 17:58 |
| devananda | Shrews: aug 18 - sep 3 | 17:59 |
| Shrews | devananda: thx. /me makes note | 17:59 |
| jroll | heya rameshg87 | 18:00 |
| linggao | devananda, basically we would like to see how images are created for ironic drivers pxe and ipa. | 18:00 |
| rameshg87 | jroll, hi | 18:00 |
| rameshg87 | jroll, just wanted to talk regarding the comment regarding test case on https://review.openstack.org/#/c/110560/ | 18:01 |
| rameshg87 | jroll, i tried adding the test case, but looks like there is nothing more i can cover than what is already available | 18:01 |
| rameshg87 | jroll, the only problem is it is not part of image_cache | 18:01 |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Make DHCP provider pluggable https://review.openstack.org/112351 | 18:01 |
| rameshg87 | jroll, s/image_cache/test_image_cache.py | 18:02 |
| *** eghobo has quit IRC | 18:02 | |
| rameshg87 | jroll, the current test_pxe.PXEPrivateFetchImagesTestCase covers all the code paths in image_cache.clean_up_caches() | 18:03 |
| jroll | rameshg87: I don't understand why you can't add tests. I understand the pxe test cases cover that code, but there are no tests just for clean_up_caches | 18:03 |
| jroll | rameshg87: for example, if I change the agent driver to use clean_up_caches, I will have to add tests there as well | 18:03 |
| jroll | rameshg87: and at that point we are duplicating code | 18:03 |
| *** eghobo has joined #openstack-ironic | 18:04 | |
| devananda | linggao: I feel bad for pointing at devstack when you asked for documentation, but it's none the less a good example of what you're looking for | 18:04 |
| devananda | linggao: http://git.openstack.org/cgit/openstack-dev/devstack/tree/lib/ironic#n462 | 18:04 |
| *** eguz has joined #openstack-ironic | 18:04 | |
| rameshg87 | jroll, adding more test cases to clean_up_caches() will duplicate all the tests in test_pxe.PXEPrivateFetchImagesTestCase | 18:05 |
| devananda | linggao: short answer is: diskimage-builder/bin/ramdisk-image-create ubuntu deploy-ironic -o /some/path/some-file | 18:05 |
| rameshg87 | jroll, actually those test cases are not testing fetch_image | 18:06 |
| rameshg87 | jroll, they are actually testing the caching mechanism. | 18:06 |
| devananda | linggao: then load some-file.kernel and some-file.initramfs into glance, and associate them with either the flavor or each node's driver_info | 18:06 |
| rameshg87 | jroll, i feel those tests can be moved to test_image_cache then. what do you feel ? | 18:06 |
| * devananda really wonders how it is that we DONT have this documented anywhere :( | 18:06 | |
| rameshg87 | jroll, my main concern i have nothing more to add than what is existing :-( | 18:07 |
| linggao | devananda that's great help. thanks. | 18:07 |
| rameshg87 | jroll, so i guess moving around the code to its proper place is the best thing to do. what's your thought on that ? | 18:07 |
| *** eghobo has quit IRC | 18:08 | |
| jroll | rameshg87: I think we should have both, I think | 18:08 |
| jroll | rameshg87: maybe they could be moved | 18:08 |
| rameshg87 | jroll, but do you feel fetch_images() should be testing the cache cleanup mechanism ? | 18:09 |
| rameshg87 | jroll, fetch_images() i feel should be testing whether cache cleanup is called or not | 18:09 |
| jroll | rameshg87: I think test_image_cache should test the image_cache code, and then test_pxe should test the subclasses of ImageCache | 18:09 |
| jroll | yes, agree | 18:09 |
| jroll | I think there should be both sets of tests | 18:09 |
| *** eghobo has joined #openstack-ironic | 18:09 | |
| jroll | code duplication isn't too horrible in tests :) | 18:09 |
| rameshg87 | jroll, well i am okay with code duplication :-) | 18:10 |
| *** eguz_ has joined #openstack-ironic | 18:10 | |
| jroll | :) | 18:10 |
| rameshg87 | jroll, but in the end if i ask a question what did you gain by adding more tests - i have nothing to say :-) | 18:10 |
| *** eguz has quit IRC | 18:10 | |
| rameshg87 | jroll, because it was already covered :-) | 18:10 |
| jroll | rameshg87: there should be tests for ImageCache and separate tests for its subclasses | 18:11 |
| rameshg87 | jroll, but subclasses don't have any functionality, right ? | 18:12 |
| jroll | true | 18:12 |
| jroll | rameshg87: maybe it is best to just move them | 18:12 |
| jroll | idk, could ask someone else's opinion too | 18:12 |
| rameshg87 | jroll, i think this is looks okay to me | 18:12 |
| *** krtaylor has quit IRC | 18:13 | |
| rameshg87 | jroll, i fetch_image() can just check whether cleanup is called, and then followed by cache.fetch_image() is called | 18:13 |
| *** eghobo has quit IRC | 18:13 | |
| rameshg87 | jroll, and then all the cache related things can goto image_cache | 18:14 |
| jroll | yes +1 | 18:14 |
| rameshg87 | jroll, probably i can put up a patch and then get your comments :-) | 18:14 |
| jroll | awesome :) | 18:14 |
| jroll | thanks rameshg87! | 18:14 |
| rameshg87 | jroll, thanks .. | 18:14 |
| rameshg87 | jroll, devananda, i had one question | 18:15 |
| *** eguz_ has quit IRC | 18:15 | |
| rameshg87 | jroll, image_source is still part of PXEDriverFields in nova ( https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/patcher.py#L95-L96 ) | 18:15 |
| *** eghobo has joined #openstack-ironic | 18:16 | |
| rameshg87 | jroll, how does ipa get what image needs to be deployed on the bm node ? | 18:16 |
| devananda | oh | 18:21 |
| devananda | jroll: that should all be moved | 18:21 |
| devananda | wtf. this isn't inheritence | 18:22 |
| rameshg87 | devananda, jroll, oh because of this: https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/patcher.py#L39 | 18:22 |
| devananda | PXEDriverFields.get_deploy_patch doesn't *ever* call the base class's method | 18:22 |
| devananda | rameshg87: what driver is in use within Ironic should not matter to Nova | 18:23 |
| rameshg87 | devananda, yes | 18:24 |
| devananda | patcher.py should not contain a class called PXEDriverFields | 18:24 |
| devananda | because nova shouldn't know whether the PXE driver is in use or not | 18:24 |
| devananda | ugh | 18:24 |
| devananda | that needs to be reworked, but that code is frozen right now while we're proposing it to nova | 18:25 |
| rameshg87 | devananda, yup. i was just wondering about ilo driver :-) | 18:25 |
| rameshg87 | devananda, because right now ilo driver wouldn't get those fields | 18:25 |
| devananda | rameshg87: exactly :( | 18:25 |
| devananda | lame. the only driver-specific fields are here: https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/patcher.py#L102-L105 | 18:26 |
| openstackgerrit | A change was merged to openstack/ironic: Expose {set,get}_boot_device in the API https://review.openstack.org/90151 | 18:26 |
| devananda | which is there only for backwards-compat. BUT there's no mention of that inline :( | 18:26 |
| rameshg87 | devananda, +1 | 18:27 |
| rameshg87 | devananda, but can something be done by juno timeframe to make the fields available for ilo driver ? | 18:27 |
| devananda | rameshg87: let me see what I can do to fix this. | 18:28 |
| rameshg87 | devananda, thanks | 18:28 |
| devananda | rameshg87: I can't promise exactly when I'll fix it -- landing the nova.virt.ironic driver in Nova is a higher project priority. BUT. This should definitely be fixed by the end of Juno. | 18:29 |
| rameshg87 | devananda, ultimately there should be only one patcher for all the drivers, right ? | 18:29 |
| devananda | correct | 18:30 |
| rameshg87 | devananda, okay | 18:30 |
| devananda | that file is where we'll handle any special cases that come up -- but in principle, Nova-compute's behavior shouldn't vary by driver at all | 18:30 |
| devananda | or we've broken the abstraction layer | 18:31 |
| rameshg87 | devananda, okay | 18:31 |
| *** rameshg87 has quit IRC | 18:32 | |
| *** pelix has quit IRC | 18:39 | |
| devananda | https://bugs.launchpad.net/ironic/+bug/1353631 | 18:41 |
| jroll | devananda, yeah, I don't love that file | 18:47 |
| *** romcheg1 has joined #openstack-ironic | 18:48 | |
| jroll | devananda: we worked the agent driver to use the same fields as the pxe driver | 18:48 |
| jroll | devananda: idk if making all drivers conform to that is reasonable (likely not) but I hate the thought of nova patches to land a driver in ironic | 18:49 |
| romcheg1 | adam_g: Hi, around? | 18:49 |
| romcheg1 | Morning devananda! | 18:49 |
| adam_g | romcheg, hi! | 18:49 |
| devananda | jroll: i hate that thought too | 18:49 |
| devananda | jroll: it essentially makes out-of-tree drivers impossible | 18:49 |
| devananda | which is seriously lame of us | 18:50 |
| romcheg1 | adam_g: So we were about to work on migration tooling together yesterday but I couldn't be here in the evening | 18:50 |
| jroll | indeed | 18:50 |
| devananda | not that I want to encourage that. but it should absolutely be possible. | 18:50 |
| devananda | also, it's a simple layer violation. nova shouldn't contain any code specific to an ironic driver | 18:50 |
| adam_g | romcheg, cool. ive got grenade to the point where im ready to make use of the migration scripts.. that is, it creates a nova-bm node inventory and brings up ironic with an empty one. | 18:51 |
| devananda | backwards compat for an early design flaw notwithstanding | 18:51 |
| * devananda foods | 18:51 | |
| romcheg1 | adam_g: So you can just cherry pick https://review.openstack.org/#/c/101920/ and try running it | 18:52 |
| romcheg1 | adam_g: I didn't create client console scripts for migration tools yet because there's no consensus on that. So probably that will be done in a separate patch | 18:53 |
| adam_g | romcheg1, sure. was just about to do that. my question was whether the scripts need to be re-proposed to the ironic tree. it seems there was some discussion at some point last week about that | 18:54 |
| romcheg1 | adam_g: at the midcycle meetup? | 18:54 |
| adam_g | romcheg1, i think? | 18:54 |
| adam_g | devananda, when you are back ^ | 18:54 |
| romcheg1 | adam_g: Idk because I wasn't there | 18:55 |
| jroll | adam_g: yeah, I don't remember the details, but they did decide on which tree they should live in. I think it was the ironic tree | 18:55 |
| romcheg1 | Does it make any difference& | 18:56 |
| jroll | I think the idea was that nova doesn't want to maintain it or have it lying around in their tree | 18:58 |
| romcheg1 | All those long discussions about what tree should a simple script live in make me feel gross | 18:58 |
| devananda | adam_g: yea. i caught a snippet of your conv with sdague yesterday about that | 18:59 |
| adam_g | romcheg1, any hints here? http://paste.openstack.org/show/91173/ | 18:59 |
| devananda | adam_g: his recollection seems to match mine -- move them to ironic's tree, along with a copy of the nova_bm sqla models and any other bits from nova we need | 18:59 |
| devananda | Shrews: oh. I just now got your reference to breaking the gate yesterday. | 19:00 |
| romcheg1 | adam_g: Hmm, it worked, I swear! :) | 19:00 |
| romcheg1 | Let me check on that | 19:00 |
| Shrews | devananda: my confirmation that i suck and need a flogging? | 19:00 |
| romcheg1 | adam_g: Looks like some import-related conflicts raised again :( | 19:01 |
| adam_g | devananda, okay, that discussion must have happened on the monday i missed. | 19:01 |
| adam_g | romcheg1, yeah, oslo.config pain | 19:01 |
| devananda | Shrews: it just means I can't yell at the Nova team any more for ignoring check-tempest-dsvm-ironic-nv FAILURE :p | 19:01 |
| romcheg1 | I had this problem in the past, then it disappeared and now it's here again | 19:02 |
| Shrews | devananda: well, i didn't ignore it (was in the middle of debugging it), but I definitely should have WIP'd it while doing so | 19:02 |
| adam_g | romcheg1, i think it can creep in when imports change in other modules, ie some module that your script pulls in begins using CONF where it didnt before | 19:02 |
| jroll | matty_dubs|gone: more pics for you: http://www.scalescale.com/the-making-of-onmetal/ | 19:02 |
| Shrews | feel free to yell at me, though | 19:03 |
| romcheg1 | adam_g: That's exactly what's happening | 19:03 |
| * devananda throws rice at Shrews instead | 19:03 | |
| jroll | devananda: we didn't ignore that job, the infra team did :P | 19:04 |
| romcheg1 | adam_g: last time I fixed it by copying the Ironic code to Nova Tree | 19:04 |
| romcheg1 | Since we need to have this script in Ironic, I will do the opposite: copy Nova models to our tree | 19:04 |
| adam_g | romcheg1, i wonder if moving the migration script to ironics tree would help? are you okay with the decision? i can put something on gerrit including your scritps and the db models later today, if its late there. | 19:04 |
| adam_g | romcheg1, okay cool | 19:05 |
| romcheg1 | adam_g: I will do that now, it's not too late | 19:05 |
| adam_g | romcheg1, awesome | 19:06 |
| Shrews | jroll: nah, that one is square on me | 19:08 |
| jroll | Shrews: emphasis on the :P | 19:09 |
| *** shakamunyi has quit IRC | 19:17 | |
| JayF | matty_dubs|gone: you were asking about our hardware --> http://www.scalescale.com/the-making-of-onmetal/ | 19:20 |
| jroll | JayF: beat you to it | 19:21 |
| jroll | devananda: ^ might be interested in that too, especially networking things | 19:21 |
| JayF | jroll: d'oh | 19:21 |
| jroll | :D | 19:21 |
| JayF | jroll: you're sick, aren't you supposed to be moving slower? | 19:21 |
| jroll | I moved not at all for a few hours so that I could move faster for a bit :P | 19:23 |
| devananda | jroll: neat! thanks | 19:25 |
| jroll | :) | 19:26 |
| *** Guest27295 is now known as mgagne | 19:31 | |
| *** mgagne has quit IRC | 19:31 | |
| *** mgagne has joined #openstack-ironic | 19:31 | |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Make DHCP provider pluggable https://review.openstack.org/112351 | 19:45 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 19:51 | |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Make DHCP provider pluggable https://review.openstack.org/112351 | 19:54 |
| openstackgerrit | Roman Prykhodchenko proposed a change to openstack/ironic: Migrate Nova BM data to Ironic https://review.openstack.org/112402 | 19:55 |
| romcheg1 | adam_g: ^ | 19:56 |
| romcheg1 | adam_g: I didn't test it myself yet :) | 19:56 |
| romcheg1 | But you can try while I'm launching the env for that | 19:56 |
| openstackgerrit | Syed Ismail Faizan Barmawer proposed a change to openstack/ironic-specs: UEFI support for Ironic deploy drivers https://review.openstack.org/99850 | 20:12 |
| *** faizan has joined #openstack-ironic | 20:26 | |
| devananda | NobodyCam: what happened withthe doc writing folks you worked with several months back? | 20:27 |
| *** Mikhail_D_ltp has quit IRC | 20:27 | |
| NobodyCam | they dropped off the planet | 20:29 |
| devananda | oh | 20:31 |
| devananda | lovely | 20:31 |
| faizan | Hi jroll: | 20:33 |
| devananda | NobodyCam: cause the diagrams seem .... slightly wrong. I think rloo got updated versions from them at some point? they also never claened up the user-guide's poor formatting | 20:33 |
| jroll | faizan: what's up | 20:33 |
| faizan | jroll: I addressed the nits in uefi specs and posted a patch. | 20:33 |
| jroll | faizan: awesome, ty | 20:34 |
| devananda | NobodyCam: so this is a lesson to us: sometimes accepting poor docs, just so that we have some docs, might be worse than accepting no docs. someone needs to fix it up | 20:34 |
| * devananda files a bug | 20:34 | |
| faizan | jroll: Can you please look at it again and give your necessary approval | 20:34 |
| jroll | faizan: will do when I have a moment, I'm a bit under the weather right now | 20:34 |
| faizan | jroll: Sure. Thanks | 20:35 |
| jroll | :) | 20:35 |
| devananda | huh | 20:41 |
| devananda | someone's been working on docs already | 20:41 |
| devananda | https://etherpad.openstack.org/p/Using_Ironic_to_launch_instances_on_real_hardware | 20:43 |
| NobodyCam | ahh | 20:44 |
| NobodyCam | brb | 20:46 |
| *** krtaylor has joined #openstack-ironic | 20:48 | |
| *** faizan has quit IRC | 20:54 | |
| *** eghobo has quit IRC | 20:54 | |
| *** eghobo has joined #openstack-ironic | 20:54 | |
| devananda | just filed https://bugs.launchpad.net/ironic/+bug/1353688 | 20:57 |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Make DHCP provider pluggable https://review.openstack.org/112351 | 21:03 |
| JayF | jroll: thought to significantly lower the size of our image's footprint: instead of putting the filesystem in a tarball, put it in something we can bind mount and not have to ever extract, so it says zipped up the whole time | 21:08 |
| *** linggao has quit IRC | 21:16 | |
| *** athomas has quit IRC | 21:16 | |
| *** romcheg1 has quit IRC | 21:19 | |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Make DHCP provider pluggable https://review.openstack.org/112351 | 21:29 |
| NobodyCam | brb... quick walkies | 21:29 |
| *** romcheg1 has joined #openstack-ironic | 21:34 | |
| *** chuckC has quit IRC | 21:38 | |
| *** lucas-afk has quit IRC | 21:41 | |
| *** wanyen has quit IRC | 21:47 | |
| *** marzif_ has joined #openstack-ironic | 21:52 | |
| *** chuckC has joined #openstack-ironic | 21:55 | |
| *** krtaylor has quit IRC | 22:10 | |
| NobodyCam | bbiaf ... quick run to the vets... to drop off a sample | 22:16 |
| *** f13o_ has joined #openstack-ironic | 22:21 | |
| jroll | JayF: sounds reasonable | 22:26 |
| devananda | adam_g: ping | 22:37 |
| adam_g | devananda, hey | 22:39 |
| devananda | adam_g: hiya! I'm updating the deprecation spec and need a blurb about the grenade test plan | 22:40 |
| devananda | adam_g: what i had clearly no longer applies :) is there something somewhere I can copy/paste? | 22:40 |
| adam_g | devananda, hmm i dont have anything. review #? | 22:41 |
| adam_g | devananda, actually this might be a good time to discuss the status and what i've got so far | 22:41 |
| devananda | adam_g: https://review.openstack.org/#/c/95025/9/specs/juno/deprecate-baremetal-driver.rst | 22:42 |
| devananda | adam_g: L200 | 22:42 |
| *** jgrimm has quit IRC | 22:42 | |
| adam_g | devananda, modifying here https://etherpad.openstack.org/p/ironic_migration | 22:44 |
| adam_g | devananda, okay, yeah, yours looks sane. since we're stuffing the nova_bm manually, we aren't really migrating instances as much as migrating nodes. | 22:47 |
| adam_g | devananda, the grenade test just duplicates a lot of what devstack lib/ironic does but instead of enrolling nodes via ironic API, its migrating them from nova_bm -> ironic using the scripts | 22:48 |
| devananda | right | 22:48 |
| romcheg1 | devananda, adam_g: it's too late here. I'll take a look on those docs in the morning | 22:49 |
| adam_g | devananda, yeah, the confirm ir-conductor stuff can go away | 22:49 |
| adam_g | romcheg1, no problem. ive started poking at your new review. do you mind if i hack on it to get it to support pxe + virtual_power -> pxe_ssh and send you a diff before your AM? | 22:50 |
| romcheg1 | adam_g: no problem at all | 22:50 |
| devananda | fwiw, i've confirmed with nova that they prefer us to put the migration tools inour code base | 22:51 |
| adam_g | romcheg1, cool. working on it. some caveats since a lot of the virtual power driver info comes from nova.conf, but i think it can be worked around | 22:51 |
| romcheg1 | adam_g: Why not add cli options for that, just like I did for the architecture? | 22:51 |
| romcheg1 | parsing a config seems to be too hard | 22:52 |
| adam_g | romcheg1, yeah, there'll be a lot of em. :) may just ConfigParser nova.conf and copy defaults from the nova-bm code | 22:52 |
| adam_g | devananda, what does 'validate data in ironic schema' entail? | 22:53 |
| devananda | adam_g: i just removed that | 22:55 |
| romcheg1 | adam_g: Well, then we need to check whether that code in nova does not change until the deprication. | 22:55 |
| romcheg1 | but it should not be a problem | 22:55 |
| devananda | adam_g: since you're populating nova_bm with real data and then booting an instance, that's "validation" IMO :) | 22:55 |
| adam_g | romcheg1, as i understand it, the attitude is that it should absolutely *not* :0 | 22:55 |
| devananda | adam_g: updated the ehterpad | 22:55 |
| romcheg1 | ok guys, it's 2am here so I have to go | 22:55 |
| devananda | romcheg1: nova core are refusing any changes to nova-baremetal at this point | 22:55 |
| adam_g | devananda, hmm i think the last two points can just be 'run tempest' like you originally had it | 22:56 |
| devananda | romcheg1: thanks! g'night! | 22:56 |
| adam_g | romcheg1, good night :) | 22:56 |
| devananda | adam_g: hm, ok | 22:56 |
| romcheg1 | devananda: I hope so! | 22:56 |
| romcheg1 | Have a good rest of the day | 22:56 |
| romcheg1 | ! | 22:56 |
| adam_g | devananda, thats the built in confirmation that grenade uses post-upgrade, at least | 22:56 |
| *** romcheg1 has quit IRC | 22:56 | |
| devananda | right. makes sense | 22:56 |
| devananda | so then this will also depend on getting all the tempest work landed | 22:57 |
| adam_g | devananda, yeah. | 22:57 |
| adam_g | devananda, adding a couple more things to your block | 22:57 |
| * NobodyCam is almost back | 22:59 | |
| devananda | adam_g: is there one (or a small list of) gerrit topics by which all the open tempest / devstack changes can be found? | 23:00 |
| devananda | rather than the giant list here: https://etherpad.openstack.org/p/IronicCI | 23:00 |
| devananda | so that I can link it in the spec | 23:00 |
| openstackgerrit | Alex Weeks proposed a change to openstack/ironic-python-agent: WIP: accept/verify SSL client certificates for authentication https://review.openstack.org/112433 | 23:03 |
| JayF | aweeks: ^ did you make a spec for that yet | 23:03 |
| adam_g | devananda, let me try to get topics added to everything. | 23:04 |
| devananda | adam_g: I think one topic for "get tempest smoke passing" and another for "grenade migration tests" would be sufficient | 23:04 |
| devananda | thanks | 23:04 |
| *** krtaylor has joined #openstack-ironic | 23:05 | |
| * aweeks dodges bus | 23:07 | |
| devananda | adam_g: given that I'm going to be gone for ~ 3 weeks between now and J3, we should probably find someone else to do the API changes in Nova | 23:09 |
| aweeks | JayF: I'm working on a spec and will submit shortly. Wanted to get the prototyping code that I did locally (to verify that it was even possible) up somewhere so it didn't languish in my local repo | 23:12 |
| * NobodyCam is now back | 23:12 | |
| JayF | aweeks: <3 backing up code to public git repos. Just make sure to add the URL to the BP into your commit message | 23:13 |
| JayF | aweeks: plus there's a post to the list (I can explain in meatspace) about how the spec process changed for Kilo, so the stuff you gotta do is even less | 23:13 |
| *** mdorman has quit IRC | 23:18 | |
| adam_g | devananda, what API changes are required for purposes of the grenade testing? | 23:21 |
| devananda | adam_g: none. but they're required as part of deprecation | 23:21 |
| devananda | adam_g: both upgrade (and its testing) and an api proxy are required for nova to drop baremetal | 23:22 |
| adam_g | devananda, okay. wasn't sure the proxy was part of the upgrade test | 23:22 |
| devananda | as an aside, I can't find any public API docs for that extension | 23:28 |
| devananda | which I think is totally fine. we don't want anyone using it anyway | 23:28 |
| adam_g | devananda, RE: topics, is that just a tag added to the commit msg or having everything squashed into a branch / dependency stack? | 23:30 |
| *** funid has joined #openstack-ironic | 23:34 | |
| devananda | adam_g: neither | 23:34 |
| *** funid has left #openstack-ironic | 23:34 | |
| devananda | -t topic, --topic=topic | 23:34 |
| devananda | Sets the target topic for this change on the gerrit server. If not specified, a bug number from the commit summary will be used. Alternatively, the local branch name will be | 23:34 |
| devananda | used if different from remote branch. | 23:34 |
| *** ellenh has quit IRC | 23:35 | |
| devananda | adam_g: an example of their usefulness: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:ironic-driver,n,z | 23:36 |
| adam_g | devananda, oh, groovy. thanks | 23:36 |
| adam_g | devananda, https://review.openstack.org/#/q/topic:ironic_grenade,n,z + https://review.openstack.org/#/q/topic:ironic_tempest,n,z | 23:53 |
| adam_g | devananda, theres one more unproposed tempest change required that Shrews is working on ATM | 23:54 |
| devananda | awesome | 23:56 |
| devananda | Shrews: anyone can set the topic name, so pls use the same one ^. thx! | 23:56 |
| *** ellenh has joined #openstack-ironic | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
