| *** ajya[m] has quit IRC | 00:00 | |
| *** ajya[m] has joined #openstack-ironic | 00:11 | |
| *** hshiina has joined #openstack-ironic | 00:58 | |
| *** rcernin has quit IRC | 01:36 | |
| *** eandersson has quit IRC | 01:45 | |
| *** edleafe has quit IRC | 03:34 | |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/bifrost master: fix grammar issues in bifrost mission statement https://review.openstack.org/619959 | 05:01 |
|---|---|---|
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Expose conductors: db and rpc https://review.openstack.org/619683 | 05:30 |
| *** skazi has quit IRC | 05:39 | |
| *** diga has joined #openstack-ironic | 06:42 | |
| *** arne_wiebalck_ has joined #openstack-ironic | 07:00 | |
| *** adrianc has joined #openstack-ironic | 07:06 | |
| jiapei | Good morning Ironicers | 07:14 |
| *** skazi has joined #openstack-ironic | 07:15 | |
| *** adrianc has quit IRC | 07:17 | |
| *** adrianc has joined #openstack-ironic | 07:17 | |
| *** jaganathan has joined #openstack-ironic | 07:20 | |
| *** krypto has quit IRC | 07:31 | |
| etingof | Happy Monday jiapei o/ | 07:34 |
| jiapei | Hey etingof \o | 07:34 |
| *** krypto has joined #openstack-ironic | 07:39 | |
| *** iurygregory has joined #openstack-ironic | 07:39 | |
| iurygregory | good morning | 07:41 |
| *** e0ne has joined #openstack-ironic | 07:46 | |
| *** moshele has joined #openstack-ironic | 07:47 | |
| *** jtomasek has joined #openstack-ironic | 07:47 | |
| *** rpittau has joined #openstack-ironic | 07:49 | |
| *** krypto has quit IRC | 07:56 | |
| *** rpittau has quit IRC | 08:01 | |
| *** rpittau_ has joined #openstack-ironic | 08:01 | |
| *** jaganathan has quit IRC | 08:03 | |
| *** gkadam has joined #openstack-ironic | 08:07 | |
| *** rpittau_ is now known as rpittau | 08:11 | |
| rpittau | good morning ironic :) | 08:11 |
| *** emine__ has joined #openstack-ironic | 08:12 | |
| iurygregory | morning rpittau o/ | 08:13 |
| *** rpittau has quit IRC | 08:14 | |
| *** rpittau has joined #openstack-ironic | 08:15 | |
| rpittau | hey iurygregory | 08:15 |
| *** iurygregory has quit IRC | 08:21 | |
| *** iurygregory has joined #openstack-ironic | 08:23 | |
| *** jaganathan has joined #openstack-ironic | 08:33 | |
| *** BOKALDO has joined #openstack-ironic | 08:41 | |
| iurygregory | if anyone have sometime to review https://review.openstack.org/#/q/46a5899543bce73e1d40f6f363031f9ac1a472e8 https://review.openstack.org/#/c/618543/ o/ tyvm =) | 08:51 |
| patchbot | patch 618543 - python-ironic-inspector-client - Add check-errors flag to Introspection - 5 patch sets | 08:51 |
| *** tssurya has joined #openstack-ironic | 09:07 | |
| etingof | iurygregory could consider adding people to the reviewers list for the patch... | 09:13 |
| iurygregory | etingof, that would be my last attempt I would say XD | 09:14 |
| *** dougsz has joined #openstack-ironic | 09:14 | |
| etingof | beer stimulus works as well ;) | 09:14 |
| iurygregory | true | 09:15 |
| iurygregory | and also coffee | 09:15 |
| *** NobodyCam_ has joined #openstack-ironic | 09:17 | |
| *** fyx_ has joined #openstack-ironic | 09:17 | |
| *** TheJulia_ has joined #openstack-ironic | 09:17 | |
| *** early` has joined #openstack-ironic | 09:18 | |
| *** mgagne_ has joined #openstack-ironic | 09:23 | |
| *** zigo has quit IRC | 09:25 | |
| *** early has quit IRC | 09:25 | |
| *** jroll has quit IRC | 09:25 | |
| *** rpioso|afk has quit IRC | 09:25 | |
| *** mgagne has quit IRC | 09:25 | |
| *** nicholas has quit IRC | 09:25 | |
| *** fyx has quit IRC | 09:25 | |
| *** TheJulia has quit IRC | 09:25 | |
| *** NobodyCam has quit IRC | 09:25 | |
| *** ildikov has quit IRC | 09:25 | |
| *** NobodyCam_ is now known as NobodyCam | 09:25 | |
| *** TheJulia_ is now known as TheJulia | 09:25 | |
| *** fyx_ is now known as fyx | 09:25 | |
| *** rpittau_ has joined #openstack-ironic | 09:27 | |
| *** rpittau has quit IRC | 09:29 | |
| *** rpittau_ is now known as rpittau | 09:29 | |
| *** jroll has joined #openstack-ironic | 09:32 | |
| *** ildikov has joined #openstack-ironic | 09:34 | |
| *** derekh has joined #openstack-ironic | 09:38 | |
| *** rpittau has quit IRC | 09:39 | |
| *** rpittau has joined #openstack-ironic | 09:40 | |
| *** rpittau has quit IRC | 09:46 | |
| *** rpittau_ has joined #openstack-ironic | 09:46 | |
| *** rpittau_ is now known as rpittau | 09:50 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironic-inspector-client master: Add check-errors flag to Introspection https://review.openstack.org/618543 | 09:53 |
| iurygregory | tks for the review rpittau o/ | 09:53 |
| rpittau | no problem :) | 09:55 |
| *** moshele has quit IRC | 10:01 | |
| *** dnuka has joined #openstack-ironic | 10:03 | |
| dnuka | Good morning ironic o/ | 10:03 |
| dnuka | morning ironic etingof, iurygregory o/ | 10:04 |
| iurygregory | morning dnuka o/ | 10:04 |
| *** moshele has joined #openstack-ironic | 10:05 | |
| *** skazi has quit IRC | 10:12 | |
| openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Add support for the `UpdateService` resource https://review.openstack.org/613828 | 10:38 |
| *** zigo has joined #openstack-ironic | 10:50 | |
| *** diga has quit IRC | 10:56 | |
| *** adrianc has quit IRC | 10:56 | |
| *** arne_wiebalck_ has quit IRC | 10:59 | |
| *** arne_wiebalck_ has joined #openstack-ironic | 11:04 | |
| *** adrianc has joined #openstack-ironic | 11:07 | |
| *** arne_wiebalck_ has quit IRC | 11:35 | |
| *** arne_wiebalck_ has joined #openstack-ironic | 11:36 | |
| *** sambetts_ has joined #openstack-ironic | 11:38 | |
| *** v12aml has joined #openstack-ironic | 11:43 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add configurable Redfish client authentication https://review.openstack.org/603818 | 11:48 |
| *** sambetts_ is now known as sambetts | 11:55 | |
| sambetts | Morning folks | 11:55 |
| *** v12aml has quit IRC | 11:57 | |
| *** arne_wiebalck_ has quit IRC | 11:58 | |
| iurygregory | morning sambetts o/ | 12:04 |
| jroll | good morning iurygregory, sambetts, and anyone else lurking :) | 12:05 |
| iurygregory | jroll, o/ | 12:05 |
| iurygregory | jroll, if you have time could you review https://review.openstack.org/#/c/618543/ ? =) | 12:06 |
| patchbot | patch 618543 - python-ironic-inspector-client - Add check-errors flag to Introspection - 6 patch sets | 12:06 |
| *** v12aml has joined #openstack-ironic | 12:06 | |
| *** arne_wiebalck_ has joined #openstack-ironic | 12:07 | |
| *** jcoufal has joined #openstack-ironic | 12:07 | |
| *** e0ne has quit IRC | 12:09 | |
| jroll | iurygregory: done, just a couple small things ilya brought up :) | 12:13 |
| iurygregory | jroll, tks o/ | 12:13 |
| *** v12aml has quit IRC | 12:13 | |
| jroll | np | 12:13 |
| TheJulia | Good morning everyone | 12:16 |
| *** v12aml has joined #openstack-ironic | 12:16 | |
| dnuka | good morning TheJulia o/ | 12:17 |
| iurygregory | TheJulia o/ | 12:22 |
| *** v12aml has quit IRC | 12:24 | |
| *** arne_wiebalck_ has quit IRC | 12:30 | |
| *** bfournie has quit IRC | 12:33 | |
| sambetts | o/ TheJulia, jroll, good thanksgiving? | 12:36 |
| *** arne_wiebalck_ has joined #openstack-ironic | 12:38 | |
| jroll | sambetts: it was, thanks! | 12:39 |
| *** v12aml has joined #openstack-ironic | 12:41 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add download link apache configuration with mod_wsgi https://review.openstack.org/611497 | 12:45 |
| *** jaganathan has quit IRC | 12:48 | |
| *** v12aml has quit IRC | 12:49 | |
| dtantsur|afk | morning ironic! as a said on Friday, I'm on meetings this week, so will be on and off | 12:49 |
| dtantsur|afk | TheJulia: if you have a minute, would you mind checking https://review.openstack.org/617953 ? | 12:50 |
| patchbot | patch 617953 - ironic-specs - Allocation API - 6 patch sets | 12:50 |
| dnuka | good morning dtantsur|afk :) | 12:51 |
| TheJulia | dtantsur|afk: you've been busy :) | 12:54 |
| *** emine__ has quit IRC | 12:56 | |
| rpittau | hi TheJulia :) | 12:57 |
| *** v12aml has joined #openstack-ironic | 12:57 | |
| dtantsur|afk | TheJulia: oh yeah :) I already have plans for 2 more specs on this topic :D | 12:58 |
| *** MattMan_ has quit IRC | 12:58 | |
| *** MattMan_ has joined #openstack-ironic | 12:58 | |
| TheJulia | dtantsur|afk: I finally picked up the owner information stuff, just because of direct metal interaction stories. I think I've got it done on my laptop sans a release note... the next logical step would be to add a capability and allow the view of nodes to be restricted... but brains | 12:59 |
| *** dtantsur|afk is now known as dtantsur|mtg | 12:59 | |
| dtantsur|mtg | oh yeah | 12:59 |
| *** rh-jelabarre has joined #openstack-ironic | 12:59 | |
| TheJulia | dtantsur|mtg: that spec has me thinking of the resource reservation work, I guess nova would need to be taught with this to be able to understand that it can consume those resources when appropriate. | 13:02 |
| TheJulia | I'm not sure how that integrates in. That spec is huge | 13:03 |
| *** rh-jelabarre has quit IRC | 13:05 | |
| *** rh-jelabarre has joined #openstack-ironic | 13:05 | |
| TheJulia | dtantsur|mtg: would it also make sense to just begin publishing directly into placement? | 13:05 |
| dtantsur|mtg | TheJulia: that is one of my planned specs :) I just wanted to deal with Placement separately | 13:05 |
| TheJulia | Understood | 13:06 |
| TheJulia | I'm _not_ sure a spec is really needed for pubishing into placement since that is already fairly well defined elsewhere now and we would just be emulating that in ironic... unless were going to do other things then a spec would make sense. | 13:06 |
| dtantsur|mtg | TheJulia: well, get the proposed allocation API pluggable backends, make placement one of them, add optional placement publishing | 13:08 |
| TheJulia | oh, yeah, I guess that whole interface could house both sides of the logic | 13:08 |
| TheJulia | makes sense to me | 13:09 |
| *** arne_wiebalck_ has quit IRC | 13:09 | |
| dtantsur|mtg | I'd not mind writing a spec at least to clear up my own mind. assuming people will review it :) | 13:09 |
| *** jaypipes has quit IRC | 13:13 | |
| TheJulia | That could be good, even as a follow-up to the one I just approved it would make sense | 13:15 |
| TheJulia | since the work is so closely intertwined | 13:15 |
| *** v12aml has quit IRC | 13:15 | |
| * TheJulia needs to go make coffee before she goes and begins looking at email | 13:15 | |
| TheJulia | sambetts: Thanksgiving was nice to have dinner with friends. Of course, not actually on thanksgiving since I was metal tubing back from Berlin | 13:16 |
| dtantsur|mtg | thanks TheJulia | 13:17 |
| TheJulia | Anyone: If there are things your aware of that need my attention sooner rather than later, please let me know and I'll get a list started | 13:18 |
| dtantsur|mtg | TheJulia: releases? | 13:18 |
| sambetts | TheJulia: thats good, did you have a good time in Berlin post-summit? | 13:18 |
| *** robbbe has joined #openstack-ironic | 13:19 | |
| *** v12aml has joined #openstack-ironic | 13:20 | |
| *** trown|outtypewww is now known as trown | 13:27 | |
| TheJulia | dtantsur|mtg: ack | 13:27 |
| TheJulia | sambetts: I did. It got colder though. :( | 13:28 |
| dtantsur|mtg | TheJulia: did you place a sticker in brewdog? :) | 13:28 |
| TheJulia | dtantsur|mtg: of course I did | 13:28 |
| dtantsur|mtg | \o/ | 13:29 |
| dtantsur|mtg | I'll go check, y'know ;) | 13:29 |
| TheJulia | I have a photo somewhere on my phone :) | 13:29 |
| *** skazi has joined #openstack-ironic | 13:30 | |
| TheJulia | looks like the bifrost release was approved. Did we need other things released? | 13:30 |
| dtantsur|mtg | TheJulia: I wonder if we need intermediary releases for ironic and inspector (and other master stuffs) | 13:30 |
| openstackgerrit | Merged openstack/ironic-specs master: Allocation API https://review.openstack.org/617953 | 13:32 |
| dtantsur|mtg | \o/ | 13:32 |
| TheJulia | https://pbs.twimg.com/media/DsiYRsPV4AIonor.jpg | 13:34 |
| *** skazi has quit IRC | 13:35 | |
| TheJulia | dtantsur|mtg: wouldn't be a bad idea... at least that gives some reason to review all outstanding patches right now | 13:35 |
| dtantsur|mtg | indeed | 13:35 |
| jroll | ++ | 13:35 |
| jroll | (on the sticker) | 13:35 |
| TheJulia | Seems like we also have some backports on track to landfor bug fixes, makes sense to kind of just go down the list | 13:37 |
| dtantsur|mtg | yeah | 13:37 |
| TheJulia | so how would people feel if we changed the port address field to support guid? | 13:39 |
| openstackgerrit | Mark Goddard proposed openstack/tenks master: Fail nicely if ironic port lookup fails https://review.openstack.org/620048 | 13:40 |
| openstackgerrit | Mark Goddard proposed openstack/tenks master: Pass node and volume name prefixes to tenks_update_state https://review.openstack.org/620049 | 13:40 |
| openstackgerrit | Mark Goddard proposed openstack/tenks master: Fix skipping glance check if deploy images not defined https://review.openstack.org/620050 | 13:40 |
| openstackgerrit | Mark Goddard proposed openstack/tenks master: Use only the first 6 characters of the node name in link names https://review.openstack.org/620051 | 13:40 |
| openstackgerrit | Mark Goddard proposed openstack/tenks master: WIP: Changes to support standalone ironic https://review.openstack.org/620052 | 13:40 |
| TheJulia | can I get one more zuul familar set of eyes to look at https://review.openstack.org/#/c/613498/4 | 13:41 |
| patchbot | patch 613498 - ironic-inspector - Zuul migration - 4 patch sets | 13:41 |
| * dtantsur|mtg looks | 13:46 | |
| dtantsur|mtg | TheJulia: I think we're going to break ppl if we allow guid's.. but dunno. maybe some microversioning could help. | 13:46 |
| TheJulia | I kind of see it as just loosing the filter and extending the field | 13:47 |
| *** skazi has joined #openstack-ironic | 13:47 | |
| jroll | what's the use case for using the guid? | 13:47 |
| TheJulia | tl;dr there is a HPC operator that is working with a new networking fabric that ports are addressed by GUIDs, and they want to represent and manage this via ironic | 13:47 |
| jroll | interesting | 13:48 |
| dtantsur|mtg | TheJulia: somebody may be consuming ironic ports DB.. | 13:48 |
| TheJulia | they really shouldn't be talking to the db directly though :\ | 13:48 |
| jroll | hard to be opposed to this given the infiniband precedent | 13:49 |
| TheJulia | That is true and we can't really prevent from breaking those cases. | 13:49 |
| TheJulia | jroll: yeah, that was kind of where my mind was at when I was talking to them in berlin | 13:49 |
| dtantsur|mtg | I mean, API, not DB. sorry | 13:49 |
| jroll | I'd love a quick spec just to be able to discuss in an async fashion, but overall not opposed | 13:50 |
| jroll | I'd support a microversion, I think | 13:50 |
| dtantsur|mtg | something like mac = port.address; # try to do something with mac | 13:50 |
| * dtantsur|mtg woke up before 6am today to flight here, please pardon my vagueness :D | 13:50 | |
| TheJulia | On a neat side, they said they were going to develop their own ML2 driver to integrate into the controller system for the network fabirc | 13:50 |
| TheJulia | dtantsur|mtg: no worries | 13:50 |
| *** dmellado has quit IRC | 13:51 | |
| TheJulia | I'm all for it, but I stressed they need to come discuss this with the community, so hopefully in the next couple weeks such a discussion gets started | 13:51 |
| TheJulia | I too am missing a lot of details sadly | 13:52 |
| *** dmellado has joined #openstack-ironic | 13:53 | |
| jroll | sounds good :) | 13:54 |
| TheJulia | I feel like I'm going to need to hire a cat wisperer because for some reason they are both up, and both trying to be annoying.... | 13:55 |
| *** edleafe has joined #openstack-ironic | 13:57 | |
| openstackgerrit | Merged openstack/bifrost master: fix grammar issues in bifrost mission statement https://review.openstack.org/619959 | 14:00 |
| openstackgerrit | Merged openstack/bifrost master: omit the twice occured words in openstacksdk-addition-efec3878ef988295.yaml https://review.openstack.org/617424 | 14:02 |
| dnuka | dtantsur|mtg, if you have time could you review https://review.openstack.org/#/c/613828/ :) | 14:03 |
| patchbot | patch 613828 - sushy - Add support for the `UpdateService` resource - 14 patch sets | 14:03 |
| *** arne_wiebalck_ has joined #openstack-ironic | 14:05 | |
| vabada | hi ironic! Quick question: after merging https://review.openstack.org/#/c/567577/ the console gets disabled when a node gets unprovisioned, which makes total sense. I was wondering whether the console gets enabled anywhere when provisioning an instance, or it needs to be an admin who enables it... | 14:06 |
| patchbot | patch 567577 - ironic - Tear down console during unprovisioning (MERGED) - 5 patch sets | 14:06 |
| *** mmethot has joined #openstack-ironic | 14:06 | |
| TheJulia | vabada: I don't think we automatically re-enable it | 14:08 |
| TheJulia | So I think it does need to be an admin. What sort of challenge have you encountered? | 14:08 |
| vabada | TheJulia: basically, users deleting/recreating or rebuilding instances complain that they can't get the console after it, so we need to re-enable it | 14:09 |
| TheJulia | Interesting... | 14:10 |
| TheJulia | vabada: master version of nova being used? | 14:10 |
| vabada | TheJulia: queens | 14:12 |
| vabada | should be fixed (https://review.openstack.org/#/c/328157/)? Maybe some of our patches break it | 14:13 |
| patchbot | patch 328157 - nova - Ironic Driver: override get_serial_console() (MERGED) - 31 patch sets | 14:13 |
| TheJulia | vabada: I would set the conductors to debug mode and jiust double check that your not getting any exceptions raised, and that console_enabled is indeed being set to false in your deployment | 14:19 |
| TheJulia | Just to make sure ironic is doing the right thing. That nova code does look like it should be doing the right thing, but it does sound like something is not playing nice. :( | 14:20 |
| TheJulia | Looking at the current state of the master code for ironic, it should be good... Then again there may also be console interface issues (hence why debug logging output check just to be on the safe side) | 14:21 |
| vabada | TheJulia: thanks. I'll double check that. It's my initial guess since I enabled manually the console on a particular node and it was being disabled on instance deletion | 14:21 |
| vabada | TheJulia: I'll let you know my findings | 14:21 |
| *** jroll has quit IRC | 14:24 | |
| *** jroll has joined #openstack-ironic | 14:25 | |
| TheJulia | vabada: thanks | 14:32 |
| TheJulia | oh the fun of having a web browser crash in the middle of typing out a bug report | 14:32 |
| *** bnemec has joined #openstack-ironic | 14:33 | |
| *** baha has joined #openstack-ironic | 14:42 | |
| *** cdearborn has joined #openstack-ironic | 14:45 | |
| TheJulia | Approaching meeting time | 14:46 |
| *** dhellmann has joined #openstack-ironic | 14:48 | |
| iurygregory | \o/ meeting yay | 14:49 |
| *** stendulker has joined #openstack-ironic | 14:52 | |
| *** kaifeng has joined #openstack-ironic | 14:53 | |
| *** hjensas has quit IRC | 14:55 | |
| *** rpittau has quit IRC | 14:55 | |
| *** rloo has joined #openstack-ironic | 14:56 | |
| TheJulia | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon Nov 26 15:00:18 2018 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| TheJulia | o/ | 15:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| etingof | o/ | 15:00 |
| iurygregory | o/ | 15:00 |
| jroll | \o | 15:00 |
| stendulker | o/ | 15:00 |
| bdodd | o/ | 15:00 |
| dnuka | o/ | 15:00 |
| TheJulia | Good morning everyone! | 15:00 |
| TheJulia | Our agenda this week can be found on the wiki. | 15:00 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic | 15:00 |
| cdearborn | o/ | 15:00 |
| *** rpioso has joined #openstack-ironic | 15:00 | |
| kaiokmo | \o | 15:01 |
| TheJulia | #topic Announcements/Reminders | 15:01 |
| rpioso | o/ | 15:01 |
| *** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)" | 15:01 | |
| mgoddard | \o/ | 15:01 |
| kaifeng | o/ | 15:01 |
| TheJulia | #info Tenks is now under Ironic's governance! | 15:01 |
| TheJulia | \o/ | 15:01 |
| rloo | o/ | 15:02 |
| * dtantsur|mtg is lurking | 15:02 | |
| rloo | yah, tenk you for tenks | 15:02 |
| TheJulia | #info The OpenStack summit was very positive, and there was lots of discussion about the future being baremetal clouds. As such things might get a bit busier in here. :) | 15:02 |
| TheJulia | Does anyone else have an announcement or reminder that they would like to raise this week? | 15:03 |
| dnuka | Hey everyone o/ You may have seen me hanging out in the IRC. So, I just wanted to introduce myself to the ironic team :) My name is Dhanuka (he/him), and I'm from Sri Lanka. I'm your Outreachy intern for the December - March round. I'll be working with my mentors dtantsur and etingof, on the sushy project. Thanks for being so welcoming and helpful community! Looking forward to contribute and learn | 15:03 |
| dnuka | a lot :) | 15:03 |
| TheJulia | dnuka: Welcome! | 15:03 |
| iurygregory | welcome =) | 15:04 |
| rpioso | dnuka: Welcome to ironic! | 15:04 |
| dnuka | TheJulia , iurygregory thank you :) | 15:04 |
| TheJulia | Anyone else? | 15:04 |
| jroll | awesome, welcome dnuka :) | 15:05 |
| TheJulia | Well, I guess we can move on then | 15:05 |
| dnuka | thank you jroll :) | 15:05 |
| openstackgerrit | Merged openstack/sushy-tools master: Make external dependencies optional https://review.openstack.org/611859 | 15:05 |
| openstackgerrit | Merged openstack/sushy-tools master: Fix wrong `set_server_metadata` args https://review.openstack.org/618110 | 15:05 |
| TheJulia | #topic Review action items from previous meeting | 15:05 |
| *** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)" | 15:05 | |
| TheJulia | Looks like I never got around to sending out a doodle for clean steps | 15:06 |
| TheJulia | #action TheJulia to send out a doodle for clean/deploy steps visibility discussion | 15:06 |
| TheJulia | Nothing besides that | 15:06 |
| TheJulia | Moving on | 15:06 |
| TheJulia | #topic Review subteam status reports | 15:07 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:07 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:07 |
| TheJulia | Starting at line 165 | 15:07 |
| dnuka | thank you rpioso :) | 15:07 |
| TheJulia | \o/ looks like we've made a little progress on switching over our jobs to be more python3 and zuul native where possible | 15:09 |
| *** e0ne has joined #openstack-ironic | 15:09 | |
| rloo | TheJulia: is the TODO L202&203 now covered by Python3First? | 15:09 |
| rloo | TheJulia: actually, L204&205 too | 15:10 |
| TheJulia | rloo: Those are more contextual, I guess we can delete the lines. 206 is totally status informational | 15:10 |
| rloo | TheJulia: i was referring to the TODO above that :) | 15:11 |
| rloo | TheJulia: oh, sorry, theTODO is part of Python3 First. | 15:11 |
| TheJulia | We kind of mashed it all together for us to just get it done. | 15:11 |
| TheJulia | Yeah | 15:11 |
| TheJulia | no worries | 15:12 |
| rloo | Great, the enhanced checksum support is done! (L224) | 15:12 |
| TheJulia | hshiina: regarding UEFI first, I wonder if we could use/need a few more CI job or a new job that explicitly supports running UEFI | 15:12 |
| TheJulia | kaifeng: is there anything else that your aware of that needs to be done regarding the checksum support? Or do you consider it done? | 15:13 |
| kaifeng | I think it's done :) | 15:13 |
| rloo | TheJulia: wrt smartnic support (L277). was there a discussion at forum, with neutron folks on it? | 15:14 |
| TheJulia | rloo: extensive, I think from our standpoint our needs/required changes has largely been dialed back | 15:15 |
| rloo | TheJulia: dialed back --> that could be good? :) | 15:16 |
| TheJulia | I'll try and follow-up on that this week. Ive seen the patches have been revised in the last few days so there is hope they are already on track. | 15:16 |
| TheJulia | rloo: yes, very good. | 15:16 |
| rloo | :) | 15:16 |
| TheJulia | etingof: How goes the task execution work? | 15:16 |
| rloo | hmm, bug stats hasn't been updated since Nov 5. Do we care? | 15:17 |
| TheJulia | Looks like graphical consoles is going to need more attention :( | 15:17 |
| TheJulia | rloo: we've not had a meeting since nov 5th | 15:18 |
| dtantsur|mtg | I'm too busy, sorry | 15:18 |
| rloo | wow, how time flies! | 15:18 |
| dtantsur|mtg | I'll get back to bug stats after the current week of meetings | 15:18 |
| rloo | no worries dtantsur|mtg | 15:18 |
| TheJulia | dtantsur|mtg: thanks! | 15:18 |
| etingof | TheJulia, I have not worked on the tasks subsystem itself much lately, but I am quite optimistic concerning ipmitool killing patch - which should improve our most immediate problem | 15:18 |
| TheJulia | It does look like one idea seems to have more support than the other, at least when I was glancing through reviewes earlier this morning | 15:19 |
| *** rpittau has joined #openstack-ironic | 15:19 | |
| rloo | if it is blocked cuz of lack of feedback, should we schedule a time to discuss? eg next Monday's meeting? | 15:19 |
| rloo | or today if it is something that we can grok in 1-2 minutes... | 15:20 |
| TheJulia | mgoddard: dtantsur|mtg: If there are only minor items on the spec, would there be a possibility of revising/reviewing and getting that moving this week? | 15:20 |
| TheJulia | I think federation discussions likely need a resync from the summit as well. | 15:21 |
| mgoddard | TheJulia: for which spec? | 15:22 |
| * dtantsur|mtg the same question | 15:22 | |
| TheJulia | deploy templates | 15:22 |
| TheJulia | sorry | 15:22 |
| mgoddard | ah | 15:22 |
| dtantsur|mtg | I don't remember for sure, but I think nothing critical there | 15:23 |
| mgoddard | yeah, I can try to get an iteration up this week, if not early next week | 15:23 |
| TheJulia | if nothing is critical, could we just merge and approve the spec and revise it? | 15:23 |
| mgoddard | not sure how well I understand the consensus around the design changes, but I'll bring those up in the spec | 15:23 |
| TheJulia | (well, approve if we have consensus) | 15:24 |
| TheJulia | mgoddard: that is a good question | 15:24 |
| TheJulia | Looks like neutron event processing needs some eyes as well | 15:25 |
| TheJulia | But there is no change there from the last meeting. | 15:25 |
| dtantsur|mtg | I don't have time to revisit right now, sorry. I'd prefer mgoddard update it and we go from there. | 15:25 |
| *** munimeha1 has joined #openstack-ironic | 15:26 | |
| *** moshele has quit IRC | 15:26 | |
| mgoddard | I'll do that. should have some time on a train tomorrow | 15:26 |
| TheJulia | Okay, I'm just worried about the inherent cost of a -1 to reviews | 15:26 |
| TheJulia | mgoddard: thanks! | 15:26 |
| TheJulia | I'm done going down the list of priorities, is everyone else done and there are no questions seeking further clarification? | 15:27 |
| TheJulia | I guess we're good to move on... | 15:29 |
| TheJulia | #topic Deciding on priorities for the coming week | 15:29 |
| *** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:29 | |
| *** coreywright_ has quit IRC | 15:30 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic stable/rocky: Don't fail when node is in CLEANFAIL state https://review.openstack.org/620086 | 15:30 |
| TheJulia | In shuffling around some of the items, it doesn't look like we have too many outstanding patches that are in a shape to receive reviews and potentially be merged this week :\ | 15:33 |
| TheJulia | I've trimmed and cleaned up the list. Is there anything anyone feels should be on the list of items to review this week? | 15:34 |
| kaifeng | https://review.openstack.org/#/c/587698/ can use some eyes if nothing else | 15:35 |
| patchbot | patch 587698 - ironic-inspector-specs - Configurable introspection data backends - 7 patch sets | 15:35 |
| TheJulia | https://review.openstack.org/#/c/619683/ might be a cantidate as well | 15:36 |
| patchbot | patch 619683 - ironic - Expose conductors: db and rpc - 2 patch sets | 15:36 |
| kaifeng | there is another patch on the way, would be better if proposed | 15:37 |
| * etingof has quite a few pending patches against sushy-tools | 15:37 | |
| TheJulia | kaifeng: will you be re-ordering the patches then? | 15:37 |
| TheJulia | etingof: priority list of 3 please :) | 15:38 |
| bdodd | https://review.openstack.org/#/c/582314/ could use some eyes | 15:38 |
| patchbot | patch 582314 - ironic - Add BIOS interface to Redfish hardware type - 7 patch sets | 15:38 |
| kaifeng | no, this is the first patch, the second will be around api | 15:38 |
| rpioso | bdodd: I'll place mine on it. | 15:38 |
| TheJulia | kaifeng: ack, I went ahead and put that one on the list, and you can add the other once it is there | 15:38 |
| TheJulia | bdodd: Added | 15:39 |
| bdodd | rpioso TheJulia - thanks! | 15:39 |
| kaifeng | TheJulia thanks:) | 15:39 |
| TheJulia | Ohh, I'm going to add https://review.openstack.org/#/c/616968/ | 15:40 |
| patchbot | patch 616968 - ironic - Let neutron regenerate mac on port unbind - 4 patch sets | 15:40 |
| TheJulia | Since that is a bug fix | 15:40 |
| etingof | TheJulia, my humble list: https://review.openstack.org/#/c/616998/ https://review.openstack.org/#/c/616998/ https://review.openstack.org/#/c/618112/ | 15:41 |
| patchbot | patch 616998 - sushy-tools - Fix UEFI firmware path and connection attribute in... - 6 patch sets | 15:41 |
| patchbot | patch 616998 - sushy-tools - Fix UEFI firmware path and connection attribute in... - 6 patch sets | 15:41 |
| patchbot | patch 618112 - sushy-tools - Fix HTTP response code on errors - 2 patch sets | 15:41 |
| etingof | ah | 15:41 |
| TheJulia | I guess one didn't copy? | 15:41 |
| etingof | yep, hold on | 15:41 |
| etingof | TheJulia, https://review.openstack.org/#/c/614316/ | 15:42 |
| patchbot | patch 614316 - sushy-tools - Fix libvirt driver to handle domains by UUID - 6 patch sets | 15:42 |
| TheJulia | okay, added | 15:42 |
| TheJulia | That looks like a healthy list for this week! | 15:42 |
| *** coreywright_ has joined #openstack-ironic | 15:43 | |
| TheJulia | Everyone good to proceed to discussion? | 15:44 |
| rloo | ++ | 15:45 |
| TheJulia | #topic Discussion | 15:45 |
| *** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:45 | |
| TheJulia | Looks like we have a single discussion topic from tonyb this week | 15:45 |
| TheJulia | In regards to providing a path forward where a bmc does not support raw commands | 15:46 |
| TheJulia | Looks like https://review.openstack.org/#/c/616053/ has a +2 and 2x+1 | 15:46 |
| patchbot | patch 616053 - ironic - Add ipmi_do_timeout_disable to avoid problematic I... - 4 patch sets | 15:46 |
| TheJulia | and https://review.openstack.org/#/c/583488/ has a -1 | 15:46 |
| patchbot | patch 583488 - ironic - Continue to set boot device even if timeout config... - 2 patch sets | 15:46 |
| TheJulia | Seems like consensus has already been built around the first one | 15:46 |
| * TheJulia is unsure there needs to be much more of a discussion | 15:47 | |
| TheJulia | I'm going to go with what the consensus is behind for now, and we can go from there | 15:48 |
| jroll | seems like kaifeng is even in favor of taking the first | 15:48 |
| TheJulia | Anyone else have thoughts on this topic? | 15:48 |
| jroll | (last comment in the second, and a +1 on the first) | 15:48 |
| TheJulia | Yeah | 15:48 |
| jroll | yeah, no discussion needed imo | 15:48 |
| kaifeng | Yeah, I plans to base my patch on tonyb's | 15:48 |
| kaifeng | er, s/plans/plan/ | 15:48 |
| TheJulia | kaifeng: I was kind of guessing that would be the next step :) | 15:48 |
| TheJulia | sounds good to me | 15:48 |
| TheJulia | I guess we can move on to Open Discussion | 15:49 |
| TheJulia | #topic Open Discussion | 15:49 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:49 | |
| TheJulia | \o/ conductor group spec in nova merged | 15:49 |
| * TheJulia dances | 15:50 | |
| TheJulia | I don't really have anything else to discuss | 15:50 |
| dtantsur|mtg | \o/ | 15:50 |
| TheJulia | Just starting to go through the tons of things from the last two weeks | 15:51 |
| TheJulia | Does anyone have anything they wish to discuss? | 15:51 |
| * TheJulia guesses the answer is no :) | 15:52 | |
| TheJulia | Well everyone, Thank you! | 15:54 |
| rloo | CRICKETS | 15:54 |
| TheJulia | #endmeeting | 15:54 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-dev mailing list." | 15:54 | |
| openstack | Meeting ended Mon Nov 26 15:54:43 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:54 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-11-26-15.00.html | 15:54 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-11-26-15.00.txt | 15:54 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-11-26-15.00.log.html | 15:54 |
| *** stendulker has quit IRC | 15:57 | |
| *** Chaserjim has joined #openstack-ironic | 16:02 | |
| *** ijw has joined #openstack-ironic | 16:09 | |
| *** dnuka has quit IRC | 16:17 | |
| rloo | TheJulia: wrt https://review.openstack.org/#/c/613426/7, you want me to edit to address dmitry's comments, before approving? | 16:20 |
| patchbot | patch 613426 - ironic-python-agent - zuul-ify primary IPA jobs - 7 patch sets | 16:20 |
| openstackgerrit | Ruby Loo proposed openstack/ironic stable/queens: Don't fail when node is in CLEANFAIL state https://review.openstack.org/620099 | 16:21 |
| openstackgerrit | Ruby Loo proposed openstack/ironic stable/pike: Don't fail when node is in CLEANFAIL state https://review.openstack.org/620100 | 16:21 |
| *** baha has quit IRC | 16:24 | |
| *** hshiina has quit IRC | 16:24 | |
| *** hshiina has joined #openstack-ironic | 16:24 | |
| *** kaifeng has quit IRC | 16:24 | |
| *** gkadam has quit IRC | 16:27 | |
| *** hjensas has joined #openstack-ironic | 16:28 | |
| *** gyee has joined #openstack-ironic | 16:31 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironic-inspector-client master: Add check-errors flag to Introspection https://review.openstack.org/618543 | 16:34 |
| *** e0ne has quit IRC | 16:39 | |
| *** dsneddon has joined #openstack-ironic | 16:41 | |
| *** baha has joined #openstack-ironic | 16:45 | |
| *** ijw_ has joined #openstack-ironic | 16:46 | |
| *** ijw has quit IRC | 16:49 | |
| rpittau | bye good evening o/ | 16:49 |
| *** rpittau has quit IRC | 16:52 | |
| TheJulia | rloo: if you have braincells to do so go right ahead | 17:19 |
| TheJulia | rloo: I'm still catching up on email | 17:19 |
| TheJulia | surprisingly... I think I have only one thread left... but it also needs strong coffee first | 17:20 |
| rloo | TheJulia: I did not partake in lots of turkey dinners :) | 17:20 |
| rloo | TheJulia: https://review.openstack.org/#/c/613426/7/zuul.d/ironic-jobs.yaml | 17:20 |
| patchbot | patch 613426 - ironic-python-agent - zuul-ify primary IPA jobs - 7 patch sets | 17:20 |
| rloo | we can delete that comment at L3? | 17:20 |
| TheJulia | Yeah | 17:21 |
| rloo | TheJulia: thx, that I can do :) | 17:23 |
| TheJulia | \o/ | 17:23 |
| openstackgerrit | Ruby Loo proposed openstack/ironic-python-agent master: zuul-ify primary IPA jobs https://review.openstack.org/613426 | 17:32 |
| *** sambetts is now known as sambetts|afk | 17:36 | |
| *** derekh has quit IRC | 17:52 | |
| *** moshele has joined #openstack-ironic | 17:55 | |
| openstackgerrit | Merged openstack/ironic stable/rocky: Fix CPU count returned by introspection in Ironic iDRAC driver https://review.openstack.org/616365 | 17:57 |
| *** sri_ has quit IRC | 17:59 | |
| *** srini_ has joined #openstack-ironic | 17:59 | |
| *** dougsz has quit IRC | 17:59 | |
| *** robbbe has quit IRC | 18:07 | |
| *** ijw_ has quit IRC | 18:07 | |
| *** kaiokmo has quit IRC | 18:38 | |
| *** moshele has quit IRC | 18:40 | |
| *** e0ne has joined #openstack-ironic | 18:41 | |
| *** adrianc has quit IRC | 18:46 | |
| *** hjensas has quit IRC | 18:50 | |
| *** tssurya has quit IRC | 18:51 | |
| *** eandersson has joined #openstack-ironic | 18:54 | |
| *** bfournie has joined #openstack-ironic | 19:58 | |
| *** ijw has joined #openstack-ironic | 20:04 | |
| tonyb | Thanks TheJulia. | 20:14 |
| TheJulia | tonyb: make sense? Do you have what you need? or will have I should say? | 20:14 |
| tonyb | TheJulia: Yup I think I'm good. I'll address the review feedback today and I *think* that will clear the path | 20:16 |
| TheJulia | Okay! | 20:16 |
| *** robbbe has joined #openstack-ironic | 20:19 | |
| *** BOKALDO has quit IRC | 20:19 | |
| *** moshele has joined #openstack-ironic | 20:35 | |
| *** arne_wiebalck_ has quit IRC | 20:37 | |
| *** moshele has quit IRC | 20:41 | |
| *** moshele has joined #openstack-ironic | 21:01 | |
| *** moshele has quit IRC | 21:15 | |
| *** moshele has joined #openstack-ironic | 21:19 | |
| *** e0ne has quit IRC | 21:21 | |
| *** e0ne has joined #openstack-ironic | 21:30 | |
| *** robbbe has quit IRC | 21:32 | |
| *** jtomasek has quit IRC | 21:39 | |
| *** moshele has quit IRC | 21:55 | |
| *** ijw has quit IRC | 22:07 | |
| *** baha has quit IRC | 22:08 | |
| *** ijw has joined #openstack-ironic | 22:09 | |
| *** sambetts|afk has quit IRC | 22:14 | |
| *** bfournie has quit IRC | 22:17 | |
| openstackgerrit | Merged openstack/ironic stable/rocky: Reuse Redfish sessions https://review.openstack.org/616242 | 22:18 |
| openstackgerrit | Merged openstack/ironic stable/queens: Reuse Redfish sessions https://review.openstack.org/616245 | 22:18 |
| *** sambetts_ has joined #openstack-ironic | 22:19 | |
| *** ijw has quit IRC | 22:21 | |
| *** e0ne has quit IRC | 22:25 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Allow erasing metadata from disk partitions https://review.openstack.org/600139 | 22:47 |
| openstackgerrit | Merged openstack/ironic-inspector master: spelling error https://review.openstack.org/619842 | 22:47 |
| *** ijw has joined #openstack-ironic | 22:53 | |
| *** rcernin has joined #openstack-ironic | 23:01 | |
| *** ijw has quit IRC | 23:20 | |
| *** ijw has joined #openstack-ironic | 23:21 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add "owner" information field https://review.openstack.org/620186 | 23:27 |
| *** cdearborn has quit IRC | 23:28 | |
| *** munimeha1 has quit IRC | 23:30 | |
| *** ijw has quit IRC | 23:35 | |
| *** jcoufal has quit IRC | 23:35 | |
| openstackgerrit | Merged openstack/ironic-inspector-specs master: Configurable introspection data backends https://review.openstack.org/587698 | 23:43 |
| *** ijw has joined #openstack-ironic | 23:44 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Mark several tests to not test cleaning https://review.openstack.org/614000 | 23:46 |
| *** ijw has quit IRC | 23:46 | |
| *** ijw has joined #openstack-ironic | 23:46 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-lib master: zuul-ify ironic-lib jobs https://review.openstack.org/613453 | 23:50 |
| *** gyee has quit IRC | 23:52 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
