openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for baremetal nodes https://review.opendev.org/c/openstack/ironic/+/763257 | 00:04 |
---|---|---|
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for ports https://review.opendev.org/c/openstack/ironic/+/763267 | 00:04 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for port groups https://review.opendev.org/c/openstack/ironic/+/763268 | 00:04 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for chassis https://review.opendev.org/c/openstack/ironic/+/763269 | 00:04 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for baremetal drivers https://review.opendev.org/c/openstack/ironic/+/763270 | 00:04 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for node and driver passthru https://review.opendev.org/c/openstack/ironic/+/763271 | 00:04 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for utility APIs https://review.opendev.org/c/openstack/ironic/+/763272 | 00:04 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: Tea pot response https://review.opendev.org/c/openstack/ironic/+/773548 | 00:04 |
*** MentalSiege has quit IRC | 00:07 | |
*** dsneddon has quit IRC | 00:35 | |
*** dsneddon has joined #openstack-ironic | 00:38 | |
*** rloo has quit IRC | 00:39 | |
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org is being quickly restarted to apply a new security patch | 00:56 | |
*** k_mouza has joined #openstack-ironic | 00:59 | |
*** k_mouza has quit IRC | 01:03 | |
*** gyee has quit IRC | 01:09 | |
openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent-builder master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/773538 | 01:28 |
*** rcernin has quit IRC | 01:44 | |
*** xinliang has joined #openstack-ironic | 02:00 | |
*** zzzeek has quit IRC | 02:07 | |
*** lmcgann has quit IRC | 02:07 | |
*** rcernin has joined #openstack-ironic | 02:09 | |
*** zzzeek has joined #openstack-ironic | 02:10 | |
*** zzzeek has quit IRC | 02:24 | |
*** zzzeek has joined #openstack-ironic | 02:25 | |
*** tzumainn has quit IRC | 02:32 | |
*** uzumaki has quit IRC | 02:32 | |
*** paras333 has quit IRC | 02:41 | |
*** Qianbiao has joined #openstack-ironic | 02:45 | |
*** zzzeek has quit IRC | 02:48 | |
*** zzzeek has joined #openstack-ironic | 02:52 | |
*** irclogbot_2 has quit IRC | 03:27 | |
*** uzumaki has joined #openstack-ironic | 03:28 | |
*** pmannidi has quit IRC | 03:54 | |
*** mkrai has joined #openstack-ironic | 04:02 | |
*** stendulker has joined #openstack-ironic | 04:08 | |
*** ricolin_ has joined #openstack-ironic | 04:13 | |
*** pmannidi has joined #openstack-ironic | 04:15 | |
*** xinliang has quit IRC | 04:26 | |
*** irclogbot_3 has joined #openstack-ironic | 04:41 | |
*** stendulker has quit IRC | 04:51 | |
*** irclogbot_3 has quit IRC | 04:54 | |
*** Qianbiao has quit IRC | 04:55 | |
*** irclogbot_0 has joined #openstack-ironic | 04:58 | |
*** stendulker has joined #openstack-ironic | 04:59 | |
*** mkrai has quit IRC | 05:06 | |
*** ociuhandu has joined #openstack-ironic | 05:07 | |
*** ociuhandu has quit IRC | 05:12 | |
*** k_mouza has joined #openstack-ironic | 05:21 | |
*** k_mouza has quit IRC | 05:26 | |
*** mkrai has joined #openstack-ironic | 05:29 | |
*** ricolin_ has quit IRC | 05:35 | |
*** ricolin has joined #openstack-ironic | 05:39 | |
*** priteau has quit IRC | 05:47 | |
*** lbragstad_ has joined #openstack-ironic | 06:24 | |
*** lbragstad has quit IRC | 06:24 | |
*** moshiur has joined #openstack-ironic | 06:27 | |
*** xinliang has joined #openstack-ironic | 06:32 | |
*** mkrai_ has joined #openstack-ironic | 06:54 | |
*** mkrai has quit IRC | 06:58 | |
*** Qianbiao has joined #openstack-ironic | 07:02 | |
*** mkrai_ has quit IRC | 07:16 | |
*** rcernin has quit IRC | 07:37 | |
openstackgerrit | ankit proposed openstack/sushy stable/victoria: Fixes issue of redfish firmware update https://review.opendev.org/c/openstack/sushy/+/771604 | 07:40 |
arne_wiebalck | Good morning, ironic! | 07:43 |
*** mkrai_ has joined #openstack-ironic | 07:54 | |
*** dsneddon has quit IRC | 08:00 | |
*** uzumaki has quit IRC | 08:09 | |
*** rcernin has joined #openstack-ironic | 08:14 | |
*** stendulker has quit IRC | 08:22 | |
*** rpittau|afk is now known as rpittau | 08:25 | |
rpittau | good morning ironic! o/ | 08:25 |
*** zaneb has quit IRC | 08:30 | |
rpittau | janders, TheJulia, I'm afraid if we want nvme-cli on tinycore will have to compile it ourselves :/ | 08:30 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Update version of tgt to 1.0.80 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/773604 | 08:31 |
*** rcernin has quit IRC | 08:31 | |
*** zaneb has joined #openstack-ironic | 08:33 | |
*** dtantsur|afk is now known as dtantsur | 08:36 | |
dtantsur | morning ironic | 08:37 |
dtantsur | rpittau: not the first time we need to compile something to tinycore | 08:37 |
rpittau | yeah | 08:37 |
rpittau | good morning dtantsur :) | 08:38 |
dtantsur | JayF, arne_wiebalck, root device hints are, in fact, supported by all 3 real deploy methods we have, although support in the ansible deploy is partial | 08:39 |
*** tosin has joined #openstack-ironic | 08:39 | |
*** dougsz has joined #openstack-ironic | 08:40 | |
arne_wiebalck | hey dtantsur o/ | 08:46 |
arne_wiebalck | thanks | 08:46 |
arne_wiebalck | dtantsur: any thoughts on: 'was "setting root device hints automatically" already discussed and discarded in the history of Ironic? (before I file a story :))' | 08:46 |
arne_wiebalck | dtantsur: or, even better, the idea? ;) | 08:47 |
dtantsur | arne_wiebalck: well, I even wrote something like that for tripleo. let me see if it still exists. | 08:47 |
arne_wiebalck | dtantsur: uh! | 08:47 |
dtantsur | arne_wiebalck: https://opendev.org/openstack/tripleo-common/src/branch/master/tripleo_common/actions/baremetal.py#L211-L299 | 08:49 |
dtantsur | I think we could have such a plugin in ironic-inspector instead | 08:50 |
* arne_wiebalck is reading ... | 08:51 | |
*** paras333 has joined #openstack-ironic | 08:55 | |
*** priteau has joined #openstack-ironic | 08:57 | |
arne_wiebalck | dtantsur: that looks nice ... for disk replacements, I would think the scsi address may be better than the name, no? | 08:57 |
dtantsur | likely, but people love using names | 08:58 |
arne_wiebalck | agree, but that is quite risky | 08:58 |
dtantsur | yep | 08:58 |
arne_wiebalck | thanks, dtantsur, I will create a story to add a plugin to the inspector | 08:59 |
dtantsur | ++ | 08:59 |
*** paras333 has quit IRC | 08:59 | |
dtantsur | if anybody has time, please review https://review.opendev.org/c/openstack/ironic/+/773382 | 08:59 |
dtantsur | and https://review.opendev.org/c/openstack/ironic/+/771493 | 09:00 |
*** rcernin has joined #openstack-ironic | 09:00 | |
*** tosky has joined #openstack-ironic | 09:02 | |
*** lucasagomes has joined #openstack-ironic | 09:06 | |
arne_wiebalck | dtantsur: https://storyboard.openstack.org/#!/story/2008578 | 09:12 |
arne_wiebalck | dtantsur: feel free to amend | 09:12 |
*** rcernin has quit IRC | 09:18 | |
*** rcernin has joined #openstack-ironic | 09:23 | |
*** ociuhandu has joined #openstack-ironic | 09:29 | |
janders | good morning arne_wiebalck rpittau dtantsur | 09:34 |
janders | (and Ironic o/) | 09:34 |
rpittau | hey janders :) | 09:34 |
janders | rpittau dtantsur thank you for the pointers regarding nvme-cli and tinycore. Is this something I really need to sort out, or a minor inconvenience? | 09:35 |
rpittau | well, do we need nvme-cli in tinycore ? | 09:37 |
rpittau | are we going to test nvme tools using tinycore ? | 09:38 |
*** derekh has joined #openstack-ironic | 09:39 | |
*** ociuhandu has quit IRC | 09:44 | |
*** ociuhandu has joined #openstack-ironic | 09:44 | |
openstackgerrit | Tianhao Hu proposed openstack/ironic master: Mark the iSCSI deploy as deprecated in the docs https://review.opendev.org/c/openstack/ironic/+/773622 | 09:52 |
viks____ | hi, when i try to create instance with `--config-drive true`, i get the below error: | 09:53 |
viks____ | https://www.irccloud.com/pastebin/L1OL5iFn/ | 09:53 |
viks____ | any suggestions? | 09:54 |
*** dsneddon has joined #openstack-ironic | 10:01 | |
dtantsur | viks____: we need more context, there should be some output around the failure | 10:02 |
*** rcernin has quit IRC | 10:08 | |
viks____ | dtantsur: here it is: http://paste.openstack.org/show/802225/ | 10:09 |
viks____ | baiscally i'm checking by enabling `--config-drive true`, if i can make ssh to baremetal instance work... | 10:11 |
*** rcernin has joined #openstack-ironic | 10:19 | |
janders | rpittau regarding tinycore- is there such thing as the most common OS ised as a base of IPA and if so what is it? | 10:27 |
*** xinliang has quit IRC | 10:28 | |
*** k_mouza has joined #openstack-ironic | 10:28 | |
*** tosky has quit IRC | 10:33 | |
*** tosky has joined #openstack-ironic | 10:34 | |
*** k_mouza has quit IRC | 10:35 | |
*** k_mouza has joined #openstack-ironic | 10:35 | |
dtantsur | viks____: you seem to have an old blkid | 10:36 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: ilo: do not change deploy_boot_mode in instance_info https://review.opendev.org/c/openstack/ironic/+/773631 | 10:41 |
iurygregory | good morning rpittau arne_wiebalck dtantsur janders and Ironic o/ | 10:54 |
arne_wiebalck | hey janders and iurygregory o/ | 11:05 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Make boot_mode more consistent with other capabilities https://review.opendev.org/c/openstack/ironic/+/773636 | 11:07 |
*** rcernin has quit IRC | 11:13 | |
*** lmcgann has joined #openstack-ironic | 11:14 | |
viks____ | dtantsur: i use `CentOS Linux release 7.9` it has default version 2.23. i could not find any newer version in centos 7 repo | 11:18 |
*** uzumaki has joined #openstack-ironic | 11:32 | |
*** rcernin has joined #openstack-ironic | 11:35 | |
*** hoonetorg has quit IRC | 11:38 | |
*** sshnaidm|ruck is now known as sshnaidm|afk | 11:38 | |
*** k_mouza has quit IRC | 11:46 | |
rpittau | janders: if you see in ipa-builder we build ipa images using different distros as base, like centos, debian, ubuntu, and fedora. These are better shaped for production usage, tinyipa is usually just used in CI for testing purpose | 11:51 |
*** mkrai_ has quit IRC | 11:53 | |
dtantsur | This looks like iSCSI deploy, so the host OS | 11:59 |
dtantsur | I don't think we support CentOS 7 as a host OS at this point | 12:00 |
*** k_mouza has joined #openstack-ironic | 12:00 | |
*** rcernin has quit IRC | 12:04 | |
*** k_mouza has quit IRC | 12:07 | |
*** k_mouza has joined #openstack-ironic | 12:07 | |
rpittau | viks____: the '-p' option in blkid is available in version 2.23 though | 12:08 |
*** iurygregory_ has joined #openstack-ironic | 12:09 | |
*** iurygregory has quit IRC | 12:09 | |
rpittau | viks____: looking at the logs you posted it seems you have a quite older version (1.0.0) | 12:11 |
openstackgerrit | Merged openstack/ironic stable/victoria: Refactor vendor detection and add Redfish implementation https://review.opendev.org/c/openstack/ironic/+/773388 | 12:25 |
openstackgerrit | Bob Fournier proposed openstack/ironic stable/victoria: For Supermicro BMCs set enable when changing boot device https://review.opendev.org/c/openstack/ironic/+/773656 | 12:29 |
*** sshnaidm|afk is now known as sshnaidm|ruck | 12:36 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Apply force_persistent_boot_device to all boot interfaces https://review.opendev.org/c/openstack/ironic/+/773382 | 12:44 |
*** anuradha1904 has joined #openstack-ironic | 12:51 | |
*** hoonetorg has joined #openstack-ironic | 13:03 | |
*** iurygregory_ is now known as iurygregory | 13:04 | |
viks____ | rpittau: srry.. blkid version is 1.0.0 .. actually when checked with yum, blkid is a part of `util-linux-2.23.2-65`.. i just realized that i have 2 systems, where both of the `util-linux`(2.23.2-65) version is same but not the blkid version.. | 13:05 |
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 | 13:06 |
janders | ack, thank you rpittau | 13:07 |
janders | see you tomorrow Ironic o/ | 13:08 |
rpittau | see ya janders :) | 13:08 |
*** moshiur has quit IRC | 13:10 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Apply force_persistent_boot_device to all boot interfaces https://review.opendev.org/c/openstack/ironic/+/773382 | 13:14 |
iurygregory | bye janders o/ | 13:23 |
*** moshiur has joined #openstack-ironic | 13:31 | |
rpittau | rmrrr the issue with bifrost in ussuri seems due to a bug in ansible..... | 13:52 |
*** ociuhandu has quit IRC | 13:53 | |
dtantsur | \o/ | 13:57 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost stable/ussuri: Fix CI issues https://review.opendev.org/c/openstack/bifrost/+/766742 | 13:59 |
openstackgerrit | Mudit proposed openstack/ironic master: More GPU support in idrac-wsman inspect interface https://review.opendev.org/c/openstack/ironic/+/773359 | 13:59 |
rpittau | problem is we used ansible 2.8 in ussuri, which is an undead version since its maintenance stopped in november 2019 | 14:00 |
rpittau | EOL 21 january 2021 | 14:00 |
rpittau | oh well, let's see how the CI goes first | 14:00 |
*** paras333 has joined #openstack-ironic | 14:04 | |
*** rloo has joined #openstack-ironic | 14:06 | |
*** dsneddon has quit IRC | 14:07 | |
*** ociuhandu has joined #openstack-ironic | 14:10 | |
TheJulia | well, maybe kill ussuri bifrost jobs at this point? | 14:11 |
rpittau | heh I wouldn't say no to that, but aren't we suppose to keep supporting ussuri ? | 14:16 |
*** tzumainn has joined #openstack-ironic | 14:26 | |
TheJulia | is it realistically to make things work that just will not work? | 14:27 |
TheJulia | especially on a master focused tool | 14:27 |
iurygregory | can't we switch to 2.9? | 14:27 |
rpittau | that's what I'm doing, I switched to 2.9 | 14:28 |
iurygregory | would be a problem because of requirements... | 14:28 |
iurygregory | ? | 14:28 |
iurygregory | oh nice! | 14:28 |
rpittau | it's a requirement, but it's broken, so I'm not sure here to be honest | 14:29 |
iurygregory | yep =) | 14:29 |
*** ociuhandu has quit IRC | 14:30 | |
rpittau | TheJulia: I understand what you mean and I agree, I don't want to keep it forcefully alive, also considering that we'll have also to switch to centos8-stream, so if we want to abandon it I'm fine with that | 14:31 |
*** ociuhandu has joined #openstack-ironic | 14:31 | |
*** ociuhandu has quit IRC | 14:36 | |
*** uzumaki has quit IRC | 14:36 | |
TheJulia | we can only do so much | 14:37 |
TheJulia | Looks like monday works best for a review jam... Well, Steve's tuesday. | 14:45 |
*** ociuhandu has joined #openstack-ironic | 14:48 | |
TheJulia | tosin: Just fyi, I have a doctor's appointment I need to go to in about an hour, I pushed our call back a half hour, but I may miss it since I will be at the mercy of the doctor's office | 14:48 |
* dtantsur approves of this goal https://review.opendev.org/c/openstack/governance/+/770616 | 14:51 | |
dtantsur | TheJulia: killing jobs for bifrost for ussuri = EOL ussuri | 14:51 |
dtantsur | not impossible, but violates what openstack expects us to | 14:51 |
dtantsur | (bifrost is different in this regard, you cannot assume it probably still works even if devstack is broken) | 14:52 |
dtantsur | speaking of bifrost, https://review.opendev.org/c/openstack/governance/+/770855 may affect us | 14:52 |
dtantsur | (not literally affect, as in the CI breakage, but spiritually) | 14:53 |
rpittau | spiritually... yeah... wow.... | 14:54 |
dtantsur | incidentally, opensuse is broken in bifrost right now | 14:54 |
dtantsur | which makes me want to Just Do It | 14:54 |
rpittau | I (sadly) approve | 14:54 |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Change metalsmith to centos8+uefi job by default https://review.opendev.org/c/openstack/metalsmith/+/773701 | 14:54 |
tosin | TheJulia: oh that's fine! I'll just stay on the call. All the best at the doctor's =) | 14:55 |
TheJulia | Am I advocating killing all jobs? | 14:55 |
rpittau | mmmm | 14:55 |
rpittau | ok, I guess it's reasonable to remove centos8 job from ussuri in bifrost considering all the changes? | 14:55 |
dtantsur | IMO no | 14:56 |
TheJulia | Is that the job that is failing? | 14:56 |
rpittau | yeah | 14:56 |
TheJulia | have we identified why it is failing? | 14:56 |
dtantsur | it means that bifrost ussuri is broken on at least (!) one of the two primary OS | 14:56 |
rpittau | the problem is not centos per-se, it's ansible | 14:56 |
TheJulia | OSes change and evolve, their packages change | 14:57 |
TheJulia | The stable commitment OSes HAD is gone. | 14:57 |
rpittau | ansible 2.8 to be precise | 14:57 |
dtantsur | 2.9 doesn't fix it? | 14:57 |
dtantsur | I'm much more open to updating the ansible version than to drop a supported OS | 14:57 |
rpittau | dtantsur: locally it seems to work, I'm waiting for CI | 14:57 |
dtantsur | (although I guess CentOS 8 won't gain a lot of popularity. sigh. SIGH) | 14:58 |
*** ociuhandu has quit IRC | 14:58 | |
rpittau | mmm I think it made it worse :D | 14:58 |
dtantsur | \o/ | 14:58 |
dtantsur | well, NOW you can EOL it :D | 14:58 |
TheJulia | \o/ | 14:58 |
rpittau | oh ok ok | 14:59 |
rpittau | it's a stupid filter, maybe I can fix it losing a reasonable amount of neurons | 14:59 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Swap Metalsmith job out for centos8-uefi https://review.opendev.org/c/openstack/ironic/+/773702 | 15:02 |
*** MentalSiege has joined #openstack-ironic | 15:03 | |
*** nam-est has joined #openstack-ironic | 15:06 | |
nam-est | Hi all, please give some review to this: https://github.com/metal3-io/ironic-image/pull/230. Thank you | 15:07 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Remove ironic-tempest-ipa-partition-pxe_ipmitool from gate https://review.opendev.org/c/openstack/ironic/+/773705 | 15:12 |
*** MentalSiege has quit IRC | 15:14 | |
TheJulia | Looks like ironic-tempest-partition-bios-redfish-pxe is redundant, but we use it on sushy instead of the standalone job | 15:15 |
*** MentalSiege has joined #openstack-ironic | 15:15 | |
*** ociuhandu has joined #openstack-ironic | 15:18 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: DNM test CI https://review.opendev.org/c/openstack/bifrost/+/773706 | 15:19 |
dtantsur | TheJulia: well, that's easy to fix | 15:19 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost stable/ussuri: Fix CI issues https://review.opendev.org/c/openstack/bifrost/+/766742 | 15:19 |
dtantsur | (left a comment re pxe_ipmitool job) | 15:19 |
TheJulia | hmm, I mus thave looked at the line above or something | 15:21 |
*** ociuhandu has quit IRC | 15:22 | |
*** ociuhandu has joined #openstack-ironic | 15:23 | |
dtantsur | I *think* we test cleaning bifrost, but it doesn't cover neutron integration, soo... | 15:23 |
*** ociuhandu has quit IRC | 15:23 | |
dtantsur | * in bifrost | 15:23 |
dtantsur | weekend project: finish bifrost neutron integration :) | 15:24 |
TheJulia | The whole intent of the standalone jobs was to peel back all of single scenario jobs | 15:24 |
* TheJulia ponders | 15:25 | |
*** ociuhandu has joined #openstack-ironic | 15:25 | |
dtantsur | I think if we could sit down with a whiteboard, plenty of time and whiskey, we would solve this problem | 15:25 |
TheJulia | would a virtual whiteboard work? | 15:27 |
TheJulia | you know, we have an automated clean knob..... | 15:28 |
TheJulia | we coul dmove it into tempest | 15:28 |
dtantsur | not for me :( I cannot feel thoughts connecting over the wires any more | 15:28 |
dtantsur | sometimes it feels like I'm talking to my laptop the whole day | 15:28 |
TheJulia | I know this feeling really well :( | 15:28 |
*** ociuhandu has quit IRC | 15:31 | |
*** ociuhandu has joined #openstack-ironic | 15:31 | |
openstackgerrit | Dmitry Tantsur proposed openstack/sushy master: Automatically retry HTTP 5xx on GET requests https://review.opendev.org/c/openstack/sushy/+/773711 | 15:35 |
*** uzumaki has joined #openstack-ironic | 15:38 | |
*** ociuhandu has quit IRC | 15:41 | |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost stable/ussuri: Fix CI issues https://review.opendev.org/c/openstack/bifrost/+/766742 | 15:43 |
*** uzumaki has quit IRC | 15:55 | |
*** uzumaki has joined #openstack-ironic | 15:57 | |
openstackgerrit | Merged openstack/ironic master: Fixes issue of redfish firmware update https://review.opendev.org/c/openstack/ironic/+/764914 | 15:58 |
*** rloo has quit IRC | 16:00 | |
*** rloo has joined #openstack-ironic | 16:00 | |
openstackgerrit | Aija Jauntēva proposed openstack/sushy master: Fix Bad request error when checking task status https://review.opendev.org/c/openstack/sushy/+/773724 | 16:15 |
openstackgerrit | Merged openstack/ironic master: Pass context objects directly to policy enforcement https://review.opendev.org/c/openstack/ironic/+/772086 | 16:17 |
*** lbragstad_ is now known as lbragstad | 16:21 | |
*** k_mouza has quit IRC | 16:25 | |
*** Qianbiao has quit IRC | 16:32 | |
*** ociuhandu has joined #openstack-ironic | 16:34 | |
*** ociuhandu has quit IRC | 16:35 | |
*** ociuhandu has joined #openstack-ironic | 16:35 | |
*** k_mouza has joined #openstack-ironic | 16:52 | |
TheJulia | tosin: just making ocffee, about five minutes | 17:01 |
TheJulia | coffee | 17:01 |
tosin | TheJulie: ok😄 | 17:03 |
openstackgerrit | Verification of a change to openstack/ironic failed: Apply force_persistent_boot_device to all boot interfaces https://review.opendev.org/c/openstack/ironic/+/773382 | 17:04 |
*** lucasagomes has quit IRC | 17:04 | |
*** ociuhandu has quit IRC | 17:06 | |
*** ociuhandu has joined #openstack-ironic | 17:07 | |
*** ociuhandu has quit IRC | 17:13 | |
*** rloo has quit IRC | 17:18 | |
*** rloo has joined #openstack-ironic | 17:19 | |
*** ociuhandu has joined #openstack-ironic | 17:19 | |
*** rloo has quit IRC | 17:22 | |
*** rloo has joined #openstack-ironic | 17:22 | |
*** ociuhandu has quit IRC | 17:25 | |
*** dougsz has quit IRC | 17:33 | |
*** ociuhandu has joined #openstack-ironic | 17:36 | |
*** ociuhandu has quit IRC | 17:40 | |
*** ociuhandu has joined #openstack-ironic | 17:52 | |
*** k_mouza has quit IRC | 17:53 | |
arne_wiebalck | bye everyone o/ | 17:56 |
*** ociuhandu has quit IRC | 17:59 | |
*** rcernin has joined #openstack-ironic | 17:59 | |
*** ociuhandu has joined #openstack-ironic | 18:02 | |
*** derekh has quit IRC | 18:04 | |
*** rcernin has quit IRC | 18:04 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Apply force_persistent_boot_device to all boot interfaces https://review.opendev.org/c/openstack/ironic/+/773382 | 18:05 |
*** rloo has quit IRC | 18:05 | |
*** rloo has joined #openstack-ironic | 18:06 | |
*** ociuhandu has quit IRC | 18:06 | |
*** k_mouza has joined #openstack-ironic | 18:09 | |
*** ociuhandu has joined #openstack-ironic | 18:15 | |
*** ociuhandu has quit IRC | 18:15 | |
*** ociuhandu has joined #openstack-ironic | 18:15 | |
bfournie | Hi TheJulia: for the BIOS registry stuff, the response from the bmc for BIOS registry is about 50k compressed. Do we have any concerns with storing this in the database? do we handle caching large entries like this elsewhere? | 18:16 |
TheJulia | bfournie: storing in the db PER node?!? | 18:22 |
TheJulia | Why would we cache it? | 18:23 |
TheJulia | Do we need it? | 18:23 |
*** k_mouza has quit IRC | 18:23 | |
*** ociuhandu has quit IRC | 18:27 | |
bfournie | TheJulia: so if we are going to add api to return the BIOS registry (per mid cycle discussion) I don't think we'd want to do it synchronously, we'd have to cache it, but we wouldn't need to do it per node - all vendors of same type will be using same registry so we'd only have to cache unique ones | 18:29 |
*** ociuhandu has joined #openstack-ironic | 18:30 | |
*** ociuhandu has quit IRC | 18:30 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Apply force_persistent_boot_device to all boot interfaces https://review.opendev.org/c/openstack/ironic/+/773382 | 18:30 |
*** sshnaidm|ruck is now known as sshnaidm|afk | 18:34 | |
*** dtantsur is now known as dtantsur|afk | 18:35 | |
dtantsur|afk | o/ | 18:35 |
rpittau | good night! o/ | 18:39 |
*** rpittau is now known as rpittau|afk | 18:39 | |
*** moshiur has quit IRC | 19:30 | |
*** dsneddon has joined #openstack-ironic | 19:56 | |
*** rcernin has joined #openstack-ironic | 20:00 | |
iurygregory | ajya, hey you around? | 20:01 |
*** rcernin has quit IRC | 20:04 | |
iurygregory | the problem with eHTML5 http://paste.openstack.org/show/802256/ looks like this ? =) | 20:08 |
*** tosin has quit IRC | 20:19 | |
*** rcernin has joined #openstack-ironic | 20:25 | |
*** hoonetorg has quit IRC | 20:38 | |
*** hoonetorg has joined #openstack-ironic | 20:51 | |
*** rcernin has quit IRC | 21:22 | |
*** rcernin has joined #openstack-ironic | 21:52 | |
*** ociuhandu has joined #openstack-ironic | 22:01 | |
*** ociuhandu has quit IRC | 22:05 | |
*** rcernin has quit IRC | 22:08 | |
*** rcernin has joined #openstack-ironic | 22:09 | |
TheJulia | stevebaker: you around? | 22:37 |
stevebaker | TheJulia: yep I'm just talking to alexmcleod | 22:37 |
TheJulia | So I'm trying to figure out how to replace the hard coded project name with a project id. Figured out importlib.reload() for testing, but I'm struggling to grok the policy testing you wrote and why its not working so thinking >1 brains may help | 22:39 |
*** hoonetorg has quit IRC | 22:40 | |
stevebaker | TheJulia: ok, where should I look to see the problem? | 22:44 |
janders | good morning Ironic o/ | 22:44 |
TheJulia | stevebaker: let me go ahead and upload the stack at this point, one minute | 22:45 |
TheJulia | stevebaker: git review is thinking | 22:47 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for volume APIs https://review.opendev.org/c/openstack/ironic/+/763273 | 22:47 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for conductor APIs https://review.opendev.org/c/openstack/ironic/+/763274 | 22:47 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for the allocation APIs https://review.opendev.org/c/openstack/ironic/+/763275 | 22:47 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for the event APIs https://review.opendev.org/c/openstack/ironic/+/763276 | 22:47 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implement secure RBAC for the deploy templates APIs https://review.opendev.org/c/openstack/ironic/+/763277 | 22:47 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: support specific project id for legacy mode https://review.opendev.org/c/openstack/ironic/+/772438 | 22:47 |
TheJulia | stevebaker: the test on the wip which should be the very last patch is ironic.tests.unit.common.test_policy.PolicyInCodeTestCase.test_creds_18 | 22:47 |
stevebaker | ok | 22:47 |
TheJulia | I can see the reload works, the rule changes | 22:49 |
TheJulia | maybe I'm missing something | 22:49 |
stevebaker | TheJulia: using importlib sounds scary | 22:51 |
TheJulia | well, there is no real way to test this without refactoring the entire policy structure | 22:52 |
TheJulia | Maybe we just change it and leave it be?! | 22:52 |
TheJulia | and not actually have tests for the conditional | 22:52 |
TheJulia | or maybe I'm over thinking it | 22:52 |
stevebaker | TheJulia: do you get the reload you need by setting policy._ENFORCER to None then calling policy.get_enforcer()? | 22:54 |
TheJulia | the lists are created on module load | 22:54 |
TheJulia | since it loads default_policy | 22:55 |
TheJulia | policy.default_policy | 22:55 |
*** uzumaki has quit IRC | 22:58 | |
stevebaker | TheJulia: yeah I see. I guess the only alternative would be to wrap those lists in a function, but that seems a lot just for testing | 22:58 |
TheJulia | yeah | 22:59 |
TheJulia | it is a conundrum | 22:59 |
*** anuradha1904 has quit IRC | 23:01 | |
stevebaker | TheJulia: are policy.RuleDefault objects mutable? If so you could change it in list_policies based on conf, then to the _ENFORCER = None thing from the test | 23:03 |
TheJulia | I don't know, not sure it would be right for it to be mutable in the grand scheme of things | 23:04 |
TheJulia | security wise | 23:04 |
stevebaker | heh | 23:04 |
TheJulia | lbragstad: oh lance! | 23:05 |
stevebaker | or you could find it in the list and replace it with a new one | 23:05 |
lbragstad | ? | 23:05 |
lbragstad | i believe that are mutable | 23:06 |
lbragstad | at least i'm not aware of any code in oslo.policy that prevents it (for better or worse) | 23:06 |
TheJulia | hmm | 23:07 |
TheJulia | I guess that could work. and just have a unit test to make sure the startup code on module load behaves as expected | 23:08 |
*** JayF has quit IRC | 23:13 | |
*** k_mouza has joined #openstack-ironic | 23:16 | |
*** JayF has joined #openstack-ironic | 23:17 | |
*** k_mouza has quit IRC | 23:20 | |
openstackgerrit | Jacob Anders proposed openstack/ironic master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic/+/771905 | 23:40 |
openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent-builder master: Add nvme-cli package to IPA image https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/773538 | 23:50 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!