| *** hwoarang has quit IRC | 00:17 | |
| *** hwoarang has joined #openstack-ironic | 00:19 | |
| jiapei | Good morning Ironicers | 01:34 |
|---|---|---|
| *** tiendc has joined #openstack-ironic | 01:46 | |
| *** openstackgerrit has joined #openstack-ironic | 02:38 | |
| openstackgerrit | melissaml proposed openstack/bifrost master: Update devel info: mailing list https://review.openstack.org/623475 | 02:38 |
| *** dnuka has joined #openstack-ironic | 04:29 | |
| dnuka | good morning everyone :) | 04:30 |
| *** hwoarang has quit IRC | 04:34 | |
| *** hwoarang has joined #openstack-ironic | 04:36 | |
| *** zzzeek has quit IRC | 04:41 | |
| *** zzzeek has joined #openstack-ironic | 04:41 | |
| *** Chaserjim has joined #openstack-ironic | 05:01 | |
| *** Chaserjim has joined #openstack-ironic | 05:02 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Expose conductors: api-ref https://review.openstack.org/622862 | 05:54 |
| openstackgerrit | Kaifeng Wang proposed openstack/python-ironic-inspector-client master: Update tox.ini to support Python 3.x unit tests https://review.openstack.org/615794 | 06:06 |
| *** dnuka has quit IRC | 06:30 | |
| *** dnuka has joined #openstack-ironic | 06:37 | |
| openstackgerrit | jiapei proposed openstack/ironic master: Although previous XClarity driver passes the unit test, we find that getting boot order function test fails in the 3rd party CI, which will throw an exception of "Unsupported boot device". After checking with the code, we find that a boot device mapping f https://review.openstack.org/624000 | 06:42 |
| openstackgerrit | jiapei proposed openstack/ironic master: Fix XClarity driver management defect https://review.openstack.org/624000 | 06:47 |
| *** moshele has joined #openstack-ironic | 06:53 | |
| openstackgerrit | jiapei proposed openstack/ironic master: Enroll XClarity machines in Ironic's devstack setting https://review.openstack.org/624003 | 07:11 |
| *** rcernin has quit IRC | 07:23 | |
| *** moshele has quit IRC | 07:26 | |
| *** moshele has joined #openstack-ironic | 07:32 | |
| *** gkadam has joined #openstack-ironic | 07:52 | |
| *** rpittau has joined #openstack-ironic | 08:27 | |
| rpittau | good morning ironic o/ | 08:27 |
| openstackgerrit | jiapei proposed openstack/ironic master: Fix XClarity driver management defect https://review.openstack.org/624000 | 08:27 |
| jiapei | Morning rpittau | 08:27 |
| dnuka | morning rpittau :) | 08:29 |
| rpittau | hey jiapei dnuka :) | 08:29 |
| *** adrianc has quit IRC | 08:31 | |
| *** adrianc has joined #openstack-ironic | 08:43 | |
| *** jtomasek has joined #openstack-ironic | 08:57 | |
| *** iurygregory has joined #openstack-ironic | 09:05 | |
| iurygregory | good morning everyone o/ | 09:05 |
| etingof | o/ Ironmen | 09:09 |
| rpittau | hi iurygregory :) | 09:09 |
| dnuka | morning etingof , iurygregory :) | 09:09 |
| iurygregory | rpittau, yo! | 09:09 |
| iurygregory | dnuka, o/ | 09:09 |
| iurygregory | etingof, your pto starts today no? XD | 09:10 |
| rpittau | hey etingof :) | 09:10 |
| etingof | iurygregory, apparently, tomorrow! I somehow confused the dates | 09:11 |
| etingof | rpittau, dnuka o/ | 09:11 |
| iurygregory | etingof, oh thats sad =( | 09:11 |
| *** dougsz has joined #openstack-ironic | 09:11 | |
| etingof | well, one more dates to iron out the bugs | 09:11 |
| iurygregory | yeah \o/ | 09:11 |
| iurygregory | im still trying to figure out the second part of a bug XD | 09:12 |
| etingof | is it a two-piece bug? | 09:12 |
| iurygregory | yeah | 09:13 |
| iurygregory | first part done | 09:13 |
| iurygregory | second is given headache haha | 09:13 |
| *** S4ren has joined #openstack-ironic | 09:14 | |
| etingof | let me know if you want to spread the headache | 09:15 |
| *** tssurya has joined #openstack-ironic | 09:17 | |
| iurygregory | etingof, tks o/ | 09:17 |
| iurygregory | i will give a little more try here if i dont figure out till lunch i will ping you =) | 09:18 |
| etingof | sure| | 09:18 |
| *** leakypipes has quit IRC | 09:19 | |
| * iurygregory grabs coffee | 09:19 | |
| *** derekh has joined #openstack-ironic | 09:31 | |
| openstackgerrit | Kaifeng Wang proposed openstack/python-ironicclient master: [WIP] Support for conductors exposed from API https://review.openstack.org/624032 | 09:32 |
| *** dougsz has quit IRC | 09:36 | |
| *** dougsz has joined #openstack-ironic | 09:41 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: OOB RAID implementation for ilo5 based HPE Proliant servers. https://review.openstack.org/555664 | 09:42 |
| *** robbbe has joined #openstack-ironic | 10:00 | |
| *** e0ne has joined #openstack-ironic | 10:03 | |
| *** betherly has joined #openstack-ironic | 10:22 | |
| *** priteau has joined #openstack-ironic | 10:28 | |
| *** e0ne has quit IRC | 10:30 | |
| *** e0ne has joined #openstack-ironic | 10:36 | |
| *** iurygregory is now known as iury|lunch | 10:59 | |
| *** tiendc has quit IRC | 11:19 | |
| *** dtantsur|afk is now known as dtantsur | 11:25 | |
| dtantsur | morning ironic | 11:25 |
| dnuka | morning dtantsur o/ | 11:28 |
| *** jtomasek has quit IRC | 11:34 | |
| jiapei | Hey morning dtantsur | 11:39 |
| dtantsur | \o | 11:39 |
| jiapei | dtantsur: I submitted 2 patches on xclarity driver, could you help have a review of it? | 11:40 |
| dtantsur | jiapei: links? | 11:49 |
| jiapei | dtantsur: They're here https://review.openstack.org/#/c/624000/ and https://review.openstack.org/#/c/624003/ | 11:50 |
| patchbot | patch 624000 - ironic - Fix XClarity driver management defect - 3 patch sets | 11:50 |
| patchbot | patch 624003 - ironic - Enroll XClarity machines in Ironic's devstack setting - 1 patch set | 11:50 |
| jiapei | 624000 is what we find during the CI test, and 624003 is to enable xclarity in devstack | 11:51 |
| dtantsur | jiapei: some comments/questions on the former, +2 on the latter | 11:54 |
| *** e0ne has quit IRC | 11:58 | |
| jiapei | dtantsur: Ah, nice findings, let me reply it and have a new patch | 11:59 |
| *** e0ne has joined #openstack-ironic | 12:00 | |
| *** e0ne has quit IRC | 12:00 | |
| *** jtomasek has joined #openstack-ironic | 12:17 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Update tox.ini to support Python 3.x unit tests https://review.openstack.org/615794 | 12:20 |
| *** iury|lunch is now known as iurygregory | 12:20 | |
| iurygregory | morning dtantsur o/ | 12:20 |
| dtantsur | \o | 12:21 |
| *** tssurya has quit IRC | 12:41 | |
| arne_wiebalck | Hi Ironic! | 12:44 |
| arne_wiebalck | TheJulia: dtantsur: As discussed, I've converted the doc I had on our software RAID prototype implementation into a spec. Could you quickly point me to where this should go in the ironic-specs repo? 11.1? Or should that go somewhere else? | 12:45 |
| dtantsur | arne_wiebalck: all new specs go into the "approved" directory | 12:45 |
| dtantsur | then a symlink should be created from "not-implemented" to it | 12:45 |
| dtantsur | we'll move the spec to the right directory when the code is done | 12:46 |
| arne_wiebalck | dtantsur: Ok, got it! | 12:46 |
| arne_wiebalck | dtantsur: Thanks | 12:46 |
| *** rh-jelabarre has joined #openstack-ironic | 12:47 | |
| dtantsur | np | 12:52 |
| *** robbbe has quit IRC | 12:56 | |
| *** jistr is now known as jistr|medchk | 12:57 | |
| *** robbbe has joined #openstack-ironic | 13:00 | |
| *** robbbe has quit IRC | 13:00 | |
| *** robbbe has joined #openstack-ironic | 13:01 | |
| *** MattMan_ has quit IRC | 13:01 | |
| *** robbbe has quit IRC | 13:01 | |
| *** MattMan_ has joined #openstack-ironic | 13:01 | |
| *** robbbe has joined #openstack-ironic | 13:03 | |
| *** robbbe has joined #openstack-ironic | 13:04 | |
| *** robbbe has quit IRC | 13:04 | |
| *** robbbe has joined #openstack-ironic | 13:07 | |
| *** robbbe has joined #openstack-ironic | 13:08 | |
| *** robbbe has quit IRC | 13:08 | |
| *** jtomasek_ has joined #openstack-ironic | 13:14 | |
| *** jtomasek has quit IRC | 13:15 | |
| *** jtomasek_ is now known as jtomasek | 13:19 | |
| *** priteau has quit IRC | 13:23 | |
| *** dtrainor has quit IRC | 13:34 | |
| jroll | morning everyone | 13:45 |
| iurygregory | morning jroll | 13:46 |
| rpittau | hi jroll :) | 13:47 |
| dtantsur | morning jroll | 13:47 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Build UEFI-only ISO for UEFI boot https://review.openstack.org/624084 | 13:47 |
| dnuka | morning jroll :) | 13:49 |
| *** priteau has joined #openstack-ironic | 13:50 | |
| *** baha has joined #openstack-ironic | 13:51 | |
| *** jistr|medchk is now known as jistr | 13:56 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client master: Zuul migration https://review.openstack.org/613216 | 13:56 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Build ISO out of EFI system partition image https://review.openstack.org/620376 | 13:57 |
| *** dtrainor has joined #openstack-ironic | 13:59 | |
| *** rloo has joined #openstack-ironic | 14:03 | |
| *** rh-jelabarre has quit IRC | 14:06 | |
| *** gaotan has quit IRC | 14:11 | |
| *** gaotan has joined #openstack-ironic | 14:11 | |
| *** rh-jelabarre has joined #openstack-ironic | 14:12 | |
| *** hjensas has quit IRC | 14:13 | |
| *** e0ne has joined #openstack-ironic | 14:15 | |
| *** openstackstatus has joined #openstack-ironic | 14:17 | |
| *** ChanServ sets mode: +v openstackstatus | 14:17 | |
| openstackgerrit | jiapei proposed openstack/ironic master: Fix XClarity driver management defect https://review.openstack.org/624000 | 14:19 |
| *** gkadam has quit IRC | 14:24 | |
| TheJulia | good morning everyone | 14:31 |
| rpittau | morning TheJulia :) | 14:31 |
| dtantsur | morning TheJulia | 14:31 |
| iurygregory | morning TheJulia o/ | 14:32 |
| dnuka | morning TheJulia :) | 14:32 |
| TheJulia | Everyone ready for a meeting in a half hour? | 14:33 |
| iurygregory | yeah | 14:35 |
| dnuka | TheJulia: thank you very much for the review :) | 14:41 |
| * etingof is meeting-hungry as always! good morning, TheJulia o/ | 14:42 | |
| dnuka | etingof: thank you very much for the review :) | 14:46 |
| TheJulia | etingof: do you think https://review.openstack.org/#/c/620376/ is ready for people to review this week? | 14:49 |
| patchbot | patch 620376 - ironic - Build ISO out of EFI system partition image - 3 patch sets | 14:49 |
| *** baha has quit IRC | 14:50 | |
| TheJulia | I guess and https://review.openstack.org/#/c/624084/ | 14:50 |
| patchbot | patch 624084 - ironic - Build UEFI-only ISO for UEFI boot - 1 patch set | 14:50 |
| TheJulia | etingof: 3 preferred sushy-tools patches for this week? | 14:54 |
| etingof | TheJulia, I think my EFI patches are ready for review | 14:54 |
| TheJulia | etingof: could you put them in the "for the next week" section on the etherpad? | 14:56 |
| etingof | TheJulia, I have both sushy and sushy-tools patches - what's my allowance: 3 or 6 patches? ;) | 14:56 |
| *** kaifeng has joined #openstack-ironic | 14:57 | |
| TheJulia | eh, make it six? | 14:57 |
| *** stendulker has joined #openstack-ironic | 14:57 | |
| TheJulia | The number of small patches out there are many, would be good to get some eyes on them | 14:57 |
| etingof | TheJulia, sushy: https://review.openstack.org/#/c/621663/ https://review.openstack.org/#/c/623028/ | 14:57 |
| patchbot | patch 621663 - sushy - Add System<->Manager linkage - 4 patch sets | 14:57 |
| patchbot | patch 623028 - sushy - Add Chassis<->ComputerSystem/Manager linkage - 2 patch sets | 14:57 |
| TheJulia | dtantsur: is there anything you think that is a must-land into ironic 12.0? | 14:58 |
| dtantsur | must? probably not | 14:58 |
| TheJulia | I think we could tag the head of master likely as soon as the ipmi option merges | 14:58 |
| dtantsur | TheJulia: maybe the xclarity fixes? | 14:58 |
| etingof | TheJulia, sushy-tools: https://review.openstack.org/614456 https://review.openstack.org/614316 https://review.openstack.org/612758 | 14:58 |
| patchbot | patch 614456 - sushy-tools - Redirect to UUID URLs - 9 patch sets | 14:58 |
| patchbot | patch 614316 - sushy-tools - Fix libvirt driver to handle domains by UUID - 11 patch sets | 14:58 |
| patchbot | patch 612758 - sushy-tools - Add memoization to expensive emulator calls - 13 patch sets | 14:58 |
| TheJulia | Oh, there was that one... well kind of two | 14:58 |
| TheJulia | dtantsur: I'd kind of prefer to not wait on xclarity interface fixes, but maybe I should go look at that second patch | 14:59 |
| kaifeng | Hi rloo, just saw your comment | 14:59 |
| dtantsur | TheJulia: I'm fine with not waiting | 14:59 |
| rloo | kaifeng: hi | 14:59 |
| kaifeng | do you have suggestions on the ipmi_disable_timeout? | 14:59 |
| *** cdearborn has joined #openstack-ironic | 14:59 | |
| kaifeng | I think it's not too late if there is a need to update | 14:59 |
| rloo | kaifeng: didn't i comment? | 14:59 |
| kaifeng | https://review.openstack.org/#/c/583488 | 15:00 |
| patchbot | patch 583488 - ironic - Introduce configuration option [ipmi]ipmi_disable_... - 5 patch sets | 15:00 |
| *** chris_j has joined #openstack-ironic | 15:00 | |
| * TheJulia is confused, although I just pushed the button that one a little while ago | 15:00 | |
| TheJulia | Anyway, Meeting time! | 15:00 |
| TheJulia | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon Dec 10 15:00:31 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 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| jroll | \o | 15:00 |
| rloo | kaifeng: the only one that can't be modified easilyis the name of the config option. | 15:00 |
| kaifeng | rloo the name of ipmi_disable_timeout took from the other patch | 15:00 |
| iurygregory | o/ | 15:00 |
| kaifeng | o/ | 15:00 |
| dnuka | o/ | 15:00 |
| bdodd | o/ | 15:00 |
| rpittau | o/ | 15:00 |
| etingof | \o | 15:00 |
| dtantsur | o/ | 15:00 |
| rloo | o/ | 15:00 |
| rpioso | o/ | 15:00 |
| TheJulia | Good morning everyone! | 15:01 |
| TheJulia | Looks like we have a very light agenda this morning. | 15:01 |
| TheJulia | Our agenda, as always can be found on the wiki. | 15:01 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| stendulker | o/ | 15:01 |
| TheJulia | #topic Announcements/Reminders | 15:01 |
| *** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)" | 15:01 | |
| TheJulia | #info TheJulia is intending to cut a release of ironic 12.0.0 this week | 15:01 |
| TheJulia | Anyone have anything they would like to announce this morning? | 15:02 |
| cdearborn | \o | 15:02 |
| dtantsur | I'll be on PTO starting next week and till January | 15:03 |
| openstackgerrit | Merged openstack/bifrost master: Update devel info: mailing list https://review.openstack.org/623475 | 15:03 |
| rloo | happy holidaze dtantsur! | 15:03 |
| * etingof will be on vacation till next year | 15:03 | |
| dtantsur | thnx :) | 15:03 |
| TheJulia | dtantsur: awesome | 15:03 |
| * TheJulia should do this same vacation thing... | 15:04 | |
| rpittau | have fun dtantsur and etingof :) | 15:04 |
| iurygregory | everybody should lol | 15:04 |
| dnuka | :) | 15:04 |
| TheJulia | We are winding down nicely for the end of the year :) | 15:04 |
| rloo | TheJulia: the release of ironic -- any ohter ironic-related projects also getting a release? | 15:05 |
| rajinir | o/ | 15:05 |
| TheJulia | rloo: likely a minor for sushy, rev for bifrost. I did IPA like two weeks ago | 15:05 |
| rloo | great. thx TheJulia | 15:05 |
| dtantsur | maybe stable releases? | 15:05 |
| TheJulia | kaifeng: any thoughts on inspector? | 15:05 |
| iurygregory | maybe python-ironic-inspector-client will need? | 15:05 |
| rloo | did we ever do stable release for the upgrade bug? | 15:06 |
| TheJulia | rloo: I _think_, I'd have to check | 15:06 |
| TheJulia | I'll look at the stable branches and do the appropriate as well | 15:06 |
| iurygregory | we add the check-errors flag to the client, not sure if will be necessary a release =) | 15:06 |
| dtantsur | iurygregory++ | 15:06 |
| TheJulia | release all the things \o/ | 15:06 |
| rloo | release often! | 15:07 |
| iurygregory | yay | 15:07 |
| TheJulia | Anyway, since we have no action items this week, lets proceed to status updates | 15:07 |
| kaifeng | TheJulia no outstanding issue in the inspector | 15:07 |
| 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 233 | 15:08 |
| TheJulia | dtantsur: re federation, you were thinking of trying to sync after the first of the year right? | 15:08 |
| dtantsur | yep | 15:08 |
| *** priteau has quit IRC | 15:10 | |
| TheJulia | dtantsur: same for ideas regarding possible further process splitting right? | 15:10 |
| TheJulia | or the capability to launch smaller portions of the conductor is the way I should put it | 15:10 |
| dtantsur | indeed | 15:10 |
| TheJulia | How does everyone feel about doing a virtual midcycle in janurary? | 15:11 |
| * iurygregory never did, +1 for the idea =) | 15:11 | |
| dtantsur | ++ (esp. mid-January) | 15:11 |
| * jroll probably won't be around, so no feelings | 15:11 | |
| * jiapei +1 | 15:12 | |
| * etingof +1 | 15:12 | |
| TheJulia | I need to check my schedule because I think mid-January I think I'll have some metal tubing | 15:12 |
| TheJulia | I guess that is an action item for me, to setup a doodle poll for us to figure out days | 15:12 |
| iurygregory | yeah =) | 15:13 |
| TheJulia | #action TheJulia to create virtual midcycle doodle poll | 15:13 |
| dtantsur | yeah, end of January is devconf.cz, then FOSDEM | 15:13 |
| TheJulia | re getting CI jobs changed over, I've been really struggling with getting the n-g-s and networking baremetal jobs changed over to use zuul based jobs and python3. If anyone wants to lend a hand, it would be appreciated | 15:13 |
| iurygregory | i can try to help TheJulia o/ | 15:14 |
| TheJulia | iurygregory: okay, I'll try and sync up with you after the meeting to chat about them | 15:15 |
| TheJulia | Everyone good with status updates for this week? | 15:15 |
| * rpittau can give a hand as well | 15:15 | |
| iurygregory | sure =) | 15:15 |
| TheJulia | rpittau: much appreciated as well | 15:15 |
| * TheJulia takes silence as we can proceed to discussing some priorities for the next week | 15:16 | |
| * TheJulia hears crickets | 15:17 | |
| TheJulia | #topic Deciding on priorities for the coming week | 15:17 |
| *** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:17 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:17 |
| TheJulia | Starting around line 113 | 15:18 |
| TheJulia | Looking at the list from last week, we got a lot merged | 15:18 |
| TheJulia | \o/ | 15:18 |
| * TheJulia proceeds with removing merged things | 15:18 | |
| *** mjturek has joined #openstack-ironic | 15:19 | |
| TheJulia | Anyone aware of items that need reviews this week? | 15:20 |
| * iurygregory still working in push some patchs =) | 15:21 | |
| kaifeng | https://review.openstack.org/#/c/622862/ | 15:21 |
| patchbot | patch 622862 - ironic - Expose conductors: api-ref - 2 patch sets | 15:21 |
| kaifeng | i guess if we need to cut a release, this one may need to have? | 15:21 |
| * etingof puts his Christmas wishes on the etherpad | 15:21 | |
| jiapei | Also this one https://review.openstack.org/#/c/624000/ | 15:21 |
| patchbot | patch 624000 - ironic - Fix XClarity driver management defect - 4 patch sets | 15:21 |
| jiapei | which fix the xclarity driver defect | 15:22 |
| TheJulia | kaifeng: it can land after since it is really for the api reference website which publishes master | 15:22 |
| kaifeng | but it's rebased on the owner patch atm | 15:22 |
| TheJulia | etingof: +++ | 15:22 |
| *** moshele has quit IRC | 15:22 | |
| kaifeng | oh, that's fine | 15:22 |
| dtantsur | looks good | 15:22 |
| *** coreywright has quit IRC | 15:23 | |
| TheJulia | jiapei: I added that one to the vendor priorites list | 15:24 |
| jiapei | TheJulia: Great :) | 15:24 |
| TheJulia | jiapei: line 179 | 15:24 |
| TheJulia | I'm going to remove smartnic support from the review list as the author is still seeking feedback it seems | 15:25 |
| TheJulia | How does everyone feel about the list? | 15:25 |
| TheJulia | I suspect dtantsur's looks good comment was the list | 15:26 |
| etingof | TheJulia, I think we should better replace the last patch for sushy-tools with this one -- https://review.openstack.org/#/c/614456/ | 15:27 |
| patchbot | patch 614456 - sushy-tools - Redirect to UUID URLs - 9 patch sets | 15:27 |
| dtantsur | TheJulia: it was about the list, yes | 15:27 |
| etingof | because they are lined up that way | 15:27 |
| TheJulia | etingof: Oh, good catch, please update then | 15:27 |
| * TheJulia thought that one has been approved... | 15:28 | |
| etingof | axk | 15:28 |
| TheJulia | Anyway, Time to proceed onward I guess | 15:28 |
| TheJulia | #topic discussion | 15:29 |
| *** openstack changes topic to "discussion (Meeting topic: ironic)" | 15:29 | |
| TheJulia | One item, regarding CI job links. rajinir do you have an ETA to when links being posted will be valid again? | 15:29 |
| rajinir | The website was down | 15:30 |
| TheJulia | looks like xclarity's log server is down as well, the status notes indicate it is being worked on | 15:30 |
| rajinir | It is up now, the logs are being synched now | 15:30 |
| TheJulia | \o/ | 15:30 |
| TheJulia | rajinir: Awesome thanks! | 15:30 |
| TheJulia | I guess with that, we can move to open discussion. \o/ | 15:30 |
| TheJulia | #topic Open Discussion | 15:30 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:30 | |
| kaifeng | i have one just before the meeting :) | 15:31 |
| kaifeng | https://review.openstack.org/#/c/583488 | 15:31 |
| etingof | rajinir should I recheck my patches to get proper links? | 15:31 |
| patchbot | patch 583488 - ironic - Introduce configuration option [ipmi]ipmi_disable_... - 5 patch sets | 15:31 |
| rajinir | etingof, give it an hr. | 15:31 |
| TheJulia | etingof: please use the dell syntax | 15:31 |
| TheJulia | etingof: if rajinir is okay with it | 15:31 |
| TheJulia | kaifeng: Yes, you said there was a comment from rloo but rloo seemed confused | 15:32 |
| kaifeng | wrt ipmi_disable_timeout, rloo thought disable_ipmi_timeout is more suitable there | 15:32 |
| rajinir | etingof,TheJulia, it must be just the logs, recheck may not be needed. I will make sure your patch links works after the sync | 15:32 |
| etingof | rajinir++ | 15:32 |
| rloo | kaifeng: i think i said 'disable_boot_timeout' ? | 15:32 |
| TheJulia | How does everyone feel about meetings later this month? Looking at the work we've accomplished recently we seem to be winding down for the year. I guess we'll see bug fixes and some minor feature work. Seems like we're going to be getting into another feature patch period for most of us | 15:32 |
| rloo | kaifeng: or maybe i mistyped. let me see. | 15:33 |
| TheJulia | I think ipmi_* is more consistent with other field names | 15:33 |
| rloo | kaifeng: ah yes. so the node's driver_info has 'ipmi_disable_timeout' which disables the 60-sec timeout for booting. It worries me that/if we add another timeout, it won't be clear what this disable-timeout is for. | 15:34 |
| *** sthussey has joined #openstack-ironic | 15:34 | |
| rloo | so i don't know if it makes sense (ie inconsistent) to name the config option 'disable_boot_timeout' so it is clearer. | 15:34 |
| rloo | and given that it is in the [ipmi] group, I see no reason to have 'ipmi' in the config option name (that is an aside though) | 15:35 |
| rloo | anyway, i was just wondering about it. | 15:35 |
| TheJulia | rloo: but the same name is also a driver_info name as well | 15:35 |
| rpittau | TheJulia, I believe if we skip next monday we're going directly to january 7 | 15:36 |
| *** coreywright has joined #openstack-ironic | 15:36 | |
| rloo | TheJulia: right, that's what i mentioned above. or if you mean the 'ipmi' part, i didn't have time to look, but i believe we have another vendor with driver_info stuff and their config options omit the prefix (i forgot which vendor) | 15:36 |
| kaifeng | that was the initial thought, yeah, be consistent to some extend | 15:36 |
| rloo | i haven't been paying much attention, don't know how long we've had the driver_info name -- is it possible/easy to change both. and/or are folks ok with existing name? This is just my opinion. | 15:37 |
| TheJulia | I'm okay with the existing name, I do concur the descriptive text could use a slight little work in follow-up | 15:38 |
| stendulker | I think rloo's suggested name looks better. The driver_info name was also recently introduced. | 15:39 |
| stendulker | driver_info anme was introduced as part of https://review.openstack.org/#/c/616053 | 15:39 |
| patchbot | patch 616053 - ironic - Add ipmi_disable_timeout to avoid problematic IPMI... (MERGED) - 7 patch sets | 15:39 |
| TheJulia | I feel like we've already committed to that, and the only thing kaifeng's patch really does is add it so it can be conductor default | 15:42 |
| TheJulia | via ironic.conf | 15:42 |
| rloo | wrt the prefix in the config option name. irmc I think has a bunch of config options that might be used instead of driver_info ones, the config options do not have 'irmc_' prefix. | 15:42 |
| TheJulia | stendulker: thanks for finding thta so quickly | 15:42 |
| rloo | [I also see that the driver_info descriptions for irmc do not mention the config options :-(] | 15:43 |
| * TheJulia is confused | 15:43 | |
| rloo | TheJulia: did i confuse you? | 15:43 |
| TheJulia | rloo: I think we're on a tangent | 15:44 |
| rloo | ? | 15:44 |
| rloo | i mentioned two issues with the naming of that configuration option. | 15:44 |
| kaifeng | i prefer to omit the ipmi_ prefix in the configuration options, open for any names (really not good at that :) | 15:44 |
| rloo | (did people want me to recap/rephrase?) | 15:46 |
| TheJulia | rloo: but your also trying to relate to something that I see in multiple drivers driver_info field options and some configuration options | 15:46 |
| TheJulia | rloo: please, start from the top | 15:46 |
| *** mmethot has quit IRC | 15:46 | |
| rloo | proposal: instead of the configuration option in the [ipmi] group being 'ipmi_disable_timeout', how about 'disable_boot_timeout'. Why? | 15:47 |
| TheJulia | kaifeng: we already merged the initial patch, I'm really confused why we're talking about changing the option name again | 15:47 |
| TheJulia | oh, just in ironic.conf? | 15:47 |
| rloo | 1. for config options that correspond to driver_info['vendor_xxx'], we don't tend to use the vendor prefix in the config option name, so we'd use just x. | 15:48 |
| TheJulia | we're not talking about anything else, correct? | 15:48 |
| rloo | 2. in this case, i feel that 'disable_timeout' isn't that specific, esp if in the future, we add another timeout for ipmi, so instead of 'disable_timeout' am thinking 'disable_boot_timeout'. | 15:48 |
| kaifeng | TheJulia technically, it's not merged yet, since rloo has concerns i think we could discuss if there is a conclusion to decide whether or not update it. | 15:48 |
| rloo | so 2 is inconsistent with the driver_info name 'ipmi_disable_timeout'. | 15:48 |
| rloo | 1 is reasonable/easy to change. unless folks really think the config option needs 'ipmi_' for some reason. | 15:49 |
| TheJulia | Okay, So then I concur can drop ipmi for the setting in ironic.conf | 15:50 |
| jroll | I do agree that disable_timeout is a confusing name, especially considering we already have a command_retry_timeout. I'd even be +1 on the driver_info option being ipmi_disable_boot_timeout for the same reason. | 15:51 |
| dtantsur | yeah | 15:51 |
| TheJulia | rloo: when your saying config option, I just want to make sure your purely talking about ironic.conf configuration options, and not driver_info configuration options | 15:51 |
| rloo | TheJulia: yup, 'config option' == ironic.conf options. | 15:51 |
| TheJulia | jroll: ditto | 15:51 |
| TheJulia | Okay, so that is where I was confused at because that was not crystal clear to me | 15:52 |
| rloo | and yes, if we could, i'd prefer changing the driver_info entry name as jroll suggests | 15:52 |
| *** bfournie has joined #openstack-ironic | 15:52 | |
| TheJulia | I think that is fine, if someone reaches out to tonyb first | 15:52 |
| TheJulia | err, but then I can't cut ironic 12.0 until this is worked out | 15:52 |
| jroll | changing something like this within a release is acceptable per stable policy, last I checked | 15:53 |
| TheJulia | I can chat with tonyb if he has concerns | 15:53 |
| jroll | we can always deprecate and remove later if we're anxious to release 12.0 | 15:54 |
| TheJulia | jroll: indeed, I think tony just needs a heads up. We've got three of us on the same page to rename the options | 15:54 |
| TheJulia | I think we can go ahead and put that patch up asap, and go from there | 15:54 |
| TheJulia | 5 minute warning | 15:55 |
| jroll | ok. | 15:55 |
| rloo | thx everyone and sorry for the confusion. | 15:55 |
| TheJulia | rloo: thank you | 15:55 |
| TheJulia | kaifeng: I removed my wf+1 on the patch | 15:56 |
| kaifeng | so the name in the driver_info would be ipmi_disable_boot_timeout, and the name in the conf would be disable_boot_timeout, right? | 15:56 |
| TheJulia | kaifeng: correct | 15:57 |
| kaifeng | TheJulia: no worries | 15:57 |
| jroll | ++ | 15:57 |
| kaifeng | i update the patch tomorrow, if that's is not asap, feel free to help :) | 15:57 |
| TheJulia | rpittau: you raised an awesome point that if we cancel next week's meeting, we won't reconvene until Janurary 7th. | 15:57 |
| TheJulia | kaifeng: I might, if I have time this morning | 15:58 |
| TheJulia | kaifeng: have a wonderful night's sleep | 15:58 |
| kaifeng | TheJulia: thanks :) | 15:58 |
| TheJulia | Re cancelling holiday meetings, I guess I'm okay with cancelling the 17th, 24th, and 1st. As long as we keep the priority list rolling for the purposes of visibility. Kind of like how we did ?last year? | 15:59 |
| rloo | ^^ 1st == 31st, right? | 15:59 |
| TheJulia | yeah, sorry | 15:59 |
| rloo | ++ good otherwise :) | 15:59 |
| jroll | +1 from me | 16:00 |
| rloo | do we need some place to record who is avail in case we have any emergencies and need eg +2s? | 16:00 |
| TheJulia | Well, I expect many new feature-ish patches to review in January :) | 16:00 |
| * jroll can be pinged on hangouts or email, I'll at least respond even if I can't help immediately | 16:00 | |
| TheJulia | I can also be as well, I don't think that will really be an issue | 16:01 |
| rloo | i know we've gone through other holidays in the past, my memory must be going ;) | 16:01 |
| TheJulia | seems we're good with cancelling the meetings and letting priorities roll forward as needed, emergency contacts are available if needed, but people will need to raise that visibility | 16:02 |
| TheJulia | I'll send an email after the meeting | 16:02 |
| rloo | thx TheJulia! (and time's up!) | 16:02 |
| TheJulia | Yup! | 16:02 |
| TheJulia | Thanks everyone! | 16:03 |
| jroll | thanks :) | 16:03 |
| iurygregory | tks o/ | 16:03 |
| TheJulia | Have a wonderful week, and if we don't chat again, see you all in the new year. | 16:03 |
| stendulker | thanks | 16:03 |
| rpittau | thanks :) | 16:03 |
| TheJulia | #endmeeting | 16:03 |
| *** 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." | 16:03 | |
| openstack | Meeting ended Mon Dec 10 16:03:22 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:03 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-12-10-15.00.html | 16:03 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-12-10-15.00.txt | 16:03 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-12-10-15.00.log.html | 16:03 |
| *** dnuka has quit IRC | 16:04 | |
| *** kaifeng has quit IRC | 16:06 | |
| rajinir | etingof, rpioso Dell CI logs should be available now. Please check | 16:07 |
| TheJulia | so I just added ironic-cores to my notification list | 16:07 |
| *** stendulker has quit IRC | 16:08 | |
| *** hjensas has joined #openstack-ironic | 16:08 | |
| rpioso | rajinir: It works for me \o/. Thank you! | 16:09 |
| etingof | rajinir, works for me as well, thank you! | 16:09 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Zuul migration https://review.openstack.org/613216 | 16:09 |
| etingof | rajinir, one minor thing is that ssl cert is expired two years ago ;) | 16:09 |
| rajinir | etingof, I will bring that up with IT. thanks | 16:13 |
| etingof | rajinir++ | 16:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Allocation API: database and RPC https://review.openstack.org/624119 | 16:14 |
| *** mmethot has joined #openstack-ironic | 16:17 | |
| TheJulia | okay, list sent | 16:21 |
| TheJulia | err, post sent to ml | 16:21 |
| *** munimeha1 has joined #openstack-ironic | 16:21 | |
| TheJulia | rpittau: iurygregory: do you guys want to jump on a hangout or something and discuss the networking-baremetal and networking-generic-switch job changes? | 16:22 |
| iurygregory | TheJulia, sure | 16:22 |
| *** mmethot has quit IRC | 16:24 | |
| TheJulia | iurygregory: rpittau http://bluejeans.com/u/jkreger/ | 16:24 |
| *** mmethot has joined #openstack-ironic | 16:26 | |
| TheJulia | https://review.openstack.org/614026 | 16:26 |
| patchbot | patch 614026 - networking-baremetal - Change networking-baremetal to zuulv3/python3 - 9 patch sets | 16:26 |
| TheJulia | https://review.openstack.org/#/c/613655/ | 16:27 |
| patchbot | patch 613655 - networking-generic-switch - Migrate n-g-s jobs to Zuulv3 - 17 patch sets | 16:27 |
| *** openstackgerrit has quit IRC | 16:35 | |
| *** gyee has joined #openstack-ironic | 16:47 | |
| *** codyhill has joined #openstack-ironic | 16:48 | |
| rpittau | bye good evening! o/ | 17:02 |
| *** rpittau has quit IRC | 17:03 | |
| *** hjensas has quit IRC | 17:03 | |
| *** munimeha1 has quit IRC | 17:14 | |
| *** dsneddon has quit IRC | 17:18 | |
| *** dsneddon has joined #openstack-ironic | 17:19 | |
| *** dtantsur is now known as dtantsur|afk | 17:21 | |
| dtantsur|afk | see you | 17:21 |
| *** e0ne has quit IRC | 17:22 | |
| *** e0ne has joined #openstack-ironic | 17:22 | |
| *** e0ne has quit IRC | 17:23 | |
| *** openstackgerrit has joined #openstack-ironic | 17:24 | |
| openstackgerrit | jiapei proposed openstack/ironic master: Fix XClarity driver management defect https://review.openstack.org/624000 | 17:24 |
| *** e0ne has joined #openstack-ironic | 17:26 | |
| *** derekh has quit IRC | 17:29 | |
| *** moshele has joined #openstack-ironic | 17:34 | |
| *** chris_j has left #openstack-ironic | 17:40 | |
| *** baha has joined #openstack-ironic | 17:44 | |
| *** moshele has quit IRC | 17:53 | |
| *** mbeierl has quit IRC | 18:00 | |
| *** dougsz has quit IRC | 18:03 | |
| *** S4ren has quit IRC | 18:08 | |
| *** e0ne has quit IRC | 18:09 | |
| *** e0ne has joined #openstack-ironic | 18:11 | |
| *** e0ne has quit IRC | 18:11 | |
| *** moshele has joined #openstack-ironic | 18:14 | |
| *** baha has quit IRC | 18:14 | |
| *** e0ne has joined #openstack-ironic | 18:34 | |
| TheJulia | o/ | 18:36 |
| *** baha has joined #openstack-ironic | 18:41 | |
| *** tssurya has joined #openstack-ironic | 18:41 | |
| *** tssurya has quit IRC | 18:53 | |
| *** jcoufal has joined #openstack-ironic | 19:01 | |
| *** e0ne has quit IRC | 19:06 | |
| *** hjensas has joined #openstack-ironic | 19:27 | |
| moshele | TheJulia: hi | 19:39 |
| TheJulia | o/ | 19:39 |
| TheJulia | moshele: I've not looked at your email yet, it has been a busy morning | 19:40 |
| moshele | TheJulia: I still trying to get hold of sam to schedule a sync | 19:40 |
| moshele | TheJulia: currently with no luck | 19:41 |
| TheJulia | :( | 19:42 |
| TheJulia | do you feel comfortable proceeding without updated input from sam? | 19:43 |
| moshele | TheJulia: I have the original proposal which sam didn't like passing the hostname and the port_id in the neutron port | 19:45 |
| moshele | TheJulia: I had some doubts with his approach, but I can say I didn't understand it | 19:47 |
| moshele | TheJulia: who do you think will be a good contact point on the neutron side? | 19:48 |
| moshele | TheJulia: the notes on the etherpad were kind of confusing to me and contracting what sam suggested | 19:48 |
| moshele | s/contracting/contradicting | 19:51 |
| TheJulia | moshele: lets try and get Miguel re-engaged and just go from there on that | 19:52 |
| moshele | TheJulia: ok | 19:53 |
| moshele | mlavalle: regarding the neutron support for baremetal with smart-nic https://review.openstack.org/#/c/582767/ and https://etherpad.openstack.org/p/BER-ironic-smartnics. can we do me TheJulia and you a sync meeting to see that we are aligned? | 19:56 |
| patchbot | patch 582767 - ironic-specs - Add Support for Smart NIC - 13 patch sets | 19:56 |
| moshele | TheJulia: the 11th at 1500 UTC works for miguel let me know if it work for you as well | 20:01 |
| *** mbeierl has joined #openstack-ironic | 20:19 | |
| *** e0ne has joined #openstack-ironic | 20:28 | |
| *** e0ne has quit IRC | 20:28 | |
| tonyb | TheJulia: renaming the option is fine and I can do that today (and rebase the ironic.conf chnage on top). It would be waaay better to release 12.0 with the new name as doign the deprecation dance would be a pain. | 20:43 |
| tonyb | my only caevat is that I can't actually test the rename on master ATM because my setup is working on ricky right now :( | 20:43 |
| TheJulia | tonyb: ack, I think I’m going to run to the store to get some stuff for dinner, and I’ll change the option | 20:44 |
| *** moshele has quit IRC | 20:46 | |
| *** moshele has joined #openstack-ironic | 20:58 | |
| *** moshele has quit IRC | 21:00 | |
| openstackgerrit | Merged openstack/ironic master: Enroll XClarity machines in Ironic's devstack setting https://review.openstack.org/624003 | 21:01 |
| openstackgerrit | Merged openstack/ironic master: spelling error https://review.openstack.org/623619 | 21:01 |
| openstackgerrit | Merged openstack/ironic stable/queens: Ensure we unbind flat network ports and clear BM mac addresses https://review.openstack.org/621400 | 21:02 |
| tonyb | TheJulia: Thanks! | 21:05 |
| *** cdearborn has quit IRC | 21:15 | |
| *** jtomasek has quit IRC | 21:32 | |
| *** bfournie has quit IRC | 21:33 | |
| *** baha has quit IRC | 21:47 | |
| *** markguz_ has joined #openstack-ironic | 21:55 | |
| *** mjturek has quit IRC | 22:03 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Introduce configuration option [ipmi]ipmi_disable_timeout https://review.openstack.org/583488 | 22:04 |
| TheJulia | tonyb: jroll: rloo: ^^^ | 22:05 |
| tonyb | TheJulia: your rockage is supreme! | 22:14 |
| tonyb | TheJulia: thanks! | 22:14 |
| TheJulia | :) | 22:18 |
| TheJulia | does that mean it is beer o'clock? | 22:19 |
| *** markguz_ has quit IRC | 22:26 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add "owner" information field https://review.openstack.org/620186 | 22:31 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add a locking interface https://review.openstack.org/599191 | 22:32 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Expose conductors: api-ref https://review.openstack.org/622862 | 22:34 |
| openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: Negative tests for testing actions with node-get command https://review.openstack.org/342060 | 22:46 |
| rloo | thx TheJulia! | 22:51 |
| TheJulia | rloo: If you want to +1, I think that would be fair given the consensus on irc and all | 22:51 |
| TheJulia | err | 22:51 |
| TheJulia | WF+! | 22:51 |
| TheJulia | WF+1 | 22:51 |
| rloo | TheJulia: oh, wanted Kaifeng to see it. | 22:52 |
| TheJulia | oh, well he can WF+1 it as well | 22:52 |
| TheJulia | its all good, I'm starting to make the list of things I need to cut releases for | 22:52 |
| rloo | TheJulia: thx! | 22:53 |
| *** rcernin has joined #openstack-ironic | 22:59 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Single process ironic https://review.openstack.org/623569 | 23:12 |
| *** hshiina has joined #openstack-ironic | 23:56 | |
| *** mmethot has quit IRC | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!