| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-07-20 | |||
| 10:47:29 | sean-k-mooney | dansmith: when you are around could you review https://review.opendev.org/c/openstack/devstack/+/888648 to unblock https://review.opendev.org/c/openstack/nova/+/886779 | |
| 10:55:30 | stephenfin | bauzas: There's a student from Oregon State University that has been working on OSC for the last few months and she's now eager to try her hand at one of the services themselves. How reasonable would it be to suggest a low-hanging nova API-related RFE for her to work on? | |
| 10:57:13 | stephenfin | I'm happy to keep covering the mentoring side of things but the ask would be for (a) suggestions of something relatively useful to do and (b) reviews if she works on it. She's around until June 2024 so it'll be relatively long-term (i.e. the C release cycle and a chunk of D) | |
| 10:58:56 | stephenfin | Easier things I can think of are changing the 'addresses' field in 'GET /servers/{serverID}' so we use an ID for the top-level key, to prevent munging of similarly named networks, and updating a few of the HTTP response codes, but I'm sure there's much more useful stuff to do | |
| 11:34:36 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove legacy placement tables from API DB https://review.opendev.org/c/openstack/nova/+/889086 | |
| 11:34:36 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove legacy tables from main DB https://review.opendev.org/c/openstack/nova/+/889087 | |
| 11:34:37 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Remove legacy columns from main DB https://review.opendev.org/c/openstack/nova/+/889088 | |
| 11:55:13 | bauzas | stephenfin: sorry was at lunch | |
| 11:56:03 | bauzas | stephenfin: we don't have a lot of low-hanging-fruits RFEs for the moment, but sure we can help | |
| 13:23:30 | dansmith | sean-k-mooney: yep | |
| 13:25:35 | kashyap | dansmith: Morning, is this the same thing that you saw the other day too? the SSL subject alt name not matching host name: https://zuul.opendev.org/t/openstack/build/f0102bbbbb854e7cb6f9d99f582f8c4c | |
| 13:26:58 | sean-k-mooney | dansmith: if you have time to add another small devstack patch to the list https://review.opendev.org/c/openstack/devstack/+/878718 fixes reboots on fedora. | |
| 13:27:24 | sean-k-mooney | i.e. makeing rebooting the devstack vm work if you use fedora/centos for the vm like it does on ubuntu/debiain | |
| 13:27:49 | sean-k-mooney | i fixed that based on the ptg feedback but havnt looked at it since | |
| 13:43:34 | dansmith | kashyap: that should be fixed now | |
| 13:43:44 | dansmith | it was a bug in the ubuntu curl package which was reverted | |
| 13:44:25 | dansmith | sean-k-mooney: got it | |
| 13:44:41 | kashyap | dansmith: Aaah, thank you! Lemme recheck | |
| 15:02:06 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add a new NumInstancesWeigher https://review.opendev.org/c/openstack/nova/+/886232 | |
| 16:13:31 | opendevreview | Maxim Monin proposed openstack/nova master: Server Rescue leads to Server ERROR state if original image is deleted https://review.opendev.org/c/openstack/nova/+/872385 | |
| 19:32:32 | opendevreview | Jay Faulkner proposed openstack/nova master: [ironic] Optionally set lessee on deployment https://review.opendev.org/c/openstack/nova/+/888290 | |
| 20:43:24 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Test tempest worker re-distribution https://review.opendev.org/c/openstack/nova/+/889130 | |
| 20:56:06 | opendevreview | Ghanshyam proposed openstack/nova master: DNM: testing Tempest run increased concurrency https://review.opendev.org/c/openstack/nova/+/889135 | |
| 20:56:55 | opendevreview | Felipe Reyes proposed openstack/nova master: Register riscv64 architecture. https://review.opendev.org/c/openstack/nova/+/889137 | |
| #openstack-nova - 2023-07-21 | |||
| 06:22:45 | opendevreview | Maxim Monin proposed openstack/nova master: Server Rescue leads to Server ERROR state if original image is deleted https://review.opendev.org/c/openstack/nova/+/872385 | |
| 10:50:15 | opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | |
| 10:50:15 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 12:00:07 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 13:32:55 | opendevreview | Takashi Natsume proposed openstack/nova-specs master: Create specs directory for 2024.1 Caracal https://review.opendev.org/c/openstack/nova-specs/+/889177 | |
| 13:36:57 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2023.2 Bobcat https://review.opendev.org/c/openstack/nova/+/876447 | |
| 13:41:33 | dansmith | bauzas: can you fix these nits real quick and I'll +2+A? https://review.opendev.org/c/openstack/nova/+/886232 | |
| 13:41:35 | dansmith | or I can do it | |
| 13:41:44 | bauzas | dansmith: sure, lemme look | |
| 13:42:18 | bauzas | dansmith: ack, thanks for reviewing, will do | |
| 13:51:07 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add a new NumInstancesWeigher https://review.opendev.org/c/openstack/nova/+/886232 | |
| 13:52:08 | auniyal | dansmith and all continuing to lock discussion - what type of action can be prevented on instance if instance is locked. | |
| 13:54:12 | dansmith | auniyal: anything with this, AFAIK: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2701 | |
| 14:00:22 | auniyal | th deorator is with stop and reboot as well, but I was able to reboot and stop locked instance | |
| 14:00:31 | dansmith | as admin or user? | |
| 14:00:37 | auniyal | yes admin | |
| 14:00:51 | dansmith | auniyal: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L240 | |
| 14:00:58 | dansmith | admins are not bothered by the lock | |
| 14:01:18 | auniyal | ack, | |
| 14:01:30 | auniyal | nova-manage can be run by user right | |
| 14:02:21 | dansmith | no | |
| 14:02:37 | dansmith | nova-manage requires physical access to the backend infrastructure | |
| 14:02:54 | dansmith | it's almost entirely direct-to-db, bypassing all of this sort of stuff in almost all cases | |
| 14:03:07 | dansmith | https://docs.openstack.org/nova/rocky/cli/nova-manage.html | |
| 14:03:14 | dansmith | "nova-manage controls cloud computing instances by managing various admin-only aspects of Nova." | |
| 14:12:23 | auniyal | thanks dansmith, | |
| 14:25:45 | auniyal | dansmith, I am facing this issue right now - https://review.opendev.org/c/openstack/nova/+/873648/comment/af765efc_a7161f39/ | |
| 14:25:46 | auniyal | so this is when the instance is not locked already. | |
| 14:25:46 | auniyal | however when I tested it by manually locking before running nova-manage cmd, db query went fine and there were no err msg. | |
| 14:25:46 | auniyal | even though in both cases while running instance_action.finish, instance was locked, query context and query values were same as required. | |
| 14:29:51 | dansmith | sorry, I'm not sure what context manager you mean you've removed, but action.finish failing means action_start never actually hit the DB, and none of that should have anything to do with lock as noted above | |
| 14:30:02 | dansmith | I've got to jump on a meeting now | |
| 14:31:30 | auniyal | okay, it did not thorugh any error, will check for if it action_start, thanks | |
| 16:52:53 | opendevreview | Merged openstack/nova master: Add a new policy for cold-migrate with host https://review.opendev.org/c/openstack/nova/+/881562 | |
| 16:56:58 | dansmith | holy crap, something merged | |
| 17:13:39 | gmann | heh | |
| 17:46:54 | sean-k-mooney | ||
| 18:26:40 | melwitt | it's a christmas miracle! (that something merged) | |
| 18:27:15 | sean-k-mooney | hehe | |
| 18:27:46 | dansmith | jgwentworth: jokes on you, that was so delayed it's *last* christmas' miracle | |
| 18:28:43 | jgwentworth | :) | |
| 19:34:37 | opendevreview | Felipe Reyes proposed openstack/nova master: Add RISCV64 support. https://review.opendev.org/c/openstack/nova/+/889137 | |
| #openstack-nova - 2023-07-23 | |||
| 08:21:30 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Include multipath_id in BDM record for iSCSI https://review.opendev.org/c/openstack/nova/+/889257 | |
| #openstack-nova - 2023-07-24 | |||
| 08:13:06 | opendevreview | Nobuhiro MIKI proposed openstack/nova master: libvirt: Add 'COMPUTE_ADDRESS_SPACE_*' traits support https://review.opendev.org/c/openstack/nova/+/873221 | |
| 11:40:28 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: Bump MIN_{LIBVIRT,QEMU} for "Bobcat" https://review.opendev.org/c/openstack/nova/+/887255 | |
| 11:48:28 | opendevreview | Friedrich Hiekel proposed openstack/nova stable/yoga: nova scheduler debug logs TypeError https://review.opendev.org/c/openstack/nova/+/889311 | |
| 14:08:55 | opendevreview | Felipe Reyes proposed openstack/nova master: Add RISCV64 support. https://review.opendev.org/c/openstack/nova/+/889137 | |
| 14:21:20 | opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | |
| 14:21:21 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 14:39:52 | opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | |
| 14:39:52 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 15:01:32 | opendevreview | Felipe Reyes proposed openstack/nova master: Add RISCV64 support. https://review.opendev.org/c/openstack/nova/+/889137 | |
| 19:21:14 | opendevreview | Felipe Reyes proposed openstack/nova master: Add RISCV64 QEMU emulation support. https://review.opendev.org/c/openstack/nova/+/889137 | |
| 21:09:30 | dansmith | gmann: do you know if anyone is looking at this? https://54345f3748db63fac266-f6d343514834eb14212ec1823288b4de.ssl.cf5.rackcdn.com/889196/2/check/tempest-slow-py3/298c784/testr_results.html | |
| 21:09:39 | dansmith | seems like maybe we need to skip it as it's failing a *lot* | |
| 21:10:40 | gmann | dansmith: humm, i saw it earlier when I made this to run in parallel and ykarel mentioned parallel run can be one of the reason. i made it to run serially 2-3 days back but it is still failing | |
| 21:11:00 | dansmith | yeah and it's been failing since before that I think | |
| 21:11:05 | gmann | confirming if it run serially | |
| 21:11:33 | dansmith | ugh, sorry, wrong channel | |
| 21:11:38 | dansmith | meant to be in -qa | |
| 23:24:15 | opendevreview | Merged openstack/nova master: Add compute_id columns to instances, migrations https://review.opendev.org/c/openstack/nova/+/879499 | |
| #openstack-nova - 2023-07-25 | |||
| 01:56:43 | sapd1 | Hi everyone, I'm running openstack train. I have a problem when resize an instance with PCI devices(GPUPassthrough) to a flavor without PCI devices. | |
| 01:56:54 | sapd1 | Does anyone know how to fix this problem. | |
| 01:57:06 | sapd1 | When I perform resize: No valid host was found. No valid host found for resize (HTTP 400) (Request-ID: req-9d6cf3f9-8f6f-482f-b0b1-95ec286a0fc3) | |
| 05:33:40 | auniyal | sapd1 - as told No Valid Host. | |
| 05:33:40 | auniyal | but if you are sure this is not correct and still looking for reason, can you add --debug in your resize cmd to find out exactly which call failed and why, then once you find out that look for req-ID in compute logs. it will give you better idea of fail. if there are any traceback's in logs - you can file a bug here: | |
| 05:33:40 | auniyal | https://bugs.launchpad.net/nova | |
| 07:07:38 | sapd1 | auniyal: this bug: https://bugs.launchpad.net/nova/+bug/1941005 | |
| 07:09:04 | auniyal | it seems fixed | |
| 07:09:55 | sapd1 | auniyal: but it's not merged yet | |
| 07:10:31 | auniyal | oh yes, in train its not merged | |
| 07:19:58 | sapd1 | auniyal: I patched it in my system. | |
| 07:40:43 | opendevreview | Konrad Gube proposed openstack/nova master: Use Cinder's os-extend_volume_completion volume action. https://review.opendev.org/c/openstack/nova/+/873560 | |
| 08:32:18 | bauzas | dansmith: gmann: it occurs to me that nova-next is almost failing on volume detach due to some kind of filesystem check (that's a guess) and kinda related to what you told this night (for me) | |
| 08:32:26 | bauzas | https://6d9b97dc35f887a99105-3214406b4544fce2f9d807df6ea4fe3f.ssl.cf5.rackcdn.com/886232/4/check/nova-next/31d3775/testr_results.html | |
| 09:02:57 | sean-k-mooney | is that using cinder lvm? | |