| gmann | I understand that, key thing is if we do and even provide all the migration way or use case, we are going to break the operator existing way and its hard to ask them to switch to the new way. | 00:05 |
|---|---|---|
| gmann | KDDI feedback (written in etherpad) and most of operator are like "we need single admin to do the things in entire deployment" - https://etherpad.opendev.org/p/rbac-operator-feedback#L88 | 00:06 |
| gmann | they know it is not the best way but they are hesitant to improve it. | 00:07 |
| gmann | anyways let's discuss in goal patch, i replied there. need to go out. | 00:08 |
| TheJulia | gmann: ack, I guess there is an associated cost we're all paying with the increased debt by not forcing change/evolution. I feel like that might have been part of the purpose originally, and maybe we've strayed from that. Who knows. Have a wonderful evening! | 00:58 |
| opendevreview | SONG SHUKUN proposed openstack/ironic master: [iRMC] Add SNMPv3 authentication functionality https://review.opendev.org/c/openstack/ironic/+/845347 | 01:22 |
| *** akahat is now known as akahat|ruck | 06:47 | |
| *** TimBeermann[m] is now known as tibeer[matrix] | 06:58 | |
| *** tibeer[matrix] is now known as TimBeermann[m] | 06:59 | |
| opendevreview | Thomas Goirand proposed openstack/ironic-ui master: Django 4.x: ugettext_lazy is removed https://review.opendev.org/c/openstack/ironic-ui/+/848236 | 09:10 |
| zigo | TheJulia: Please merge this in, it unfixed building my ironic-ui package in Debian unstable (which contains Django 4.1). | 09:12 |
| opendevreview | waleed mousa proposed openstack/ironic-python-agent master: Fix passing kwargs in clean steps https://review.opendev.org/c/openstack/ironic-python-agent/+/845583 | 09:13 |
| opendevreview | Merged openstack/ironic master: Fix Redfish RAID to update raid_config https://review.opendev.org/c/openstack/ironic/+/842979 | 09:50 |
| *** akahat is now known as akahat|ruck | 10:08 | |
| pamsoo | hi. I have openstack victoria environment and I'm trying to deploy on a server which has nvme devices. It looks that the nvme devices are not detected by IPA. | 10:37 |
| pamsoo | IPA said "Could not find the SCSI address (HCTL) for device /dev/nvme0n1. Skipping: FileNotFoundError: [Errno 2] No such file or directory: '/sys/block/nvme0n1/device/scsi_device'" | 10:37 |
| pamsoo | are nvme devices unusable for root device? | 10:38 |
| arne_wiebalck | mraineri_: Mike, are there any plans to package the redfish tacklebox as an rpm? | 10:38 |
| pamsoo | any answer would be greatly appreciated. | 10:42 |
| iurygregory | good morning Ironic | 11:26 |
| iurygregory | zigo, I would +2 the ironic-ui patch but CI failed .-. (maybe is just a random failure) | 11:27 |
| iurygregory | pamsoo, so nvme devices can be used as root device | 11:30 |
| iurygregory | but seems like you IPA is not able to find it .-. | 11:30 |
| iurygregory | did you build the IPA ramdisk on you own or you are using the one from tarballs ? | 11:31 |
| pamsoo | i'm using the one from tarballs. | 11:32 |
| pamsoo | I'm now trying to deploy with root device hint. | 11:32 |
| pamsoo | I think that nvme devices are also stored in block_devices (appeared in ironic_python_agent/hardware.py) but one of them is not selected. | 11:36 |
| arne_wiebalck | pamsoo: I don't think this is a fatal error, but only a warning. | 11:48 |
| arne_wiebalck | pamsoo: I have these messages in my logs as well. | 11:49 |
| arne_wiebalck | pamsoo: My point is: this may be a false lead. | 11:49 |
| arne_wiebalck | pamsoo: in the clean logs tarball, there is the lsblk output, does that list your devices? | 11:56 |
| zigo | iurygregory: The log says there's an invalid char in openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/flavor/flavor.controller.js:116 this is really unrelated to my patch. | 11:59 |
| zigo | Is ironic-ui's CI broken somehow ? | 11:59 |
| iurygregory | zigo, we don't push a lot of patches in ironic-ui... normally it just works | 11:59 |
| iurygregory | can you push a DNM so we can test? | 11:59 |
| opendevreview | Aija Jauntēva proposed openstack/ironic stable/yoga: Fix Redfish RAID to update raid_config https://review.opendev.org/c/openstack/ironic/+/848128 | 12:04 |
| pamsoo | arne_wiebalck, iurygregory: Thanks for advices. I'll try more. (If there were no try, I'll do. :-P) | 12:14 |
| iurygregory | =) | 12:33 |
| TheJulia | good morning | 12:50 |
| iurygregory | good morning TheJulia =) | 12:53 |
| TheJulia | hmm... it looks like openstack_dashboards has an isse | 12:56 |
| TheJulia | uhh yup | 13:01 |
| TheJulia | we... have... better.. testing than horizon | 13:01 |
| TheJulia | oh god | 13:01 |
| TheJulia | https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/flavor/flavor.controller.js#L116 | 13:01 |
| iurygregory | WOOT?! | 13:02 |
| iurygregory | I'm in shock by "we... have... better.. testing than horizon" | 13:02 |
| TheJulia | cloning horizon, will post a patch shortly | 13:02 |
| iurygregory | shocked* | 13:03 |
| TheJulia | hmm, javascript | 13:05 |
| TheJulia | oh, I see the comment | 13:06 |
| TheJulia | ugh | 13:06 |
| opendevreview | Julia Kreger proposed openstack/ironic-ui master: Update npm packages for linting https://review.opendev.org/c/openstack/ironic-ui/+/848294 | 13:11 |
| TheJulia | I *hope* that fixes it | 13:11 |
| TheJulia | it has been a very long time since I hacked on that code | 13:11 |
| TheJulia | well, the error is different. That is a good sign | 13:19 |
| * TheJulia schedules a meeting first | 13:25 | |
| TheJulia | before resuming | 13:25 |
| mraineri_ | arne_wiebalck: Just saw your note; no plans to push Tacklebox as an RPM, but I don't see why we couldn't do that | 13:34 |
| mraineri_ | I assume you just mean to package it and post the RPM on GitHub and not necessarily push it to other package repositories? | 13:35 |
| *** mraineri_ is now known as mraineri | 13:36 | |
| mraineri | Looks like those files could be pushed to PyPI too if needed | 13:51 |
| kubajj | loop | 14:05 |
| opendevreview | Julia Kreger proposed openstack/ironic-ui master: Update npm packages for linting https://review.opendev.org/c/openstack/ironic-ui/+/848294 | 14:13 |
| opendevreview | Julia Kreger proposed openstack/ironic-ui master: Update karma and related npm package versions https://review.opendev.org/c/openstack/ironic-ui/+/848299 | 14:13 |
| opendevreview | Julia Kreger proposed openstack/ironic-ui master: Update npm packages for linting https://review.opendev.org/c/openstack/ironic-ui/+/848294 | 14:29 |
| opendevreview | Julia Kreger proposed openstack/ironic-ui master: Update npm packages for linting https://review.opendev.org/c/openstack/ironic-ui/+/848294 | 14:48 |
| mraineri | arne_wiebalck: If we were to build RPM packages, would it be more useful to distribute them on PyPI or attach them to GitHub releases? | 14:55 |
| dmellado | It can be done both ways. Personally, I'd go for pypi then rpm that | 14:59 |
| ajya | iurygregory, TheJulia and anyone else around, can this patch be prioritized for review https://review.opendev.org/c/openstack/ironic/+/848107 ? Would like to get it merged and backported. | 15:30 |
| TheJulia | ajya: any library dependencies, or is it just that? | 15:35 |
| TheJulia | ajya: it would be super helpful if once merged, you can do the backports, but it doesn't look like there is any dependency, so it has a +2 from me | 15:38 |
| ajya | TheJulia: no dependencies, thanks | 15:38 |
| iurygregory | ajya, will look at it later today (a lot of downstream things happening today for me) | 15:40 |
| ajya | thanks, iurygregory | 15:42 |
| arne_wiebalck | mraineri: if we had the srpm, we would just rebuild it locally ... or we add it to epel/fedora (not sure what the process is, how licenses are affected etc) | 16:20 |
| TheJulia | iurygregory: I think we're going to have to update our version of angular take care of zigo's problem ultimately | 16:28 |
| TheJulia | iurygregory: I don't think we had any volunteers on the mailing list? | 16:28 |
| iurygregory | we didn't | 16:41 |
| TheJulia | iurygregory: I suspect it might be like a weekend project then | 16:44 |
| TheJulia | iurygregory: I approved the dib change so hopefully we get our gate back in a clean state | 17:13 |
| TheJulia | ajya: if https://review.opendev.org/c/openstack/ironic/+/848128 needs to go to other branches, like bugfix branches or to wallaby, you may want to create the backports | 17:16 |
| ajya | TheJulia: yup, in process, older branches need to have merge conflicts resolved | 17:19 |
| TheJulia | okay | 17:19 |
| opendevreview | Verification of a change to openstack/ironic master failed: [Minor] Fix misspellings of "insufficient" https://review.opendev.org/c/openstack/ironic/+/845789 | 17:39 |
| mraineri | arne_wiebalck: I may need some help getting things sorted on which RPM you'd like to be built and where to place it. It seems like if I simply add "bdist_rpm" to our build process, it should push src.rpm and noarch.rpm to PyPI along with the rest of the release. If that's all you need, then that's a simple change I can do very quickly. | 17:40 |
| mraineri | If there are other things needed and other locations they should be published, then I'll need some help with that | 17:40 |
| arne_wiebalck | mraineri: we have some colleagues here which build images for Fedora, let me check with them what to do ... thanks for your help so far! | 19:49 |
| arne_wiebalck | mraineri: I will let you know | 19:49 |
| arne_wiebalck | Bare Metal SIG: would there be interest in a session on July 12th (if I manage to get a speaker) or should we rather break for the summer and resume in September? | 19:50 |
| TheJulia | arne_wiebalck: Are you thinking August is lost to vacations? | 20:02 |
| TheJulia | arne_wiebalck: I think I might actually have a speaker I could line up | 20:02 |
| TheJulia | I just need to circle back to the thread where the idea is planted | 20:03 |
| arne_wiebalck | TheJulia: yeah, we took a break last time since many are away | 20:07 |
| arne_wiebalck | TheJulia: but we don't have to :) | 20:07 |
| TheJulia | I think the earliest I'm going to circle back to that email thread is tomorrow | 20:07 |
| TheJulia | so let me see if the idea resonates and we can go from there | 20:08 |
| arne_wiebalck | sure, I may have a speaker for July, still trying to sort out details | 20:08 |
| arne_wiebalck | TheJulia: thanks! | 20:08 |
| TheJulia | okay, cool! | 20:09 |
| opendevreview | Verification of a change to openstack/ironic master failed: Auto-populate lessee for deployments https://review.opendev.org/c/openstack/ironic/+/818641 | 20:57 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Drop python2 from bindep.txt https://review.opendev.org/c/openstack/ironic-python-agent/+/845371 | 22:26 |
| JayF | TheJulia: I *think* IPA gate is still a little broken, the same job keeps failing | 22:45 |
| JayF | TheJulia: wondering if IPA needs some kind of update you may have already pushed to Ironic-proper, perhaps? | 22:45 |
| JayF | hmm, maybe not always the same one | 22:47 |
| ashinclouds[m] | Got an example link? | 23:08 |
| JayF | the two fails above | 23:13 |
| JayF | are fresh rechecks by me | 23:13 |
| JayF | OH, that's why it looked different | 23:13 |
| JayF | one is ironic one is ipa | 23:13 |
| JayF | but the ipa ones all fail on ipa-tempest-bios-ipmi-direct-src that I've seen | 23:14 |
| JayF | > [ERROR] /opt/stack/ironic/devstack/lib/ironic:2322 Some nodes did not finish cleaning: e9bb2d93-8fd7-4183-9da8-4d1710eece99 | 23:14 |
| JayF | is the error at the end, but I know that's semi-useless | 23:14 |
| JayF | oh, you probably wanted this -> https://zuul.opendev.org/t/openstack/build/5a3c06b14cf048f5b115d009098ccc27 | 23:17 |
| JayF | looks like lots of errors related to bad networking on node 1 | 23:20 |
| JayF | https://zuul.opendev.org/t/openstack/build/5a3c06b14cf048f5b115d009098ccc27/log/controller/logs/ironic-bm-logs/node-1_no_ansi_2022-06-30-21:37:44_log.txt | 23:20 |
| TheJulia | jmmm | 23:25 |
| * JayF saw centos 9 stream + NetworkManager and lolsob'd | 23:25 | |
| TheJulia | i bet we need a dib release | 23:26 |
| JayF | that's a good thought | 23:28 |
| JayF | so was ironic itself never broken with this? | 23:29 |
| JayF | because there's no reason ipa would need a dib release and ironic wouldn't've | 23:29 |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Pull in diskimage-builder until new release is cut https://review.opendev.org/c/openstack/ironic/+/848345 | 23:29 |
| TheJulia | that *should* fix things for now | 23:29 |
| JayF | ima dep that bindep one on that change | 23:30 |
| JayF | to see if it fixes | 23:30 |
| JayF | will a commit message change remove the +A? | 23:30 |
| TheJulia | yes, but as a core you can just re-approve | 23:30 |
| TheJulia | just leave a comment as to why | 23:30 |
| JayF | ack | 23:30 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Drop python2 from bindep.txt https://review.opendev.org/c/openstack/ironic-python-agent/+/845371 | 23:32 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Drop python2 from bindep.txt https://review.opendev.org/c/openstack/ironic-python-agent/+/845371 | 23:33 |
| opendevreview | Merged openstack/ironic-python-agent master: Use json for lsblk output https://review.opendev.org/c/openstack/ironic-python-agent/+/775391 | 23:38 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!