| *** munimeha1 has quit IRC | 00:06 | |
| TheJulia | codyhill: good to hear, sorry for the headache :( | 00:24 |
|---|---|---|
| *** mjturek has quit IRC | 00:25 | |
| rpioso | TheJulia: I created a new story -- https://storyboard.openstack.org/#!/story/2004592 | 00:28 |
| rpioso | I would appreciate any feedback you and other cores could provide. | 00:29 |
| *** mjturek has joined #openstack-ironic | 00:57 | |
| *** rpioso is now known as rpioso|afk | 00:57 | |
| rpioso|afk | Good nite :) | 00:57 |
| *** dsneddon has quit IRC | 00:59 | |
| *** mjturek has quit IRC | 01:06 | |
| *** dnuka has joined #openstack-ironic | 01:08 | |
| dnuka | good morning everyone :) | 01:09 |
| *** sthussey has quit IRC | 01:19 | |
| *** rh-jelabarre has quit IRC | 01:24 | |
| *** dsneddon has joined #openstack-ironic | 01:25 | |
| *** dsneddon has quit IRC | 01:30 | |
| *** hwoarang has quit IRC | 01:48 | |
| *** hwoarang has joined #openstack-ironic | 01:53 | |
| *** mbeierl has quit IRC | 02:06 | |
| *** mbeierl has joined #openstack-ironic | 02:11 | |
| *** jrist has quit IRC | 02:25 | |
| *** rloo has quit IRC | 02:59 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: [wip] Single process ironic https://review.openstack.org/623569 | 03:23 |
| *** dsneddon has joined #openstack-ironic | 03:26 | |
| *** dnuka has quit IRC | 03:32 | |
| *** dsneddon has quit IRC | 03:40 | |
| *** stendulker has joined #openstack-ironic | 03:42 | |
| *** hwoarang has quit IRC | 03:56 | |
| *** hwoarang has joined #openstack-ironic | 04:03 | |
| *** v12aml has quit IRC | 04:10 | |
| openstackgerrit | Tony Breeds proposed openstack/ironic stable/rocky: [Stable Only] rename ipmi_disable_timeout to ipmi_disable_boot_timeout https://review.openstack.org/624569 | 04:29 |
| tonyb | TheJulia: ^^ that's my attempt at backporting just the driver_info rename | 04:29 |
| *** dsneddon has joined #openstack-ironic | 04:36 | |
| *** dsneddon has quit IRC | 04:41 | |
| *** dhellmann has quit IRC | 05:19 | |
| *** dhellmann has joined #openstack-ironic | 05:20 | |
| *** jaganathan has joined #openstack-ironic | 05:25 | |
| *** dnuka has joined #openstack-ironic | 05:58 | |
| *** hwoarang has quit IRC | 06:16 | |
| *** hwoarang has joined #openstack-ironic | 06:22 | |
| *** dsneddon has joined #openstack-ironic | 06:31 | |
| *** dsneddon has quit IRC | 06:36 | |
| openstackgerrit | Kaifeng Wang proposed openstack/python-ironicclient master: Support for conductors exposed from API https://review.openstack.org/624032 | 06:40 |
| *** dsneddon has joined #openstack-ironic | 06:41 | |
| *** hjensas has quit IRC | 06:54 | |
| *** trungnv has joined #openstack-ironic | 06:59 | |
| *** moshele has joined #openstack-ironic | 07:00 | |
| *** e0ne has joined #openstack-ironic | 07:13 | |
| iurygregory | good morning everyone o/ | 07:24 |
| dnuka | morning iurygregory o/ | 07:25 |
| openstackgerrit | jiapei proposed openstack/ironic master: Fix XClarity driver management defect https://review.openstack.org/624000 | 07:26 |
| iurygregory | dnuka o/ | 07:26 |
| *** openstackgerrit has quit IRC | 07:29 | |
| *** rcernin has quit IRC | 07:30 | |
| *** hjensas has joined #openstack-ironic | 07:47 | |
| *** jtomasek has joined #openstack-ironic | 08:06 | |
| *** rpittau has joined #openstack-ironic | 08:19 | |
| rpittau | good morning ironic! o/ | 08:22 |
| rpittau | happy hump day :) | 08:22 |
| dnuka | morning rpittau :) | 08:23 |
| rpittau | hi dnuka :) | 08:23 |
| *** e0ne has quit IRC | 08:31 | |
| iurygregory | morning rpittau o/ | 08:35 |
| * iurygregory has no idea what is hump day lol | 08:35 | |
| rpittau | hey iurygregory :) | 08:35 |
| rpittau | :D | 08:35 |
| rpittau | hump day is wednesday | 08:36 |
| rpittau | midpoint of typical working week | 08:36 |
| iurygregory | oh lol | 08:36 |
| rpittau | ^^ | 08:36 |
| iurygregory | so every wed is hump right? | 08:36 |
| rpittau | correct :) | 08:37 |
| iurygregory | nice =) | 08:37 |
| *** openstackgerrit has joined #openstack-ironic | 08:47 | |
| openstackgerrit | Kaifeng Wang proposed openstack/networking-baremetal master: Change networking-baremetal to zuulv3/python3 https://review.openstack.org/614026 | 08:47 |
| *** tssurya has joined #openstack-ironic | 09:00 | |
| *** dougsz has joined #openstack-ironic | 09:02 | |
| *** dtantsur|afk is now known as dtantsur | 09:19 | |
| dtantsur | jroll: I'm sceptical about all sorts of visual programming since I extensively used Delphi :) but I'd like a framework that simple | 09:20 |
| *** S4ren has joined #openstack-ironic | 09:21 | |
| dtantsur | morning ironic | 09:22 |
| iurygregory | morning dtantsur o/ | 09:22 |
| rpittau | good morning dtantsur :) | 09:22 |
| dtantsur | iurygregory, rpittau, I prefer "Odin's day" which is what Wednesday essentially means :) | 09:22 |
| iurygregory | Odin's day sounds awesome \o/ | 09:23 |
| dtantsur | did my last comment actually go through? My wifi misbehaves again.. | 09:26 |
| rpittau | dtantsur, for my people is Mercury Day :P | 09:27 |
| dtantsur | heh, cool :) | 09:28 |
| dtantsur | iurygregory https://bugzilla.redhat.com/show_bug.cgi?id=1658331 maybe another case of the problem you're debugging | 09:28 |
| openstack | bugzilla.redhat.com bug 1658331 in openstack-ironic "Overcloud deployment fails not changing provisioning state or powering off baremetal servers" [Unspecified,New] - Assigned to rhos-maint | 09:28 |
| iurygregory | dtantsur, yeah | 09:29 |
| iurygregory | will take a look now | 09:29 |
| dtantsur | iurygregory: you can ping bjacot later today for a reproducer environment | 09:29 |
| iurygregory | dtantsur, ack | 09:30 |
| dnuka | good morning dtantsur o/ | 09:35 |
| dtantsur | \o | 09:37 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Ignore newly introduced tables in pre-upgrade versions check https://review.openstack.org/624449 | 09:42 |
| *** moshele has quit IRC | 09:42 | |
| *** derekh has joined #openstack-ironic | 09:44 | |
| iurygregory | dtantsur, there is a way to get old logs from a specific patch on CI? like for https://review.openstack.org/#/c/613659/ | 09:47 |
| patchbot | patch 613659 - networking-baremetal - Testing CI - 1 patch set | 09:47 |
| dtantsur | iurygregory: I'm not aware of such | 09:48 |
| iurygregory | i will post a recheck just to see if i can compare the local.conf to try to debug https://review.openstack.org/#/c/614026 XD | 09:49 |
| patchbot | patch 614026 - networking-baremetal - Change networking-baremetal to zuulv3/python3 - 10 patch sets | 09:49 |
| openstackgerrit | paresh sao proposed openstack/ironic master: OOB RAID implementation for ilo5 based HPE Proliant servers. https://review.openstack.org/555664 | 09:51 |
| *** gkadam has joined #openstack-ironic | 10:05 | |
| *** gkadam has quit IRC | 10:05 | |
| *** S4ren has quit IRC | 10:06 | |
| *** e0ne has joined #openstack-ironic | 10:08 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allocation API: database and RPC https://review.openstack.org/624119 | 10:11 |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: [WIP] Replace use of Q_USE_PROVIDERNET_FOR_PUBLIC https://review.openstack.org/621146 | 10:19 |
| dtantsur | dnuka: re https://review.openstack.org/#/c/613828/15/sushy/resources/updateservice/mappings.py@32 you need "NSF" to be mapped to the same value as NFS | 10:31 |
| patchbot | patch 613828 - sushy - Add support for the `UpdateService` resource - 15 patch sets | 10:31 |
| dtantsur | so you need to update the mapping, not just add a constant (you actually do not need a new constant) | 10:32 |
| *** jaganathan has quit IRC | 10:38 | |
| dnuka | dtantsur: is this the way, https://review.openstack.org/#/c/613828/12/sushy/resources/updateservice/mappings.py@32 | 10:39 |
| patchbot | patch 613828 - sushy - Add support for the `UpdateService` resource - 15 patch sets | 10:39 |
| *** stendulker has quit IRC | 10:39 | |
| dtantsur | dnuka: it was close, you just needed TRANSFER_PROTOCOL_TYPE_VALUE_MAP['NSF'] = ups_cons.TRANSFER_PROTOCOL_TYPE_NFS | 10:41 |
| *** dsneddon has quit IRC | 10:41 | |
| dtantsur | i.e. use constant instead of literal string value | 10:41 |
| dnuka | thank you dtantsur :) | 10:41 |
| openstackgerrit | Hamdy Khader proposed openstack/ironic master: Add support for smartNICs https://review.openstack.org/583573 | 10:45 |
| openstackgerrit | Merged openstack/sushy master: Unify sushy models by Redfish schema bundle https://review.openstack.org/616534 | 10:54 |
| *** dsneddon has joined #openstack-ironic | 10:55 | |
| dnuka | \o/ | 10:55 |
| dtantsur | congrats :) | 11:00 |
| dnuka | thank you dtantsur :) | 11:00 |
| dnuka | dtantsur: I'll fix the things you've mentioned in a follow-up patch. | 11:02 |
| dtantsur | ack | 11:02 |
| *** moshele has joined #openstack-ironic | 11:13 | |
| *** v12aml has joined #openstack-ironic | 11:28 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/networking-baremetal master: [DNM] Tests https://review.openstack.org/624671 | 11:28 |
| *** e0ne has quit IRC | 11:30 | |
| *** dnuka has quit IRC | 11:37 | |
| dtantsur | TheJulia: I got a feeling that the master CI is broken.. | 11:52 |
| openstackgerrit | melissaml proposed openstack/bifrost master: Add Python 3.6 classifier to setup.cfg https://review.openstack.org/624679 | 11:55 |
| *** dtantsur is now known as dtantsur|brb | 11:56 | |
| *** rpittau is now known as rpittau|lunch | 11:59 | |
| *** moshele has quit IRC | 12:03 | |
| *** moshele has joined #openstack-ironic | 12:04 | |
| jroll | morning ironic | 12:13 |
| jroll | oof, CI | 12:13 |
| jroll | :( | 12:13 |
| iurygregory | morning jroll | 12:14 |
| iurygregory | jroll, if you have time can you check https://review.openstack.org/#/c/624397/ ? =) | 12:15 |
| patchbot | patch 624397 - ironic-inspector (stable/rocky) - Accept None value for set-attribute - 1 patch set | 12:15 |
| jroll | iurygregory: I just sat down, coffee and CI have priority right now :) | 12:15 |
| jroll | actually backports are easy, one sec | 12:16 |
| iurygregory | sure =) | 12:16 |
| jroll | +W :) | 12:16 |
| iurygregory | ty sir o/ | 12:16 |
| jroll | np! | 12:17 |
| jroll | dtantsur|brb: hrm, it looks like it's bionic, I wonder if my depends-on to try it didn't do what I thought? | 12:17 |
| jroll | console says no bootable device | 12:18 |
| jroll | blurgh | 12:18 |
| jroll | oh where oh where to begin | 12:19 |
| *** hjensas has quit IRC | 12:21 | |
| jroll | ugh, everything looks normal in logs, AFAICT. this is gonna be some digging :( | 12:25 |
| *** dsneddon has quit IRC | 12:30 | |
| *** dnuka has joined #openstack-ironic | 12:46 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:50 | |
| jroll | oh, I was depending on the devstack change as requested by QA folks, but this was what I needed: https://review.openstack.org/#/c/618169/11 | 12:52 |
| patchbot | patch 618169 - tempest - Migrate tempest provided base jobs to bionic (MERGED) - 11 patch sets | 12:52 |
| jroll | :/ | 12:52 |
| dnuka | morning jroll o/ | 12:52 |
| *** hjensas has joined #openstack-ironic | 12:56 | |
| *** dnuka has quit IRC | 12:56 | |
| *** moshele has quit IRC | 12:59 | |
| *** dnuka has joined #openstack-ironic | 12:59 | |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: Switch the default NIC driver to e1000 https://review.openstack.org/624690 | 13:01 |
| derekh | dtantsur|brb: jroll ^ I suspect we're getting hit by this | 13:01 |
| jroll | oof, yeah that could be | 13:02 |
| *** MattMan_ has quit IRC | 13:02 | |
| derekh | jroll: we got hit a while back but in the grenade job only https://review.openstack.org/#/c/604506/2 | 13:02 |
| patchbot | patch 604506 - ironic - Switch grenade nic driver to e1000 (MERGED) - 2 patch sets | 13:02 |
| *** MattMan_ has joined #openstack-ironic | 13:02 | |
| jroll | derekh: yeah, I recall that | 13:03 |
| jroll | here's where we switched away: https://github.com/openstack/ironic/commit/ee4f24bbb9760fb524fd5cbac763b8b84b122123 | 13:03 |
| jroll | hopefully going back doesn't hurt performance too bad | 13:03 |
| derekh | fingers crossed, if it does we can maybe look into blacklisting the versions of ipxe that doesn't work or something | 13:05 |
| jroll | ++ | 13:05 |
| * derekh lunches | 13:05 | |
| jroll | I'll keep an eye on that patch :) | 13:05 |
| jroll | thanks! | 13:05 |
| *** rpittau|lunch is now known as rpittau | 13:06 | |
| *** trown|outtypewww is now known as trown | 13:07 | |
| *** dnuka has quit IRC | 13:12 | |
| *** dnuka has joined #openstack-ironic | 13:14 | |
| *** dtantsur|brb is now known as dtantsur | 13:20 | |
| dtantsur | great, thansk derekh! | 13:20 |
| dnuka | dtantsur: can you please explain this to me, https://review.openstack.org/#/c/613828/15/sushy/resources/updateservice/updateservice.py@104 :) | 13:28 |
| patchbot | patch 613828 - sushy - Add support for the `UpdateService` resource - 15 patch sets | 13:28 |
| *** bfournie has joined #openstack-ironic | 13:30 | |
| dtantsur | dnuka: you can google for "set comprehension". What I mean is that you can do something like {TRANSFER_PROTOCOL_TYPE_VALUE_MAP[v] for v in simple_update_action.transfer_protocol if v in TRANSFER_PROTOCOL_TYPE_VALUE_MAP} | 13:31 |
| dtantsur | there is no need for set([]). you're creating a list just to create a set from it. you can create a set directly | 13:32 |
| dnuka | thank you very much dtantsur :) | 13:32 |
| *** dsneddon has joined #openstack-ironic | 13:32 | |
| *** dsneddon has quit IRC | 13:37 | |
| * TheJulia tries to wake up | 13:54 | |
| dnuka | good morning TheJulia :) | 13:54 |
| rpittau | hey TheJulia :) | 13:55 |
| dtantsur | morning TheJulia | 13:56 |
| *** dtantsur is now known as dtantsur|brb | 13:56 | |
| iurygregory | good morning TheJulia o/ | 13:59 |
| *** rloo has joined #openstack-ironic | 14:15 | |
| *** irclogbot_1 has quit IRC | 14:32 | |
| *** jrist has joined #openstack-ironic | 14:37 | |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: [WIP] Replace use of Q_USE_PROVIDERNET_FOR_PUBLIC https://review.openstack.org/621146 | 14:38 |
| *** e0ne has joined #openstack-ironic | 14:38 | |
| derekh | hmm, the NIC driver update seems to have fixed some jobs, others have failed | 14:39 |
| jroll | probably just timeouts then? | 14:40 |
| jroll | especially if e1000 performance isn't as good | 14:40 |
| *** irclogbot_1 has joined #openstack-ironic | 14:41 | |
| derekh | possibly, trying to see if I can find out the problem | 14:42 |
| *** moshele has joined #openstack-ironic | 14:42 | |
| *** mmethot has quit IRC | 14:43 | |
| openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Add support for the `UpdateService` resource https://review.openstack.org/613828 | 14:43 |
| jroll | cool, thanks! | 14:43 |
| jroll | might be easiest to just wait for the results to post and get the full log | 14:44 |
| derekh | jroll: the full logs are available on zuul for finished jobs, only jobs yet to finish are the grenade jobs | 14:45 |
| jroll | ah ok | 14:46 |
| *** mmethot has joined #openstack-ironic | 14:46 | |
| *** moshele has quit IRC | 14:47 | |
| *** rpioso|afk is now known as rpioso | 14:52 | |
| rpioso | Good morning, ironicers. | 14:52 |
| dnuka | good morning rpioso | 14:52 |
| rpioso | dnuka: :-) | 14:53 |
| rpittau | hi rpioso :) | 14:53 |
| rpioso | rpittau: o/ | 14:53 |
| iurygregory | good morning rpioso o/ | 14:53 |
| rpioso | iurygregory: Hey! | 14:54 |
| *** irclogbot_1 has quit IRC | 14:55 | |
| derekh | hmm, 1 X ssh timeout error, 1 X BFV volume creation error and 2 X Error Executing Command 'set -eu -o pipefail; PATH=$PATH:/sbin; mount | grep '/dev/vda1' | cut -d' ' -f3', exit status: 1, stderr: | 14:56 |
| TheJulia | :( | 14:57 |
| jroll | O_o | 14:57 |
| openstackgerrit | Merged openstack/ironic-inspector stable/rocky: Accept None value for set-attribute https://review.openstack.org/624397 | 14:57 |
| rpioso | I would appreciate a review of "[RFE] Define iDRAC support of Redfish power and management interfaces" (https://storyboard.openstack.org/#!/story/2004592) to determine if it needs a spec, and for feedback on the enhancement and approach. | 14:58 |
| TheJulia | rpioso: It is so detailed that I hit information overload just skimming it.. which is a good sign a spec is needed | 15:01 |
| TheJulia | I think your likely okay if you pan to alias the classes internally in case people have written code. That would also help map/migrate unit tests across so they could be done in two changes since you could then verify that your major changes are not breaking your tests without modifying your tests (which is an important thing at that scale) | 15:05 |
| TheJulia | s/pan/plan/ | 15:05 |
| TheJulia | otherwise, the basic concept, I think aligns with our intent of driver decomposition since, if I am understanding correctly your seeking to begin switching "native" interfaces to redfish based communication and deprecate out the old interfaces, but allow them in the same hardware type | 15:07 |
| *** dsneddon has joined #openstack-ironic | 15:15 | |
| *** mbeierl has quit IRC | 15:15 | |
| *** dsneddon has quit IRC | 15:19 | |
| rpioso | TheJulia: Does "I think your likely okay if you plan to alias the classes internally in case people have written code." refer to the first two subsections in the design description, "Rename idrac Interface Classes" and "Deprecate Existing idrac Interfaces"? | 15:20 |
| *** irclogbot_1 has joined #openstack-ironic | 15:20 | |
| *** jrist has quit IRC | 15:21 | |
| *** dtantsur|brb is now known as dtantsur | 15:25 | |
| dtantsur | iurygregory: I suggest backporting the set-attribute fix to queens as well, wdyt? | 15:27 |
| iurygregory | dtantsur, sure =) | 15:27 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector stable/queens: Accept None value for set-attribute https://review.openstack.org/624738 | 15:28 |
| rpioso | TheJulia: Or, I should ask, is it covered by those two subsections? | 15:29 |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: [WIP] Replace use of Q_USE_PROVIDERNET_FOR_PUBLIC https://review.openstack.org/621146 | 15:46 |
| *** coreywright has quit IRC | 15:46 | |
| TheJulia | rpioso: you have to understand, I've had maybe 5 minutes to look at it and think about it thus far | 15:48 |
| TheJulia | rpioso: so reading that specifically in detail, logging on __init__ is likely not going to work as desired because each driver load would log the message | 15:50 |
| rpioso | TheJulia: I understand completely :) | 15:51 |
| rpioso | TheJulia: Is there another point in time when it could notify the operator that it's being deprecated and only occurs once? | 15:52 |
| TheJulia | rpioso: we've tended as of recent to use direct mapping with a todo note to remove older interfaces. I guess an eventual question is a data migration upon new interfaces | 15:52 |
| TheJulia | rpioso: release notes are intended to convey that sort of information. It seems like your also hoping to just quietly transition idrac hardware type users over to redfish | 15:53 |
| TheJulia | I feel like you'd need to migrate them over and let them choose :\ | 15:54 |
| *** jrist has joined #openstack-ironic | 15:54 | |
| TheJulia | It would naturally be bad if someone upgraded ironic and suddenly someone was unable to talk to any of their dell servers that worked an hour earlier because we went to redfish defaults | 15:54 |
| rpioso | TheJulia: This story is about giving operators the option to use Redfish or WS-Man. The defaults will remain WS-Man (idrac). | 15:55 |
| rpioso | However, we do want to inform operators that will change. Release notes and driver doc will cover it. | 15:56 |
| TheJulia | that doesn't align with deprecate, and the choice already exists if the new interfaces are added to the hardware type interface lists | 15:56 |
| *** tssurya has quit IRC | 15:57 | |
| rpioso | By deprecate, I mean that in the future the WS-Man interfaces will no longer be first in the supported lists. | 15:58 |
| TheJulia | if the intent is to maintain ws-man and just offer idrac, then perhaps idrac-redfish as an interface becoming a superset of the existing redfish to support other additional logic that may be needed. If the intent is to deprecate and remove, then the plan needs to be the eventual removal of the interfaces and a path forward where they are no longer the explicit defaults after a specific point in time. In median we | 15:58 |
| TheJulia | would need to be careful with operators that create nodes from before that time | 15:58 |
| TheJulia | deprecate to us essentially means eventual removal | 15:59 |
| rpioso | How is changing supported list order dealt with? | 15:59 |
| *** coreywright has joined #openstack-ironic | 15:59 | |
| TheJulia | if we're just talking about list ordering changes, we still need nail the specific interfaces that are in use before the new interfaces are loaded | 15:59 |
| TheJulia | otherwise upon restart with new options, a new default would be leveraged if they are not explicitly set | 16:00 |
| TheJulia | since hardware types work by calculated defaults unless specifically overriden at an interface level | 16:00 |
| rpioso | Isn't that what the two releases deprecation period allows? | 16:01 |
| rpioso | The operator would be able to explicitly configure their nodes to use the desired interfaces whose priorities are being lowered in the supported lists. | 16:01 |
| TheJulia | in theory yes, but the specific interfaces would need to be explicitly set to handle the migration boundrary | 16:02 |
| rpioso | What does "migration boundary" mean? | 16:03 |
| TheJulia | think of it this way. node-zed is added with just idrac hardware type. | 16:03 |
| TheJulia | The interface fields are all calculated, it is using wsman and working | 16:03 |
| TheJulia | The interfaces are implied from the idrac hardware type | 16:03 |
| TheJulia | so operator upgrades conductor and gets new idrac hardware type | 16:04 |
| TheJulia | upon startup, if redfish is first in the list, the conductor will treat that as the calculated default and try to use redfish since it won't know otherwise | 16:04 |
| TheJulia | Does this make more sense? | 16:04 |
| rpioso | Yes, it does. Wouldn't the deprecation notice, release note, and doc inform the operator that's changing? | 16:06 |
| rpioso | TheJulia: Also, couldn't __init__ print the message only once, instead of for every node that uses a particular interface? | 16:10 |
| rpioso | TheJulia: The subject story 1) will not change the order of the legacy interfaces in the supported lists, 2) will not change the entrypoint names of the legacy interfaces, and 3) will not remove any legacy interfaces from the supported lists. | 16:13 |
| rpioso | A future story will do 1). | 16:14 |
| rpioso | For now, I believe we just need to settle on how to notify operators that will occur. A release note and doc is clear. Perhaps log messages could, too. | 16:15 |
| *** rpioso is now known as rpioso|afk | 16:17 | |
| * rpioso|afk will be back :) | 16:17 | |
| *** dnuka has quit IRC | 16:18 | |
| *** gyee has joined #openstack-ironic | 16:19 | |
| TheJulia | rpioso|afk: That is not the purpose of a deprecation, it would be a behavior change between change across releases... at least once item 1 is changed | 16:38 |
| *** moshele has joined #openstack-ironic | 16:40 | |
| *** tssurya has joined #openstack-ironic | 16:51 | |
| *** robbbe has joined #openstack-ironic | 16:52 | |
| derekh | So this fixed some of the jobs https://review.openstack.org/#/c/624690/ looks like we might have 3 other failuers in other jobs, | 16:58 |
| patchbot | patch 624690 - ironic - Switch the default NIC driver to e1000 - 1 patch set | 16:58 |
| derekh | I'm guessing they are all triggered because we switched from xenial to bionic this morning | 16:58 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: [wip] Single process ironic https://review.openstack.org/623569 | 17:01 |
| *** iurygregory has quit IRC | 17:01 | |
| *** robbbe has quit IRC | 17:03 | |
| TheJulia | derekh: yeah :( | 17:13 |
| TheJulia | derekh: thanks for keeping an eye on it! | 17:14 |
| NobodyCam | good morning ironic'ers | 17:14 |
| derekh | TheJulia: I'm trying to reproduce, but might take some time to get through it | 17:14 |
| *** dsneddon has joined #openstack-ironic | 17:16 | |
| TheJulia | derekh: completely understand :) | 17:18 |
| *** e0ne has quit IRC | 17:21 | |
| *** dtantsur is now known as dtantsur|afk | 17:26 | |
| dtantsur|afk | see you tomorrow! | 17:27 |
| TheJulia | goodnight dtantsur|afk | 17:27 |
| TheJulia | derekh: btw, looks like I have +a's on the neutron patches, to fix option tagging logic and give us an ipxe6 tag, just gate stability issues are the hurdle at the moment | 17:27 |
| derekh | TheJulia: ack, nice one | 17:28 |
| *** dsneddon has quit IRC | 17:29 | |
| *** rpioso|afk is now known as rpioso | 17:31 | |
| *** dsneddon has joined #openstack-ironic | 17:33 | |
| rpioso | TheJulia: Is the RFE good to go? | 17:37 |
| *** ianychoi has joined #openstack-ironic | 17:38 | |
| *** trown is now known as trown|lunch | 17:41 | |
| rpittau | bye all, good evening! o/ | 17:52 |
| rpioso | rpittau: Good nite :) | 17:52 |
| *** rpittau has quit IRC | 17:53 | |
| *** iurygregory has joined #openstack-ironic | 17:55 | |
| *** derekh has quit IRC | 18:04 | |
| *** jrist has quit IRC | 18:33 | |
| *** dougsz has quit IRC | 18:42 | |
| *** moshele has quit IRC | 18:44 | |
| *** trown|lunch is now known as trown | 18:50 | |
| *** tssurya has quit IRC | 18:51 | |
| *** dougsz has joined #openstack-ironic | 19:07 | |
| *** cdearborn has joined #openstack-ironic | 19:19 | |
| *** e0ne has joined #openstack-ironic | 19:20 | |
| *** e0ne_ has joined #openstack-ironic | 19:27 | |
| *** e0ne has quit IRC | 19:27 | |
| *** e0ne_ has quit IRC | 19:30 | |
| *** e0ne has joined #openstack-ironic | 19:34 | |
| *** dougsz has quit IRC | 19:36 | |
| *** e0ne_ has joined #openstack-ironic | 19:39 | |
| *** e0ne has quit IRC | 19:41 | |
| *** e0ne has joined #openstack-ironic | 19:44 | |
| *** e0ne_ has quit IRC | 19:45 | |
| *** e0ne has quit IRC | 19:45 | |
| *** e0ne has joined #openstack-ironic | 19:46 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: [wip] Single process ironic https://review.openstack.org/623569 | 19:52 |
| *** e0ne has quit IRC | 19:54 | |
| *** vkmc has quit IRC | 20:03 | |
| *** vkmc has joined #openstack-ironic | 20:07 | |
| *** e0ne has joined #openstack-ironic | 20:15 | |
| *** e0ne has quit IRC | 20:16 | |
| *** jcoufal has joined #openstack-ironic | 20:17 | |
| *** rloo has quit IRC | 20:19 | |
| *** rloo has joined #openstack-ironic | 20:19 | |
| *** jrist has joined #openstack-ironic | 20:30 | |
| *** iurygregory has quit IRC | 20:41 | |
| *** iurygregory has joined #openstack-ironic | 20:42 | |
| *** rcernin has joined #openstack-ironic | 21:06 | |
| TheJulia | rpioso: as an rfe and not deprecating, I guess. I'm not sure how other cores feel. I'll ask dtantsur to look tomorrow | 21:22 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/networking-generic-switch master: Migrate n-g-s jobs to Zuulv3 https://review.openstack.org/613655 | 21:33 |
| *** jcoufal has quit IRC | 21:34 | |
| TheJulia | have I ever mentioned that I really don't like how we log 2 lines for every task invocation.... :\ | 21:36 |
| * TheJulia feels like she had a good locking change in the task execution mode poc patch | 21:38 | |
| TheJulia | s/locking/periodic/ | 21:38 |
| TheJulia | rpioso: I did think about the upgrade status check work, it could be possible to evaluate nodes looking for the older driver, and provide non-fatal warnings as time goes on so operators know once deprecated that the drivers will need to be explicitely set before the next upgrade... and in that next version we actually fail the upgrade check if the data does not validate out | 21:46 |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: [WIP] CI Tests https://review.openstack.org/624690 | 21:53 |
| *** trown is now known as trown|outtypewww | 22:04 | |
| *** mnasiadka_ has joined #openstack-ironic | 22:23 | |
| *** mnasiadka has quit IRC | 22:29 | |
| *** rh-jelabarre has quit IRC | 22:29 | |
| *** logan- has quit IRC | 22:29 | |
| *** mnasiadka_ is now known as mnasiadka | 22:29 | |
| *** rh-jelabarre has joined #openstack-ironic | 22:29 | |
| *** logan- has joined #openstack-ironic | 22:31 | |
| *** bfournie has quit IRC | 22:35 | |
| *** cdearborn has quit IRC | 22:41 | |
| iurygregory | yay CI is happy https://review.openstack.org/613655 | 22:58 |
| patchbot | patch 613655 - networking-generic-switch - Migrate n-g-s jobs to Zuulv3 - 19 patch sets | 22:58 |
| *** dtrainor has quit IRC | 23:09 | |
| *** bfournie has joined #openstack-ironic | 23:18 | |
| TheJulia | \o/ | 23:28 |
| *** dtrainor has joined #openstack-ironic | 23:31 | |
| *** iurygregory has quit IRC | 23:38 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Single process ironic https://review.openstack.org/623569 | 23:53 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!