| *** mrunge has quit IRC | 00:15 | |
| *** mrunge has joined #openstack-ironic | 00:19 | |
| *** mrunge_ has joined #openstack-ironic | 00:24 | |
| *** mrunge has quit IRC | 00:24 | |
| *** hshiina|afk is now known as hshiina | 00:33 | |
| *** rh-jelabarre has joined #openstack-ironic | 00:52 | |
| *** liuzz has joined #openstack-ironic | 01:03 | |
| *** trungnv has joined #openstack-ironic | 01:04 | |
| *** phuongnh has joined #openstack-ironic | 01:07 | |
| *** openstack has joined #openstack-ironic | 01:19 | |
| *** ChanServ sets mode: +o openstack | 01:19 | |
| *** tiendc has joined #openstack-ironic | 01:19 | |
| *** yangyapeng has joined #openstack-ironic | 01:29 | |
| *** zhangfei has joined #openstack-ironic | 01:30 | |
| openstackgerrit | qingszhao proposed openstack/ironic-tempest-plugin master: change default python 3 env in tox to 3.5 https://review.openstack.org/574507 | 01:46 |
|---|---|---|
| *** zhangfei has quit IRC | 01:47 | |
| openstackgerrit | Merged openstack/ironic master: Fix conductor manager unit tests https://review.openstack.org/577479 | 02:15 |
| *** rh-jelabarre has quit IRC | 02:37 | |
| *** jaganathan has joined #openstack-ironic | 02:51 | |
| *** zshi has joined #openstack-ironic | 02:54 | |
| *** liuzz_ has joined #openstack-ironic | 03:06 | |
| *** liuzz has quit IRC | 03:10 | |
| *** liuzz has joined #openstack-ironic | 03:13 | |
| *** liuzz_ has quit IRC | 03:16 | |
| *** zshi has quit IRC | 03:56 | |
| *** kiseok7 has quit IRC | 04:13 | |
| *** d0ugal_ has joined #openstack-ironic | 04:15 | |
| *** d0ugal has quit IRC | 04:15 | |
| *** parlos has joined #openstack-ironic | 04:48 | |
| *** parlos has quit IRC | 05:10 | |
| *** e0ne has joined #openstack-ironic | 05:17 | |
| *** yangyapeng has quit IRC | 05:33 | |
| *** hamzy_ has quit IRC | 05:36 | |
| *** hamzy has joined #openstack-ironic | 05:38 | |
| *** tiendc has quit IRC | 05:39 | |
| *** hamzy has quit IRC | 05:40 | |
| *** hamzy has joined #openstack-ironic | 05:42 | |
| *** e0ne has quit IRC | 05:46 | |
| *** mjura has joined #openstack-ironic | 05:47 | |
| *** e0ne has joined #openstack-ironic | 05:48 | |
| *** yolanda has joined #openstack-ironic | 05:53 | |
| *** yangyapeng has joined #openstack-ironic | 05:57 | |
| *** yangyapeng has quit IRC | 06:02 | |
| *** tiendc has joined #openstack-ironic | 06:03 | |
| *** yangyapeng has joined #openstack-ironic | 06:03 | |
| *** zshi has joined #openstack-ironic | 06:14 | |
| *** jcoufal has joined #openstack-ironic | 06:16 | |
| *** jcoufal has quit IRC | 06:16 | |
| *** jcoufal has joined #openstack-ironic | 06:17 | |
| *** hoonetorg has quit IRC | 06:20 | |
| *** parlos has joined #openstack-ironic | 06:26 | |
| *** mrunge_ is now known as mrunge | 06:27 | |
| *** hoonetorg has joined #openstack-ironic | 06:32 | |
| *** phuongnh has quit IRC | 06:33 | |
| *** phuongnh has joined #openstack-ironic | 06:33 | |
| *** e0ne has quit IRC | 06:37 | |
| *** jtomasek has joined #openstack-ironic | 06:43 | |
| *** lifeless has quit IRC | 06:49 | |
| *** rbartal has joined #openstack-ironic | 06:58 | |
| *** ajya has joined #openstack-ironic | 07:04 | |
| *** pcaruana has joined #openstack-ironic | 07:07 | |
| *** vmud213 has joined #openstack-ironic | 07:13 | |
| *** phuongnh has quit IRC | 07:15 | |
| *** phuongnh has joined #openstack-ironic | 07:16 | |
| *** tesseract has joined #openstack-ironic | 07:20 | |
| *** hjensas has quit IRC | 07:37 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Add configurable option for Ironic master tftp images path https://review.openstack.org/577071 | 07:44 |
| *** toabctl has quit IRC | 07:58 | |
| openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Introduce BIOS API https://review.openstack.org/570555 | 08:02 |
| *** serlex has joined #openstack-ironic | 08:09 | |
| *** jistr|off is now known as jistr | 08:14 | |
| *** athomas has joined #openstack-ironic | 08:16 | |
| openstackgerrit | Lenny Verkhovsky proposed openstack/ironic master: WIP Change SOFT_REBOOT to reboot instead of power off/on https://review.openstack.org/577748 | 08:16 |
| *** d0ugal_ has quit IRC | 08:16 | |
| *** d0ugal has joined #openstack-ironic | 08:17 | |
| *** d0ugal has quit IRC | 08:17 | |
| *** d0ugal has joined #openstack-ironic | 08:17 | |
| *** racedo has joined #openstack-ironic | 08:20 | |
| *** dougsz has joined #openstack-ironic | 08:21 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic master: Add CUSTOM_CPU_FPGA Traits value to ironic inspection https://review.openstack.org/569377 | 08:23 |
| *** vmud213 has quit IRC | 08:23 | |
| *** arxcruz|off is now known as arxcruz | 08:26 | |
| *** derekh has joined #openstack-ironic | 08:30 | |
| *** toabctl has joined #openstack-ironic | 08:31 | |
| *** e0ne has joined #openstack-ironic | 08:34 | |
| *** mgoddard has joined #openstack-ironic | 08:36 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/ironic stable/queens: Raise TemporaryFailure if no conductors are online https://review.openstack.org/577752 | 08:41 |
| *** vmud213 has joined #openstack-ironic | 08:42 | |
| *** namnh has joined #openstack-ironic | 08:43 | |
| *** rbartal has quit IRC | 08:48 | |
| *** rbartal has joined #openstack-ironic | 08:49 | |
| *** zshi has quit IRC | 08:50 | |
| *** MrBerns has joined #openstack-ironic | 08:56 | |
| *** dtantsur|afk is now known as dtantsur | 09:03 | |
| dtantsur | morning ironic | 09:03 |
| mgoddard | morning dtantsur | 09:05 |
| etingof | happy Monday mgoddard, dtantsur and the rest of ironic! ;) | 09:05 |
| mgoddard | happy Monday indeed etingof | 09:07 |
| *** parlos has quit IRC | 09:09 | |
| *** jcoufal has quit IRC | 09:10 | |
| *** jcoufal has joined #openstack-ironic | 09:13 | |
| *** jcoufal has quit IRC | 09:13 | |
| *** jcoufal has joined #openstack-ironic | 09:14 | |
| *** parlos has joined #openstack-ironic | 09:16 | |
| *** ajya has quit IRC | 09:21 | |
| *** ajya has joined #openstack-ironic | 09:21 | |
| olivierbourdon38 | good monday mgoddard dtantsur & all Ironicers | 09:27 |
| dtantsur | o/ | 09:30 |
| dtantsur | mgoddard: mind checking https://review.openstack.org/#/c/576574/ ? we need it for the release | 09:31 |
| patchbot | patch 576574 - ironic - Release note clean-ups for ironic release | 09:31 |
| mgoddard | dtantsur: sure | 09:33 |
| *** hjensas has joined #openstack-ironic | 09:33 | |
| *** hjensas has quit IRC | 09:33 | |
| *** hjensas has joined #openstack-ironic | 09:33 | |
| mgoddard | dtantsur: how strict are we on use of english in release notes? | 09:36 |
| *** namnh has quit IRC | 09:36 | |
| dtantsur | mgoddard: it must be understandable by any reader familiar with the language and not contain obvious errors | 09:36 |
| *** rbartal has quit IRC | 09:36 | |
| *** kiseok7 has joined #openstack-ironic | 09:39 | |
| *** namnh has joined #openstack-ironic | 09:41 | |
| openstackgerrit | Debayan Ray proposed openstack/sushy master: Add system simple storage resource support https://review.openstack.org/561582 | 09:41 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Refuse secure erase if ATA command does not work https://review.openstack.org/575206 | 09:42 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Provide knob to disable ata secure erase https://review.openstack.org/575207 | 09:42 |
| dtantsur | TheJulia: FYI doing master releases for a lot of things. waiting for your ack for ironic though, and waiting for not-yet-merged patches for IPA and bifrost. | 09:46 |
| dtantsur | etingof: mind following up on a virtualbmc reno about deprecating the old behavior of launching server from the cli? | 09:49 |
| dtantsur | then I can release it | 09:49 |
| dtantsur | mgoddard: are we ready for an n-g-s release (not final yet)? | 09:50 |
| etingof | dtantsur, what do you mean -- deprecating? just stop launching server from cli? | 09:51 |
| dtantsur | etingof: yep, a formal deprecation release note | 09:51 |
| dtantsur | see TheJulia's comment on a patch I just approved | 09:51 |
| *** vmud213 has quit IRC | 09:51 | |
| *** hjensas has quit IRC | 09:51 | |
| *** rbartal has joined #openstack-ironic | 09:51 | |
| etingof | dtantsur, ah, I see! sure! | 09:52 |
| *** vmud213 has joined #openstack-ironic | 09:52 | |
| dtantsur | thanks! | 09:52 |
| mgoddard | dtantsur: I'll check the backlog | 09:52 |
| openstackgerrit | Merged openstack/ironic master: Release note clean-ups for ironic release https://review.openstack.org/576574 | 09:53 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy stable/queens: Gate fix: Cap hacking to avoid gate failure https://review.openstack.org/577770 | 09:56 |
| mgoddard | dtantsur: I can't see anything that can't wait for another (rocky) release in NGS | 09:56 |
| dtantsur | ack | 09:56 |
| openstackgerrit | Merged openstack/bifrost master: Add configurable option for Ironic master tftp images path https://review.openstack.org/577071 | 09:57 |
| openstackgerrit | Merged openstack/virtualbmc master: Improve the documentation https://review.openstack.org/575482 | 09:58 |
| *** hjensas has joined #openstack-ironic | 09:59 | |
| *** hjensas has quit IRC | 09:59 | |
| *** hjensas has joined #openstack-ironic | 09:59 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy stable/pike: Avoid tox_install.sh for constraints support https://review.openstack.org/577771 | 10:01 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Add manage_boot parameter to introspection API https://review.openstack.org/316801 | 10:03 |
| *** rbartal_ has joined #openstack-ironic | 10:04 | |
| *** rbartal has quit IRC | 10:04 | |
| openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Rework documentation layout https://review.openstack.org/577774 | 10:06 |
| *** hshiina has quit IRC | 10:07 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy stable/queens: Change BootSourceOverrideMode from BIOS to Legacy https://review.openstack.org/577581 | 10:07 |
| openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Add documentation on configuration of synchronisation https://review.openstack.org/577430 | 10:11 |
| openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Add support for Dell PowerConnect switches https://review.openstack.org/573380 | 10:11 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy stable/pike: Change BootSourceOverrideMode from BIOS to Legacy https://review.openstack.org/577582 | 10:14 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Add manage_boot parameter to introspection API https://review.openstack.org/316801 | 10:20 |
| *** namnh_ has joined #openstack-ironic | 10:21 | |
| *** namnh_ has quit IRC | 10:21 | |
| *** namnh has quit IRC | 10:24 | |
| *** milan has joined #openstack-ironic | 10:26 | |
| *** phuongnh has quit IRC | 10:30 | |
| *** vmud213 has quit IRC | 11:06 | |
| *** vmud213 has joined #openstack-ironic | 11:06 | |
| *** jcoufal has quit IRC | 11:08 | |
| *** tiendc has quit IRC | 11:08 | |
| *** MattMan has quit IRC | 11:19 | |
| *** MattMan has joined #openstack-ironic | 11:19 | |
| *** bfournie has quit IRC | 11:26 | |
| *** zshi has joined #openstack-ironic | 11:30 | |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: Add reno for release notes management https://review.openstack.org/577795 | 11:42 |
| dtantsur | etingof: ouch, we did not have reno for vbmc? | 11:46 |
| *** vmud213 has quit IRC | 11:46 | |
| dtantsur | I would not ask you do introduce it if I realized it :) | 11:46 |
| etingof | dtantsur, we did not have it, apparently | 11:50 |
| etingof | dtantsur, but we'd inevitably add it sooner or later ;) | 11:50 |
| dtantsur | etingof: not necessary, it's not a production service.. but I don't mind it either | 11:51 |
| *** bfournie has joined #openstack-ironic | 12:08 | |
| *** dprince has joined #openstack-ironic | 12:14 | |
| *** trown|outtypewww has quit IRC | 12:15 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:15 | |
| *** weshay_ is now known as weshay|ruck | 12:22 | |
| *** juhak has joined #openstack-ironic | 12:24 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove the iDRAC classic drivers https://review.openstack.org/577452 | 12:32 |
| *** trown has joined #openstack-ironic | 12:35 | |
| dtantsur | rpioso|afk, rajinir, FYI sushy 1.5.0 released with the boot mode fix, should start appearing in the runs soon(ish) | 12:35 |
| TheJulia | dtantsur: ack. Trying to wake up now | 12:39 |
| *** yangyapeng has quit IRC | 12:41 | |
| TheJulia | Really bad jet lag | 12:44 |
| dtantsur | TheJulia: ugh :( good ehmmm morning? | 12:47 |
| dtantsur | how was China? | 12:47 |
| TheJulia | It is morning ;) | 12:54 |
| TheJulia | It is always morning, espescialy when the coffee is brewing. | 12:54 |
| *** rloo has joined #openstack-ironic | 12:55 | |
| TheJulia | China was very insightful. Interesting to hear their perceptions and issues. | 12:56 |
| TheJulia | rloo: did oath ever release their background power state checker/updater? | 12:57 |
| rloo | TheJulia: good morning. To be honest (maybe I am still not awake), I don't know. Is that something that jroll mentioned? | 12:58 |
| rloo | TheJulia: if it is something someone said they'd do in the last 3 months, the answer would be no. | 13:00 |
| TheJulia | rloo: penick mentioned it a while back, i only asked as i just woke up as well | 13:00 |
| TheJulia | rloo: okay | 13:00 |
| TheJulia | Turns out oath is not the only deployment with that sort of scale issues. | 13:01 |
| rloo | TheJulia: oh. hmm, let me check with him. I'll get back to you on that later. | 13:01 |
| TheJulia | rloo: awesome, thanks | 13:01 |
| rloo | :) | 13:01 |
| * TheJulia sips coffee | 13:01 | |
| TheJulia | My fitbit says 9 PM, my phone says 9 AM | 13:02 |
| rloo | <don't trust your fitbit> | 13:02 |
| dtantsur | TheJulia: I remember chatting with devananda on moving all periodic task into a new process. it may be the time finally.. | 13:03 |
| *** cdearborn has joined #openstack-ironic | 13:03 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove the deprecated classic drivers for Cisco UCS hardware https://review.openstack.org/577810 | 13:04 |
| TheJulia | Periodic task conductor mose maybe? Then again a big part of the problwm with ipmi is the exec of ipmitool | 13:04 |
| openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Rework documentation layout https://review.openstack.org/577774 | 13:06 |
| openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Add documentation on configuration of synchronisation https://review.openstack.org/577430 | 13:06 |
| openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Add support for Dell PowerConnect switches https://review.openstack.org/573380 | 13:06 |
| *** Guest71763 is now known as honza | 13:07 | |
| dtantsur | TheJulia: time for Execute IpmiTool as a Service? | 13:08 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove the deprecated pxe_snmp driver https://review.openstack.org/577812 | 13:08 |
| dtantsur | TheJulia: btw we need reviews on https://review.openstack.org/577770 and https://review.openstack.org/577771 to fix sushy stable | 13:09 |
| patchbot | patch 577770 - sushy (stable/queens) - Gate fix: Cap hacking to avoid gate failure | 13:09 |
| patchbot | patch 577771 - sushy (stable/pike) - Avoid tox_install.sh for constraints support | 13:10 |
| TheJulia | dtantsur: maybe, but the rolling power check is what really impacts performance. The deployment i spoke with at one point had 11 conductors.... | 13:10 |
| TheJulia | dtantsur: finishing coffee and will go open my laptop shortly | 13:11 |
| dtantsur | TheJulia: if we split them into separate services, maybe we can make it less a bottleneck | 13:11 |
| dtantsur | e.g. we'll be able to run several on one host | 13:11 |
| sambetts|afk | I think penick mentioned it being a network issue too though | 13:12 |
| sambetts|afk | so making more requests would just result in more missing ipmi packets | 13:12 |
| TheJulia | We would have to dig uo the data he collected on it | 13:12 |
| *** ajya has quit IRC | 13:13 | |
| *** ajya has joined #openstack-ironic | 13:13 | |
| *** bdodd_ has joined #openstack-ironic | 13:14 | |
| dtantsur | sambetts|afk: https://review.openstack.org/316801 has celebrated two years since revision 1, maybe congratulate it with some reviews? :) | 13:16 |
| patchbot | patch 316801 - ironic-inspector - Add manage_boot parameter to introspection API | 13:16 |
| *** mjturek has joined #openstack-ironic | 13:16 | |
| *** bdodd has quit IRC | 13:16 | |
| *** yangyapeng has joined #openstack-ironic | 13:21 | |
| TheJulia | ugh, looks like my ipa jobs are hitting all sorts of fun timeouts | 13:31 |
| sambetts|afk | dtantsur: I'll take a look | 13:31 |
| TheJulia | ipa patches I should say | 13:31 |
| *** arnewiebalck_ has joined #openstack-ironic | 13:32 | |
| openstackgerrit | Merged openstack/ironic master: Remove the ilo classic drivers https://review.openstack.org/577443 | 13:32 |
| *** rbartal has joined #openstack-ironic | 13:33 | |
| *** bdodd_ has quit IRC | 13:33 | |
| *** bdodd has joined #openstack-ironic | 13:34 | |
| sambetts|afk | dtantsur: still looks good since the last time I reviewed it :) | 13:35 |
| *** rbartal_ has quit IRC | 13:35 | |
| dtantsur | heh, cool :) | 13:39 |
| dtantsur | TheJulia: yeah today is timeouts day, quite a few happened in the morning | 13:39 |
| TheJulia | dtantsur: I looked at maybe a dozen failures last week, and they were all timeouts pulling from pypi :\ | 13:40 |
| dtantsur | huh | 13:40 |
| * dtantsur sees iRMC and iLO BIOS interfaces proposed, nice! | 13:43 | |
| *** jaganathan has quit IRC | 13:47 | |
| dtantsur | sambetts|afk: this needs fixing ASAP http://192.133.158.227/10/577810/1/check/ironic-dsvm-tempest-ironic-cimc-current-centos-7-cimc/c30ba2e/logs/devstacklog.txt.gz#_2018-06-25_13_14_51_770 | 13:48 |
| dtantsur | the patch removing it is proposed | 13:48 |
| dtantsur | ditto http://192.133.158.227/10/577810/1/check/ironic-dsvm-tempest-ironic-ucsm-current-centos-7-ucsm/8d0304a/logs/devstacklog.txt.gz#_2018-06-25_13_15_18_130 | 13:49 |
| sambetts|afk | dtantsur: looking :) | 13:50 |
| openstackgerrit | Merged openstack/sushy stable/queens: Gate fix: Cap hacking to avoid gate failure https://review.openstack.org/577770 | 13:51 |
| openstackgerrit | Merged openstack/sushy stable/pike: Avoid tox_install.sh for constraints support https://review.openstack.org/577771 | 13:51 |
| TheJulia | dtantsur: \o/ | 13:51 |
| TheJulia | dtantsur: got any more that your aware of that need reviews? | 13:51 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: [DNM] Testing Cisco Third Party CI https://review.openstack.org/514312 | 13:54 |
| sambetts|afk | dtantsur: testing https://github.com/CiscoSystems/project-config-third-party/pull/101 ^ | 13:54 |
| *** cjloader has joined #openstack-ironic | 13:57 | |
| *** sungn_ has joined #openstack-ironic | 13:59 | |
| *** jcoufal has joined #openstack-ironic | 14:00 | |
| *** hjensas has quit IRC | 14:08 | |
| *** zshi has quit IRC | 14:09 | |
| dtantsur | cool | 14:09 |
| dtantsur | TheJulia: you can review the sushy boot mode backports now https://review.openstack.org/#/c/577581/ https://review.openstack.org/#/c/577582/ | 14:11 |
| patchbot | patch 577581 - sushy (stable/queens) - Change BootSourceOverrideMode from BIOS to Legacy | 14:11 |
| patchbot | patch 577582 - sushy (stable/pike) - Change BootSourceOverrideMode from BIOS to Legacy | 14:11 |
| NobodyCam | Good Morning Ironic'ers | 14:12 |
| dtantsur | morning NobodyCam | 14:12 |
| NobodyCam | hey hey dtantsur :) | 14:12 |
| NobodyCam | how goes the monday | 14:12 |
| dtantsur | as painfully as expected :D | 14:12 |
| dtantsur | and for you? | 14:13 |
| openstackgerrit | Merged openstack/sushy-tools master: fix documentation on simulators https://review.openstack.org/572833 | 14:13 |
| *** zshi has joined #openstack-ironic | 14:14 | |
| *** markguz has joined #openstack-ironic | 14:18 | |
| NobodyCam | its be a roller coaster of a weekend. | 14:18 |
| *** baha has joined #openstack-ironic | 14:19 | |
| NobodyCam | aged, lost one of great danes to cancer, and got a new puppy! :p | 14:19 |
| NobodyCam | but we're all doing good now :) | 14:19 |
| * TheJulia gives NobodyCam hugs | 14:20 | |
| NobodyCam | Thank you TheJulia :) | 14:21 |
| NobodyCam | and Good Morning :) | 14:21 |
| *** rpioso|afk is now known as rpioso | 14:22 | |
| rpioso | Good morning | 14:22 |
| NobodyCam | Good Morning rpioso :) | 14:23 |
| * etingof is looking for an example project having in-tree reno job configured... the ones I checked have it in project-config/zuul.d/projects.yaml | 14:23 | |
| openstackgerrit | Julia Kreger proposed openstack/sushy-tools master: Fix minor typo in user doc https://review.openstack.org/577837 | 14:23 |
| etingof | rpioso, NobodyCam good morning o/ | 14:23 |
| NobodyCam | morning etingof :) | 14:23 |
| dtantsur | etingof: checkout metalsmith | 14:30 |
| dtantsur | https://github.com/openstack/metalsmith/blob/master/.zuul.yaml#L103-L113 | 14:31 |
| *** yangyapeng has quit IRC | 14:34 | |
| rpioso | etingof, NobodyCam: Hey! | 14:36 |
| * rpioso is in a meeting | 14:36 | |
| NobodyCam | Morning rpioso | 14:36 |
| jiapei | Morning Guys | 14:40 |
| etingof | dtantsur, thanks, but why don't I see 'release-notes-jobs' there? probably this one -- https://docs.openstack.org/infra/openstack-zuul-jobs/project-templates.html#project_template-release-notes-jobs | 14:40 |
| *** sekharvajjula has joined #openstack-ironic | 14:41 | |
| *** hjensas has joined #openstack-ironic | 14:42 | |
| NobodyCam | Morning jape :) | 14:43 |
| NobodyCam | gah | 14:43 |
| NobodyCam | Morning jaipei :) | 14:43 |
| *** arnewiebalck_ has quit IRC | 14:45 | |
| *** hshiina has joined #openstack-ironic | 14:46 | |
| TheJulia | rloo: By chance did you see comments on https://review.openstack.org/#/c/577490/ ? | 14:48 |
| patchbot | patch 577490 - ironic - During cleaning, use current node.driver_internal_... | 14:48 |
| TheJulia | good morning jiapei | 14:48 |
| rloo | TheJulia: oh, forgot to look, thx for pinging me. | 14:48 |
| rloo | TheJulia: ah, i was wondering about that. Ok, will do it today. | 14:48 |
| TheJulia | no problem :) Thanks! | 14:49 |
| etingof | dtantsur, the more I look the more I get confused - does metalsmith have release notes? | 14:50 |
| dtantsur | etingof: nope | 14:52 |
| sekharvajjula | TheJulia: Good morning. I am in for today's meeting for https://review.openstack.org/#/c/543936. | 14:52 |
| patchbot | patch 543936 - ironic-specs - Added new spec for L3 based Ironic deployment | 14:52 |
| TheJulia | etingof: re: https://review.openstack.org/#/c/389736/ if I were you, I'd just go ahead and rev it so we can land it | 14:52 |
| patchbot | patch 389736 - ironic - Adds snmp driver auto discovery for APC and baytech | 14:52 |
| dtantsur | just take the required template in the same way | 14:52 |
| TheJulia | sekharvajjula: Excellent! | 14:52 |
| TheJulia | sekharvajjula: Also, good morning! | 14:53 |
| etingof | TheJulia, looks like you are me - I am on it for a couple of days | 14:53 |
| mordred | TheJulia, dtantsur: I just got a pile of bifrost failures that the same patch didn't get yesterday - is this a known issue? | 14:54 |
| mordred | http://logs.openstack.org/58/577658/1/gate/bifrost-integration-tinyipa/44e4010/ | 14:54 |
| etingof | dtantsur, so how does metalsmith (you linked) help figuring out in-tree zuul config for reno jobs? %-] | 14:54 |
| dtantsur | etingof: s/for reno jobs// | 14:54 |
| dtantsur | it does not matter which jobs you add - the config is the same (both in-tree and in project-config) | 14:55 |
| dtantsur | so I'm not really sure what the source of confusion is.. | 14:55 |
| *** stendulker has joined #openstack-ironic | 14:57 | |
| openstackgerrit | Merged openstack/sushy stable/queens: Change BootSourceOverrideMode from BIOS to Legacy https://review.openstack.org/577581 | 14:57 |
| dtantsur | etingof: templates are sets of jobs. both jobs and templates can be added to a 'project' structure | 14:58 |
| TheJulia | mordred: not a known issue. Interesting: http://logs.openstack.org/58/577658/1/gate/bifrost-integration-tinyipa/44e4010/job-output.txt.gz#_2018-06-25_14_11_57_580741 | 14:58 |
| etingof | dtantsur, yes, I see | 14:58 |
| dtantsur | templates are added under 'templates', while jobs - under one of the queues (check, gate, etc) | 14:58 |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: Add reno for release notes management https://review.openstack.org/577795 | 14:58 |
| dtantsur | TheJulia: a side effect of changing the error code to 503? | 14:58 |
| TheJulia | dtantsur: I don't think that would do it because the conductor should be up. The url path seems odd to me though | 14:59 |
| etingof | dtantsur, like this? -- https://review.openstack.org/#/c/577795/2/zuul.d/project.yaml | 14:59 |
| patchbot | patch 577795 - virtualbmc - Add reno for release notes management | 14:59 |
| * TheJulia gets water | 14:59 | |
| dtantsur | etingof: yep | 14:59 |
| etingof | 10x | 14:59 |
| TheJulia | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon Jun 25 15:00:15 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 | |
| TheJulia | o/ everyone! | 15:00 |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| dtantsur | o/ | 15:00 |
| TheJulia | Our agenda this week, as every week, is on the wiki! | 15:00 |
| rpioso | o/ | 15:00 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic | 15:00 |
| bdodd | o/ | 15:00 |
| etingof | o/ | 15:00 |
| TheJulia | #topic Announcements / Reminder | 15:00 |
| rloo | o/ | 15:00 |
| *** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)" | 15:00 | |
| hshiina | o/ | 15:01 |
| jiapei | o/ | 15:01 |
| *** e0ne has quit IRC | 15:01 | |
| hjensas | u/ | 15:01 |
| stendulker | o/ | 15:01 |
| mgoddard | o/ | 15:01 |
| TheJulia | This week is R-9. Typically we have to cut our final release around R-5 or R-4 in order not to be completely broken at the end of the cycle for a few weeks. So as a reminder, we have about a month of time left of time where we can merge things(). | 15:02 |
| *** mjura has quit IRC | 15:02 | |
| cdearborn | \o | 15:02 |
| TheJulia | I have no other announcements, other than I've got jetlag. ;) Does anyone have anything they would like to annouce? | 15:02 |
| TheJulia | announce ? | 15:03 |
| rajinir | o/ | 15:03 |
| dtantsur | I've requested quite a few releases | 15:03 |
| TheJulia | Ahh, yes! | 15:03 |
| dtantsur | the master ones are done already, with ironic pike and ocata pending | 15:03 |
| dtantsur | the ironic master release is awaiting the drivers removal I guess? | 15:03 |
| openstackgerrit | Merged openstack/networking-generic-switch master: support hpe device type for the HPE 5900 series switches https://review.openstack.org/471051 | 15:04 |
| TheJulia | dtantsur: I'd like to wait until we have them removed, since we're after the middle of the cycle | 15:04 |
| dtantsur | yeah | 15:04 |
| * dtantsur urgently fixes bifrost, yay | 15:04 | |
| rpioso | dtantsur: How about sushy? | 15:04 |
| dtantsur | rpioso: sushy master released | 15:05 |
| dtantsur | 1.5.0, will hit upper-constraints soon(ish) | 15:05 |
| TheJulia | dtantsur: :( | 15:05 |
| rpioso | dtantsur: \o/ | 15:05 |
| *** rh-jelabarre has quit IRC | 15:05 | |
| TheJulia | I guess we're ready to move on then | 15:05 |
| TheJulia | #topic Review action items from previous meeting | 15:06 |
| *** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)" | 15:06 | |
| openstackgerrit | Merged openstack/sushy stable/pike: Change BootSourceOverrideMode from BIOS to Legacy https://review.openstack.org/577582 | 15:06 |
| TheJulia | #info We had no action items last week \o/ | 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 around line 156 | 15:07 |
| *** e0ne has joined #openstack-ironic | 15:08 | |
| TheJulia | ohh, neutron event processing spec | 15:09 |
| rloo | yay, graphical console spec was approved! | 15:10 |
| TheJulia | yup | 15:10 |
| *** VisBits has joined #openstack-ironic | 15:10 | |
| rloo | wrt L203, dtantsur asks a question -- did we answer it? | 15:10 |
| TheJulia | rloo: I've been slammed, I'll try to answer that and more this week | 15:11 |
| rloo | TheJulia: no worries, is that an action item? :) | 15:11 |
| TheJulia | #action TheJulia will try and condense operator feedback/asks from summit and OpenInfra Days Beijing in a written form this week. | 15:11 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Remove support for classic drivers https://review.openstack.org/577848 | 15:12 |
| dtantsur | TheJulia: this looks like sad panda ^^^ but apparently needed... | 15:12 |
| rloo | dtantsur: wrt the classic driver removal. the list there -- is that it or does it continue to grow? (L249 etc) | 15:12 |
| TheJulia | Speaking of which, there are some surprisingly large ironic deployments in china | 15:12 |
| TheJulia | dtantsur: c'est la vie | 15:12 |
| dtantsur | rloo: it grows until the last enemy falls victim of my code removal skills | 15:12 |
| rloo | dtantsur: heh. so we don't know how close we are to finishing that :-( | 15:12 |
| TheJulia | dtantsur: I put the three you had up earlier this morning on the priorities to review this week, we should add the others I think | 15:12 |
| rloo | wonder if we should push out a master release regardless of whether we remove them all or not. | 15:13 |
| *** pcaruana has quit IRC | 15:13 | |
| TheJulia | yolanda: You have an outstanding question on cleanhold it looks like. I'll try and answer that today | 15:13 |
| rloo | not that i care that much, but the original intent was to release often, which means there is no way big features would be done w/i a release | 15:13 |
| dtantsur | rloo: updated. it's not a lot of work in reality, just some consistent reviews (and rechecks) | 15:13 |
| rloo | dtantsur: ok, i thought last week there was only one patch left so didn't keep an eye out. | 15:14 |
| yolanda | hi TheJulia yes... may be because i don't get the whole workflow, but i was not understanding this part | 15:14 |
| rloo | I'm deleting python 3.5 compat since it is being tracked in storyboard | 15:15 |
| rloo | (L307) | 15:15 |
| rloo | poof | 15:15 |
| TheJulia | Re: actually writing down our vision. I think that can wait until we've cut our stable branch for this cycle. | 15:15 |
| TheJulia | That is a nice quiet time typically | 15:16 |
| * TheJulia laughs at the idea | 15:16 | |
| rloo | assuming someone has time to actually write that down. maybe we should have made a video at the ptg :D | 15:16 |
| TheJulia | ++++++ | 15:16 |
| TheJulia | Maybe ideas for the next PTG | 15:17 |
| TheJulia | Anyway, I'm good with the updates. Shall we move on? | 15:17 |
| rloo | +movin' | 15:17 |
| *** rh-jelabarre has joined #openstack-ironic | 15:18 | |
| TheJulia | #topic Deciding on priorities for the coming week | 15:18 |
| *** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:18 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:18 |
| TheJulia | Starting on line 98 | 15:18 |
| rloo | hardware type cleaning should be priority #1 since it is delaying our release | 15:18 |
| TheJulia | good point | 15:19 |
| TheJulia | moving | 15:19 |
| TheJulia | I think that is about it.... | 15:19 |
| TheJulia | I moved the ipa fix up | 15:20 |
| TheJulia | for secure erase | 15:20 |
| dtantsur | I'll try to finish the remaining patches tomorrow | 15:20 |
| dtantsur | it's no longer as difficult as it used to be with fake drivers - real drivers are rarely used in unit tests | 15:20 |
| TheJulia | I'll try and get that hammered out this week, the failures seem to be sporatic timeouts on the ci jobs since we download and build lots of things in those jobs that are not really cached :\ | 15:21 |
| TheJulia | dtantsur: Awesome, thanks | 15:21 |
| rloo | thx TheJulia | 15:21 |
| rloo | TheJulia: can I put down that you will be looking at the CI failures? | 15:21 |
| dtantsur | I think we have ironic-lib broken for.. time.. | 15:21 |
| TheJulia | rloo: yes | 15:21 |
| TheJulia | dtantsur: similar issues, we need to adjust all the timeouts on it I think. :\ | 15:22 |
| rloo | Thx TheJulia | 15:22 |
| TheJulia | I started down that path but didn't get too far... | 15:22 |
| * TheJulia wonders if we have anything outstanding there | 15:22 | |
| dtantsur | ironic-lib jobs also build IPA from source | 15:22 |
| TheJulia | hmm, some stuff mjturek would likely prefer to see landed | 15:23 |
| TheJulia | #action TheJulia to look at ironic-lib and IPA build timeouts this week | 15:23 |
| TheJulia | yeah.. :( \o/ | 15:23 |
| TheJulia | I think we're good to move on? | 15:23 |
| rloo | + moving | 15:24 |
| TheJulia | #topic Discussion | 15:25 |
| *** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:25 | |
| TheJulia | As some of you may know, sekharvajjula has been working on proposing a pure l3 deployment method for nokia hardware using virtual media, where there is no DHCP. | 15:26 |
| TheJulia | #link https://review.openstack.org/#/c/543936/13/specs/approved/L3-based-deployment.rst | 15:26 |
| patchbot | patch 543936 - ironic-specs - Added new spec for L3 based Ironic deployment | 15:26 |
| dtantsur | I like the idea, but left two comments there | 15:26 |
| dtantsur | only one of them may be an issue (the generic 'properties') | 15:26 |
| TheJulia | Because of a virtual media limitation for their hardware where only one device can be attached, his current idea is to enable support for cases where the networking information gets appended to the end of the vmedia iso | 15:27 |
| dtantsur | I assume it's still a valid ISO, right? | 15:27 |
| TheJulia | I'm not sure there is a better way short of rebuilding the ISO, and I don't think Ironic should do that | 15:28 |
| * dtantsur recalls that it was possible to attach data filesystem to audio CDs | 15:28 | |
| TheJulia | dtantsur: I believe so yes, the file map is allocated upfront and maps to the blocks on disk, so appended data at the end shouldn't matter | 15:28 |
| dtantsur | yeah, as long as we don't end with invalid ISO (which won't work), it's fine | 15:28 |
| dtantsur | I wonder if it can happen that a BMC will strip the added part (or rather not expose it to the OS) | 15:29 |
| TheJulia | sekharvajjula: do you know if iso tools handle the appended data? | 15:29 |
| TheJulia | or if OSes handle it properly? | 15:29 |
| dtantsur | (including BMCs themselves) | 15:29 |
| TheJulia | dtantsur: that is a good question, I think they should just present a block device as long as the image. | 15:30 |
| sekharvajjula | We have been using dd to append the network config to the end of ISO and read it, when os boots up from /dev/sr0 (CD) device | 15:30 |
| dtantsur | that's a reasonable expectation, but may be worth calling out in the spec explicitly | 15:30 |
| dtantsur | aha, so it works. good. | 15:30 |
| sekharvajjula | We have this method tested on HP and on our own hardware as well | 15:31 |
| TheJulia | sekharvajjula: okay, in that case, if it works, and your aware that you'll have to add some more mechanics to handle that in some cases, I'm good with that then. I'm not aware of dtantsur's comments he mentioned at the beginning of the discussion. | 15:31 |
| dtantsur | yeah, just posted them | 15:31 |
| TheJulia | sekharvajjula: That is awesome news | 15:32 |
| *** e0ne has quit IRC | 15:32 | |
| dtantsur | but they may be easy to resolve | 15:32 |
| TheJulia | okay, sounds like we have consensus on my concern, so Im good. | 15:32 |
| TheJulia | dtantsur: do you wish to discuss your concerns? | 15:32 |
| dtantsur | I can voice them | 15:32 |
| dtantsur | 1. a free form port.properties. NOPE. please nope. if we need a new field, let's just add a purposed field. | 15:33 |
| openstackgerrit | Merged openstack/networking-baremetal master: Remove the duplicated "the" https://review.openstack.org/576317 | 15:33 |
| dtantsur | 2. I did not get the bit about virtmedia boot interfaces. are the existing (ilo-virtual-media, etc) fine or do we need yet another set? | 15:33 |
| openstackgerrit | Merged openstack/networking-baremetal master: Switch to using stestr https://review.openstack.org/574723 | 15:33 |
| dtantsur | if the latter, I need to understand why | 15:33 |
| TheJulia | dtantsur: why? We have portgroup.properties? | 15:34 |
| dtantsur | and node.properties. and it's ugly | 15:34 |
| TheJulia | which too is just a json field | 15:34 |
| dtantsur | well, we cannot fix the mistakes of the past so easily :) but we can avoid making new | 15:34 |
| rloo | wrt 1. I agree with dtantsur, we have to start using fields instead of stuffing things in .properties. | 15:34 |
| dtantsur | let me ask it this way: why do we need a field that is not versioned and is not introspectable? | 15:34 |
| TheJulia | Okay, I'm convinced | 15:34 |
| rloo | we've been meaning to clean up properties (eg capabilities) for a long time... | 15:35 |
| TheJulia | yeah | 15:35 |
| TheJulia | I just don't want to see us using extra, although people have been using extra fields for a while for misc things. | 15:35 |
| dtantsur | note, I'm fine if the new field is also a JSON field, I just don't want it generic | 15:35 |
| rloo | nope, not extra. we cannot/should not code anything that relies on anything from .extra | 15:35 |
| TheJulia | rloo: agreed | 15:36 |
| dtantsur | i.e. I want it to have a schema. and since we're in the microversion business, it should ideally be microversioned | 15:36 |
| TheJulia | the fields existence, or the field contents, because content revision control seems overly restricting and hampering adaptation and improvement.... given our microversioning | 15:37 |
| rloo | we've been using schemas for other things, so that makes sense. Have we used microversions? I don't recall any schemas changing. | 15:37 |
| dtantsur | TheJulia: that's exactly what microversions are for ;) | 15:37 |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: Add reno noting recent changes https://review.openstack.org/577851 | 15:37 |
| TheJulia | dtantsur: up for rewriting how we handle microversioning then? | 15:37 |
| dtantsur | so yes, the field contents. and don't shoot the messenger - I'm not the one who proposed them :) | 15:37 |
| TheJulia | so we can land more microversion impacting changes without needing new livers. | 15:38 |
| dtantsur | TheJulia: why rewrite? in this case we just have different versions of JSON schema, similar to how Nova does it | 15:38 |
| *** stendulker has quit IRC | 15:38 | |
| dtantsur | I don't propose to retrospectively start versioning other JSON fields (I propose killing them eventually) | 15:38 |
| dtantsur | but this new is going to have a specific schema (based on os-net-config IIRC) | 15:39 |
| rloo | before i forget, and i just skimmed the spec so don't grok it all, is there any security issue in appending that 'vfat image' to the boot iso? (can we validate the vfat image?) | 15:39 |
| TheJulia | dtantsur: I mean all of the way that we structure the microversion code, means every change merge conflicts and has to be stacked. We've managed to stack some in advance, but yeah... | 15:39 |
| dtantsur | TheJulia: I don't disagree with that, but it's a generic objection against microversions | 15:39 |
| dtantsur | it's one of the reasons I proposed using "features" (like, strings) instead long ago (it was shot down instantly) | 15:39 |
| TheJulia | I'm for it, but as our API is currently coded and maintained, I'm worried that we will hamper our ability further without some improvements. I almost feel like maybe we're inching towards a PTG topic of making API stuffs easier to land in the ironic code bsae | 15:40 |
| dtantsur | ++ (modulo /me not going to the PTG) | 15:41 |
| TheJulia | Lets decouple this discussion. Add a new field, and at the same time try and figure out how to achieve improved API versioning and schema control on that new field | 15:41 |
| dtantsur | yeah, we can add it without thinking this through as long as it has a fixed schema | 15:41 |
| rloo | I'm good with that | 15:42 |
| sekharvajjula | so shall i update the spec with a new feild l3_network_configuration in both port and port_group? | 15:42 |
| TheJulia | sounds good to me | 15:42 |
| * TheJulia is good with the name | 15:42 | |
| dtantsur | yep. and then the last question will be: what to do if this field is set to something for a boot interface that does not support it? | 15:42 |
| dtantsur | i.e. the PXE boot? | 15:42 |
| dtantsur | fail validation? | 15:43 |
| TheJulia | dtantsur: yeah, we kind of went down the same road with | 15:43 |
| TheJulia | with bfv | 15:43 |
| dtantsur | right, it's similar | 15:43 |
| TheJulia | and I think that is fine, which will kill the deployment early on | 15:43 |
| TheJulia | that being if scheduling/config mismatches. | 15:43 |
| dtantsur | okay, so fail validation? | 15:43 |
| TheJulia | Yeah | 15:44 |
| dtantsur | cool. sekharvajjula please reflect ^^^ in the spec | 15:44 |
| TheJulia | boot interface validation, that leaves the remaining question of interface naming, and can this be wrapped in or not | 15:44 |
| TheJulia | If we do the fail validation, I think we can use existing vmedia interfaces tbh | 15:44 |
| sekharvajjula | dtantsur: good. I will update spec accordingly. | 15:45 |
| dtantsur | I hope we do. I don't feel like we should multiply the interfaces without the vendor actually supporting several technologies | 15:45 |
| *** pbourke has quit IRC | 15:45 | |
| TheJulia | Agreed, if adding "distinct" to that statement :) | 15:45 |
| dtantsur | yeah :) | 15:46 |
| TheJulia | Okay! Awesome | 15:46 |
| TheJulia | sekharvajjula: thanks! | 15:46 |
| TheJulia | Time to move on to open discussion! | 15:46 |
| jiapei | Yean | 15:46 |
| rloo | i had a security-related question ^^ | 15:46 |
| TheJulia | #topic Open Discussion | 15:46 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:46 | |
| sekharvajjula | still didn't get answer for dtantsur: 2nd question | 15:46 |
| TheJulia | #undo | 15:46 |
| openstack | Removing item from minutes: #topic Open Discussion | 15:46 |
| TheJulia | rloo: yes? | 15:46 |
| jiapei | yeah, I just want to confirm the CI setup schedule | 15:47 |
| rloo | before i forget, and i just skimmed the spec so don't grok it all, is there any security issue in appending that 'vfat image' to the boot iso? (can we validate the vfat image?) | 15:47 |
| TheJulia | jiapei: I looked at the email, hold on and let me compare to the rocky cycle schedule | 15:47 |
| *** pbourke has joined #openstack-ironic | 15:47 | |
| jiapei | TheJulia: Sure | 15:48 |
| hjensas | sekharvajjula: Not to sure about l3_network_configuration is the best name? to me it's host_net_config? (bonds l2, ifconfig l3 ?) (it's a nit anyway ...) | 15:48 |
| rloo | (I didn't grok yet where the info comes from for generating that vfat image) | 15:48 |
| TheJulia | jiapei: It looks like your schedule was based on the rocky release schedule, however ironic releases earlier than that because of stable branch handling | 15:48 |
| etingof | rloo, what can be a threat in case of a malformed vfat? | 15:48 |
| rloo | etingof: i don't know. i'm not a security person. | 15:48 |
| *** dprince has quit IRC | 15:49 | |
| rloo | etingof: but ipa is going to do something with that. just want to make sure the info is from a trusted source or whatever. | 15:49 |
| TheJulia | rloo: I believe the conductor in the case of this usage would create the data and apend it, while offering up the vmedia image to the BMC | 15:49 |
| *** stendulker has joined #openstack-ironic | 15:49 | |
| rloo | etingof: or maybe we don't have to worry about it.just asking. | 15:49 |
| jiapei | TheJulia: well, so is there a last time for the CI? Currently we encounter some problems when setting up, still debuging it | 15:49 |
| etingof | rloo, I am just trying to understand how that could be exploited | 15:50 |
| jiapei | I mean when should the CI ready for the Ironic release | 15:50 |
| etingof | rloo, is appended vfat anyhow different from just iso from the security standpoint? | 15:50 |
| TheJulia | rloo: It would appear as local block device data, so I'm not entirely sure there is an issue there as long as however the file is shared is read-only (which should always be the case if a CD device is being emulated) | 15:50 |
| dtantsur | jiapei: if you mean the xclarity CI, it was supposed to be ready by queens final.. | 15:51 |
| jiapei | Ah... | 15:51 |
| sekharvajjula | TheJulia: +2 | 15:52 |
| rloo | and if we had had that CI working, we would have known that there were issues with the xclarity driver in queens. (Or else we would have found/fixed those issues) :-( | 15:52 |
| rloo | ok, so no one thinks there is a security issue. we can move on then :) | 15:52 |
| openstackgerrit | Merged openstack/sushy master: Introduce BIOS API https://review.openstack.org/570555 | 15:53 |
| jiapei | :) Yeah, some issues can only be tested by the CI hardware, and we have tested it | 15:54 |
| jiapei | We're building puppet and configure it this week | 15:54 |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: Add CI job to publish docs https://review.openstack.org/577853 | 15:55 |
| TheJulia | jiapei: What dtantsur said. We've seen some progress. But... yeah, it needs to appear soon. | 15:56 |
| TheJulia | Four minutes left | 15:56 |
| TheJulia | #topic Open Disucssion | 15:56 |
| *** openstack changes topic to "Open Disucssion (Meeting topic: ironic)" | 15:56 | |
| TheJulia | #undo | 15:56 |
| openstack | Removing item from minutes: #topic Open Disucssion | 15:56 |
| TheJulia | #topic Open Discussion | 15:56 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:56 | |
| *** vmud213 has joined #openstack-ironic | 15:56 | |
| TheJulia | Anyone have anything else to discuss today? | 15:56 |
| jiapei | TheJulia: We'll try our best to make it ready as soon as possible | 15:57 |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: Add reno for release notes management https://review.openstack.org/577795 | 15:57 |
| TheJulia | jiapei: Thanks! | 15:58 |
| rloo | crickets | 15:58 |
| TheJulia | Thanks everyone! | 15:58 |
| dtantsur | thanks | 15:58 |
| TheJulia | #endmeeting | 15:59 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 15:59 | |
| openstack | Meeting ended Mon Jun 25 15:59:05 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:59 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-06-25-15.00.html | 15:59 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-06-25-15.00.txt | 15:59 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-06-25-15.00.log.html | 15:59 |
| *** sungn_ has quit IRC | 15:59 | |
| jiapei | Thank you, it's my work, :D | 15:59 |
| dtantsur | g'night all | 15:59 |
| *** dtantsur is now known as dtantsur|afl | 15:59 | |
| *** dtantsur|afl is now known as dtantsur|afk | 15:59 | |
| jiapei | Good night | 15:59 |
| rloo | good night jiapei; thanks for staying up to attend the meeting! | 16:00 |
| *** hjensas is now known as hjensas|afk | 16:00 | |
| jiapei | Haha, have a good day rloo | 16:00 |
| rloo | jiapei: :) | 16:00 |
| *** hshiina has quit IRC | 16:01 | |
| *** baha has quit IRC | 16:02 | |
| *** sekharvajjula has quit IRC | 16:03 | |
| NobodyCam | Night dtantsur|afk | 16:07 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add helper method for testing node fields https://review.openstack.org/577856 | 16:07 |
| NobodyCam | night jiapei :) | 16:07 |
| *** mjturek has quit IRC | 16:12 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic master: [WIP] iLO BIOS interface implementation https://review.openstack.org/576803 | 16:13 |
| *** stendulker has quit IRC | 16:15 | |
| *** tesseract has quit IRC | 16:18 | |
| rloo | vdrok: are you around? I don't understand your comment: https://review.openstack.org/#/c/577490/1/ironic/conductor/manager.py | 16:21 |
| patchbot | patch 577490 - ironic - During cleaning, use current node.driver_internal_... | 16:21 |
| vdrok | rloo: yup i'm here | 16:21 |
| rloo | vdrok: where is the code you mention in 'execute_clean_step' call? | 16:22 |
| rloo | vdrok: or is that hypothetical? | 16:22 |
| vdrok | rloo: I'm trying to understand in what case it will break | 16:22 |
| vdrok | yeah, just hypothetical | 16:22 |
| rloo | vdrok: ah. yes, if task.node.driver_internal_info got changed. | 16:23 |
| vdrok | I guess it is only in case 2 when someone does copy.deepcopy of the dict and then reassigns | 16:23 |
| rloo | right | 16:23 |
| rloo | which is rare | 16:23 |
| TheJulia | and which really shouldn't be done to a task... | 16:23 |
| *** VisBits has quit IRC | 16:24 | |
| vdrok | rloo: ok that makes sense then | 16:24 |
| rloo | (so do I need a bug for it? :)) | 16:24 |
| *** derekh has quit IRC | 16:24 | |
| rloo | i will clarify in the commit msg. | 16:24 |
| vdrok | dunno :) If someone gets broken having some trace in the bug report is convenient but as this is a rare case noone should be looking for it :) | 16:25 |
| vdrok | thanks! | 16:25 |
| rloo | vdrok: i need to look/think about that, cuz i think i tested (e.g. my unit test) and it isn't a copy, and it broke | 16:26 |
| vdrok | rloo: hrm, weird, I thought task manager only does obj_what_changed during the db save, not during plain assignments | 16:26 |
| rloo | vdrok: or my test is wrong. hmm. | 16:26 |
| rloo | vdrok: no, well, err, dictionaries are odd but i don't recall the details now. | 16:27 |
| *** AlexeyAbashkin has joined #openstack-ironic | 16:28 | |
| vdrok | rloo: ah, https://github.com/openstack/ironic/blob/master/ironic/objects/base.py#L273 | 16:28 |
| vdrok | this might be the reason, as you call .save() in the test which calls this in the end | 16:29 |
| vdrok | if you were just assigning the value without save it should work as-is I think. but people usually want to persist something with assignment I think, so save case should be taken care of | 16:29 |
| rloo | yeah, needs to work in both cases. | 16:30 |
| *** vmud213 has quit IRC | 16:31 | |
| * TheJulia wonders where the security interface spec went to that she wrote on an aircraft that somehow became a truncated TODO | 16:33 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: During cleaning, use current node.driver_internal_info https://review.openstack.org/577490 | 16:39 |
| *** baha has joined #openstack-ironic | 16:41 | |
| *** mgoddard has quit IRC | 16:43 | |
| *** dprince has joined #openstack-ironic | 16:45 | |
| TheJulia | yolanda: responded to your question on https://review.openstack.org/#/c/507910/3 which I hope makes sense and provides additional background | 16:45 |
| patchbot | patch 507910 - ironic-specs - Add cleanhold specification | 16:45 |
| *** dsneddon has quit IRC | 16:48 | |
| rpioso | TheJulia: dtantsur|afk stated above, "rpioso: sushy master released 1.5.0, will hit upper-constraints soon(ish) | 16:51 |
| *** dougsz has quit IRC | 16:51 | |
| rpioso | TheJulia: Is soon(ish) now -- https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L337 ? | 16:51 |
| *** cjloader_ has joined #openstack-ironic | 16:52 | |
| *** cjloader has quit IRC | 16:52 | |
| *** dprince has quit IRC | 16:53 | |
| *** dprince has joined #openstack-ironic | 16:54 | |
| *** gyee has joined #openstack-ironic | 16:55 | |
| *** cjloader_ has quit IRC | 16:55 | |
| *** cjloader has joined #openstack-ironic | 16:56 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Follow the new PTI for document build https://review.openstack.org/556604 | 16:56 |
| openstackgerrit | Merged openstack/ironic stable/queens: Raise TemporaryFailure if no conductors are online https://review.openstack.org/577752 | 17:04 |
| *** Alexey_Abashkin has joined #openstack-ironic | 17:04 | |
| *** AlexeyAbashkin has quit IRC | 17:07 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 17:07 | |
| *** Alexey_Abashkin has quit IRC | 17:09 | |
| *** elbragstad is now known as lbragstad | 17:16 | |
| *** MrBerns has quit IRC | 17:17 | |
| openstackgerrit | Merged openstack/ironic master: fix typos https://review.openstack.org/576442 | 17:25 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Try to unlock failed device before proceeding https://review.openstack.org/575205 | 17:25 |
| *** e0ne has joined #openstack-ironic | 17:26 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Cleanup CI jobs for IPA https://review.openstack.org/577879 | 17:27 |
| *** dprince has quit IRC | 17:27 | |
| TheJulia | rpioso: it takes a background job finishing if memory serves | 17:29 |
| TheJulia | rpioso: https://review.openstack.org/#/c/577804/ | 17:30 |
| patchbot | patch 577804 - requirements - update constraint for sushy to new release 1.5.0 | 17:30 |
| TheJulia | looks like there is some unrelated nova failure right now :\ | 17:30 |
| TheJulia | yolanda: I proposed https://review.openstack.org/577879 for ipa, i noticed only the coreos jobs had extended timeouts, so for good measure I extended the base, and the only value that was extended. I figure tinyipa typically builds in under 600 seconds which is the otherwise default. | 17:32 |
| patchbot | patch 577879 - ironic-python-agent - Cleanup CI jobs for IPA | 17:32 |
| rpioso | TheJulia: Thank you :) | 17:32 |
| TheJulia | rpioso: you might want to ask in #openstack-requirements, since it looks like it was rechecked | 17:34 |
| rpioso | TheJulia: Will do when I return to a real keyboard. | 17:35 |
| openstackgerrit | Merged openstack/ironic master: Fix error when deleting a non-existent port https://review.openstack.org/577218 | 17:37 |
| *** cjloader has quit IRC | 17:39 | |
| *** cjloader has joined #openstack-ironic | 17:40 | |
| *** e0ne has quit IRC | 17:40 | |
| TheJulia | rpioso: matthew just rechecked it again | 17:42 |
| *** trown is now known as trown|lunch | 17:43 | |
| rpioso | TheJulia: My fingers are crossed :) | 17:46 |
| rpioso | TheJulia: Thanks, again. | 17:46 |
| *** AlexeyAbashkin has quit IRC | 17:49 | |
| *** murrdoc has joined #openstack-ironic | 17:53 | |
| *** e0ne has joined #openstack-ironic | 17:53 | |
| *** harlowja has joined #openstack-ironic | 17:55 | |
| openstackgerrit | Merged openstack/ironic master: Fix for Unable to create RAID1 on Dell BOSS card https://review.openstack.org/574879 | 18:09 |
| *** jiapei has quit IRC | 18:17 | |
| *** mjturek has joined #openstack-ironic | 18:18 | |
| *** hjensas|afk is now known as hjensas | 18:19 | |
| *** mjturek has quit IRC | 18:30 | |
| *** baha has quit IRC | 18:30 | |
| *** ajya has quit IRC | 18:31 | |
| *** jcoufal has quit IRC | 18:43 | |
| *** mjturek has joined #openstack-ironic | 18:47 | |
| *** baha has joined #openstack-ironic | 18:48 | |
| *** cjloader has quit IRC | 18:49 | |
| *** jcoufal has joined #openstack-ironic | 18:53 | |
| *** jcoufal has quit IRC | 18:54 | |
| *** cjloader has joined #openstack-ironic | 18:54 | |
| *** karimull has quit IRC | 18:54 | |
| *** karimull has joined #openstack-ironic | 18:55 | |
| *** cjloader has quit IRC | 18:56 | |
| *** cjloader has joined #openstack-ironic | 18:57 | |
| *** cjloader has quit IRC | 19:07 | |
| *** jaypipes has joined #openstack-ironic | 19:08 | |
| hamzy | is "openstack baremetal node delete ${UUID}" the equivalent for "ironic node-set-provision-state ${UUID} deleted"? It doesn't seem like it to me. | 19:11 |
| TheJulia | hamzy: no, you want openstack baremetal node undeploy | 19:11 |
| hamzy | ah, thanks TheJulia! | 19:11 |
| *** dprince has joined #openstack-ironic | 19:18 | |
| *** racedo has quit IRC | 19:19 | |
| *** pcaruana has joined #openstack-ironic | 19:27 | |
| *** dprince has quit IRC | 19:35 | |
| *** sekharvajjula has joined #openstack-ironic | 19:39 | |
| *** trown|lunch is now known as trown | 19:42 | |
| *** dsneddon has joined #openstack-ironic | 19:56 | |
| *** yolanda has quit IRC | 20:01 | |
| *** yolanda has joined #openstack-ironic | 20:01 | |
| *** racedo has joined #openstack-ironic | 20:06 | |
| *** milan has quit IRC | 20:17 | |
| *** cjloader has joined #openstack-ironic | 20:18 | |
| *** serlex has left #openstack-ironic | 20:19 | |
| rloo | TheJulia: hi, oath didn't upstream anything wrt background state power thing. | 20:36 |
| *** mjturek has quit IRC | 20:38 | |
| cdearborn | Hey guys, I'm working on setting up a new Fedora dev VM for ironic, and I'm having issues getting the docs to build. The error that I'm seeing is here: http://paste.openstack.org/show/724260/ | 20:39 |
| cdearborn | Any thoughts? Everything else is building correctly. Only the docs build is failing | 20:39 |
| *** dsneddon has quit IRC | 20:43 | |
| *** pcaruana has quit IRC | 20:43 | |
| *** rbartal has quit IRC | 20:55 | |
| *** e0ne has quit IRC | 20:57 | |
| *** e0ne has joined #openstack-ironic | 20:57 | |
| *** e0ne has quit IRC | 20:59 | |
| *** e0ne has joined #openstack-ironic | 20:59 | |
| *** e0ne has quit IRC | 21:00 | |
| *** racedo has quit IRC | 21:00 | |
| *** trown is now known as trown|outtypewww | 21:00 | |
| *** EmilienM_PTO has quit IRC | 21:04 | |
| *** harlowja has quit IRC | 21:04 | |
| *** e0ne has joined #openstack-ironic | 21:04 | |
| *** EmilienM has joined #openstack-ironic | 21:05 | |
| *** ptx0_ has joined #openstack-ironic | 21:05 | |
| *** ptx0 has quit IRC | 21:06 | |
| *** e0ne has quit IRC | 21:07 | |
| *** e0ne has joined #openstack-ironic | 21:07 | |
| *** baha has quit IRC | 21:11 | |
| *** e0ne has quit IRC | 21:21 | |
| *** dsneddon has joined #openstack-ironic | 21:27 | |
| TheJulia | rloo: do you know if there are any plans to? | 21:30 |
| rloo | TheJulia: I don't know :-( | 21:30 |
| TheJulia | cdearborn: that is new :( | 21:31 |
| rloo | TheJulia: we haven't been good wrt upstreaming. too busy drowning, er working downstream. | 21:31 |
| *** cjloader has quit IRC | 21:33 | |
| TheJulia | :( | 21:33 |
| *** dsneddon has quit IRC | 21:35 | |
| *** rcernin has joined #openstack-ironic | 21:45 | |
| *** bfournie has quit IRC | 21:48 | |
| *** dsneddon has joined #openstack-ironic | 21:53 | |
| cdearborn | TheJulia: I've tried deleting the .tox directory and the doc directory, repulled the doc directory from git. Still no love | 21:55 |
| TheJulia | cdearborn: how are you trying to build the docs? | 22:02 |
| *** cdearborn has quit IRC | 22:06 | |
| *** baha has joined #openstack-ironic | 22:13 | |
| *** harlowja has joined #openstack-ironic | 22:14 | |
| *** rloo has quit IRC | 22:16 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Separate unit tests into different classes https://review.openstack.org/577939 | 22:20 |
| *** yolanda_ has joined #openstack-ironic | 22:21 | |
| *** yolanda has quit IRC | 22:22 | |
| markguz | not sure if i should ask this question here or in nova, but since upgrading to ocata my ironic baremetal node has started being picked by the nova-scheduler. | 22:26 |
| markguz | for normal vms | 22:27 |
| markguz | i have everything partitioned in host-aggregates, but they seem to be getting ignored now | 22:27 |
| markguz | anyone else seen this? | 22:27 |
| *** yolanda__ has joined #openstack-ironic | 22:28 | |
| *** yolanda_ has quit IRC | 22:31 | |
| *** baha has quit IRC | 22:36 | |
| *** baha has joined #openstack-ironic | 22:37 | |
| *** baha has quit IRC | 22:39 | |
| *** bfournie has joined #openstack-ironic | 22:40 | |
| *** bfournie has quit IRC | 22:41 | |
| *** bfournie has joined #openstack-ironic | 22:42 | |
| *** ElCoyote_ has quit IRC | 22:52 | |
| *** pmannidi has joined #openstack-ironic | 23:14 | |
| *** markguz has quit IRC | 23:44 | |
| *** gyee has quit IRC | 23:51 | |
| *** mjturek has joined #openstack-ironic | 23:52 | |
| *** rh-jelabarre has quit IRC | 23:53 | |
| *** rpioso is now known as rpioso|afk | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!