| rpittau | fungi: great news, thanks a bunch :) | 08:05 |
|---|---|---|
| rpittau | good morning ironic! o/ | 08:05 |
| espenfl | Hi there. I just made some updates to the bootloader element in DIB and would like to submit it for review. I understand what needs to be done is just to make a sensible series of commits and launch git review? I honestly have no experience with Gerrit, so sorry for the stupid question. | 08:29 |
| frickler | espenfl: maybe https://docs.opendev.org/opendev/infra-manual/latest/developers.html and https://docs.openstack.org/ironic/latest/contributor/contributing.html can help | 08:43 |
| espenfl | Thanks frickler. I also have a question regarding the setup of the root filesystem that is put into the kernel boot line when building images with DIB. Out of the box, it somehow ends up being /dev/mapper/loop6p1. This I guess is the loop used when setting up the image. It should preferably be set to the UUID I think. After being put on the disk, it of course fails to boot | 08:52 |
| espenfl | and drops to the shell. Any advice on how to address this? | 08:52 |
| espenfl | For reference I build with the elements: ubuntu vm openssh-server dynamic-login dhcp-all-interfaces mellanox bootloader block-device-mbr | 08:53 |
| espenfl | Guess this is related to the bootloader element. | 08:53 |
| espenfl | Found this: https://opendev.org/openstack/diskimage-builder/src/commit/174089a6a5f1ff4628a9d103e4328a1b1298e241/diskimage_builder/elements/bootloader/finalise.d/50-bootloader#L85. Maybe that should just be GRUB_DEVICE_LABEL? | 09:21 |
| espenfl | neh, it is correct I think, but it still is not set right | 09:24 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Support ansible-collections-openstack 2 and later https://review.opendev.org/c/openstack/bifrost/+/875750 | 09:29 |
| espenfl | Okey, it was a mistake on my part. Sorry for the noise. Will try to submit to Gerrit using linked description today or tomorrow. | 09:46 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Do not move nodes to CLEAN FAILED with empty last_error https://review.opendev.org/c/openstack/ironic/+/874784 | 10:19 |
| dtantsur | I think I got to the bottom of it ^^^ | 10:19 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic-ui/+/875867 | 10:48 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic-ui/+/875868 | 10:48 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/ironic-ui/+/875869 | 10:48 |
| iurygregory | good morning Ironic | 11:10 |
| opendevreview | Merged openstack/networking-generic-switch master: Add Fake device type https://review.opendev.org/c/openstack/networking-generic-switch/+/873098 | 11:24 |
| opendevreview | Merged openstack/networking-generic-switch master: Add ngs-stress test script https://review.opendev.org/c/openstack/networking-generic-switch/+/874789 | 11:24 |
| opendevreview | Merged openstack/ironic-ui master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/ironic-ui/+/875869 | 11:50 |
| opendevreview | Merged openstack/ironic-ui stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic-ui/+/875867 | 11:50 |
| opendevreview | Merged openstack/ironic-ui stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic-ui/+/875868 | 11:50 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/sushy master: Handle non-default language for registries https://review.opendev.org/c/openstack/sushy/+/872049 | 11:59 |
| iurygregory | dtantsur, I just added https://review.opendev.org/c/openstack/ironic/+/874784 to the ironic-week-prio | 13:06 |
| iurygregory | I will try to take a look today | 13:07 |
| dtantsur | thx! | 13:07 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/875901 | 13:15 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/875902 | 13:15 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/875903 | 13:15 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Refactoring: extract some common functions from the inspector code https://review.opendev.org/c/openstack/ironic/+/875915 | 13:22 |
| rpittau | well we have bifrost CI working with ansible-collections-openstack 2 now, just checking the failed non voting jobs but so far it seems they're not failing for the change https://review.opendev.org/c/openstack/bifrost/+/875750 | 13:25 |
| dtantsur | w00t | 13:28 |
| rpittau | going to submit a couple of fixes in parallel, we have some non voting jobs failing since a while for other reasons | 13:29 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Fix enabling epel repo for rpm distributions https://review.opendev.org/c/openstack/bifrost/+/875929 | 13:46 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Restructure the inspector module in preparation for its expansion https://review.opendev.org/c/openstack/ironic/+/874677 | 13:51 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Refactoring: extract some common functions from the inspector code https://review.opendev.org/c/openstack/ironic/+/875915 | 13:51 |
| fungi | JayF: rpittau: right, i assume you'll want to follow up with governance adoption and a project rename once we confirm ironic control of the repository, but i'll let you drive those pieces (happy to provide pointers when the time comes of course) | 14:03 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-lib master: Use direct writes when erasing the GPT https://review.opendev.org/c/openstack/ironic-lib/+/875931 | 14:13 |
| TheJulia | good morning | 14:32 |
| iurygregory | good morning TheJulia | 14:33 |
| arne_wiebalck | Good morning TheJulia and iurygregory o/ | 14:33 |
| TheJulia | just what I needed this morning.... crashing laptop | 14:42 |
| iurygregory | arne_wiebalck, o/ | 14:42 |
| iurygregory | oh ouch! | 14:42 |
| * TheJulia can't decide if power button or terminal window to kill -9 the browser PID will be faster | 14:43 | |
| TheJulia | and power button it is | 14:43 |
| opendevreview | Furkat proposed openstack/sushy-tools master: Add fake_ipa to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 15:07 |
| * TheJulia did not order a month of rain in the last 2 hours | 15:14 | |
| dtantsur | wanna see something crazy? | 16:03 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Migrate the inspector's /continue API https://review.opendev.org/c/openstack/ironic/+/875944 | 16:03 |
| dtantsur | here goes ^^^ :) | 16:04 |
| dtantsur | I'm interested in feedback on both the API/RPC/driver design AND the idea to have the API before the new interface implementation | 16:06 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Fix enabling epel repo for rpm distributions https://review.opendev.org/c/openstack/bifrost/+/875929 | 16:15 |
| JayF | we should consider enabling this: https://review.opendev.org/c/openstack/tempest/+/873653 | 16:21 |
| JayF | it came up in TC meeting | 16:21 |
| JayF | they added a setting you can flip to make MYSQL use less ram, and leave more ram available to other things | 16:21 |
| dtantsur | mmm, neat! | 16:21 |
| JayF | would be interesting to see if it helps us w/memory constraints | 16:21 |
| JayF | apparently some Nova jobs were OOM'ing; this solved it | 16:22 |
| JayF | so not exactly aligned to us but anything that can get us more ram for fake-hardware-vms is a win imo | 16:22 |
| rpittau | interesting, although reading the description in devstack makes me wonder :) | 16:26 |
| rpittau | https://github.com/openstack/devstack/blob/master/stackrc#L204 | 16:26 |
| rpittau | this is how it's implemented https://github.com/openstack/devstack/blob/master/lib/databases/mysql#L179 | 16:27 |
| rpittau | we could always give it a try | 16:27 |
| rpittau | good night! o/ | 16:50 |
| arne_wiebalck | bye everyone o/ | 17:18 |
| *** jamesdenton_ is now known as jamesdenton | 17:54 | |
| JayF | I wonder if we have any metal3 users who could be nominated for a superuser award. https://superuser.openstack.org/articles/2023-superuser-awards-nominations-open/ | 18:17 |
| JayF | it'd get more visibility on our k8s integrations | 18:17 |
| TheJulia | I suspect this might go back to the classical feedback cycle loop issue | 19:09 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/875901 | 20:38 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/875902 | 20:41 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/875903 | 20:44 |
| fungi | rpittau: when you wake up, one of the virtualpdu maintainers added you to the core and release groups for it already, so shouldn't require any intervention on my part now | 23:08 |
| JayF | \o/ | 23:19 |
| TheJulia | woot! | 23:26 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Support batching up commands https://review.opendev.org/c/openstack/networking-generic-switch/+/743283 | 23:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!