| *** Haomeng has joined #openstack-ironic | 00:03 | |
| *** Haomeng|2 has quit IRC | 00:06 | |
| Haomeng | mrda: :) | 00:12 |
|---|---|---|
| *** achanda has quit IRC | 00:22 | |
| *** overlayer has quit IRC | 00:36 | |
| *** kkoski has joined #openstack-ironic | 00:36 | |
| *** achanda has joined #openstack-ironic | 00:53 | |
| *** kkoski has quit IRC | 01:10 | |
| *** kkoski has joined #openstack-ironic | 01:11 | |
| *** kkoski has quit IRC | 01:11 | |
| *** chenglch has joined #openstack-ironic | 01:16 | |
| *** oomichi_ has joined #openstack-ironic | 01:44 | |
| *** kkoski has joined #openstack-ironic | 01:49 | |
| *** achanda has quit IRC | 01:57 | |
| *** spandhe has quit IRC | 02:08 | |
| *** mtanino has joined #openstack-ironic | 02:08 | |
| *** chenglch has quit IRC | 02:09 | |
| *** chenglch has joined #openstack-ironic | 02:09 | |
| *** kkoski has quit IRC | 02:36 | |
| *** kkoski has joined #openstack-ironic | 02:36 | |
| *** ramineni has joined #openstack-ironic | 02:37 | |
| *** yuanying has quit IRC | 02:56 | |
| *** achanda has joined #openstack-ironic | 02:58 | |
| *** achanda has quit IRC | 03:03 | |
| *** ramineni has quit IRC | 03:07 | |
| *** yuanying has joined #openstack-ironic | 03:09 | |
| *** spandhe has joined #openstack-ironic | 03:16 | |
| *** ramineni has joined #openstack-ironic | 03:16 | |
| *** coolsvap|afk is now known as coolsvap | 03:24 | |
| *** kkoski has quit IRC | 03:34 | |
| openstackgerrit | Merged openstack/ironic: Hide new fields via single method https://review.openstack.org/166056 | 04:01 |
| *** naohirot has joined #openstack-ironic | 04:04 | |
| *** achanda has joined #openstack-ironic | 04:08 | |
| openstackgerrit | Michael Davies proposed openstack/ironic: New field 'name' not supported in port REST API https://review.openstack.org/163730 | 04:17 |
| *** spandhe has quit IRC | 04:18 | |
| *** sandhya has joined #openstack-ironic | 04:28 | |
| *** achanda has quit IRC | 04:34 | |
| *** Nisha has joined #openstack-ironic | 04:41 | |
| *** Marga_ has quit IRC | 05:01 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: ILO:Check the link status after reset_ilo https://review.openstack.org/166126 | 05:06 |
| *** pcaruana has quit IRC | 05:07 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node https://review.openstack.org/155731 | 05:08 |
| *** achanda has joined #openstack-ironic | 05:08 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: ILO:Check the link status after reset_ilo https://review.openstack.org/166126 | 05:08 |
| *** Marga_ has joined #openstack-ironic | 05:21 | |
| *** achanda has quit IRC | 05:24 | |
| *** achanda has joined #openstack-ironic | 05:26 | |
| *** rameshg87 has joined #openstack-ironic | 05:31 | |
| rameshg87 | good morning ironic | 05:31 |
| mrda | hi rameshg87 | 05:39 |
| rameshg87 | hello mrda | 05:40 |
| *** pradipta has joined #openstack-ironic | 05:40 | |
| *** spandhe has joined #openstack-ironic | 05:50 | |
| *** killer_prince has quit IRC | 05:59 | |
| *** rameshg87 is now known as rameshg87-away | 05:59 | |
| *** killer_prince has joined #openstack-ironic | 06:01 | |
| *** killer_prince is now known as lazy_prince | 06:01 | |
| *** Marga_ has quit IRC | 06:21 | |
| *** mtanino has quit IRC | 06:22 | |
| *** jamielennox is now known as jamielennox|away | 06:28 | |
| *** saripurigopi has joined #openstack-ironic | 06:35 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Fixed nits for secure boot support for iLO Drivers https://review.openstack.org/166704 | 06:37 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: ILO:Check the link status after reset_ilo https://review.openstack.org/166126 | 06:41 |
| *** Marga_ has joined #openstack-ironic | 06:57 | |
| *** pcaruana has joined #openstack-ironic | 06:58 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: ILO:Check the link status after reset_ilo https://review.openstack.org/166126 | 07:00 |
| *** ukalifon1 has joined #openstack-ironic | 07:00 | |
| *** dlpartain has joined #openstack-ironic | 07:10 | |
| *** yog__ has joined #openstack-ironic | 07:11 | |
| rameshg87-away | mrda: are you around ? | 07:26 |
| rameshg87-away | mrda: any reason to still keep that AttributeError code in https://review.openstack.org/#/c/163730/9/ironic/api/controllers/v1/utils.py ? | 07:26 |
| mrda | hey rameshg87-away | 07:26 |
| *** rameshg87-away is now known as rameshg87 | 07:26 | |
| * rameshg87 always forgets to change the nickname :) | 07:26 | |
| rameshg87 | mrda: it has been removed in rloos commit | 07:27 |
| mrda | So when we tried to call allow_logical_name outside a pecan context, it threw an Attribute error | 07:27 |
| rameshg87 | mrda: yeah it can, but do we have any code which does that today ? :) | 07:27 |
| mrda | So I'm not sure that all the callers of allow_logical_name are inside the context | 07:28 |
| mrda | I figure iut doesn't actually hurt, given that it was added by Deva originally because one code path needed it | 07:28 |
| mrda | it might not be needed now | 07:28 |
| rameshg87 | mrda: as far as your patch is concerned (which introduces this function), i don't see anyone calling it outside pecan context | 07:28 |
| mrda | if we want it gone, I can remove it | 07:28 |
| rameshg87 | mrda: i am not sure who introduced it :) | 07:28 |
| rameshg87 | mrda: so if you have some idea, we can check with them too | 07:29 |
| mrda | so we had a failing lest when we introduced logical_name | 07:29 |
| *** Marga_ has quit IRC | 07:29 | |
| mrda | s/lest/test | 07:29 |
| rameshg87 | mrda: unit test ? | 07:29 |
| *** achanda has quit IRC | 07:29 | |
| mrda | and deva kindly found the source of the problem over (my) weekend | 07:29 |
| mrda | rameshg87: ya | 07:29 |
| rameshg87 | mrda: oh okay | 07:29 |
| rameshg87 | mrda: if there is a good reason to keep it, we should keep it :) | 07:29 |
| rameshg87 | mrda: may be we should check with devananda then | 07:30 |
| mrda | it _was_ needed once upon a time | 07:30 |
| mrda | but things might have changed | 07:30 |
| *** pas-ha has joined #openstack-ironic | 07:30 | |
| rameshg87 | mrda: yeah, i can't see anything being done in actual code outside of pecan context | 07:31 |
| mrda | in 13 hours I'll be back online, as will deva. I'll check with him then | 07:31 |
| rameshg87 | mrda: if it's only unit tests, i would think we should mock it - we shouldn't change code to make it work for unit tests (unless it's absolutely necessary) | 07:31 |
| rameshg87 | mrda: but we can check with devananda | 07:31 |
| rameshg87 | mrda: if i get to him before you, i can check with him | 07:32 |
| mrda | actually, it might have been one of the tempest tests | 07:32 |
| * mrda can't remember | 07:32 | |
| rameshg87 | okay | 07:32 |
| mrda | I think it was tempest, or else I would have seen it ahead of checkin, right? | 07:32 |
| rameshg87 | yeah | 07:32 |
| rameshg87 | might be .. | 07:32 |
| rameshg87 | but tempest at max should use ironic api | 07:33 |
| mrda | But | 07:33 |
| rameshg87 | which is always in pecan context | 07:33 |
| rameshg87 | :) | 07:33 |
| mrda | Ruby's fix passed tempest | 07:33 |
| rameshg87 | yes | 07:33 |
| mrda | so that case might not be present anymore | 07:33 |
| rameshg87 | okay | 07:34 |
| mrda | I'll push a new rev | 07:34 |
| rameshg87 | okay | 07:37 |
| mrda | if it fails tempest, we'll need to put it back | 07:42 |
| rameshg87 | yeah okay .. | 07:43 |
| openstackgerrit | Michael Davies proposed openstack/ironic: New field 'name' not supported in port REST API https://review.openstack.org/163730 | 07:43 |
| *** killer_prince has joined #openstack-ironic | 07:45 | |
| *** lazy_prince has quit IRC | 07:46 | |
| *** killer_prince is now known as lazy_prince | 07:46 | |
| *** dtantsur|afk is now known as dtantsur | 07:50 | |
| dtantsur | Morning Ironic | 07:52 |
| *** lazy_prince has quit IRC | 07:52 | |
| *** killer_prince has joined #openstack-ironic | 07:54 | |
| *** killer_prince is now known as lazy_prince | 07:54 | |
| openstackgerrit | Merged stackforge/ironic-discoverd: Fix typos in ironic-discoverd https://review.openstack.org/166538 | 07:57 |
| *** chlong has quit IRC | 07:58 | |
| *** oomichi_ has quit IRC | 08:04 | |
| mrda | Morning dtantsur | 08:05 |
| *** ifarkas has joined #openstack-ironic | 08:06 | |
| dtantsur | hey mrda! :) | 08:06 |
| dtantsur | isn't it late evening for you already? | 08:06 |
| mrda | Just knocking off :) | 08:07 |
| mrda | 6:40pm, so not very late | 08:07 |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Add supporting generate config files https://review.openstack.org/163281 | 08:07 |
| *** mrda is now known as mrda-away | 08:07 | |
| mrda-away | Night Ironic :) | 08:07 |
| dtantsur | g'night then :) | 08:08 |
| *** dlpartain has left #openstack-ironic | 08:18 | |
| *** ndipanov has joined #openstack-ironic | 08:21 | |
| *** absubram has quit IRC | 08:24 | |
| rameshg87 | dtantsur: o/ | 08:25 |
| dtantsur | o/ | 08:25 |
| dtantsur | Eventlet is finally fully compatible with Python 3. What great time to live! | 08:27 |
| *** andreykurilin_ has joined #openstack-ironic | 08:40 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: ILO:Check the link status after reset_ilo https://review.openstack.org/166126 | 08:45 |
| ramineni | rameshg87: ^^ | 08:46 |
| *** dtantsur is now known as dtantsur|brb | 08:48 | |
| rameshg87 | ramineni: done :) | 08:49 |
| ramineni | rameshg87: gr8 , thanks | 08:50 |
| *** yuanying has quit IRC | 08:54 | |
| *** Nisha has quit IRC | 08:57 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-python-agent: Do udevadm settle after running partx in image extn https://review.openstack.org/164931 | 09:01 |
| *** jistr has joined #openstack-ironic | 09:02 | |
| *** rsalevsky has joined #openstack-ironic | 09:09 | |
| *** lucasagomes has joined #openstack-ironic | 09:09 | |
| *** lsmola has joined #openstack-ironic | 09:10 | |
| *** Nisha has joined #openstack-ironic | 09:11 | |
| openstackgerrit | Merged openstack/ironic: Fixed nits for secure boot support for iLO Drivers https://review.openstack.org/166704 | 09:16 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: ILO:Check if iLO is up after reset_ilo https://review.openstack.org/166126 | 09:18 |
| *** mgoddard has joined #openstack-ironic | 09:21 | |
| openstackgerrit | Merged stackforge/proliantutils: ILO:Check if iLO is up after reset_ilo https://review.openstack.org/166126 | 09:24 |
| *** derekh has joined #openstack-ironic | 09:25 | |
| *** andreykurilin_ has quit IRC | 09:26 | |
| *** pradipta has quit IRC | 09:26 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add whole disk image support for iscsi_ilo using agent ramdisk https://review.openstack.org/165847 | 09:34 |
| openstackgerrit | Merged openstack/ironic: fix invalid asserts in tests https://review.openstack.org/166352 | 09:35 |
| openstackgerrit | Merged openstack/ironic: fix invalid asserts in tests https://review.openstack.org/166352 | 09:35 |
| *** kbyrne has joined #openstack-ironic | 09:35 | |
| *** viktors has joined #openstack-ironic | 09:36 | |
| *** naohirot has quit IRC | 09:45 | |
| *** tiagogomes_ has joined #openstack-ironic | 09:49 | |
| *** pelix has joined #openstack-ironic | 09:57 | |
| *** athomas has joined #openstack-ironic | 09:58 | |
| *** athomas has quit IRC | 09:58 | |
| *** athomas has joined #openstack-ironic | 09:59 | |
| *** subscope has joined #openstack-ironic | 10:06 | |
| *** pelix has quit IRC | 10:06 | |
| *** pelix has joined #openstack-ironic | 10:07 | |
| *** Nisha_away has joined #openstack-ironic | 10:20 | |
| *** Nisha has quit IRC | 10:20 | |
| *** Nisha has joined #openstack-ironic | 10:28 | |
| *** Nisha_away has quit IRC | 10:28 | |
| *** vdrok_afk is now known as vdrok | 10:33 | |
| vdrok | morning ironic | 10:33 |
| *** dtantsur|brb is now known as dtantsur | 10:34 | |
| dtantsur | vdrok, morning | 10:34 |
| vdrok | morning dtantsur | 10:34 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node https://review.openstack.org/155731 | 10:35 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: ILO:(PART-1) Test cases for RIS Operations https://review.openstack.org/166779 | 10:52 |
| *** Nisha has quit IRC | 10:57 | |
| *** ramineni has quit IRC | 11:00 | |
| *** sandhya has quit IRC | 11:00 | |
| *** bauwser is now known as bauzas | 11:01 | |
| *** coolsvap is now known as coolsvap|afk | 11:11 | |
| * rameshg87 goes home | 11:21 | |
| *** rameshg87 has left #openstack-ironic | 11:21 | |
| *** Haomeng|2 has joined #openstack-ironic | 11:22 | |
| *** Haomeng has quit IRC | 11:25 | |
| *** jcoufal has joined #openstack-ironic | 11:27 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add local boot with partition images documentation https://review.openstack.org/166788 | 11:46 |
| *** saripurigopi has quit IRC | 11:46 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add local boot with partition images documentation https://review.openstack.org/166788 | 11:49 |
| *** bradjones has quit IRC | 12:05 | |
| *** bradjones has joined #openstack-ironic | 12:07 | |
| *** athomas has quit IRC | 12:20 | |
| *** mjturek1 has joined #openstack-ironic | 12:20 | |
| *** jcoufal has quit IRC | 12:20 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add root device hints documentation https://review.openstack.org/166801 | 12:31 |
| *** lucasagomes is now known as lucas-hungry | 12:31 | |
| *** jcoufal has joined #openstack-ironic | 12:36 | |
| *** trown|outttypeww is now known as trown | 12:36 | |
| *** lazy_prince is now known as killer_prince | 12:45 | |
| *** saripurigopi has joined #openstack-ironic | 12:46 | |
| *** athomas has joined #openstack-ironic | 12:47 | |
| *** kbs has joined #openstack-ironic | 12:52 | |
| *** rameshg87 has joined #openstack-ironic | 12:54 | |
| *** dprince has joined #openstack-ironic | 12:59 | |
| *** Marga_ has joined #openstack-ironic | 12:59 | |
| *** rameshg87 has quit IRC | 12:59 | |
| *** rloo has joined #openstack-ironic | 13:06 | |
| *** mjturek1 has quit IRC | 13:11 | |
| *** kkoski has joined #openstack-ironic | 13:13 | |
| *** coolsvap|afk is now known as coolsvap | 13:25 | |
| *** rameshg87 has joined #openstack-ironic | 13:25 | |
| *** alexpilotti has joined #openstack-ironic | 13:31 | |
| *** mjturek1 has joined #openstack-ironic | 13:33 | |
| *** lucas-hungry is now known as lucasagomes | 13:35 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add root device hints documentation https://review.openstack.org/166801 | 13:39 |
| *** saripurigopi has quit IRC | 13:40 | |
| openstackgerrit | Martin Kletzander proposed openstack/ironic: Fix common misspellings https://review.openstack.org/166827 | 13:40 |
| lucasagomes | rameshg87, Shrews hi there, once you have some time mind taking a look at the comments by dtantsur here https://review.openstack.org/#/c/166201/ ? | 13:45 |
| lucasagomes | would like to know what people think about having that config drive creation on the client | 13:46 |
| lucasagomes | (no rush for that, just look when you guys get some free time) | 13:46 |
| Shrews | lucasagomes: will put it on my list | 13:46 |
| lucasagomes | thakns | 13:46 |
| lucasagomes | thanks* | 13:46 |
| * rameshg87 looks | 13:49 | |
| rameshg87 | lucasagomes: ack. i will take same to go through it completely :) | 13:51 |
| rameshg87 | lucasagomes: will have a look at it today itself | 13:51 |
| lucasagomes | rameshg87, cool, no problem :) take ur time | 13:52 |
| *** dprince has quit IRC | 13:52 | |
| *** dprince has joined #openstack-ironic | 13:52 | |
| *** rsalevsky has quit IRC | 13:58 | |
| rloo | good day Ironic'ers! | 14:02 |
| rloo | rameshg87: question about https://review.openstack.org/#/c/155731/. Why does the milestone in the bug need to be set? | 14:03 |
| rameshg87 | rloo: hello | 14:03 |
| rameshg87 | rloo: i didn't know. since we are in rc, i thought we should get in only approved bugs | 14:03 |
| rameshg87 | rloo: is it that way ? i just wasn't sure | 14:04 |
| lucasagomes | rloo, morning | 14:04 |
| rloo | rameshg87: good question. I don't know. I don't think so. In any case, it is a 'high' priority bug. | 14:04 |
| rloo | hi lucasagomes :) | 14:04 |
| rloo | rameshg87: we should be fixing bugs now | 14:05 |
| *** rsalevsky has joined #openstack-ironic | 14:05 | |
| *** rsalevsky has quit IRC | 14:05 | |
| rameshg87 | rloo: yeah. i agree. so can we get that in ? | 14:05 |
| *** rsalevsky has joined #openstack-ironic | 14:05 | |
| rloo | rameshg87: besides, I think Shivanand wins the prize on the number of rebases in a patch | 14:05 |
| *** coolsvap is now known as coolsvap|afk | 14:05 | |
| rameshg87 | rloo: everything else is fine with except that someone has set milestone as "kilo-rc1" | 14:05 |
| rameshg87 | :) | 14:05 |
| *** Marga_ has quit IRC | 14:05 | |
| rloo | rameshg87: yeah, I think we can get that in. | 14:05 |
| *** saripurigopi has joined #openstack-ironic | 14:06 | |
| rameshg87 | rloo: okay, +2ing | 14:06 |
| rameshg87 | rloo: do you want to have a look at it and +A or is it fine if i do it ? | 14:06 |
| rloo | rameshg87: you mean everything else if fine except that it has not been set to kilo-rc1 milestone? | 14:06 |
| rloo | rameshg87: please +A. I can't do an honest review of that one any more. I've looked at too many rebased versions of it. | 14:07 |
| rameshg87 | :) | 14:07 |
| rameshg87 | okay doing right away ... | 14:07 |
| rloo | oh, rameshg87. is Shivanand in hp too? | 14:07 |
| rameshg87 | rloo: yes. we work in the same team. | 14:07 |
| *** chenglch has quit IRC | 14:08 | |
| rloo | rameshg87: argh. guess i should take a look then. not that I care if you +A it. | 14:08 |
| *** jerryz has joined #openstack-ironic | 14:08 | |
| rameshg87 | rloo: okay. it's all yours. | 14:09 |
| rameshg87 | rloo: just remove my +A :) | 14:09 |
| rameshg87 | rloo: i have +1ed/+2ed it many times. so it seemed fine to me anyway. | 14:09 |
| rameshg87 | :) | 14:09 |
| rloo | rameshg87: thx ;) | 14:09 |
| rameshg87 | anyone still pursuing this spec: https://review.openstack.org/#/c/117040/ ? | 14:11 |
| rameshg87 | i was reading through zoo keeper on the weekend, it seemed like a built-in solution for most of the tasks that we do - stale locks, notifications, etc .. | 14:12 |
| *** zz_jgrimm- is now known as jgrimm | 14:12 | |
| rloo | rameshg87: it was on my 'list' to do for kilo, look at improving our locking situation, etc, and zookeeper was an option. | 14:13 |
| rloo | rameshg87: you could shoot kyle an email asking if they are still interested in pursuing it. | 14:13 |
| rameshg87 | rloo: yeah, i think the J's might know about | 14:14 |
| rameshg87 | rloo: Kyle was from rackspace iirc | 14:14 |
| rameshg87 | jroll: JayF: ^^^ | 14:14 |
| rloo | rameshg87: ah. I am finding it hard to remember who works with/for what company. Guess I should have a cheat sheet ;) | 14:14 |
| rameshg87 | rloo: stackalytics is the cheatsheet (it already has your records too btw :) ) | 14:15 |
| rloo | rameshg87: oh, right! | 14:15 |
| *** absubram has joined #openstack-ironic | 14:17 | |
| BadCub_Hiding | morning everyone | 14:19 |
| *** coolsvap has joined #openstack-ironic | 14:19 | |
| TheJulia | good morning BadCub_Hiding | 14:20 |
| *** BadCub_Hiding is now known as BadCub | 14:20 | |
| BadCub | morning TheJulia | 14:20 |
| *** hj-hp has joined #openstack-ironic | 14:21 | |
| *** mtanino has joined #openstack-ironic | 14:21 | |
| *** hj-hp has quit IRC | 14:24 | |
| NobodyCam | Good Morning Ironicers | 14:24 |
| TheJulia | good morning | 14:24 |
| NobodyCam | morning TheJulia :) | 14:25 |
| rloo | Good morning BadCub, TheJulia, NobodyCam | 14:25 |
| NobodyCam | morning rloo and BadCub :) | 14:27 |
| BadCub | morning rloo and NobodyCam | 14:27 |
| NobodyCam | I'm sure there is dtantsur, lucasagomes, rameshg87, and Shrews :) | 14:27 |
| NobodyCam | too | 14:27 |
| lucasagomes | NobodyCam, BadCub TheJulia good morning! | 14:27 |
| BadCub | mornin lucasagomes | 14:28 |
| rameshg87 | morning NobodyCam :) | 14:28 |
| NobodyCam | :) | 14:28 |
| lucasagomes | TheJulia, hi, re IPA and 64MB I left a comment on that review | 14:28 |
| *** hj-hp has joined #openstack-ironic | 14:28 | |
| lucasagomes | TheJulia, I think that's a bug on IPA, because if not vfat the ramdisk doesn't have a fixed size and can be > 64MB and that will break IPA | 14:28 |
| * lucasagomes opened a bug about it | 14:28 | |
| Shrews | morning NobodyCam, BadCub, rloo, rameshg87, lucasagomes, TheJulia, Santa, Dasher, Dancer, Prancer, Vixen, etc.... | 14:29 |
| dtantsur | morning NobodyCam, rloo, TheJulia, Shrews, BadCub, rameshg87, and everyone else :) | 14:29 |
| NobodyCam | :) hehe | 14:29 |
| rloo | hi dtantsur and Shrews. (And Rudolph!) | 14:29 |
| NobodyCam | oh did I miss jlvillal ? | 14:29 |
| TheJulia | lucasagomes: I've seen references to 64MiB in other contexties, it may just not be codified in a spec anywhere.. :( | 14:29 |
| lucasagomes | TheJulia, oh in our code? or nova? | 14:30 |
| BadCub | mornin Shrews and dtantsur | 14:30 |
| * rameshg87 will be back for the meeting | 14:30 | |
| rameshg87 | good morning all :) | 14:31 |
| *** rameshg87 has quit IRC | 14:31 | |
| lucasagomes | TheJulia, oh I see an old note in iscsi_deploy.py | 14:33 |
| lucasagomes | will remove that | 14:33 |
| TheJulia | lucasagomes: not entirely sure, I think it will require some researching into the history of configdrives, and as such while the hardcoding is not the best, entirely removing it doesn't seem ideal because it would be kind of silly to allow a user to submit a .99TB custom configdrive | 14:33 |
| *** achanda has joined #openstack-ironic | 14:34 | |
| lucasagomes | TheJulia, right, yeah well for the maximum size I think we should respect the iso9660 | 14:35 |
| lucasagomes | but yeah, depends on how nova create it | 14:35 |
| lucasagomes | it doesn't seem to limit the configdrive size for the default iso9660 format. Only vfat | 14:35 |
| * lucasagomes will investigate more | 14:35 | |
| TheJulia | ok, let me know what you find since I am curious, just looking at a different issue right now | 14:36 |
| lucasagomes | TheJulia, cool np | 14:36 |
| *** Marga_ has joined #openstack-ironic | 14:38 | |
| NobodyCam | brb | 14:40 |
| *** achanda has quit IRC | 14:41 | |
| *** pelix has quit IRC | 14:41 | |
| * BadCub needs more coffee | 14:44 | |
| *** bnemec has quit IRC | 14:46 | |
| *** Marga_ has quit IRC | 14:46 | |
| *** Marga_ has joined #openstack-ironic | 14:46 | |
| *** pelix has joined #openstack-ironic | 14:47 | |
| *** bnemec has joined #openstack-ironic | 14:49 | |
| *** r-daneel has joined #openstack-ironic | 14:49 | |
| *** rsalevsky has quit IRC | 14:51 | |
| NobodyCam | does the white board look strange to anyone eles? | 14:57 |
| NobodyCam | like gone | 14:57 |
| dtantsur | ifarkas, hi, could you review https://review.openstack.org/#/c/165852/ ? | 14:58 |
| dtantsur | NobodyCam, oh yes, it does Oo | 14:59 |
| ifarkas | dtantsur, sure! | 14:59 |
| NobodyCam | O.o | 14:59 |
| NobodyCam | humm, I was going to check the sttaus | 14:59 |
| NobodyCam | status* reports | 14:59 |
| rloo | NobodyCam: OMG. what happened to the whiteboard? | 15:00 |
| NobodyCam | ???? | 15:00 |
| rloo | NobodyCam: I didn't realize an etherpad could be deleted | 15:00 |
| dtantsur | NobodyCam, and the whole history seems to be lost... every point looks like that | 15:00 |
| JayF | rloo: jfyi; kylestev isn't working for us anymore. You'd have to ask him directly if he is doing anything, but I'd presume any assigned bugs are up for grabs | 15:01 |
| dtantsur | I mean, there are lots of revisions, but they all are identical to the current one O_o | 15:01 |
| NobodyCam | I think etherpad is broken | 15:01 |
| rloo | dtantsur: yeah, i remember looking at the history of an etherpad before. maybe it was the whiteboard. it seemed to only have history for the past 24 or so hours. | 15:02 |
| NobodyCam | I'm unable to go back to saved versions | 15:02 |
| NobodyCam | ya | 15:02 |
| * NobodyCam pops over to infra | 15:02 | |
| rloo | so here's an etherpad and it still exists: https://etherpad.openstack.org/p/kilo-ironic-making-it-simple | 15:03 |
| rloo | NobodyCam: I am wondering if someone accidentally deleted the whiteboard (or deleted the contents of the whiteboard -- would you get what we see if you did that?) | 15:03 |
| BadCub | that is strange | 15:04 |
| lucasagomes | damn, well there's a historic on etherpads | 15:05 |
| lucasagomes | we can restore it | 15:05 |
| jroll | morning all :) | 15:05 |
| jroll | that seems bad | 15:05 |
| jroll | lucasagomes: the history is gone too :| | 15:05 |
| lucasagomes | ouch | 15:05 |
| lucasagomes | x.x | 15:05 |
| rloo | lucasagomes: can you get history. When dtantsur and I look, we only see history for the past 24 hours or so and it shows the same :-( | 15:05 |
| pshige | jroll: morning | 15:05 |
| jroll | morning pshige :) | 15:06 |
| lucasagomes | I see that sucks :( | 15:06 |
| dtantsur | jroll, morning | 15:06 |
| jroll | \o | 15:06 |
| * dtantsur backups important etherpads | 15:06 | |
| BadCub | FYI: I will be out of the office tomorrow for most of the day | 15:06 |
| jroll | BadCub: you're never in my office anyway! | 15:07 |
| rloo | I just looked at another etherpad, and it has history beyond one day. I think there was something odd with the whiteboard etherpad before this happened. | 15:07 |
| BadCub | jroll: are you sure about that. /me is known for silently stalking spaces LOL | 15:07 |
| * jroll is suddenly scared | 15:08 | |
| BadCub | lol | 15:08 |
| TheJulia | jroll: just give BadCub a cookie, it will all be okay :) | 15:10 |
| jroll | lol | 15:10 |
| jroll | 15:10:00 fungi | jroll: i can probably look at it after my current meeting is over | 15:10 |
| * BadCub like cookies | 15:10 | |
| jroll | ^ wrt whiteboard | 15:10 |
| NobodyCam | yep so it may be gone :( | 15:11 |
| rloo | thx jroll. Should we wait for fungi to confirm. and/or start now, trying to recreate the page? | 15:11 |
| * rloo votes to procrastinate and wait for confirmation of the extinction of the whiteboard. | 15:11 | |
| NobodyCam | I would say wait so we don't make his job harder | 15:12 |
| * BadCub seconds rloo 's motion | 15:12 | |
| rloo | NobodyCam: so ping us if you want help recreating it ;) | 15:12 |
| BadCub | must go assault coffee pot yet again | 15:13 |
| NobodyCam | rloo: ack | 15:13 |
| TheJulia | BadCub: you should post a video of this one of these days, I'm sure it would be amusing :) | 15:13 |
| NobodyCam | lets see what fungi can find | 15:13 |
| jroll | BadCub: stalking, assault, you're extra bad today | 15:14 |
| NobodyCam | lol | 15:14 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: pxe partition image deploy fails in UEFI boot mode https://review.openstack.org/166886 | 15:17 |
| BadCub | jroll: but that is "normal" behavior for me LOL | 15:17 |
| jlvillal | NobodyCam: I just got online. So you didn't miss me :) | 15:17 |
| jroll | lol | 15:18 |
| NobodyCam | hehehe in that case.... Good mornign jlvillal :) | 15:18 |
| BadCub | TheJulia: video? Oh nooooo... We don't do video. My hair is still a mess this early! LOL | 15:18 |
| jlvillal | Good morning to you NobodyCam and the rest of Ironic | 15:18 |
| BadCub | mornin jlvillal | 15:18 |
| pshige | jlvillal: morning | 15:19 |
| jlvillal | So what are the top things to review/work on today? | 15:20 |
| BadCub | jlvillal: I think we are needed to make sure docs are completed and so forth. Still waking up a bit, so forgive vagueness :-) | 15:22 |
| jlvillal | BadCub: I'm only mostly woke up... | 15:23 |
| jlvillal | devananda: adam_g: I noticed that https://bugs.launchpad.net/bugs/1433805 got closed. I thought that was opened to help bring attention to the issue. | 15:23 |
| openstack | Launchpad bug 1433805 in OpenStack Compute (nova) juno "Absence of python-ironicclient in nova requirements.txt making upgrades awkward, python-ironicclient features diffucult" [Undecided,New] | 15:23 |
| BadCub | jlvillal: working on achieving "mostly" stage. only second cup of blessed coffee | 15:23 |
| jlvillal | dtantsur: Not sure if they already discussed https://bugs.launchpad.net/bugs/1433805 with you. | 15:25 |
| openstack | Launchpad bug 1433805 in OpenStack Compute (nova) juno "Absence of python-ironicclient in nova requirements.txt making upgrades awkward, python-ironicclient features diffucult" [Undecided,New] | 15:25 |
| *** stendulker has joined #openstack-ironic | 15:25 | |
| * BadCub goes to look | 15:26 | |
| dtantsur | jlvillal, I closes the Ironic part because I assumed there's nothing to do on Ironic side | 15:26 |
| jlvillal | dtantsur: From what I recall they had opened that on Friday because it is going problems with Nova not picking an exact version of python-ironicclient. From what I recall | 15:26 |
| jroll | sooooooo. https://bugs.launchpad.net/nova/+bug/1174153 is more on my mind. | 15:26 |
| openstack | Launchpad bug 1174153 in OpenStack Security Notes "data from previous tenants accessible with nova baremetal" [High,Fix released] - Assigned to Robert Clark (robert-clark) | 15:26 |
| * jroll talks to dims | 15:26 | |
| jlvillal | dtantsur: Ah, so there were two bugs... | 15:26 |
| dtantsur | jlvillal, note that bugs are very bad way "to help bring attention to the issue". ML would work much much better | 15:26 |
| jlvillal | dtantsur: I know adam_g and devananda were discussing it on Friday. I may be misquoting the reason. | 15:27 |
| dtantsur | jlvillal, bug against Nova is valid, because Nova should act somehow/// | 15:27 |
| *** stendulker has quit IRC | 15:27 | |
| dtantsur | we sometimes disagree on how to use bug tracker :) | 15:27 |
| jlvillal | dtantsur: Thanks | 15:27 |
| BadCub | so we basically closed the ironic bug on this as a dup of the Nova bug from what I can see. | 15:28 |
| jlvillal | BadCub: Okay. Works for me. Thanks | 15:29 |
| *** rsalevsky has joined #openstack-ironic | 15:29 | |
| BadCub | :-) | 15:30 |
| BadCub | I think that is valid to place expectation on Nova to fix from their side | 15:30 |
| openstackgerrit | Merged stackforge/ironic-discoverd: Documentation update https://review.openstack.org/165852 | 15:31 |
| rloo | BadCub: I don't know that the ironic bug is a dup (if we're talking about https://bugs.launchpad.net/bugs/1433805 | 15:32 |
| openstack | Launchpad bug 1433805 in OpenStack Compute (nova) juno "Absence of python-ironicclient in nova requirements.txt making upgrades awkward, python-ironicclient features diffucult" [Undecided,New] | 15:32 |
| rloo | BadCub: I think it's more that there's nothing (or we don't think there is anything) to do on the Ironic side to fix the problem | 15:32 |
| BadCub | rloo: that may be the case. I am looking at both now to see why Dmitry noted it as largely dup | 15:34 |
| *** absubram has quit IRC | 15:35 | |
| rloo | BadCub: oh, I didn't notice dtantsur's comment. Will leave it for you to figure out ;) | 15:36 |
| BadCub | hehehe. Will chat with Dmitry when he comes on as well. But I think I see his rationale here | 15:37 |
| rloo | BadCub: Dmitry should be here, he's dtantsur | 15:38 |
| dtantsur | here I am \o/ | 15:38 |
| dtantsur | :) | 15:38 |
| BadCub | yep. FOrgot we said good morning earlier.. Coffee not kicking in | 15:38 |
| dtantsur | yes, the idea was that we have https://bugs.launchpad.net/ironic/+bug/1433727 for upgrade problem | 15:39 |
| openstack | Launchpad bug 1433727 in Ironic "partial upgrade not possible, 'reason': u'Unknown argument: "configdrive" (HTTP 400)'" [High,Confirmed] | 15:39 |
| dtantsur | other than that we can't do anything (IMO), so it's better we keep more detailed bug on Ironic part | 15:40 |
| dtantsur | that's why I closes Ironic part on https://review.openstack.org/165852 | 15:40 |
| BadCub | dtantsur: i more detail is always good! | 15:40 |
| BadCub | especially on bugs that cross projects | 15:41 |
| *** stendulker has joined #openstack-ironic | 15:41 | |
| BadCub | if Nova is not listing the Ironic by not listing the python-ironicclient, the ball really does fall into their court to address for user sanity (imo) | 15:44 |
| * BadCub rereads his statement and realizes he sounds like a crazy cat lady in the first part | 15:46 | |
| jroll | BadCub: jfyi, people in openstack don't like declaring optional dependencies | 15:46 |
| BadCub | jroll: yeah, I have gathered that from the cross project meetings. | 15:47 |
| JayF | What time is the Ironic meeting now? in 13 minutes or 73 minutes? | 15:47 |
| *** stendulker has quit IRC | 15:48 | |
| BadCub | I am still confused on the meeting times with the day light savings jumble too | 15:48 |
| Shrews | 71 min | 15:49 |
| * jroll hmm | 15:49 | |
| jroll | my calendar says 11, my heart says 71 | 15:49 |
| NobodyCam | yep 10 am pst tiem today :) | 15:50 |
| NobodyCam | time even | 15:50 |
| jroll | cool | 15:50 |
| JayF | jroll: /me fixes calendar | 15:50 |
| BadCub | I deleted the schedule from my calendar out of frustration | 15:50 |
| JayF | this means the other meeting is now 10pm, which means I'll likely never be able to attend | 15:50 |
| jroll | JayF: thanks :) | 15:50 |
| jroll | dude, I attended last weeks, it was rough | 15:50 |
| BadCub | yeah, 2200hrs is a bit late for me too | 15:52 |
| *** spandhe has quit IRC | 15:52 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Correct update_dhcp_opts methods https://review.openstack.org/166903 | 16:00 |
| NobodyCam | Nice TheJulia :) | 16:00 |
| *** ijw has joined #openstack-ironic | 16:02 | |
| *** ijw has quit IRC | 16:04 | |
| *** ijw has joined #openstack-ironic | 16:04 | |
| *** yog__ has quit IRC | 16:05 | |
| NobodyCam | jroll: lucasagomes: rloo: others: I think we will need more tests like ^^^^^^^^^ | 16:06 |
| *** jcoufal has quit IRC | 16:11 | |
| *** Nisha has joined #openstack-ironic | 16:11 | |
| jlvillal | BadCub: JayF: You can setup Google Calendar and Outlook to use UTC for appointments. There was a little work involved. I did it after the time switch. | 16:15 |
| jlvillal | Happy to explain what I did if any interest. | 16:15 |
| BadCub | jlvillal: I just re-entered them with the correct times. With my luck if I switched it would garble up all my internal meetings too /me has a volatile relationship with Outlook | 16:16 |
| NobodyCam | we also try and keep the timaanddate link up to date: http://www.timeanddate.com/worldclock/fixedtime.html?iso=20150323T1700 | 16:16 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: More generic option for tuning adding ports https://review.openstack.org/166908 | 16:17 |
| NobodyCam | which should give local time. | 16:17 |
| *** jcoufal has joined #openstack-ironic | 16:17 | |
| jlvillal | NobodyCam: Yep | 16:17 |
| *** Nisha has quit IRC | 16:17 | |
| *** Nisha has joined #openstack-ironic | 16:18 | |
| NobodyCam | :) | 16:18 |
| *** coolsvap has quit IRC | 16:19 | |
| *** spandhe has joined #openstack-ironic | 16:21 | |
| TheJulia | NobodyCam: Thanks! | 16:22 |
| *** spandhe has quit IRC | 16:22 | |
| *** Nisha has quit IRC | 16:22 | |
| NobodyCam | TheJulia: no no Thank you! | 16:22 |
| NobodyCam | :) | 16:22 |
| *** Nisha has joined #openstack-ironic | 16:22 | |
| BadCub | morning Nisha | 16:23 |
| Nisha | morning BadCub | 16:23 |
| NobodyCam | Hi Nisha :) | 16:23 |
| Nisha | Hi NobodyCam :) | 16:23 |
| rloo | NobodyCam: wrt tests, yes, that's a nice one. (I'm sure I reviewed the patch that broke it. too many things to remember.) | 16:25 |
| *** Marga_ has quit IRC | 16:25 | |
| NobodyCam | rloo: ya. but more tests like that will deff help us not to that again :-p | 16:26 |
| NobodyCam | I added it to the list of summit things to talk about | 16:26 |
| rloo | NobodyCam: as long as we remember to update them eg if a new method is added | 16:26 |
| rloo | NobodyCam: why add to list of summit things? Just open a bug or something, or do it. | 16:27 |
| *** ukalifon1 has quit IRC | 16:27 | |
| NobodyCam | hummm :) (incase its not all done by summit time) | 16:27 |
| *** Nisha_away has joined #openstack-ironic | 16:28 | |
| *** Nisha has quit IRC | 16:28 | |
| *** Nisha_away is now known as Nisha | 16:28 | |
| rloo | NobodyCam: I suppose we could open a bug for each 'thing' that needs the test, they could all be 'wishlist'. I don't know what needs to be talked about? | 16:29 |
| *** absubram has joined #openstack-ironic | 16:29 | |
| NobodyCam | rloo: ya that just seems like over kill. but ya I just didn't want to forgot about it | 16:30 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Correct update_dhcp_opts methods https://review.openstack.org/166903 | 16:32 |
| NobodyCam | meeting in about 1/2 an hour | 16:34 |
| * TheJulia goes and finds food | 16:35 | |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: More generic option for tuning adding ports https://review.openstack.org/166908 | 16:37 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add option to delete ports after introspection https://review.openstack.org/166915 | 16:37 |
| *** Nisha has quit IRC | 16:37 | |
| *** Nisha has joined #openstack-ironic | 16:38 | |
| sirushti | Hi, could any of the cores have a look at this bug fix, it's got several +1 | 16:39 |
| sirushti | https://review.openstack.org/#/c/164440/ | 16:39 |
| sirushti | and a +2 | 16:39 |
| sirushti | ah, pressed enter too soon there! | 16:40 |
| *** Nisha_away has joined #openstack-ironic | 16:44 | |
| *** Nisha has quit IRC | 16:44 | |
| *** stendulker has joined #openstack-ironic | 16:46 | |
| *** Marga_ has joined #openstack-ironic | 16:47 | |
| *** trown is now known as trown|lunch | 16:48 | |
| *** Nisha_brb has joined #openstack-ironic | 16:50 | |
| *** Nisha_away has quit IRC | 16:50 | |
| *** Nisha_away has joined #openstack-ironic | 16:52 | |
| *** Nisha_brb has quit IRC | 16:52 | |
| NobodyCam | back in 7 minutes | 16:53 |
| *** Nisha_away is now known as Nisha | 16:54 | |
| *** BertieFulton has joined #openstack-ironic | 16:55 | |
| * jlvillal hopes the other people in #openstack-meeting-3 finish in time for our meeting ;) | 16:56 | |
| *** spandhe has joined #openstack-ironic | 16:57 | |
| NobodyCam | hehehe | 16:58 |
| NobodyCam | they are very good about that :) | 16:58 |
| jlvillal | Who is chair for the meeting today? | 17:01 |
| NobodyCam | i got it | 17:01 |
| *** Nisha has quit IRC | 17:02 | |
| *** absubram has quit IRC | 17:02 | |
| *** Nisha has joined #openstack-ironic | 17:02 | |
| *** davide has joined #openstack-ironic | 17:04 | |
| tiagogomes_ | why, I read here that Ironic only works with flat networks https://software.intel.com/en-us/articles/physical-server-provisioning-with-openstack. Does that mean that I can't have an existing DHCP server on the network where I am trying to do the deployment? | 17:05 |
| tiagogomes_ | s/why/hi | 17:05 |
| JayF | that's a bit of a strange article | 17:05 |
| *** Nisha has quit IRC | 17:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/166435 | 17:06 |
| *** Nisha has joined #openstack-ironic | 17:06 | |
| *** dan has quit IRC | 17:07 | |
| tiagogomes_ | perhaps, but the Ironic documentation sets the networks to be flat. Does that mean that my VM with the Ironic service and the baremetal service needs to be on its own network? | 17:07 |
| *** saripurigopi has quit IRC | 17:08 | |
| tiagogomes_ | I am getting a "not boot file found" when Ironic runs the deployment, so I wonder if it is trying to DHCP from the wrong DHCP server | 17:09 |
| JayF | jroll: ^ did your multiple networks stuff land for K? IDR | 17:09 |
| jroll | no | 17:09 |
| jroll | tiagogomes_: just fyi, we're in the middle of our meeting right now, so responses may be slow :) | 17:09 |
| *** hj-hp has quit IRC | 17:10 | |
| tiagogomes_ | jroll, np | 17:10 |
| *** Nisha has quit IRC | 17:11 | |
| *** Nisha has joined #openstack-ironic | 17:11 | |
| *** dan has joined #openstack-ironic | 17:12 | |
| *** ijw has quit IRC | 17:17 | |
| JayF | http://logs.openstack.org/31/164931/5/gate/gate-ironic-python-agent-pep8/e542931/console.html | 17:17 |
| JayF | did someone enable some new hacking rule | 17:17 |
| JayF | that IPA is now in violation of? | 17:17 |
| JayF | i..e H405 | 17:17 |
| JayF | this is re: getting https://review.openstack.org/#/c/164931/ merged | 17:18 |
| *** hj-hp has joined #openstack-ironic | 17:18 | |
| jroll | huh | 17:18 |
| jroll | JayF: I vote disable them with a TODO | 17:18 |
| jroll | and rebase that patch on top | 17:19 |
| JayF | I gotta check it locally to see if I can reproduce the fail | 17:19 |
| jroll | side note: shouldn't this be named test_errors.py? I thought only test_* was picked up. https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/tests/errors.py | 17:19 |
| JoshNang | i thought it was any class that subclassed a test case, functions named test_*. though ++ to renaming them | 17:20 |
| JayF | https://github.com/openstack/ironic-python-agent/tree/master/ironic_python_agent/tests | 17:20 |
| JayF | no other files in there are test_* | 17:20 |
| *** absubram has joined #openstack-ironic | 17:21 | |
| jroll | JoshNang: yeah, no idea | 17:21 |
| jroll | oh | 17:21 |
| jroll | ok | 17:21 |
| JayF | No distributions at all found for Babel>=1.3 (from -r /data/Users/jay/dev/ironic-python-agent/requirements.txt (line 4)) | 17:22 |
| JayF | I'm getting that for both py27 and pep8 | 17:22 |
| JayF | on IPA | 17:22 |
| JayF | when trying tox -r | 17:22 |
| *** Marga_ has quit IRC | 17:22 | |
| jroll | seems bad | 17:23 |
| jroll | are you using regular pypi or devpi or? | 17:23 |
| JayF | my VM dns is hosed | 17:23 |
| JayF | oc | 17:23 |
| jroll | ah ok | 17:23 |
| JayF | *ofc | 17:23 |
| *** derekh has quit IRC | 17:23 | |
| JayF | https://review.openstack.org/#/c/164931/5 once I got it running, pep8 passed locally | 17:26 |
| JayF | so I'm rechecking | 17:26 |
| *** harlowja has joined #openstack-ironic | 17:27 | |
| *** Nisha has quit IRC | 17:28 | |
| *** achanda has joined #openstack-ironic | 17:29 | |
| *** Nisha has joined #openstack-ironic | 17:29 | |
| *** Nisha has quit IRC | 17:32 | |
| *** Nisha_away has joined #openstack-ironic | 17:32 | |
| *** Nisha_away is now known as Nisha | 17:32 | |
| *** hj-hp has quit IRC | 17:36 | |
| *** Nisha has quit IRC | 17:37 | |
| *** Nisha has joined #openstack-ironic | 17:37 | |
| *** Nisha has quit IRC | 17:42 | |
| *** Nisha has joined #openstack-ironic | 17:42 | |
| *** Nisha has quit IRC | 17:47 | |
| *** Nisha has joined #openstack-ironic | 17:48 | |
| *** rfchapman has quit IRC | 17:49 | |
| openstackgerrit | Merged openstack/ironic: Ilo drivers sets capabilities:boot_mode in node https://review.openstack.org/155731 | 17:50 |
| *** Nisha has quit IRC | 17:53 | |
| *** Nisha has joined #openstack-ironic | 17:53 | |
| *** jistr has quit IRC | 17:54 | |
| NobodyCam | stendulker: I don't think we'er going to hit your topic.. will pick it up here in channel if htats ok | 17:56 |
| *** achanda has quit IRC | 17:57 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Correct update_dhcp_opts methods https://review.openstack.org/166903 | 17:58 |
| stendulker | NobodyCam: :) | 17:59 |
| stendulker | NobodyCam: Should be ok | 17:59 |
| NobodyCam | :) ty | 17:59 |
| *** Nisha has quit IRC | 17:59 | |
| *** Nisha has joined #openstack-ironic | 18:00 | |
| jroll | I have to run for a while, sorry :/ | 18:00 |
| adam_g | bb in a few. i moved houses and forgot to bring my coffee maker. need to grab one around the corner | 18:00 |
| JayF | Something there was no time for in the meeting: | 18:01 |
| NobodyCam | adam_g: ieek | 18:01 |
| BadCub | https://etherpad.openstack.org/p/IronicReviewDay is now updated with all the bugs | 18:01 |
| JayF | I'm working on docs for Cleanign | 18:01 |
| NobodyCam | how could you forget the coffee makere | 18:01 |
| JayF | if anyone has any suggestions, please toss them in here: https://etherpad.openstack.org/p/ironic-cleaning-docs | 18:01 |
| NobodyCam | thak you BadCub | 18:01 |
| NobodyCam | stendulker: we ran out of time before we hit your topic | 18:02 |
| *** igordcard has quit IRC | 18:02 | |
| NobodyCam | Grub2 as a alternate bootloader. It can co-exist with elilo (default). | 18:02 |
| stendulker | NobodyCam: no problem | 18:02 |
| stendulker | Have raised a review for the solution wherein grub2 could be a alternate uefi bootloader and can co-exist with elilo. | 18:03 |
| JayF | I can see grub2 support being useful ... for L | 18:03 |
| stendulker | https://review.openstack.org/#/c/166192 | 18:03 |
| JayF | it's a little late for K though | 18:03 |
| stendulker | Also elilo is out of active development | 18:03 |
| NobodyCam | JayF: the issue was the orginal version removed elilo and would break takeover | 18:03 |
| rloo | stendulker: isn't that a new feature? | 18:03 |
| *** vdrok is now known as vdrok_afk | 18:04 | |
| stendulker | rloo: It was part of secure boot for pxe_ilo | 18:04 |
| rloo | NobodyCam: then shouldn't there be a bug associated with that patch? | 18:04 |
| NobodyCam | stendulker: awesome so we can review the changes to that review | 18:04 |
| *** BertieFulton has quit IRC | 18:04 | |
| NobodyCam | rloo: the orgianl never landed to create the bug. it was cought in review | 18:05 |
| rloo | stendulker: so what's the question/discussion you wanted to have about it? | 18:05 |
| *** Nisha has quit IRC | 18:05 | |
| BadCub | was this 154808 for the UEFI Bp? | 18:05 |
| stendulker | rloo, NobodyCam: Just wanted to check if the co-existence of the two bootloaders could be a solution... | 18:05 |
| NobodyCam | stendulker: I have to look at the latest rev of the patch :) | 18:06 |
| *** Nisha has joined #openstack-ironic | 18:06 | |
| stendulker | NobodyCam: ok | 18:06 |
| NobodyCam | oh and I'm late with bbt.... brb | 18:06 |
| *** Nisha has quit IRC | 18:07 | |
| *** rfchapman has joined #openstack-ironic | 18:07 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Correct update_dhcp_opts methods https://review.openstack.org/166903 | 18:07 |
| *** Nisha has joined #openstack-ironic | 18:07 | |
| rloo | stendulker: congrats on https://review.openstack.org/155731 finally merging! | 18:07 |
| stendulker | rloo: Thank you for giving +2 for som many times :) | 18:07 |
| rloo | stendulker: :) | 18:08 |
| *** ChuckC has quit IRC | 18:08 | |
| stendulker | rloo, NobodyCam, JayF: PLease have a look at the grub2 based review, and let me know your thoughts on the same. Will fine tune accordingly, even if it meant for L release | 18:08 |
| *** dan has quit IRC | 18:08 | |
| rloo | stendulker: I'll look in L, or if someone prioritizes it for K (but I don't see the urgency for K) | 18:09 |
| rloo | stendulker: I mean "I'll try to look at it in...". I don't want to make promises ;) | 18:10 |
| stendulker | rloo: Thank you :) | 18:10 |
| *** Nisha has quit IRC | 18:11 | |
| *** Nisha has joined #openstack-ironic | 18:11 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: pxe partition image deploy fails in UEFI boot mode https://review.openstack.org/166886 | 18:11 |
| NobodyCam | stendulker: thank you ... I'll try and see if I can take a look at the direction thats going | 18:12 |
| stendulker | NobodyCam: Thank you :) | 18:12 |
| *** trown|lunch is now known as trown | 18:13 | |
| stendulker | NobodyCam, rloo: Have not based the pxe_ilo secure boot patch on grub2 yet. But have raised grub2 to get the sense of cores, if that is the kind of direction we would want to go. | 18:14 |
| *** Marga_ has joined #openstack-ironic | 18:14 | |
| *** dank_ has joined #openstack-ironic | 18:14 | |
| lucasagomes | folks I will call it a day | 18:14 |
| lucasagomes | have a good night everyone | 18:14 |
| NobodyCam | night lucasagomes | 18:14 |
| NobodyCam | :) | 18:14 |
| NobodyCam | stendulker: ack | 18:14 |
| BadCub | g'night lucasagomes | 18:14 |
| *** igordcard has joined #openstack-ironic | 18:14 | |
| BadCub | brb | 18:14 |
| *** lucasagomes is now known as lucas-dinner | 18:14 | |
| stendulker | thank you all. | 18:15 |
| *** achanda has joined #openstack-ironic | 18:15 | |
| stendulker | logging out. good night all :) | 18:15 |
| *** dtantsur is now known as dtantsur|afk | 18:15 | |
| *** stendulker has quit IRC | 18:15 | |
| *** enikanorov__ has joined #openstack-ironic | 18:16 | |
| *** harlowja has quit IRC | 18:18 | |
| *** enikanorov has quit IRC | 18:19 | |
| *** Nisha has quit IRC | 18:19 | |
| *** Nisha has joined #openstack-ironic | 18:19 | |
| *** achanda has quit IRC | 18:19 | |
| *** achanda has joined #openstack-ironic | 18:20 | |
| *** tiagogomes_ has quit IRC | 18:21 | |
| *** harlowja has joined #openstack-ironic | 18:21 | |
| NobodyCam | night ststendulker | 18:26 |
| NobodyCam | night dtantsur|afk | 18:26 |
| NobodyCam | brb | 18:27 |
| *** achanda has quit IRC | 18:27 | |
| *** Marga_ has quit IRC | 18:27 | |
| *** achanda has joined #openstack-ironic | 18:28 | |
| *** Nisha has quit IRC | 18:28 | |
| *** Nisha has joined #openstack-ironic | 18:28 | |
| *** Nisha has quit IRC | 18:33 | |
| *** thiagop has joined #openstack-ironic | 18:34 | |
| *** ParsectiX has joined #openstack-ironic | 18:37 | |
| *** ParsectiX has quit IRC | 18:39 | |
| *** ParsectiX has joined #openstack-ironic | 18:40 | |
| *** ParsectiX has quit IRC | 18:40 | |
| thiagop | Hello people, quick question: I have some nodes on Ironic and these nodes have correspondent hypervisors on nova, but these hypervisors are showing without memory_mb=0 and vcpus=0 and so on. Does the driver needs to validate the nodes completely for this information to be available on the hipervisor? | 18:45 |
| thiagop | showing with memory_mb=0* | 18:45 |
| JayF | thiagop: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#enrollment | 18:47 |
| JayF | did you update node properties to include cpus/memory/cpu_arch/local_gb as indicated in step 3? | 18:47 |
| thiagop | yep | 18:50 |
| thiagop | but my driver is validating some other stuff | 18:50 |
| thiagop | (that should not be on the node until the actual deployment takes place) | 18:50 |
| *** Marga_ has joined #openstack-ironic | 18:50 | |
| JayF | your driver? | 18:51 |
| thiagop | should I remove this new validation for the driver to validate? | 18:51 |
| JayF | If you're using an out of tree driver not sure there's a lot I can do to help | 18:51 |
| JayF | I'm confused as to what you're meaning | 18:51 |
| thiagop | Yeah, our team is implementing a new deployment driver for HP's OneView | 18:51 |
| JayF | If it's open source somewhere, I would gladly look to try and help find where your issue is | 18:52 |
| JayF | If it's not, then I'd rather focus my time on stuff that is open :/ | 18:52 |
| thiagop | HP requires that our code pass some security checks before go public and ours isn't yet validated, but thank you for your disposal | 18:54 |
| thiagop | :) | 18:55 |
| JayF | :( There are some HP folk in channel, maybe they can help with the code? Good luck | 18:57 |
| NobodyCam | huh | 18:58 |
| NobodyCam | what | 18:58 |
| NobodyCam | who | 18:58 |
| BadCub | who | 18:59 |
| NobodyCam | :) | 18:59 |
| BadCub | lol | 18:59 |
| * NobodyCam reads scroll back | 18:59 | |
| NobodyCam | thiagop: are the nodes deployed? | 19:00 |
| *** raginbajin has quit IRC | 19:04 | |
| *** SpamapS has quit IRC | 19:04 | |
| *** Nick_A has quit IRC | 19:04 | |
| *** SpamapS has joined #openstack-ironic | 19:04 | |
| *** SpamapS has quit IRC | 19:04 | |
| *** SpamapS has joined #openstack-ironic | 19:04 | |
| *** Nick_A has joined #openstack-ironic | 19:04 | |
| thiagop | not yet. nova-compute keep saying that my node hasn't ram (ramfilter says it has memory_mb=0) | 19:04 |
| *** raginbajin has joined #openstack-ironic | 19:04 | |
| *** mikal has quit IRC | 19:04 | |
| *** mikal has joined #openstack-ironic | 19:04 | |
| NobodyCam | thiagop: can you paste a sample of ironic node-show for that node | 19:05 |
| thiagop | for what I saw in the docs, the process of having these hypervisors populated is a nova periodic task, I just wonder if these values are populated only when the power and deploy interfaces are validated | 19:05 |
| * BadCub steps away for a few short minutes | 19:05 | |
| *** pelix has quit IRC | 19:07 | |
| *** jerryz has quit IRC | 19:07 | |
| thiagop | NobodyCam: http://paste.openstack.org/show/195510/ | 19:08 |
| *** Marga_ has quit IRC | 19:08 | |
| * NobodyCam looks | 19:08 | |
| *** Marga_ has joined #openstack-ironic | 19:09 | |
| NobodyCam | thiagop: as I have not seen the one view driver: i will ask what do you get with node-validate | 19:13 |
| NobodyCam | and have you looked at the nova driver? (https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L511-L543 ) | 19:13 |
| NobodyCam | actually this section: https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L202-L281 is prob better | 19:17 |
| NobodyCam | thiagop: do ypu see anything strange in the nova conductor log? | 19:17 |
| NobodyCam | or the ironic conductor log | 19:18 |
| NobodyCam | *nova-compute log! | 19:18 |
| thiagop | NobodyCam: node-validate fails because of a "server_profile_template_uri" property (new) that is set on the flavor and passed only on the deployment process | 19:19 |
| thiagop | right on the mark, NobodyCam | 19:20 |
| thiagop | the _node_resources_unavailable function checks if the power driver is invalid | 19:20 |
| NobodyCam | humm the power interface should not depend on a deployment parameter | 19:23 |
| NobodyCam | can you correct that in the driver? | 19:24 |
| thiagop | yes, I can. Will try now. | 19:24 |
| thiagop | ;) | 19:24 |
| thiagop | Thank you very much | 19:24 |
| NobodyCam | :) | 19:26 |
| NobodyCam | we're if you have more questions | 19:26 |
| *** Marga_ has quit IRC | 19:26 | |
| NobodyCam | brb... quit smoke break | 19:27 |
| NobodyCam | lol | 19:39 |
| * BadCub thinks it be time for food | 19:39 | |
| NobodyCam | quick even | 19:39 |
| *** BadCub is now known as BadCub_Feeding | 19:39 | |
| *** alexpilotti has quit IRC | 19:39 | |
| jlvillal | JayF: Did you ever figure out why the pep8 keeps failing for: https://review.openstack.org/#/c/164931/ ? Locally I tried and everything works fine. Seems like something might be wrong with the Zuul process??? | 19:44 |
| NobodyCam | jlvillal: wow looks like lots of things: http://logs.openstack.org/31/164931/5/check/gate-ironic-python-agent-pep8/1a0efa7/console.html#_2015-03-23_17_41_29_463 | 19:47 |
| *** hyakuhei has joined #openstack-ironic | 19:47 | |
| jlvillal | NobodyCam: I think something is wrong with that Zuul test | 19:47 |
| jlvillal | NobodyCam: tox runs just fine in my environment. And JayF also ran the pep8 test without issue. | 19:48 |
| jlvillal | NobodyCam: And also it has errors for files not touched by that patch. Strange... | 19:48 |
| NobodyCam | jlvillal: ya ??? | 19:49 |
| JayF | jlvillal: just asked about it in -infra | 19:50 |
| JoshNang | in a blank env, pep8 failed just like in zuul for me | 19:50 |
| JayF | in a "blank env" ? | 19:51 |
| JoshNang | freshly downloaded, tox had never been run (my devstack box i just re-setup) | 19:51 |
| JayF | tox -r did not reproduce it :x | 19:51 |
| JayF | wtf | 19:51 |
| *** romcheg has joined #openstack-ironic | 19:52 | |
| trown | Is there more documentation on IPA than the wiki? | 19:53 |
| *** BadCub_Feeding is now known as BadCub | 19:53 | |
| JayF | There are some README files in the repository | 19:53 |
| trown | JayF: I am trying to wrap my head around how the agent deploy driver works. It looks like the main deploy method boots into the agent ramdisk, and then the vendor passthru methods continue the deployment. Is that correct? | 19:57 |
| JayF | On a very high level | 19:58 |
| JayF | ramdisk boots | 19:58 |
| JayF | Ironic says "prepare image" to the ramdisk REST API | 19:58 |
| JayF | IPA (in the ramdisk) downloads the image requested, writes it to disk | 19:58 |
| jlvillal | JayF: JoshNang: I did a 'rm -rf .tox' and then it still passed for me. | 19:58 |
| JoshNang | weird | 19:59 |
| jlvillal | JoshNang: But I'm not using devstack | 19:59 |
| JayF | on each heartbeat, conductor will check /v1/commands in IPA, if prepare_image shows as SUCCESS it'll consider the image done and deploy | 19:59 |
| JayF | that's a very high level overview | 19:59 |
| JoshNang | fwiw, i'm running this on a fresh ubuntu 14.04 box, all i've done is run stack.sh | 19:59 |
| jlvillal | JoshNang: I've just done a 'git clone' on my Fedora 20 box. | 20:00 |
| trown | JayF: that helps thanks | 20:00 |
| jlvillal | JoshNang: Tried on my Ubuntu 14.04 system and it worked also. But I don't use devstack to do the 'tox' check. | 20:02 |
| *** hj-hp has joined #openstack-ironic | 20:02 | |
| JoshNang | i'm not using devstack either | 20:02 |
| JoshNang | it just happened to have checked out a fresh ipa as part of stack.sh | 20:02 |
| jlvillal | JoshNang: You mentioned 'stack.sh' so I thought that is what you meant. | 20:02 |
| JayF | did you guys see the response in -infra? | 20:02 |
| JayF | have we compared pip freeze for a fail vs a pass? | 20:02 |
| JoshNang | JayF: yup | 20:02 |
| JoshNang | yup to seeing it, not to comparing | 20:02 |
| JayF | lets look at it? | 20:03 |
| * JayF going to gist his pip freeze | 20:03 | |
| *** jgrimm is now known as zz_jgrimm | 20:05 | |
| JayF | passing: https://gist.github.com/jayofdoom/cd559fa685736421ceff | 20:06 |
| JayF | hacking: 0.10.1 vs hacking: 0.8.1 | 20:07 |
| *** Marga_ has joined #openstack-ironic | 20:07 | |
| BadCub | Hey folks.. Just follow-up. Spoke with Deva. We are not planning to open specs for L until RC1 is out of our hair | 20:07 |
| JayF | and the new hacking breaks all the things | 20:07 |
| JoshNang | i'm surprised tox -r doesn't install 0.10.1 | 20:08 |
| JayF | so am I | 20:08 |
| JayF | test-requirements.txt:hacking>=0.8.0,<0.9 isn't being respected | 20:09 |
| NobodyCam | humm ironic has: hacking>=0.9.2,<0.10 | 20:10 |
| BadCub | RPC recivers may starve periodic tasks - https://bugs.launchpad.net/ironic/+bug/1308680 (Per Deva: do a documented workaround for K, and move the bug & fix for bug to L) | 20:10 |
| openstack | Launchpad bug 1308680 in Ironic "RPC receivers may starve periodic tasks" [Medium,Triaged] | 20:10 |
| *** ifarkas has quit IRC | 20:10 | |
| JayF | NobodyCam: exactly, working it with fungo | 20:11 |
| JayF | *fungo | 20:11 |
| NobodyCam | :) | 20:11 |
| jroll | JayF: maybe upgrade tox/pbr locally? | 20:11 |
| BadCub | I have updated the reviewday pad with same notes | 20:12 |
| jroll | JayF: those are installed outside the venv I believe | 20:12 |
| JayF | I'm working it in -infra | 20:12 |
| jroll | k | 20:13 |
| *** mrda-away is now known as mrda | 20:14 | |
| *** andreykurilin_ has joined #openstack-ironic | 20:15 | |
| mrda | Morning Ironic | 20:15 |
| mrda | NobodyCam: yes, I was really away :) | 20:15 |
| NobodyCam | :-p | 20:15 |
| NobodyCam | that was a ping from the meeting this moring | 20:15 |
| mrda | yes, I just saw that (but haven't read the minutes yet) | 20:15 |
| *** pas-ha has quit IRC | 20:16 | |
| NobodyCam | :) | 20:16 |
| mrda | Anything in particular that I can help with? | 20:16 |
| NobodyCam | was a good meeting so please do | 20:16 |
| mrda | ok, shall do | 20:16 |
| jlvillal | JayF: JoshNang NobodyCam: Glad you guys figured out the issue :) tox 1.9.1 | 20:16 |
| mrda | I do have an issue right now with database migrations, if anyone here considers themselves familiar with that code | 20:16 |
| jlvillal | Bad news it is breaking the gate :( | 20:16 |
| NobodyCam | we where chatting about returning 400 vs 406 I believe, but please read the minutes to be sure. :-p | 20:17 |
| JoshNang | yeah :( i'm surprised the entire gate isn't falling over | 20:17 |
| JoshNang | (it might be and i haven't noticed) | 20:17 |
| jlvillal | JoshNang: :) | 20:17 |
| jlvillal | JoshNang: They did say sphinx might also be affected by it. | 20:17 |
| NobodyCam | jlvillal: have you checked if this is / will affect our other repos? or just IPa? | 20:18 |
| JayF | I put a note on 164931 as to why it's failing the gate | 20:18 |
| jlvillal | NobodyCam: I would imagine it might affect everyone as JoshNang said. | 20:18 |
| jlvillal | NobodyCam: Seems to be that tox or pip is ignoring the version restriction. | 20:19 |
| JoshNang | here's the bug: https://bitbucket.org/hpk42/tox/issue/230/tox-191-ignores-package-versions-specified | 20:19 |
| mrda | wow | 20:19 |
| jlvillal | JoshNang: Thanks! | 20:19 |
| *** andreykurilin_ has quit IRC | 20:19 | |
| *** andreykurilin__ has joined #openstack-ironic | 20:19 | |
| JayF | It only breaks other things in the gate | 20:20 |
| JayF | if the wrong versions of the deps break them | 20:20 |
| JayF | we could update our tox.ini to ignore E265, H405 E131 and our stuff would pass. | 20:20 |
| *** lucas-dinner has quit IRC | 20:20 | |
| jlvillal | JayF: I would also be concerned with all the wrong version libraries coming in. | 20:21 |
| jlvillal | JayF: I'm assuming that it is also pulling in wrong library version for other libraries. | 20:21 |
| JoshNang | jlvillal: most of our requirements are > or >= some version, so pip will likely install the latest, just like if it was ignoring the version | 20:22 |
| jlvillal | JoshNang: Ah okay. | 20:22 |
| jlvillal | JoshNang: Does sound like they are working on fixing it on the infra side though | 20:23 |
| JayF | Yeah, fungi is going to push the old, working images | 20:23 |
| JayF | then we should be able to recheck | 20:23 |
| JoshNang | sweet | 20:23 |
| NobodyCam | so I'm guessing the WhiteBoard os a lost cause? | 20:26 |
| NobodyCam | s/os/is/ | 20:26 |
| BadCub | that sucks | 20:27 |
| JayF | What happened to the whiteboard?! | 20:27 |
| JayF | it's been replaced with a markdown howto | 20:27 |
| JayF | lol | 20:27 |
| NobodyCam | as has all of the history | 20:27 |
| JayF | wtf | 20:27 |
| mrda | did they upgrade or something? | 20:27 |
| jroll | NobodyCam: did fungi ever get back to you about it? | 20:27 |
| NobodyCam | fungi was looking in to it... jroll no? | 20:28 |
| JayF | IPA gate is broken, should be fixed in a matter of minutes :D | 20:28 |
| JayF | fixed by .... fungi | 20:28 |
| JayF | lol | 20:28 |
| jroll | NobodyCam: ok, I'll check in when he's done with this | 20:28 |
| NobodyCam | i didn;t want to distract him from his current task :-p | 20:28 |
| NobodyCam | jroll: ack :) | 20:28 |
| *** ndipanov has quit IRC | 20:29 | |
| mrda | So, back to database migrations... In test_migrations.py, if you add 'import q; q("I am the walrus")' to any of the _check_* functions, you will see that our migration tests never run. It's because _walk_versions is never invoked directly - it's always mocked for every test case. Does anyone know/remember anything about this? My guess is that this has been broken since the migration tests were | 20:32 |
| mrda | refactored back in July 2014. But I'm still investigating. Does anyone have any idea on this? | 20:32 |
| jroll | :| | 20:33 |
| jroll | I don't, sorry | 20:33 |
| mrda | (this is the reason review 165666 is blocked - it's waiting for a migration test that is kinda impossible right now) | 20:34 |
| mrda | (which is the fix is_hostname_safe function review) | 20:34 |
| *** andreykurilin__ has quit IRC | 20:37 | |
| *** andreykurilin_ has joined #openstack-ironic | 20:37 | |
| mrda | ok, I'll keep plugging away then | 20:39 |
| NobodyCam | mrda: ugg | 20:40 |
| *** hyakuhei has quit IRC | 20:41 | |
| rloo | JayF, JoshNang, jroll, jlvillal: would this patch help (maybe not): https://review.openstack.org/#/c/166436/ | 20:48 |
| JayF | rloo: no, the bug was that tox 0.9.1 wasn't honoring version restrictions | 20:48 |
| rloo | ^^ wrt pep8 stuff. although it is a bug in tox so probably not. | 20:48 |
| JayF | although that being said; I should land that. I'll wait until the bugged patch passes and gets in first though | 20:48 |
| rloo | JayF: ok thx. | 20:49 |
| jroll | JayF: yeah, I just left the same comment :P | 20:49 |
| *** vipuls is now known as vipul | 20:49 | |
| JayF | jroll: also trying to avoid making that patch that's failed the broken gate twice need a rebase | 20:50 |
| jroll | heh | 20:50 |
| jroll | it won't need a rebase, they don't touch the same thing | 20:51 |
| JayF | I just threw a recheck on it | 20:51 |
| JayF | so hopefully it's coming in soon | 20:51 |
| jroll | I'll leave it for you to approve then :) | 20:51 |
| JayF | fungi said wait 30m to be safe, 10m was probably enough | 20:51 |
| JayF | I rechecked at 15ish | 20:51 |
| JayF | lol | 20:51 |
| NobodyCam | brb | 20:53 |
| JayF | woo, pep8 passed on that one test | 20:58 |
| JayF | finally | 20:58 |
| jroll | nice | 20:58 |
| *** jamielennox|away is now known as jamielennox | 21:02 | |
| *** romcheg has quit IRC | 21:05 | |
| *** romcheg has joined #openstack-ironic | 21:05 | |
| openstackgerrit | Clif Houck proposed openstack/ironic: Add support for image caching for the agent driver https://review.openstack.org/161832 | 21:06 |
| *** igordcard_ has joined #openstack-ironic | 21:13 | |
| *** kkoski has quit IRC | 21:14 | |
| *** kkoski has joined #openstack-ironic | 21:14 | |
| *** kkoski has joined #openstack-ironic | 21:15 | |
| *** Marga_ has quit IRC | 21:18 | |
| *** kkoski has quit IRC | 21:18 | |
| *** hyakuhei has joined #openstack-ironic | 21:20 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Support the equivalent of the Juno API https://review.openstack.org/164369 | 21:25 |
| *** kkoski has joined #openstack-ironic | 21:28 | |
| *** kkoski has quit IRC | 21:29 | |
| JoshNang | so i found a race in cleaning, not sure the best way to close it. if cleaning takes any decent amount of time (>novas timeout), it fails because it can't remove the instance uuid, because this line: https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L1016 | 21:32 |
| JayF | Why isn't CLEANING an UPDATE_ALLOWED_STATE ? | 21:33 |
| JoshNang | now it does work if i add CLEANING to the update_allowed_states (nearly instant, yay!). jroll suggested we clean instance_uuid when we clear the instance_info in the conductor, but not sure we can get 3rd patch to land in nova before rc1 | 21:33 |
| JoshNang | JayF: i'm fine with it being one, but there's issues | 21:34 |
| JoshNang | like, if you have a validate hardware clean step, and update a nodes properties before or after..you'd have different results. maybe others? | 21:34 |
| JoshNang | also, not sure if that's reason enough not to do it :P | 21:35 |
| JayF | Why is it Nova's job to remove instance uuid? | 21:35 |
| JayF | Why not just remove it automatically as part of tear down? | 21:35 |
| JayF | Just trying to wrap my brain around the interactions | 21:36 |
| JoshNang | so as it stands, its 'set node to deleting, wait to hear that happened, then remove instance uuid' | 21:36 |
| JayF | why shouldn't it be "set node to deleting, poll to ensure instance_uuid is gone" ? | 21:37 |
| *** kkoski has joined #openstack-ironic | 21:37 | |
| JoshNang | yeah or until provision_state == cleaning/available/cleanfail, which is how it is now | 21:37 |
| JayF | I mean, I think instance_uuid makes sense as a thing to key on for Nova | 21:37 |
| JayF | because cases where Nova thinks it has an instance and Ironic doesn't | 21:38 |
| JayF | or vice versa | 21:38 |
| JayF | are particularly problematic | 21:38 |
| *** kkoski has quit IRC | 21:40 | |
| *** athomas has quit IRC | 21:41 | |
| *** jcoufal has quit IRC | 21:42 | |
| JoshNang | right, but this is only in the deletes, its about equal. i think matching up instance_uuids in the resource tracker is probably a good idea, but a different topic | 21:42 |
| NobodyCam | br | 21:44 |
| NobodyCam | brb | 21:44 |
| *** kkoski has joined #openstack-ironic | 21:44 | |
| *** kkoski has quit IRC | 21:46 | |
| JoshNang | i think i'll add cleaning to update_allowed_states with a note to revist how we do instance_uuid's in L | 21:49 |
| NobodyCam | JoshNang: ++ | 21:50 |
| JoshNang | thanks JayF :) | 21:50 |
| JayF | both of those changes are going through the gate for IPA | 21:50 |
| JoshNang | \o/ | 21:50 |
| JayF | the udevadm settle + the global req bump | 21:50 |
| *** dprince has quit IRC | 21:50 | |
| *** jcoufal has joined #openstack-ironic | 21:54 | |
| *** hyakuhei has quit IRC | 21:55 | |
| NobodyCam | oh the udev settle worked :) nice | 22:01 |
| JayF | NobodyCam: yeah, I need to factor out full disk imaging to use that code as well | 22:02 |
| JayF | NobodyCam: because right now we have partx and partprobe both in use, and I think partx + udevadm settle would potentially be enough | 22:02 |
| JayF | TheJulia: around? got a sec to test a thing? | 22:02 |
| NobodyCam | :) | 22:02 |
| NobodyCam | JayF: kewl | 22:03 |
| *** Marga_ has joined #openstack-ironic | 22:06 | |
| JayF | Is there still an etherpad or lp link with the things we're supposed to be reviewing? | 22:07 |
| NobodyCam | https://etherpad.openstack.org/p/IronicReviewDay | 22:07 |
| BadCub | I updated it with the bugs today | 22:07 |
| BadCub | It still needs to be updated with Code Clean and Docs I believe, unless we typically do not put those on the list (but probably should) | 22:08 |
| TheJulia | JayF: sure, I might not be able to test since I'm in the middle of cleaning up playbooks | 22:08 |
| JayF | TheJulia: just curious if you did partx command+udevadm settle (but not partprobe) in write_image.sh if it resolves your issue | 22:09 |
| *** kkoski has joined #openstack-ironic | 22:09 | |
| JayF | TheJulia: we just landed some code in the partition image section of IPA that uses that pattern and works | 22:09 |
| TheJulia | JayF: I did not try udevadm settle, but I can likely give it a try a little later | 22:10 |
| JayF | TheJulia: if you do, lmk | 22:10 |
| TheJulia | will do | 22:10 |
| TheJulia | It will be after dinner though, summer and I are patiently waiting for noms | 22:11 |
| JayF | Nice, you're still on the east coast then? | 22:11 |
| JayF | TheJulia: it's not a rush, just if you can it'd be awesome | 22:11 |
| * JayF works on other upstream things | 22:11 | |
| NobodyCam | on out this way we have Norms! | 22:11 |
| openstackgerrit | Merged openstack/ironic-python-agent: Do udevadm settle after running partx in image extn https://review.openstack.org/164931 | 22:12 |
| TheJulia | JayF: Yup, in the raleigh area atm | 22:12 |
| jroll | idk, NobodyCam, I eat noms | 22:13 |
| JayF | TheJulia: Next time I'm that way to visit family I expect dinner w/you and Summer :D | 22:13 |
| TheJulia | JayF: still own the house... on a mission to get rid of it this year | 22:13 |
| *** kkoski has quit IRC | 22:13 | |
| *** andreykurilin_ has quit IRC | 22:15 | |
| NobodyCam | JoshNang: are you updating 161474? | 22:15 |
| JayF | BadCub: https://bugs.launchpad.net/ironic/+bug/1432918 is fix committed; I updated the status | 22:16 |
| openstack | Launchpad bug 1432918 in Ironic "pxe deploy with agent ramdisk fails on localboot" [Critical,Fix committed] - Assigned to Ramakrishnan G (rameshg87) | 22:16 |
| BadCub | JayF: awesome! TY :-) | 22:16 |
| JayF | https://bugs.launchpad.net/ironic/+bug/1407820 is attached to https://review.openstack.org/#/c/151268/ which is abandoned | 22:16 |
| openstack | Launchpad bug 1407820 in Ironic "server will not boot from pxe if it does not support command 'chassis bootdev pxe ' with 'persistent' option" [Medium,Triaged] | 22:16 |
| JayF | so if that's important someone probably should pick it up | 22:17 |
| BadCub | Updated the pad | 22:17 |
| NobodyCam | is anyone on bug #1432498 | 22:19 |
| openstack | bug 1432498 in Ironic "Whole disk image feature is broken in UEFI environment for pxe drivers" [High,Confirmed] https://launchpad.net/bugs/1432498 | 22:19 |
| JayF | NobodyCam: there's another fix that depends on that needs +2s | 22:19 |
| JayF | NobodyCam: https://review.openstack.org/#/c/166886/ | 22:20 |
| JayF | 3x+1 but in need of core reviews | 22:20 |
| * jroll will review | 22:21 | |
| JoshNang | NobodyCam: shouldn't need an update | 22:21 |
| openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/166436 | 22:21 |
| JoshNang | the fix will just be in the ironic api | 22:22 |
| NobodyCam | jroll: it adds a = | 22:22 |
| JoshNang | (for now) | 22:22 |
| jroll | NobodyCam: JayF: +2 | 22:22 |
| JayF | one of you +A it, right? | 22:22 |
| NobodyCam | jroll: you wanta +a I have +2 on it twoo | 22:22 |
| JayF | you both +2'd it nobody +a'd it | 22:22 |
| JayF | merge something, dammit! | 22:22 |
| JayF | :P | 22:22 |
| jroll | boom | 22:23 |
| jroll | JayF: sorry, I can't merge anything | 22:23 |
| * NobodyCam hands JayF a claming glass of vodka | 22:23 | |
| * BadCub wonders what a claming glass is | 22:23 | |
| NobodyCam | (large) | 22:23 |
| BadCub | guest vodka or sacred vodka? lol | 22:24 |
| JayF | NobodyCam: you appear to have misjudged how I relax | 22:24 |
| NobodyCam | lol | 22:24 |
| jroll | I thought that was a typo for flaming | 22:24 |
| NobodyCam | lol | 22:24 |
| jroll | or vodka + clamato | 22:24 |
| jroll | in which case, yes pls | 22:24 |
| BadCub | gray goose martini/extra dry/ twist of orange | 22:25 |
| jroll | vodka + clamato + worcestire + A1 + pickle juice + salt + pepper + tabasco + all the veggies. | 22:25 |
| * jroll <3 bloody marys | 22:26 | |
| JayF | I don't think I've had more than a single drink in one sitting since Christmas | 22:26 |
| BadCub | now why would one want to taint good vodka with all those extra things? LOL | 22:26 |
| JayF | when I may or may not have remininised too hard with my best friend back home | 22:26 |
| JayF | lol | 22:26 |
| jroll | BadCub: because breakfast | 22:26 |
| NobodyCam | lol | 22:26 |
| BadCub | jroll: breakfast of champions! | 22:26 |
| jroll | :) | 22:27 |
| jroll | owait there's work people here | 22:27 |
| jroll | I mean.... cereal. yeah, cereal. | 22:27 |
| NobodyCam | JayF: was that all that was needed for #1432498 | 22:27 |
| BadCub | gray goose and OJ works as a good breakfast suppliment | 22:27 |
| JayF | NobodyCam: no | 22:27 |
| NobodyCam | ack didn't think so | 22:28 |
| NobodyCam | :-p | 22:28 |
| JayF | NobodyCam: that bug is basically "once I fix the template, it gets further but is still broken | 22:28 |
| JayF | I had just reviewed the easy change though and wanted it in while people were paying attention | 22:28 |
| BadCub | er..um.. cereal is GOOD.. Yeah...... lol | 22:28 |
| jroll | :P | 22:28 |
| NobodyCam | ahh | 22:29 |
| NobodyCam | lol while the wind has died down here I'm going to see if I can get on the roof and look at a vent pipe. | 22:30 |
| NobodyCam | happy happy joy joy | 22:30 |
| NobodyCam | brb | 22:30 |
| * BadCub watches NobodyCam in a supporting manner with cell phone in hand | 22:31 | |
| * TheJulia gets a good laugh out of the buffer | 22:33 | |
| *** romcheg has quit IRC | 22:37 | |
| BadCub | well, no pictures for FaceBook or calls to 911, so all is well | 22:40 |
| NobodyCam | and /me is still alive unhurt | 22:40 |
| NobodyCam | lol | 22:40 |
| *** jcoufal has quit IRC | 22:40 | |
| *** ijw has joined #openstack-ironic | 22:52 | |
| *** absubram has quit IRC | 22:55 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Allow instance_uuid to be removed during cleaning https://review.openstack.org/167033 | 22:58 |
| JayF | JoshNang: you should add that to the bugs list | 23:00 |
| JayF | like the review thinger for etherpad | 23:00 |
| JoshNang | JayF: kk | 23:00 |
| NobodyCam | anyone booked hotel yet? | 23:01 |
| BadCub | already added | 23:01 |
| JayF | For what? | 23:01 |
| JoshNang | BadCub: you rock | 23:01 |
| JayF | The summit? | 23:01 |
| NobodyCam | ya | 23:01 |
| JayF | not yet | 23:02 |
| JayF | My talk got approved | 23:02 |
| mrda | airbnb here :) | 23:02 |
| BadCub | JoshNang: I added it under Med & Low cuz it was undecided on level :-) | 23:02 |
| JayF | BadCub: this is "lol can't release" level of bug | 23:02 |
| JoshNang | BadCub: totally reasonable | 23:02 |
| JayF | well I guess | 23:02 |
| JayF | lol can't release with cleaning enabled | 23:02 |
| BadCub | eep | 23:02 |
| *** Marga_ has quit IRC | 23:02 | |
| JoshNang | cleaning isn't enabled yet (updating that review right now) | 23:03 |
| BadCub | added that little note to it on pad as well! | 23:03 |
| JoshNang | thanks :) | 23:03 |
| *** Marga_ has joined #openstack-ironic | 23:03 | |
| BadCub | yw :-) | 23:03 |
| jroll | TheJulia: left a note on 166903 | 23:03 |
| NobodyCam | oh JoshNang did you my question? | 23:03 |
| NobodyCam | JoshNang: are you updating 161474? | 23:04 |
| JoshNang | NobodyCam: yeah, the nova patch doesn't need updates for this to work (tested in devstack) | 23:04 |
| * jroll goes away for a while | 23:04 | |
| NobodyCam | JoshNang: the dep is outdated on the review | 23:04 |
| JoshNang | NobodyCam: ahh, i wasn't going to rebase just to rebase. i think deva made a note of not doing that | 23:05 |
| NobodyCam | ahh ok, but it will have to be done before it can land | 23:06 |
| JoshNang | gotcha | 23:07 |
| openstackgerrit | Merged openstack/ironic: Fix common misspellings https://review.openstack.org/166827 | 23:09 |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/166435 | 23:10 |
| JayF | okay 166827 was a damn cool patch | 23:10 |
| TheJulia | heh | 23:11 |
| TheJulia | doh | 23:11 |
| JoshNang | JayF: that is pretty sweet. maybe it should be part of pep8? | 23:12 |
| JayF | heh | 23:12 |
| JayF | that's a long awful mailing list thread, just waiting to happen | 23:12 |
| TheJulia | jroll: pondering, so this might be a newb quesiton, but are tests executed in a shared process/memory with no ordering control? | 23:14 |
| JayF | TheJulia: AIUI you should assume tests in the same test class share state | 23:14 |
| JoshNang | TheJulia: yup! basically | 23:14 |
| TheJulia | wow... that sucks | 23:15 |
| JayF | all software is terrible | 23:15 |
| JayF | some is just slightly less terrible than others | 23:15 |
| TheJulia | BadCub: please send vodka to my house | 23:15 |
| * TheJulia packs up and goes home | 23:15 | |
| openstackgerrit | Merged openstack/ironic: pxe partition image deploy fails in UEFI boot mode https://review.openstack.org/166886 | 23:19 |
| *** yuanying has joined #openstack-ironic | 23:22 | |
| *** ijw_ has joined #openstack-ironic | 23:24 | |
| *** ijw_ has quit IRC | 23:25 | |
| *** ijw_ has joined #openstack-ironic | 23:26 | |
| *** ijw has quit IRC | 23:27 | |
| *** chlong has joined #openstack-ironic | 23:30 | |
| BadCub | you know yoru artwork is safe when you must use a hammer drill and masonry anchors to hang pictures lol | 23:31 |
| *** ijw has joined #openstack-ironic | 23:33 | |
| BadCub | TheJulia: any preference? | 23:33 |
| BadCub | remember I consider kettle One *guest vodka* lol | 23:33 |
| * BadCub heads off to ponder dinner-y things and stuff. | 23:35 | |
| *** mtanino has quit IRC | 23:35 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Enable cleaning by default https://review.openstack.org/165496 | 23:35 |
| BadCub | See everyone Wednesday. Possibly tomorrow for guest appearance(s) | 23:35 |
| JoshNang | have a good night! | 23:36 |
| *** BadCub is now known as BadCub_Hiding | 23:36 | |
| BadCub_Hiding | you too JoshNang ! | 23:36 |
| *** ijw_ has quit IRC | 23:36 | |
| *** Haomeng has joined #openstack-ironic | 23:53 | |
| JayF | JoshNang: https://review.openstack.org/#/c/165496/ is showing merge conflict | 23:55 |
| JayF | JoshNang: I wonder if ^ that needs to be git-dependant on the Ironic patches it depends on? | 23:55 |
| JoshNang | JayF: hmm | 23:55 |
| JayF | re: udevadm settle: | 23:56 |
| JayF | 16:55:25 <marineam> JayF: Even udevadm settle hasn't been enough in my experience. If you call settle right after | 23:56 |
| JayF | closing the device it is possible for udev to get the settle command before it gets the partition | 23:56 |
| JayF | rescan event from the kernel. | 23:56 |
| JoshNang | it wasn't complaining about the ironic driver patch earlier | 23:56 |
| JayF | NobodyCam: ^ | 23:56 |
| JayF | that's from one of the CoreOS devs who works with udev+systemd all the time | 23:56 |
| *** Haomeng|2 has quit IRC | 23:56 | |
| *** korekhov has joined #openstack-ironic | 23:57 | |
| *** ijw_ has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!