Posted	Nick	Remark
#openstack-nova - 2017-05-04
01:58:48	jaypipes	mriedem: I don't see a question...
01:58:53	mriedem	this is a host aggregate with cpu_allocation_ratio meta set on it?
01:59:00	mriedem	is that the idea?
01:59:34	mriedem	jaypipes: so with placement in the filter scheduler we basically ignore the corefilter because we get compute node RPs via the VCPU resource class filtering right?
01:59:42	mriedem	we don't ignore it, but it's redundant
01:59:42	openstackgerrit	fengzhr proposed openstack/nova master: Modify the description of flat_injected in nova.conf https://review.openstack.org/458789
01:59:58	mriedem	and i think we said you could/should disable the CoreFilter, RAMFilter and DiskFilter once you're using placement
02:00:20	jaypipes	yeah...
02:00:34	jaypipes	but aggregate metadata should still be processed, IIRC.
02:01:00	mriedem	yeah i'm not sure why this would be a problem,
02:01:15	mriedem	it's just extra filtering once we get the list of compute node resource providers filtered on VCPU to start
02:01:36	mriedem	Kevin_Zheng: it's probably best to just report a bug with details
02:01:51	mriedem	especially because i have a date with my couch and tv
02:02:54	Kevin_Zheng	ha， i will
02:03:03	mriedem	good night all
02:03:08	mriedem	and day
02:05:47	Kevin_Zheng	As described in aggregatecorefilter it will try to get ratio from aggregate first and if not possible then fall back to per host ratio
02:08:03	Kevin_Zheng	If we use placement, then it directly selected hosts with per host ratio, if we have different configurations in aggregate meta, then the host selection might not be what we expected
02:10:04	Kevin_Zheng	Say, if people only use aggregatecorefilter as it will fall back to per host ratio, it won't work with placement
02:22:49	openstackgerrit	shaofeng cheng proposed openstack/nova master: Update nova version newton to ocata in upgrade.rst https://review.openstack.org/457197
03:18:35	openstackgerrit	Jackie Truong proposed openstack/nova master: Add missing rootwrap filter for cryptsetup https://review.openstack.org/462348
03:33:09	openstackgerrit	chenaidong1 proposed openstack/nova master: Fix a typo https://review.openstack.org/462349
03:38:41	openstackgerrit	chenaidong1 proposed openstack/python-novaclient master: Fix a typo https://review.openstack.org/462351
04:03:22	openstackgerrit	LiChunlin proposed openstack/nova master: update the description of hypervisor statistics response https://review.openstack.org/462355
04:52:56	openstackgerrit	ChangBo Guo(gcb) proposed openstack/nova master: Remove oslo.config deprecated parameter enforce_type https://review.openstack.org/462364
07:25:46	openstackgerrit	jichenjc proposed openstack/nova master: Use plain routes list for os-instance-usage-audit-log endpoint instead of stevedore https://review.openstack.org/461647
07:27:02	openstackgerrit	jichenjc proposed openstack/nova master: use plain routes list for os-simple-tenant-usage https://review.openstack.org/460821
07:33:47	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Use plain routes list for os-server-groups endpoint instead of stevedore https://review.openstack.org/462169
07:38:58	openstackgerrit	Andreas Scheuring proposed openstack/nova master: DO NOT MERGE https://review.openstack.org/462421
07:48:04	openstackgerrit	jichenjc proposed openstack/nova master: Use plain routes list for os-quota-classes endpoint instead of stevedore https://review.openstack.org/461641
07:49:29	openstackgerrit	jichenjc proposed openstack/nova master: refactory some code on get_password https://review.openstack.org/449458
07:52:26	openstackgerrit	Alex Xu proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470
07:53:57	openstackgerrit	Alex Xu proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470
08:13:21	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049
08:19:48	sfinucan	mdbooth: Could you take a look at https://review.openstack.org/#/q/topic:disk_info_processing today? It's been around a long time and it seems like something you should look at first
08:26:32	openstackgerrit	shaofeng cheng proposed openstack/nova master: Replace newton to release_name in upgrade.rst https://review.openstack.org/457197
08:48:12	openstackgerrit	jichenjc proposed openstack/nova master: [placement] Add POST for resource provider classes https://review.openstack.org/462441
08:49:51	sfinucan	johnthetubaguy: Are we allowed to delete configuration without a deprecation cycle, even if they do nothing? https://review.openstack.org/#/c/459470/
08:50:46	johnthetubaguy	sfinucan: so cloudpipe only works with nova-network, and nova-network has been deprecated (on and off) for a long time
08:50:52	johnthetubaguy	sfinucan: I would give that one a pass
08:51:51	sfinucan	johnthetubaguy: Cool. Sounds fair
08:54:18	openstackgerrit	jichenjc proposed openstack/nova master: [placement] Add POST for resource provider classes https://review.openstack.org/462441
08:56:49	johnthetubaguy	sfinucan: +1 deleting that already deprecated option that does nothing now
08:56:59	abhi89	johnthetubaguy: can you please review https://review.openstack.org/#/c/460305/
08:57:28	johnthetubaguy	abhi89: ah, I should take a look at that now
09:02:35	johnthetubaguy	abhi89: I really like that regression test now, really nice proof that your change works
09:04:00	abhi89	johnthetubaguy: yes.. i have well tested the changes also
09:04:55	johnthetubaguy	abhi89: I just love seeing the automated tests like that, really stops us breaking your fix by accident later
09:05:46	abhi89	johnthetubaguy: yes.. Thanks John
09:06:09	johnthetubaguy	abhi89: thank you for sticking with it, great to get that fixed
09:10:13	sfinucan	johnthetubaguy: Could you take a punt at https://review.openstack.org/#/c/431037/ and successors today?
09:11:18	johnthetubaguy	sfinucan: yeah, I want to sit down with the quota code again at some point though
09:11:30	openstackgerrit	falseuser proposed openstack/nova master: [BugFix] Change the parameter of the exception error message https://review.openstack.org/461985
09:14:40	sfinucan	johnthetubaguy: ah, at some point then :)
09:24:01	sfinucan	johnthetubaguy: Ta :)
09:58:51	openstackgerrit	Stephen Finucane proposed openstack/nova master: Fix default_availability_zone docs https://review.openstack.org/438887
09:58:52	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Move availability zones opts to a group https://review.openstack.org/462469
10:13:07	openstackgerrit	Stephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625
10:13:07	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add PCIWeigher https://review.openstack.org/379524
10:14:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: tools/xenserver: Remove 'cleanup_sm_locks' https://review.openstack.org/416520
10:15:17	sfinucan	bauzas: Fancy taking a look at two availability zone changes? https://review.openstack.org/438887 https://review.openstack.org/462469
11:16:28	openstackgerrit	Matthew Booth proposed openstack/nova master: Fix scope of instance error context manager in resize_instance https://review.openstack.org/462499
11:16:54	mdbooth	Hopefully pretty simple bugfix ^^^
11:20:46	sean-k-mooney	hi
11:21:02	sean-k-mooney	is there a way to make devstack not use systemd still
11:21:23	mdbooth	sean-k-mooney: That got pushed yesterday, I think.
11:21:36	mdbooth	https://review.openstack.org/#/c/462163/3
11:21:45	sean-k-mooney	swapping to systemd or keeping the legacy way
11:22:02	mdbooth	Keeps systemd option, but defaults to legacy
11:22:37	sean-k-mooney	i pulled devstack today and it defaulted to systemd instead of screen
11:23:55	sean-k-mooney	but ok ill look into it. i hate systemd... this might actully be the thing that makes me stop using devstack for my development
11:24:55	sean-k-mooney	ah i have to set USE_SCREEN=True to get the old behavior
11:25:21	dpawlik	Hello. Can someone of you tell me if I should add DocImpact to the commit message in this PS: https://review.openstack.org/#/c/459753/ ?
11:27:00	sean-k-mooney	dpawlik: you will need to update the docs for the new image metadata item but i belive that would be in glance's docs not novas
11:28:39	dpawlik	sean-k-mooney: hey. So better when I add here DocImpact and ask glance team to add it in its docs
11:30:01	sean-k-mooney	dpawlik: https://wiki.openstack.org/wiki/Documentation/DocImpact
11:30:38	dpawlik	sean-k-mooney: thx
11:44:23	openstackgerrit	Daniel Pawlik proposed openstack/nova master: Added kvm_hidden feature https://review.openstack.org/459753
11:45:07	sean-k-mooney	sdague: hi, 2 quick quetstions regarding the new systemd workflow for devstack will the screen workflow be tested and maintined going forword? also will devstack now work across reboots again?
11:49:44	sdague	sean-k-mooney: screen goes away in Queens
11:50:03	sdague	http://lists.openstack.org/pipermail/openstack-dev/2017-May/116301.html
11:50:15	sdague	I'm not sure what works across reboots means
11:50:37	sdague	mdbooth: that's entirely possible...
11:50:51	mdbooth	I don't think the journal itself drops messages
11:51:02	sdague	mdbooth: we did disable it now, so we're out of that woods for the moment
11:51:09	mdbooth	Yeah, makes sense
11:51:20	mdbooth	One to investigate though, for sure
11:53:37	sean-k-mooney	sdague: ok i might need to make changes to our devstack plugins/CIs.
11:54:11	sean-k-mooney	i think they will work but they may not
11:54:32	sdague	sean-k-mooney: yeh, that thread has some common issues people hit
11:56:21	sean-k-mooney	sdague: i guess il have to give it a fair shot but honestly the fact that the process ran in screen sessions was one of the things i liked about devstack as it made it easy to wrap your head around when you were new to openstack
11:57:06	kashyap	sean-k-mooney: It isn't any more? I haven't run a DevStack run in a couple of weeks
11:57:12	sdague	sean-k-mooney: ok, an equally large number of people have stated that screen was the most confusing thing ever
11:57:24	sdague	but, regardless, there are other reasons we need this change
11:57:35	kashyap	I've seen the systemd / 'screen' related change whoosh by. /me looks closer...
11:57:50	kashyap	Hmm, it's this patch: "make USE_SCREEN=False imply USE_SYSTEMD=True"
11:58:50	sean-k-mooney	yes i swapped the flag back and it still failed to stack so my local failure wasnt related to the systemd change so im going to start with a clean environment again
11:59:14	sean-k-mooney	kashyap: ^
11:59:31	kashyap	sean-k-mooney: What is the implication of that patch? Still reading this - https://review.openstack.org/#/c/460062/
11:59:36	sean-k-mooney	sdague: screen not being maintianed for one

Earlier   Later