| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-06-09 | |||
| 13:56:36 | dansmith | but the local share thing should really be a tack-on at the end of this right? | |
| 13:56:59 | dansmith | the set is extremely long right now so just breaking that stuff off the end for now would make sense to me and focus on getting the manila bit working as the primary goal | |
| 13:57:02 | sean-k-mooney | well i atcully dont think we shoudl do the local share as part of this work this cyle | |
| 13:58:07 | sean-k-mooney | i was ortgnallly hoping that the virtiofs xml change woudl be first then scaphandre abd mainila could progess in parallel | |
| 13:58:45 | sean-k-mooney | but i entrily expect us to be able to merge scaphandre support without any of the manilla supprot being merge outside the virtiofs part in the libvirt driver | |
| 13:59:20 | dansmith | yes I would think so too, but you're also saying do that after manila (now) right? | |
| 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 | |