Posted	Nick	Remark
#openstack-nova - 2017-10-03
00:06:40	openstackgerrit	Jay Pipes proposed openstack/nova master: rp: remove bad comment in AllocList.create_all() https://review.openstack.org/509034
00:06:40	openstackgerrit	Jay Pipes proposed openstack/nova master: rp: fix up AllocList.get_by_resource_provider_uuid https://review.openstack.org/509033
00:06:41	openstackgerrit	Jay Pipes proposed openstack/nova master: rp: remove _HasAResourceProvider mixin https://review.openstack.org/509036
00:06:41	openstackgerrit	Jay Pipes proposed openstack/nova master: rp: rework AllocList.get_all_by_consumer_id() https://review.openstack.org/509035
00:51:54	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove 400 as expected error https://review.openstack.org/509039
01:51:53	openstackgerrit	melanie witt proposed openstack/nova-specs master: Propose counting quota usage from placement https://review.openstack.org/509042
03:12:56	openstackgerrit	Tony Breeds proposed openstack/nova master: [DNM] Yesting the legacy-requirements job https://review.openstack.org/509054
03:45:43	openstackgerrit	Philip Choi proposed openstack/nova master: commit ce26619af3f7ac8c55567d6aa8125b77e457af19 Author: Philip Choi Date: Fri Sep 8 10:52:42 2017 +0800 https://review.openstack.org/509060
07:07:48	openstackgerrit	Merged openstack/nova master: check query param for service's index function https://review.openstack.org/489492
07:08:05	openstackgerrit	Merged openstack/nova master: Do not setup conductor in BaseAPITestCase https://review.openstack.org/508120
07:27:29	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove 400 as expected error https://review.openstack.org/509039
07:44:25	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
07:47:41	bauzas	good specs day to everyone
07:48:45	openstackgerrit	Radoslav Gerganov proposed openstack/nova master: VMware: serial console log (completed) https://review.openstack.org/450636
07:53:08	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Move 'ips' field from Subnet object to VIF object https://review.openstack.org/508498
07:58:22	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix test_get_volume_config method https://review.openstack.org/489467
08:19:50	openstackgerrit	sahid proposed openstack/nova master: pci: add for PciDevice object new field mdev https://review.openstack.org/508176
08:19:50	openstackgerrit	sahid proposed openstack/nova master: pci: update PciDevice object field 'address' to accept NULL https://review.openstack.org/508175
08:19:51	openstackgerrit	sahid proposed openstack/nova master: pci: add support for mdev device type request https://review.openstack.org/508178
08:19:51	openstackgerrit	sahid proposed openstack/nova master: pci: generalize object unit-tests for different framework https://review.openstack.org/508177
08:19:52	openstackgerrit	sahid proposed openstack/nova master: pci: add support for mdev devices type in devspec https://review.openstack.org/508180
08:19:52	openstackgerrit	sahid proposed openstack/nova master: pci: generalize stats unit-tests for different framework https://review.openstack.org/508179
08:19:53	openstackgerrit	sahid proposed openstack/nova master: pci: make manager to accept handling mdev devices https://review.openstack.org/508182
08:19:53	openstackgerrit	sahid proposed openstack/nova master: pci: add support for resource pool stats of mdev devices https://review.openstack.org/508181
08:19:54	openstackgerrit	sahid proposed openstack/nova master: libvirt: report mdev resources https://review.openstack.org/508184
08:19:54	openstackgerrit	sahid proposed openstack/nova master: libvirt: update PCI node device to report mdev devices https://review.openstack.org/508183
08:19:55	openstackgerrit	sahid proposed openstack/nova master: functional: rework fakelibvirt host pci devices https://review.openstack.org/508186
08:19:55	openstackgerrit	sahid proposed openstack/nova master: libvirt: add support to start vm with using mdev (vGPU) https://review.openstack.org/508185
08:19:56	openstackgerrit	sahid proposed openstack/nova master: WIP - functional: rewrite class to generate pci devices https://review.openstack.org/508835
08:19:56	openstackgerrit	sahid proposed openstack/nova master: functional: resuse SRIOV funtional tests for MDEV devices https://review.openstack.org/508187
08:25:11	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Change Subnet.dhcp_server to contain multiple IPs https://review.openstack.org/509107
09:13:20	openstackgerrit	Jan Zerebecki proposed openstack/nova master: Fix wording of debug message for future releases https://review.openstack.org/508261
09:17:53	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405
09:22:40	openstackgerrit	Jan Zerebecki proposed openstack/nova master: Fix wording of debug message for future releases https://review.openstack.org/508261
09:27:59	cdent	bauzas: that’s a great looking dog you have
09:28:17	bauzas	cdent: hah thanks
09:28:20	bauzas	Eurasier FTW
09:30:32	bauzas	FWIW, since it's specs review day, I'm just fixing my own dashs because of Gerrit 2.13 :)
09:31:52	bauzas	in case people face the same problem, sdague explained the issue in http://lists.openstack.org/pipermail/openstack-dev/2017-September/122277.html
09:53:58	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Intel Fortville Dynamic Device Personalization (DDP) https://review.openstack.org/503001
09:59:10	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Intel Fortville Dynamic Device Personalization (DDP) https://review.openstack.org/503001
10:20:22	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/504895
10:31:50	openstackgerrit	Chris Dent proposed openstack/nova-specs master: Fix issues for post-allocations spec https://review.openstack.org/509136
10:32:07	cdent	efried: that ^ should fix some of your comments, thanks for them
10:37:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove dead code of api.fault notification sending https://review.openstack.org/505164
10:45:11	openstackgerrit	Stephen Finucane proposed openstack/nova master: doc: Add documentation for cpu_realtime, cpu_realtime_mask https://review.openstack.org/502056
10:46:21	stephenfin	gibi, bauzas: Any chance of a (re-)review of ^
10:52:36	openstackgerrit	Chris Dent proposed openstack/nova-specs master: Spec for limiting GET /allocation_candidates https://review.openstack.org/504540
10:59:11	tetsuro	The patch I brought in the latest PTG is ready for review https://review.openstack.org/#/c/465160/.
10:59:37	tetsuro	This is a bug that falls into silent error with NUMATopology when virt_type is not set to kvm.
11:18:09	gibi	stephenfin: done. +2
11:18:44	gibi	stephenfin: thanks for the respin of the api.fault check. Is there something on master that will help pleasing jenkins? Should I rebase my other patches as well?
11:18:58	gibi	s/check/patch.//
11:54:49	openstackgerrit	Chris Dent proposed openstack/nova master: Update RT aggregate map less frequently https://review.openstack.org/489633
12:40:58	stephenfin	sahid: Can you reword the comment here? I'm not sure what you mean https://review.openstack.org/#/c/461456/5/nova/virt/hardware.py
12:45:44	sahid	stephenfin: yes i will, there are several points on what i'm not agree with that change
12:46:15	stephenfin	sahid: Cool, thanks. I don't personally see the downside so there's a chance I'm missing something
12:49:15	sahid	well firstable we add new syntax to provide the same fonctionnality, the previous one already brought bugs, adding a new one is not going to help
12:51:33	sahid	then with this new syntax, we allow users to specify the set of vCPUs used to apply the mask on where in my sense that shouldn't be allowed, the mask is applied on the set of vCPUs of the guest which Nova do provide
12:54:03	stephenfin	Yeah, they do achieve basically the same thing. However, from a usability perspective, I do think this second pattern is more intuitive
12:54:43	stephenfin	I mean, if the option was called 'cpu_realtime_excludes' or something, then the current pattern would make sense (and the carat wouldn't be necessary)
12:55:46	stephenfin	However, while nova definitely should keep track of total number of CPUs, I don't see why we should allow "explicit exclude-implicit include", but not "explicit include-implicit exclude"
12:56:36	stephenfin	"explicit include-explicit exclude" is an odd one, but it's no harm and already works, so if someone's silly enough to do it I don't see why we shouldn't just let them
12:57:03	stephenfin	This is a usability improvement and nothing more, IMO
12:57:31	sahid	ok that is the subjective point, i can hear it. what about the fact we add more and more code to achieve the same result?
13:13:20	mriedem	blarg my nova-specs dashboard doesn't work with new gerrit
13:14:14	cdent	mriedem: bauzas had some links earlier today about ways to fix that
13:14:39	bauzas	mriedem: yeah, I had to modify my own dashes
13:14:44	cdent	mriedem: http://lists.openstack.org/pipermail/openstack-dev/2017-September/122277.html
13:14:59	sdague	bauzas / mriedem are you using the ones from upstream?
13:15:04	bauzas	mriedem: tl;dr: labeling your votes doesn't work, you need to use another one
13:15:17	sdague	I tried to fix everything in that repo
13:15:25	sdague	but, I'm sure they could use tweaks
13:15:33	stephenfin	Yeah, https://github.com/openstack/gerrit-dash-creator/blob/master/dashboards/nova-specs.dash looks good to me
13:15:45	stephenfin	and I'm using https://github.com/openstack/gerrit-dash-creator/blob/master/dashboards/nova.dash daily
13:15:59	bauzas	sdague: I'm using the same queries than yours, but I use https://review.openstack.org/#/settings/preferences
13:16:10	sdague	bauzas: ok, cool
13:16:31	sdague	bauzas: yeh, I end up just using browser bookmarks for them as they sync between chrome
13:16:34	bauzas	sdague: so, tbh I modified my queries thanks to you :)
13:17:03	bauzas	that, plus Zuul v3 \o/
13:17:12	bauzas	https://docs.openstack.org/infra/manual/zuulv3.html
13:17:58	bauzas	is anyone currently working on moving our jobs to the nova repo btw. ?
13:18:12	bauzas	sdague: do you know that ^ ?
13:19:04	jaypipes	sdague, mriedem: is it worth rechecking anything at the moment?
13:19:26	sdague	jaypipes: I don't know, I rechecked a few things, I'll let you know if anything works
13:19:32	jaypipes	kk
13:19:48	mriedem	jaypipes: don't think so
13:19:54	sdague	bauzas: no, but given that zuul v3 is still not passing many things, it didn't seem really useful to change the queries yet
13:19:54	jaypipes	sdague: seeing a lot of POST_FAILURE stuff right now...
13:19:56	mriedem	what i have rechecked is not queueing up
13:20:11	sdague	jaypipes: yeh... that's what I saw from stuff that hit lastnight
13:20:36	jaypipes	mordred: any particular activity that would be useful from us in identifying issues with Zuulv3?
13:20:39	bauzas	sdague: well, you're right
13:20:53	bauzas	jaypipes: there is an etherpad
13:21:04	bauzas	jaypipes: https://etherpad.openstack.org/p/zuulv3-migration-faq
13:21:08	jaypipes	bauzas: cheers
13:21:54	sdague	mriedem: you want me to restrict down the specs dashboard to only stuff proposed for queens?
13:22:26	bauzas	sdague: it's another dash I guess
13:22:39	bauzas	sdague: honestly, I'm directly querying for that
13:23:17	mriedem	sdague: no that's fine

Earlier   Later