| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-04-04 | |||
| 18:29:20 | sean-k-mooney | Uggla: you used `'` not `"` | |
| 18:29:31 | sean-k-mooney | they are json so the have to be double quotes | |
| 18:30:06 | dansmith | ooh, yeah I see | |
| 18:31:16 | sean-k-mooney | you get a really unintuivie error | |
| 18:31:18 | sean-k-mooney | BadRequestException: 400: Client Error for url: http://192.168.16.185/compute/v2.1/servers, Invalid PCI alias definition: Expecting property name enclosed in double quotes: line 1 column 2 (char 1) | |
| 18:31:42 | sean-k-mooney | i mean it makes sense but i dont expect end users to get that excption | |
| 18:31:56 | sean-k-mooney | wehn its an error in your alias in the config file | |
| 18:32:44 | sean-k-mooney | its really a server side error not a client side one | |
| 18:33:11 | dansmith | ah yeah, exposing that to the user is probably not a great idea either | |
| 18:33:59 | sean-k-mooney | it might be becaus i did that as admin | |
| 18:34:06 | sean-k-mooney | ill check that after im done | |
| 18:36:45 | sean-k-mooney | https://paste.opendev.org/show/bwJuKNE7VW3NvAjnxiWr/ | |
| 18:36:53 | sean-k-mooney | so cool i was able to boot a vm | |
| 18:37:01 | sean-k-mooney | it reserved the device | |
| 18:37:10 | sean-k-mooney | and after deleting it i now get a no valid host | |
| 18:38:01 | dansmith | \o/ | |
| 18:40:07 | sean-k-mooney | and cool i can reset it and boot again | |
| 18:40:37 | dansmith | \o\ /o/ \o/ | |
| 18:41:15 | sean-k-mooney | im too tired this envning to review the final functional test patch but ill do that on monday | |
| 18:41:22 | sean-k-mooney | i was testing with the reno patch just before it | |
| 18:41:41 | dansmith | no worries, good to have another confirmation, thanks | |
| 18:41:45 | dansmith | rest up for next week ;) | |
| 18:43:35 | sean-k-mooney | o/ | |
| 19:03:42 | dansmith | gmann: melwitt: this recently-added test seems to be failing a *lot* https://review.opendev.org/c/openstack/tempest/+/858885 | |
| 19:04:13 | dansmith | and it seems to be failing because the swap devices don't actually show up or disappear in the guest | |
| 19:05:01 | dansmith | probably because it doesn't wait for SSHABLE I'm guessing | |
| 19:05:37 | sean-k-mooney | that shoudl not matter for rezise since there is no hotplug | |
| 19:05:49 | sean-k-mooney | but i dotn think it will auto swap on | |
| 19:06:11 | sean-k-mooney | so depending on what the test is doing it might now be stable | |
| 19:06:46 | sean-k-mooney | hum blkid looking for type swap | |
| 19:07:02 | sean-k-mooney | do we format the disk with a swap partion | |
| 19:07:10 | dansmith | what I mean is, I'm not sure it's waiting for the instance to have actually been resized | |
| 19:07:20 | sean-k-mooney | oh | |
| 19:07:36 | sean-k-mooney | well that optentaly differnet | |
| 19:08:01 | dansmith | I guess the underlying stuff waits for the verify stage, so perhaps not | |
| 19:08:09 | dansmith | either way, seems to be quite flaky | |
| 19:08:13 | melwitt | sean-k-mooney: swap is formatted | |
| 19:08:19 | sean-k-mooney | ack | |
| 19:08:24 | sean-k-mooney | i tought it was but was not sure | |
| 19:08:59 | sean-k-mooney | im nmot sure why there are reboots in those tests | |
| 19:09:19 | dansmith | me either, but it's failing before the reboot | |
| 19:09:39 | sean-k-mooney | i get why its creating flaovr but im not sure why that part of the resize helper | |
| 19:10:00 | sean-k-mooney | i mena that fine just not what i expect form the function name | |
| 19:10:41 | dansmith | I also don't think the 2048_to_1024 test is doing anything particularly useful since it's not asserting the size of the device | |
| 19:11:39 | sean-k-mooney | https://github.com/openstack/tempest/blob/b1e168015316f3f73131957e9fb6abfd2fdc20f1/tempest/api/compute/base.py#L441-L454 should be confirming as you said | |
| 19:11:50 | dansmith | yeah | |
| 19:12:09 | sean-k-mooney | dansmith: thse looks like the tests were partly ported form the functional tests for the same thing | |
| 19:12:17 | sean-k-mooney | but those had more asserts | |
| 19:13:53 | dansmith | either way, I wonder if we should fast revert that.. it's been merged for less than 24 hours | |
| 19:15:04 | sean-k-mooney | probably if its flaky i say the pin to review a few days ago but didnt really have time to look at it | |
| 19:16:19 | dansmith | proposed https://review.opendev.org/c/openstack/tempest/+/946379 | |
| 19:18:04 | sean-k-mooney | its the remote command | |
| 19:18:37 | dansmith | looks like all the multicell jobs are failing this since it merged (which is only three amazingly) https://zuul.opendev.org/t/openstack/builds?job_name=nova-multi-cell&skip=0 | |
| 19:18:59 | sean-k-mooney | [20:16:32]❯ blkid | |
| 19:19:01 | sean-k-mooney | /dev/nvme0n1p3: UUID="9e5b21f3-dd4d-425a-b309-dcfbf585ad16" TYPE="crypto_LUKS" PARTUUID="82a4c723-1748-43f6-b82d-f051b8d3a6c5" | |
| 19:19:03 | sean-k-mooney | /dev/nvme0n1p1: UUID="F3DE-29C6" BLOCK_SIZE="512" TYPE="vfat" PARTUUID="7208aa8f-8f44-432c-8163-693e4c1dcb8b" | |
| 19:19:05 | sean-k-mooney | /dev/nvme0n1p2: UUID="f84e29ae-bdd6-45a2-9590-f7d9fc71f90c" BLOCK_SIZE="4096" TYPE="ext4" PARTUUID="b2216aac-c4e1-4cf1-930b-37a879cb1cd3" | |
| 19:19:07 | sean-k-mooney | /dev/mapper/nvme0n1p3_crypt: UUID="0ebbc49c-51a0-4b8c-bdc1-702bf9d178b7" UUID_SUB="83086ee4-93b7-49c7-9915-0b8da9589da6" BLOCK_SIZE="4096" TYPE="btrfs" | |
| 19:19:09 | sean-k-mooney | ~ via 🐹 v1.21.11 via 🐍 v3.11.11 | |
| 19:19:11 | sean-k-mooney | [20:16:35]➜ lsblk | |
| 19:19:13 | sean-k-mooney | NAME MAJ:MIN RM SIZE RO TYPE MOUNTPOINTS | |
| 19:19:15 | sean-k-mooney | loop0 7:0 0 76.8M 1 loop | |
| 19:19:17 | sean-k-mooney | loop1 7:1 0 76.8M 1 loop | |
| 19:19:19 | sean-k-mooney | zram0 252:0 0 15.6G 0 disk [SWAP] | |
| 19:19:21 | sean-k-mooney | nvme0n1 259:0 0 953.9G 0 disk | |
| 19:19:23 | sean-k-mooney | ├─nvme0n1p1 259:1 0 1.1G 0 part /boot/efi | |
| 19:19:25 | sean-k-mooney | ├─nvme0n1p2 259:2 0 14.9G 0 part /boot | |
| 19:19:27 | sean-k-mooney | └─nvme0n1p3 259:3 0 937.9G 0 part | |
| 19:19:29 | sean-k-mooney | └─nvme0n1p3_crypt 253:0 0 937.8G 0 crypt /var/lib/docker/btrfs | |
| 19:19:31 | sean-k-mooney | /home | |
| 19:19:33 | sean-k-mooney | /swap | |
| 19:19:35 | sean-k-mooney | / | |
| 19:19:37 | sean-k-mooney | paste.o.o is donw | |
| 19:19:39 | sean-k-mooney | blkid wiht no args does not alwasy show swap | |
| 19:20:41 | sean-k-mooney | ah | |
| 19:20:50 | sean-k-mooney | so with sudo the swap disk show up | |
| 19:20:52 | sean-k-mooney | but not without | |
| 19:21:02 | sean-k-mooney | at least initally | |
| 19:21:22 | sean-k-mooney | once i run blkid with sudo the non sudo one has the swap partion too | |
| 19:21:31 | sean-k-mooney | so it must be caching or something like that | |
| 19:23:08 | dansmith | I suppose I won't recheck my patches since this has failed twice in a row on the same thing | |
| 19:23:23 | sean-k-mooney | https://paste.centos.org/view/ebce235e | |
| 19:24:15 | dansmith | yeah weird, but... busybox | |
| 19:24:33 | sean-k-mooney | well the last two are the gnu versions | |
| 19:24:48 | sean-k-mooney | busybox apprently does not show anything as non root | |
| 19:25:02 | sean-k-mooney | which could also be the problme here since cirror is useing busybox | |
| 19:25:08 | sean-k-mooney | which is why i cheked it | |
| 19:25:53 | dansmith | makes me think that this test wasn't actually running in whatever tempest configs are required to merge | |
| 19:27:24 | dansmith | anyway, nothing much I can do about it now I guess | |
| 19:27:37 | dansmith | I'mma go get food | |
| 19:28:12 | sean-k-mooney | did you hit the revert button | |
| 19:28:24 | dansmith | pasted above: https://review.opendev.org/c/openstack/tempest/+/946379 | |
| 19:28:41 | sean-k-mooney | ack | |
| 19:31:07 | sean-k-mooney | ok commented on that too and now i need to go before the shops start to close o/ | |
| 19:42:16 | gmann | dansmith: ok, I did nto check multicell job but it was passing on regular one. agree to merge the revert first and then we can fix the test | |
| 20:11:44 | dansmith | gmann: I'm really not sure why the multicell job would be special here, almost like it is missing a regex exclude or something | |
| #openstack-nova - 2025-04-05 | |||
| 01:03:21 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2025.2 Flamingo https://review.opendev.org/c/openstack/nova/+/944603 | |
| 14:41:33 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove object_class_action https://review.opendev.org/c/openstack/nova/+/946497 | |
| 14:45:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove workaround for old oslo.versionedobjects https://review.opendev.org/c/openstack/nova/+/946497 | |
| 15:29:39 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove workaround for old oslo.versionedobjects https://review.opendev.org/c/openstack/nova/+/946497 | |