stevebaker | hey | 00:08 |
---|---|---|
*** rcernin has joined #openstack-ironic | 00:14 | |
*** rcernin has quit IRC | 00:22 | |
*** rcernin has joined #openstack-ironic | 00:22 | |
*** tosky has quit IRC | 00:26 | |
*** stevebaker has quit IRC | 02:15 | |
*** JayF has quit IRC | 02:23 | |
*** JayF has joined #openstack-ironic | 02:39 | |
*** ricolin has joined #openstack-ironic | 03:08 | |
*** rcernin has quit IRC | 03:22 | |
*** mkrai has joined #openstack-ironic | 03:24 | |
*** uzumaki has joined #openstack-ironic | 03:55 | |
*** rcernin has joined #openstack-ironic | 04:02 | |
*** rcernin has quit IRC | 04:07 | |
*** mkrai has quit IRC | 04:10 | |
*** mkrai has joined #openstack-ironic | 04:15 | |
*** rcernin has joined #openstack-ironic | 04:48 | |
*** rcernin has quit IRC | 04:48 | |
*** rcernin has joined #openstack-ironic | 05:26 | |
*** ricolin_ has joined #openstack-ironic | 05:52 | |
*** anuradha1904 has joined #openstack-ironic | 06:02 | |
arne_wiebalck | Good morning, ironic! | 06:52 |
janders | good morning arne_wiebalck o/ | 06:58 |
arne_wiebalck | hey janders o/ | 06:59 |
*** mkrai has quit IRC | 07:39 | |
*** mkrai_ has joined #openstack-ironic | 07:39 | |
*** ociuhandu has joined #openstack-ironic | 07:46 | |
*** ociuhandu has quit IRC | 07:52 | |
*** mkrai_ has quit IRC | 07:58 | |
*** rpittau|afk is now known as rpittau | 08:19 | |
rpittau | good morning ironic! o/ | 08:19 |
rpittau | if anyone has a moment I think we're good to go with this https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/781052 | 08:22 |
*** rcernin has quit IRC | 08:23 | |
*** uzumaki has quit IRC | 08:27 | |
*** mkrai has joined #openstack-ironic | 08:28 | |
*** ociuhandu has joined #openstack-ironic | 08:33 | |
*** ociuhandu has quit IRC | 08:38 | |
openstackgerrit | Riccardo Pittau proposed openstack/sushy stable/ussuri: Fix ExtendedInfo error handling for non-list item https://review.opendev.org/c/openstack/sushy/+/782029 | 08:42 |
*** tosky has joined #openstack-ironic | 08:49 | |
*** ociuhandu has joined #openstack-ironic | 08:56 | |
*** lucasagomes has joined #openstack-ironic | 08:56 | |
*** rcernin has joined #openstack-ironic | 08:58 | |
*** ociuhandu has quit IRC | 09:18 | |
*** mkrai has quit IRC | 09:22 | |
*** mkrai_ has joined #openstack-ironic | 09:22 | |
*** derekh has joined #openstack-ironic | 09:27 | |
*** moshiur has joined #openstack-ironic | 09:30 | |
moshiur | Good morning. rpittau: can we please move forward and merge this PR?: https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/778726 | 09:32 |
rpittau | moshiur: good morning, as far I would like to since that is approved, we can't because the patch it depends on is still not merged | 09:34 |
moshiur | Hi TheJulia: is there anything that is blocking us to get merge the following PR?: https://review.opendev.org/c/openstack/diskimage-builder/+/778723 | 09:35 |
moshiur | rpittau: Thanks, I asked TheJulia. | 09:36 |
rpittau | moshiur: we don't control the diskimage-builder project, you should ask in the #openstack-infra or in the #opendev channels | 09:36 |
*** rcernin has quit IRC | 09:37 | |
moshiur | ok | 09:37 |
moshiur | rpittau: I asked for review in #openstack-infra. lets see.. | 09:43 |
*** ociuhandu has joined #openstack-ironic | 09:44 | |
*** ociuhandu has quit IRC | 09:46 | |
*** ociuhandu_ has joined #openstack-ironic | 09:46 | |
arne_wiebalck | rpittau: I am having trouble getting devstack going (with Ironic), tried with bionic and c8 so far ... any recommendations on what has some chances to succeed? | 09:51 |
*** viks____ has joined #openstack-ironic | 09:56 | |
*** dtantsur|afk is now known as dtantsur | 10:06 | |
dtantsur | morning ironic | 10:06 |
dtantsur | moshiur: actually, there is #openstack-dib specifically for diskimage-builder | 10:06 |
*** mkrai_ has quit IRC | 10:13 | |
*** ricolin_ has quit IRC | 10:16 | |
*** ociuhandu_ has quit IRC | 10:16 | |
*** mkrai has joined #openstack-ironic | 10:16 | |
*** ociuhandu has joined #openstack-ironic | 10:34 | |
*** ociuhandu has quit IRC | 10:36 | |
*** ociuhandu has joined #openstack-ironic | 10:37 | |
rpittau | d'oh! how did I forget that? | 10:37 |
*** ociuhandu has quit IRC | 10:37 | |
*** ociuhandu has joined #openstack-ironic | 10:38 | |
*** ociuhandu has quit IRC | 10:38 | |
*** ociuhandu has joined #openstack-ironic | 10:38 | |
*** ociuhandu has quit IRC | 10:38 | |
rpittau | arne_wiebalck: What kind of trouble? To be honest it's a while I don't run devstack on centos8, but at least with ussuri I don't remember any issue, bionic should work the same, but I suggest to run it on focal if possible, especially if victoria or wallaby. | 10:39 |
arne_wiebalck | I'll try focal now | 10:40 |
arne_wiebalck | this kind of issue: https://stackoverflow.com/questions/63439723/error-in-installation-of-openstack-in-ubuntu | 10:40 |
*** ociuhandu has joined #openstack-ironic | 10:40 | |
arne_wiebalck | (on centos8) | 10:41 |
arne_wiebalck | on bionic, there was a neutron issue when running stack.sh | 10:41 |
arne_wiebalck | this is all with the local.conf we have in our docs | 10:41 |
arne_wiebalck | our == upstream ironic | 10:41 |
arne_wiebalck | rpittau: and, this is master :) | 10:42 |
*** ociuhandu has quit IRC | 10:43 | |
*** ociuhandu has joined #openstack-ironic | 10:44 | |
rpittau | arne_wiebalck: for master definitely with focal, was that centos8 stream? | 10:44 |
arne_wiebalck | rpittau: no, centos | 10:44 |
arne_wiebalck | rpittau: ok, let me try with focal ... | 10:44 |
rpittau | mmm I may try with centos8 stream, some packages may not have been updated with older centos | 10:45 |
arne_wiebalck | the devstack upstream docs recommend bionic, I think, that's why I started there | 10:49 |
rpittau | arne_wiebalck: is that devstack docs or ironic docs ? | 10:49 |
arne_wiebalck | rpittau: devstack | 10:49 |
* arne_wiebalck hopes he was not on some outdated page now ... | 10:49 | |
arne_wiebalck | https://docs.openstack.org/devstack/latest/ recommends bionic | 10:51 |
rpittau | arne_wiebalck: no, it's actually the latest docs that suggest bionic, which is... weird | 10:51 |
rpittau | we stopped testing on bionic in ussuri AFAIR, from victoria the official testing OS is focal | 10:52 |
arne_wiebalck | thanks! | 10:57 |
*** mkrai has quit IRC | 11:00 | |
moshiur | Thanks dtantsur: I ask for review in #openstack-dib | 11:04 |
openstackgerrit | Merged openstack/ironic master: Add anaconda configuration and template https://review.opendev.org/c/openstack/ironic/+/772166 | 11:05 |
*** rcernin has joined #openstack-ironic | 11:13 | |
*** ociuhandu has quit IRC | 11:18 | |
*** rcernin has quit IRC | 11:18 | |
*** uzumaki has joined #openstack-ironic | 11:21 | |
openstackgerrit | Dhuldev Valekar proposed openstack/ironic master: DRAC: Added redfish management clean steps https://review.opendev.org/c/openstack/ironic/+/721593 | 11:27 |
*** ociuhandu has joined #openstack-ironic | 11:28 | |
*** ociuhandu has quit IRC | 11:30 | |
*** ociuhandu has joined #openstack-ironic | 11:31 | |
openstackgerrit | sonali bhausaheb borkar proposed openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port https://review.opendev.org/c/openstack/ironic/+/693328 | 11:35 |
*** rcernin has joined #openstack-ironic | 11:44 | |
*** monica_pardhi has joined #openstack-ironic | 11:45 | |
*** abhishekyadav has joined #openstack-ironic | 11:47 | |
abhishekyadav | @mon | 11:47 |
iurygregory | good morning Ironic! | 11:52 |
*** rh-jelabarre has joined #openstack-ironic | 11:55 | |
*** monica_pardhi has quit IRC | 11:55 | |
janders | good morning / afternoon rpittau dtantsur iurygregory o/ | 11:56 |
*** mkrai has joined #openstack-ironic | 11:56 | |
iurygregory | janders, o/ | 11:57 |
openstackgerrit | Jacob Anders proposed openstack/ironic master: Generic way to configure clean step priorites https://review.opendev.org/c/openstack/ironic/+/744117 | 11:57 |
iurygregory | janders, dtantsur hey regarding the way of setting MultiOpt, if oslo supports both ways we will be able to validate we don't support? .-. | 12:07 |
*** ociuhandu has quit IRC | 12:09 | |
iurygregory | in cinder when they are using multiopt with dict it's possible to configure things that way https://docs.openstack.org/cinder/pike/contributor/replication.html#service-configuration | 12:09 |
openstackgerrit | Dhuldev Valekar proposed x/sushy-oem-idrac master: Update management clean steps https://review.opendev.org/c/x/sushy-oem-idrac/+/782219 | 12:09 |
openstackgerrit | Merged openstack/ironic stable/ussuri: Enable swift temporary URLs in grenade and provide a good error message https://review.opendev.org/c/openstack/ironic/+/777430 | 12:29 |
*** rcernin has quit IRC | 12:31 | |
*** rcernin has joined #openstack-ironic | 12:31 | |
*** ociuhandu has joined #openstack-ironic | 12:39 | |
*** jamesdenton has quit IRC | 12:48 | |
*** jamesdenton has joined #openstack-ironic | 12:49 | |
dtantsur | iurygregory, janders, mmmmm... I'm not sure we should artificially restrict this feature, maybe just don't document it? | 12:56 |
dtantsur | I mean, don't document setting multiple values per line (and use singular for the option name)? | 12:57 |
openstackgerrit | Merged openstack/ironic-python-agent stable/victoria: Fix root UUID for streamed partition images https://review.opendev.org/c/openstack/ironic-python-agent/+/781814 | 13:01 |
openstackgerrit | Merged openstack/ironic stable/train: DRAC: Fix a failure to create virtual disk https://review.opendev.org/c/openstack/ironic/+/769719 | 13:01 |
*** ociuhandu has quit IRC | 13:02 | |
iurygregory | dtantsur, I'm ok with it =) | 13:04 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Make baremetal --debug actually work https://review.opendev.org/c/openstack/python-ironicclient/+/781100 | 13:05 |
iurygregory | habemus Backport Candidate label =D | 13:07 |
dtantsur | \o/ | 13:07 |
*** ociuhandu has joined #openstack-ironic | 13:08 | |
* dtantsur uses | 13:08 | |
iurygregory | now I will update my script to usse gerrit api to see if I can do some magic with it :D | 13:09 |
iurygregory | after fixing some unit tests | 13:09 |
TheJulia | good morning | 13:11 |
*** rloo has joined #openstack-ironic | 13:11 | |
dtantsur | morning TheJulia | 13:11 |
iurygregory | good morning TheJulia | 13:11 |
arne_wiebalck | rpittau: With focal, I hit the issue mentioned in the stackoverflow link mentioned above (and the fix mentioned there seems to indeed fix this). | 13:12 |
arne_wiebalck | rpittau: The local.conf from the Ironic docs did not manage to bring up devstack, the local.conf from the samples dir seems to work ok. | 13:12 |
TheJulia | arne_wiebalck: that is not terribly srprising :( | 13:14 |
arne_wiebalck | TheJulia: the sample local.conf is pretty minimal I realise now | 13:14 |
arne_wiebalck | TheJulia: why not? | 13:15 |
openstackgerrit | Merged openstack/ironic master: Use OOB inspection to fetch MACs for IB inspection https://review.opendev.org/c/openstack/ironic/+/774391 | 13:16 |
TheJulia | What is in the docs is kind of old and easy for people to miss | 13:16 |
janders | dtantsur iurygregory not restricting and not documenting sounds reasonable | 13:18 |
janders | see you tomorrow Ironic o/ | 13:18 |
iurygregory | bye janders o/ | 13:18 |
janders | iurygregory dtantsur thanks heaps for your help with https://review.opendev.org/c/openstack/ironic/+/744117 today, much appreciated! :) | 13:18 |
janders | cheers | 13:18 |
iurygregory | janders, yw =) | 13:19 |
arne_wiebalck | TheJulia: Do we have a newer version of how devstack for Ironic should be configured somewhere? | 13:20 |
iurygregory | arne_wiebalck, you can take the local.conf from the jobs :D | 13:20 |
iurygregory | it's not a minimal example, but it would help I would say | 13:21 |
arne_wiebalck | iurygregory: Thanks! Could you point me to one you would deem a sensible starting point? | 13:22 |
rpittau | arne_wiebalck: the example in the docs definitely needs updating | 13:23 |
arne_wiebalck | rpittau: :-D | 13:23 |
iurygregory | arne_wiebalck, sure! | 13:23 |
arne_wiebalck | rpittau: happy to do that once I got one running ;) | 13:23 |
*** tzumainn has joined #openstack-ironic | 13:25 | |
rpittau | I'm still at lunch, end of meal coffee takes time in Italy :) | 13:26 |
rpittau | I'll be happy to review that, I'll check the jobs values later | 13:26 |
*** abhishekyadav has quit IRC | 13:27 | |
openstackgerrit | Dhuldev Valekar proposed x/sushy-oem-idrac master: Update management clean step https://review.opendev.org/c/x/sushy-oem-idrac/+/782254 | 13:32 |
TheJulia | arne_wiebalck: no, But | 13:34 |
TheJulia | arne_wiebalck: but I thinkI started heading into updating those docs and got stuck in a loop :) | 13:34 |
arne_wiebalck | TheJulia: iurygregory made some suggestions, I'll see if I can get things going. If yes, we can circle back to the docs. | 13:35 |
iurygregory | the only thing is that it will be tied to a job definition hehe | 13:35 |
rpittau | well, you can make it generic | 13:39 |
iurygregory | yeah, after removing some variables and checking if it would work :D | 13:41 |
rpittau | eheh yeah | 13:42 |
TheJulia | One thing I noticed, etcd3 needs to be disabled | 13:43 |
TheJulia | It also likely needs to be removed or fixed with devstack, but...yeah | 13:43 |
*** ociuhandu has quit IRC | 13:44 | |
TheJulia | Review Jam in 15 minutes | 13:44 |
*** ociuhandu has joined #openstack-ironic | 13:46 | |
*** ociuhandu has quit IRC | 13:46 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Refactor: use mounted from ironic-lib https://review.opendev.org/c/openstack/ironic-python-agent/+/782259 | 13:47 |
openstackgerrit | Dhuldev Valekar proposed x/sushy-oem-idrac master: Update the resources to put idrac in known good state https://review.opendev.org/c/x/sushy-oem-idrac/+/782254 | 13:47 |
*** ociuhandu has joined #openstack-ironic | 13:49 | |
*** ociuhandu has quit IRC | 13:50 | |
*** ociuhandu has joined #openstack-ironic | 13:50 | |
*** ociuhandu has quit IRC | 13:55 | |
*** ociuhandu has joined #openstack-ironic | 13:58 | |
*** ociuhandu has quit IRC | 14:00 | |
*** ociuhandu has joined #openstack-ironic | 14:01 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Hardware manager call to load configuration from vmedia https://review.opendev.org/c/openstack/ironic-python-agent/+/782262 | 14:01 |
*** jhesketh has quit IRC | 14:02 | |
*** jhesketh has joined #openstack-ironic | 14:04 | |
*** rcernin has quit IRC | 14:05 | |
*** ociuhandu_ has joined #openstack-ironic | 14:21 | |
*** ociuhandu has quit IRC | 14:22 | |
*** ociuhandu has joined #openstack-ironic | 14:23 | |
*** lmcgann has joined #openstack-ironic | 14:24 | |
*** ociuhand_ has joined #openstack-ironic | 14:24 | |
*** ociuhandu has quit IRC | 14:24 | |
*** ociuhandu has joined #openstack-ironic | 14:25 | |
*** ociuhandu_ has quit IRC | 14:26 | |
*** ociuhand_ has quit IRC | 14:28 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Support pre-built deploy/rescue ISO in Redfish https://review.opendev.org/c/openstack/ironic/+/782266 | 14:32 |
*** ociuhandu_ has joined #openstack-ironic | 14:33 | |
*** ociuhandu has quit IRC | 14:34 | |
*** ociuhand_ has joined #openstack-ironic | 14:34 | |
*** lbragstad__ is now known as lbragstad | 14:36 | |
*** ociuhandu_ has quit IRC | 14:38 | |
*** kaifeng has joined #openstack-ironic | 14:48 | |
*** dansmith has quit IRC | 14:53 | |
rpittau | ajya, rpioso, are you going to cut a stable/wallaby for sushy-oem-idrac ? besides the new 2.0.0 version I mean | 14:54 |
*** frigo has joined #openstack-ironic | 14:54 | |
TheJulia | zer0c00l: Hey, when you around, I have a question w/r/t https://review.opendev.org/c/openstack/ironic/+/775891 | 14:55 |
*** stendulker has joined #openstack-ironic | 14:56 | |
*** Qianbiao has joined #openstack-ironic | 14:56 | |
*** dansmith has joined #openstack-ironic | 14:57 | |
TheJulia | #startmeeting ironic | 15:00 |
openstack | Meeting started Mon Mar 22 15:00:14 2021 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
TheJulia | o/ | 15:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
iurygregory | o/ | 15:00 |
*** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
openstack | The meeting name has been set to 'ironic' | 15:00 |
bfournie | o/ | 15:00 |
bdodd | o/ | 15:00 |
dtantsur | o/ | 15:00 |
arne_wiebalck | o/ | 15:00 |
ajya | o/ | 15:00 |
rpittau | o/ | 15:00 |
erbarr | o/ | 15:00 |
rpioso | \o | 15:00 |
TheJulia | Our agenda for this week can be found on the wiki. | 15:00 |
rloo | o/ | 15:00 |
TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:00 |
TheJulia | #topic Announcements / Reminders | 15:01 |
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)" | 15:01 | |
Qianbiao | o/ | 15:01 |
kaifeng | o/ | 15:01 |
stendulker | o/ | 15:01 |
TheJulia | #info This week is week R-3 in the Wallaby release schedule. Most projects will be cutting their RC1 and stable branches this week. | 15:01 |
*** frigo has quit IRC | 15:02 | |
TheJulia | #info Due to other projects releasing, we may encounter devstack changes which will may force us to release early. Currently we're hoping to release next week. | 15:02 |
TheJulia | #info PTG has been scheduled and a number of topics for discussion are on the etherpad. | 15:03 |
TheJulia | #link https://april2021-ptg.eventbrite.com/ | 15:03 |
TheJulia | #link https://etherpad.opendev.org/p/ironic-xena-ptg | 15:03 |
TheJulia | Does anyone have anything to announce or remind us of this week? | 15:03 |
iurygregory | we have the Backport-Candidate label \o/ | 15:04 |
iurygregory | it will probably help us track things we want to be sure to backport =) | 15:04 |
iurygregory | only cores can set the label, so when reviewing a patch if you think we should backport add the +1 =) | 15:05 |
TheJulia | #info We now have a backport-candidate label \o/ | 15:05 |
*** frigo has joined #openstack-ironic | 15:05 | |
*** moshiur has quit IRC | 15:05 | |
TheJulia | iurygregory: Thanks for taking that on and getting the patches through Infra | 15:05 |
iurygregory | np o/ | 15:05 |
TheJulia | Looks like we had no action items from last week. | 15:06 |
TheJulia | Does anyone have anything else to announce? | 15:07 |
TheJulia | FWIW, I *may* try to take the week of R-1 or R-0 off, but have not solidified any plans as of yet. | 15:07 |
TheJulia | I guess the only other thing to mention is I believe daylight savings time in Europe is next week? | 15:08 |
rpittau | yep | 15:09 |
TheJulia | :( | 15:09 |
JayF | Are there any outstanding patches that need review/merge targetted for the Wallaby release? | 15:09 |
rpittau | march 28, end of this week | 15:09 |
TheJulia | JayF: I believe there are, but I think we'll get to that during our next segment | 15:09 |
TheJulia | Well, if there is nothing else to announce or remind ourselves of, then I believe we can proceed | 15:10 |
TheJulia | #topic Review subteam status reports | 15:10 |
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:10 | |
TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:11 |
TheJulia | Starting around line 260 | 15:11 |
TheJulia | dtantsur: do you have anything w/r/t dhcp-less to merge this cycle? | 15:13 |
dtantsur | TheJulia: I'm concentrated on glean at this moment | 15:14 |
TheJulia | dtantsur: ack | 15:14 |
TheJulia | rpittau: For the anaconda stuff since it seems to be merging, maybe we add the ironic-week-prio hashtag if not already present to everything except the very last patch in that chain? | 15:15 |
rpittau | TheJulia: sounds good | 15:15 |
rpittau | done | 15:17 |
TheJulia | ajya: thank you for noting the status of the config molds item | 15:17 |
ajya | TheJulia: yw | 15:17 |
TheJulia | Onward to priorities for the week? | 15:18 |
rpittau | let's | 15:18 |
TheJulia | #topic Deciding on priorities for the coming week | 15:19 |
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:19 | |
TheJulia | The main question I want to ask during this is what is needed to release, and make sure we get our eyes on those items during the next few days | 15:20 |
TheJulia | Are there any new items that need to be considered? | 15:20 |
ajya | the ones from review jam | 15:20 |
TheJulia | I believe they already have the hashtag, but will check in a minute | 15:20 |
rpioso | TheJulia: We have a couple of oldie, but goodie changes, too: https://review.opendev.org/c/openstack/ironic/+/693328 and https://review.opendev.org/c/openstack/ironic/+/721593 | 15:21 |
rpittau | well we need ironic, ironic-inspector, ironic-python-agent, and bifrost at least | 15:22 |
iurygregory | https://review.opendev.org/c/openstack/ironic/+/721593 - merge conflict | 15:22 |
iurygregory | and missing the hashtag ironic-week-prio (it will help to get visibility) | 15:22 |
ajya | https://review.opendev.org/c/openstack/ironic/+/769928 https://review.opendev.org/c/openstack/ironic/+/759428 - these don't have hashtag | 15:23 |
ajya | can I add it myself? | 15:23 |
rpioso | iurygregory: May I add that tag? | 15:23 |
TheJulia | Tags added | 15:24 |
iurygregory | ajya, in the ones you are the owner/Uploader you can | 15:24 |
iurygregory | rpioso, ^ same | 15:24 |
iurygregory | if you think the patch is in good shape and ready for review =) | 15:24 |
ajya | ok, I'll add updates as discussed in the review jam and then hashtag | 15:25 |
TheJulia | I was about to propose a bifrost change and loaded it and see it was already approved in the time between the list view and me loading the patch | 15:25 |
iurygregory | *magic* | 15:25 |
rpittau | no priority for release but we need this for the exporter https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/781927 | 15:26 |
TheJulia | Does https://review.opendev.org/c/openstack/ironic-python-agent/+/782259 make sense for priorites for this week? | 15:26 |
rpioso | TheJulia: Thank you for adding the tags :) | 15:26 |
JayF | TheJulia: I'll review (and likely land) that right now-ish so may not be worth the paperwork :) | 15:26 |
iurygregory | rpittau, +W =) quick fix | 15:26 |
TheJulia | JayF: ack | 15:26 |
TheJulia | nothing else on IPA looks really pressing at the moment | 15:27 |
TheJulia | unless someone has something that is hiding from the list of last 50 patches | 15:27 |
rpittau | thanks iurygregory | 15:27 |
TheJulia | I think https://review.opendev.org/c/openstack/ironic-inspector/+/770708 might make sense, but since they haven't done the release note can I get a volunteer? | 15:28 |
*** mkrai has quit IRC | 15:28 | |
iurygregory | TheJulia, I can | 15:28 |
TheJulia | iurygregory: thanks | 15:29 |
iurygregory | will push an update today | 15:29 |
rpittau | a quick one more https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/781052 | 15:30 |
TheJulia | That one was easy :) | 15:31 |
rpittau | :) | 15:31 |
TheJulia | I found an important patch | 15:33 |
TheJulia | https://review.opendev.org/c/openstack/ironic/+/780974\ | 15:33 |
rpittau | oh wow, that is indeed | 15:33 |
TheJulia | and https://review.opendev.org/c/openstack/ironic/+/780973 | 15:33 |
rpittau | couple of comments to sort out | 15:34 |
TheJulia | Well, https://review.opendev.org/q/hashtag:%2522ironic-week-prio%2522+(status:open) looks pretty good :) | 15:35 |
TheJulia | https://review.opendev.org/c/openstack/ironic/+/767432 <-- really needs to make 17 or I need to do more secure-rbac policy patches | 15:36 |
rpittau | oh yeah, the depends on patch has merged, good | 15:37 |
TheJulia | Well, I think we're done with priorities and can proceed onward! | 15:38 |
dtantsur | yep | 15:38 |
TheJulia | ohh, we'll need to update the release mapping | 15:38 |
TheJulia | I can do that this week | 15:38 |
dtantsur | and fix API versions list | 15:38 |
openstackgerrit | Aija Jauntēva proposed openstack/ironic master: Add configuration mold storage https://review.opendev.org/c/openstack/ironic/+/769928 | 15:38 |
*** frigo has quit IRC | 15:38 | |
TheJulia | dtantsur: yup | 15:38 |
TheJulia | arne_wiebalck: anything for the Baremetal SIG this week? | 15:39 |
TheJulia | Looks like arne_wiebalck is away from his keyboard at the moment | 15:40 |
TheJulia | I guess we can proceed to Open Discussion | 15:40 |
TheJulia | #topic Open Discussion | 15:40 |
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:40 | |
TheJulia | Does anyone have anything they would like to bring up this week? | 15:40 |
arne_wiebalck | Bare Metal SIG: NTR I think :) | 15:41 |
rpittau | just one thing, please keep an eye on the tinyipa jobs as they may require RAM increase for the recent tinycore 12.x update | 15:41 |
TheJulia | #info please keep an eye on the tinyipa jobs as they may require RAM increase for the recent tinycore 12.x update | 15:41 |
rpittau | thanks :) | 15:41 |
TheJulia | Anything else for this week? | 15:44 |
TheJulia | Or have the crickets arrived? | 15:44 |
iurygregory | probably | 15:45 |
TheJulia | Well, Thanks everyone, and have a wonderful week! | 15:46 |
rpittau | thanks! | 15:46 |
dtantsur | o/ | 15:46 |
Qianbiao | thanks, have a good week | 15:46 |
arne_wiebalck | thanks TheJulia | 15:46 |
iurygregory | \o/ | 15:46 |
TheJulia | Final reminder, Add PTG topics! | 15:46 |
TheJulia | :) | 15:47 |
TheJulia | #endmeeting | 15:47 |
*** 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." | 15:47 | |
openstack | Meeting ended Mon Mar 22 15:47:04 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:47 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-22-15.00.html | 15:47 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-22-15.00.txt | 15:47 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-22-15.00.log.html | 15:47 |
*** stendulker has quit IRC | 15:48 | |
rloo | TheJulia: wrt https://review.opendev.org/c/openstack/ironic/+/767432. Thinking you want it addressed sooner rather than later. I commented... | 15:52 |
TheJulia | rloo: its been deprecated for like ?5? years | 15:53 |
rloo | TheJulia: did we deprecate it or say that it would still work for < 1.28 version? That's what I don't recall. | 15:54 |
rloo | to be honest, now that i think about it, what's the diff wrt 'deprecate' and using microversions. think i need more caffeine... | 15:55 |
TheJulia | deprecate it... I think you led the last charge to rip it out :) | 15:55 |
* rloo looks in release notes... | 15:55 | |
TheJulia | I don't think we ever committed to keeping it working for the API version <1.28 | 15:56 |
rloo | yeah, i think i did too. but my comment in the code seems to indicate that i thought we still needed to support it if < 1.28. | 15:56 |
TheJulia | But even if we did, we quite literally cannot | 15:56 |
TheJulia | We cannot because it bypasses RBAC checking for what could be a privileged node action | 15:56 |
rloo | i don't see how RBAC in a newer microversion, is expected to 'work properly' with an older microversion. | 15:57 |
TheJulia | RBAC is not microversioned behavior, and is operator overridable via custom policy files. | 15:58 |
rloo | i thought we up | 15:58 |
rloo | ed the microversino when we added rbac? | 15:58 |
TheJulia | As a signal to indicate "capability should be present" | 15:58 |
TheJulia | That will allow us to customize tempest tests | 15:58 |
*** Qianbiao has quit IRC | 15:59 | |
TheJulia | Because in the future, some of the tests will no longer be valid depending on the exact operating mode they are connecting with | 15:59 |
TheJulia | or need to use specific credentials | 15:59 |
JayF | It's worth calling attention to RBAC/policy having a separate deprecation path than API | 16:00 |
JayF | the old rules are in there, and will be for a deprecation period if enabled | 16:00 |
TheJulia | The conundrum is using extra info is an end-run around it all. | 16:01 |
rloo | i skimmed and don't see any mention of deprecating that in the release notes. going to do a more thorough search... | 16:03 |
TheJulia | so I have two paths... rip out the capability completely, or build rbac rules around it. Only two x/ namespace projects even still reference extra['vif_port_id'] | 16:03 |
* iurygregory lunch time | 16:03 | |
rloo | am wondering if deleting this can wait another cycle. put a deprecation notice in wallaby. | 16:04 |
TheJulia | can't | 16:04 |
TheJulia | I either need to completely rip it out, or I need to put rbac rules in place | 16:04 |
rloo | putting rbac rules in place is to say that ironic has 'rbac' completed? | 16:04 |
TheJulia | https://docs.openstack.org/releasenotes/ironic/ocata.html#deprecation-notes | 16:05 |
rloo | ++ thx for finding that. | 16:05 |
rloo | ok, i think we're good then. | 16:05 |
TheJulia | rloo: Well, capability to enforce access controls for non-privilged users | 16:06 |
openstackgerrit | Aija Jauntēva proposed openstack/ironic master: Add import, export configuration to idrac-redfish https://review.opendev.org/c/openstack/ironic/+/759428 | 16:06 |
TheJulia | owner/lessee provided some of that framework/capability, but rbac just allowed us to basically take it to the next level | 16:06 |
TheJulia | just... the glaring security issue in all of it is that field :) | 16:06 |
ajya | rpittau: re sushy-oem-idrac wallaby branch - yes, there will be (cc rpioso) | 16:06 |
rpittau | ajya: thanks | 16:06 |
openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Increase RAM for ironic vm https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/781927 | 16:09 |
openstackgerrit | Verification of a change to openstack/bifrost failed: Relax format of checksum file https://review.opendev.org/c/openstack/bifrost/+/778578 | 16:15 |
TheJulia | iurygregory: I commented w/r/t netboot on https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/741445 | 16:16 |
TheJulia | fwiw | 16:16 |
*** anuradha1904 has quit IRC | 16:22 | |
rpioso | rpittau: We are likely going to create a 2.1.0. There could be a 2.1.1, too. Those would both be done before Wallaby ironic is released. Is there any reason why we should not wait to cut stable/wallaby until the dust settles? | 16:27 |
rpittau | rpioso: not really, it was mainly a curiosity, that is an independent project so it's totally up to you. If it was for me I would try to follow the non-client libs release schedule for consistency with sushy, but it's not mandatory in the case of sushy-oem-idrac, in the end it really all depends if you have more features to add or not. | 16:32 |
*** penick has joined #openstack-ironic | 16:34 | |
*** penick has quit IRC | 16:35 | |
*** penick has joined #openstack-ironic | 16:36 | |
rpioso | rpittau: Thank you for clarifying :) | 16:42 |
TheJulia | ajya: I spotted a minor s/web/http/ on https://review.opendev.org/c/openstack/ironic/+/769928. I also added more verbose comments for other reviewers | 16:42 |
* TheJulia is now on cat attempt to capture the microphone # 6 for today. | 16:42 | |
*** ociuhand_ has quit IRC | 16:42 | |
*** ociuhandu has joined #openstack-ironic | 16:43 | |
ajya | TheJulia: thanks, will update | 16:43 |
iurygregory | TheJulia, looking now | 16:46 |
*** ociuhandu_ has joined #openstack-ironic | 16:55 | |
ajya | TheJulia: for that retry thing, are there re-usable conf values or should I create new ones? | 16:55 |
TheJulia | It was just a thought. I would just add to the molds section of the configuration | 16:56 |
*** ociuhandu has quit IRC | 16:58 | |
*** ociuhandu_ has quit IRC | 16:59 | |
zer0c00l | TheJulia: pong | 17:04 |
TheJulia | question for you, I noticed you were adding arguments in the middle of kwargs on the anaconda pxe template changes | 17:05 |
TheJulia | change | 17:05 |
zer0c00l | i can move that kwarg to the end. | 17:05 |
TheJulia | At the end is preferred | 17:05 |
zer0c00l | i thought ramdisk_boot and anaconda_boot go together. | 17:05 |
zer0c00l | but if it can cause problems - yes i can move it | 17:05 |
TheJulia | Since things like tests also just do value checking in order | 17:05 |
TheJulia | yeah, in case someone did something like create_ramdisk(1,2,3,4,5,6,7,9) | 17:06 |
TheJulia | instead of actually defining the kw to associate | 17:06 |
zer0c00l | ok. | 17:07 |
*** lucasagomes has quit IRC | 17:07 | |
dtantsur | going to get some fresh air while it's still allowed | 17:09 |
dtantsur | o/ | 17:09 |
*** dtantsur is now known as dtantsur|afk | 17:09 | |
rpittau | dtantsur|afk: enjoy! :) | 17:09 |
TheJulia | ugh :( | 17:09 |
TheJulia | enjoy! | 17:09 |
TheJulia | I take it germany is going into another hard lockdown? | 17:09 |
rpittau | I think I'm going to get some fresh pizza, for the same reason :P | 17:09 |
TheJulia | :( | 17:09 |
rpittau | TheJulia: unfortunately yes, until end of April if I understood correctly | 17:10 |
TheJulia | ugh | 17:10 |
*** penick has quit IRC | 17:11 | |
rpittau | bye everyone! o/ | 17:11 |
*** rpittau is now known as rpittau|afk | 17:11 | |
openstackgerrit | Aija Jauntēva proposed openstack/ironic master: Add configuration mold storage https://review.opendev.org/c/openstack/ironic/+/769928 | 17:16 |
openstackgerrit | Merged openstack/python-ironicclient master: Fix --fields network_data https://review.opendev.org/c/openstack/python-ironicclient/+/781112 | 17:19 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Revise release notes for 17.0 release https://review.opendev.org/c/openstack/ironic/+/780973 | 17:19 |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Add a job to test tinyipa in ussuri https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/781052 | 17:22 |
*** jamesdenton has quit IRC | 17:24 | |
*** jamesdenton has joined #openstack-ironic | 17:24 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Revise release notes for 17.0 release https://review.opendev.org/c/openstack/ironic/+/780973 | 17:39 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Add prelude for Ironic 17.0. https://review.opendev.org/c/openstack/ironic/+/780974 | 17:39 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Version 17.0 mapping/doc updates https://review.opendev.org/c/openstack/ironic/+/782306 | 17:39 |
openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Add Python3 xena unit tests https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/781846 | 17:41 |
openstackgerrit | Verification of a change to openstack/ironic failed: Fix broken configdrive_use_object_store https://review.opendev.org/c/openstack/ironic/+/777349 | 18:18 |
arne_wiebalck | bye everyone o/ | 18:25 |
*** kaifeng has quit IRC | 18:27 | |
*** dtantsur has joined #openstack-ironic | 18:34 | |
*** dtantsur|afk has quit IRC | 18:41 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Automaticaly set cipher suite https://review.opendev.org/c/openstack/ironic/+/770526 | 18:47 |
openstackgerrit | Arun S A G proposed openstack/ironic master: Add anaconda support in the pxe boot driver https://review.opendev.org/c/openstack/ironic/+/775891 | 18:59 |
openstackgerrit | Arun S A G proposed openstack/ironic master: Validate the kickstart template and file before use https://review.opendev.org/c/openstack/ironic/+/777448 | 18:59 |
openstackgerrit | Arun S A G proposed openstack/ironic master: Add agent_state and agent_status params to heartbeat https://review.opendev.org/c/openstack/ironic/+/778058 | 18:59 |
openstackgerrit | Arun S A G proposed openstack/ironic master: Add support for configdrive in anaconda interface https://review.opendev.org/c/openstack/ironic/+/780398 | 18:59 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Remove extra/vif_port_id https://review.opendev.org/c/openstack/ironic/+/767432 | 19:04 |
JayF | TheJulia: are we expecting review jam to move with DST, or remain the same UTC time? | 19:16 |
TheJulia | with UTC | 19:16 |
JayF | ack ty | 19:16 |
TheJulia | it makes my life less complicated then :) | 19:17 |
JayF | Tues 6pm UTC counting DST becomes ... 11-11:50 | 19:17 |
JayF | oh hey, I put it in the calendar at utc | 19:18 |
JayF | go me | 19:18 |
iurygregory | TheJulia, quick question, for https://review.opendev.org/c/openstack/ironic-inspector/+/770708 the release note can be something like http://paste.openstack.org/show/803792/ ? | 19:21 |
iurygregory | I'm wondering if it should be a feature or just upgrade | 19:21 |
TheJulia | iurygregory: i would say just upgrades is fine | 19:24 |
TheJulia | its not really a feature | 19:24 |
iurygregory | ack I will push o/ | 19:24 |
TheJulia | iurygregory: wait | 19:24 |
iurygregory | ok | 19:24 |
TheJulia | iurygregory: alow the operator to use the `ibmc` driver for | 19:25 |
TheJulia | introspection. "allow" and it is more allowing introspection to try and match the BMC address if no ports are defined. | 19:25 |
iurygregory | "allowing introspection to try and match the BMC address if no ports are defined when using the `ibmc` driver." | 19:28 |
*** Anticime1 is now known as Anticimex | 19:29 | |
TheJulia | Yup, I think that works | 19:30 |
* iurygregory waits for the delay... | 19:31 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Add ibmc_address to ipmi_address_fields https://review.opendev.org/c/openstack/ironic-inspector/+/770708 | 19:31 |
iurygregory | not bad, 14sec | 19:31 |
*** thiagop has joined #openstack-ironic | 19:38 | |
*** uzumaki has quit IRC | 19:53 | |
TheJulia | heh | 20:08 |
tzumainn | TheJulia, I noticed on the ptg topics list, you mention the ceph iscsi driver and note that there's a question of whether it works - is "working" something beyond boot-from-volume? | 20:12 |
TheJulia | tzumainn: it just merged recently, and yeah, in part the question is does it work, does it have the right info, do we need to do something slightly different | 20:13 |
tzumainn | TheJulia, ah, I can say that it worked in testing at the MOC | 20:13 |
TheJulia | so literal start with ceph volume, tell things to attach and magic happens and just works? | 20:14 |
tzumainn | if the volume is defined in cinder, yeah | 20:15 |
TheJulia | \o/ | 20:15 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Add ibmc_address to ipmi_address_fields https://review.opendev.org/c/openstack/ironic-inspector/+/770708 | 20:24 |
*** rcernin has joined #openstack-ironic | 20:25 | |
*** adrianc has quit IRC | 20:31 | |
*** adrianc has joined #openstack-ironic | 20:32 | |
*** ociuhandu has joined #openstack-ironic | 20:37 | |
*** rcernin has quit IRC | 20:38 | |
*** rcernin has joined #openstack-ironic | 21:03 | |
*** gyee has joined #openstack-ironic | 21:04 | |
*** thiagop is now known as outbrito | 21:08 | |
*** ociuhandu has quit IRC | 21:15 | |
*** rcernin has quit IRC | 21:17 | |
*** derekh has quit IRC | 21:32 | |
*** bdodd has quit IRC | 21:35 | |
*** lmcgann has quit IRC | 21:36 | |
*** bdodd has joined #openstack-ironic | 21:38 | |
*** stevebaker has joined #openstack-ironic | 21:42 | |
*** bdodd has quit IRC | 21:48 | |
*** rcernin has joined #openstack-ironic | 21:51 | |
*** rcernin has quit IRC | 21:51 | |
*** bdodd has joined #openstack-ironic | 21:52 | |
*** rcernin has joined #openstack-ironic | 21:52 | |
*** uzumaki has joined #openstack-ironic | 21:57 | |
*** ociuhandu has joined #openstack-ironic | 22:13 | |
*** ociuhandu has quit IRC | 22:14 | |
*** hoonetorg has quit IRC | 22:37 | |
*** rloo has quit IRC | 23:34 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!