| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-06-09 | |||
| 13:59:32 | sean-k-mooney | not really | |
| 13:59:43 | dansmith | [06:57:02] <sean-k-mooney> well i atcully dont think we shoudl do the local share as part of this work this cyle | |
| 13:59:51 | dansmith | what does that mean then? | |
| 14:00:06 | sean-k-mooney | the intoduction of a local share type | |
| 14:00:28 | sean-k-mooney | if we were to have that i would prefer to defer that to the other uscase of epmperal fs or simialr | |
| 14:00:30 | dansmith | the scaphandre case is just a special configuration of a local share no? | |
| 14:00:54 | sean-k-mooney | im conlifcted on how to respond to that | |
| 14:01:20 | sean-k-mooney | i want to say no because i dont think scaphandre's use of virtio-fs shoudl me modeled as a share in the db | |
| 14:01:23 | dansmith | by "local share" I'm meaning the "operator has configured nova to always insert this share into all guests on this host" | |
| 14:01:28 | sean-k-mooney | so its a special case fo virtio-fs | |
| 14:01:36 | dansmith | maybe we need to define some terms | |
| 14:02:14 | sean-k-mooney | i would cinsider it a special case foinsterting this "filesystem path" into all guest that request it | |
| 14:02:44 | dansmith | okay I'm a bit confused then | |
| 14:02:44 | sean-k-mooney | i would reall like to keep "share" to refer only to manialla shares | |
| 14:02:52 | dansmith | I agree with that | |
| 14:03:27 | sean-k-mooney | if we want to allow arbairy addtional filesystem to be provided to a guest via an api that woudl be a new filesystem attach api | |
| 14:03:48 | sean-k-mooney | for scaphandre as you said i think config drive is a better mental model | |
| 14:04:15 | sean-k-mooney | we have an image property/extra spec that act as a request for it and a trait on host that supprot it | |
| 14:04:38 | dansmith | how about a 15 minute meet on the topic real quick? | |
| 14:04:42 | sean-k-mooney | there is no requirement for modileign it in the db or as object or in the api beyond the image property/extra spec | |
| 14:04:45 | dansmith | Uggla_: ? | |
| 14:06:07 | sean-k-mooney | sure i can do that | |
| 14:06:27 | dansmith | meet.google.com/duc-bfai-mrv | |
| 14:07:47 | sean-k-mooney | anyone who is interested is welcome to join by the way ^ | |
| 14:07:54 | dansmith | yup | |
| 14:14:21 | Uggla_ | dansmith, sean-k-mooney sorry was out to get Rose, reading | |
| 14:18:52 | bauzas | dansmith: sean-k-mooney: Uggla_ : doh, I need to go to get my daughter in 5 mins | |
| #openstack-nova - 2023-06-11 | |||
| 16:19:50 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: Refactor CinderFixture https://review.opendev.org/c/openstack/nova/+/885756 | |
| 16:19:50 | opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer for dangling bdms https://review.opendev.org/c/openstack/nova/+/881457 | |
| 16:19:51 | opendevreview | Amit Uniyal proposed openstack/nova master: Delete dangling bdms https://review.opendev.org/c/openstack/nova/+/882284 | |
| #openstack-nova - 2023-06-12 | |||
| 06:47:53 | opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer for dangling bdms https://review.opendev.org/c/openstack/nova/+/881457 | |
| 06:47:54 | opendevreview | Amit Uniyal proposed openstack/nova master: Delete dangling bdms https://review.opendev.org/c/openstack/nova/+/882284 | |
| 07:38:16 | opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer for dangling bdms https://review.opendev.org/c/openstack/nova/+/881457 | |
| 07:38:17 | opendevreview | Amit Uniyal proposed openstack/nova master: Delete dangling bdms https://review.opendev.org/c/openstack/nova/+/882284 | |
| 09:10:01 | opendevreview | Sofia Enriquez proposed openstack/nova-specs master: Implement support for LUKS inside qcow2 volumes when using Libvirt https://review.opendev.org/c/openstack/nova-specs/+/883516 | |
| 09:34:46 | opendevreview | Sofia Enriquez proposed openstack/nova-specs master: Implement support for LUKS inside qcow2 volumes when using Libvirt https://review.opendev.org/c/openstack/nova-specs/+/883516 | |
| 11:12:49 | opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer for dangling bdms https://review.opendev.org/c/openstack/nova/+/881457 | |
| 11:12:49 | opendevreview | Amit Uniyal proposed openstack/nova master: Delete dangling bdms https://review.opendev.org/c/openstack/nova/+/882284 | |
| 16:42:25 | opendevreview | Merged openstack/nova-specs master: Add compute-object-ids spec for 2023.2 https://review.opendev.org/c/openstack/nova-specs/+/877291 | |
| #openstack-nova - 2023-06-13 | |||
| 08:14:13 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 15:33:58 | manuvakery1 | Hi, Few of my VM live migration never completes, it always get stuck at the end of memory copy , going from 1% to 0 and then to 1%. I tried to set the following params | |
| 15:33:58 | manuvakery1 | live_migration_downtime: 500 | |
| 15:33:58 | manuvakery1 | live_migration_downtime_steps: 10 | |
| 15:33:58 | manuvakery1 | live_migration_downtime_delay: 50, Still its not helping | |
| 15:34:32 | manuvakery1 | I see its sets downtime only once to 50ms | |
| 15:35:04 | manuvakery1 | Any help on this | |
| 15:36:38 | manuvakery1 | 2023-06-13 15:01:20.336 3223605 INFO nova.virt.libvirt.migration [req-42a1d50d-1411-416e-8084-cf929cf892f9 - - - - -] [instance: 09658040-2af3-4fbf-884f-601a4d1f2ce2] Increasing downtime to 50 ms after 0 sec elapsed time | |
| 15:37:02 | frickler | manuvakery1: you may wnat to look into auto-completion or post-copy https://docs.openstack.org/nova/yoga/admin/configuring-migrations.html | |
| 15:38:18 | manuvakery1 | frickler: Sure i will .. but based on the configs I set it should increase the downtime to 100ms and then 150ms until it reaches 500ms | |
| 18:47:42 | stephenfin | sean-k-mooney[m]: melwitt: Would you folks be able to look at https://review.opendev.org/c/openstack/placement/+/880623/ | |
| 18:48:27 | stephenfin | (Context is we discussed bumping SQLAlchemy to 2.x at a TC forum session this morning and I realised this was proposed but never merged /o\) | |
| 18:52:07 | melwitt | stephenfin: sure, I can take a look later | |
| 18:52:30 | stephenfin | ty 🙏 | |
| 20:10:22 | opendevreview | Amit Uniyal proposed openstack/nova master: Allow swap resize from non-zero to zero https://review.opendev.org/c/openstack/nova/+/857339 | |
| 21:50:31 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Update test for DB special characters https://review.opendev.org/c/openstack/nova/+/880669 | |
| 22:44:57 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Store unmasked password https://review.opendev.org/c/openstack/nova/+/880669 | |
| #openstack-nova - 2023-06-14 | |||
| 07:41:43 | opendevreview | Pierre Riteau proposed openstack/nova stable/victoria: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858051 | |
| 08:25:39 | opendevreview | Kiran Pawar proposed openstack/nova-specs master: SR-IOV NIC device tracking in Placement https://review.opendev.org/c/openstack/nova-specs/+/884569 | |
| 08:53:07 | opendevreview | Vitalii Vrublevskyi proposed openstack/nova master: Napatech SmartNIC support https://review.opendev.org/c/openstack/nova/+/859577 | |
| 15:25:20 | opendevreview | Artom Lifshitz proposed openstack/nova master: Call Neutron immediately upon _post_live_migration() start https://review.opendev.org/c/openstack/nova/+/883682 | |
| 15:25:21 | opendevreview | Artom Lifshitz proposed openstack/nova master: Parallelize calls to Neutron and Cinder in post_live_migration https://review.opendev.org/c/openstack/nova/+/883678 | |
| 17:53:39 | opendevreview | Navid Pustchi proposed openstack/nova-specs master: dnm test https://review.opendev.org/c/openstack/nova-specs/+/886116 | |
| 17:55:41 | opendevreview | Navid Pustchi proposed openstack/nova-specs master: dnm test https://review.opendev.org/c/openstack/nova-specs/+/886116 | |
| 21:27:19 | melwitt | stephenfin, sean-k-mooney: I have a couple of patches related to database archiving that need review if yall get a chance. the second change looks big but it's because of the addition of a for loop (and the subsequent indentation of the code block under it). I added a comment on the review to explain the change in hopes of easing review https://review.opendev.org/q/topic:archive-trees-one-at-a-time | |
| 21:43:57 | opendevreview | Merged openstack/nova stable/xena: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858024 | |
| 22:48:13 | opendevreview | Merged openstack/placement master: db: Replace use of deprecated API https://review.opendev.org/c/openstack/placement/+/880623 | |
| #openstack-nova - 2023-06-15 | |||
| 00:42:48 | opendevreview | Artom Lifshitz proposed openstack/nova master: Call Neutron immediately upon _post_live_migration() start https://review.opendev.org/c/openstack/nova/+/883682 | |
| 00:42:49 | opendevreview | Artom Lifshitz proposed openstack/nova master: Parallelize calls to Neutron and Cinder in post_live_migration https://review.opendev.org/c/openstack/nova/+/883678 | |
| 11:49:17 | opendevreview | Merged openstack/placement master: tests: Use base class for all functional tests https://review.opendev.org/c/openstack/placement/+/880624 | |
| 21:54:19 | opendevreview | Stephen Finucane proposed openstack/placement master: Add job to test with SQLAlchemy master (2.x) https://review.opendev.org/c/openstack/placement/+/886229 | |
| 21:58:33 | opendevreview | Stephen Finucane proposed openstack/placement master: Add job to test with SQLAlchemy master (2.x) https://review.opendev.org/c/openstack/placement/+/886229 | |
| 21:58:41 | opendevreview | Stephen Finucane proposed openstack/nova master: Add job to test with SQLAlchemy master (2.x) https://review.opendev.org/c/openstack/nova/+/886230 | |
| 23:45:36 | opendevreview | Merged openstack/placement master: tests: Warn on *any* SAWarning warning https://review.opendev.org/c/openstack/placement/+/880625 | |
| 23:49:50 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add a new NumInstancesWeigher https://review.opendev.org/c/openstack/nova/+/886232 | |
| #openstack-nova - 2023-06-16 | |||
| 06:37:12 | opendevreview | melanie witt proposed openstack/nova master: testing: Fix and robustify archive_deleted_rows test https://review.opendev.org/c/openstack/nova/+/877055 | |
| 06:37:12 | opendevreview | melanie witt proposed openstack/nova master: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/877056 | |
| 19:08:02 | opendevreview | melanie witt proposed openstack/nova master: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/877056 | |
| 20:51:31 | melwitt | fyi just got two CI failures on the same patch in the nova-live-migration and nova-grenade-multinode jobs https://bugs.launchpad.net/neutron/+bug/1940425 and it reminded me I have been seeing this failure often lately | |
| 20:52:05 | melwitt | we likely need to look into this to improve gate fail rate | |
| #openstack-nova - 2023-06-17 | |||
| 02:41:47 | opendevreview | Stephen Finucane proposed openstack/nova master: Add job to test with SQLAlchemy master (2.x) https://review.opendev.org/c/openstack/nova/+/886230 | |
| #openstack-nova - 2023-06-19 | |||
| 13:03:08 | opendevreview | Danila Balagansky proposed openstack/nova master: Show free vCPUs in HostState __repr__ https://review.opendev.org/c/openstack/nova/+/886400 | |
| #openstack-nova - 2023-06-20 | |||
| 10:56:05 | noonedeadpunk | Hey folks! Completely forgot to ask that during PTG last week :( But wdyt about this bug report? For me it sounds more like a nova issue, that service fails and do not try to re-auth? https://bugs.launchpad.net/openstack-ansible/+bug/2023370 | |
| 10:57:04 | noonedeadpunk | So what's hapenning in OSA here, is that we reset a password (in 99.9% to exactly same one). While I kinda agree we should not do that at the first place, but this should not break service either? | |
| 11:26:17 | stephenfin | melwitt: Sorry for the delay, reviewed that DB archive series. I addressed your feedback on https://review.opendev.org/c/openstack/nova/+/880669/ also. Thanks for that | |
| 11:49:31 | opendevreview | Merged openstack/nova master: testing: Fix and robustify archive_deleted_rows test https://review.opendev.org/c/openstack/nova/+/877055 | |
| 12:52:17 | sean-k-mooney | noonedeadpunk: reset the password where | |
| 12:52:32 | sean-k-mooney | noonedeadpunk: you update the db password? rabbit? | |
| 12:52:35 | noonedeadpunk | sean-k-mooney: in keystone. | |
| 12:52:51 | noonedeadpunk | so issued previously tokens are invalidated | |
| 12:53:03 | noonedeadpunk | despite password is kinda the same | |
| 12:53:07 | sean-k-mooney | oh that sound like a keystone issue then | |
| 12:53:28 | sean-k-mooney | password or what by the way | |
| 12:53:44 | noonedeadpunk | but it happens kinda only when asking neutron api by nova service, and only first request fails, and then it re-asks for the token I believe | |
| 12:53:51 | sean-k-mooney | the password of the nova user? or the enduser creating the vm | |
| 12:53:57 | noonedeadpunk | of the nova user | |
| 12:54:15 | noonedeadpunk | and system service user per say | |
| 12:54:28 | sean-k-mooney | right so if you udpate it in keystoen you need to update it in the nova.conf and then restart the nova service | |
| 12:54:53 | sean-k-mooney | unless this is marked as mutable a sighup wont be enough | |
| 12:54:59 | noonedeadpunk | yeah, but it's reseted to the same value | |
| 12:55:12 | noonedeadpunk | So you kinda don't need to update conf | |