Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-03
00:06:38 openstackgerrit Jay Pipes proposed openstack/nova master: rp: streamline InventoryList.get_all_by_rp_uuid() https://review.openstack.org/509030
00:06:38 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove CRUD operations on Inventory class https://review.openstack.org/509029
00:06:39 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove ability to delete 1 allocation record https://review.openstack.org/509032
00:06:39 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove dead code in Allocation._create_in_db() https://review.openstack.org/509031
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

Earlier   Later