Posted	Nick	Remark
#openstack-nova - 2017-10-02
21:36:08	melwitt	you can set those limits to be different per project or per project and user
21:36:49	sean-k-mooney	melwitt: so there is no global limit today for how much data a single project/user can store across all there instances today
21:37:06	mriedem	well, personality files are not stored in the db
21:37:25	mriedem	i think the point is that they default to a global value, checked per request,
21:37:33	mriedem	but you can override those defaults per project or per project/user
21:37:37	mriedem	using the os-quota-sets API
21:39:09	melwitt	sean-k-mooney: right. injected file data isn't counted across projects or anything. it's only evaluated at the time of instance create, that the values you've requested don't exceed the limits. by default you can't boot an instance with more than 5 injected_files, each file has to be <= a certain length, each path has to be <= a certain length
21:39:36	sean-k-mooney	mriedem: ah thats the delta to user-data then which is stored in the db.
21:40:01	mriedem	yeah
21:40:13	mriedem	spec will be incoming shortly
21:41:30	sean-k-mooney	so are injected files kept in ram until the compute node actully creates the instance. if so that was proably part of the reason for limiting there overall lenght per request
21:46:20	melwitt	yeah, could be. I don't know the history about it
21:52:55	openstackgerrit	Ed Leafe proposed openstack/nova-specs master: Return Alternate Hosts https://review.openstack.org/504275
21:53:23	sean-k-mooney	johnthetubaguy: just getting ready to reupload https://review.openstack.org/#/c/375580/2/specs/ocata/approved/neutron-new-port-binding-api.rst is the createive common license correct for specs?
21:57:47	openstackgerrit	sean mooney proposed openstack/nova-specs master: WIP: Use neutron's new port binding API https://review.openstack.org/375580
21:57:59	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Deprecate file injection https://review.openstack.org/509013
21:58:17	mriedem	sean-k-mooney: it's part of the spec template
21:59:04	sean-k-mooney	mriedem: cool i just assumed it would be apache 2 is all but i guess its really a doc and not code so makes sense
23:02:14	openstackgerrit	Matt Riedemann proposed openstack/python-novaclient stable/newton: Fix aggregate_update name and availability_zone clash https://review.openstack.org/507816
#openstack-nova - 2017-10-03
00:06:36	openstackgerrit	Jay Pipes proposed openstack/nova master: rp: Move RP._get|set_aggregates() to module scope https://review.openstack.org/509026
00:06:36	openstackgerrit	Jay Pipes proposed openstack/nova master: rp: de-ORM ResourceProvider.get_by_uuid() https://review.openstack.org/509025
00:06:37	openstackgerrit	Jay Pipes proposed openstack/nova master: rp: move RP._set_traits() to module scope https://review.openstack.org/509028
00:06:37	openstackgerrit	Jay Pipes proposed openstack/nova master: rp: Remove RP.get_traits() method https://review.openstack.org/509027
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

Earlier   Later