| *** sdake has joined #openstack-ironic | 00:47 | |
| *** _fragatina_ has quit IRC | 01:02 | |
| *** _fragatina has joined #openstack-ironic | 01:02 | |
| *** sdake has quit IRC | 01:03 | |
| *** sdake has joined #openstack-ironic | 01:06 | |
| *** hwoarang has quit IRC | 01:14 | |
| *** hwoarang has joined #openstack-ironic | 01:16 | |
| *** mcarden has joined #openstack-ironic | 01:16 | |
| *** sdake has quit IRC | 01:19 | |
| *** tiendc has joined #openstack-ironic | 01:38 | |
| *** yaawang has joined #openstack-ironic | 02:41 | |
| *** sdake has joined #openstack-ironic | 03:03 | |
| *** whoami-rajat has joined #openstack-ironic | 03:12 | |
| *** sdake has quit IRC | 03:36 | |
| *** sdake has joined #openstack-ironic | 03:41 | |
| *** sdake has quit IRC | 03:56 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: [docs] IPv6 support for iLO https://review.openstack.org/638590 | 03:56 |
|---|---|---|
| *** sdake has joined #openstack-ironic | 04:22 | |
| *** sdake has quit IRC | 04:25 | |
| *** sdake has joined #openstack-ironic | 04:26 | |
| *** sdake has quit IRC | 04:30 | |
| *** sdake has joined #openstack-ironic | 04:31 | |
| openstackgerrit | Merged openstack/python-ironicclient master: [Trivial] Allocation API: fix incorrect parameter description https://review.openstack.org/638702 | 04:32 |
| *** sdake has quit IRC | 04:35 | |
| *** sdake has joined #openstack-ironic | 04:38 | |
| *** sdake has quit IRC | 04:40 | |
| *** sdake has joined #openstack-ironic | 04:43 | |
| *** sdake has quit IRC | 04:45 | |
| *** sdake has joined #openstack-ironic | 04:46 | |
| *** sdake has quit IRC | 04:48 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Update the proliantutils version in documentation https://review.openstack.org/638754 | 04:52 |
| *** sdake has joined #openstack-ironic | 04:52 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: [docs] IPv6 support for iLO https://review.openstack.org/638590 | 04:53 |
| *** sdake has quit IRC | 04:55 | |
| *** sdake has joined #openstack-ironic | 04:56 | |
| *** sdake has quit IRC | 05:00 | |
| *** sdake has joined #openstack-ironic | 05:02 | |
| *** sdake has quit IRC | 05:10 | |
| *** sdake_ has joined #openstack-ironic | 05:12 | |
| *** sdake_ has quit IRC | 05:15 | |
| *** sdake has joined #openstack-ironic | 05:17 | |
| *** sdake has quit IRC | 05:21 | |
| *** sdake has joined #openstack-ironic | 05:22 | |
| *** sdake has quit IRC | 05:26 | |
| *** sdake has joined #openstack-ironic | 05:28 | |
| *** sdake has quit IRC | 05:30 | |
| *** sdake_ has joined #openstack-ironic | 05:33 | |
| *** sdake_ has quit IRC | 05:35 | |
| *** sdake has joined #openstack-ironic | 05:37 | |
| *** yaawang has quit IRC | 05:45 | |
| *** sdake has quit IRC | 05:45 | |
| *** yaawang has joined #openstack-ironic | 05:45 | |
| *** sdake has joined #openstack-ironic | 05:49 | |
| *** sdake has quit IRC | 05:51 | |
| *** sdake has joined #openstack-ironic | 05:53 | |
| *** sdake has quit IRC | 05:55 | |
| *** sdake has joined #openstack-ironic | 05:56 | |
| *** sdake has quit IRC | 06:00 | |
| *** sdake has joined #openstack-ironic | 06:02 | |
| openstackgerrit | Merged openstack/ironic master: Follow up to node description https://review.openstack.org/637483 | 06:03 |
| *** sdake has quit IRC | 06:06 | |
| *** sdake_ has joined #openstack-ironic | 06:08 | |
| *** yaawang has quit IRC | 06:09 | |
| *** sdake_ has quit IRC | 06:10 | |
| *** jtomasek has joined #openstack-ironic | 06:16 | |
| *** trungnv has quit IRC | 06:20 | |
| *** tiendc has quit IRC | 06:36 | |
| *** dnuka has joined #openstack-ironic | 06:39 | |
| dnuka | good morning ironic | 06:39 |
| *** toabctl has joined #openstack-ironic | 06:46 | |
| *** ltomasbo has joined #openstack-ironic | 07:04 | |
| *** andrein has joined #openstack-ironic | 07:23 | |
| *** andrein has quit IRC | 07:23 | |
| *** hamdyk has joined #openstack-ironic | 07:31 | |
| *** arne_wiebalck_ has joined #openstack-ironic | 07:37 | |
| *** arne_wiebalck_ has left #openstack-ironic | 07:37 | |
| *** arne_wiebalck has joined #openstack-ironic | 07:40 | |
| arne_wiebalck | Good morning, ironic! | 07:40 |
| dnuka | good morning arne_wiebalck o/ | 07:41 |
| *** arne_wiebalck_ has joined #openstack-ironic | 07:41 | |
| arne_wiebalck | hey dnuka o/ | 07:41 |
| *** arne_wiebalck_ has quit IRC | 07:45 | |
| *** arne_wiebalck_ has joined #openstack-ironic | 08:03 | |
| *** dtantsur|afk is now known as dtantsur | 08:21 | |
| dtantsur | morning ironic | 08:21 |
| *** amoralej|off is now known as amoralej | 08:26 | |
| dnuka | good morning dtantsur o/ | 08:28 |
| openstackgerrit | John Haan proposed openstack/bifrost master: Add dnsmasq_ntp_servers variable in the main https://review.openstack.org/639029 | 08:30 |
| openstackgerrit | Dmitry Tantsur proposed openstack/metalsmith master: [DNM] Test omitting configdrive fields https://review.openstack.org/639030 | 08:32 |
| *** sburette has joined #openstack-ironic | 08:33 | |
| rpittau|sardegna | good morning ironic! o/ | 08:35 |
| dnuka | good morning rpittau|sardegna :) | 08:36 |
| rpittau|sardegna | hey dnuka :) | 08:36 |
| *** yaawang has joined #openstack-ironic | 08:37 | |
| asettle | Goooooood morning ironic | 08:39 |
| dnuka | good morning asettle | 08:39 |
| rpittau|sardegna | hi asettle :) | 08:40 |
| dtantsur | \o | 08:43 |
| asettle | rpittau|sardegna, saw your note. You need me to manually remove the source/contributor/api files and then rebuild? | 08:43 |
| rpittau|sardegna | asettle, the code that I submitted does that | 08:44 |
| rpittau|sardegna | it will need to be backported to rocky and eventually to queens | 08:44 |
| asettle | Ah right, sorry. Yes. Well, we tested that one (manually and otherwise) on Friday and I don't remeber it working | 08:44 |
| asettle | Running it again now | 08:44 |
| rpittau|sardegna | there's one little issue with that, actually I think there will be an issue anyway | 08:45 |
| rpittau|sardegna | it will not work with python 3.7 | 08:45 |
| asettle | That's.. okay sure | 08:45 |
| *** iurygregory has joined #openstack-ironic | 08:46 | |
| iurygregory | good morning o/ | 08:46 |
| rpittau|sardegna | yeah it's a lot of fun :) | 08:46 |
| rpittau|sardegna | hi iurygregory :) | 08:46 |
| asettle | rpittau|sardegna, manual deletion still runs a failure :( | 08:47 |
| asettle | (not a fresh clone - Friday's clone) | 08:47 |
| *** tssurya has joined #openstack-ironic | 08:48 | |
| rpittau|sardegna | asettle, weird :/ still the same duplicate error ? | 08:49 |
| asettle | Sure do. | 08:49 |
| *** e0ne has joined #openstack-ironic | 08:51 | |
| *** rcernin has quit IRC | 09:00 | |
| *** stendulker has joined #openstack-ironic | 09:07 | |
| *** dougsz has joined #openstack-ironic | 09:09 | |
| *** derekh has joined #openstack-ironic | 09:30 | |
| openstackgerrit | John Haan proposed openstack/bifrost master: Add dnsmasq_ntp_servers variable in the main https://review.openstack.org/639029 | 09:31 |
| dtantsur | TheJulia, rpittau|sardegna, iurygregory, hi! so, we talked about configdrive in gophercloud. I've prepared an RFE to do it in ironic, I can prototype it today if you like it. https://storyboard.openstack.org/#!/story/2005083 | 09:37 |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: WIP: Support reapply with supplied introspection data https://review.openstack.org/639039 | 09:40 |
| dnuka | good morning iurygregory o/ | 09:42 |
| iurygregory | dtantsur, cool! | 09:44 |
| rpittau|sardegna | dtantsur, looks very interesting :) | 09:47 |
| dtantsur | rpittau|sardegna, iurygregory, the question is whether it will actually help our case with gophercloud | 09:47 |
| iurygregory | dtantsur, lets see what people think in the downstream first maybe? | 09:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/metalsmith master: [DNM] Test omitting configdrive fields https://review.openstack.org/639030 | 09:48 |
| *** yaawang has quit IRC | 09:53 | |
| dtantsur | iurygregory, rpittau|sardegna, an ignition file, is it a text file? | 09:55 |
| dtantsur | seems like JSON, right? | 09:55 |
| rpittau|sardegna | dtantsur, my understanding is that you're suggesting a standardization of the configdrive creation, directly handling it on ironic side | 09:56 |
| rpittau|sardegna | dtantsur, yes it's json basically | 09:56 |
| iurygregory | dtantsur, it looks like json for me | 09:56 |
| dtantsur | okie, then what I suggest should work | 09:56 |
| dtantsur | rpittau|sardegna: well, yeah. at least for the case of metadata+user_data, which seems to match what is happening at https://github.com/metalkube/dev-scripts/blob/6613c13e70f301c4f11d8d18d8ee8940610ff446/07_deploy_masters.sh#L28 | 09:57 |
| dtantsur | derekh: do you have a real example of such master.ign ^^^? | 09:57 |
| rpittau|sardegna | dtantsur, yes, I'm all for it, I think it will make life easier on client side, and help with the gophercloud case | 10:03 |
| *** stendulker has quit IRC | 10:12 | |
| derekh | dtantsur: https://goodsquishy.com/upload/424e84caabac886570c3 | 10:26 |
| dtantsur | nice, thanks derekh | 10:28 |
| derekh | np | 10:29 |
| *** MattMan has quit IRC | 10:31 | |
| *** MattMan has joined #openstack-ironic | 10:31 | |
| openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Add unit test case to `get_allowed_transfer_protocol_values()` https://review.openstack.org/637799 | 10:37 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Allow building configdrive from JSON in the API https://review.openstack.org/639050 | 10:42 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Allow building configdrive from JSON in the API https://review.openstack.org/639050 | 10:44 |
| dtantsur | prototype ^^ | 10:45 |
| *** trident has quit IRC | 10:47 | |
| *** trident has joined #openstack-ironic | 10:50 | |
| openstackgerrit | Dhanuka Warusadura proposed openstack/sushy master: Introduce default value for `transfer_protocol` parameter https://review.openstack.org/639051 | 10:51 |
| asettle | rpittau|sardegna, I'm going to get my colleague to test your fix and see if we can get that working. | 10:53 |
| *** andrein has joined #openstack-ironic | 10:54 | |
| rpittau|sardegna | asettle, ok, let me know how it goes, I'm seeing | 10:56 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add read-only Virtual Media support https://review.openstack.org/638787 | 10:58 |
| asettle | rpittau|sardegna, you got it | 10:58 |
| rpittau|sardegna | some erratic behavior, I wonder if it's a bug in sphinx ? | 10:58 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Allow building configdrive from JSON in the API https://review.openstack.org/639050 | 11:04 |
| openstackgerrit | Dmitry Tantsur proposed openstack/metalsmith master: [DNM] Test using ironic for generating configdrive https://review.openstack.org/639030 | 11:05 |
| asettle | rpittau|sardegna, I dunno. I haven't had these kinds of issues with any other project. Just here. | 11:24 |
| openstackgerrit | caoyuan proposed openstack/python-ironicclient master: Update json module to jsonutils https://review.openstack.org/638980 | 11:41 |
| asettle | rpittau|sardegna, haven't finished yet. But it's the same error so far. | 11:44 |
| asettle | And it works on Fedora you say? | 11:45 |
| *** sdake has joined #openstack-ironic | 11:46 | |
| openstackgerrit | Mark Goddard proposed openstack/python-ironicclient master: Deploy templates: client support https://review.openstack.org/636931 | 11:46 |
| *** sdake has quit IRC | 11:50 | |
| *** sdake has joined #openstack-ironic | 11:51 | |
| *** yaawang has joined #openstack-ironic | 11:53 | |
| *** cmurphy has joined #openstack-ironic | 11:53 | |
| *** sdake has quit IRC | 11:55 | |
| *** sdake has joined #openstack-ironic | 11:57 | |
| *** sdake has quit IRC | 12:00 | |
| *** dnuka is now known as dnuka|brb | 12:14 | |
| openstackgerrit | Mark Goddard proposed openstack/python-ironicclient master: Deploy templates: client support https://review.openstack.org/636931 | 12:25 |
| *** bfournie has quit IRC | 12:33 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Allow image checksum to be a URL https://review.openstack.org/639079 | 12:34 |
| *** rh-jelabarre has joined #openstack-ironic | 12:46 | |
| *** e0ne has quit IRC | 12:47 | |
| *** dnuka|brb is now known as dnuka | 13:06 | |
| *** bfournie has joined #openstack-ironic | 13:07 | |
| *** jistr is now known as jistr|mtg | 13:07 | |
| *** jistr|mtg is now known as jistr | 13:08 | |
| *** mjturek has joined #openstack-ironic | 13:13 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Allow image checksum to be a URL https://review.openstack.org/639079 | 13:28 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Adding bios_interface reference to api docs https://review.openstack.org/639088 | 13:32 |
| *** e0ne has joined #openstack-ironic | 13:35 | |
| *** amoralej is now known as amoralej|lunch | 13:36 | |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Removing python3.5 tests in favor of 3.6 https://review.openstack.org/638638 | 13:43 |
| mjturek | morning ironic! | 13:45 |
| iurygregory | morning mjturek | 13:46 |
| dtantsur | morning mjturek | 13:49 |
| dnuka | morning mjturek :) | 13:52 |
| *** rloo has joined #openstack-ironic | 13:53 | |
| *** w14161_1 has joined #openstack-ironic | 13:57 | |
| *** andrein has quit IRC | 14:01 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allow building configdrive from JSON in the API https://review.openstack.org/639050 | 14:01 |
| *** cmurphy has left #openstack-ironic | 14:01 | |
| *** jcoufal has joined #openstack-ironic | 14:01 | |
| *** mmethot has joined #openstack-ironic | 14:03 | |
| w14161_1 | HI, All, did anyone know whether devstack could have automatic script to provision every nodes at network with ironic? I used devstack to deploy ironic at one node of physical machine, but to provision other physical nodes at the network, I need to mannually use "openstack baremetal xxx" commands to do this one by one, whether devstack could do all these things like MAAS did, MAAS could | 14:04 |
| w14161_1 | provision all the physical nodes at the network automatically even include IPMI user name and password of every machines. | 14:04 |
| openstackgerrit | Mark Goddard proposed openstack/ironic-tempest-plugin master: WIP: Deploy templates: add API tests https://review.openstack.org/637187 | 14:05 |
| *** amoralej|lunch is now known as amoralej | 14:05 | |
| jroll | morning everyone | 14:06 |
| openstackgerrit | Jakub Libosvar proposed openstack/ironic-tempest-plugin master: Fix multitenancy skip https://review.openstack.org/639098 | 14:06 |
| jroll | w14161_1: you can set IRONIC_HWINFO_FILE and make a file like this: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L97-L122 | 14:07 |
| jroll | w14161_1: that will automatically enroll the machines, but not deploy them | 14:07 |
| dtantsur | morning jroll | 14:08 |
| jroll | hey dtantsur | 14:08 |
| w14161_1 | jroll, thanks for information. | 14:08 |
| rpittau|sardegna | hi jroll :0 | 14:08 |
| dnuka | morning jroll o/ | 14:10 |
| jroll | morning rpittau|sardegna dnuka :) | 14:10 |
| openstackgerrit | Jakub Libosvar proposed openstack/ironic master: Set available_nodes in tempest conf https://review.openstack.org/639101 | 14:10 |
| openstackgerrit | Jakub Libosvar proposed openstack/ironic-tempest-plugin master: Fix multitenancy skip https://review.openstack.org/639098 | 14:10 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allow building configdrive from JSON in the API https://review.openstack.org/639050 | 14:11 |
| openstackgerrit | Mark Goddard proposed openstack/ironic-tempest-plugin master: Deploy templates: add API tests https://review.openstack.org/637187 | 14:17 |
| *** andrein has joined #openstack-ironic | 14:18 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic-tempest-plugin master: Deploy templates: add API tests https://review.openstack.org/637187 | 14:19 |
| TheJulia | Good morning everyone | 14:28 |
| rpittau|sardegna | hi TheJulia :) | 14:28 |
| TheJulia | dtantsur: will you be able to run the library releases for me this week? | 14:28 |
| * TheJulia gets on a plane early Wednesday morning, and doesn't get home until Saturday :( | 14:28 | |
| *** arne_wiebalck_ has quit IRC | 14:29 | |
| dtantsur | TheJulia: on Thursday, yes. Will be enjoying the USA embassy in Frankfurt on Wednesday :) | 14:29 |
| *** arne_wiebalck_ has joined #openstack-ironic | 14:31 | |
| dnuka | morning TheJulia | 14:33 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector master: Add ironic API url to inspector IPA config https://review.openstack.org/636778 | 14:35 |
| w14161_1 | jroll, I checked the file, but the usage was not clear, did we have user guide for this? | 14:37 |
| openstackgerrit | Merged openstack/ironic master: Update the proliantutils version in documentation https://review.openstack.org/638754 | 14:41 |
| TheJulia | dtantsur: joy! | 14:46 |
| dtantsur | oh yeah | 14:46 |
| TheJulia | do you want me to mention that your out on wednesday as well in the meeting? | 14:47 |
| dtantsur | TheJulia: yes please | 14:47 |
| * TheJulia updates notes | 14:47 | |
| dtantsur | These awesome people don't even allow a mobile phone inside the embassy. so I'll be pretty much offline. | 14:47 |
| dnuka | :) | 14:47 |
| TheJulia | funf un | 14:48 |
| openstackgerrit | Merged openstack/ironic-ui master: add python 3.7 unit test job https://review.openstack.org/637260 | 14:48 |
| TheJulia | I updated and clarified my own note because it is likely possible that I can be reached, the question will be if I have spoons after deplaning and getting to my hotel. | 14:49 |
| TheJulia | rpittau|sardegna: you have a note on the whiteboard around graphical console, could you add a date when you get a chance? | 14:51 |
| rpittau|sardegna | TheJulia: I will as soon as the electricity comes back :/ not a fun day today..... | 14:53 |
| TheJulia | :( | 14:53 |
| TheJulia | rpittau|sardegna: thanks | 14:53 |
| *** kaifeng has joined #openstack-ironic | 14:54 | |
| jroll | w14161_1: that's the best guide I could find, sorry | 14:55 |
| w14161_1 | NP, thanks anyway | 14:56 |
| *** stendulker has joined #openstack-ironic | 14:57 | |
| etingof | dtantsur should get a Frankfurter Würstchen | 14:59 |
| dtantsur | totally, if I have a minute of time there.. | 14:59 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Add status field to port and portgroup data model https://review.openstack.org/638579 | 14:59 |
| TheJulia | #startmeeting ironic | 15:00 |
| TheJulia | o/ | 15:00 |
| openstack | Meeting started Mon Feb 25 15:00:29 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 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| iurygregory | o/ | 15:00 |
| TheJulia | Good morning everyone! | 15:00 |
| iurygregory | morning | 15:00 |
| kaifeng | o/ | 15:00 |
| bdodd | o/ | 15:00 |
| dnuka | o/ | 15:00 |
| arne_wiebalck_ | o/ | 15:00 |
| jroll | \o | 15:00 |
| rpittau|sardegna | o/ | 15:01 |
| rloo | o/ | 15:01 |
| etingof | o/ | 15:01 |
| w14161_1 | o/ | 15:01 |
| mgoddard | \o | 15:01 |
| *** baha has joined #openstack-ironic | 15:01 | |
| dtantsur | o/ | 15:02 |
| TheJulia | Our agenda this morning is fairly run of the mill with some announcements, so hopefully we'll get through things quickly. | 15:02 |
| TheJulia | Of course this is always my hope. | 15:02 |
| baha | o/ | 15:02 |
| TheJulia | Our agenda can be found on the wiki, as always. | 15:02 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:02 |
| TheJulia | #topic Announcements/Reminders | 15:02 |
| *** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)" | 15:02 | |
| mjturek | o/ | 15:02 |
| stendulker | o/ | 15:02 |
| hjensas | o/ | 15:03 |
| TheJulia | #info This week is the Stein release deadline for non-client libraries. (R-6) | 15:03 |
| TheJulia | #link https://releases.openstack.org/stein/schedule.html | 15:03 |
| *** e0ne has quit IRC | 15:03 | |
| TheJulia | #info Forum/Topic discussions are underway at our forum/ptg planning etherpad. Please add ideas. We should co-ordinate submitting topics this week. | 15:04 |
| TheJulia | #link https://etherpad.openstack.org/p/DEN-train-ironic-brainstorming | 15:04 |
| TheJulia | #info TheJulia will be traveling Wednesday and at an on-site meeting Thursday/Friday so will likely be moderately unavailable. | 15:04 |
| *** e0ne has joined #openstack-ironic | 15:05 | |
| TheJulia | #info dtantsur will be handling non-client library releases on Thursday | 15:05 |
| rloo | TheJulia: how do we decide what is in the Forum and what is in PTG? | 15:05 |
| TheJulia | #info dtantsur will also be out on Wednesday. | 15:05 |
| TheJulia | rloo: Forum is what runs during the summit in order to serve as requirements collection/larger discussions with operators/users to identify needs/wants/gaps | 15:05 |
| TheJulia | PTG is our time as a team to have focused discussions on how to solve those problems. | 15:06 |
| *** cdearborn has joined #openstack-ironic | 15:06 | |
| TheJulia | So there is some things that will naturally blend across both, and may need time in both. Those are likely the best topics! | 15:06 |
| TheJulia | Does anyone else have anything to announce or remind us of this week? | 15:06 |
| *** hamdyk has quit IRC | 15:06 | |
| rloo | TheJulia: thx, so people don't need to specify forum or ptg when they put things down on that etherpad | 15:07 |
| TheJulia | ++ | 15:07 |
| rloo | TheJulia: might be useful to mention, client packages are next week? | 15:07 |
| rloo | TheJulia: stein release deadline for client packages | 15:07 |
| TheJulia | #info Client library release deadline is next week. | 15:08 |
| TheJulia | Anything else? | 15:08 |
| dtantsur | and FF is also next week | 15:09 |
| TheJulia | #info Feature freeze is next week as well. Direct user facing features will likely be pushed to Stein. Non-using facing features "MAY" be considered before we need to branch for Stein. As we have learned int he past, ironic has to be ready to branch earlier than other projects, so that will also reduce chances of new features merging. | 15:11 |
| TheJulia | Are we good to proceed? | 15:12 |
| * dtantsur is good | 15:12 | |
| TheJulia | Since we have no action items from last week, we will go directly to subteam status | 15:12 |
| TheJulia | #topic Review subteam status reports | 15:12 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:12 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:13 |
| TheJulia | Starting at line 292. | 15:13 |
| TheJulia | I looked through things earlier and, and everything looked good to me | 15:13 |
| dtantsur | I think we should clear it from items that are clearly not making stein | 15:13 |
| TheJulia | Yeah | 15:13 |
| TheJulia | I can do that after the meeting | 15:14 |
| dtantsur | thnx | 15:14 |
| rloo | for py3, if we want to update python-ironicclient to py3, do we need to do it before end of next week cuz of deadline? it doesn't affect the code (i hope not) | 15:14 |
| jroll | rloo: as in, update the CI jobs? | 15:15 |
| rloo | jroll: yup. L306 | 15:15 |
| jroll | that seems fine to me to do any time | 15:16 |
| jroll | if it affects the code, and we want that code released for stein, it needs to be before the deadline. otherwise press on | 15:16 |
| TheJulia | It seem slike we really should just launch functional or tempest with py3 | 15:17 |
| TheJulia | Does anyone have a few minutes to submit that as a change to python-ironicclient this week? | 15:20 |
| rloo | wrt smartnic support, is all the stuff on ironic side done? I only see one PR for neutron (L417ish) | 15:20 |
| TheJulia | well, in the next couple days | 15:20 |
| asettle | Question to all: We've been looking at this Ironic stable/rocky doc build error for 2 days now and it's not solving itself. For context's sake: Why is Ironic publishing the API docs? | 15:20 |
| rpittau|sardegna | TheJulia, I can check that as I'm already on the py3 wave | 15:20 |
| dtantsur | asettle: we're in a meeting now, maybe ask in the Open Discussion section? | 15:21 |
| TheJulia | rloo: Seems that way minus some docs I believe | 15:21 |
| asettle | dtantsur, in a chan? | 15:21 |
| TheJulia | but additional docs are likely useless until neutron also merges the change | 15:21 |
| dtantsur | asettle: yeah, we moved the meeting here some time ago | 15:21 |
| *** beekneemech is now known as bnemec | 15:21 | |
| asettle | Sorry dtantsur | 15:21 |
| * asettle backs out | 15:21 | |
| dtantsur | no worries, it's not obviously different from our usual chatter :) | 15:22 |
| rpittau|sardegna | so many strikes in the whiteboard | 15:22 |
| rpittau|sardegna | :/ | 15:23 |
| dtantsur | yeah, this also needs periodic cleanup | 15:23 |
| TheJulia | Yeah, struck out things that I'll remove after the meeting | 15:24 |
| TheJulia | Are we good to proceed? | 15:25 |
| dtantsur | ++ | 15:26 |
| rpittau|sardegna | ++ | 15:26 |
| iurygregory | ++ | 15:27 |
| TheJulia | #topic Priorities for the coming week | 15:27 |
| *** openstack changes topic to "Priorities for the coming week (Meeting topic: ironic)" | 15:27 | |
| * TheJulia removes the merged items from the list for this next week :) | 15:27 | |
| TheJulia | (of which, there are a decent number \o/) | 15:27 |
| TheJulia | I think that looks fairly good for the next week | 15:31 |
| mgoddard | shall I add deploy templates patches? | 15:32 |
| TheJulia | mgoddard: please | 15:32 |
| dtantsur | mgoddard++ | 15:32 |
| dtantsur | otherwise looking good | 15:32 |
| TheJulia | mgoddard: do you feel the conductor patch is going to be ready in the next week or two? | 15:32 |
| mgoddard | TheJulia: yeah, I have a fairly free week this week | 15:33 |
| mgoddard | API is ready for review, once I push (today) | 15:33 |
| TheJulia | Awesome | 15:33 |
| mgoddard | assume we can accept API before conductor? | 15:33 |
| mgoddard | would take the pressure off, and allow client and tempest to go in | 15:34 |
| TheJulia | I'd prefer not to, but if it is an easy wire-together and would land in short order, I guess it would be okay | 15:34 |
| TheJulia | Is the conductor patch actually really minor? | 15:34 |
| dtantsur | ideally API goes last to avoid a period of non-working API. but it's not hard blocking. | 15:34 |
| TheJulia | i.e. not api invoked? | 15:34 |
| mgoddard | the conductor side isn't minor, but it's not huge | 15:35 |
| TheJulia | Yeah, broken api stuffs is ultimately what we want to avoid | 15:35 |
| mgoddard | it's not directly triggered by the new API, it happens during validate/deploy | 15:35 |
| TheJulia | I'm okay with it on a case by case basis | 15:35 |
| TheJulia | Okay, that sounds promising | 15:36 |
| mgoddard | I'll reorder the patches to put conductor last, we don't have to +A the API but that frees me up a bit to iterate and makes it easier to review the API | 15:36 |
| TheJulia | okay, thanks | 15:37 |
| TheJulia | I've updated the list to be a little cleaner, and moved sushy/sushy tools to the top since those are the main things with outstanding patches | 15:38 |
| TheJulia | Getting those merged will help us in the long run | 15:38 |
| TheJulia | Anyway, are we good to proceed | 15:38 |
| TheJulia | ? | 15:38 |
| * TheJulia hears crickets | 15:39 | |
| rpittau|sardegna | let's move on :) | 15:39 |
| dtantsur | yep | 15:39 |
| TheJulia | #topic RFE Review | 15:39 |
| *** openstack changes topic to "RFE Review (Meeting topic: ironic)" | 15:39 | |
| * TheJulia hands dtantsur the microphone | 15:39 | |
| dtantsur | #link https://storyboard.openstack.org/#!/story/2005083 Allow building simple configdrives inside Ironic | 15:39 |
| dtantsur | what I suggest is building a simple configdrive building facility in ironic | 15:40 |
| dtantsur | to stop people from cargo culting out genisoimage code from ironicclient ~ everywhere | 15:40 |
| dtantsur | #link https://review.openstack.org/#/c/639050/ ready to review patch | 15:40 |
| patchbot | patch 639050 - ironic - Allow building configdrive from JSON in the API - 5 patch sets | 15:40 |
| TheJulia | Seems simple to me | 15:40 |
| dtantsur | (note that it's less than 200 LoC) | 15:40 |
| hjensas | I like it, maby in the future also add support for vendor_data and network_data? (Looks like a trivial change in the sdk and then ironic). | 15:40 |
| jroll | dtantsur: whether we do this RFE or not, we're going to pull that (and nova's) code out into a library, right? | 15:41 |
| dtantsur | jroll: I'm taking it from openstacksdk where I put it for metalsmith (and which is already our indirect dependency) | 15:41 |
| dtantsur | * taking = importing, not copying | 15:41 |
| dtantsur | hjensas: yeah, I did not want to extend openstacksdk so close to various deadlines. but yes, we can. | 15:41 |
| TheJulia | those seem like reasonable iterations to me | 15:42 |
| dtantsur | with cloud-init you can do anything via user_data though. ditto for ignition. | 15:42 |
| jroll | cloud-init's networking doesn't come from user_data | 15:42 |
| jroll | and for some clouds, that's the most important bit | 15:43 |
| kaifeng | only nova has the network info i think | 15:43 |
| TheJulia | But ignition, it does come purely from user data. | 15:43 |
| jroll | (once it's full featured) should we use this thing instead of building the config drive in nova? | 15:43 |
| TheJulia | I believe kaifeng is correct | 15:43 |
| dtantsur | jroll: if nova can provide it to us in a sane format - sure | 15:43 |
| TheJulia | jroll: possibly, worth a discussion with the nova folks I think | 15:44 |
| jroll | ok | 15:44 |
| jroll | this feels like a bigger conversation IMO, figure out where we want to be instead of slapping a second format on the configdrive parameter | 15:44 |
| dtantsur | okay, I can take a follow-up to extend sdk+ironic to support network stuff. but it will slip into train likely, so I'd like to start with user_data. | 15:44 |
| dtantsur | jroll: I'm not sure I get the "where we want to be" bit | 15:45 |
| kaifeng | will we support multi-file packaging for the user_data? | 15:45 |
| jroll | dtantsur: for example, maybe it's worth deprecating passing a iso image in, and pushing everyone to pass user_data/etc instead | 15:45 |
| dtantsur | kaifeng: user_data is one file from our perspective. then each vendor is doing something of their choice. | 15:45 |
| TheJulia | kaifeng: I feel like user data would need to be pre-assembled however it is posted | 15:45 |
| dtantsur | jroll: dunno, I see both as useful | 15:46 |
| TheJulia | jroll: I'm -1 to deprecating passing an pre-built config drive image. We have operators that build in tools and additional files they need when they use standalone | 15:46 |
| dtantsur | e.g. for particularly crazy cases with custom layout etc | 15:46 |
| TheJulia | We already do a "is this a url or is this a blob of encoded data, adding a third of "is this a dict" seems reasonable to me | 15:47 |
| jroll | another example, are we sure we don't want specific fields for this? | 15:47 |
| jroll | which should nova use? | 15:47 |
| kaifeng | okay, i just feel that would be easier for users, but no strong demands | 15:47 |
| jroll | etc | 15:47 |
| jroll | just feels like lots of questions | 15:47 |
| * dtantsur has not seen lots of questions so far | 15:48 | |
| * rloo wonders if it would be useful to have the configdrive building code in a library, outside of openstacksdk. | 15:48 | |
| dtantsur | I mean, we can argue about small details and let standalone users suffer. or we can agree on something that may not be 100% perfect. | 15:48 |
| TheJulia | I think the questions are that jroll is seeing it through eyes of nova, where dtantsur is seeing it purely from a standalone use case | 15:48 |
| rloo | i think it is worth discussing with nova first | 15:48 |
| dtantsur | we've been nova-centric for quite some time | 15:48 |
| TheJulia | I don't think that actually helps us | 15:49 |
| rloo | is dtantsur in a hurry cuz he wants to get something in stein? | 15:49 |
| jroll | TheJulia: dtantsur: I'm seeing it from a perspective of "should we give folks more than 5 minutes in a meeting to think about this" | 15:49 |
| dtantsur | rloo: I'd prefer to get it in stein, yes, although patching things downstream is certainly an option. | 15:49 |
| jroll | you both know I'm open to small additions that aren't 100% perfect | 15:49 |
| jroll | this is an API thing we need to support forever, I prefer not to slam those in a week before feature freeze | 15:50 |
| dtantsur | I don't quite get why Nova is an argument here. Nova doesn't have problems with our configdrive building AFAIK. Standalone users seem to (esp. coming from new languages). | 15:50 |
| TheJulia | The background on this is the gophecloud are rejecting config drive building support | 15:50 |
| dtantsur | I nearly see it as a technical debt we've been shying away from because standalone users are 2nd class | 15:50 |
| TheJulia | so either it gets cargo culted around in code that uses gophercloud | 15:50 |
| dtantsur | (ditto for allocation API) | 15:51 |
| TheJulia | or we add an API extension to support it | 15:51 |
| jroll | I don't have an argument. I have questions about consistency, that also involve nova | 15:51 |
| TheJulia | I think it is a good topic to discuss with them at the PTG | 15:51 |
| mgoddard | I agree with the point about FF. We've got quite a lot features that have been planned for some time | 15:52 |
| TheJulia | "We're thinking of heading further in this direction, but intend to maintain compatability for prior methods... does nova care?" in essence | 15:52 |
| jroll | or would we prefer to do it this way in nova | 15:52 |
| TheJulia | exactly | 15:52 |
| jroll | TheJulia | The background on this is the gophecloud are rejecting config drive building support <- thank you for giving us the primary reason to do this | 15:53 |
| dtantsur | I still think we should become less Nova-centric with time | 15:53 |
| * TheJulia somehow missed "folks" after gophercloud | 15:53 | |
| jroll | to be absolutely clear, this is a good idea and I'm not trying to block it. I'm just trying to slow down a bit and make sure we get it right | 15:53 |
| TheJulia | dtantsur: I concur, and I think we already have the pattern established of double use of the field. so triple use is not a huge deal in my book | 15:54 |
| jroll | dtantsur: I think of the nova driver as our code, not nova's code. I'm purely asking if we should also consider that code in part of this. (I have the same question for bifrost) | 15:54 |
| dtantsur | for the record, I'm fine with 2-3-5 separate fields as well | 15:54 |
| * TheJulia worries 2-3-5 fields may actually cause lots of confusion.... | 15:55 | |
| dtantsur | jroll: I think bifrost, metalsmith, etc will immediately benefit from this | 15:55 |
| dtantsur | (I can surely say for metalsmith) | 15:55 |
| TheJulia | Well, early next cycle if people have time | 15:55 |
| TheJulia | Anyway, do we have consensus that it is a generally good idea? | 15:55 |
| dtantsur | jroll: changing nova requires coupling between how nova builds configdrives and how we do | 15:55 |
| dtantsur | which is something I'm a bit reserved about | 15:56 |
| dtantsur | because I'm not planning on something that covers every possible layout of a configdrive in the world | 15:56 |
| TheJulia | Which we may not be the only user of that code path either | 15:56 |
| dtantsur | right, you can have configdrives with VMs | 15:56 |
| dtantsur | (dunno why) | 15:56 |
| TheJulia | heh | 15:56 |
| mgoddard | security? | 15:57 |
| TheJulia | magical metadata service is not that magical | 15:57 |
| jroll | because metadata service was/is hard to scale | 15:57 |
| hjensas | dtantsur: ipv6? afik metadata does' | 15:57 |
| jroll | (probably more toward was) | 15:57 |
| hjensas | nt work with v6. | 15:57 |
| dtantsur | ah | 15:57 |
| rloo | i don't think anyone disagrees; it is a good idea. | 15:57 |
| TheJulia | anyway, we're heading in a bit of a tangent | 15:57 |
| jroll | right, good idea, I would like more discussion, I can drop questions on the story | 15:57 |
| TheJulia | I believe that is reasonable | 15:58 |
| TheJulia | So we have 2 minutes left | 15:58 |
| TheJulia | I guess time for open discussion and let asettle ask her question | 15:58 |
| TheJulia | #topic Open Discussion | 15:58 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:58 | |
| TheJulia | asettle: could you provide some background regarding your question | 15:59 |
| asettle | Mellow hello yes | 15:59 |
| asettle | Sorry for interrupting | 15:59 |
| asettle | Please :) thanks TheJulia | 15:59 |
| *** arne_wiebalck_ has quit IRC | 16:02 | |
| *** w14161_1 has quit IRC | 16:03 | |
| jroll | I suspect asettle is waiting for TheJulia to provide background, and vice versa :) | 16:03 |
| TheJulia | jroll: I was just wondering the same thing | 16:03 |
| jroll | should we end the meeting and just discuss as regular chatter? | 16:03 |
| TheJulia | Yeah, I think so | 16:03 |
| TheJulia | Thanks everyone! | 16:03 |
| rpioso | o/ | 16:03 |
| TheJulia | #endmeeting | 16:03 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-dev mailing list." | 16:03 | |
| openstack | Meeting ended Mon Feb 25 16:03:43 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:03 |
| rpittau|sardegna | thanks! | 16:03 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-02-25-15.00.html | 16:03 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-02-25-15.00.txt | 16:03 |
| dnuka | o/ | 16:03 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-02-25-15.00.log.html | 16:03 |
| NobodyCam | good morning ironic'ers | 16:03 |
| rpittau|sardegna | hey NobodyCam :) | 16:04 |
| dnuka | good morning NobodyCam | 16:04 |
| TheJulia | asettle: So your wondering why we have an api-ref and why we publish it? | 16:04 |
| TheJulia | asettle: and your saying rocky is broken? | 16:04 |
| NobodyCam | hey hey good morning rpittau|sardegna and dnuka | 16:04 |
| asettle | TheJulia, sorry, ha. Didn't realise you were waiting for me. | 16:05 |
| dnuka | o/ | 16:06 |
| *** dnuka has quit IRC | 16:06 | |
| asettle | So, my question is (and i'll provide context secondary to the question) Why do you need the API docs being published in doc/source/contributor/api/? What's the value in publishing these docs? | 16:06 |
| NobodyCam | :) | 16:06 |
| asettle | Context being: The api docs are causing the stable/rocky doc builds to fail if you decide to build a second time. | 16:07 |
| TheJulia | asettle: AHH! | 16:07 |
| asettle | As discussed, the stale files are *not* properly being removed when tox.ini is run for the second time | 16:07 |
| TheJulia | Well, we have out of tree drivers, so it makes sense to have both | 16:07 |
| asettle | WE've come up with several iterations and no matter what we do... we end up with duplications | 16:07 |
| rpittau|sardegna | or.. rmmmr... after a random x time :/ | 16:07 |
| asettle | ^^ good point, It's not realy consistent. It could be every second, third, or forth time | 16:07 |
| TheJulia | asettle: I thought we expcluded the build output... | 16:07 |
| asettle | Depending on the doc build's mood | 16:07 |
| asettle | TheJulia, didn't work. Resulting in (inconsistent) failures that look like what's in yellow on this etherpad: https://etherpad.openstack.org/p/ZhOwYHGE4J | 16:08 |
| jroll | so to be clear, doc/source/contributor/api/ is the python API docs, not the REST API docs | 16:08 |
| jroll | (before we go any deeper here) | 16:08 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Adding bios_interface reference to api docs https://review.openstack.org/639088 | 16:09 |
| asettle | jroll, thank you, good clarification | 16:09 |
| TheJulia | asettle: tox -edocs? | 16:11 |
| asettle | ye | 16:11 |
| asettle | (but every second/third/forth run... the error will eventually appear) | 16:11 |
| * TheJulia plays the jepordy music as tox -edocs runs | 16:12 | |
| * asettle dances | 16:12 | |
| jroll | tbh I would be very very surprised if anyone ever looked at the python api docs | 16:13 |
| TheJulia | asettle: "docstring of ironic.api.controllers.v1.Controller.get:1:duplicate object description of get, other instance in /home/jkreger/ironic3/doc/source/contributor/api/ironic.api.controllers.root.rst, use :noindex: for one of them | 16:13 |
| TheJulia | " sort of errors? | 16:13 |
| asettle | Ya that's it | 16:14 |
| rpittau|sardegna | yay! | 16:14 |
| rpittau|sardegna | I mean... oh no! :/ | 16:14 |
| asettle | Hooray rpittau|sardegna now other people can enjoy in our pain | 16:14 |
| TheJulia | yeah, I think there is a patch we can backport | 16:14 |
| * TheJulia looks | 16:14 | |
| asettle | wait what what | 16:14 |
| asettle | There's just a PATCH LYING AROUND | 16:14 |
| rpittau|sardegna | oO | 16:14 |
| * asettle tries not to screech | 16:14 | |
| kaifeng | https://review.openstack.org/#/c/638662/ | 16:14 |
| patchbot | patch 638662 - ironic - Remove docs build before building docs - 5 patch sets | 16:14 |
| jroll | lol | 16:14 |
| kaifeng | sounds like this? | 16:14 |
| *** stendulker has quit IRC | 16:14 | |
| asettle | kaifeng, aye no. Doesn't solve this error I'm afraid. | 16:14 |
| rpittau|sardegna | oh no, that is THE test | 16:14 |
| asettle | Tried that one | 16:14 |
| dtantsur | jroll: responses (largely reiterating what was said on the meeting): https://storyboard.openstack.org/#!/story/2005083#comment-117047 | 16:15 |
| dtantsur | jroll: also, cloud-init docs are pretty bad, but they do seem to support networking config via user_data | 16:16 |
| dtantsur | it's pretty implicit in https://cloudinit.readthedocs.io/en/latest/topics/network-config.html#disabling-network-configuration | 16:17 |
| kaifeng | dtantsur: iirc cloud-init support executing scripts, if you mean purely config networks by custom scripts | 16:17 |
| openstackgerrit | Merged openstack/ironic master: [trivial] Removing python 3.5 template jobs https://review.openstack.org/638632 | 16:17 |
| dtantsur | nope, they actually seem to support some networking configuration | 16:17 |
| * dtantsur remembers how much time he spent going through nova and cloud-init sources to understand all this | 16:18 | |
| TheJulia | asettle: Does anyone remember how we fixed the fun doc build issues | 16:18 |
| kaifeng | kaifeng reads code each time he hacks cloud-init :( | 16:18 |
| TheJulia | err | 16:18 |
| TheJulia | that was meant as a general question | 16:18 |
| asettle | TheJulia, no :p not personally. | 16:19 |
| TheJulia | oh, I think i found it | 16:19 |
| TheJulia | or at least one of them | 16:19 |
| asettle | TheJulia, could you explain to me what you're thinking here? I'm keen to know what this issue is and how it'll solve it :) | 16:20 |
| TheJulia | checking the backport locally | 16:22 |
| TheJulia | dpj | 16:22 |
| TheJulia | blah | 16:23 |
| asettle | AHUH | 16:23 |
| asettle | Sorry | 16:23 |
| asettle | Same error? | 16:23 |
| TheJulia | asettle: I had a general fix patch that I put together | 16:23 |
| TheJulia | testing | 16:23 |
| *** kaifeng has quit IRC | 16:23 | |
| *** mjturek has quit IRC | 16:24 | |
| TheJulia | some wip trash from the folder, trying... again \o/ | 16:24 |
| *** mjturek has joined #openstack-ironic | 16:25 | |
| TheJulia | re-running | 16:25 |
| *** e0ne has quit IRC | 16:26 | |
| TheJulia | 5th run with the backport | 16:26 |
| asettle | 5th run hollaaa | 16:26 |
| * rpittau|sardegna deja bu | 16:27 | |
| rpittau|sardegna | or vu | 16:27 |
| rpittau|sardegna | better vu | 16:27 |
| asettle | You pick | 16:27 |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/rocky: Fix doc builds for ironic https://review.openstack.org/639148 | 16:27 |
| TheJulia | asettle: ^^^ | 16:27 |
| asettle | TheJulia, looking | 16:28 |
| asettle | Was zum... | 16:29 |
| asettle | Well this is not what I had in mind at all | 16:29 |
| *** e0ne has joined #openstack-ironic | 16:31 | |
| *** Chaserjim has joined #openstack-ironic | 16:33 | |
| *** gyee has joined #openstack-ironic | 16:37 | |
| TheJulia | lol | 16:38 |
| TheJulia | Yeah, it was FUN! | 16:38 |
| asettle | TheJulia, and this was for master originally? | 16:39 |
| * dtantsur looks at the dead virtualbmc on his environment | 16:39 | |
| * asettle is finally building now | 16:39 | |
| TheJulia | asettle: yes, but during stein | 16:39 |
| asettle | Oh grand | 16:39 |
| rpittau|sardegna | well that was easy xD | 16:44 |
| rloo | TheJulia: wrt the question jroll asked on Friday about nova taking too long to remove vifs (because ironic's lock on node for cleaning), do you know if a bug was opened? i looked in nova and ironic's bugs and didn't see anything. | 16:51 |
| dtantsur | rloo: I think we ended up removing all VIFs ourselves in the end of deployment | 16:53 |
| rloo | dtantsur: nova code still tries to do it and is blocked. sec. | 16:53 |
| dtantsur | ugh | 16:53 |
| rloo | dtantsur: https://github.com/openstack/nova/blob/d5bde60e5680962394e263a662a2f331b6da93cd/nova/virt/ironic/driver.py#L1244 | 16:54 |
| rloo | dtantsur: i hacked something downstream but someone was asking me if there was an upstream bug | 16:55 |
| dtantsur | right.. | 16:55 |
| rloo | or if i should open one | 16:55 |
| * dtantsur is not aware of an existing bug | 16:55 | |
| rloo | dtantsur: julia discussed with jim on Friday: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2019-02-22.log.html#t2019-02-22T19:30:23 | 16:55 |
| dtantsur | it seems to me we can just stop unplugging VIFs in nova, since it's unlikely to work anyway? | 16:56 |
| rpittau|sardegna | good night o/ | 16:57 |
| rloo | dtantsur: i'm using ocata so can't verify wrt master, but in ocata, all it does is remove the info we've stashed in the port. i think we need to think about it conceptually, the _cleanup_deploy in the virt driver does other things. they will all block on the ironic node. | 16:57 |
| dtantsur | right, I don't think we backported it so far | 16:58 |
| dtantsur | in master we can drop many of cleanups since ironic does them | 16:58 |
| dtantsur | ocata.. hmm.. | 16:58 |
| rloo | dtantsur: i've hacked it for ocata, not a problem there. just want to make sure we fix it upstream. | 16:58 |
| TheJulia | rloo: and everyone, sorry brain is in another meeting at the moment | 16:59 |
| dtantsur | I'm not sure where my brain is, but apparently not here either | 16:59 |
| * dtantsur blames the visa stuffs | 17:00 | |
| rloo | heh. not asking for a solution now, just want to make sure we're tracking this bug. seems like the bug is in nova? (well, apparent from there anyway) | 17:00 |
| TheJulia | melwitt got involved in the discussion ?last? or was it ?2? weeks ago, we can drop the check or catch it in nova since we explicitly do the clean-up in ironic. someone needs to just get to submit the patch | 17:01 |
| rloo | TheJulia: do you know of a nova bug about it? i looked and didn't see anything. | 17:02 |
| TheJulia | rloo: I'm juggling like 10 different things right now, so I'm afraid I'm not aware of a bug #. I believe one was filed though | 17:02 |
| rloo | TheJulia: ok thx. | 17:03 |
| TheJulia | at some point... I'll get to my email :( | 17:06 |
| *** _fragatina_ has joined #openstack-ironic | 17:08 | |
| *** _fragatina has quit IRC | 17:10 | |
| *** spotz has joined #openstack-ironic | 17:10 | |
| *** sburette has quit IRC | 17:12 | |
| *** andrein has quit IRC | 17:17 | |
| *** e0ne has quit IRC | 17:18 | |
| *** Chaserjim has quit IRC | 17:19 | |
| *** dtantsur is now known as dtantsur|afk | 17:23 | |
| dtantsur|afk | \o | 17:23 |
| openstackgerrit | Merged openstack/ironic-inspector master: Revise driver api for introspection data backend https://review.openstack.org/637673 | 17:24 |
| *** sdake has joined #openstack-ironic | 17:26 | |
| *** _fragatina_ has quit IRC | 17:51 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add read-only Virtual Media support https://review.openstack.org/638787 | 17:56 |
| *** dougsz has quit IRC | 18:00 | |
| *** derekh has quit IRC | 18:01 | |
| *** trown is now known as trown|lunch | 18:04 | |
| *** sdake_ has joined #openstack-ironic | 18:05 | |
| *** sdake has quit IRC | 18:07 | |
| *** sdake_ has quit IRC | 18:10 | |
| *** sdake has joined #openstack-ironic | 18:11 | |
| *** sdake has quit IRC | 18:16 | |
| *** sdake_ has joined #openstack-ironic | 18:16 | |
| *** sdake_ has quit IRC | 18:20 | |
| *** baha has quit IRC | 18:22 | |
| *** sdake has joined #openstack-ironic | 18:22 | |
| *** sdake has quit IRC | 18:26 | |
| *** sdake_ has joined #openstack-ironic | 18:27 | |
| *** amoralej is now known as amoralej|off | 18:33 | |
| *** ijw has joined #openstack-ironic | 18:35 | |
| *** sdake_ has quit IRC | 18:35 | |
| *** ijw has quit IRC | 18:36 | |
| *** sdake has joined #openstack-ironic | 18:36 | |
| *** ijw has joined #openstack-ironic | 18:36 | |
| *** bnemec has quit IRC | 18:40 | |
| *** sdake has quit IRC | 18:41 | |
| *** sdake_ has joined #openstack-ironic | 18:42 | |
| *** e0ne has joined #openstack-ironic | 18:44 | |
| *** bnemec has joined #openstack-ironic | 18:45 | |
| *** sdake_ has quit IRC | 18:45 | |
| *** baha has joined #openstack-ironic | 18:45 | |
| *** sdake has joined #openstack-ironic | 18:47 | |
| *** tssurya has quit IRC | 18:47 | |
| *** sdake has quit IRC | 18:50 | |
| *** sdake has joined #openstack-ironic | 18:51 | |
| *** mjturek has quit IRC | 18:52 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Removing python 3.5 testing in favor of 3.6 https://review.openstack.org/638625 | 18:54 |
| openstackgerrit | Merged openstack/ironic master: Set available_nodes in tempest conf https://review.openstack.org/639101 | 18:54 |
| *** sdake has quit IRC | 18:55 | |
| *** sdake has joined #openstack-ironic | 18:57 | |
| *** sdake has quit IRC | 19:01 | |
| *** sdake has joined #openstack-ironic | 19:01 | |
| *** sdake has quit IRC | 19:05 | |
| *** sdake has joined #openstack-ironic | 19:07 | |
| *** sdake has quit IRC | 19:11 | |
| *** sdake has joined #openstack-ironic | 19:11 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Deploy templates: API & notifications https://review.openstack.org/631845 | 19:11 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: WIP: Deploy templates: conductor https://review.openstack.org/634732 | 19:11 |
| *** yolanda has joined #openstack-ironic | 19:15 | |
| *** sdake has quit IRC | 19:15 | |
| *** sdake_ has joined #openstack-ironic | 19:15 | |
| *** trown|lunch is now known as trown | 19:19 | |
| *** sdake_ has quit IRC | 19:20 | |
| *** andrein has joined #openstack-ironic | 19:22 | |
| *** _fragatina has joined #openstack-ironic | 19:22 | |
| openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Add public resource loading and message parsing https://review.openstack.org/590747 | 19:24 |
| openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Add @Redfish.Settings update status and expose it for BIOS https://review.openstack.org/591635 | 19:31 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add read-only Virtual Media support https://review.openstack.org/638787 | 19:52 |
| *** mjturek has joined #openstack-ironic | 19:55 | |
| *** arne_wiebalck_ has joined #openstack-ironic | 19:57 | |
| *** jtomasek has quit IRC | 20:39 | |
| *** jtomasek has joined #openstack-ironic | 20:41 | |
| *** jtomasek has quit IRC | 20:42 | |
| *** jtomasek has joined #openstack-ironic | 20:43 | |
| *** jrist has quit IRC | 20:59 | |
| *** lucasagomes has quit IRC | 21:01 | |
| *** jrist has joined #openstack-ironic | 21:03 | |
| *** cdearborn has quit IRC | 21:07 | |
| *** jrist has quit IRC | 21:09 | |
| *** jtomasek has quit IRC | 21:09 | |
| *** jrist has joined #openstack-ironic | 21:10 | |
| *** jcoufal has quit IRC | 21:16 | |
| *** e0ne has quit IRC | 21:21 | |
| TheJulia | jroll: rloo: dtantsur|afk: https://review.openstack.org/#/c/639148/ only needs a single core approval as a stable backport to fix doc builds. | 21:23 |
| patchbot | patch 639148 - ironic (stable/rocky) - Fix doc builds for ironic - 1 patch set | 21:23 |
| jroll | done | 21:23 |
| *** sdake has joined #openstack-ironic | 21:28 | |
| *** mjturek has quit IRC | 21:29 | |
| rloo | thx TheJulia, jroll, dtantsur|afk! | 21:41 |
| *** rcernin has joined #openstack-ironic | 21:46 | |
| *** whoami-rajat has quit IRC | 21:47 | |
| *** sdake has quit IRC | 21:48 | |
| *** baha has quit IRC | 21:48 | |
| TheJulia | thanks everyone! | 21:52 |
| *** e0ne has joined #openstack-ironic | 21:54 | |
| *** e0ne has quit IRC | 21:55 | |
| *** arne_wiebalck_ has quit IRC | 21:56 | |
| *** trown is now known as trown|outtypewww | 22:00 | |
| *** baha has joined #openstack-ironic | 22:02 | |
| *** sdake has joined #openstack-ironic | 22:03 | |
| *** e0ne has joined #openstack-ironic | 22:05 | |
| *** sdake has quit IRC | 22:05 | |
| *** e0ne has quit IRC | 22:08 | |
| *** sdake has joined #openstack-ironic | 22:08 | |
| *** MattMan_1 has joined #openstack-ironic | 22:09 | |
| *** sdake has quit IRC | 22:10 | |
| *** MattMan has quit IRC | 22:10 | |
| *** sdake has joined #openstack-ironic | 22:13 | |
| *** sdake has quit IRC | 22:13 | |
| *** anupn has joined #openstack-ironic | 22:24 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Add fast-track testing capability https://review.openstack.org/637039 | 23:05 |
| *** anupn has quit IRC | 23:05 | |
| *** baha has quit IRC | 23:25 | |
| *** hwoarang has quit IRC | 23:32 | |
| *** hwoarang has joined #openstack-ironic | 23:33 | |
| *** bfournie has quit IRC | 23:39 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: fast tracked deployment support https://review.openstack.org/635996 | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!