*** tosky has quit IRC | 00:10 | |
*** xinliang has joined #openstack-ironic | 01:11 | |
*** zzzeek has quit IRC | 01:15 | |
*** zzzeek has joined #openstack-ironic | 01:17 | |
*** ricolin_ has joined #openstack-ironic | 01:21 | |
*** ricolin_ has quit IRC | 02:11 | |
*** ricolin_ has joined #openstack-ironic | 02:11 | |
*** rcernin has quit IRC | 02:37 | |
*** rh-jelabarre has quit IRC | 03:01 | |
*** rloo has quit IRC | 03:17 | |
*** mkrai has joined #openstack-ironic | 03:22 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Reduce list_all policy rules to SYSTEM_ADMIN scope https://review.opendev.org/c/openstack/ironic/+/777291 | 03:47 |
---|---|---|
*** xinliang has quit IRC | 04:01 | |
*** bdodd has quit IRC | 04:29 | |
*** rcernin has joined #openstack-ironic | 04:44 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Expose owner and lessee in baremetal volume policy https://review.opendev.org/c/openstack/ironic/+/767631 | 04:50 |
*** rcernin has quit IRC | 04:54 | |
*** rcernin has joined #openstack-ironic | 04:55 | |
*** tzumainn has quit IRC | 04:55 | |
*** rcernin has quit IRC | 05:14 | |
*** rcernin has joined #openstack-ironic | 05:21 | |
*** gyee has quit IRC | 05:34 | |
*** ricolin has quit IRC | 06:00 | |
*** ricolin_ has quit IRC | 06:27 | |
*** ricolin has joined #openstack-ironic | 06:28 | |
*** uzumaki has joined #openstack-ironic | 06:44 | |
*** mkrai has quit IRC | 06:47 | |
*** sshnaidm|afk is now known as sshnaidm | 07:38 | |
*** rcernin has quit IRC | 07:39 | |
arne_wiebalck | Good morning, ironic! | 07:51 |
janders | good morning arne_wiebalck o/ | 07:59 |
arne_wiebalck | hey janders o/ | 07:59 |
*** rpittau|afk is now known as rpittau | 08:11 | |
rpittau | good morning ironic! o/ | 08:11 |
*** rcernin has joined #openstack-ironic | 08:15 | |
janders | good morning rpittau o/ | 08:17 |
rpittau | hey janders :) | 08:17 |
*** fdegir has joined #openstack-ironic | 08:18 | |
*** ociuhandu has joined #openstack-ironic | 08:20 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-lib master: [DNM] test unprivileged qemu convert https://review.opendev.org/c/openstack/ironic-lib/+/777312 | 08:21 |
*** ociuhandu has quit IRC | 08:31 | |
*** rcernin has quit IRC | 08:31 | |
*** rcernin has joined #openstack-ironic | 08:33 | |
rpittau | if anyone has a minute please check https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776642 thanks! | 08:35 |
*** ociuhandu has joined #openstack-ironic | 08:36 | |
rpittau | iurygregory: I did some tests on the privsep change and I hopefully found the issue, there are a couple of things we may want to discuss before moving forward, but at the moment I'm testing here https://review.opendev.org/c/openstack/ironic-lib/+/777312 | 08:37 |
*** mkowalski_ has quit IRC | 08:39 | |
*** mkowalski_ has joined #openstack-ironic | 08:40 | |
*** anuradha1904 has joined #openstack-ironic | 08:46 | |
rpittau | mmm grenade job seems broken -> http://paste.openstack.org/show/802952/ | 08:55 |
*** lucasagomes has joined #openstack-ironic | 09:07 | |
*** rcernin has quit IRC | 09:08 | |
*** ociuhandu has quit IRC | 09:09 | |
*** uzumaki has quit IRC | 09:13 | |
*** hoonetorg has joined #openstack-ironic | 09:16 | |
*** dougsz has joined #openstack-ironic | 09:17 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Prepare to use tinycore 12 for tinyipa https://review.opendev.org/c/openstack/ironic/+/777342 | 09:18 |
*** tosky has joined #openstack-ironic | 09:18 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Use tinycore 12 to build tinyipa https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776587 | 09:18 |
*** dougsz has quit IRC | 09:23 | |
*** derekh has joined #openstack-ironic | 09:36 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [trivial] Remove default parameter from execute https://review.opendev.org/c/openstack/ironic/+/777348 | 09:38 |
*** ociuhandu has joined #openstack-ironic | 09:40 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Use tinycore 12 to build tinyipa https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776587 | 09:40 |
*** ociuhandu has quit IRC | 09:49 | |
*** ociuhandu has joined #openstack-ironic | 09:52 | |
*** ociuhandu has quit IRC | 09:53 | |
*** ociuhandu has joined #openstack-ironic | 09:53 | |
*** ociuhandu has quit IRC | 09:58 | |
*** ociuhandu has joined #openstack-ironic | 10:00 | |
*** ociuhandu has quit IRC | 10:00 | |
*** ociuhandu has joined #openstack-ironic | 10:01 | |
*** ociuhandu has quit IRC | 10:03 | |
*** ociuhandu has joined #openstack-ironic | 10:03 | |
*** dtantsur|afk is now known as dtantsur | 10:07 | |
dtantsur | morning ironic | 10:07 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Fix broken configdrive_use_object_store https://review.opendev.org/c/openstack/ironic/+/777349 | 10:09 |
dtantsur | folks, we need to release ironic-lib next week, and I'll be out since Friday. could you review https://review.opendev.org/c/openstack/ironic-lib/+/774354 before then? | 10:10 |
dtantsur | it's a lot of code, but it's copy-pasted from ironic with minimal changes | 10:10 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-lib master: [DNM] test unprivileged qemu convert https://review.opendev.org/c/openstack/ironic-lib/+/777312 | 10:15 |
*** dougsz has joined #openstack-ironic | 10:15 | |
rpittau | dtantsur: I'm reviewing that now | 10:15 |
rpittau | dtantsur: for the release, I can take care of them, we have also sushy and metalsmith if I remember right | 10:18 |
dtantsur | rpittau: correct. metalsmith has already been released, I don't think we've had any changes since then (but worth double-checking) | 10:19 |
rpittau | oh right, yeah, I checked already for metalsmith and there are not open patches | 10:21 |
*** rcernin has joined #openstack-ironic | 10:27 | |
*** dougsz has quit IRC | 10:39 | |
*** rcernin has quit IRC | 10:42 | |
janders | good morning dtantsur o/ | 10:45 |
*** k_mouza has joined #openstack-ironic | 10:48 | |
*** rcernin has joined #openstack-ironic | 10:52 | |
*** k_mouza has quit IRC | 10:55 | |
*** k_mouza has joined #openstack-ironic | 10:56 | |
*** tosin has joined #openstack-ironic | 10:56 | |
*** k_mouza_ has joined #openstack-ironic | 11:03 | |
*** k_mouza has quit IRC | 11:06 | |
*** anuradha1904 has quit IRC | 11:06 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch iLO and iRMC to the new secure boot framework https://review.opendev.org/c/openstack/ironic/+/770122 | 11:06 |
*** rcernin has quit IRC | 11:12 | |
*** ociuhandu has quit IRC | 11:19 | |
*** ociuhandu has joined #openstack-ironic | 11:20 | |
*** ociuhandu has quit IRC | 11:25 | |
*** ociuhandu has joined #openstack-ironic | 11:26 | |
*** ociuhandu has quit IRC | 11:30 | |
*** ociuhandu has joined #openstack-ironic | 11:34 | |
iurygregory | good morning dtantsur rpittau arne_wiebalck janders and Ironic! | 11:34 |
janders | good morning iurygregory o/ | 11:35 |
iurygregory | rpittau, tks! we can discuss about it in a few o/ | 11:35 |
rpittau | Hey iurygregory :) | 11:35 |
openstackgerrit | Merged openstack/sushy stable/victoria: Fixes issue of redfish firmware update https://review.opendev.org/c/openstack/sushy/+/771604 | 11:36 |
rpittau | iurygregory: no problem, I have a couple of.meetings lined up but I will have time at the end of my day | 11:38 |
iurygregory | rpittau, ack | 11:39 |
*** ociuhandu has quit IRC | 11:41 | |
*** k_mouza_ has quit IRC | 11:45 | |
*** k_mouza has joined #openstack-ironic | 11:46 | |
openstackgerrit | Merged openstack/sushy master: Refactor TaskMonitor and update Volume methods https://review.opendev.org/c/openstack/sushy/+/774532 | 11:48 |
*** ociuhandu has joined #openstack-ironic | 11:58 | |
*** akahat has quit IRC | 12:05 | |
*** akahat has joined #openstack-ironic | 12:05 | |
*** k_mouza has quit IRC | 12:08 | |
*** k_mouza has joined #openstack-ironic | 12:21 | |
*** ociuhandu has quit IRC | 12:24 | |
*** ociuhandu has joined #openstack-ironic | 12:24 | |
*** ociuhandu has quit IRC | 12:29 | |
dtantsur | I think I know what is wrong with grenade | 12:34 |
rpittau | the only thing I found is that I'm not sure we're actually setting the temp ke | 12:35 |
rpittau | key* | 12:35 |
dtantsur | yep | 12:35 |
iurygregory | yay | 12:35 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Enable swift temporary URLs in grenade and provide a good error message https://review.opendev.org/c/openstack/ironic/+/777375 | 12:35 |
dtantsur | we're not, devstack should, but ^^^ | 12:35 |
rpittau | oh yeah | 12:36 |
rpittau | dtantsur: isn't going to fail before reaching that ? | 12:37 |
rpittau | I mean, if the key is not set | 12:37 |
openstackgerrit | Verification of a change to openstack/ironic failed: Implement system scoped RBAC for the event APIs https://review.opendev.org/c/openstack/ironic/+/763276 | 12:42 |
arne_wiebalck | hey iurygregory o/ | 12:50 |
dtantsur | rpittau: not sure I get the question, currently it fails there | 12:59 |
*** dougsz has joined #openstack-ironic | 13:15 | |
*** rh-jelabarre has joined #openstack-ironic | 13:20 | |
rpittau | dtantsur: sorry, let me explain better, it seems like it fails when trying to get x-account-meta-temp-url-key on L134, it doesn't go to L135, so maybe we can wrap that in try-except ? | 13:23 |
TheJulia | good morning | 13:23 |
rpittau | good morning TheJulia :) | 13:23 |
openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Implement secure RBAC https://review.opendev.org/c/openstack/ironic-inspector/+/772771 | 13:26 |
iurygregory | good morning TheJulia =) | 13:27 |
*** dougsz has quit IRC | 13:31 | |
*** k_mouza has quit IRC | 13:31 | |
*** k_mouza has joined #openstack-ironic | 13:32 | |
dtantsur | TheJulia: good morning | 13:34 |
*** pmannidi has quit IRC | 13:34 | |
dtantsur | rpittau: get returns None if the key is not set, you're confusing it with pop likely | 13:34 |
*** lmcgann has joined #openstack-ironic | 13:34 | |
*** dtantsur is now known as dtantsur|brb | 13:35 | |
rpittau | dtantsur|brb: true, I got confused from the exception in the logs | 13:36 |
*** pmannidi has joined #openstack-ironic | 13:37 | |
*** dougsz has joined #openstack-ironic | 13:38 | |
openstackgerrit | Merged openstack/ironic master: [doc-only] Add BFV basic flow and networking context https://review.opendev.org/c/openstack/ironic/+/762873 | 13:39 |
* TheJulia awaits the sacred cup of coffee | 13:40 | |
*** ociuhandu has joined #openstack-ironic | 13:42 | |
mgoddard | Hi o/ | 13:42 |
mgoddard | We're hitting some kayobe CI unreliability in our bare metal deploy tests | 13:43 |
mgoddard | limited data, but it seems to happen on vexxhost | 13:43 |
*** dougsz has quit IRC | 13:43 | |
mgoddard | is this something that has come up before? | 13:43 |
TheJulia | instability on vexxhost, not that I'm aware of | 13:44 |
rpittau | not that I remember | 13:44 |
TheJulia | Aside from the standard mirror issues | 13:44 |
mgoddard | this almost looks like entropy issues - the VM consoles are at different points in the boot process | 13:45 |
mgoddard | although they could be hung | 13:45 |
mgoddard | hopefully we'll work it out, and will share if relevant | 13:46 |
openstackgerrit | Verification of a change to openstack/ironic failed: Review feedback follow-up on Node System Scoped RBAC https://review.opendev.org/c/openstack/ironic/+/777016 | 13:47 |
TheJulia | mgoddard: I tend to always think of memory first since there is 8gb of ram and if you hit swap the job is likely to fail. | 13:48 |
mgoddard | TheJulia: yeah, although vexxhost kindly provides 32G | 13:48 |
TheJulia | :( | 13:48 |
TheJulia | sadness | 13:48 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-lib master: [DNM] test unprivileged qemu convert https://review.opendev.org/c/openstack/ironic-lib/+/777312 | 13:57 |
*** rloo has joined #openstack-ironic | 14:00 | |
*** bdodd has joined #openstack-ironic | 14:01 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Prepare to use tinycore 12 for tinyipa https://review.opendev.org/c/openstack/ironic/+/777342 | 14:01 |
openstackgerrit | Verification of a change to openstack/ironic failed: Implement system scoped RBAC for the event APIs https://review.opendev.org/c/openstack/ironic/+/763276 | 14:03 |
rpittau | see ya tomorrow o/ | 14:04 |
*** rpittau is now known as rpittau|afk | 14:04 | |
TheJulia | gah, ci | 14:06 |
TheJulia | This... feels... weird. I don't thinik I really ahve any big meetings today | 14:07 |
TheJulia | well, corgi objecting to something is always a thing | 14:07 |
TheJulia | The corgi has like a million times my energy this morning | 14:21 |
TheJulia | It is frankly, disturbing. | 14:21 |
*** ociuhandu has quit IRC | 14:29 | |
*** dhellmann has quit IRC | 14:30 | |
*** ociuhandu has joined #openstack-ironic | 14:30 | |
*** dhellmann has joined #openstack-ironic | 14:31 | |
*** ociuhandu has quit IRC | 14:34 | |
*** juanoterocas has joined #openstack-ironic | 14:41 | |
*** ociuhandu has joined #openstack-ironic | 14:44 | |
*** tzumainn has joined #openstack-ironic | 14:46 | |
*** dsneddon has quit IRC | 14:52 | |
*** dsneddon has joined #openstack-ironic | 14:54 | |
*** tosin has quit IRC | 15:04 | |
*** dsneddon has quit IRC | 15:06 | |
*** dsneddon has joined #openstack-ironic | 15:09 | |
*** dhellmann has quit IRC | 15:12 | |
*** dtantsur|brb is now known as dtantsur | 15:18 | |
TheJulia | + | 15:22 |
*** dhellmann has joined #openstack-ironic | 15:22 | |
TheJulia | today feels like a "should have stayed in bed" day | 15:25 |
*** outbrito has quit IRC | 15:29 | |
*** tosin has joined #openstack-ironic | 15:44 | |
dtantsur | TheJulia: could you check https://review.opendev.org/c/openstack/ironic/+/777375 before going to bed? it fixes the CI | 15:53 |
TheJulia | done | 15:56 |
dtantsur | thx! | 15:56 |
* TheJulia goes back to unit tests | 15:57 | |
*** dougsz has joined #openstack-ironic | 16:03 | |
*** ociuhandu has quit IRC | 16:09 | |
*** ociuhandu has joined #openstack-ironic | 16:10 | |
*** k_mouza_ has joined #openstack-ironic | 16:14 | |
*** ociuhandu has quit IRC | 16:14 | |
*** k_mouza has quit IRC | 16:17 | |
*** juanoterocas has quit IRC | 16:19 | |
*** dougsz has quit IRC | 16:25 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/victoria: Enable swift temporary URLs in grenade and provide a good error message https://review.opendev.org/c/openstack/ironic/+/777429 | 16:29 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Fix broken configdrive_use_object_store https://review.opendev.org/c/openstack/ironic/+/777349 | 16:30 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Enable swift temporary URLs in grenade and provide a good error message https://review.opendev.org/c/openstack/ironic/+/777430 | 16:30 |
*** gyee has joined #openstack-ironic | 16:41 | |
*** anuradha1904 has joined #openstack-ironic | 16:43 | |
*** ociuhandu has joined #openstack-ironic | 16:47 | |
*** dtantsur is now known as dtantsur|brb | 16:50 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Allow instance_info to override node interface https://review.opendev.org/c/openstack/ironic/+/777434 | 16:52 |
*** lucasagomes has quit IRC | 17:01 | |
*** ociuhandu_ has joined #openstack-ironic | 17:04 | |
*** ociuhandu has quit IRC | 17:07 | |
*** ociuhandu_ has quit IRC | 17:08 | |
*** k_mouza_ has quit IRC | 17:14 | |
*** k_mouza has joined #openstack-ironic | 17:14 | |
*** ociuhandu has joined #openstack-ironic | 17:18 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Port/Portgroup project scoped access https://review.opendev.org/c/openstack/ironic/+/775465 | 17:19 |
*** ociuhandu has quit IRC | 17:22 | |
*** derekh has quit IRC | 17:23 | |
*** derekh has joined #openstack-ironic | 17:23 | |
*** derekh has quit IRC | 17:26 | |
*** derekh has joined #openstack-ironic | 17:26 | |
zer0c00l | i wanted to use pykickstart library in ironic. I will be just calling the binary 'ksvalidator' from ironic to validate the kickstart file | 17:35 |
zer0c00l | i am just realizing that pykickstart is GPL-v2 | 17:35 |
TheJulia | ugh, just all the binary! just remember to also forget everything you remembered from pykickstart first | 17:35 |
zer0c00l | and GPL-v2 isnot compatible with ironic. | 17:36 |
zer0c00l | TheJulia: so no on pykickstart? | 17:36 |
zer0c00l | i guess we can't validate kickstart files, we can validate the template though. | 17:37 |
zer0c00l | "[A]GPL libraries used during validation or testing phases of development fall into a gray area - they are not presumed to be compatible or incompatible and instead are reviewed on a case by case basis. Please use the legal-discuss mailing list to bring up any such cases." | 17:37 |
zer0c00l | https://governance.openstack.org/tc/reference/licensing.html | 17:37 |
TheJulia | yeah, afaik not compatible for end operating state | 17:37 |
TheJulia | but literally if it is just calling an external tool why not just call it? | 17:38 |
TheJulia | as long as it won't leak anything | 17:38 |
zer0c00l | The tool is writtern in 'python' | 17:38 |
zer0c00l | it will be an external dependency. | 17:39 |
zer0c00l | i guess we can check if the tool exists on the conductor, if it does we invoke it. If it doesn't then we ignore it. | 17:39 |
zer0c00l | Will that work? | 17:39 |
TheJulia | I think that would be fine | 17:39 |
zer0c00l | cool! | 17:40 |
zer0c00l | wish openstack is GPL :) | 17:40 |
zer0c00l | i guess it will scare some of the companies | 17:41 |
TheJulia | execing out to a gpl thing is fine | 17:43 |
*** derekh has quit IRC | 17:46 | |
*** derekh has joined #openstack-ironic | 17:47 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Enable swift temporary URLs in grenade and provide a good error message https://review.opendev.org/c/openstack/ironic/+/777375 | 17:57 |
arne_wiebalck | bye everyone o/ | 17:59 |
TheJulia | goodnight! | 17:59 |
*** derekh has quit IRC | 18:00 | |
*** tosin has quit IRC | 18:04 | |
*** bkslash has joined #openstack-ironic | 18:04 | |
TheJulia | https://meet.google.com/vaf-jzoy-kqp <-- Discussing security interfaces | 18:04 |
*** bkslash has quit IRC | 18:04 | |
*** iurygregory_ has joined #openstack-ironic | 18:04 | |
*** iurygregory has quit IRC | 18:05 | |
*** iurygregory_ is now known as iurygregory | 18:05 | |
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:29 |
*** dtantsur|brb is now known as dtantsur | 18:31 | |
*** sdanni has joined #openstack-ironic | 18:47 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Enable swift temporary URLs in grenade and provide a good error message https://review.opendev.org/c/openstack/ironic/+/777375 | 18:57 |
*** k_mouza has quit IRC | 19:00 | |
stevebaker | morning | 19:03 |
*** gregwork has joined #openstack-ironic | 19:06 | |
TheJulia | good morning | 19:07 |
TheJulia | stevebaker: take a look at the portg/portgroup change again :) | 19:08 |
stevebaker | oki | 19:08 |
TheJulia | if that makes sense to you'll rinse/repeat again | 19:08 |
dtantsur | morning stevebaker :) seeing you here is a good reminder I should close my laptop :) | 19:10 |
stevebaker | dtantsur: lol! | 19:10 |
*** rcernin has joined #openstack-ironic | 19:10 | |
*** rcernin has quit IRC | 19:15 | |
*** rh-jelabarre has quit IRC | 19:27 | |
*** rh-jelabarre has joined #openstack-ironic | 19:30 | |
*** dtantsur is now known as dtantsur|afk | 19:56 | |
*** k_mouza has joined #openstack-ironic | 20:14 | |
*** k_mouza has quit IRC | 20:19 | |
*** bdodd has quit IRC | 20:27 | |
*** sshnaidm is now known as sshnaidm|afk | 20:28 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP Volume targets/connectors Project Scoped RBAC https://review.opendev.org/c/openstack/ironic/+/776314 | 20:32 |
*** rcernin has joined #openstack-ironic | 20:33 | |
TheJulia | stevebaker: ^^^ was remarkably easy | 20:33 |
* stevebaker is mired in dracut problems | 20:33 | |
*** zzzeek has quit IRC | 20:34 | |
TheJulia | eek | 20:34 |
*** zzzeek has joined #openstack-ironic | 20:34 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Volume targets/connectors Project Scoped RBAC https://review.opendev.org/c/openstack/ironic/+/776314 | 20:42 |
*** anuradha1904 has quit IRC | 20:42 | |
*** juanoterocas has joined #openstack-ironic | 20:42 | |
*** rcernin has quit IRC | 20:47 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Project scope driver vendor pass-through https://review.opendev.org/c/openstack/ironic/+/776767 | 20:52 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Follow-up on project scoped trait tests https://review.opendev.org/c/openstack/ironic/+/776768 | 20:52 |
* TheJulia attempts to wrap brain around allocations | 21:02 | |
stevebaker | TheJulia: alright, I've done a review on port/portgroups | 21:02 |
TheJulia | much appreciated | 21:03 |
*** bdodd has joined #openstack-ironic | 21:08 | |
*** rcernin has joined #openstack-ironic | 21:14 | |
TheJulia | dtantsur|afk: so w/r/t allocation, was the idea behind the owner field to allow something along the lines of "make this allocation and give it to x tenant... oh and only use their owned/leased nodes for the allocation??" I'm thinking the change would need to effecitiely be "if not system admin, filter nodes by project id and auto-add project id to owner field for the allocation itself. | 21:17 |
* TheJulia is starting to wonder if this becomes an api anyone can post against | 21:17 | |
*** rcernin has quit IRC | 21:19 | |
*** zzzeek has quit IRC | 21:26 | |
*** zzzeek has joined #openstack-ironic | 21:28 | |
*** rcernin has joined #openstack-ironic | 21:32 | |
*** rcernin has quit IRC | 21:37 | |
*** rcernin has joined #openstack-ironic | 21:40 | |
*** rcernin has quit IRC | 21:45 | |
*** rcernin has joined #openstack-ironic | 21:46 | |
*** zzzeek has quit IRC | 21:49 | |
tzumainn | TheJulia, that was indeed the idea | 21:50 |
tzumainn | there are actually two policy rules for creating an allocation - baremetal:allocation:create and baremetal:allocation:create_restricted | 21:50 |
tzumainn | the idea being that the former is for admins who can create an allocation and set the owner field to anything | 21:50 |
TheJulia | Ahh, haven't looked there yet | 21:51 |
tzumainn | and the latter is for non-admins, and forces the owner field into their project | 21:51 |
*** rcernin has quit IRC | 21:51 | |
TheJulia | That works quite well then | 21:51 |
tzumainn | I actually kinda forgot about it until you asked :) but the code is in ironic/api/controllers/v1/allocation.py if you're curious | 21:52 |
*** zzzeek has joined #openstack-ironic | 21:53 | |
TheJulia | I took an initial look but didnt' dig too deeply because I figured I'll try and code on it tomorrow | 21:53 |
TheJulia | today, I just wanted to wrap my head around it | 21:53 |
tzumainn | makes sense | 21:54 |
tzumainn | I think there were attempts to make nodes, ports, and allocations do sensible things for the weird admin/non-admin cases - like having list versus list_all policy rules, and create create_restricted | 21:55 |
TheJulia | That actually came in handy if you checkout the ports/portgroups patch | 21:55 |
TheJulia | so there are knobs at various levels | 21:55 |
TheJulia | anyway, I actually need to go to a bank branch :\ | 21:55 |
*** devtolu1__ has joined #openstack-ironic | 21:56 | |
*** sdanni has quit IRC | 21:57 | |
*** juanoterocas has quit IRC | 21:58 | |
tzumainn | ah, I see - I understand now I completely whiffed on the portgroup case | 21:59 |
TheJulia | Yeah, I think I only had to change two minor things in ports | 22:00 |
*** rh-jelabarre has quit IRC | 22:08 | |
*** rcernin has joined #openstack-ironic | 22:15 | |
*** devtolu1__ has quit IRC | 22:19 | |
*** juanoterocas has joined #openstack-ironic | 22:19 | |
*** lmcgann has quit IRC | 22:35 | |
*** rloo has quit IRC | 22:54 | |
janders | good morning Ironic o/ | 23:05 |
*** pmannidi has quit IRC | 23:05 | |
iurygregory | morning janders o/ | 23:06 |
*** pmannidi has joined #openstack-ironic | 23:10 | |
*** juanoterocas has quit IRC | 23:13 | |
*** zzzeek has quit IRC | 23:14 | |
*** zzzeek has joined #openstack-ironic | 23:15 | |
openstackgerrit | Merged openstack/ironic master: Enable swift temporary URLs in grenade and provide a good error message https://review.opendev.org/c/openstack/ironic/+/777375 | 23:17 |
janders | TheJulia IRT https://bugzilla.redhat.com/show_bug.cgi?id=1670336 - thank you for responding to Antonios. I've got a patch up for reviews: https://review.opendev.org/c/openstack/ironic/+/774391, linked it to the BZ | 23:17 |
openstack | janders: Error: Error getting bugzilla.redhat.com bug #1670336: NotPermitted | 23:17 |
TheJulia | No problem, thank you for that! | 23:26 |
janders | by when would this patch have to merge to make it to OSP17? | 23:26 |
janders | Dmitry mentioned yesterday Ironic has different timelines than other OpenStack projects but I didn't follow up and ask for the date | 23:27 |
TheJulia | Sometimes in the next 3-4 weeks most likely | 23:30 |
janders | cool, thank you TheJulia | 23:31 |
stevebaker | TheJulia: how is your dracut fu? I'm finding that unless the 'network-legacy' module is explicitly omitted on RHEL then dracut fails. I want to know if it is ok to omit it by default in the general case | 23:45 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!