| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-03-12 | |||
| 15:46:52 | gibi | sean-k-mooney: tox -e py36 resource_tracker -r passed on master for me | |
| 15:48:15 | sean-k-mooney | it looks like im missing https://github.com/openstack/nova/commit/91973ec26c3ae77ffe349bc4c5c03f13065d9b6f | |
| 15:48:39 | sean-k-mooney | but i did a rebase against master just before running the tests | |
| 15:48:54 | kashyap | lyarwood: Responded... | |
| 15:48:58 | sean-k-mooney | i wonder if the repos are all in sync and what one im using as my origin | |
| 15:49:13 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Mask the token used to allow access to consoles https://review.opendev.org/708876 | |
| 15:49:20 | sean-k-mooney | ill rebase against the gerrit remote which shoudl be up to date | |
| 15:50:05 | lyarwood | kashyap: thanks | |
| 15:50:26 | gibi | sean-k-mooney: I have 91973ec26c3ae77ffe349bc4c5c03f13065d9b6f in my repo | |
| 15:50:37 | kashyap | lyarwood: Not sure how much of what I wrote makes sense :-); I'll check back in later | |
| 15:52:38 | lyarwood | kashyap: yup it does, I wasn't aware of the other API | |
| 15:52:45 | lyarwood | kashyap: I'll take a look after $meeetings | |
| 15:53:13 | kashyap | lyarwood: It's part of the (you wouldn't believe) 5-ish year effort in QEMU/libvirt to get the incremental backup stuff and all the relevant things | |
| 15:53:29 | sean-k-mooney | gibi: my origin was set to https://review.openstack.org/p/openstack/nova i wonder if one fo the servers is out of sync | |
| 15:54:01 | gibi | sean-k-mooney: my setup is a bit complicated as I push through nordix, gerrit.nordix.org | |
| 15:54:37 | gibi | and I'm not sure what nordix mirrors | |
| 15:54:41 | sean-k-mooney | gibi: ya no worries i can fix this by swaping my origin to opendev or gitub | |
| 15:54:49 | sean-k-mooney | instead of directly form gerrit | |
| 15:55:01 | sean-k-mooney | but its just strange that it thinks im up to date | |
| 15:55:22 | gibi | nova team meeting starts in 5 minutes on #openstack-meeting-3 (please note the place!) | |
| 15:56:18 | sean-k-mooney | oh it moved | |
| 15:56:23 | sean-k-mooney | as part of the time chagne | |
| 15:57:03 | gibi | yeah, #openstack-meeting was booked this time | |
| 16:14:49 | openstackgerrit | John Garbutt proposed openstack/nova master: Add tests for API using unified limits driver https://review.opendev.org/712140 | |
| 16:15:37 | openstackgerrit | John Garbutt proposed openstack/nova master: Make unified limits APIs return reserved of 0 https://review.opendev.org/712141 | |
| 16:15:53 | openstackgerrit | John Garbutt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/712142 | |
| 16:16:09 | openstackgerrit | John Garbutt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/712143 | |
| 16:16:23 | openstackgerrit | John Garbutt proposed openstack/nova master: Update limit APIs https://review.opendev.org/712707 | |
| 16:28:00 | openstackgerrit | Merged openstack/nova master: Deprecate the vmwareapi driver https://review.opendev.org/712101 | |
| 16:45:39 | gibi | artom, bauzas: I have to leave for today, but let's discuss the meeting time clash at some point | |
| 16:45:52 | bauzas | gibi: we're trying to fix it | |
| 16:46:13 | artom | gibi, we *have* fixed it :) | |
| 16:46:15 | bauzas | you shouldn't impact the whole community for a specific internal-only issue | |
| 16:46:20 | artom | Just not this week, because it was too last minute | |
| 16:46:29 | artom | Next week our call in on Wednesday, so no conflict | |
| 16:46:38 | artom | (Well, unless someone strongly objects internally) | |
| 16:46:59 | bauzas | gibi: technically, for a worldwide distributed team, timeslots around 5-6pm our time are strategic | |
| 16:47:05 | gibi | artom, bauzas: thanks | |
| 16:47:17 | gibi | and sorry for the short notice | |
| 16:47:25 | bauzas | but we only have 5 of those slots per week :) | |
| 16:47:28 | gibi | I would not able to chari the night meeting | |
| 16:47:31 | bauzas | so, conflics are usual :) | |
| 16:47:41 | bauzas | conflicts* even | |
| 16:52:44 | mnaser | hi nova friends -- this is kind-of-a-big-change and it'd be nice if opinions came in here -- https://review.opendev.org/#/c/712696/4 | |
| 16:54:35 | bauzas | mnaser: that's surely something I can comment on | |
| 16:54:43 | mnaser | bauzas: please :) | |
| 16:55:13 | sean-k-mooney | gibi: i dont think we will have an issue aft the eu moves to DST | |
| 16:55:25 | sean-k-mooney | gibi: we are also looking at another time internally | |
| 16:55:42 | sean-k-mooney | gibi: so we shoudl be about to sort it out internally | |
| 17:09:25 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Allocate mdevs when resizing or reverting resize https://review.opendev.org/712741 | |
| 17:13:22 | openstackgerrit | John Garbutt proposed openstack/nova master: Update limit APIs https://review.opendev.org/712707 | |
| 17:13:23 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Enforce unified limits using oslo.limit https://review.opendev.org/615180 | |
| 17:15:46 | openstackgerrit | John Garbutt proposed openstack/nova master: DNM: Turn on unified limits to see if tests pass https://review.opendev.org/708387 | |
| 17:22:51 | kplant | jkulik: for the record, the problem seems to have been that the AZ which the instance was created for no longer existed | |
| 17:22:59 | kplant | the resize function kept reqesting an AZ with the original name | |
| 17:23:21 | kplant | so i just created an AZ with that name. not sure if that's intended behavior | |
| 17:30:41 | johnthetubaguy | kplant: if you requested the AZ in the first place, I believe that is expected | |
| 17:32:39 | kplant | johnthetubaguy: you're everywhere :-) | |
| 17:33:10 | johnthetubaguy | heh, sometimes | |
| 17:41:16 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Fix up quota classes APIs https://review.opendev.org/712749 | |
| 17:42:55 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Fix up quota sets APIs https://review.opendev.org/712749 | |
| 17:49:51 | openstackgerrit | Elod Illes proposed openstack/nova stable/rocky: Remove global state from the FakeDriver https://review.opendev.org/712751 | |
| 17:51:19 | stephenfin | johnthetubaguy: if you're about for another few minutes, care to finish off the remove-nova-network series? https://review.opendev.org/#/c/703974/ | |
| 18:06:06 | openstackgerrit | melanie witt proposed openstack/nova master: Add config option for neutron client retries https://review.opendev.org/712226 | |
| 18:15:36 | sean-k-mooney | anyone know where the placement inventory class is defiend in nova | |
| 18:21:00 | sean-k-mooney | ok its not a class or object its just a random dict https://github.com/openstack/nova/blob/2f88e09609ee3075d675c1c41358f23901e56bce/nova/compute/provider_tree.py#L63 | |
| 18:22:10 | sean-k-mooney | ah # dict of inventory records, keyed by resource class | |
| 18:22:16 | sean-k-mooney | that is what i was looking for | |
| 18:33:22 | openstackgerrit | Sasha Andonov proposed openstack/nova master: rbd_utils: increase _destroy_volume timeout https://review.opendev.org/705764 | |
| 18:43:50 | openstackgerrit | sean mooney proposed openstack/nova master: Provider Config File: YAML file loading and schema validation https://review.opendev.org/673341 | |
| 18:43:51 | openstackgerrit | sean mooney proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | |
| 18:43:51 | openstackgerrit | sean mooney proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | |
| 18:43:52 | openstackgerrit | sean mooney proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | |
| 18:46:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Handle flavor disk mismatches when resizing https://review.opendev.org/712766 | |
| 18:49:43 | stephenfin | melwitt: did you # scsi > 7 patch land? Have you a link? | |
| 18:50:58 | melwitt | stephenfin: what? the scsi limited to 7 thing is a libvirt bug. previous to that bug, the limit was 26 and a patch of mine from stein changed it to be configurable | |
| 18:51:16 | melwitt | which thing are you looking for? | |
| 18:51:38 | stephenfin | Ah, I got that assways, sorry /o\ | |
| 18:52:00 | stephenfin | I was conflating the two | |
| 18:52:06 | openstack | Launchpad bug 1864279 in OpenStack Compute (nova) "Unable to attach more than 6 scsi volumes" [Undecided,New] | |
| 18:52:06 | stephenfin | https://bugs.launchpad.net/nova/+bug/1864279 | |
| 18:52:16 | stephenfin | trying to find a dupe to associate with that | |
| 18:54:26 | melwitt | stephenfin: ok, just looked. yeah, I remember we discussed downstream whether to propose a patch that basically checks "!= 7" and skips, but the consensus was for libvirt to fix its bug | |
| 18:56:12 | melwitt | and apparently that did not happen yet? this has been going on for a long time and it looks like we need to talk about it again | |
| 18:57:21 | melwitt | it doesn't really seem right for us to have to cover over the libvirt bug. and I don't know why it's not getting fixed on their side | |
| 18:57:50 | melwitt | last I heard, they said they were fixing it | |
| 18:58:01 | stephenfin | I've no idea whether it has been fixed yet or not. Could be that the fix just hasn't been backported? I can poke danpb or kashyap about it in the AM | |
| 18:58:31 | stephenfin | I'm tempted to close this bug as not an issue here though. It's clearly a libvirt issue | |
| 18:58:35 | melwitt | yeah I mean it seems not fixed if these ppl are hitting it | |
| 18:59:03 | stephenfin | The only resolution I feel we could do is add a warning for the affected versions. We have done that before, in fairness | |
| 18:59:04 | melwitt | but yeah def let's ask danpb or kashyap tomorrow | |
| 18:59:44 | stephenfin | cool, I'll do that and consider adding the nova runtime version check if it's worth it | |
| 19:00:16 | openstackgerrit | Merged openstack/nova master: Run sdk functional tests on nova changes https://review.opendev.org/712092 | |
| 19:00:23 | openstackgerrit | Merged openstack/nova master: [Trivial] Fix code comment of admin password tests https://review.opendev.org/711734 | |
| 19:06:12 | openstack | melwitt: Error: Error getting bugzilla.redhat.com bug #1744079: NotPermitted | |
| 19:06:12 | melwitt | stephenfin: here's the bug ticket https://bugzilla.redhat.com/show_bug.cgi?id=1744079 says it's fixed, so need to look at what versions this is showing up in? maybe they just need to update or maybe a backport is missing like you said? | |
| 19:07:02 | openstack | melwitt: Error: Error getting bugzilla.redhat.com bug #1741782: NotPermitted | |
| 19:07:02 | melwitt | sorry that was a clone https://bugzilla.redhat.com/show_bug.cgi?id=1741782 | |
| 19:09:57 | kplant | if i have a instance on say "compute-01" and virsh shows it there, but nova thinks it's on "compute-02" how can i fix that? | |
| 19:12:29 | stephenfin | melwitt: Yup, looks like this is the commit we need https://github.com/libvirt/libvirt/commit/c8007fdc5d2ce43fec2753cda60fb4963f55abd5 | |