| * NobodyCam loves that his irc host's time zone is set so that at 5 pm he gets Day changed to 02 Jul 2014 | 00:01 | |
| jroll | heh | 00:04 |
|---|---|---|
| NobodyCam | what the day doesn't start at 5:00pm | 00:04 |
| NobodyCam | :-p | 00:04 |
| *** chuckC has quit IRC | 00:07 | |
| * NobodyCam calls it a : ψ ︿_____︿_ψ_ ☼ | 00:08 | |
| * devananda packs up for dinner, bbl | 00:11 | |
| *** Penick has quit IRC | 00:13 | |
| NobodyCam | night devananda | 00:14 |
| mrda | Night Deva | 00:22 |
| *** hemna has quit IRC | 00:24 | |
| *** ccrouch has joined #openstack-ironic | 00:50 | |
| openstackgerrit | Dan Prince proposed a change to openstack/ironic: Port iBoot PDU driver from Nova https://review.openstack.org/50977 | 00:52 |
| *** Sam_S has joined #openstack-ironic | 00:54 | |
| *** mitz_ has joined #openstack-ironic | 01:11 | |
| openstackgerrit | Dan Prince proposed a change to openstack/ironic: Port iBoot PDU driver from Nova https://review.openstack.org/50977 | 01:18 |
| *** nosnos has joined #openstack-ironic | 01:49 | |
| *** nosnos has quit IRC | 01:55 | |
| *** nosnos has joined #openstack-ironic | 01:55 | |
| *** hemna has joined #openstack-ironic | 01:56 | |
| *** max_lobur has quit IRC | 01:59 | |
| *** chuckC has joined #openstack-ironic | 02:44 | |
| *** harlowja is now known as harlowja_away | 02:50 | |
| *** ajc_ has joined #openstack-ironic | 02:50 | |
| *** killer_prince is now known as lazy_prince | 02:59 | |
| *** lazy_prince is now known as killer_prince | 03:00 | |
| *** hemna has quit IRC | 03:11 | |
| *** nosnos has quit IRC | 03:27 | |
| *** sabah has joined #openstack-ironic | 03:46 | |
| *** coolsvap|afk is now known as coolsvap | 03:53 | |
| *** sabah has quit IRC | 03:53 | |
| *** Poornima has joined #openstack-ironic | 03:59 | |
| *** pcrews has quit IRC | 04:02 | |
| *** rakesh_hs has joined #openstack-ironic | 04:05 | |
| *** rakesh_hs has quit IRC | 04:16 | |
| *** rakesh_hs has joined #openstack-ironic | 04:17 | |
| *** lazy_prince has joined #openstack-ironic | 04:25 | |
| *** bvivek has joined #openstack-ironic | 04:36 | |
| *** rakesh_hs2 has joined #openstack-ironic | 04:36 | |
| *** rakesh_hs has quit IRC | 04:36 | |
| *** nosnos has joined #openstack-ironic | 04:42 | |
| *** k4n0 has joined #openstack-ironic | 04:42 | |
| *** lazy_prince has quit IRC | 04:46 | |
| *** killer_prince is now known as lazy_prince | 04:47 | |
| *** geekyogi has joined #openstack-ironic | 04:52 | |
| *** geekyogi has quit IRC | 05:05 | |
| *** Nisha has joined #openstack-ironic | 05:06 | |
| *** kpavel has quit IRC | 05:07 | |
| *** amitpp has joined #openstack-ironic | 05:21 | |
| adam_g | Shrews, https://review.openstack.org/104069 you might be interested in this, it overlaps a bunch with your rebuild scenario test. | 05:22 |
| *** rwsu has quit IRC | 05:24 | |
| *** pradipta_away is now known as pradipta | 05:30 | |
| *** eghobo has joined #openstack-ironic | 05:30 | |
| *** romcheg has joined #openstack-ironic | 05:31 | |
| *** sabah has joined #openstack-ironic | 05:32 | |
| *** amitpp has quit IRC | 05:35 | |
| *** amitpp has joined #openstack-ironic | 05:40 | |
| *** pradipta is now known as pradipta_away | 05:47 | |
| *** pradipta_away is now known as pradipta | 05:47 | |
| *** pradipta is now known as pradipta_away | 05:50 | |
| *** bvivek has quit IRC | 05:53 | |
| *** eguz has joined #openstack-ironic | 05:53 | |
| *** bvivek has joined #openstack-ironic | 05:56 | |
| *** eghobo has quit IRC | 05:57 | |
| *** geekyogi has joined #openstack-ironic | 06:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/103291 | 06:02 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 06:09 | |
| *** Haomeng has joined #openstack-ironic | 06:12 | |
| *** bvivek2 has joined #openstack-ironic | 06:25 | |
| *** bvivek has quit IRC | 06:25 | |
| *** radsy has quit IRC | 06:34 | |
| *** max_lobur has joined #openstack-ironic | 06:45 | |
| *** mitz_ has quit IRC | 06:48 | |
| *** ramineni has joined #openstack-ironic | 06:49 | |
| *** mitz_ has joined #openstack-ironic | 06:50 | |
| *** eguz has quit IRC | 06:55 | |
| GheRivero_afk | morning all | 06:56 |
| *** GheRivero_afk is now known as GheRivero | 06:56 | |
| *** pcrews has joined #openstack-ironic | 07:01 | |
| *** foexle has joined #openstack-ironic | 07:04 | |
| *** wanyen has quit IRC | 07:06 | |
| *** romcheg has quit IRC | 07:09 | |
| *** geekyogi has quit IRC | 07:10 | |
| mrda | Night all, see you tomorrow | 07:11 |
| *** mrda is now known as mrda_away | 07:11 | |
| *** pradipta_away is now known as pradipta | 07:11 | |
| *** bvivek2 has quit IRC | 07:14 | |
| *** pcrews has quit IRC | 07:17 | |
| *** foexle has quit IRC | 07:20 | |
| *** bvivek has joined #openstack-ironic | 07:28 | |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties for iLO driver https://review.openstack.org/103007 | 07:33 |
| *** romcheg has joined #openstack-ironic | 07:34 | |
| *** jcoufal has joined #openstack-ironic | 07:38 | |
| *** geekyogi has joined #openstack-ironic | 07:41 | |
| *** mkerrin has joined #openstack-ironic | 07:45 | |
| *** geekyogi has quit IRC | 07:50 | |
| * devananda reviews all the specs | 07:50 | |
| * devananda reviews ^Das many specs as he can before falling asleep | 07:51 | |
| *** ifarkas has joined #openstack-ironic | 07:51 | |
| *** lucasagomes has joined #openstack-ironic | 07:51 | |
| *** athomas has joined #openstack-ironic | 07:53 | |
| *** geekyogi has joined #openstack-ironic | 07:55 | |
| *** lsmola__ has joined #openstack-ironic | 07:56 | |
| *** max_lobur has quit IRC | 08:00 | |
| *** ndipanov has joined #openstack-ironic | 08:02 | |
| *** foexle has joined #openstack-ironic | 08:07 | |
| *** loki184 has joined #openstack-ironic | 08:11 | |
| *** amitpp has quit IRC | 08:14 | |
| *** amitpp has joined #openstack-ironic | 08:14 | |
| *** rakesh_hs2 has quit IRC | 08:19 | |
| *** rakesh_hs has joined #openstack-ironic | 08:22 | |
| *** bvivek has quit IRC | 08:22 | |
| openstackgerrit | A change was merged to openstack/ironic-specs: Update pbr version https://review.openstack.org/103334 | 08:31 |
| openstackgerrit | A change was merged to openstack/ironic: PEP fixes for the Nova driver https://review.openstack.org/103162 | 08:32 |
| openstackgerrit | A change was merged to openstack/ironic: pep8 cleanup of Nova code https://review.openstack.org/103001 | 08:32 |
| openstackgerrit | A change was merged to openstack/ironic: Fix the section name in CONTRIBUTING.rst https://review.openstack.org/103189 | 08:32 |
| *** derekh_ has joined #openstack-ironic | 08:41 | |
| *** bvivek has joined #openstack-ironic | 08:41 | |
| *** jcoufal has quit IRC | 08:48 | |
| *** pelix has joined #openstack-ironic | 08:56 | |
| openstackgerrit | Imre Farkas proposed a change to openstack/ironic-specs: DRAC power driver https://review.openstack.org/99352 | 09:14 |
| *** romcheg has quit IRC | 09:21 | |
| *** romcheg has joined #openstack-ironic | 09:24 | |
| romcheg | Morning Ironic | 09:24 |
| romcheg | It looks like I won't be able to come to our mid-cycle meet up | 09:25 |
| romcheg | :( | 09:25 |
| *** overlayer has joined #openstack-ironic | 09:25 | |
| *** sabah has quit IRC | 09:36 | |
| *** loki184 has quit IRC | 09:36 | |
| *** rakesh_hs has quit IRC | 09:41 | |
| *** sabah has joined #openstack-ironic | 10:03 | |
| *** Nisha has quit IRC | 10:08 | |
| *** sabah has quit IRC | 10:08 | |
| *** jcoufal has joined #openstack-ironic | 10:13 | |
| *** sabah has joined #openstack-ironic | 10:14 | |
| *** takadayuiko has joined #openstack-ironic | 10:19 | |
| *** rakesh_hs has joined #openstack-ironic | 10:42 | |
| *** Alexei_9871 has joined #openstack-ironic | 10:43 | |
| *** ramineni has quit IRC | 10:56 | |
| *** loki184 has joined #openstack-ironic | 11:05 | |
| openstackgerrit | Syed Ismail Faizan Barmawer proposed a change to openstack/ironic-specs: UEFI support for Ironic deploy drivers https://review.openstack.org/99850 | 11:06 |
| *** subah has joined #openstack-ironic | 11:07 | |
| *** sabah has quit IRC | 11:07 | |
| *** bvivek has quit IRC | 11:08 | |
| *** pradipta is now known as pradipta_away | 11:15 | |
| *** pradipta_away is now known as pradipta | 11:18 | |
| *** coolsvap is now known as coolsvap|afk | 11:23 | |
| *** Poornima has quit IRC | 11:25 | |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic-specs: iLO Virtual Media Deploy Driver https://review.openstack.org/97744 | 11:27 |
| *** athomas has quit IRC | 11:29 | |
| *** athomas has joined #openstack-ironic | 11:31 | |
| *** subah has quit IRC | 11:38 | |
| *** faizan has joined #openstack-ironic | 11:40 | |
| *** loki184 has quit IRC | 11:41 | |
| *** amitpp has quit IRC | 11:50 | |
| *** pradipta is now known as pradipta_away | 11:58 | |
| *** ajc_ has quit IRC | 11:58 | |
| *** nosnos has quit IRC | 11:59 | |
| *** nosnos has joined #openstack-ironic | 11:59 | |
| *** Shrews has quit IRC | 12:00 | |
| *** Shrews has joined #openstack-ironic | 12:01 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic-specs: iPXE boot https://review.openstack.org/98822 | 12:04 |
| *** nosnos has quit IRC | 12:04 | |
| *** takadayuiko has quit IRC | 12:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/102126 | 12:24 |
| *** jdob has joined #openstack-ironic | 12:25 | |
| *** k4n0 has quit IRC | 12:41 | |
| *** Poornima has joined #openstack-ironic | 12:43 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Rename/update common/tftp to drivers/modules/pxe_utils https://review.openstack.org/103595 | 12:45 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add iPXE support for Ironic https://review.openstack.org/99318 | 12:45 |
| *** linggao has joined #openstack-ironic | 12:46 | |
| *** jbjohnso has joined #openstack-ironic | 12:55 | |
| *** rloo has joined #openstack-ironic | 13:17 | |
| *** ramineni has joined #openstack-ironic | 13:20 | |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Cleanup mock patch without `with` part 3 https://review.openstack.org/73256 | 13:23 |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Get rid of the newline "\" https://review.openstack.org/66793 | 13:23 |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Check whether specified FS is supported https://review.openstack.org/98102 | 13:25 |
| *** Mikhail_D_ltp has quit IRC | 13:27 | |
| NobodyCam | good morning Ironic | 13:32 |
| NobodyCam | I have some family business to take care of this morning / today so I will in and out most of the day | 13:35 |
| *** pcrews has joined #openstack-ironic | 13:37 | |
| NobodyCam | romcheg: :( why | 13:37 |
| romcheg | I'm not likely to be funded for some unknown an bureaucratic reasons | 13:37 |
| NobodyCam | :( | 13:37 |
| romcheg | Morning NobodyCam lucasagomes and everyone else! | 13:38 |
| NobodyCam | morning romcheg | 13:38 |
| lucasagomes | morning NobodyCam romcheg | 13:38 |
| NobodyCam | morning lucasagomes | 13:38 |
| romcheg | 1.5k for tickets is too much for me | 13:38 |
| NobodyCam | ya | 13:38 |
| lucasagomes | NobodyCam, one thing before you go | 13:38 |
| romcheg | *only for tickets | 13:38 |
| lucasagomes | NobodyCam, can you take a look at the ironic -> ceilometer spec ? | 13:38 |
| lucasagomes | if u have time, if not that's fine | 13:38 |
| NobodyCam | is that the send sensor data to ceilometer | 13:40 |
| *** romcheg has quit IRC | 13:40 | |
| *** romcheg has joined #openstack-ironic | 13:40 | |
| rloo | hello ironickers! | 13:42 |
| NobodyCam | morning rloo | 13:42 |
| romcheg | Morning rloo! | 13:43 |
| NobodyCam | lucasagomes: my concern was if we have a large pool of nodes and several conductors. with out the tracking the last time we sent data, on ring reblaance every node would queue up for data colection | 13:44 |
| NobodyCam | if we have 10k nodes thats a lot of traffic | 13:45 |
| *** lazy_prince is now known as killer_prince | 13:45 | |
| lucasagomes | NobodyCam, hmm on ring rebalance we don't trigger the periodic tasks to run immediately | 13:46 |
| lucasagomes | NobodyCam, so it would wait the for the interval | 13:46 |
| lucasagomes | (which is by default 600) | 13:46 |
| NobodyCam | ahh ok | 13:46 |
| rloo | quick question, the etherpad shows that the gate status is DOWN due to glanceclient but that has been merged. Are there other issues with Gate or is everything grand in ironicland? | 13:47 |
| NobodyCam | rloo that should be fixed now | 13:47 |
| lucasagomes | rloo, oh it was fixed yesterday I think | 13:47 |
| lucasagomes | yeah | 13:47 |
| romcheg | Guys, do we have any sprints in EU? | 13:47 |
| lucasagomes | romcheg, yes, I'm in one right now | 13:47 |
| romcheg | fffuuu | 13:47 |
| rloo | thx NobodyCam, lucasagomes , I'll update the status. | 13:47 |
| lucasagomes | romcheg, https://wiki.openstack.org/wiki/Sprints/ParisJuno2014 | 13:48 |
| lucasagomes | in paris | 13:48 |
| lucasagomes | romcheg, it started today and will end on friday | 13:48 |
| *** rakesh_hs has quit IRC | 13:50 | |
| ramineni | Morning Ironic | 13:50 |
| rameshg87 | ramineni: good morning :-) | 13:51 |
| NobodyCam | lucasagomes: changed my vote. on the spec.. feel free to land away | 13:51 |
| romcheg | I will be in Spain and Amsterdam this August so I can meet some of you if you have time to drink beer and discuss the things | 13:51 |
| romcheg | :) | 13:51 |
| rameshg87 | rloo: please have a look at https://review.openstack.org/#/c/97455/ , got approval from devananda. he wanted you to take a look at it before approving :-) | 13:51 |
| lucasagomes | NobodyCam, Yay! | 13:51 |
| lucasagomes | NobodyCam, thanks for raising those concerns anyway, make me think/learn more about ceilometer | 13:52 |
| lucasagomes | data points | 13:52 |
| lucasagomes | romcheg, nice, where in spain? | 13:52 |
| NobodyCam | as long as we dont trigger a rdata poll on rebalance I'm ok with it | 13:52 |
| ramineni | lucasgomes,romcheg,rloo ,NobodyCam , can u guys have a look at firmware update design spec , if you have some time | 13:52 |
| rameshg87 | rloo: this is for ilo power spec review | 13:52 |
| lucasagomes | romcheg, and what day? | 13:52 |
| NobodyCam | brb | 13:52 |
| rloo | rameshg87: will look in a few minutes (catching up on stuff still) | 13:54 |
| romcheg | lucasagomes: Barcelona 8-12, Madrid 12-15, Porto (not Spain but very close) 15-18, Amsterdam 18-25 | 13:54 |
| rameshg87 | rloo: thanks | 13:54 |
| lucasagomes | romcheg, cool | 13:57 |
| *** amitpp has joined #openstack-ironic | 13:57 | |
| romcheg | lucasagomes: Would you be able to join me somewhere? | 13:57 |
| lucasagomes | romcheg, I can check that out, I'm planning to go to spain has a while | 13:57 |
| lucasagomes | so it may work out | 13:57 |
| lucasagomes | romcheg, ur going to the midcycle in beaverton? | 13:58 |
| romcheg | lucasagomes: I'd say no, because I'm not likely to be funded for that | 13:59 |
| *** amitpp has quit IRC | 14:00 | |
| romcheg | lucasagomes: It's been another re-org here + my relocation so there's no budged for Ironic midcycle event | 14:01 |
| lucasagomes | romcheg, :( I see | 14:01 |
| *** dkehnx has joined #openstack-ironic | 14:02 | |
| *** rloo has quit IRC | 14:05 | |
| *** max_lobur has joined #openstack-ironic | 14:09 | |
| *** rwsu has joined #openstack-ironic | 14:10 | |
| *** rloo_ has joined #openstack-ironic | 14:20 | |
| rameshg87 | rloo_: just wanted to discuss on the comments :-) | 14:27 |
| rameshg87 | rloo_: is it okay to discuss now ? | 14:27 |
| *** stendulker has joined #openstack-ironic | 14:29 | |
| *** Nisha has joined #openstack-ironic | 14:30 | |
| rloo_ | rameshg87: hi | 14:33 |
| *** jcoufal has quit IRC | 14:34 | |
| rameshg87 | rloo_: i just posted back some comments | 14:34 |
| rameshg87 | rloo_: for https://review.openstack.org/#/c/97455/13 | 14:35 |
| rameshg87 | rloo_: for the security impact section, since storing passwords in db is the same with every available driver, i thought i can skip that part | 14:35 |
| rameshg87 | rloo_: do you feel i should add i note there ? | 14:35 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 14:36 | |
| *** jcoufal has joined #openstack-ironic | 14:36 | |
| stendulker | jayF: Hi | 14:36 |
| rloo_ | rameshg87: I don't know if you should or not. Personally, I would prefer if the spec mentioned that info, but I don't want to delay the spec if others don't think it is worth mentioning. | 14:37 |
| stendulker | jayF: can you please review the design spec for firmware settings https://review.openstack.org/#/c/101122/ | 14:38 |
| rameshg87 | rloo_: okay, i can add a note there. not a problem. | 14:38 |
| rameshg87 | rloo_: regarding the tempest testing, we wanted to add but don't have any plans now. | 14:39 |
| rameshg87 | rloo_: since power is a very small module and not much useful by itself, it might be worth thinking about it when more advanced features like deploy are available in ilo driver | 14:40 |
| rloo_ | rameshg87: if you're going to update the spec, maybe it is worth mentioning that there will be tempest tests in the future when you add an ilo deploy. or not mention . i don't know, i got the info I wanted ;) | 14:40 |
| stendulker | jroll: Hi | 14:40 |
| stendulker | jroll: I have updated the firmware setting spec resolving your comments https://review.openstack.org/#/c/101122/ | 14:41 |
| *** Poornima has quit IRC | 14:41 | |
| rameshg87 | rloo_: i will add a note saying tempest tests will be considered when more advanced modules like deploy are submitted | 14:41 |
| rameshg87 | rloo_: thanks, i will revise the spec | 14:41 |
| rloo_ | rameshg87: sounds good to me. thx! | 14:41 |
| stendulker | jroll: Can you please check the same. Also wanted to know more about the comments related to security concerns | 14:42 |
| jroll | stendulker: I just think there should be a note about: if an unauthorized person can access this endpoint, that's very very bad | 14:43 |
| *** jgrimm has joined #openstack-ironic | 14:44 | |
| *** stendulker has quit IRC | 14:46 | |
| *** dwalleck has joined #openstack-ironic | 14:46 | |
| *** ramineni has quit IRC | 14:46 | |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic-specs: iLO Power Driver for Ironic https://review.openstack.org/97455 | 14:48 |
| *** max_lobur has quit IRC | 14:48 | |
| rameshg87 | rloo_: i have added the notes as discussed, please have a look at the spec again when you get time :-) | 14:49 |
| *** ramineni has joined #openstack-ironic | 14:49 | |
| *** stendulker has joined #openstack-ironic | 14:50 | |
| *** coolsvap|afk is now known as coolsvap | 14:53 | |
| *** foexle has quit IRC | 14:53 | |
| rloo_ | rameshg87: thx. I just reviewed it. | 14:53 |
| *** rloo_ has left #openstack-ironic | 14:54 | |
| *** rloo_ has joined #openstack-ironic | 14:55 | |
| stendulker | dtantsur: Hi | 14:56 |
| stendulker | dtantsur: I have resolved your reviw comments for firmware settings design spec https://review.openstack.org/#/c/101122 | 14:57 |
| stendulker | dtantsur: Can you please review the same. | 14:57 |
| rameshg87 | rloo_: thanks. :-) | 14:59 |
| jroll | stendulker: I just think there should be a note about: if an unauthorized person can access this endpoint, that's very very bad | 15:01 |
| stendulker | jroll: this endpoint would be similar to other endpoint like power, where it could impact the BM. I can add a note on similar lines under security section. | 15:04 |
| *** ndipanov_ has joined #openstack-ironic | 15:05 | |
| *** krtaylor has quit IRC | 15:05 | |
| jroll | stendulker: I would want to see a similar note if someone added the power state endpoint today :) | 15:06 |
| stendulker | jroll: Do you feel, any other thing that needs to be updated with respect to security | 15:06 |
| jroll | stendulker: no, just a note that this is very scary (tm) :) | 15:06 |
| stendulker | jroll: Sure, will do that. | 15:07 |
| *** ndipanov has quit IRC | 15:08 | |
| stendulker | jroll: I have resolved your other comments. Please look at the new version of the same | 15:08 |
| jroll | thanks | 15:11 |
| jroll | will do | 15:11 |
| *** dwalleck has quit IRC | 15:18 | |
| shardy | Hey all | 15:18 |
| shardy | FYI I've posted some heat resources which expose the Ironic API: | 15:19 |
| shardy | https://review.openstack.org/#/q/status:open+project:openstack/heat+branch:master+topic:bp/ironic-resource,n,z | 15:19 |
| shardy | Currently an initial draft but review feedback from ironic experts welcome ;) | 15:19 |
| shardy | lucasagomes: ^^ | 15:19 |
| *** max_lobur has joined #openstack-ironic | 15:27 | |
| *** dwalleck has joined #openstack-ironic | 15:30 | |
| stendulker | jroll: thank you | 15:31 |
| lucasagomes | shardy, w00t! thanks I will take a look at it | 15:40 |
| stendulker | lucasgomes: Hi | 15:41 |
| stendulker | lucasgomes: Can you please review firmware settings related to review https://review.openstack.org/#/c/101122 | 15:41 |
| *** jcoufal has quit IRC | 15:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/102126 | 15:42 |
| stendulker | lucasgomes: this review is for design spec for this feature | 15:42 |
| *** faizan has quit IRC | 15:42 | |
| lucasagomes | stendulker, hi yeah, I will take a look... I probably will have to see it tomorrow because it's a bit late here | 15:43 |
| stendulker | lucasgomes: Thank you. | 15:46 |
| *** rameshg87 is now known as rameshg87_afk | 15:52 | |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic-specs: API to get driver_info properties https://review.openstack.org/102914 | 15:54 |
| *** rakesh_hs has joined #openstack-ironic | 15:57 | |
| JayF | stendulker: I put a comment on that, it looks good just had a question about the scope and wanted it explicitly stated | 15:58 |
| *** eghobo has joined #openstack-ironic | 16:03 | |
| *** killer_prince is now known as lazy_prince | 16:05 | |
| *** ndipanov_ has quit IRC | 16:07 | |
| *** eghobo has quit IRC | 16:08 | |
| *** ramineni has quit IRC | 16:08 | |
| *** eghobo has joined #openstack-ironic | 16:08 | |
| stendulker | JayF: Thank you for the review. | 16:10 |
| rloo_ | devananda, lucasagomes, NobodyCam, romcheg, anyone: how are we handling patches that change the nova ironic driver. We can +2 but we shouldn't approve? | 16:11 |
| stendulker | JayF: I'm planning to change the scope statement more explicit to cover in-band mechanism IPA as well. | 16:11 |
| *** coolsvap is now known as coolsvap|afk | 16:12 | |
| stendulker | It proposes to introduce standard APIs to support common firmware settings. These firmware settings could be retrieved out-of-band using IPMI, iLO, DRAC etc. or via some in-band mechanism, e.g. IPA. | 16:12 |
| *** dwalleck has quit IRC | 16:13 | |
| lucasagomes | rloo_, hmm I'm also not sure... but that sounds correct | 16:13 |
| stendulker | JayF: It proposes to introduce standard APIs to support common firmware settings. These firmware settings could be retrieved out-of-band using IPMI, iLO, DRAC etc. or via some in-band mechanism, e.g. IPA. | 16:13 |
| JayF | stendulker: So first of all, we should have these chats somewhat in the spec so others can see/comment async :) | 16:14 |
| stendulker | JayF: I think this should make it explicit that this feature is applicable to both OOB and IB mechanism | 16:14 |
| JayF | stendulker: but generally, I don't think we should make it applicable to IB | 16:14 |
| JayF | stendulker: primarily because there /is no/ generic way to modify BIOS settings in band | 16:14 |
| JayF | stendulker: for instance, on the servers we're using for OnMetal, we can't 'flip' any single setting, we have to apply a complete set of settings | 16:15 |
| stendulker | Jay: ok | 16:15 |
| JayF | stendulker: so I'm not sure that the API should or could be the same even across different IB implementations of BIOS/Firmware settings | 16:15 |
| stendulker | JayF: ok. | 16:15 |
| JayF | I'd be +1 right now if IB was taken out of scope | 16:15 |
| JayF | also curious as to what jroll ^ thinks of all this | 16:15 |
| stendulker | JayF: I was not aware of this part. | 16:15 |
| JayF | stendulker: That's probably why jroll wanted me to look at it :) I've been doing most of the stuff on-server for these boxse | 16:16 |
| JayF | and pretty much setting BIOS settings from in-band requires proprietary tooling, and is different for each bios | 16:16 |
| stendulker | JayF: with respect to inability to apply the individual firmware settings . | 16:16 |
| JayF | for ours, which is running an AMI bios, it's a proprietary utility that only takes in full configuration sets, not individual settings | 16:16 |
| JayF | I can't speak to others, or if it's even possible on others, I just know the one example I have doesn't align well at all | 16:17 |
| devananda | morning, all | 16:17 |
| NobodyCam | good morning devananda | 16:17 |
| devananda | rloo_: right now, I'd prefer to be a gatekeeper for any patch that touches the nova driver in our tree | 16:17 |
| stendulker | JayF: Ok. Yes, but there could be more such platforms. | 16:18 |
| devananda | rloo_: simply because I need to merge all those changes into the proposal to Nova | 16:18 |
| rloo_ | devananda: that makes sense. how about you -2 them, so that we can't accidentally approve? | 16:18 |
| devananda | rloo_: ack | 16:18 |
| JayF | stendulker: I personally haven't seen many IB implementations that allow single options to be flipped (I wish they did exist though :C) | 16:18 |
| devananda | rloo_: got one in mind? | 16:18 |
| stendulker | JayF: I'm fine with stating that this spec is applicable to OOB mechanisms. | 16:19 |
| rloo_ | devananda: https://review.openstack.org/#/c/103136/ | 16:19 |
| rloo_ | devananda: https://review.openstack.org/#/c/92416/ | 16:19 |
| JayF | stendulker: cool, thanks | 16:19 |
| rloo_ | devananda: https://review.openstack.org/#/c/103137/ | 16:19 |
| *** jcoufal has joined #openstack-ironic | 16:20 | |
| devananda | rloo_: woo, a bunch! thanks | 16:20 |
| *** ndipanov_ has joined #openstack-ironic | 16:20 | |
| rloo_ | devananda: https://review.openstack.org/#/c/97536/ | 16:20 |
| stendulker | JayF: jroll has reviewed it and in the current patch I have addressed his comments. He had a concern related to security, which I have discussed with him and would add a note related to the same. jroll have agreed to look at the current patch. | 16:20 |
| rloo_ | devananda: there may be more. Those were the ones I noticed. Did you want me to look through them all? | 16:21 |
| stendulker | devananda: Hi | 16:21 |
| stendulker | devananda: Can you please review firmware settings related to review https://review.openstack.org/#/c/101122 | 16:21 |
| devananda | rloo_: hmm. no, but pls point them out if you see them | 16:21 |
| rloo_ | devananda: will do ;) | 16:21 |
| devananda | stendulker: hi! not right now, no. I've already outlined what my review priorities are, and will get to additional specs later in the cycle, probably around the midcycle meeting | 16:22 |
| *** dwalleck has joined #openstack-ironic | 16:23 | |
| devananda | stendulker: thank you for the ping, though. I wish I had more time to review everything, but unfortunately, do not | 16:23 |
| devananda | mrda_away: https://review.openstack.org/#/c/102695 looks like a failure in your changeset, but also looks like a valuable change which I'd love to be able to land soonish ... | 16:26 |
| jroll | stendulker, JayF: I don't think we should restrict it to oob implementations. could still do single settings flip inband... would just require the implementation to do something like get all settings -> apply all settings with this one change (not ideal, I know). the driver could also just raise NotImplementedError if it's not possible. | 16:26 |
| stendulker | devananda: I understand. I pinged you as you had API related concern. I wished to convey you that its being addressed with the help of dtantsur, jroll, JayF and NobodyCam | 16:26 |
| jroll | we don't need everybody to review, especially for lower priority things :) | 16:27 |
| *** overlayer has quit IRC | 16:28 | |
| JayF | jroll: honestly from my experience with the tooling, at least that we're using right now, I think that's a pretty bad road to go down | 16:28 |
| stendulker | jroll, JayF: thank you for your feedback. If JayF is fine, I will retain references to IB mechanisms in the spec. | 16:29 |
| jroll | JayF: then leave it up to the driver | 16:29 |
| lucasagomes | devananda, morning | 16:29 |
| JayF | jroll: I could be OK with a "here's a collection of settings, apply them", but the in band tools to do this really are awful | 16:29 |
| *** athomas has quit IRC | 16:29 | |
| lucasagomes | I will have to go, have a good night everybody | 16:29 |
| jroll | JayF: there might be wrird hw where ipa *can* do it inband | 16:29 |
| jroll | weird* | 16:29 |
| jroll | night lucasagomes | 16:30 |
| stendulker | lucasgomes: good night :) | 16:30 |
| *** lucasagomes has quit IRC | 16:30 | |
| *** krtaylor has joined #openstack-ironic | 16:30 | |
| JayF | jroll: I just think that in a lot of ways, applying firmware settings in-band is a radically different thing than applying them out of band | 16:31 |
| jroll | JayF: I just don't see an issue with having the same api | 16:32 |
| JayF | and I think there are inherent differences that are going to be incredibly difficult, if not borderline impossible in some cases, to abstract away | 16:32 |
| jroll | JayF: and an implementation can choose not to implement it | 16:33 |
| *** lazy_prince is now known as killer_prince | 16:34 | |
| JayF | Would this preclude a non-managementinterface implementation of firmware setting in-band? | 16:34 |
| *** lsmola__ has quit IRC | 16:34 | |
| JayF | if so, then I think those differences are relevant now from the perspective of that spec | 16:34 |
| *** ifarkas has quit IRC | 16:34 | |
| devananda | JayF, jroll: have you both seen the async API spec's latest incarnation? | 16:34 |
| jroll | JayF: unclear what you mean | 16:34 |
| Nisha | devananda: lifeless Hi, Did you had a chance to look at Discover node properties at node-create/node-update https://review.openstack.org/#/c/100951/16? | 16:35 |
| jroll | devananda: no | 16:35 |
| JayF | I haven't read any incarnation of it, but I think from what you said I probably should :) | 16:35 |
| devananda | I think it presents a framework for a REST API where a single ManagementInterface could handle both in- and out-of-band | 16:35 |
| devananda | yes | 16:35 |
| devananda | basically it's a proposal for a /v2/ API | 16:35 |
| devananda | to address several issues we're seeing | 16:35 |
| jroll | right | 16:35 |
| jroll | I haven't looked in a week or two | 16:36 |
| devananda | I suspect we'll all end up talking at length about it at the midcycle, but it's gonna end up being a lot more work than we can do in Juno, clearly | 16:36 |
| devananda | so iterative improvements for now, while we discuss if larger changes are really necessary // how to address issues caused by the underlying API architecture we have today | 16:37 |
| stendulker | jayF, jroll: just curious, given that all of the firmware settings needs to be applied in one go for in-band mechanism. what kind of mangement interface would address it? | 16:38 |
| JayF | stendulker: I'm not saying for /all/ in-band mechanisms, just the ones I'm familiar with | 16:38 |
| romcheg | rloo_: having a class right now | 16:38 |
| JayF | stendulker: and honestly, I don't know, but I would see something that talks more about the purpose of the settings, and leaves the implementation to the hardware manager in IPA for in band | 16:39 |
| rloo_ | romcheg: no worries, I got the answer from deva. | 16:39 |
| *** dkehn__ has joined #openstack-ironic | 16:39 | |
| JayF | stendulker: for instance, "set bios settings to VM_OPTIMAL" might enable VT, disable Turbo mode and power saving | 16:39 |
| stendulker | jayF, jroll: ok. but again, it would be a binary patch, I suppose. | 16:39 |
| jroll | devananda: agreed | 16:39 |
| JayF | stendulker: or "set bios settings to secure" might set some BIOS settings that 'lock down' the instance for a customer | 16:39 |
| JayF | stendulker: but the key is that it has to be generic enough that the IPA HardwareManager has lots of freedom to implement it in whatever crazy way a vendor decided you should set bios settings for their box | 16:40 |
| jroll | "binary patch"? | 16:40 |
| stendulker | jayF: ok. so single setting would change the related settings as well. | 16:40 |
| *** geekyogi1 has joined #openstack-ironic | 16:40 | |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Add more INFO logging to ironic/common https://review.openstack.org/99495 | 16:41 |
| devananda | stendulker: Jayf: https://review.openstack.org/94923 | 16:41 |
| stendulker | jroll: since it was mentioned that multiple settings are updated through single shot, i thought the propritary tool generates a patch based upon settings input. My mistake | 16:41 |
| JayF | stendulker: I'm just thinking more about talking about what, on a high level, we're trying to do rather than trying to make Ironic (and IPA) smart enough to know how to flip single settings on and off | 16:41 |
| devananda | proposal for a "convergence API" | 16:41 |
| jroll | stendulker: oh. dunno. | 16:41 |
| *** dkehnx has quit IRC | 16:41 | |
| devananda | based on an approach like that, an inband firmware update could be done in a single pass | 16:42 |
| devananda | whereby the user makes one or more API requests for the state they want, and then at some time (eg, just before deploy, or something else... TBD) the agent applies them | 16:42 |
| Nisha | devananda could you please review https://review.openstack.org/#/c/100951/16? Got +1 from NobodyCam | 16:42 |
| *** geekyogi has quit IRC | 16:43 | |
| *** Isotopp has quit IRC | 16:43 | |
| devananda | Nisha: I will take a look when I have time | 16:43 |
| *** Isotopp has joined #openstack-ironic | 16:43 | |
| devananda | which may be a while. I alraedy spent ~4 hours reviewing specs today | 16:43 |
| Nisha | devananda: Thanks... | 16:43 |
| stendulker | devananda: thanks for pointing to this spec. | 16:43 |
| *** Isotopp has quit IRC | 16:44 | |
| *** Isotopp has joined #openstack-ironic | 16:45 | |
| stendulker | NobodyCam: I have responded to your queries wrt firmware settings review https://review.openstack.org/#/c/101122 Please have a look. | 16:45 |
| *** hemna__ is now known as hemna | 16:45 | |
| stendulker | jroll, JayF: JayF, if you agree, I will mention that this spec applies to in-band mechanisms as well. | 16:47 |
| JayF | I disagree but I'll have a battle royale with jroll when he gets into the office since we work together :) | 16:48 |
| jroll | ^ | 16:48 |
| jroll | there may be something I'm missing | 16:48 |
| stendulker | oh :) | 16:48 |
| stendulker | jayF, jroll: May be we can take a final call, once both of you could discuss this in person. | 16:49 |
| jroll | stendulker: we'll comment on the review so that everyone can see | 16:50 |
| stendulker | jroll: that would help. I opened the discussion here for quicker resolution as we work across time zones. Shall I capture this discussion in the review as a comment? | 16:51 |
| NobodyCam | stendulker: will do... | 16:51 |
| jroll | stendulker: right. do that if you like | 16:51 |
| stendulker | NobodyCam: thank you | 16:52 |
| stendulker | jroll: will do that. will add summary on this discussion | 16:52 |
| *** pelix has quit IRC | 16:52 | |
| *** harlowja_away is now known as harlowja | 16:52 | |
| jroll | stendulker: for what its worth, there's no rush on this spec as it is not a priority for juno... but still faster is better :) | 16:52 |
| stendulker | jroll: yes, unfortunately. But if this spec could get into right shape, it may have a luck in later cycle. | 16:54 |
| jroll | stendulker: of course | 16:55 |
| *** dkehn__ is now known as dkehnx | 16:57 | |
| *** ndipanov_ has quit IRC | 17:01 | |
| *** Penick has joined #openstack-ironic | 17:07 | |
| *** Alexei_9871 has quit IRC | 17:09 | |
| *** derekh_ has quit IRC | 17:11 | |
| *** loki184 has joined #openstack-ironic | 17:13 | |
| *** eghobo has quit IRC | 17:14 | |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Update nova rebuild to account for new image https://review.openstack.org/103227 | 17:20 |
| *** dkehn__ has joined #openstack-ironic | 17:21 | |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Update nova rebuild to account for new image https://review.openstack.org/103227 | 17:23 |
| *** dkehn__ has quit IRC | 17:23 | |
| *** dkehn__ has joined #openstack-ironic | 17:24 | |
| *** dkehnx has quit IRC | 17:25 | |
| *** dkehn__ is now known as dkehnx | 17:26 | |
| *** romcheg has quit IRC | 17:29 | |
| JoshNang | NobodyCam: in the swift spec, about setting the tenant_id as a conf option limiting it to a single tenant, that's true. i'm not sure how to work around it though. do i have access to the user's tenant_id at deploy time? | 17:31 |
| devananda | GheRivero: that ^ has unresolved merge conflict at L742 of driver.py | 17:32 |
| GheRivero | devananda: agh! I fixed another, but miss that one! silly me. | 17:34 |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Update nova rebuild to account for new image https://review.openstack.org/103227 | 17:38 |
| *** stendulker has quit IRC | 17:38 | |
| lifeless | Nisha: no I haven't, as I said, early next week. | 17:40 |
| linggao | Hi Shrews, is the bug that you mentioned to me yesterday landed? | 17:43 |
| *** chuckC has quit IRC | 17:43 | |
| *** datajerk has quit IRC | 17:43 | |
| Shrews | linggao: yes | 17:43 |
| *** Penick has quit IRC | 17:45 | |
| linggao | Shrews, do you know the bug number? | 17:45 |
| linggao | I need to recheck my patch. | 17:45 |
| Shrews | linggao: no. you’ll need to rebase before you recheck though | 17:46 |
| linggao | oh, no. | 17:46 |
| jroll | JoshNang: I believe you don't have the user's tenant id in callbacks | 17:48 |
| *** Penick has joined #openstack-ironic | 17:48 | |
| *** openstackgerrit has quit IRC | 17:49 | |
| JoshNang | jroll: ahh yeah. but i could save it in instance info or something beforehand if that's the only blocker. or generate the temp url earlier or something. | 17:50 |
| *** openstackgerrit has joined #openstack-ironic | 17:50 | |
| jroll | gah. | 17:51 |
| JoshNang | though, i'd still need to know the backend container the image is stored in. i'm leaning towards documenting that if you don't have direct_url you're limited to a single tenant_id/container | 17:51 |
| jroll | disclaimer, didn't read the comment yet | 17:51 |
| jroll | maybe | 17:51 |
| jroll | but like | 17:51 |
| jroll | I don't want to be limited to that | 17:51 |
| *** rameshg87_afk is now known as rameshg87 | 17:51 | |
| JoshNang | i agree. i wonder if there's a way i could query glance for it. you'd think there'd be a way to not enable direct_url in the standard glance image-show, but allow admins to see it only. | 17:52 |
| *** Shrews has quit IRC | 17:55 | |
| *** sseago has quit IRC | 17:56 | |
| *** loki184 has quit IRC | 17:57 | |
| *** rameshg87 is now known as rameshg87_afk | 18:00 | |
| jroll | JoshNang: I mean, glance has like a storage url or something, yes? | 18:01 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Add current directory logging to build script https://review.openstack.org/104276 | 18:04 |
| JoshNang | jroll: wdym? | 18:05 |
| jroll | JoshNang: I think glance-show gives you a 'storage url'. kind of like a direct url without creds. | 18:06 |
| jroll | I might be completely wrong | 18:06 |
| JoshNang | jroll: i don't think so. i'll go check though | 18:07 |
| jroll | I swear I've heard of it | 18:08 |
| jroll | idk | 18:08 |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Add current directory logging to build script https://review.openstack.org/104276 | 18:12 |
| *** sseago has joined #openstack-ironic | 18:12 | |
| *** Shrews has joined #openstack-ironic | 18:12 | |
| devananda | rloo, NobodyCam: either of you want to review another cleanup to the nova driver? https://review.openstack.org/#/c/97536/ | 18:13 |
| rloo_ | devananda: I'll take a look | 18:17 |
| *** sseago has quit IRC | 18:17 | |
| JoshNang | jroll: this is all i see in an image-show (no direct_url, swift as backend). https://gist.github.com/pcsforeducation/ac2e8b8e0aea4968ed62 | 18:17 |
| jroll | JoshNang: 'nova image-show' might not display all the data | 18:19 |
| *** amitpp has joined #openstack-ironic | 18:19 | |
| jroll | JoshNang: if you run with --debug, you should see the actual json flying around | 18:19 |
| *** faizan has joined #openstack-ironic | 18:25 | |
| NobodyCam | rloo_: I +2's that review (97536) but did not +a so you can look it over :) | 18:29 |
| NobodyCam | s/\'s/\'d/ | 18:29 |
| rloo_ | NobodyCam: thx. Am almost done. | 18:29 |
| *** sseago has joined #openstack-ironic | 18:29 | |
| rloo_ | devananda: wrt 97536, all yours to approve when you want! | 18:30 |
| *** chuckC has joined #openstack-ironic | 18:31 | |
| *** wanyen has joined #openstack-ironic | 18:32 | |
| *** overlayer has joined #openstack-ironic | 18:39 | |
| *** amitpp has quit IRC | 18:41 | |
| devananda | NobodyCam: have a minute to take a look at https://review.openstack.org/#/c/102373/ again? | 18:43 |
| NobodyCam | I might :) | 18:45 |
| *** eghobo has joined #openstack-ironic | 18:47 | |
| *** eghobo has quit IRC | 18:47 | |
| *** eghobo has joined #openstack-ironic | 18:48 | |
| *** overlayer has quit IRC | 18:49 | |
| *** overlayer has joined #openstack-ironic | 18:50 | |
| devananda | GheRivero: https://review.openstack.org/#/c/103227/4 still failing | 18:50 |
| NobodyCam | devananda: vote updated to +2 thou I did not +a, leaving that click to you :-p | 18:53 |
| *** romcheg has joined #openstack-ironic | 18:53 | |
| *** chuckC has quit IRC | 18:54 | |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Update nova rebuild to account for new image https://review.openstack.org/103227 | 18:54 |
| GheRivero | devananda: fixed. with the rebase some ironic.nova pep8 changes landed and break the new tests. My bad for not testing it locally before pushing it | 18:55 |
| devananda | NobodyCam: ack, ty | 18:55 |
| devananda | GheRivero: ah yea. several fixes are in-flight right now :) trying to land all the driver cleanup so I can update the driver proposal to Nova | 18:56 |
| NobodyCam | devananda: Thank you for comments | 18:56 |
| NobodyCam | lol makes more sense reading it then just thinkng thru the tests :-p | 18:57 |
| *** chuckC has joined #openstack-ironic | 18:57 | |
| *** harlowja is now known as harlowja_away | 18:59 | |
| *** Penick has quit IRC | 19:02 | |
| *** harlowja_away is now known as harlowja | 19:03 | |
| *** Nisha has quit IRC | 19:06 | |
| *** Penick has joined #openstack-ironic | 19:12 | |
| openstackgerrit | A change was merged to openstack/ironic: Add the remaining unittests to the ClientWrapper class https://review.openstack.org/92416 | 19:13 |
| *** datajerk has joined #openstack-ironic | 19:13 | |
| *** faizan has quit IRC | 19:23 | |
| openstackgerrit | A change was merged to openstack/ironic: Nova Ironic driver get_info() to return memory stats in KBytes https://review.openstack.org/103136 | 19:25 |
| openstackgerrit | A change was merged to openstack/ironic: Update Nova Ironic Driver destroy() method https://review.openstack.org/103137 | 19:25 |
| openstackgerrit | A change was merged to openstack/ironic: Add/Update docstrings in the Nova Ironic Driver https://review.openstack.org/97536 | 19:27 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Remove unused classmethod from nova.virt.ironic driver https://review.openstack.org/104322 | 19:27 |
| NobodyCam | really thats not used :-p | 19:31 |
| devananda | NobodyCam: where is it used? | 19:32 |
| NobodyCam | no no I'm just kinda shocked its not. I have not found it in use yet | 19:33 |
| devananda | i've grepped for self.instance and IronicDriver.instance -- nothing | 19:33 |
| NobodyCam | instance or *._instance? | 19:34 |
| NobodyCam | if not hasattr(cls, '_instance'): | 19:34 |
| devananda | NobodyCam: it is creating an instance of the driver, then stashing it on the class ... but why? I don't see anything in nova that uses that | 19:36 |
| * devananda lunches, bbi20 | 19:39 | |
| NobodyCam | ya not seeing it used either | 19:39 |
| NobodyCam | but /me opts to wait for tempest tests | 19:39 |
| NobodyCam | :-p | 19:39 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Minor cleanups to nova virt driver and tests https://review.openstack.org/104322 | 19:39 |
| NobodyCam | enjoy lunch | 19:39 |
| devananda | updating with two other fixes :) | 19:39 |
| NobodyCam | ack | 19:39 |
| NobodyCam | food sounds good | 19:40 |
| NobodyCam | brb | 19:41 |
| *** overlayer has quit IRC | 19:42 | |
| *** overlayer has joined #openstack-ironic | 19:43 | |
| openstackgerrit | linggao proposed a change to openstack/ironic-specs: Add console log support https://review.openstack.org/96538 | 19:51 |
| JoshNang | jroll: that was glance image-show. even with debug its pretty scarce on details. nova image-show with debug shows endpoints from keystone. the token gives us the tenant_id. but we'd still be short glance/swift container | 19:55 |
| jroll | hrm, ok | 19:56 |
| *** jdob has quit IRC | 19:57 | |
| *** jdob has joined #openstack-ironic | 19:58 | |
| openstackgerrit | linggao proposed a change to openstack/ironic: Fix exception handling in console https://review.openstack.org/102318 | 19:59 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-specs: Swift Temporary URLs Spec https://review.openstack.org/98904 | 20:02 |
| *** rakesh_hs has quit IRC | 20:06 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Allow noauth for Neutron https://review.openstack.org/98922 | 20:08 |
| *** harlowja is now known as harlowja_away | 20:09 | |
| *** zdiN0bot has joined #openstack-ironic | 20:10 | |
| *** romcheg has quit IRC | 20:14 | |
| *** zdiN0bot has quit IRC | 20:17 | |
| *** sseago has quit IRC | 20:22 | |
| *** sseago has joined #openstack-ironic | 20:22 | |
| openstackgerrit | Alex Weeks proposed a change to openstack/ironic-specs: Add metric reporting spec. https://review.openstack.org/100729 | 20:28 |
| Shrews | Anyone familiar with the disk_available_least compute node value? | 20:33 |
| *** sseago has quit IRC | 20:37 | |
| rloo_ | JoshNang: I just commented on 98922 (noauth for neutron). let me know if you have questions. | 20:38 |
| *** romcheg has joined #openstack-ironic | 20:38 | |
| *** harlowja_away is now known as harlowja | 20:39 | |
| *** Mikhail_D_ltp has quit IRC | 20:40 | |
| Shrews | adam_g: we have 4 bugs marked with 'ironic-tempest' tag, and 3 marked with 'tempest'. Wondering which ones are mis-tagged | 20:45 |
| Shrews | Neither tag appears in our "official" list: https://wiki.openstack.org/wiki/Bug_Tags#Ironic | 20:46 |
| adam_g | Shrews, they're mostly tempest bugs, didnt realize they needed to be 'official' tags. was tagging for my own personal book keeping. ill bring that up with tempest guys tomorrow. | 20:48 |
| Shrews | oh, those are under the tempest bugs | 20:48 |
| JoshNang | rloo_: nope that makes sense to me! | 20:48 |
| Shrews | adam_g: my bad... thought they were filed under ironic | 20:49 |
| rloo_ | JoshNang: ok thx. Sorry, I totally forgot about that patch; I should have given then feedback sooner. | 20:49 |
| JoshNang | rloo_: no worries! | 20:50 |
| * devananda is back | 20:50 | |
| *** krtaylor has quit IRC | 20:50 | |
| *** zdiN0bot has joined #openstack-ironic | 20:52 | |
| *** dwalleck has quit IRC | 20:54 | |
| *** zdiN0bot has quit IRC | 20:55 | |
| NobodyCam | wb devananda | 20:57 |
| Shrews | devananda: fyi, https://bugs.launchpad.net/ironic/+bug/1336937 | 20:57 |
| Shrews | probably should fix that before merging the driver back into nova | 20:59 |
| Shrews | but i have no idea what that value is yet | 20:59 |
| *** mrda_away is now known as mrda | 21:02 | |
| *** linggao has quit IRC | 21:02 | |
| mrda | Morning Ironic! | 21:02 |
| Shrews | morning mrda | 21:02 |
| NobodyCam | morning mrda | 21:03 |
| mrda | devananda: I'll be working on that changeset re: caching today. Looks like a little more work, but hoping to get another patch up soon (might need some work in ironicclient) | 21:03 |
| NobodyCam | filename.docx is a M$ word file, I think? | 21:03 |
| mrda | hey Shrews & NobodyCam :) | 21:03 |
| devananda | mrda: ack, thanks! | 21:03 |
| devananda | NobodyCam: yes | 21:04 |
| devananda | mrda: and g'morning! | 21:04 |
| * mrda is off to cafe hack, bbs (\o deva) | 21:04 | |
| devananda | Shrews: interesting... | 21:04 |
| *** romcheg has quit IRC | 21:04 | |
| devananda | Shrews: not causing a functional problem for us, but def needs to be fixed as part of the tempest cleanup | 21:05 |
| devananda | Shrews: it's not immediately clear to me what we should set that to | 21:05 |
| *** zdiN0bot has joined #openstack-ironic | 21:05 | |
| devananda | Shrews: size of the root_gb on the smallest node? | 21:06 |
| devananda | *shrug* | 21:06 |
| Shrews | devananda: me neither. best i could find so far is the libvirt driver comments and http://www-01.ibm.com/support/knowledgecenter/#!/SS4KMC_2.3.0/com.ibm.sco.doc_2.3/sr/hv/hvr_trohyer.html | 21:06 |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Fix tear_down a node with missing info https://review.openstack.org/103685 | 21:07 |
| *** jbjohnso has quit IRC | 21:08 | |
| devananda | adam_g: have you been hitting https://bugs.launchpad.net/ironic/+bug/1326364 in the last ~3 weeks? it looks like both fixes landed but the bug wasn't closed yet | 21:10 |
| *** steveh has joined #openstack-ironic | 21:11 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Allow noauth for Neutron https://review.openstack.org/98922 | 21:14 |
| JoshNang | rloo_: so i basically took your advice but made it a bit more explicit. a few other OS services use auth_strategy=keystone as default, so i stuck to that and added error checking. thanks for the review | 21:15 |
| *** romcheg has joined #openstack-ironic | 21:19 | |
| *** zdiN0bot has quit IRC | 21:21 | |
| *** zdiN0bot has joined #openstack-ironic | 21:21 | |
| adam_g | devananda, the unique constraint is only a partial fix. i had a patch for it, one min | 21:27 |
| adam_g | devananda, https://review.openstack.org/#/c/99519/ was my fix. didn't get li9nked | 21:28 |
| devananda | adam_g: I think the "." on the closes-bug line breaks the parsing sometimes | 21:29 |
| adam_g | devananda, yeah, thats my hunch too. ive stopped adding it lately | 21:29 |
| *** sseago has joined #openstack-ironic | 21:29 | |
| devananda | adam_g: so you feel the bug is fixed now? | 21:30 |
| devananda | adam_g: if so, i'll link and close it | 21:30 |
| adam_g | devananda, it should be, yes. | 21:30 |
| devananda | adam_g: awesome | 21:30 |
| adam_g | thanks | 21:30 |
| *** overlayer has quit IRC | 21:30 | |
| *** jdob has quit IRC | 21:31 | |
| rloo_ | JoshNang: just one little thing now ;) | 21:31 |
| openstackgerrit | A change was merged to openstack/ironic: Update Nova's available resources at termination https://review.openstack.org/102373 | 21:36 |
| *** overlayer has joined #openstack-ironic | 21:40 | |
| rloo_ | JoshNang: wrt your spec: https://review.openstack.org/#/c/98904/. Did some changes get lost? eg the para under Performance Impact is diff from revision 7. | 21:42 |
| *** romcheg has quit IRC | 21:49 | |
| *** Haomeng|2 has joined #openstack-ironic | 21:50 | |
| *** Haomeng has quit IRC | 21:50 | |
| * NobodyCam is back with doped up bubbie. :-p | 21:54 | |
| openstackgerrit | A change was merged to openstack/ironic: Updated from global requirements https://review.openstack.org/102126 | 21:55 |
| *** zdiN0bot has quit IRC | 22:04 | |
| *** ccrouch has quit IRC | 22:04 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Run make with sudo when building https://review.openstack.org/104361 | 22:11 |
| *** ccrouch has joined #openstack-ironic | 22:17 | |
| *** radsy has joined #openstack-ironic | 22:27 | |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Add more INFO logging to ironic/conductor https://review.openstack.org/99744 | 22:27 |
| *** jgrimm has quit IRC | 22:37 | |
| *** Penick has quit IRC | 22:38 | |
| NobodyCam | hum we have a blueprint for secure storage for IPMI creds (kinda) but not a spec :-p | 22:48 |
| *** zdiN0bot has joined #openstack-ironic | 22:52 | |
| *** krtaylor has joined #openstack-ironic | 22:56 | |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Add more INFO logging to ironic/common https://review.openstack.org/99495 | 22:58 |
| *** blamar_ has joined #openstack-ironic | 23:01 | |
| *** blamar has quit IRC | 23:01 | |
| *** blamar_ is now known as blamar | 23:01 | |
| *** faizan has joined #openstack-ironic | 23:04 | |
| *** blamar has quit IRC | 23:04 | |
| NobodyCam | rloo_: LOL - pros: no code changes at all, no need to write this specification | 23:09 |
| *** zdiN0bot has quit IRC | 23:09 | |
| JayF | NobodyCam: doped up bubbie? haha I don't know what that means but it sounds fun and hilarious | 23:13 |
| NobodyCam | JayF: Bubbie is our name for our male Great Dane. just got him back from surgery | 23:15 |
| JayF | Aha. | 23:15 |
| JayF | I went almost the entire other direction :) | 23:15 |
| NobodyCam | :) all is good with him... just groggy | 23:15 |
| *** ccrouch has quit IRC | 23:26 | |
| *** zdiN0bot has joined #openstack-ironic | 23:30 | |
| *** jcoufal has quit IRC | 23:48 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-python-agent: Adding support for decommissioning https://review.openstack.org/104379 | 23:51 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Use "incubated" doc theme https://review.openstack.org/104380 | 23:51 |
| devananda | one more review for the nova driver, if anyone's still around: https://review.openstack.org/#/c/103227/ | 23:54 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/88722 | 23:54 |
| *** zdiN0bot has quit IRC | 23:54 | |
| *** max_lobur has quit IRC | 23:58 | |
| faizan | Devananda: NobodyCam: thanks for reviewing uefi spec 99850. I have address most of your comments and posted a patch. I need to discuss on few of the review comments | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!