Posted	Nick	Remark
#openstack-nova - 2019-03-06
07:04:41	openstackgerrit	melanie witt proposed openstack/nova master: Count instances from mappings and cores/ram from placement https://review.openstack.org/638073
07:04:42	openstackgerrit	melanie witt proposed openstack/nova master: Use instance mappings to count server group members https://review.openstack.org/638324
07:06:35	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Add description for BDM volume_size https://review.openstack.org/641255
07:10:54	openstackgerrit	melanie witt proposed openstack/nova master: Add user_id column to the instance_mappings table https://review.openstack.org/633349
07:10:55	openstackgerrit	melanie witt proposed openstack/nova master: Populate InstanceMapping.user_id during migrations and schedules https://review.openstack.org/638574
07:10:55	openstackgerrit	melanie witt proposed openstack/nova master: Add user_id field to InstanceMapping https://review.openstack.org/633350
07:10:56	openstackgerrit	melanie witt proposed openstack/nova master: Add get_counts() to InstanceMappingList https://review.openstack.org/638072
07:10:56	openstackgerrit	melanie witt proposed openstack/nova master: Add online data migration for populating user_id https://review.openstack.org/633351
07:10:57	openstackgerrit	melanie witt proposed openstack/nova master: Use instance mappings to count server group members https://review.openstack.org/638324
07:10:57	openstackgerrit	melanie witt proposed openstack/nova master: Count instances from mappings and cores/ram from placement https://review.openstack.org/638073
07:16:36	openstackgerrit	Abhishek Kekane proposed openstack/nova-specs master: Support multiple backend of Glance https://review.openstack.org/641210
07:47:05	yonglihe	sean-k-mooney: ping
07:56:02	yonglihe	https://docs.google.com/document/d/1kRRZFq_ha0T9mFDOEzv0PMvXgtnjGm5ii9mSzdqt1VM/edit#heading=h.kn16u3i4dj85
07:57:56	yonglihe	I collected your commets in google docs and anyone could edit or comment it, i given my opion about them, some of them needs discuss.
08:42:40	openstackgerrit	Merged openstack/nova master: Validate PCI aliases early in resize https://review.openstack.org/641131
08:50:33	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Should not skip volume_size check for bdm.image_id == image_ref case https://review.openstack.org/641282
09:05:35	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Use Selection object to fill request group mapping https://review.openstack.org/638711
10:05:29	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Warn if group_policy is missing from flavor https://review.openstack.org/641289
10:06:09	gibi	simple patch ^^
10:19:24	bauzas	gibi: +2d
10:21:22	openstackgerrit	Merged openstack/nova master: Move additional IP address management to privsep. https://review.openstack.org/624591
10:23:30	gibi	bauzas: thanks
10:24:18	gibi	bauzas: this the that API patch for bandwidht, turning the functionalty on https://review.openstack.org/#/c/636360/
10:24:28	gibi	bauzas: mriedem already +2-d it
10:24:44	bauzas	I know, I was planning to vote on it soon
10:24:49	gibi	bauzas: cool. thanks
10:33:38	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Test macvtap port with resource request https://review.openstack.org/641309
10:33:59	gibi	it is a functional test only patch ^^
10:34:02	gibi	sean-k-mooney: ^^
10:52:58	openstackgerrit	Neil Jerram proposed openstack/nova master: Allow utime call to fail on Flat image base file https://review.openstack.org/641315
11:06:21	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Store InstancePCIRequest.numa_policy in DB https://review.openstack.org/624444
11:06:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Create PCI tests for NUMA'y tests https://review.openstack.org/640826
11:06:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: fakelibvirt: Add ability to generate fake PCI devices https://review.openstack.org/640409
11:06:48	stephenfin	gibi, bauzas: Any chance you could take a look at those today (or at least the bottom one)?
11:07:47	bauzas	stephenfin: sure, I dedicated my time today for reviews
11:07:57	bauzas	spread the word
11:08:26	gibi	stephenfin: looking
11:14:09	gibi	stephenfin: I'm +2 on the whole series. It really made those sriov tests more readable at the end
11:14:32	stephenfin	gibi: Sweet. Thanks.
11:15:37	stephenfin	gibi: I'd been thinking about maybe using Jinja2 templates for that entire file, but I'll tackle that later/separately :)
11:16:00	stephenfin	gibi: If it's not too much, this might also be of interest to you https://review.openstack.org/#/c/636383/ (another clean up'y one)
11:18:31	gibi	stephenfin: I will check that after launch
11:18:34	gibi	lunch
11:18:38	stephenfin	ta :)
11:46:11	openstackgerrit	Neil Jerram proposed openstack/nova master: Allow utime call to fail on qcow2 image base file https://review.openstack.org/641315
12:25:51	openstackgerrit	Merged openstack/nova master: Fix WeighedHost logging regression https://review.openstack.org/641143
13:15:58	openstackgerrit	Eric Fried proposed openstack/nova master: Warn if group_policy is missing from flavor https://review.openstack.org/641289
13:23:21	frickler	eandersson: check whether your compute endpoint is v2 or v2.1. the former doesn't support microversions.
13:24:47	frickler	probably osc should do some better guarding against that
13:25:39	sean-k-mooney	frickler: osc does not support auto microversions either but yes if you pass one explcitly i coudl check that you are not using the v2 endpoint
13:26:36	frickler	sean-k-mooney: see the backlog from yesterday. "openstack --os-compute-api-version 2.15 server group create anti-affinity-test --policy soft-anti-affinity" fails when I change my compute endpoint url from /v2.1 to /v2
13:27:39	sean-k-mooney	frickler: sure you are using an explcitly microverions it could guard against that but osc doese not really have any special logic to handel microversions
13:29:38	frickler	sean-k-mooney: hmm, o.k., maybe then nova should throw an error when the client requests 2.15 instead of silently falling back to 2.0 behaviour
13:30:03	sean-k-mooney	that woudl be a backwards incompatiable chagne
13:30:12	sean-k-mooney	and would break those that expect it
13:30:34	sean-k-mooney	the v2 endpoint is unversioned so by deffinion it shoudl not be doing version checks
13:30:54	yonglihe	sean-k-mooney: hi, good morning
13:31:26	openstackgerrit	Andrey Volkov proposed openstack/nova stable/rocky: Check hosts have no instances for AZ rename https://review.openstack.org/641351
13:32:05	frickler	sean-k-mooney: hmm, o.k., let's wait for eandersson to confirm whether I located his issue before continuing that discussion ;)
13:38:19	mriedem	sean-k-mooney: https://blueprints.launchpad.net/nova/+spec/libvirt-neutron-sriov-livemigration is next in the runways slot but is that going to make stein in the next 24 hours?
13:38:40	mriedem	my guess is no looking at the state of the series
13:40:34	yonglihe	mriedem, sean-k-mooney: any suggest on https://review.openstack.org/#/c/621476/? sean-k-mooney had quiet a lot concern about the sockets/threads/cores that reported to user.
13:41:39	mriedem	yonglihe: the contents of that regarding numa are not my strong area so i will have to defer to sean-k-mooney and stephenfin and others
13:43:03	yonglihe	ok, i collected them into https://docs.google.com/document/d/1kRRZFq_ha0T9mFDOEzv0PMvXgtnjGm5ii9mSzdqt1VM/edit# , might be easier to work on. sean-k-mooney , stephenfin.
13:43:19	mriedem	tssurya: i copied you on an email to jose about https://review.openstack.org/#/c/613039/ since he's at cern as well - just fyi
13:43:33	yonglihe	but seems time for stein is quite tight.
13:43:34	mriedem	tssurya: since we've got <24 hours to land that
13:43:42	mriedem	yonglihe: yes it is
13:43:53	tssurya	mriedem: ack, thanks
13:44:17	mriedem	tssurya: i'm of course assuming cern is very small and everyone knows everyone else and what they are working on...
13:44:42	sean-k-mooney	yonglihe: o/
13:44:43	tssurya	mriedem: yep I am going to pass by his office now :)
13:44:50	mriedem	tssurya: oh heh i was just joking
13:44:51	mriedem	but cool
13:45:13	openstackgerrit	Merged openstack/nova master: Validate bandwidth configuration for other VIF types https://review.openstack.org/636383
13:45:22	openstackgerrit	Merged openstack/nova master: Further de-dupe os-vif VIF tests https://review.openstack.org/636384
13:46:40	sean-k-mooney	yonglihe: so overall i dont think the datamodel is quite right. given the time constraitns i personally would be more comfortable waiting for train to finalise it but i can take a look at the google doc. etherpad thens to be a good choice for this kind of thing also
13:47:54	sean-k-mooney	yonglihe: one of the main issues is the current proposed api is coupeling cpu topology and numa topology but they are independnt fo each other which is why im concerned the current data model is missleading
13:48:31	openstackgerrit	Matt Riedemann proposed openstack/nova stable/rocky: Fix WeighedHost logging regression https://review.openstack.org/641355
13:49:37	yonglihe	sean-k-mooney, yeah, i get that. it's mass. how about delete the cpu topology toltally.
13:51:00	bauzas	gibi: +Wd with a comment on https://review.openstack.org/#/c/636360/23
13:51:02	yonglihe	considerating the time factor, it's up to you choosing what we can do on the stein, or postpone to next release. both ok for me.
13:51:19	sean-k-mooney	you could or you could group it seperatly. e.g. in the toployg endpoint you get back a dict with two fields {cpu_toplolgoy:{}, numa_toplogy[{},{}]}
13:51:49	gibi	bauzas: thanks, replied
13:52:05	tssurya	mriedem: josecastroleon is working on it
13:52:10	bauzas	gibi: great, that works with me
13:52:17	gibi	cool
13:52:19	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: Fix WeighedHost logging regression https://review.openstack.org/641359
13:52:20	bauzas	for docs, I dunno where to land docs
13:52:30	bauzas	given that's both neutron/nova
13:53:03	bauzas	stephenfin: thoughts for doc'ing bandwith RPs feature ?
13:53:08	bauzas	in nova, neutron or both ?
13:53:22	gibi	bauzas: from nova perspective only the API limitations and the vrit driver dependency are externally visible, the rest is neutron configuration and neutron API
13:53:30	bauzas	stephenfin: how can we write some admin notes given docs are now per project ?
13:53:47	stephenfin	bauzas: Depends on where most of the work has to be done. It _feels_ like more of a neutron thing, IMO
13:54:02	bauzas	oki doki, I'm good with this
13:54:22	stephenfin	bauzas: SR-IOV networking is probably the closest thing we have and that is in the neutron guide, though we do have references to it in our docs
13:54:43	mriedem	tssurya: thanks
13:54:44	bauzas	that's my point, we probably still need references to it
13:55:32	stephenfin	sean-k-mooney: what was the reply to mriedem's question about the libvirt-neutron-sriov-livemigration spec? You think it's reasonable to land

Earlier   Later