*** jamesdenton has quit IRC | 00:02 | |
*** jamesdenton has joined #openstack-ironic | 00:02 | |
*** rloo has quit IRC | 00:02 | |
*** rloo has joined #openstack-ironic | 00:03 | |
*** rloo has quit IRC | 00:03 | |
*** sshnaidm|ruck is now known as sshnaidm|afk | 00:08 | |
*** tosky has quit IRC | 00:09 | |
iurygregory | good morning janders o/ | 00:29 |
---|---|---|
janders | hey iurygregory o/ | 00:29 |
*** rcernin has quit IRC | 00:32 | |
*** tzumainn has quit IRC | 00:34 | |
*** rcernin has joined #openstack-ironic | 00:37 | |
*** lbragstad_ has joined #openstack-ironic | 00:38 | |
*** lbragstad has quit IRC | 00:40 | |
*** tosin has quit IRC | 01:20 | |
*** xinliang has joined #openstack-ironic | 01:34 | |
*** lbragstad_ is now known as lbragstad | 01:37 | |
*** lbragstad has quit IRC | 01:46 | |
openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: [WIP] Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 02:11 |
openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 02:13 |
openstackgerrit | Jacob Anders proposed openstack/ironic master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic/+/771905 | 02:14 |
*** rh-jelabarre has quit IRC | 02:22 | |
openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 02:24 |
janders | dtantsur|afk rpittau|afk ^ this change ( https://review.opendev.org/q/I2f63db9b739e53699bd5f164b79640927bf757d7 ) can likely replace the more complex NVMe cleaning proposal ( https://review.opendev.org/c/openstack/ironic-python-agent/+/770237 ) for the time being. We can re-visit the more complex approach when sanitize support becomes more | 02:42 |
janders | mainstream. What are your thoughts? I'd welcome your feedback when you have some time to look at this. Thank you! | 02:42 |
*** rcernin has quit IRC | 03:08 | |
*** ianw is now known as ianw_pto | 03:12 | |
*** lbragstad has joined #openstack-ironic | 03:25 | |
*** dsneddon has quit IRC | 03:32 | |
*** rcernin has joined #openstack-ironic | 03:33 | |
*** gyee has quit IRC | 03:36 | |
*** rcernin has quit IRC | 03:48 | |
*** rcernin has joined #openstack-ironic | 04:04 | |
*** rcernin has quit IRC | 04:05 | |
*** rcernin has joined #openstack-ironic | 04:05 | |
*** k_mouza has joined #openstack-ironic | 04:18 | |
*** mkrai has joined #openstack-ironic | 04:20 | |
*** xinliang has quit IRC | 04:20 | |
*** k_mouza has quit IRC | 04:22 | |
*** uzumaki has joined #openstack-ironic | 04:33 | |
*** lbragstad has quit IRC | 05:11 | |
*** mkrai has quit IRC | 05:22 | |
*** mkrai_ has joined #openstack-ironic | 05:22 | |
*** mkrai_ has quit IRC | 05:51 | |
*** mkrai has joined #openstack-ironic | 05:52 | |
*** sdanni has quit IRC | 05:58 | |
zer0c00l | I am wondering if i can make 'anaconda' as a boot option capablity? https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/deploy_utils.py#L58 | 06:04 |
zer0c00l | basically this will allow the PXEBoot interface to identify anaconda boot and the users intent. | 06:06 |
zer0c00l | Looks like 'ramdisk' is already a boot option. | 06:07 |
*** k_mouza has joined #openstack-ironic | 06:19 | |
*** k_mouza has quit IRC | 06:23 | |
*** anuradha1904 has joined #openstack-ironic | 06:30 | |
zer0c00l | I am also planning to reuse http_root and http_url configuration options from ipxe deploy. These directories will be used to "Store" and server kickstart and anaconda squashfs via "http(s)" | 06:38 |
*** uzumaki has quit IRC | 06:52 | |
*** moshiur has joined #openstack-ironic | 07:20 | |
*** rcernin has quit IRC | 07:25 | |
*** rpittau|afk is now known as rpittau | 07:34 | |
rpittau | good morning ironic! o/ | 07:34 |
rpittau | Happy Friday! | 07:34 |
*** rcernin has joined #openstack-ironic | 07:37 | |
*** rcernin has quit IRC | 07:42 | |
openstackgerrit | Yogesh proposed openstack/ironic master: WIP Add idrac HW type IPMI interface support https://review.opendev.org/c/openstack/ironic/+/771862 | 07:44 |
rpittau | janders: re nvme-lite it looks like a good starting approach, and as you said, we can revisit that later | 07:45 |
arne_wiebalck | Good morning, ironic! | 07:50 |
*** rcernin has joined #openstack-ironic | 07:50 | |
rpittau | hey arne_wiebalck :) | 07:50 |
arne_wiebalck | hey rpittau, good morning o/ | 07:51 |
*** rcernin has quit IRC | 07:55 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Update minversion of tox https://review.opendev.org/c/openstack/ironic-inspector/+/771940 | 07:56 |
janders | good morning arne_wiebalck and rpittau, Happy Friday! | 07:58 |
*** rcernin has joined #openstack-ironic | 08:08 | |
*** mgoddard has quit IRC | 08:11 | |
*** mgoddard has joined #openstack-ironic | 08:11 | |
*** rcernin has quit IRC | 08:13 | |
*** rcernin has joined #openstack-ironic | 08:14 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Update minversion of tox https://review.opendev.org/c/openstack/ironic-python-agent/+/771942 | 08:16 |
*** rcernin has quit IRC | 08:19 | |
*** dsneddon has joined #openstack-ironic | 08:24 | |
*** rcernin has joined #openstack-ironic | 08:26 | |
*** dsneddon has quit IRC | 08:30 | |
openstackgerrit | Riccardo Pittau proposed openstack/sushy master: Update minversion of tox https://review.opendev.org/c/openstack/sushy/+/771943 | 08:30 |
*** rcernin has quit IRC | 08:31 | |
*** rcernin has joined #openstack-ironic | 08:44 | |
*** tosky has joined #openstack-ironic | 08:44 | |
*** mkrai has quit IRC | 08:45 | |
*** dougsz has joined #openstack-ironic | 08:47 | |
openstackgerrit | Riccardo Pittau proposed openstack/sushy-tools master: Move some test requirements to tox.ini https://review.opendev.org/c/openstack/sushy-tools/+/771945 | 08:49 |
*** rcernin has quit IRC | 08:49 | |
*** jamesdenton has quit IRC | 08:49 | |
*** jamesdenton has joined #openstack-ironic | 08:49 | |
*** ociuhandu has joined #openstack-ironic | 09:02 | |
*** lucasagomes has joined #openstack-ironic | 09:02 | |
*** rcernin has joined #openstack-ironic | 09:08 | |
openstackgerrit | Riccardo Pittau proposed openstack/python-ironicclient master: Update minversion of tox https://review.opendev.org/c/openstack/python-ironicclient/+/771948 | 09:09 |
arne_wiebalck | hey janders o/ | 09:12 |
*** rcernin has quit IRC | 09:13 | |
*** andy_ has quit IRC | 09:14 | |
*** PrinzElvis has quit IRC | 09:14 | |
*** zigo has quit IRC | 09:14 | |
*** andy_ has joined #openstack-ironic | 09:15 | |
*** PrinzElvis has joined #openstack-ironic | 09:15 | |
*** zigo has joined #openstack-ironic | 09:15 | |
*** mkowalski has quit IRC | 09:21 | |
*** eagereagle1 has quit IRC | 09:24 | |
*** rcernin has joined #openstack-ironic | 09:25 | |
openstackgerrit | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Update minversion of tox https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/771953 | 09:26 |
*** rcernin has quit IRC | 09:30 | |
*** rcernin has joined #openstack-ironic | 09:32 | |
*** derekh has joined #openstack-ironic | 09:48 | |
openstackgerrit | Riccardo Pittau proposed openstack/virtualbmc master: Update minversion of tox https://review.opendev.org/c/openstack/virtualbmc/+/771955 | 09:50 |
*** k_mouza has joined #openstack-ironic | 09:59 | |
*** dtantsur|afk is now known as dtantsur | 09:59 | |
dtantsur | morning ironic | 09:59 |
janders | good morning dtantsur | 10:00 |
janders | SPUC anyone? | 10:00 |
dtantsur | omw | 10:01 |
dtantsur | rpittau, ajya, arne_wiebalck ^^ | 10:01 |
janders | (we can do a shorter one if people are busy - I wouldnt mind submitting one more revision of my NVMe patch before the weekend :) ) | 10:01 |
rpittau | not sure I will be able to connect this morning, I have another meeting in 30 mins and have to finish a couple of things before then :/ | 10:02 |
*** k_mouza_ has joined #openstack-ironic | 10:06 | |
*** Dhuldev_Valekar has joined #openstack-ironic | 10:08 | |
*** k_mouza has quit IRC | 10:09 | |
*** mkowalski has joined #openstack-ironic | 10:10 | |
*** tosin has joined #openstack-ironic | 10:20 | |
arne_wiebalck | sorry, in another meeting atm | 10:34 |
*** k_mouza has joined #openstack-ironic | 10:40 | |
*** k_mouza_ has quit IRC | 10:44 | |
*** hoonetorg has quit IRC | 11:10 | |
iurygregory | good morning arne_wiebalck janders rpittau dtantsur and Ironic o/ happy friday | 11:24 |
arne_wiebalck | hey iurygregory o/ | 11:24 |
iurygregory | \o/ | 11:27 |
*** mgoddard has quit IRC | 11:30 | |
*** rcernin has quit IRC | 11:31 | |
*** Dhuldev_Valekar has quit IRC | 11:34 | |
*** rcernin has joined #openstack-ironic | 11:38 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Removes unnecessary requirement of sdparm https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/771965 | 11:41 |
dtantsur | rpittau, ^^^ | 11:41 |
rpittau | well that makes things easier :D | 11:57 |
*** rcernin has quit IRC | 12:00 | |
iurygregory | dtantsur is killing things with fire \o/ | 12:06 |
dtantsur | :) | 12:08 |
nam-est | Please give some reviews to these PRs: | 12:09 |
nam-est | https://github.com/metal3-io/ironic-image/pull/230 | 12:09 |
nam-est | https://github.com/metal3-io/ironic-inspector-image/pull/70 | 12:09 |
nam-est | All comments have been addressed, and also the PRs have passed the CI. | 12:09 |
janders | see you next week Ironic o/ | 12:22 |
janders | I'll be off Monday-Tuesday but will be back on Wednesday. Will attend Tuesday's midcycle session (that's Wednesday in my timezone). | 12:23 |
janders | have a good weekend everyone | 12:23 |
*** rcernin has joined #openstack-ironic | 12:27 | |
*** iurygregory_ has joined #openstack-ironic | 12:28 | |
*** iurygregory has quit IRC | 12:28 | |
*** iurygregory_ is now known as iurygregory | 12:29 | |
*** k_mouza has quit IRC | 12:43 | |
*** k_mouza has joined #openstack-ironic | 12:44 | |
*** rcernin has quit IRC | 12:48 | |
*** k_mouza has quit IRC | 12:54 | |
*** k_mouza_ has joined #openstack-ironic | 12:54 | |
*** rh-jelabarre has joined #openstack-ironic | 12:55 | |
*** k_mouza has joined #openstack-ironic | 12:55 | |
*** ociuhandu has quit IRC | 12:55 | |
*** ociuhandu has joined #openstack-ironic | 12:56 | |
*** ociuhandu has quit IRC | 12:58 | |
*** k_mouza_ has quit IRC | 12:58 | |
*** ociuhandu has joined #openstack-ironic | 12:59 | |
*** k_mouza has quit IRC | 13:00 | |
*** k_mouza has joined #openstack-ironic | 13:02 | |
*** ociuhandu has quit IRC | 13:02 | |
*** ociuhandu has joined #openstack-ironic | 13:02 | |
*** ociuhandu has quit IRC | 13:02 | |
*** k_mouza has quit IRC | 13:03 | |
*** ociuhandu has joined #openstack-ironic | 13:03 | |
*** k_mouza has joined #openstack-ironic | 13:05 | |
*** k_mouza_ has joined #openstack-ironic | 13:07 | |
*** k_mouza has quit IRC | 13:11 | |
*** ociuhandu has quit IRC | 13:15 | |
*** tkajinam_ has quit IRC | 13:16 | |
*** paras333 has joined #openstack-ironic | 13:28 | |
tosin | Good day. Can someone please help check this? https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770845/6/plugins/modules/baremetal_node_action.py | 13:30 |
*** mgoddard has joined #openstack-ironic | 13:32 | |
*** moshiur has quit IRC | 13:36 | |
*** moshiur has joined #openstack-ironic | 13:36 | |
*** hoonetorg has joined #openstack-ironic | 13:37 | |
*** ociuhandu has joined #openstack-ironic | 13:39 | |
*** lbragstad has joined #openstack-ironic | 13:42 | |
*** ociuhandu has quit IRC | 13:49 | |
*** ociuhandu has joined #openstack-ironic | 13:50 | |
*** rloo has joined #openstack-ironic | 14:03 | |
*** jdandrea has joined #openstack-ironic | 14:08 | |
*** ociuhandu has quit IRC | 14:20 | |
*** jamesdenton has quit IRC | 14:20 | |
ajya | iurygregory: got some light on the EjectMedia. There is no underlying issue with EjectMedia as such, what has changed is what iDRAC returns in case of 415 error. There is workaround for 415 error as iDRAC and others don't like empty body. | 14:23 |
ajya | In older versions it returned HTML body with the error, now it returns JSON with ExtendedInfo that is not list. | 14:24 |
ajya | so it's only error handling that fails now, previously different block handled it because there was no JSON | 14:24 |
iurygregory | ajya, gotcha we can add a workaround in sushy for this | 14:25 |
iurygregory | but the InsertMedia is broken right? | 14:25 |
ajya | yes, that's my suggestion | 14:25 |
ajya | iurygregory: so about InsertMedia - the issue is not widespread, I tried with another server having 4.40.00.00 and it worked. Still investigating about the cases where it does not work. I'd say could still give 4.40.00.00 a try once error handling is fixed | 14:26 |
ajya | it looks like I run into InsertMedia thing accidentally while trying it out, for now I don't see that it is related | 14:27 |
ajya | iurygregory: do you want to update error handling or can I? | 14:27 |
iurygregory | ajya, feel free to update =) | 14:28 |
iurygregory | you can push a patch to master | 14:28 |
*** moshiur has quit IRC | 14:28 | |
ajya | iurygregory: ok | 14:28 |
TheJulia | good morning | 14:32 |
iurygregory | good morning TheJulia | 14:34 |
*** jamesdenton has joined #openstack-ironic | 14:45 | |
* TheJulia needs to switch coffees again | 14:46 | |
*** ociuhandu has joined #openstack-ironic | 14:49 | |
openstackgerrit | Aija Jauntēva proposed openstack/sushy master: Fix ExtendedInfo error handling for non-list item https://review.opendev.org/c/openstack/sushy/+/771988 | 14:53 |
rpittau | bye everyone! Have a great weekend! o/ | 14:58 |
*** rpittau is now known as rpittau|afk | 14:58 | |
dtantsur | morning TheJulia | 15:00 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] redfish-virtual-media: allow a link to raw configdrive image https://review.opendev.org/c/openstack/ironic/+/771989 | 15:00 |
*** lmcgann has joined #openstack-ironic | 15:00 | |
*** MentalSiege has joined #openstack-ironic | 15:03 | |
iurygregory | dtantsur, wdyt of https://review.opendev.org/c/openstack/sushy/+/771988 ? makes sense to me | 15:05 |
* iurygregory brb | 15:05 | |
*** MentalSiege has quit IRC | 15:08 | |
*** mugsie has joined #openstack-ironic | 15:09 | |
dtantsur | lgtm module missing unit test | 15:17 |
*** mugsie has quit IRC | 15:18 | |
ajya | dtantsur: yup, I said that to iurygregory, he wanted to see this first. So create a new file? | 15:18 |
dtantsur | ajya: we don't have unit tests for exceptions? Oo | 15:18 |
dtantsur | they might be in test_connector.py | 15:19 |
ajya | yes, look like there is smth dedicated for that, other than that the code was covered | 15:21 |
*** sdanni has joined #openstack-ironic | 15:25 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: redfish-virtual-media: allow a link to raw configdrive image https://review.opendev.org/c/openstack/ironic/+/771989 | 15:27 |
openstackgerrit | Aija Jauntēva proposed openstack/sushy master: Fix ExtendedInfo error handling for non-list item https://review.opendev.org/c/openstack/sushy/+/771988 | 15:41 |
*** dougsz has quit IRC | 15:44 | |
TheJulia | spuc in 1.25 hours right? | 15:46 |
*** mugsie has joined #openstack-ironic | 15:51 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: Correct base64 error handling https://review.opendev.org/c/openstack/ironic-lib/+/771998 | 15:55 |
iurygregory | TheJulia, yeah | 15:56 |
TheJulia | so many executions of unit tests... so so so many executions | 15:59 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: Get rid of oslo.serialization https://review.opendev.org/c/openstack/ironic-lib/+/771999 | 16:03 |
dtantsur | TheJulia: re SPUC: yes | 16:03 |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Removes unnecessary requirement of sdparm https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/771965 | 16:05 |
*** ociuhandu has quit IRC | 16:18 | |
*** ociuhandu has joined #openstack-ironic | 16:19 | |
*** ociuhandu has quit IRC | 16:24 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: Correct base64 error handling https://review.opendev.org/c/openstack/ironic-lib/+/771998 | 16:25 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: Allow binary data for configdrive https://review.opendev.org/c/openstack/ironic-lib/+/772062 | 16:33 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: Get rid of oslo.serialization https://review.opendev.org/c/openstack/ironic-lib/+/771999 | 16:34 |
*** ociuhandu has joined #openstack-ironic | 16:38 | |
*** ociuhandu has quit IRC | 16:42 | |
*** ociuhandu has joined #openstack-ironic | 16:47 | |
*** k_mouza_ has quit IRC | 16:52 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Replace pysendfile with os.sendfile https://review.opendev.org/c/openstack/ironic/+/772066 | 16:53 |
*** bburns has quit IRC | 17:00 | |
*** bburns has joined #openstack-ironic | 17:04 | |
dtantsur | SPUC is going on for those interested | 17:05 |
*** lucasagomes has quit IRC | 17:08 | |
*** k_mouza has joined #openstack-ironic | 17:09 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for node and driver passthru https://review.opendev.org/c/openstack/ironic/+/763271 | 17:10 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for utility APIs https://review.opendev.org/c/openstack/ironic/+/763272 | 17:10 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for volume APIs https://review.opendev.org/c/openstack/ironic/+/763273 | 17:10 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for conductor APIs https://review.opendev.org/c/openstack/ironic/+/763274 | 17:10 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for the allocation APIs https://review.opendev.org/c/openstack/ironic/+/763275 | 17:10 |
TheJulia | boom | 17:10 |
*** zzzeek has quit IRC | 17:12 | |
*** zzzeek has joined #openstack-ironic | 17:13 | |
*** ociuhandu_ has joined #openstack-ironic | 17:22 | |
iurygregory | wow gerrit is a bit slow or just with me? | 17:23 |
dtantsur | a bit?? | 17:23 |
dtantsur | uploading any patch takes 30+ seconds for me | 17:23 |
*** ociuhandu has quit IRC | 17:26 | |
*** ociuhandu_ has quit IRC | 17:27 | |
iurygregory | I gave a +2 it took 30s to apply (I was like woot?!) | 17:30 |
*** ociuhandu has joined #openstack-ironic | 17:47 | |
*** gyee has joined #openstack-ironic | 17:51 | |
*** ociuhandu has quit IRC | 17:53 | |
*** derekh has quit IRC | 18:01 | |
arne_wiebalck | bye everyone, have a good week-end o/ | 18:06 |
iurygregory | bye arne_wiebalck enjoy the weekend! | 18:08 |
*** lbragstad has quit IRC | 18:16 | |
TheJulia | has anyone mentioned this to opendev folks? | 18:17 |
dtantsur | not me | 18:17 |
* dtantsur waves | 18:17 | |
*** dtantsur is now known as dtantsur|afk | 18:17 | |
iurygregory | I haven't | 18:17 |
iurygregory | maybe was just my connection .-. | 18:18 |
TheJulia | it was super slow to uplaod the last batch of rbac stuffs | 18:18 |
*** lbragstad has joined #openstack-ironic | 18:22 | |
zer0c00l | The current ramdisk deploy driver uses 'ramdisk' as a 'boot_option' I am wondering if i can make 'anaconda' as a boot option capablity for anaconda deploy driver https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/deploy_utils.py#L58 | 18:27 |
zer0c00l | so i can determine if the node is going to boot 'anaconda' in the boot interface | 18:27 |
zer0c00l | any reservations about this approach? | 18:28 |
* TheJulia ponders | 18:32 | |
TheJulia | lbragstad: if you have a minute, I could use a set of eyes on http://paste.openstack.org/show/801896/, if anything looks obvious to you. | 18:34 |
TheJulia | zer0c00l: I guess that is likely okay and it would be a good flavor and or glance attribute path in | 18:35 |
zer0c00l | Sorry i don't understand your last sentence "it would be a good flavor and or glance attribute path in" | 18:36 |
zer0c00l | Thanks. | 18:36 |
TheJulia | sorry, I mean, it would be a viable path to get that information in from a flavor scheudling or maybe even a glance image attribute | 18:36 |
zer0c00l | TheJulia: I was also thinking about reusing 'CONF.deploy.http_url' and 'CONF.deploy.http_root' (these are right now required only if ipxe is enabled) | 18:37 |
TheJulia | zer0c00l: I think that is fine | 18:37 |
zer0c00l | Basically those two are needed to store and transfer kickstart file and squashfs. | 18:37 |
zer0c00l | Thanks | 18:37 |
TheJulia | ++ | 18:37 |
lbragstad | TheJulia sounds good - i have it in the queue, wrapping up some placement policy stuff now and i should be able to take a look | 18:40 |
TheJulia | lbragstad: thanks | 18:41 |
*** dsneddon has joined #openstack-ironic | 18:43 | |
*** k_mouza has quit IRC | 18:56 | |
openstackgerrit | Merged openstack/sushy master: Fix ExtendedInfo error handling for non-list item https://review.opendev.org/c/openstack/sushy/+/771988 | 19:18 |
openstackgerrit | Lance Bragstad proposed openstack/ironic master: Pass context objects directly to policy enforcement https://review.opendev.org/c/openstack/ironic/+/772086 | 19:26 |
lbragstad | TheJulia I think you might need that ^ | 19:27 |
lbragstad | i proposed it outside the chain you're working on | 19:27 |
TheJulia | lets see! | 19:29 |
lbragstad | TheJulia i think you're getting that error even with system scope because we're missing this | 19:29 |
lbragstad | https://opendev.org/openstack/oslo.policy/src/branch/master/oslo_policy/policy.py#L1073-L1082 | 19:29 |
lbragstad | oslo.policy will handle that case for you - but ironic attempts to map context attributes into credential dictionaries prior to enforcement | 19:29 |
lbragstad | which is really good, just not needed anymore since oslo.policy is smart enough to do that for people now | 19:30 |
TheJulia | b'{"error_message": "{\\"faultcode\\": \\"Server\\", \\"faultstring\\": \\"name \'cdict\' is not defined\\", \\"debuginfo\\": null}"}' | 19:30 |
TheJulia | :\ | 19:30 |
*** anuradha1904 has quit IRC | 19:31 | |
lbragstad | bah - sorry | 19:31 |
lbragstad | fixing | 19:31 |
TheJulia | ok | 19:31 |
* TheJulia cooks lunch | 19:31 | |
*** jamesdenton has quit IRC | 19:43 | |
*** jamesdenton has joined #openstack-ironic | 19:43 | |
*** eagereagle1 has joined #openstack-ironic | 19:51 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/sushy stable/victoria: Fix ExtendedInfo error handling for non-list item https://review.opendev.org/c/openstack/sushy/+/772089 | 20:00 |
iurygregory | TheJulia, if you have some minutes later today ^ | 20:01 |
TheJulia | iurygregory: sure | 20:01 |
iurygregory | tyvm! | 20:01 |
iurygregory | have a great weekend Ironic o/ | 20:02 |
*** rh-jelabarre has quit IRC | 20:12 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIPImplement secure RBAC for the event APIs https://review.opendev.org/c/openstack/ironic/+/763276 | 20:19 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for the deploy templates APIs https://review.opendev.org/c/openstack/ironic/+/763277 | 20:19 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for baremetal nodes https://review.opendev.org/c/openstack/ironic/+/763257 | 20:24 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for ports https://review.opendev.org/c/openstack/ironic/+/763267 | 20:25 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for port groups https://review.opendev.org/c/openstack/ironic/+/763268 | 20:25 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for chassis https://review.opendev.org/c/openstack/ironic/+/763269 | 20:25 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for baremetal drivers https://review.opendev.org/c/openstack/ironic/+/763270 | 20:26 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for node and driver passthru https://review.opendev.org/c/openstack/ironic/+/763271 | 20:27 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for utility APIs https://review.opendev.org/c/openstack/ironic/+/763272 | 20:27 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for volume APIs https://review.opendev.org/c/openstack/ironic/+/763273 | 20:27 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for conductor APIs https://review.opendev.org/c/openstack/ironic/+/763274 | 20:28 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for the allocation APIs https://review.opendev.org/c/openstack/ironic/+/763275 | 20:29 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIPImplement secure RBAC for the event APIs https://review.opendev.org/c/openstack/ironic/+/763276 | 20:29 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for the deploy templates APIs https://review.opendev.org/c/openstack/ironic/+/763277 | 20:29 |
TheJulia | I really need to add coauthored by flags | 20:30 |
* TheJulia things she has hit spoonlimit for the week | 20:38 | |
openstackgerrit | Lance Bragstad proposed openstack/ironic master: Pass context objects directly to policy enforcement https://review.opendev.org/c/openstack/ironic/+/772086 | 20:44 |
lbragstad | TheJulia ^ fixed the failing policy tests | 20:44 |
TheJulia | applied on top of my local branch and running all py3 tests | 20:45 |
TheJulia | this should be fun :) | 20:45 |
* lbragstad readies the popcorn | 20:47 | |
TheJulia | - Failed: 0 | 20:47 |
TheJulia | \o/ | 20:47 |
lbragstad | nice - i'll be curious if your functional tests start passing now without that InvalidScope error | 20:50 |
*** viks____ has quit IRC | 20:50 | |
*** k_mouza has joined #openstack-ironic | 20:56 | |
*** rcernin has joined #openstack-ironic | 20:58 | |
TheJulia | should be fine | 20:59 |
*** tosin has quit IRC | 21:00 | |
TheJulia | Anyway, my spoon reserve is empty | 21:00 |
*** k_mouza has quit IRC | 21:01 | |
*** rcernin has quit IRC | 21:02 | |
*** rcernin has joined #openstack-ironic | 21:03 | |
*** k_mouza has joined #openstack-ironic | 21:15 | |
*** k_mouza has quit IRC | 21:20 | |
*** rcernin has quit IRC | 21:28 | |
*** jdandrea has quit IRC | 22:09 | |
*** rcernin has joined #openstack-ironic | 22:46 | |
*** zzzeek has quit IRC | 22:56 | |
*** zzzeek has joined #openstack-ironic | 22:57 | |
*** ociuhandu has joined #openstack-ironic | 22:59 | |
*** ociuhandu has quit IRC | 23:03 | |
*** lmcgann has quit IRC | 23:07 | |
*** sdanni has quit IRC | 23:38 | |
*** jamesdenton has quit IRC | 23:40 | |
*** jamesdenton has joined #openstack-ironic | 23:41 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!