| *** goldyfruit_ has joined #openstack-ironic | 00:02 | |
| *** Goneri has quit IRC | 00:12 | |
| *** Goneri has joined #openstack-ironic | 00:14 | |
| *** dsneddon_ has joined #openstack-ironic | 00:23 | |
| *** dsneddon_ has quit IRC | 00:28 | |
| *** dsneddon_ has joined #openstack-ironic | 01:00 | |
| *** tbachman has joined #openstack-ironic | 01:02 | |
| *** dsneddon_ has quit IRC | 01:05 | |
| *** tbachman_ has joined #openstack-ironic | 01:08 | |
| *** tbachman has quit IRC | 01:09 | |
| *** tbachman_ is now known as tbachman | 01:09 | |
| *** dsneddon_ has joined #openstack-ironic | 01:40 | |
| *** dsneddon_ has quit IRC | 01:44 | |
| *** dsneddon_ has joined #openstack-ironic | 02:16 | |
| *** BjoernT has joined #openstack-ironic | 02:20 | |
| *** dsneddon_ has quit IRC | 02:21 | |
| *** BjoernT_ has joined #openstack-ironic | 02:29 | |
| *** BjoernT has quit IRC | 02:31 | |
| *** dsneddon_ has joined #openstack-ironic | 02:52 | |
| *** dsneddon_ has quit IRC | 02:57 | |
| *** bobmel has joined #openstack-ironic | 03:07 | |
| *** dsneddon_ has joined #openstack-ironic | 03:08 | |
| *** bobmel has quit IRC | 03:11 | |
| *** e0ne has joined #openstack-ironic | 03:15 | |
| *** ricolin_ has joined #openstack-ironic | 03:18 | |
| *** e0ne has quit IRC | 03:19 | |
| *** BjoernT_ has quit IRC | 03:26 | |
| *** BjoernT has joined #openstack-ironic | 03:27 | |
| *** BjoernT has quit IRC | 03:28 | |
| *** BjoernT_ has joined #openstack-ironic | 03:29 | |
| *** BjoernT_ has quit IRC | 03:34 | |
| *** BjoernT has joined #openstack-ironic | 03:34 | |
| *** BjoernT has quit IRC | 04:10 | |
| *** BjoernT has joined #openstack-ironic | 04:12 | |
| *** igordc has joined #openstack-ironic | 04:19 | |
| *** BjoernT has quit IRC | 04:26 | |
| *** SuiongNg|ITRI_TW has joined #openstack-ironic | 05:08 | |
| *** bobmel has joined #openstack-ironic | 05:08 | |
| *** bobmel has quit IRC | 05:13 | |
| *** igordc has quit IRC | 05:22 | |
| *** HagunKim has joined #openstack-ironic | 05:28 | |
| *** igordc has joined #openstack-ironic | 05:51 | |
| *** mkowalski__ has quit IRC | 05:56 | |
| HagunKim | Hello, I used ironic baremetal node with centos cloud image. | 05:58 |
|---|---|---|
| HagunKim | When I create a new baremetal instance, there is wrong DNS 10.0.2.3 in /etc/resolv.conf. | 05:58 |
| HagunKim | How can I create a instance without 10.0.2.3 DNS? | 05:58 |
| *** mkowalski__ has joined #openstack-ironic | 05:59 | |
| *** whoami-rajat has joined #openstack-ironic | 06:02 | |
| *** hwoarang has quit IRC | 06:03 | |
| *** hwoarang has joined #openstack-ironic | 06:06 | |
| *** KeenWarrior has joined #openstack-ironic | 06:09 | |
| *** mkowalski__ has quit IRC | 06:10 | |
| openstackgerrit | yatin proposed openstack/networking-baremetal stable/train: Fix unit tests with ironicclient >=3.0.0 https://review.opendev.org/686965 | 06:12 |
| *** pcaruana has joined #openstack-ironic | 06:22 | |
| *** SuiongNg|ITRI_TW has quit IRC | 06:27 | |
| *** mkowalski__ has joined #openstack-ironic | 06:34 | |
| *** rachit7 has joined #openstack-ironic | 06:35 | |
| arne_wiebalck | Good morning, ironic! | 06:36 |
| openstackgerrit | Rachit Kapadia proposed openstack/ironic stable/train: DRAC: Fix a bug for job creation when only required https://review.opendev.org/686969 | 07:03 |
| *** tssurya has joined #openstack-ironic | 07:06 | |
| *** jtomasek has joined #openstack-ironic | 07:10 | |
| *** iurygregory has joined #openstack-ironic | 07:24 | |
| iurygregory | good morning Ironic o/ | 07:24 |
| *** rpittau|afk is now known as rpittau | 07:30 | |
| rpittau | good morning ironic! o/ | 07:30 |
| *** igordc has quit IRC | 07:50 | |
| *** alexmcleod has joined #openstack-ironic | 07:54 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: [WIP] use openstacksdk for ironic module https://review.opendev.org/672179 | 08:01 |
| *** ociuhandu has joined #openstack-ironic | 08:10 | |
| *** pcaruana has quit IRC | 08:13 | |
| *** pcaruana has joined #openstack-ironic | 08:18 | |
| *** ociuhandu has quit IRC | 08:28 | |
| *** ociuhandu has joined #openstack-ironic | 08:28 | |
| *** ociuhandu has quit IRC | 08:33 | |
| *** ociuhandu has joined #openstack-ironic | 08:36 | |
| mgoddard | morning arne_wiebalck iurygregory rpittau, happy monday | 08:40 |
| *** derekh has joined #openstack-ironic | 08:40 | |
| arne_wiebalck | good morning mgoddard o/ | 08:40 |
| *** ociuhandu has quit IRC | 08:41 | |
| iurygregory | morning mgoddard arne_wiebalck rpittau o/ | 08:41 |
| rpittau | hey mgoddard iurygregory arne_wiebalck :) | 08:41 |
| arne_wiebalck | gm iurygregory rpittau :-D | 08:42 |
| * arne_wiebalck likes good morning spanning trees | 08:42 | |
| rachit7 | Good morning rpittau arne_wiebalck iurygregory | 08:46 |
| rpittau | hey rachit7 :) | 08:47 |
| arne_wiebalck | good morning rachit7 o/ | 08:47 |
| rachit7 | rpittau arne_wiebalck 8) | 08:47 |
| rachit7 | Hey rpittau mgoddard can you please add this patch in your review list : https://review.opendev.org/#/c/680660/ | 08:49 |
| patchbot | patch 680660 - ironic - DRAC: Fix a bug for delete_config with multiple co... - 21 patch sets | 08:49 |
| *** e0ne has joined #openstack-ironic | 08:55 | |
| *** dsneddon_ has quit IRC | 09:03 | |
| iurygregory | hey rachit7 | 09:14 |
| *** rachit7 has quit IRC | 09:15 | |
| *** ricolin_ is now known as ricolin | 09:29 | |
| *** dsneddon_ has joined #openstack-ironic | 09:38 | |
| *** whoami-rajat has quit IRC | 09:42 | |
| *** whoami-rajat has joined #openstack-ironic | 09:42 | |
| *** rachit7 has joined #openstack-ironic | 10:11 | |
| *** dsneddon_ has quit IRC | 10:20 | |
| *** derekh has quit IRC | 10:25 | |
| openstackgerrit | Pranjali Srivastava proposed openstack/ironic master: Improve iDrac Documentation https://review.opendev.org/668987 | 10:30 |
| *** jawad_axd has joined #openstack-ironic | 10:33 | |
| *** dsneddon_ has joined #openstack-ironic | 10:38 | |
| *** tbachman has quit IRC | 10:40 | |
| openstackgerrit | Pranjali Srivastava proposed openstack/ironic master: Improve iDrac Documentation https://review.opendev.org/668987 | 10:42 |
| *** dsneddon_ has quit IRC | 10:42 | |
| openstackgerrit | Merged openstack/ironic stable/train: Fix devstack installation failure https://review.opendev.org/686893 | 10:50 |
| *** dsneddon_ has joined #openstack-ironic | 11:09 | |
| *** pcaruana has quit IRC | 11:11 | |
| *** derekh has joined #openstack-ironic | 11:12 | |
| *** Lucas_Gray has joined #openstack-ironic | 11:41 | |
| *** pcaruana has joined #openstack-ironic | 11:42 | |
| *** dsneddon_ has quit IRC | 11:49 | |
| *** rh-jelabarre has joined #openstack-ironic | 11:53 | |
| *** HagunKim has quit IRC | 11:57 | |
| *** Goneri has quit IRC | 11:58 | |
| *** dsneddon_ has joined #openstack-ironic | 11:59 | |
| *** dsneddon_ has quit IRC | 12:04 | |
| *** Lucas_Gray has quit IRC | 12:07 | |
| *** tbachman has joined #openstack-ironic | 12:07 | |
| *** Lucas_Gray has joined #openstack-ironic | 12:09 | |
| *** goldyfruit_ has quit IRC | 12:21 | |
| *** dsneddon_ has joined #openstack-ironic | 12:32 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: [WIP] use openstacksdk for ironic module https://review.opendev.org/672179 | 12:44 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Raising minimum version of oslo.db https://review.opendev.org/687016 | 12:50 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Raising minimum version of oslo.db https://review.opendev.org/687018 | 12:52 |
| jroll | morning ironic | 13:04 |
| *** Goneri has joined #openstack-ironic | 13:08 | |
| *** jcoufal has joined #openstack-ironic | 13:10 | |
| *** jcoufal has quit IRC | 13:12 | |
| *** dsneddon_ has quit IRC | 13:12 | |
| *** BjoernT has joined #openstack-ironic | 13:18 | |
| *** rloo has joined #openstack-ironic | 13:20 | |
| arne_wiebalck | hey jroll o/$ | 13:22 |
| *** derekh has quit IRC | 13:24 | |
| *** ociuhandu has joined #openstack-ironic | 13:25 | |
| *** beekneemech is now known as bnemec | 13:29 | |
| *** ociuhandu has quit IRC | 13:31 | |
| *** tzumainn has joined #openstack-ironic | 13:34 | |
| *** goldyfruit_ has joined #openstack-ironic | 13:35 | |
| *** ociuhandu has joined #openstack-ironic | 13:40 | |
| *** dsneddon_ has joined #openstack-ironic | 13:46 | |
| *** jawad_axd has quit IRC | 13:50 | |
| *** jawad_axd has joined #openstack-ironic | 13:51 | |
| *** jawad_axd has quit IRC | 13:51 | |
| *** jawad_axd has joined #openstack-ironic | 13:52 | |
| *** jawad_axd has quit IRC | 13:52 | |
| *** ociuhandu has quit IRC | 13:53 | |
| *** ociuhandu has joined #openstack-ironic | 13:54 | |
| *** ociuhandu has quit IRC | 13:57 | |
| *** ociuhandu has joined #openstack-ironic | 13:57 | |
| TheJulia | good morning everyone | 14:01 |
| TheJulia | My coffee is very clearly not strong enough this morning | 14:06 |
| rpittau | morning TheJulia :) | 14:12 |
| *** csmart has quit IRC | 14:20 | |
| *** ajya[m] has quit IRC | 14:20 | |
| *** rachit7 has quit IRC | 14:27 | |
| TheJulia | too many emails.... | 14:29 |
| *** Lucas_Gray has quit IRC | 14:33 | |
| *** hjensas has quit IRC | 14:34 | |
| *** TxGirlGeek has joined #openstack-ironic | 14:36 | |
| *** hjensas has joined #openstack-ironic | 14:41 | |
| TheJulia | rpioso: What is the root cause of oob inspection when not in uefi mode? | 14:41 |
| iurygregory | morning TheJulia o/ | 14:45 |
| *** stendulker has joined #openstack-ironic | 14:46 | |
| *** jawad_axd has joined #openstack-ironic | 14:48 | |
| *** cdearborn has joined #openstack-ironic | 14:51 | |
| *** jawad_axd has quit IRC | 14:52 | |
| rpioso | TheJulia: I don't understand your question. | 14:56 |
| rpioso | Good morning, ironicers. | 14:56 |
| iurygregory | morning rpioso | 14:59 |
| rpioso | iurygregory: o/ | 15:00 |
| TheJulia | #startmeeting ironic | 15:00 |
| TheJulia | o/ | 15:00 |
| openstack | Meeting started Mon Oct 7 15:00:19 2019 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 |
| *** bdodd has joined #openstack-ironic | 15:00 | |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| TheJulia | o/ everyone | 15:00 |
| hjensas | o/ | 15:00 |
| rpioso | \o | 15:00 |
| etingof | \o | 15:00 |
| bdodd | o/ | 15:00 |
| TheJulia | rpioso: I guess we'll get to that topic, just curious after reading the meeting agenda | 15:00 |
| iurygregory | o/ | 15:00 |
| stendulker | o/ | 15:00 |
| cdearborn | \o | 15:00 |
| rpioso | TheJulia: ack | 15:01 |
| TheJulia | Greetings everyone! | 15:01 |
| TheJulia | Our meeting agenda can be found on the wiki! | 15:01 |
| *** TxGirlGeek has quit IRC | 15:01 | |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| jroll | \o | 15:01 |
| arne_wiebalck | o/ | 15:02 |
| TheJulia | #topic Announcements / Reminders | 15:02 |
| *** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)" | 15:02 | |
| TheJulia | I guess I can say I'm back! Until Shanghai that is. | 15:02 |
| rpioso | \o/ | 15:02 |
| TheJulia | A quick reminder, The Shanghai summit is in just about one month, so I'm sure presentations will be the order of the coming weeks. | 15:02 |
| TheJulia | Does anyone have any reminders or things they would like to announce? | 15:03 |
| rpittau | o/ | 15:03 |
| *** mkrai has joined #openstack-ironic | 15:04 | |
| rloo | (o/) | 15:04 |
| mkrai | o/ | 15:04 |
| TheJulia | Speaking of reminders! I guess if there are any thoughts for things that need to be discussed during the PTG, it would be good to add/note that on the etherpad. | 15:04 |
| TheJulia | #link https://etherpad.openstack.org/p/PVG-Ironic-Planning | 15:04 |
| mgoddard | \o (sorry I'm late) | 15:05 |
| TheJulia | no worries! | 15:05 |
| TheJulia | Looks like we had no action items from the last meeting, so I guess we can jump directly to subteam status review if there are no objections? | 15:05 |
| * iurygregory in doubt why the etherpad has PVG o.o | 15:06 | |
| TheJulia | iurygregory: Shanhai Pudong International Airport | 15:07 |
| TheJulia | Shanghai | 15:07 |
| rpioso | lol | 15:07 |
| iurygregory | good! | 15:07 |
| TheJulia | Moving on | 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:08 |
| TheJulia | Starting around line 226 | 15:08 |
| rpioso | Who's taking over the Bare Metal Program from hogepodge? | 15:09 |
| arne_wiebalck | FWIU the fondation will not have the resources to have someone working on this. | 15:10 |
| arne_wiebalck | foundation | 15:10 |
| TheJulia | Taht is a very good question rpioso | 15:10 |
| TheJulia | I did reach out to the foundation, after discussing this with arne_wiebalck, to at least identify a primary point of contact that we as a larger community could co-ordinate with on such things should the community carry things forward. | 15:10 |
| *** bfournie has joined #openstack-ironic | 15:11 | |
| TheJulia | I suspect my email was lost, because multiple people were posting from all sorts of places around the world around and after I sent the emails, so I'll try to follow-up in a few days | 15:11 |
| rpioso | TheJulia: I look forward to hearing what you learn. arne_wiebalck, TheJulia: Thank you! | 15:11 |
| TheJulia | A question popped up on the etherpad regarding python 2.x testing, and I believe we can start to drop py2 testing this cycle. We should likely make sure all of our jobs are py3 soon-ish. | 15:12 |
| TheJulia | Would someone be willing to take the action item to check the documentation and let us know? | 15:13 |
| rpittau | TheJulia: that was my question as I saw some projects already starting tp drop py2 testing | 15:13 |
| iurygregory | I think we have python3 jobs for all projects | 15:13 |
| iurygregory | maybe bifrost doesn't | 15:13 |
| TheJulia | iurygregory: our integration tests are still somewhat mixed unless we quietly changed the defaults all over and didn't fix job names | 15:13 |
| mkrai | TheJulia, I will check the documentation. Thanks! | 15:13 |
| rpittau | TheJulia: I can check that, can't wait to get rid of Py 2 :D | 15:14 |
| TheJulia | #action mkrai to check Python testing documentation and let us know when we can begin removing python2 | 15:14 |
| TheJulia | rpittau: So... Python3.8 is something else we need to circle back kto | 15:14 |
| iurygregory | TheJulia, i can look at the integration tests | 15:14 |
| rpittau | TheJulia: yep, and py3.7 too, like oslo.db min version compatibility | 15:14 |
| TheJulia | #link https://etherpad.openstack.org/p/Python_3.8_Deprecation_Hunt | 15:15 |
| TheJulia | iurygregory: action item for you to check py3.8 unit testing? or am I misinterpretting your comment | 15:15 |
| TheJulia | oh | 15:15 |
| TheJulia | integration tests | 15:15 |
| TheJulia | okay | 15:15 |
| *** jawad_axd has joined #openstack-ironic | 15:15 | |
| * TheJulia needs more coffee | 15:15 | |
| TheJulia | #action iurygregory to look at changing all of our integration tests over to py3 | 15:15 |
| rpittau | TheJulia: I'll keep working on py38 and py37 | 15:16 |
| TheJulia | iurygregory: one thing to keep in mind, we have a duplicate or two for jobs that were cross py2/py3 | 15:16 |
| TheJulia | iurygregory: so we can remove the duplicate :) | 15:16 |
| TheJulia | that uses py2 | 15:16 |
| TheJulia | #action rpittau to look at py37/py38 comparability issues | 15:17 |
| iurygregory | TheJulia, ack | 15:17 |
| TheJulia | arne_wiebalck: so we got power callback done this cycle, but nothing state wise right? | 15:17 |
| arne_wiebalck | TheJulia: correct | 15:17 |
| TheJulia | okay | 15:17 |
| TheJulia | So that is largely the list. I think those action items do cover the most important things in the next few weeks which should set us on a good path for the next cycle. | 15:18 |
| TheJulia | Is everyone ready to move on? | 15:19 |
| rpittau | let's | 15:19 |
| TheJulia | #topic Priorities for the coming week | 15:19 |
| *** openstack changes topic to "Priorities for the coming week (Meeting topic: ironic)" | 15:19 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:19 |
| TheJulia | Starting at line 151 | 15:20 |
| TheJulia | So it looks like most of the items from last week need some reviews/rebases, or revision. I think we just clean the list up and use it as is for this week. That is unless there are additional items people are aware of at this time? | 15:21 |
| *** jawad_axd has quit IRC | 15:21 | |
| * TheJulia removes merged items from the list | 15:21 | |
| hjensas | I added https://review.opendev.org/686965 to the backports section. | 15:21 |
| patchbot | patch 686965 - networking-baremetal (stable/train) - Fix unit tests with ironicclient >=3.0.0 - 1 patch set | 15:21 |
| rpittau | this ^^ has 1 +2 | 15:22 |
| TheJulia | That seems reasonable | 15:22 |
| TheJulia | Anything else? | 15:22 |
| TheJulia | Any other backports? | 15:22 |
| rpittau | yep, the log level change | 15:23 |
| rpittau | https://review.opendev.org/#/q/b8fa188a29481b11b2f3117cdeecfd920a8f5a0e | 15:23 |
| hjensas | .. if puppet-ironic counts - https://review.opendev.org/#/q/Ic95dbf1867fb5397f1b6d8f23466910a97051cb6 | 15:24 |
| TheJulia | Anything else today? | 15:24 |
| mkrai | I added this to the ironic list https://review.opendev.org/#/c/675853/ | 15:24 |
| patchbot | patch 675853 - ironic - Add support for replacing WSME - 3 patch sets | 15:24 |
| TheJulia | hjensas: largely that is TripleO controlled, we don't have have rights on puppet-ironic | 15:24 |
| TheJulia | mkrai: thanks | 15:24 |
| TheJulia | That seems reasonable | 15:24 |
| hjensas | TheJulia: ack | 15:24 |
| rpittau | hjensas: no train yet for puppet-ironic ? | 15:24 |
| TheJulia | mkrai: adding a quick description | 15:24 |
| TheJulia | mkrai: nevermind, I see you just did it! | 15:25 |
| iurygregory | Im core on puppet-ironic | 15:25 |
| TheJulia | Thanks! | 15:25 |
| hjensas | rpittau: the change merged before train was cut on master. | 15:25 |
| mkrai | TheJulia, Thanks! | 15:25 |
| rpittau | hjensas: oh kk thanks | 15:25 |
| iurygregory | hjensas, +A on puppet-ironic =) | 15:25 |
| TheJulia | \o/ | 15:25 |
| hjensas | iurygregory: thank. | 15:25 |
| TheJulia | Excellent! | 15:25 |
| TheJulia | Anything else before we move on? | 15:25 |
| *** ociuhandu has quit IRC | 15:27 | |
| TheJulia | Then I guess we're good to move to Discussion | 15:27 |
| TheJulia | #topic Discussion | 15:27 |
| *** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:27 | |
| *** ociuhandu has joined #openstack-ironic | 15:27 | |
| TheJulia | First discussion item comes from rpioso! | 15:27 |
| rpioso | TheJulia: Thank you | 15:28 |
| TheJulia | rpioso: Would you like to describe the situation and root problem that exists? | 15:28 |
| rpioso | Absolutely | 15:28 |
| rpioso | About a year ago, Dell EMC added UEFI boot mode support to the idrac hardware type. | 15:28 |
| rpioso | At that time, we did not realize IPA in-band inspection sets boot_mode on the ironic node. | 15:29 |
| rpioso | For UEFI boot mode, it must be set to 'uefi'; otherwise, deployment fails. | 15:30 |
| rpioso | If we had known about IPA's behavior, idrac's out-of-band inspection would have been changed to do the same. | 15:30 |
| rpioso | However, at the time, we were not aware of it. | 15:31 |
| TheJulia | Okay, so what actually fails today? | 15:31 |
| rpioso | Therefore, we have always considered it a bug. | 15:31 |
| *** ociuhandu has quit IRC | 15:31 | |
| TheJulia | Where does a deployment fail today? | 15:32 |
| rpioso | If memory serves, every booting of the ramdisk. | 15:32 |
| TheJulia | So your saying, UEFI ramdisk booting is broken on idrac hardware? | 15:32 |
| TheJulia | the driver interaction their-in | 15:33 |
| rpioso | When boot_mode is not set to 'uefi'. | 15:33 |
| cdearborn | if the user uses IB inspection, then it works fine. if they use OOB introspection then it doesn't work because OOB introspection does not set boot_mode | 15:33 |
| rpioso | Right! IPA in-band inspection causes it to be automagically set. | 15:33 |
| cdearborn | needless to say, this can be very confusing and lead to a bad user experience | 15:34 |
| *** ociuhandu has joined #openstack-ironic | 15:34 | |
| TheJulia | agreed, also seems rather confusing. So the minimal impact fix sounds like boot mode should be set as part of OOB inspection? Or should boot mode be asserted just prior to ramdisk being booted? I thought the latter is what we ended up doing and should be in the code paths, however it may rely upon an method that might not be present for idrac or something along those lines | 15:35 |
| jroll | so what is the question at hand here? | 15:36 |
| stendulker | Why do you need to set boot mode in OOB inspection? Do you power on the node? | 15:36 |
| TheJulia | jroll: they want to backport a fix | 15:36 |
| TheJulia | well, feature/fix which entirely depends on point of view. | 15:36 |
| jroll | TheJulia: thanks, that's what I thought, just making sure | 15:36 |
| rpioso | We strongly prefer the former, so that OOB inspection is on par with in-band. That what the topic's change proposes. | 15:36 |
| * TheJulia gives coffee to the opendev webpage | 15:38 | |
| *** alexmcleod has quit IRC | 15:38 | |
| * iurygregory hopes TheJulia is not using google-chrome, otherwise she would need to give memory | 15:39 | |
| TheJulia | iurygregory: heh | 15:39 |
| jroll | both tonyb and dtantsur|afk indicate that they believe it is a feature, though they seem close to the line | 15:39 |
| jroll | I tend to trust their opinion, personally | 15:39 |
| TheJulia | jroll: where do you see that? | 15:39 |
| jroll | TheJulia: comments in https://review.opendev.org/#/c/656425/2 | 15:40 |
| patchbot | patch 656425 - ironic (stable/rocky) - Set boot_mode in node properties during OOB Intros... - 2 patch sets | 15:40 |
| TheJulia | I ask because there was another issue that was similar in nature | 15:40 |
| rloo | it seems like a feature to me. but it causes problems cuz w/o it, things are inconsistent to your customers/users. | 15:40 |
| cdearborn | stendulker, we don't want to set boot mode on the server, we want to discover what boot mode the server is set to as part of OOB inspection. this will make servers that are in uefi boot mode "just work" | 15:40 |
| stendulker | cdearborn: oh ok. Thanks. | 15:40 |
| rloo | so this change went into master in March -- is that in Stein then? | 15:41 |
| TheJulia | Separate change, the change being asked abou tis | 15:41 |
| jroll | rloo: it is in stein | 15:41 |
| TheJulia | oh, nope | 15:41 |
| rloo | sorry, is there a PR to backport this already? i thought that as the link jroll gave | 15:42 |
| TheJulia | I'm wrong | 15:42 |
| TheJulia | rloo: sorry, I got confused | 15:42 |
| jroll | 656425 is the rocky backport, yes | 15:42 |
| rloo | and who has a link to the docs wrt backporting so I can read/remember what is allowed? | 15:42 |
| jroll | https://docs.openstack.org/project-team-guide/stable-branches.html | 15:42 |
| TheJulia | I need to dig through through the code because there seems to be a separate bug | 15:43 |
| TheJulia | Which should negate the need to make any change to the inspection path | 15:43 |
| jroll | rocky is in "maintained" mode per https://releases.openstack.org | 15:43 |
| *** tssurya has quit IRC | 15:43 | |
| jroll | actually, re-reading the "appropriate fixes" section, I'm probably +0 on backporting this. I believe it's technically a feature, if we're only looking at inspector. if we're looking at a full baremetal environment/workflow, it seems like more of a bug. it definitely fits the criteria given here, otherwise. | 15:45 |
| rpioso | I believe idrac UEFI boot mode support landed in Queens. | 15:45 |
| rpioso | A very similar idrac bug fix/feature was recently backported and merged into stable/rocky, https://review.opendev.org/#/c/648360/ | 15:46 |
| patchbot | patch 648360 - ironic (stable/rocky) - DRAC: Fix OOB introspection to use pxe_enabled fla... (MERGED) - 32 patch sets | 15:46 |
| rpioso | That merged change simplified the already small-ish change we're discussing. | 15:46 |
| rloo | Some types of changes are completely forbidden: New features | 15:46 |
| rloo | the 'story' is an RFE: https://storyboard.openstack.org/#!/story/2005119 | 15:46 |
| rpioso | We're discussing what Dell EMC has viewed as a bug from day one. | 15:46 |
| TheJulia | The Story also doesn't have the level of detail that we now understand through the meeting | 15:47 |
| rloo | rpioso: is that the story ^^ ? | 15:47 |
| JayF | Nobody has asked the reverse question: if this patch were backported, what's the likelyhook that it would break an existing user | 15:47 |
| rloo | if it doesn't have the level of detail -- shouldn't someone update it? | 15:47 |
| rpioso | rloo: We didn't label it that. | 15:47 |
| JayF | it sounds like pretty close to zero if it's considered a bug by iDRAC users? | 15:47 |
| TheJulia | In late 2017, we retooled boot mode setting code, in that process the prepare_ramdisk method should be setting the boot mode as user/system requested, so completely absent is a valid case for a user | 15:47 |
| * jroll adds a full comment and a +1 on https://review.opendev.org/#/c/656425 | 15:48 | |
| patchbot | patch 656425 - ironic (stable/rocky) - Set boot_mode in node properties during OOB Intros... - 2 patch sets | 15:48 |
| rloo | rpioso: if you didn't label it that, and given the docs wrt backporting, how would you decide, given the current description in the story? | 15:48 |
| *** ociuhandu has quit IRC | 15:48 | |
| rloo | so i'm fine wrt the impact, risk etc. all are low and to make things consistent, it makes sense. | 15:49 |
| *** ociuhandu has joined #openstack-ironic | 15:49 | |
| TheJulia | so I see the issue with the code since 2017, the idrac management interfaces lack the get/set bood mode methods | 15:49 |
| rloo | however, whoever wants this change, needs to do their part (which i guess they're doing now). but change that to a bug not an rfe. or at least elaborate in the story. | 15:49 |
| TheJulia | rloo: +++ | 15:49 |
| rpioso | TheJulia: Correct. They're optional methods. | 15:50 |
| TheJulia | not if you want boot mode assetion to work correctly for your hardware type prior to ramdisk booting | 15:50 |
| jroll | the fact that the release note claims it as a feature tells me that Dell EMC has not viewed this as a bug from day one | 15:50 |
| jroll | https://review.opendev.org/#/c/656425/2/releasenotes/notes/set-boot-mode-4c42b3fd0b5f5b37.yaml | 15:50 |
| patchbot | patch 656425 - ironic (stable/rocky) - Set boot_mode in node properties during OOB Intros... - 2 patch sets | 15:50 |
| rpioso | jroll: dtantsur tagged it as an RFE. We did not agree with that. | 15:50 |
| jroll | that said, I'm good with backporting it | 15:50 |
| rpioso | Still don't. | 15:51 |
| rpioso | A follow-on change could change the release note from feature to bug fix. | 15:51 |
| jroll | sure, I said the same in the rocky review | 15:51 |
| TheJulia | If we can clarify what is broken in the user experience/interaction and focus on that, then I think that would be the correct path for mutual understanding as well. | 15:52 |
| jroll | anyway, I think from a user POV, this is a bug. from an ironic ecosystem POV, it's a bug. for inspector specifically, it's a feature. the right thing to do is to backport it. | 15:52 |
| rpioso | TheJulia: We're glad to do that. Where? | 15:52 |
| *** ociuhandu has quit IRC | 15:53 | |
| TheJulia | rpioso: In a story in storyboard so we focus on what is broken, the bug | 15:53 |
| rpioso | TheJulia: +1 | 15:54 |
| TheJulia | I completely agree with jroll, it is all about how the context is set for reviewers | 15:54 |
| rpioso | TheJulia: May the existing story be updated. | 15:54 |
| TheJulia | yes, please add a task to the changeset | 15:54 |
| TheJulia | the task: ##### tag | 15:54 |
| rpioso | Hrm ... the original change doesn't have a tag :-( | 15:55 |
| jroll | I have to run for another meeting - please ping me to make my +1 a +2 if we end up agreeing to backport it :) | 15:55 |
| TheJulia | It is fine to edit the commit message | 15:55 |
| rpioso | +1 | 15:55 |
| TheJulia | jroll: I think we agree, but I need to clear my mind... and do expense reporting first :) | 15:56 |
| cdearborn | +1 thx much! | 15:56 |
| rpioso | Thank you! | 15:56 |
| jroll | sure, no rush :) | 15:56 |
| TheJulia | Okay, we have like four minutes left | 15:56 |
| TheJulia | I think we'll skip RFE review and just jump to open discussion for a couple minutes | 15:57 |
| TheJulia | And we can review the RFE next week | 15:57 |
| TheJulia | (For context, that RFE is https://storyboard.openstack.org/#!/story/2003594) | 15:57 |
| TheJulia | #topic Open Discussion | 15:57 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:57 | |
| TheJulia | Does anyone have anything they would like to bring up in the next couple of minutes? | 15:57 |
| cdearborn | qq: do we have any guidelines for docs changes? | 15:57 |
| TheJulia | cdearborn: what do you mean by guidelines? | 15:58 |
| TheJulia | I think the question may next be what do you feel needs to change? | 15:58 |
| cdearborn | format, when to ``asdf`` things, etc | 15:58 |
| TheJulia | That is a really good question | 15:58 |
| cdearborn | we are at long last going to be bringing the iDRAC drivers docs up to date | 15:58 |
| TheJulia | I don't think there are a unified formatting guidelines, but there are structural guidelines | 15:59 |
| *** TxGirlGeek has joined #openstack-ironic | 15:59 | |
| TheJulia | but for a driver, structural really doesn't matter | 15:59 |
| rpioso | TheJulia: Another qq. Have community goals been determined for Ussuri? I wonder if projects' implementation of driver matrices for the OpenStack Marketplace is a goal. | 15:59 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `instance_info/kernel_append_params` to `redfish` https://review.opendev.org/687092 | 15:59 |
| cdearborn | and just wondering what tech docs guidelines we should adhere to if any to ease reviews, etc | 15:59 |
| tzumainn | I've gotten good feedback on https://review.opendev.org/#/c/681066/ , if anyone else has comments I'd be happy to hear them! | 15:59 |
| patchbot | patch 681066 - ironic-specs - Expose node owner information to oslo.policy checks - 7 patch sets | 15:59 |
| TheJulia | `constant` <-- constant such as driver name or variable kind of thing. :) | 15:59 |
| TheJulia | rpioso: I think we would want the operator feedback session first before we really set that forward path | 16:00 |
| TheJulia | tzumainn: awesome! | 16:00 |
| TheJulia | Anyway, we're at our time. Thanks for attending today's meeting! | 16:00 |
| TheJulia | #endmeeting | 16:00 |
| *** 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-discuss mailing list." | 16:00 | |
| openstack | Meeting ended Mon Oct 7 16:00:55 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:00 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-10-07-15.00.html | 16:00 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-10-07-15.00.txt | 16:00 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-10-07-15.00.log.html | 16:01 |
| TheJulia | cdearborn: I would honestly ask in #openstack-doc | 16:01 |
| TheJulia | since some of the doc writers and they may have some feedback. There are a few things in the contributor guides, nothing ironic specific | 16:01 |
| TheJulia | But if there is any one single place or doc, they should know of it | 16:01 |
| cdearborn | TheJulia - will do - thx | 16:01 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: [WIP] use openstacksdk for ironic module https://review.opendev.org/672179 | 16:01 |
| rpittau | bye everyone! o/ | 16:02 |
| *** rpittau is now known as rpittau|adk | 16:02 | |
| *** rpittau|adk is now known as rpittau|afk | 16:02 | |
| rpioso | rpittau|afk: Good nite! | 16:03 |
| TheJulia | Now, I just wish I didn't come back with a sinus infection | 16:08 |
| *** goldyfruit___ has joined #openstack-ironic | 16:09 | |
| *** stendulker has quit IRC | 16:09 | |
| *** goldyfruit_ has quit IRC | 16:11 | |
| *** e0ne has quit IRC | 16:17 | |
| *** iurygregory has quit IRC | 16:17 | |
| *** mkrai has quit IRC | 16:18 | |
| *** ociuhandu has joined #openstack-ironic | 16:18 | |
| *** ociuhandu_ has joined #openstack-ironic | 16:20 | |
| arne_wiebalck | bye, everyone o/ | 16:21 |
| TheJulia | goodnight! | 16:22 |
| *** gyee has joined #openstack-ironic | 16:22 | |
| *** gyee_ has joined #openstack-ironic | 16:22 | |
| *** gyee_ has quit IRC | 16:23 | |
| *** gyee has quit IRC | 16:23 | |
| *** gyee has joined #openstack-ironic | 16:23 | |
| *** ociuhandu has quit IRC | 16:23 | |
| *** ociuhandu_ has quit IRC | 16:25 | |
| *** trident has quit IRC | 16:33 | |
| *** ajya[m] has joined #openstack-ironic | 16:35 | |
| *** TxGirlGeek has quit IRC | 16:35 | |
| *** ociuhandu has joined #openstack-ironic | 16:36 | |
| *** trident has joined #openstack-ironic | 16:37 | |
| *** billp has joined #openstack-ironic | 16:40 | |
| *** ociuhandu has quit IRC | 16:41 | |
| *** mkrai has joined #openstack-ironic | 16:48 | |
| *** mkrai has quit IRC | 16:54 | |
| *** ociuhandu has joined #openstack-ironic | 16:54 | |
| *** mgoddard has quit IRC | 16:57 | |
| *** mgoddard has joined #openstack-ironic | 17:02 | |
| *** TxGirlGeek has joined #openstack-ironic | 17:05 | |
| *** pcaruana has quit IRC | 17:21 | |
| *** dsneddon_ has quit IRC | 17:25 | |
| *** csmart has joined #openstack-ironic | 17:36 | |
| *** bdodd has quit IRC | 17:39 | |
| *** dsneddon_ has joined #openstack-ironic | 17:59 | |
| *** pcaruana has joined #openstack-ironic | 18:00 | |
| *** ociuhandu has quit IRC | 18:01 | |
| *** ociuhandu has joined #openstack-ironic | 18:02 | |
| *** goldyfruit___ has quit IRC | 18:03 | |
| *** goldyfruit has joined #openstack-ironic | 18:06 | |
| *** ociuhandu has quit IRC | 18:06 | |
| *** ociuhandu has joined #openstack-ironic | 18:16 | |
| *** ociuhandu has quit IRC | 18:20 | |
| *** ricolin has quit IRC | 18:21 | |
| *** ociuhandu has joined #openstack-ironic | 18:21 | |
| *** ociuhandu has quit IRC | 18:23 | |
| *** ociuhandu has joined #openstack-ironic | 18:24 | |
| *** e0ne has joined #openstack-ironic | 18:28 | |
| *** ociuhandu has quit IRC | 18:29 | |
| *** ociuhandu has joined #openstack-ironic | 18:29 | |
| *** gyee has quit IRC | 18:33 | |
| *** ociuhandu has quit IRC | 18:34 | |
| *** ociuhandu has joined #openstack-ironic | 18:39 | |
| *** ociuhandu has quit IRC | 18:49 | |
| *** tbachman has quit IRC | 18:54 | |
| *** tbachman has joined #openstack-ironic | 19:09 | |
| *** spartakos has joined #openstack-ironic | 19:16 | |
| *** gyee has joined #openstack-ironic | 19:18 | |
| *** tbachman has quit IRC | 19:27 | |
| *** pcaruana has quit IRC | 19:32 | |
| *** spartakos has quit IRC | 19:50 | |
| *** spartakos has joined #openstack-ironic | 19:59 | |
| *** cdearborn has quit IRC | 20:01 | |
| *** spartakos has quit IRC | 20:02 | |
| *** rnoriega- has quit IRC | 20:03 | |
| *** igordc has joined #openstack-ironic | 20:13 | |
| *** dsneddon_ has quit IRC | 20:20 | |
| *** dsneddon_ has joined #openstack-ironic | 20:21 | |
| *** naved001 has quit IRC | 20:41 | |
| *** tonyb has joined #openstack-ironic | 20:53 | |
| *** Goneri has quit IRC | 20:55 | |
| *** goldyfruit_ has joined #openstack-ironic | 21:05 | |
| *** goldyfruit has quit IRC | 21:08 | |
| *** goldyfruit_ has quit IRC | 21:22 | |
| *** TxGirlGeek has quit IRC | 21:30 | |
| *** BjoernT has quit IRC | 21:35 | |
| *** tbachman has joined #openstack-ironic | 21:41 | |
| *** tbachman has quit IRC | 21:48 | |
| *** tbachman has joined #openstack-ironic | 21:52 | |
| *** jtomasek has quit IRC | 21:54 | |
| *** jtomasek has joined #openstack-ironic | 21:55 | |
| *** jtomasek has quit IRC | 21:55 | |
| *** whoami-rajat has quit IRC | 22:02 | |
| *** trident has quit IRC | 22:03 | |
| *** trident has joined #openstack-ironic | 22:04 | |
| *** goldyfruit_ has joined #openstack-ironic | 22:10 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: grub configuration should use user kernel & ramdisk https://review.opendev.org/685539 | 22:23 |
| *** e0ne has quit IRC | 22:26 | |
| *** BjoernT has joined #openstack-ironic | 22:37 | |
| openstackgerrit | Richard G. Pioso proposed openstack/ironic master: Add Redfish vmedia boot interface to idrac HW type https://review.opendev.org/672498 | 22:37 |
| rpioso | dtantsur, etingof: Please review ^^^ | 22:38 |
| *** BjoernT has quit IRC | 22:42 | |
| * rpioso meant dtantsur|afk | 22:47 | |
| rpioso | dtantsur|afk: ^^^ | 22:47 |
| rpioso | TheJulia: You, too, please ^^^ | 22:47 |
| * rpioso looks forward to Halloween, when scary things are invisible ;-) | 22:49 | |
| *** rcernin has joined #openstack-ironic | 23:08 | |
| *** hwoarang has quit IRC | 23:12 | |
| openstackgerrit | Merged openstack/ironic stable/train: Change log level based on node status https://review.opendev.org/686594 | 23:17 |
| *** goldyfruit_ has quit IRC | 23:20 | |
| openstackgerrit | Merged openstack/ironic stable/stein: Change log level based on node status https://review.opendev.org/686595 | 23:23 |
| *** hwoarang has joined #openstack-ironic | 23:25 | |
| *** hoonetorg has quit IRC | 23:53 | |
| *** goldyfruit_ has joined #openstack-ironic | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!