Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-07
20:34:48 mriedem the spec just says, "operators will have to use host aggregates"
20:35:19 artom I sort of assumed it would be in the same way that we have pinned CPUs now
20:35:31 artom Aggregates for trusted VFs and aggregated for normal VFs
20:35:46 artom So that any VF in a 'trusted' aggregate is, err, trusted
20:38:27 dansmith that would kindof suck
20:39:40 mriedem the PciPassthroughFilter is looking at HostState.pci_stats, but i don't know what goes into that thing saying "this host reported it supports trusted VFs"
20:40:09 openstackgerrit Merged openstack/nova master: Only attempt a rebuild claim for an evacuation to a new host https://review.openstack.org/546268
20:42:13 mriedem i guess https://review.openstack.org/#/c/458514/13/releasenotes/notes/trusted-vfs-abee6dff7c9b6940.yaml ?
20:42:27 mriedem so the pci passthrough_whitelist has to report the devices that are trusted,
20:42:37 mriedem and the PciPassthroughFilter links those to the ports in the pci request spec?
20:43:25 mriedem and pci passthrough_whitelist is our poor man's placement inventory system yes?
20:43:26 mriedem jaypipes: ^?
20:43:47 openstackgerrit Dan Smith proposed openstack/nova master: Make nova-manage db purge take --all-cells https://review.openstack.org/550502
20:44:52 dansmith mriedem: this ^ passed, so I converted the hook to run --all-cells on the purge
20:45:20 dansmith all the other patches have a clean swath of green from zuul too
20:45:22 mriedem +2 on the bottom change in the series
20:46:06 dansmith the other guy hasn't updated the archive patch yet, but if he has tomorrow I'll rebase on this and convert that part of the hook too
20:46:26 mriedem Kevin_Zheng ?
20:46:34 dansmith someone else was responding right?
20:46:39 mriedem oh idk
20:47:06 dansmith also downstream QE here is going to try to get some real world database snapshots and run archive/purge against them
20:48:51 bauzas jaypipes: mriedem: dansmith: as we discussed at the PTG, I'm about to provide a new spec for NUMA topology, should I use that one https://review.openstack.org/#/c/543062/1 or just a new one given we agreed on a different way
20:49:00 tssurya dansmith : Zhenyu Zheng is Kevin_Zheng :)
20:49:14 dansmith tssurya: oh, hah
20:49:57 dansmith I stopped pattern matching at "Z != K; break"
20:50:25 tssurya dansmith: :D
20:54:34 openstackgerrit Surya Seetharaman proposed openstack/nova master: Update the nova-manage db archive_deleted_rows description https://review.openstack.org/550600
20:54:48 artom dansmith, actually, about that, if we grab that mitaka dump from the "original" bug a long time ago, and just run it through nova-manage db-sync...?
20:55:06 dansmith artom: it was newton I think
20:55:13 dansmith artom: we could, but it'd be nicer to have something a little fresher
20:55:17 jaypipes mriedem: yes, you are correct. (about the pci_passthrough_whitelist and PciPassthroughFilter cruft.
20:55:19 dansmith but if nothing else, that will work
20:55:40 jaypipes bauzas: definitely a new one.
20:55:56 bauzas jaypipes: roger.
20:58:18 mriedem jaypipes: ok cool,
20:58:28 mriedem it's about time, since juno, that i start to try to understand how that stuff works
20:58:39 mriedem i would really rather not have to though
20:59:06 openstackgerrit Merged openstack/nova stable/queens: Check for multiattach before removing connections https://review.openstack.org/550221
21:03:27 bauzas just to make it clear : "le"/"la"
21:03:36 bauzas le woof, la bye
21:03:50 bauzas :p
21:09:39 mriedem maybe in le france
21:09:42 mriedem i'm in 'merica
21:18:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Rename the 'recreate' param in rebuild_instance to 'evacuate' https://review.openstack.org/508190
21:22:45 bauzas oh man, getting mad with the imagebackend
21:30:28 mriedem dansmith: not sure if you want to clean up the nits in https://review.openstack.org/#/c/550182/ before i +W?
21:31:16 mriedem bauzas: https://i.chzbgr.com/full/6425329152/hA7A48AD5/
21:32:03 bauzas meh
21:32:14 bauzas trying to understand spaghetti code
21:32:36 mriedem i think mdbooth said the libvirt imagebackend code was done
21:36:12 bauzas mriedem: man, it's your fault
21:36:39 bauzas mriedem: I need to understand why we preallocate LVM volumes
21:36:41 bauzas :p
21:37:00 mriedem is pdragbrady still around?
21:37:06 mriedem *pdraigbrady?
21:37:50 mriedem oh facebook i see
21:40:56 dansmith mriedem: sure, give me a sec
21:43:10 openstackgerrit Dan Smith proposed openstack/nova master: Add --purge helper flag to archive_deleted_rows https://review.openstack.org/550182
21:43:11 openstackgerrit Dan Smith proposed openstack/nova master: Make nova-manage db purge take --all-cells https://review.openstack.org/550502
21:43:17 dansmith mriedem: le boom ^
21:43:28 mriedem le thanks
21:47:42 mriedem dansmith: assuming we should get this into a 17.0.1 as well https://review.openstack.org/#/c/550557/ if you want to fast approve
21:47:46 mriedem then i can request the release
21:48:39 dansmith mriedem: hmm, for the backport we should probably clamp to the 4.x latest right?
21:48:52 mriedem wasn't sure
21:48:57 dansmith it should be the minimum that release supports
21:49:10 dansmith so actually not 4.max
21:49:32 dansmith 4.18 or 'pike'
21:49:40 mriedem yeah was going to say 4.18
21:49:59 dansmith although
21:50:11 mriedem then why didn't we need that on master?
21:50:12 dansmith if someone is starting up a new deployment of queens...
21:50:23 dansmith because 5.0 is the minimum we support on master
21:50:38 dansmith actually, you know, yeah, let's just leave it
21:50:44 dansmith it doesn't cache it,
21:50:50 dansmith so if it's really first startup, 5.0 is fine on queens
21:51:07 dansmith and if something else comes online that is older, it won't have cached it and will do the right thing
21:51:50 dansmith mriedem: I will say this is not critical on queens though, since queens will speak 4.11 if you ask it to,
21:51:55 dansmith so it won't break like master did
21:52:12 mriedem i wanted to ask odyssey4me if they hit this in OSA on queens or master
21:52:32 mriedem if we don't need this in queens, i'm fine to abandon
21:52:36 dansmith really shouldn't on queens, it's after we dropped 4.x support in master that pinning to 4.11 breaks
21:52:53 dansmith let's just hold off, ask odyssey4me, and release 17.0.1 without it
21:52:56 openstackgerrit Sylvain Bauza proposed openstack/nova master: Deprecate sparse LVs https://review.openstack.org/549771
21:53:00 dansmith and if I'm missing something we can merge it later, okay?
21:53:11 mriedem yeah
21:53:11 mriedem abandoned
21:58:09 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094
22:05:41 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - List/Show all server migration types https://review.openstack.org/430839
22:11:44 openstackgerrit Merged openstack/nova stable/queens: Allow 'network' in RequestContext service_catalog https://review.openstack.org/550259
22:16:53 openstackgerrit Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/550569
22:55:52 openstackgerrit Merged openstack/nova master: Run post-test archive against cell1 https://review.openstack.org/550194
23:01:40 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094
23:07:00 openstackgerrit Surya Seetharaman proposed openstack/nova-specs master: Support disabling a cell https://review.openstack.org/546684
23:30:59 openstackgerrit Surya Seetharaman proposed openstack/nova master: Update the nova-manage db archive_deleted_rows description https://review.openstack.org/550600
#openstack-nova - 2018-03-08
00:11:34 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc (1/3) https://review.openstack.org/528201
00:17:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc (1/3) https://review.openstack.org/528201
00:26:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc (2/3) https://review.openstack.org/550648
00:33:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc (2/3) https://review.openstack.org/550648
00:47:44 tetsuro artom: I’ve updated. ready for review https://review.openstack.org/#/c/543805/

Earlier   Later