| *** rajinir has quit IRC | 00:16 | |
| *** arne_wiebalck has quit IRC | 00:17 | |
| *** nandita has quit IRC | 00:18 | |
| *** arne_wiebalck has joined #openstack-ironic | 00:18 | |
| *** rajinir has joined #openstack-ironic | 00:18 | |
| *** vdrok has quit IRC | 00:18 | |
| *** nandita has joined #openstack-ironic | 00:19 | |
| *** vdrok has joined #openstack-ironic | 00:19 | |
| *** k_mouza has joined #openstack-ironic | 00:37 | |
| *** k_mouza has quit IRC | 00:42 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic master: [WIP] For Supermicro BMCs, set the BootSourceOverrideEnable value when changing boot device https://review.opendev.org/c/openstack/ironic/+/772239 | 01:05 |
|---|---|---|
| *** xinliang has joined #openstack-ironic | 01:50 | |
| *** rcernin has quit IRC | 02:22 | |
| *** zzzeek has quit IRC | 02:28 | |
| *** zzzeek has joined #openstack-ironic | 02:30 | |
| *** rcernin has joined #openstack-ironic | 02:48 | |
| *** mkrai has joined #openstack-ironic | 02:55 | |
| *** xinliang has quit IRC | 03:49 | |
| *** rnoriega has quit IRC | 03:58 | |
| *** rnoriega has joined #openstack-ironic | 04:01 | |
| *** mkrai has quit IRC | 04:49 | |
| *** mkrai_ has joined #openstack-ironic | 04:49 | |
| openstackgerrit | Arun S A G proposed openstack/ironic master: Add kickstart configuration and template https://review.opendev.org/c/openstack/ironic/+/772166 | 05:04 |
| *** rcernin has quit IRC | 05:08 | |
| *** rcernin has joined #openstack-ironic | 05:18 | |
| *** viks____ has joined #openstack-ironic | 05:30 | |
| *** ociuhandu has joined #openstack-ironic | 06:06 | |
| *** ociuhandu has quit IRC | 06:10 | |
| *** k_mouza has joined #openstack-ironic | 06:16 | |
| *** k_mouza has quit IRC | 06:20 | |
| *** k_mouza has joined #openstack-ironic | 06:25 | |
| *** k_mouza has quit IRC | 06:29 | |
| *** nandita has quit IRC | 06:36 | |
| *** mkrai_ has quit IRC | 06:51 | |
| *** ociuhandu has joined #openstack-ironic | 07:03 | |
| *** moshiur has joined #openstack-ironic | 07:05 | |
| *** Qianbiao has joined #openstack-ironic | 07:06 | |
| *** ociuhandu has quit IRC | 07:07 | |
| *** mkrai has joined #openstack-ironic | 07:14 | |
| *** ociuhandu has joined #openstack-ironic | 07:24 | |
| *** ociuhandu has quit IRC | 07:24 | |
| *** ociuhandu has joined #openstack-ironic | 07:25 | |
| *** ociuhandu has quit IRC | 07:30 | |
| arne_wiebalck | Good morning, ironic! | 07:38 |
| *** ociuhandu has joined #openstack-ironic | 07:42 | |
| *** ociuhandu has quit IRC | 07:42 | |
| *** ociuhandu has joined #openstack-ironic | 07:44 | |
| *** rcernin has quit IRC | 07:48 | |
| *** ociuhandu has quit IRC | 07:49 | |
| *** ociuhandu has joined #openstack-ironic | 07:52 | |
| *** ociuhandu has quit IRC | 07:58 | |
| *** k_mouza has joined #openstack-ironic | 07:59 | |
| *** k_mouza has quit IRC | 08:03 | |
| *** tosin has joined #openstack-ironic | 08:06 | |
| *** ociuhandu has joined #openstack-ironic | 08:11 | |
| *** rpittau|afk is now known as rpittau | 08:35 | |
| rpittau | good morning ironic! o/ | 08:35 |
| *** ociuhandu has quit IRC | 08:36 | |
| *** ociuhandu has joined #openstack-ironic | 08:36 | |
| *** ociuhandu has quit IRC | 08:36 | |
| *** ociuhandu has joined #openstack-ironic | 08:38 | |
| *** ociuhandu has quit IRC | 08:42 | |
| *** mkrai has quit IRC | 08:45 | |
| *** lucasagomes has joined #openstack-ironic | 08:56 | |
| *** rcernin has joined #openstack-ironic | 09:04 | |
| *** ociuhandu has joined #openstack-ironic | 09:15 | |
| openstackgerrit | Riccardo Pittau proposed openstack/sushy-tools master: Move some test requirements to tox.ini https://review.opendev.org/c/openstack/sushy-tools/+/771945 | 09:16 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: Reconfigure tox.ini https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/772161 | 09:18 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-tempest-plugin master: Reconfigure tox.ini https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/772162 | 09:19 |
| *** alexmcleod has joined #openstack-ironic | 09:23 | |
| *** tosky has joined #openstack-ironic | 09:23 | |
| *** rcernin has quit IRC | 09:41 | |
| *** derekh has joined #openstack-ironic | 09:45 | |
| *** mkrai has joined #openstack-ironic | 09:46 | |
| *** k_mouza has joined #openstack-ironic | 09:58 | |
| *** rcernin has joined #openstack-ironic | 09:59 | |
| *** rcernin has quit IRC | 10:14 | |
| iurygregory | good morning arne_wiebalck rpittau and Ironic o/ | 10:36 |
| rpittau | hey iurygregory :) | 10:39 |
| *** sshnaidm is now known as sshnaidm|ruck | 10:39 | |
| *** rcernin has joined #openstack-ironic | 10:45 | |
| arne_wiebalck | hey iurygregory o/ | 10:51 |
| *** dougsz has joined #openstack-ironic | 10:52 | |
| *** dougsz has quit IRC | 10:53 | |
| *** dougsz has joined #openstack-ironic | 10:53 | |
| *** dtantsur|afk is now known as dtantsur | 11:11 | |
| * dtantsur rubs his eyes | 11:11 | |
| iurygregory | hehehe | 11:14 |
| *** Qianbiao has quit IRC | 11:27 | |
| *** Qianbiao has joined #openstack-ironic | 11:42 | |
| *** mkrai has quit IRC | 11:56 | |
| *** rcernin has quit IRC | 11:58 | |
| *** ianychoi__ has joined #openstack-ironic | 12:16 | |
| *** gudrutis has quit IRC | 12:17 | |
| *** gudrutis has joined #openstack-ironic | 12:18 | |
| *** ianychoi_ has quit IRC | 12:19 | |
| *** early has quit IRC | 12:19 | |
| *** early has joined #openstack-ironic | 12:20 | |
| *** ociuhandu has quit IRC | 12:48 | |
| *** nam-est has joined #openstack-ironic | 12:50 | |
| *** ociuhandu has joined #openstack-ironic | 12:52 | |
| *** ociuhandu has quit IRC | 12:53 | |
| nam-est | Hi all, | 12:53 |
| nam-est | Please give some reviews to these PRs | 12:53 |
| nam-est | https://github.com/metal3-io/ironic-image/pull/230 | 12:53 |
| nam-est | https://github.com/metal3-io/ironic-inspector-image/pull/70 | 12:53 |
| nam-est | All comments have been addressed, and also the PRs have passed the CI. Thank you. | 12:53 |
| *** ociuhandu has joined #openstack-ironic | 12:53 | |
| *** ociuhandu has quit IRC | 12:58 | |
| tosin | Hi Ironic! I keep getting this error and I'm not sure what it means: | 13:00 |
| tosin | InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings | 13:00 |
| *** Qianbiao has quit IRC | 13:01 | |
| *** anuradha1904 has joined #openstack-ironic | 13:05 | |
| iurygregory | tosin, what command gave you this error? | 13:07 |
| openstackgerrit | sonali bhausaheb borkar proposed x/sushy-oem-idrac master: Add get pxe port macs for bios mode https://review.opendev.org/c/x/sushy-oem-idrac/+/770904 | 13:07 |
| tosin | iurygregory: I pushed a change and got that report from Zuul | 13:10 |
| tosin | Here's the patch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770845/6/plugins/modules/baremetal_node_action.py | 13:10 |
| tosin | Here's the full log | 13:12 |
| tosin | https://zuul.opendev.org/t/openstack/build/4c28582c648d44c8a3b09fd46ccd5233 | 13:12 |
| iurygregory | tosin, looking now | 13:13 |
| tosin | thank you | 13:13 |
| iurygregory | I have the feeling it was a temporary problem in the CI jobs | 13:15 |
| iurygregory | I will put a recheck | 13:15 |
| tosin | oh ok | 13:17 |
| *** rh-jelabarre has joined #openstack-ironic | 13:19 | |
| iurygregory | let's see how it goes =) | 13:19 |
| tosin | =) | 13:21 |
| *** ociuhandu has joined #openstack-ironic | 13:28 | |
| *** jamesdenton has joined #openstack-ironic | 13:33 | |
| *** moshiur has quit IRC | 13:34 | |
| *** ociuhandu has quit IRC | 13:35 | |
| *** moshiur has joined #openstack-ironic | 13:35 | |
| dtantsur | nam-est: at least one comment is still outstanding on the ironic-image patch (and I've added one more) | 13:42 |
| *** Qianbiao has joined #openstack-ironic | 13:42 | |
| iurygregory | tosin, you need to run the ansible linters locally to find the real problem tox -e linters-2.9 and tox -e linters | 13:43 |
| tosin | ok i'll try this | 13:45 |
| iurygregory | dtantsur, please correct if I'm wrong since I never used ansible-collections hehe ^ | 13:45 |
| *** paras333 has joined #openstack-ironic | 13:55 | |
| *** iurygregory_ has joined #openstack-ironic | 13:59 | |
| *** ociuhandu has joined #openstack-ironic | 14:00 | |
| *** iurygregory has quit IRC | 14:00 | |
| *** iurygregory_ is now known as iurygregory | 14:00 | |
| TheJulia | good morning everyone | 14:06 |
| *** rloo has joined #openstack-ironic | 14:10 | |
| iurygregory | good morning TheJulia =) | 14:11 |
| nam-est | dtantsur: Oh, sorry I missed that. I will have a fix soon. Thanks for your comments and also for your approval in ironic-inspector-image repo. | 14:11 |
| *** lbragstad has joined #openstack-ironic | 14:18 | |
| *** ociuhandu has quit IRC | 14:28 | |
| *** ociuhandu has joined #openstack-ironic | 14:29 | |
| dtantsur | nam-est: please also see the email I've just sent to metal3-dev re potentially merging ironic-inspector and ironic image repos | 14:32 |
| *** lbragstad has quit IRC | 14:34 | |
| TheJulia | Does anyone have any review candidates to propose for the week? https://etherpad.opendev.org/p/IronicWhiteBoard line 196 | 14:35 |
| dtantsur | I definitely do, will add in a few | 14:36 |
| TheJulia | ok | 14:36 |
| *** lbragstad has joined #openstack-ironic | 14:37 | |
| *** tzumainn has joined #openstack-ironic | 14:37 | |
| dtantsur | speaking of which, looking for a 2nd +2 on https://review.opendev.org/c/openstack/ironic/+/771989 | 14:38 |
| iurygregory | dtantsur, looking now | 14:42 |
| TheJulia | Just added a +2 from myself | 14:45 |
| *** mkrai has joined #openstack-ironic | 14:52 | |
| TheJulia | Ironic has been holding meetings for approaching *8* years.... *brainexplodes* | 14:56 |
| *** MentalSiege has joined #openstack-ironic | 14:56 | |
| dtantsur | \o/ | 14:56 |
| TheJulia | May 3rd, 2013 is when the meetings started | 14:56 |
| *** kaifeng has joined #openstack-ironic | 14:57 | |
| dtantsur | and I'll celebrate 7 years around OpenStack next week | 14:57 |
| TheJulia | \o/ | 14:57 |
| TheJulia | #startmeeting ironic | 15:00 |
| TheJulia | o/ | 15:00 |
| openstack | Meeting started Mon Jan 25 15:00:08 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 |
| 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 |
| dtantsur | o/ | 15:00 |
| rpittau | o/ | 15:00 |
| rpioso | \o | 15:00 |
| kaifeng | o/ | 15:00 |
| Qianbiao | o/ | 15:00 |
| ajya | o/ | 15:00 |
| bdodd | o/ | 15:00 |
| TheJulia | Good morning everyone, our weekly meeting agenda is on the wiki at https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:00 |
| TheJulia | I'm hoping our meeting today goes quickly since today is the first of three scheduled "Mid-Cycle" calls | 15:01 |
| TheJulia | As well. | 15:01 |
| TheJulia | #topic Announcements / Reminders | 15:01 |
| *** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)" | 15:01 | |
| TheJulia | #Info Midcycle the next three days | 15:01 |
| TheJulia | #link https://etherpad.opendev.org/p/ironic-wallaby-midcycle | 15:01 |
| *** stendulker has joined #openstack-ironic | 15:01 | |
| TheJulia | The first call starts in about two hours. All are welcome to bring topics as long as we respect people's time. | 15:01 |
| iurygregory | o/ | 15:02 |
| stendulker | o/ | 15:02 |
| TheJulia | #info this week is R-11 in the OpenStack Wallaby release schedule. R-6 is non-client library freeze. R-5 is client library and requirements freeze. | 15:02 |
| dtantsur | how long till the next releases? 2 weeks? | 15:03 |
| TheJulia | #info The Ironic team's second sprint is scheduled to end February 8th with our second release of the development cycle. | 15:03 |
| TheJulia | well, week of February 8th, since CI | 15:03 |
| TheJulia | Does anyone have anything they would like to announce or remind us of this week? | 15:03 |
| TheJulia | I have one item, I have some electrical work being done on my home this week. I'm supposed to get a heads up when they will cut power but I guess we all know how these sorts of things go. | 15:04 |
| iurygregory | good luck =) | 15:04 |
| dtantsur | heh | 15:05 |
| TheJulia | Thanks | 15:05 |
| TheJulia | No announcements or reminders? | 15:05 |
| rloo | o/ | 15:05 |
| TheJulia | dtantsur: should I remind myself to announce you've been in the community for 7 years week? | 15:05 |
| dtantsur | why not :) | 15:06 |
| TheJulia | \o/ | 15:06 |
| TheJulia | Will do | 15:06 |
| TheJulia | Anyway, we have no noted action items from last week. I guess we shall proceed on subteam status reports. | 15:06 |
| TheJulia | #topic Review subteam status reports | 15:06 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:06 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:06 |
| *** lmcgann has joined #openstack-ironic | 15:07 | |
| TheJulia | Starting at line 266 of that etehrpad | 15:07 |
| TheJulia | Etherpad | 15:07 |
| TheJulia | \o/ looks like we can review redfish raid | 15:07 |
| dtantsur | yeah, this is great news | 15:08 |
| bdodd | \o/ | 15:09 |
| * iurygregory brb - I've updated status for privsep | 15:10 | |
| dtantsur | bdodd: the patch doesn't change the sushy requirement, is it correct? | 15:10 |
| TheJulia | iurygregory: thanks | 15:10 |
| *** MentalSiege has quit IRC | 15:10 | |
| bdodd | dtantsur Actually, it does require a very recent sushy change. | 15:11 |
| * dtantsur reminds himself to remind everyone about FOSDEM next time | 15:11 | |
| TheJulia | I've updated the community goal json to yaml notes, also added a note to try and delineate it from RBAC which is related, but a whole larger thing. The policy json to yaml is literally one patch for ironic, and one patch for ironic-inspector. | 15:11 |
| dtantsur | bdodd: than please update requirements.txt | 15:11 |
| TheJulia | ++ | 15:12 |
| TheJulia | ajya: Looks like you've been busy on configuration molds. Can you get together a single list of patches needing to be reviewed since it looks like your touching multiple repositories. | 15:13 |
| ajya | TheJulia: will post a link | 15:13 |
| bdodd | dtantsur Is there a new sushy release version that includes https://review.opendev.org/c/openstack/sushy/+/758874 ? I haven't been able to tell. | 15:13 |
| dtantsur | bdodd: yes, 3.6.0 | 15:13 |
| dtantsur | released last week | 15:13 |
| ajya | TheJulia: posted in WB | 15:13 |
| bdodd | Great, thanks! I'll update requirements.txt | 15:14 |
| dtantsur | thx | 15:14 |
| TheJulia | ajya: Thanks | 15:14 |
| *** sdanni has joined #openstack-ironic | 15:14 | |
| TheJulia | Looks like Arun also posted an anaconda patch over the weekend | 15:14 |
| TheJulia | \o/ | 15:14 |
| dtantsur | much progress. wow! | 15:15 |
| TheJulia | kaifeng: still working on snapshot support? | 15:15 |
| * dtantsur has a feeling that node history is something metal3 could benefit from | 15:15 | |
| TheJulia | ++ | 15:15 |
| kaifeng | no op on the snapshot, but squeezing sometime for the node history :) | 15:16 |
| TheJulia | sounds good to me :) | 15:16 |
| TheJulia | Looks like the security interface needs reviews as well. | 15:17 |
| arne_wiebalck | o/ | 15:18 |
| TheJulia | Everyone good to proceed to priorities for the week? | 15:18 |
| dtantsur | yeah | 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 | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:19 |
| TheJulia | Starting at line 128 | 15:19 |
| * TheJulia removes merged items | 15:19 | |
| TheJulia | I'm going to drop the minor ironic-lib improvement I've been pondering | 15:21 |
| TheJulia | No time right now to update it | 15:21 |
| TheJulia | Just a couple items to add it looks like | 15:22 |
| TheJulia | Sounds good to me, any objections? | 15:23 |
| arne_wiebalck | shall I add the GPT tiny partitions issue to ironic-lib? | 15:23 |
| TheJulia | arne_wiebalck: oh yes, please | 15:23 |
| arne_wiebalck | I need to add some tests first | 15:23 |
| arne_wiebalck | TheJulia: ok | 15:23 |
| *** ociuhandu has quit IRC | 15:23 | |
| *** Qianbiao has quit IRC | 15:23 | |
| TheJulia | Line 174/175 | 15:25 |
| TheJulia | That has been hanging out for a while and it seems like it is getting zero traction to the question | 15:25 |
| * TheJulia hears crickets | 15:27 | |
| TheJulia | arne_wiebalck: anything for the baremetal sig this week?? | 15:27 |
| TheJulia | Otherwise I think Open Discussion is next | 15:27 |
| arne_wiebalck | TheJulia: nothing for the SIG | 15:28 |
| TheJulia | #topic Open Discussion | 15:28 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:28 | |
| TheJulia | Well, I hope everyone had a wonderful weekend, but that can also be a topic of discussion during the midcycle call for those who plan on attending | 15:29 |
| rloo | crickets until midcycle... | 15:30 |
| TheJulia | I guess so.... | 15:31 |
| TheJulia | Well, Thanks everyone! | 15:31 |
| arne_wiebalck | TheJulia: Thanks! | 15:31 |
| TheJulia | Talk to you all soon | 15:31 |
| TheJulia | #endmeeting | 15:31 |
| *** 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:31 | |
| openstack | Meeting ended Mon Jan 25 15:31:32 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:31 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-01-25-15.00.html | 15:31 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-01-25-15.00.txt | 15:31 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-01-25-15.00.log.html | 15:31 |
| *** mkrai has quit IRC | 15:36 | |
| *** ociuhandu has joined #openstack-ironic | 15:36 | |
| openstackgerrit | Merged openstack/networking-baremetal master: Add doc/requirements https://review.opendev.org/c/openstack/networking-baremetal/+/769172 | 15:41 |
| nam-est | dtansur: Yes, I have got your email. I am discussing it with other people in EST team and we will share our thoughts soon in the mailing list or in the next community meeting. | 16:01 |
| *** jamesdenton has quit IRC | 16:03 | |
| *** jamesdenton has joined #openstack-ironic | 16:03 | |
| *** moshiur has quit IRC | 16:11 | |
| *** stendulker has quit IRC | 16:18 | |
| dtantsur | TheJulia: FYI I've split the event framework into a non-proxying part https://storyboard.openstack.org/#!/story/2008366 and a proxy part https://storyboard.openstack.org/#!/story/2008555 | 16:44 |
| dtantsur | the former is something we need for metal3, the latter is cool to have for ironic, but I don't know if any of us will ever have time for it | 16:44 |
| dtantsur | (I've removed needs-spec for a new discussion) | 16:45 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add centralized secure boot documentation https://review.opendev.org/c/openstack/ironic/+/772376 | 16:49 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Redfish secure boot https://review.opendev.org/c/openstack/ironic/+/771493 | 16:50 |
| openstackgerrit | Riccardo Pittau proposed openstack/sushy stable/victoria: Raise exception when async operation fails https://review.opendev.org/c/openstack/sushy/+/772397 | 16:50 |
| *** ociuhandu_ has joined #openstack-ironic | 16:51 | |
| *** ociuhandu has quit IRC | 16:55 | |
| *** ociuhandu_ has quit IRC | 16:56 | |
| *** ociuhandu has joined #openstack-ironic | 17:10 | |
| *** ociuhandu has quit IRC | 17:15 | |
| *** lucasagomes has quit IRC | 17:17 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: redfish-virtual-media: allow a link to raw configdrive image https://review.opendev.org/c/openstack/ironic/+/771989 | 17:20 |
| *** dougsz has quit IRC | 17:22 | |
| openstackgerrit | Bill Dodd proposed openstack/ironic master: Add Redfish RAID management to Ironic https://review.opendev.org/c/openstack/ironic/+/730000 | 17:33 |
| *** MentalSiege has joined #openstack-ironic | 17:43 | |
| *** kaifeng has quit IRC | 17:46 | |
| *** jdandrea has quit IRC | 17:52 | |
| *** jdandrea has joined #openstack-ironic | 17:54 | |
| TheJulia | dtantsur: ack, thanks | 17:56 |
| TheJulia | dtantsur: bringing up on meeting next week? | 17:56 |
| dtantsur | yeah | 17:57 |
| TheJulia | sounds good | 17:57 |
| dtantsur | rloo: recollected the context and posting a comment on the deploy-failed-to-active spec | 17:57 |
| rloo | thx dtantsur, will look ... after lunch :) | 17:58 |
| dtantsur | enjoy! | 17:58 |
| *** derekh has quit IRC | 17:59 | |
| openstackgerrit | Arne Wiebalck proposed openstack/ironic-lib master: Handle EBRs and tiny partitions when removing metadata https://review.opendev.org/c/openstack/ironic-lib/+/771788 | 18:00 |
| * TheJulia looks at her empty coffee mug and thinks it is time for a refill | 18:01 | |
| arne_wiebalck | bye everyone o/ | 18:03 |
| rpittau | good night! o/ | 18:10 |
| *** rpittau is now known as rpittau|afk | 18:10 | |
| iurygregory | any core has some time to give a second +2 in https://review.opendev.org/c/openstack/sushy/+/772397 ? =D | 18:12 |
| *** rnoriega has quit IRC | 18:19 | |
| dtantsur | done | 18:20 |
| dtantsur | I guess we need a release after it merges | 18:20 |
| *** nam-est has quit IRC | 18:20 | |
| *** rnoriega has joined #openstack-ironic | 18:21 | |
| *** gyee has joined #openstack-ironic | 18:21 | |
| iurygregory | dtantsur, yeah I will push one for victoria | 18:23 |
| *** dsneddon has joined #openstack-ironic | 18:40 | |
| *** irclogbot_1 has quit IRC | 18:46 | |
| *** mgoddard has quit IRC | 18:48 | |
| *** irclogbot_2 has joined #openstack-ironic | 18:49 | |
| *** dtantsur is now known as dtantsur|afk | 18:50 | |
| dtantsur|afk | o/ | 18:50 |
| *** mgoddard has joined #openstack-ironic | 18:50 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: redfish-virtual-media: allow a link to raw configdrive image https://review.opendev.org/c/openstack/ironic/+/771989 | 18:53 |
| *** mgoddard has quit IRC | 18:58 | |
| stevebaker | morning | 19:24 |
| TheJulia | Good morning | 19:25 |
| openstackgerrit | Merged openstack/sushy stable/victoria: Raise exception when async operation fails https://review.opendev.org/c/openstack/sushy/+/772397 | 19:25 |
| iurygregory | do we also want to update ACL for tenks? just wondering | 19:33 |
| *** mgoddard has joined #openstack-ironic | 19:35 | |
| TheJulia | mgoddard: should iury also update the acl for tenks? | 19:36 |
| iurygregory | lol mgoddard appeared after my message hehe | 19:40 |
| iurygregory | *magic* | 19:40 |
| TheJulia | It is magic! The magic of IRC! | 19:42 |
| TheJulia | Queue mystical music | 19:42 |
| openstackgerrit | Merged openstack/virtualbmc master: Update minversion of tox https://review.opendev.org/c/openstack/virtualbmc/+/771955 | 19:43 |
| iurygregory | yeah | 19:43 |
| *** k_mouza has quit IRC | 19:55 | |
| iurygregory | TheJulia, fyi https://review.opendev.org/c/openstack/project-config/+/772427 let's see what zuul will say | 20:00 |
| TheJulia | iurygregory: ack | 20:03 |
| * TheJulia comes to the realization that there is no "clean/easy" way to test major default policy changes | 20:04 | |
| stevebaker | hmm | 20:22 |
| *** anuradha1904 has quit IRC | 20:25 | |
| TheJulia | stevebaker: I'm going to upload a wip, and I think you'll get the idea of what I'm trying to do. Bottom line is we create lists upon first interpretation of policy.py | 20:27 |
| TheJulia | because they are constants which can be overridden by config | 20:27 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: support specific project id for legacy mode https://review.opendev.org/c/openstack/ironic/+/772438 | 20:54 |
| * TheJulia now tries to remember all of the fun test names she was thinking of at 5 AM on saturday morning | 20:59 | |
| *** ociuhandu has joined #openstack-ironic | 21:20 | |
| openstackgerrit | Merged openstack/ironic master: redfish-virtual-media: allow a link to raw configdrive image https://review.opendev.org/c/openstack/ironic/+/771989 | 21:22 |
| *** tosky has quit IRC | 21:35 | |
| *** tosky has joined #openstack-ironic | 21:36 | |
| *** TheJulia has quit IRC | 21:36 | |
| *** tosin has quit IRC | 21:37 | |
| *** TheJulia has joined #openstack-ironic | 21:37 | |
| *** rpittau|afk has quit IRC | 21:37 | |
| *** johnsom has quit IRC | 21:37 | |
| *** rpittau|afk_ has joined #openstack-ironic | 21:37 | |
| *** buhman has quit IRC | 21:37 | |
| *** johnsom has joined #openstack-ironic | 21:37 | |
| *** buhman_ has joined #openstack-ironic | 21:37 | |
| *** NobodyCam_ has joined #openstack-ironic | 21:38 | |
| *** vdrok has quit IRC | 21:38 | |
| *** NobodyCam has quit IRC | 21:38 | |
| *** NobodyCam_ is now known as NobodyCam | 21:38 | |
| *** viks____ has quit IRC | 21:38 | |
| *** tosin has joined #openstack-ironic | 21:38 | |
| *** k_mouza has joined #openstack-ironic | 21:39 | |
| *** vdrok has joined #openstack-ironic | 21:40 | |
| *** k_mouza has quit IRC | 21:44 | |
| *** rh-jelabarre has quit IRC | 22:08 | |
| TheJulia | stevebaker: so for project, do you think it would be fine to skip some of the redundant checks. say admin's can read a node when the permission is the reader? Trying to think of ways to model it mentally to keep things from getting too out of hand | 22:17 |
| stevebaker | TheJulia: for a start how about skipping endpoints which share the same policy rule? | 22:18 |
| *** rcernin has joined #openstack-ironic | 22:18 | |
| TheJulia | I was kind of thinking the same, where instead we change the data being tested, headers translating through properly, etc | 22:18 |
| TheJulia | let me paste an example of the direction I'm heading | 22:19 |
| stevebaker | as long as there is one allow and one deny test for each endpoint, the rest of the tests can be targetting the policy itself on a single endpoint | 22:19 |
| TheJulia | stevebaker: http://paste.openstack.org/show/B3kDotPNGJDyALQkmP9Y/ | 22:20 |
| stevebaker | TheJulia: then for policies which are allowed for readers, do we really need to test against member and admin? How about just a test with a reader and a no-roles? We have a commitment from keystone that users with admin or member role will explicitly have the reader role too | 22:23 |
| TheJulia | I was kind of thinking in that same vain | 22:24 |
| stevebaker | those 2 approaches might save a few hundred tests :) | 22:25 |
| TheJulia | yeah, I'm basically thinking that, but we also need to test things like "oh, well they are just going to get no data back" or whatever | 22:25 |
| TheJulia | "third party" | 22:26 |
| TheJulia | my third party admin is a admin,member,reader :) | 22:27 |
| stevebaker | they are third party because its in a different project? | 22:28 |
| TheJulia | exactly | 22:31 |
| TheJulia | lbragstad: does it make sense to create some randomly named freeform role and just hit endpoints with it? | 22:31 |
| lbragstad | just to make sure the policy is loading the roles we expect? | 22:32 |
| TheJulia | Yeah, I guess | 22:32 |
| lbragstad | you could - i don't think we take that approach in keystone (not implying you need to follow what we do) | 22:32 |
| TheJulia | I guess I'm more wondering if there is a larger scope of concern, but I guess not. | 22:33 |
| lbragstad | i think it would just test that nothing in ironic's middleware, api, or policy enforcement is overloading roles with something else | 22:33 |
| TheJulia | I can't think of anything in there, so I guess that is just a mater of over thinking it | 22:34 |
| *** ociuhandu has quit IRC | 22:34 | |
| *** jdandrea has quit IRC | 22:34 | |
| *** ociuhandu has joined #openstack-ironic | 22:35 | |
| lbragstad | i can see it if you wanted to test policy overrides, in which case you'd need to add a new role (call it foo) and then muck with the policy to make sure you override it so that foo works | 22:35 |
| *** lmcgann has quit IRC | 22:35 | |
| lbragstad | but that may be out of scope for this (technically that should already work) | 22:36 |
| stevebaker | TheJulia: just a handful of third-party-denied tests should be enough IMO | 22:36 |
| TheJulia | lbragstad: ack | 22:37 |
| TheJulia | stevebaker: agreed, additionally many of the endpoints will remain system scoped... possibly forever | 22:37 |
| TheJulia | since they don't *need* to be exposed as they are purely admin only | 22:38 |
| stevebaker | yep | 22:38 |
| * TheJulia receives the explicit "it is cold outside, make something warm for dinner" request. | 22:38 | |
| *** ociuhandu has quit IRC | 22:39 | |
| TheJulia | it is like 40F out and one of the two nearby mountains is masked by rain/snow | 22:40 |
| *** tosin has quit IRC | 22:43 | |
| *** ociuhandu has joined #openstack-ironic | 22:44 | |
| *** ociuhandu has quit IRC | 22:48 | |
| *** ociuhandu has joined #openstack-ironic | 22:48 | |
| TheJulia | Field patching wise, we're likely going to need to expand upon such testing :\ | 22:50 |
| *** ociuhandu has quit IRC | 22:53 | |
| *** uzumaki has joined #openstack-ironic | 22:53 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: RBAC System Scope: observer -> reader https://review.opendev.org/c/openstack/ironic/+/772450 | 22:56 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: Project scoped tests https://review.opendev.org/c/openstack/ironic/+/772451 | 22:56 |
| TheJulia | stevebaker: it is a start ^ :) | 22:57 |
| lbragstad | \o/ | 22:58 |
| TheJulia | lbragstad: well, there is a lot from the system scope tests in that file | 22:59 |
| TheJulia | and it is not actually tested... and there are just notes of "hey, we should test this" | 22:59 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: Project scoped tests https://review.opendev.org/c/openstack/ironic/+/772451 | 23:11 |
| TheJulia | a little more in the way of notes. I'm going to go start dinner, brain demanding it. | 23:12 |
| TheJulia | Also, I'm going to have to run down that entire list with the spec in front of me tomorrow | 23:13 |
| stevebaker | TheJulia: thanks, I'll take a look | 23:30 |
| *** jdandrea has joined #openstack-ironic | 23:35 | |
| * TheJulia goes and exercises | 23:37 | |
| *** jdandrea has quit IRC | 23:39 | |
| *** rcernin has quit IRC | 23:40 | |
| *** tosky has quit IRC | 23:47 | |
| *** k_mouza has joined #openstack-ironic | 23:56 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!