Wednesday, 2016-11-30

melwittyeah, I'm sure he has actually00:00
*** Swami_ has quit IRC00:00
jaypipesedleafe: each ironic node will be a separate resource provider in the placement API. and each of those providers will have a single inventory record with a resource class that matches the node.resource_class returned from the Ironic API for that node.00:00
edleafejaypipes: ah, so nested turtles all the way down.00:02
*** ducttape_ has joined #openstack-nova00:03
*** WillUK2010 has quit IRC00:04
*** kristian__ has quit IRC00:05
*** ijw has quit IRC00:06
*** sambetts|afk has quit IRC00:08
*** annegentle has quit IRC00:10
openstackgerritTakashi NATSUME proposed openstack/nova: Remove mox in test_block_device_mapping_v1.py  https://review.openstack.org/30874600:14
*** penick_ has quit IRC00:16
*** ijw has joined #openstack-nova00:20
*** hongbin has quit IRC00:20
*** thorst_ has joined #openstack-nova00:21
*** hfu has joined #openstack-nova00:22
*** thorst_ has quit IRC00:22
*** thorst_ has joined #openstack-nova00:23
*** diogogmt has joined #openstack-nova00:24
*** ijw has quit IRC00:25
*** annegentle has joined #openstack-nova00:26
*** thorst_ has quit IRC00:31
*** Swami has joined #openstack-nova00:32
*** liverpooler has quit IRC00:32
*** hfu has quit IRC00:35
*** edleafe- has joined #openstack-nova00:36
*** edleafe has quit IRC00:36
melwittmriedem: on https://review.openstack.org/#/c/401375 I'm good with the change but based on my comment it seems like I should update the commit message to explain more. what do you think?00:37
*** hfu has joined #openstack-nova00:39
*** diga has joined #openstack-nova00:40
digaHi00:40
digaI am getting errot in 2016-11-30 00:30:15.293 |     output_file = etc/nova/policy.yaml00:41
diga2016-11-30 00:30:15.292 | error: expected '<document start>', but found '<scalar>'00:41
digawhile installing devstack00:41
digathis error comes when devstakc try to run below command00:41
*** yuntongjin has joined #openstack-nova00:41
digacommand /usr/local/bin/nova-manage --config-file /etc/nova/nova.conf db sync00:42
*** ijw has joined #openstack-nova00:42
*** dimtruck is now known as zz_dimtruck00:43
*** yingjun has joined #openstack-nova00:44
*** nherciu has quit IRC00:44
*** ijw has quit IRC00:47
*** unicell has quit IRC00:48
digaedleafe-: Hi00:49
*** ducttape_ has quit IRC00:51
*** hoangcx has joined #openstack-nova00:55
*** jwcroppe has joined #openstack-nova00:55
*** nic has quit IRC00:56
*** jwcroppe has quit IRC00:56
*** jwcroppe has joined #openstack-nova00:56
openstackgerritliaonanhai proposed openstack/nova: Use uuidutils instead of uuid.uuid4()  https://review.openstack.org/39477500:57
*** mtanino has quit IRC00:59
*** ducttape_ has joined #openstack-nova01:01
*** gjayavelu has quit IRC01:01
*** gjayavelu has joined #openstack-nova01:03
*** gjayavelu has quit IRC01:04
*** giri has joined #openstack-nova01:04
openstackgerritSTEW TY proposed openstack/nova: Transform instance.rescue notifications  https://review.openstack.org/38564401:04
*** rama_y has quit IRC01:05
*** thorst_ has joined #openstack-nova01:06
*** esberglu has joined #openstack-nova01:06
*** dave-mccowan has joined #openstack-nova01:07
*** yuntongjin has quit IRC01:07
*** diga has quit IRC01:08
mriedemmelwitt: i agree the commit message could be better,01:08
mriedemmelwitt: if you want, you could just tweak that yourself and then +W01:08
mriedemor i can re-+201:08
mriedemi'll be on for awhile01:08
melwittmriedem: cool, that's what I was thinking01:09
*** jamesdenton has joined #openstack-nova01:10
*** thorst_ has quit IRC01:13
*** tovin07_ has quit IRC01:16
*** esberglu has quit IRC01:17
*** baoli has joined #openstack-nova01:18
*** baoli has quit IRC01:19
*** baoli has joined #openstack-nova01:19
*** jamesdenton has quit IRC01:21
*** Swami_ has joined #openstack-nova01:22
*** Swami has quit IRC01:22
*** ijw has joined #openstack-nova01:24
openstackgerritmelanie witt proposed openstack/nova: Fix wait for detach code to handle 'disk not found error'  https://review.openstack.org/40137501:26
melwittmriedem: I updated the commit message and +2ed01:27
mriedem+W, thanks01:28
mriedemhttps://review.openstack.org/#/c/403988/ should also fix some of our ceph job woes01:28
melwittmriedem: a-ha, nice01:29
mriedemturns out if you're testing and creating >= 8 1gb volumes at the same time, the host should probably have at least 8gb of storage capacity01:30
melwittI never knew that was related to the disk size01:30
melwittthe NoValidHost01:30
melwitthaha, really!01:30
*** ijw has quit IRC01:31
*** annp has joined #openstack-nova01:32
*** neilhuang has quit IRC01:34
*** vearnx has joined #openstack-nova01:34
*** Zer0Byte__ has quit IRC01:35
*** Swami_ has quit IRC01:40
mriedemwznoinsk: looks like intel nfv ci hits some timeouts during test runs http://intel-openstack-ci-logs.ovh/25/403925/1/check/tempest-dsvm-full-nfv-xenial/c0d0c17/console.html01:40
mriedemwznoinsk: how big are the vms that those jobs run on?01:42
mriedemupstream is i think 8vcpu 8gb of ram01:42
*** kristian__ has joined #openstack-nova01:49
*** gcb has joined #openstack-nova01:50
*** kristian__ has quit IRC01:53
*** thorst_ has joined #openstack-nova01:55
*** kevinz has joined #openstack-nova02:00
*** tbachman has quit IRC02:01
*** thorst_ has quit IRC02:02
*** tbachman has joined #openstack-nova02:05
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port xenapi test_vm_utils to Python 3  https://review.openstack.org/40182702:07
*** kaisers_ has joined #openstack-nova02:07
*** ijw has joined #openstack-nova02:08
*** tlian has quit IRC02:09
openstackgerritZhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_security_group  https://review.openstack.org/30713502:09
openstackgerritZhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_instance_info_cache  https://review.openstack.org/30715502:09
openstackgerritZhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_compute_node  https://review.openstack.org/30762502:09
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port test_virt_drivers to Python 3  https://review.openstack.org/40341702:10
*** kaisers has quit IRC02:10
*** ijw has quit IRC02:13
*** sam_nowitzki has joined #openstack-nova02:13
*** liusheng has quit IRC02:14
*** tovin07 has joined #openstack-nova02:16
*** ijw has joined #openstack-nova02:18
*** ijw has quit IRC02:18
*** esberglu has joined #openstack-nova02:20
*** esberglu has joined #openstack-nova02:20
*** hongbin has joined #openstack-nova02:21
*** liusheng has joined #openstack-nova02:22
*** esberglu has quit IRC02:25
*** annegentle has quit IRC02:26
*** ijw has joined #openstack-nova02:29
*** stevemar__ has joined #openstack-nova02:29
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: cleanup network volume driver auth config  https://review.openstack.org/38938902:29
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: prefer cinder rbd auth values over nova.conf  https://review.openstack.org/38939902:29
mriedemmikal: i had to move ^ back to before your change, as it broke the unit tests02:30
mikalmriedem: huh02:30
mikalmriedem: I wonder why that tweak broke unit tests02:30
mriedemsee my reply in ps402:30
mikalOkie02:30
mriedemit's an if/elif, not an if/else02:31
*** diogogmt has quit IRC02:31
mikalOh, poo02:31
mriedemso it's valid to set no values on that conf thing02:31
mikalSorry, business like02:31
mikalBottom02:31
mikalMy apologies02:31
mriedempoopy butt farts is fine in here02:31
mikalAwesome02:31
*** tuanluong has joined #openstack-nova02:31
*** hongbin has quit IRC02:32
openstackgerritHuan Xie proposed openstack/nova: [DO NOT MERGE] Use os-xenapi repo  https://review.openstack.org/39310302:32
*** ijw has quit IRC02:33
smcginnislol02:33
openstackgerritHuan Xie proposed openstack/nova: [DO NOT MERGE] Use os-xenapi for nova  https://review.openstack.org/39310302:34
*** kevinz has quit IRC02:35
mriedemsmcginnis: welcome to nova: after dark02:35
*** kevinz has joined #openstack-nova02:35
*** tlian has joined #openstack-nova02:37
*** mlavalle has quit IRC02:37
openstackgerritMatt Riedemann proposed openstack/nova: Add 'disabled' to WatchdogAction field  https://review.openstack.org/38622102:38
*** yufei has joined #openstack-nova02:39
*** thorst_ has joined #openstack-nova02:39
*** ducttape_ has quit IRC02:40
*** thorst_ has quit IRC02:40
*** baoli has quit IRC02:41
smcginnis;)02:41
mikalOr: Nova, Sensible Timezone Edition02:43
smcginnismriedem: Since you're around - is status.openstack.org not showing anything, or is it a local issue for me?02:43
mriedemsmcginnis: works for me02:44
smcginnispoopy fart butt then.02:44
mriedem:)02:45
smcginnisDerp, helps if I actually go to the /zuul page.02:46
*** kevinz has quit IRC02:50
*** kevinz has joined #openstack-nova02:50
*** browne has quit IRC02:52
*** zz_dimtruck is now known as dimtruck02:53
*** ayoung has joined #openstack-nova03:02
*** giri has quit IRC03:07
*** shihanzhang has quit IRC03:08
*** ducttape_ has joined #openstack-nova03:08
*** zhenq has quit IRC03:08
*** yuntongjin has joined #openstack-nova03:09
*** gjayavelu has joined #openstack-nova03:09
*** gjayavelu has quit IRC03:14
*** Apoorva has quit IRC03:16
*** code-R has joined #openstack-nova03:20
*** kevinz has quit IRC03:20
*** code-R_ has joined #openstack-nova03:21
*** code-R has quit IRC03:25
*** code-R_ has quit IRC03:25
*** code-R has joined #openstack-nova03:25
*** links has joined #openstack-nova03:28
openstackgerritHuan Xie proposed openstack/nova: [DO NOT MERGE] Use os-xenapi for nova  https://review.openstack.org/39310303:39
*** Cristina_ has joined #openstack-nova03:42
*** tlbr has quit IRC03:44
*** sreeram_v has joined #openstack-nova03:45
*** CristinaPauna has quit IRC03:45
*** thorst_ has joined #openstack-nova03:45
*** tlbr has joined #openstack-nova03:45
*** ducttape_ has quit IRC03:46
*** yuntongjin has quit IRC03:47
*** sreeram_v has quit IRC03:47
*** zhangjl has joined #openstack-nova03:48
*** sreeram_v has joined #openstack-nova03:48
*** sreeram__ has joined #openstack-nova03:49
*** bks has joined #openstack-nova03:52
*** sreeram_v has quit IRC03:52
*** thorst_ has quit IRC03:52
*** zhangjl1 has joined #openstack-nova03:53
*** sreeram__ has quit IRC03:53
*** zhangjl has quit IRC03:55
*** nicolasbock has quit IRC03:57
*** sreeram_v has joined #openstack-nova03:57
*** sreeram__ has joined #openstack-nova03:57
*** psachin has joined #openstack-nova04:01
*** sreeram_v has quit IRC04:02
*** code-R_ has joined #openstack-nova04:04
*** code-R has quit IRC04:07
*** dave-mccowan has quit IRC04:14
*** mdnadeem has joined #openstack-nova04:15
*** hieulq_ has joined #openstack-nova04:17
*** stevemar__ has quit IRC04:19
*** stevemar__ has joined #openstack-nova04:19
*** hieulq_ has quit IRC04:21
*** stevemar__ has quit IRC04:24
*** takashin has quit IRC04:26
*** hongbin has joined #openstack-nova04:30
*** hongbin has quit IRC04:30
*** sudipto has joined #openstack-nova04:31
*** sudipto_ has joined #openstack-nova04:31
*** prateek has joined #openstack-nova04:31
*** baoli has joined #openstack-nova04:32
*** hoangcx has quit IRC04:35
*** yamahata has quit IRC04:35
*** hoangcx has joined #openstack-nova04:35
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port compute.test_extended_ip* to Python 3  https://review.openstack.org/40354804:39
*** Jeffrey4l has quit IRC04:46
*** dimtruck is now known as zz_dimtruck04:47
*** zz_dimtruck is now known as dimtruck04:47
*** unicell has joined #openstack-nova04:52
*** kaisers_ has quit IRC04:53
*** prateek has quit IRC04:53
*** sudipto_ has quit IRC04:54
*** sudipto has quit IRC04:54
*** prateek has joined #openstack-nova04:54
*** ayogi has joined #openstack-nova04:54
*** jgriffith is now known as jgriffith_away04:54
*** unicell1 has joined #openstack-nova04:56
*** unicell has quit IRC04:57
*** Jeffrey4l has joined #openstack-nova04:59
*** dimtruck is now known as zz_dimtruck05:01
openstackgerritMaho Koshiya proposed openstack/nova: Add metadata functional negative tests  https://review.openstack.org/39174405:01
*** ducttape_ has joined #openstack-nova05:10
*** thorst_ has joined #openstack-nova05:11
*** ducttape_ has quit IRC05:15
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Remove more tests from tests-py3.txt  https://review.openstack.org/40454305:17
openstackgerritHuan Xie proposed openstack/nova: [DO NOT MERGE] Use os-xenapi for nova  https://review.openstack.org/39310305:18
*** catinthe_ has quit IRC05:19
*** sudipto has joined #openstack-nova05:19
*** sudipto_ has joined #openstack-nova05:20
*** thorst_ has quit IRC05:20
*** catintheroof has joined #openstack-nova05:21
*** code-R_ has quit IRC05:26
*** catintheroof has quit IRC05:26
*** mirrorbo1 has quit IRC05:35
*** rha has quit IRC05:37
*** rha has joined #openstack-nova05:38
*** mirrorbo1 has joined #openstack-nova05:41
*** markmc has quit IRC05:42
*** markmc has joined #openstack-nova05:43
openstackgerritZhenyu Zheng proposed openstack/nova: Don't delete neutron port when attach failed  https://review.openstack.org/40356805:46
*** kaisers has joined #openstack-nova05:53
*** gcb has quit IRC05:54
*** kristian__ has joined #openstack-nova05:55
*** kristian__ has quit IRC05:59
*** kristian__ has joined #openstack-nova06:00
*** kaisers has quit IRC06:02
*** prometheanfire has joined #openstack-nova06:06
prometheanfirebooo06:06
prometheanfirenova-api: error: unrecognized arguments: --06:06
*** gouthamr has joined #openstack-nova06:06
prometheanfire-- is a standard construct for starting options06:06
*** jaosorior has joined #openstack-nova06:06
*** gcb has joined #openstack-nova06:07
*** code-R has joined #openstack-nova06:08
*** masuberu has quit IRC06:13
*** esberglu has joined #openstack-nova06:15
*** thorst_ has joined #openstack-nova06:18
*** esberglu has quit IRC06:19
*** gyx has joined #openstack-nova06:22
*** masuberu has joined #openstack-nova06:24
*** thorst_ has quit IRC06:25
*** yamahata has joined #openstack-nova06:26
*** code-R_ has joined #openstack-nova06:26
*** masber has joined #openstack-nova06:27
*** armax has quit IRC06:27
*** stevemar__ has joined #openstack-nova06:29
*** markus_z has joined #openstack-nova06:29
*** masuberu has quit IRC06:29
*** code-R has quit IRC06:30
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port compute unit tests to Python 3  https://review.openstack.org/40201006:30
openstackgerritYingxin Cheng proposed openstack/nova: Unexpected API error during server create  https://review.openstack.org/40456506:30
*** rha has quit IRC06:32
*** stevemar__ has quit IRC06:33
*** avolkov` has joined #openstack-nova06:34
*** avolkov has quit IRC06:35
*** ijw has joined #openstack-nova06:37
*** ijw has quit IRC06:41
*** sandanar has joined #openstack-nova06:42
*** code-R_ has quit IRC06:42
*** tuanluong has quit IRC06:42
*** psachin_ has joined #openstack-nova06:42
*** edand has joined #openstack-nova06:42
*** psachin has quit IRC06:45
*** kristian__ has joined #openstack-nova06:47
*** kristian__ has quit IRC06:47
*** kristia__ has joined #openstack-nova06:47
openstackgerritYingxin Cheng proposed openstack/nova: split host_state.least_disk_mb out of free_disk_mb  https://review.openstack.org/29030206:50
*** kristia__ has quit IRC06:50
openstackgerritzhaolihui proposed openstack/nova: Fix pci_alias that include white spaces  https://review.openstack.org/39207706:50
*** kristian__ has joined #openstack-nova06:51
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Check destination_type when booting with bdm provided  https://review.openstack.org/40236406:51
*** kristia__ has joined #openstack-nova06:53
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port virt.libvirt.test_imagebackend to Python 3  https://review.openstack.org/40343106:54
openstackgerrithuangtianhua proposed openstack/nova: Update hostname of server when updating its name  https://review.openstack.org/40458306:54
*** sridharg has joined #openstack-nova06:55
*** sreeram__ has quit IRC06:56
*** josecastroleon has joined #openstack-nova06:57
*** kristian__ has quit IRC06:57
openstackgerritZhenyu Zheng proposed openstack/nova: Don't delete neutron port when attach failed  https://review.openstack.org/40356806:58
*** kaisers has joined #openstack-nova06:58
*** zenoway has joined #openstack-nova07:00
*** zenoway has quit IRC07:00
*** zenoway has joined #openstack-nova07:00
*** oanson has joined #openstack-nova07:01
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port xenapi test_vm_utils to Python 3  https://review.openstack.org/40182707:02
*** mjura has joined #openstack-nova07:04
*** kaisers has quit IRC07:06
*** yuntongjin has joined #openstack-nova07:06
*** jaosorior has quit IRC07:08
*** jaosorior has joined #openstack-nova07:08
*** gyx has quit IRC07:09
*** esberglu has joined #openstack-nova07:09
*** gongysh has joined #openstack-nova07:10
openstackgerritYuYang proposed openstack/nova: VMware:fix get vm'root deivce conditional judgment  https://review.openstack.org/40225607:11
*** mjura has quit IRC07:11
*** ducttape_ has joined #openstack-nova07:11
*** mjura has joined #openstack-nova07:11
*** esberglu has quit IRC07:14
*** ducttape_ has quit IRC07:15
*** yuntongjin has quit IRC07:16
openstackgerritJianghua Wang proposed openstack/nova: XenAPI: fix memory usage miscalculation  https://review.openstack.org/39485707:19
*** irenab has joined #openstack-nova07:20
*** hoonetorg has quit IRC07:21
*** hoonetorg has joined #openstack-nova07:21
*** thorst_ has joined #openstack-nova07:24
*** hoonetorg has quit IRC07:24
*** hoonetorg has joined #openstack-nova07:24
*** sreeram_v has joined #openstack-nova07:26
*** moshele has joined #openstack-nova07:28
*** rcernin has joined #openstack-nova07:28
*** thorst_ has quit IRC07:29
*** Cristina_ has quit IRC07:30
*** mkoderer has joined #openstack-nova07:30
*** sreeram_v has quit IRC07:31
*** hieulq_ has joined #openstack-nova07:32
*** andreas_s has joined #openstack-nova07:36
*** gyx has joined #openstack-nova07:37
openstackgerritYingxin Cheng proposed openstack/nova-specs: The ResourceProviderTraits API - Manage traits with ResourceProviders  https://review.openstack.org/34513807:38
*** sridharg has quit IRC07:40
*** Cristina_ has joined #openstack-nova07:42
*** sreeram_v has joined #openstack-nova07:42
*** gyx has quit IRC07:43
*** gjayavelu has joined #openstack-nova07:45
*** ekuris has joined #openstack-nova07:45
openstackgerritHussain Chachuliya proposed openstack/nova: Refactor: Move post method to APIValidationTestCase base class  https://review.openstack.org/36468707:47
*** sreeram_v has quit IRC07:47
*** sudipto has quit IRC07:50
*** sudipto_ has quit IRC07:50
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: [WIP] Add version field to LibvirtLiveMigrateData  https://review.openstack.org/40460707:50
*** matrohon has joined #openstack-nova07:50
*** gcb has quit IRC07:51
*** gcb has joined #openstack-nova07:53
*** avolkov` has quit IRC07:54
*** gyx has joined #openstack-nova07:54
*** avolkov` has joined #openstack-nova07:55
*** gyx has quit IRC07:57
openstackgerritint32bit proposed openstack/nova: Choose VMware datastore by the least provisioned space  https://review.openstack.org/40263207:59
*** sridharg has joined #openstack-nova08:00
*** josecastroleon has quit IRC08:02
*** kaisers has joined #openstack-nova08:03
*** artom_ has joined #openstack-nova08:04
*** artom_ has quit IRC08:05
*** artom_ has joined #openstack-nova08:05
*** links has quit IRC08:06
*** artom has quit IRC08:07
*** josecastroleon has joined #openstack-nova08:09
*** aloga_ has joined #openstack-nova08:09
*** sreeram_v has joined #openstack-nova08:11
*** rha has joined #openstack-nova08:13
*** kaisers has quit IRC08:13
*** rmart04 has joined #openstack-nova08:13
*** aloga_ has quit IRC08:15
*** links has joined #openstack-nova08:18
*** ralonsoh has joined #openstack-nova08:19
*** Jeffrey4l has quit IRC08:22
*** Jeffrey4l has joined #openstack-nova08:23
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Remove more tests from tests-py3.txt  https://review.openstack.org/40454308:23
*** amoralej|off is now known as amoralej08:25
*** longlq_ has joined #openstack-nova08:26
*** thorst_ has joined #openstack-nova08:26
*** sam_nowitzki has quit IRC08:29
*** stevemar__ has joined #openstack-nova08:30
*** stevemar__ has quit IRC08:34
*** thorst_ has quit IRC08:34
*** sorrison_laptop has joined #openstack-nova08:36
*** sorrison_laptop has left #openstack-nova08:36
*** yuntongjin has joined #openstack-nova08:38
*** gongysh has quit IRC08:39
*** sudipto_ has joined #openstack-nova08:40
*** sudipto has joined #openstack-nova08:40
*** diogogmt has joined #openstack-nova08:49
*** fandi has joined #openstack-nova08:51
*** esberglu has joined #openstack-nova08:58
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-nova09:00
*** esberglu has quit IRC09:03
*** yuntongjin has quit IRC09:04
*** paul-carlton2 has joined #openstack-nova09:06
*** aloga_ has joined #openstack-nova09:08
*** yamahata has quit IRC09:09
*** kaisers has joined #openstack-nova09:10
*** lpetrut has joined #openstack-nova09:10
*** ducttape_ has joined #openstack-nova09:12
*** jaosorior is now known as jaosorior_lunch09:13
*** ducttape_ has quit IRC09:17
tdurakovbauzas: hi, could you review this one again https://review.openstack.org/#/c/398956/09:17
bauzastdurakov: with pleasure09:17
*** kaisers has quit IRC09:17
tdurakovbauzas: thank you09:17
openstackgerritTeng Fei proposed openstack/nova: Pass the parameter of extra image properties to create_snapshot_force()  https://review.openstack.org/40465609:21
openstackgerritZhenyu Zheng proposed openstack/nova: Don't delete neutron port when attach failed  https://review.openstack.org/40356809:21
*** hieulq_ has quit IRC09:21
*** zhangjl1 has quit IRC09:24
*** hieulq_ has joined #openstack-nova09:24
*** aloga_ has quit IRC09:24
*** hieulq_ has quit IRC09:26
*** hieulq_ has joined #openstack-nova09:28
*** baoli_ has joined #openstack-nova09:31
*** yassine has joined #openstack-nova09:32
*** thorst_ has joined #openstack-nova09:33
*** lucas-afk is now known as lucasagomes09:33
*** yassine is now known as Guest6200909:33
*** baoli has quit IRC09:33
*** gjayavelu has quit IRC09:35
johnthetubaguyBobBall: is there a meeting today?09:35
*** masuberu has joined #openstack-nova09:36
tdurakovbauzas: thank you09:37
*** jotr has quit IRC09:37
*** thorst_ has quit IRC09:39
*** masber has quit IRC09:39
*** jotr has joined #openstack-nova09:46
*** bkopilov_ has quit IRC09:49
*** yingjun has quit IRC09:50
*** yingjun has joined #openstack-nova09:51
*** esberglu has joined #openstack-nova09:53
openstackgerritPaul Carlton proposed openstack/nova: Live migration of instances with encrypted volumes  https://review.openstack.org/38960809:54
*** hfu has quit IRC09:54
*** zenoway has quit IRC09:54
*** lpetrut has quit IRC09:55
*** yingjun has quit IRC09:55
*** esberglu has quit IRC09:57
*** lpetrut has joined #openstack-nova10:03
*** aloga_ has joined #openstack-nova10:03
*** code-R has joined #openstack-nova10:04
*** links has quit IRC10:05
*** code-R_ has joined #openstack-nova10:07
*** code-R has quit IRC10:10
*** gjayavelu has joined #openstack-nova10:11
*** kaisers has joined #openstack-nova10:14
*** hoonetorg has quit IRC10:15
*** eglynn has joined #openstack-nova10:15
*** Cristina__ has joined #openstack-nova10:15
*** hieulq_ has quit IRC10:16
*** baoli has joined #openstack-nova10:16
*** gjayavelu has quit IRC10:16
*** eglynn has quit IRC10:16
*** brault has joined #openstack-nova10:17
*** andreaf has quit IRC10:17
*** sapcc-bot has quit IRC10:17
*** carthaca_1 has quit IRC10:17
*** sapcc-bot has joined #openstack-nova10:17
*** ihrachys has joined #openstack-nova10:17
*** david_1 has joined #openstack-nova10:17
*** tpatzig_ has joined #openstack-nova10:17
*** databus23_ has joined #openstack-nova10:17
*** carthaca_ has joined #openstack-nova10:17
*** mkoderer_ has joined #openstack-nova10:17
*** dgonzalez_ has joined #openstack-nova10:17
*** tommy_ has joined #openstack-nova10:17
*** kwathore_ has joined #openstack-nova10:18
*** tommy_ is now known as Guest2455610:18
*** andreaf has joined #openstack-nova10:18
*** pkholkin_ has joined #openstack-nova10:18
*** lyarwood_ has joined #openstack-nova10:18
*** mkoderer___ has joined #openstack-nova10:18
*** mkoderer__ has quit IRC10:19
*** logan- has quit IRC10:19
*** pkholkin has quit IRC10:19
*** pkholkin_ is now known as pkholkin10:19
*** lyarwood has quit IRC10:19
*** htruta has quit IRC10:19
*** Gorian has quit IRC10:19
*** anteaya has quit IRC10:19
*** baoli_ has quit IRC10:19
*** brault_ has quit IRC10:19
*** betherly has quit IRC10:19
*** rwmjones has quit IRC10:19
*** zz_dimtruck has quit IRC10:19
*** mrodden has quit IRC10:19
*** Cristina_ has quit IRC10:19
*** jaosorior_lunch has quit IRC10:19
*** ayoung has quit IRC10:19
*** bnemec has quit IRC10:19
*** mrsoul has quit IRC10:19
*** brad[] has quit IRC10:19
*** rdo has quit IRC10:19
*** briancurtin has quit IRC10:19
*** esmiurium has quit IRC10:19
*** DinaBelova has quit IRC10:19
*** JayF has quit IRC10:19
*** leifz has quit IRC10:19
*** darrenc has quit IRC10:19
*** jschwarz has quit IRC10:19
*** carthaca_ has quit IRC10:19
*** tpatzig_ has quit IRC10:19
*** Guest24556 has quit IRC10:19
*** dgonzalez_ has quit IRC10:19
*** mkoderer_ has quit IRC10:19
*** databus23_ has quit IRC10:19
*** david_1 has quit IRC10:19
*** kwathore has quit IRC10:19
*** mfisch has quit IRC10:19
*** bswartz has quit IRC10:19
*** dhellmann has quit IRC10:19
*** morgabra has quit IRC10:19
*** hshiina has joined #openstack-nova10:20
*** lpetrut has quit IRC10:20
*** darrenc has joined #openstack-nova10:20
*** logan- has joined #openstack-nova10:20
*** JayF has joined #openstack-nova10:20
*** carthaca_ has joined #openstack-nova10:20
*** bnemec has joined #openstack-nova10:21
*** DinaBelova has joined #openstack-nova10:21
*** kaisers has quit IRC10:21
*** inara` has joined #openstack-nova10:22
*** links has joined #openstack-nova10:22
*** tbachman_ has joined #openstack-nova10:22
*** alineb- has joined #openstack-nova10:22
*** mfisch has joined #openstack-nova10:23
*** mfisch has quit IRC10:23
*** mfisch has joined #openstack-nova10:23
*** alineb- is now known as alineb10:23
*** htruta has joined #openstack-nova10:23
*** dhellmann has joined #openstack-nova10:23
*** briancurtin has joined #openstack-nova10:24
*** hoonetorg has joined #openstack-nova10:24
*** mrsoul has joined #openstack-nova10:24
*** rwmjones has joined #openstack-nova10:24
*** Gorian has joined #openstack-nova10:24
*** mrodden has joined #openstack-nova10:24
*** brad[] has joined #openstack-nova10:24
*** _gryf_ has joined #openstack-nova10:25
*** leifz has joined #openstack-nova10:25
*** zz_dimtruck has joined #openstack-nova10:25
*** markmcclain has quit IRC10:25
*** zeroDivisible has quit IRC10:25
*** tbachman has quit IRC10:25
*** Daviey has quit IRC10:25
*** _gryf has quit IRC10:25
*** sgordon has quit IRC10:25
*** inara has quit IRC10:25
*** alex_xu has quit IRC10:25
*** boris-42 has quit IRC10:25
*** code-R_ has quit IRC10:25
*** tbachman_ is now known as tbachman10:25
*** rdo has joined #openstack-nova10:25
*** hoangcx has quit IRC10:25
*** zz_dimtruck is now known as dimtruck10:25
*** markmcclain has joined #openstack-nova10:26
*** code-R has joined #openstack-nova10:26
*** sgordon has joined #openstack-nova10:26
*** jschwarz has joined #openstack-nova10:26
*** alex_xu has joined #openstack-nova10:27
*** zeroDivisible has joined #openstack-nova10:28
*** boris-42 has joined #openstack-nova10:28
*** esmiurium has joined #openstack-nova10:28
*** morgabra has joined #openstack-nova10:28
*** morgabra has quit IRC10:28
*** morgabra has joined #openstack-nova10:28
*** slunkad has joined #openstack-nova10:29
*** anteaya has joined #openstack-nova10:31
*** jaosorior_lunch has joined #openstack-nova10:31
*** gjayavelu has joined #openstack-nova10:32
*** annp has quit IRC10:34
*** pcaruana has joined #openstack-nova10:35
*** derekh has joined #openstack-nova10:36
*** Daviey has joined #openstack-nova10:36
*** betherly has joined #openstack-nova10:36
*** betherly has quit IRC10:36
*** betherly has joined #openstack-nova10:36
*** gjayavelu has quit IRC10:36
*** thorst_ has joined #openstack-nova10:36
*** cdent has joined #openstack-nova10:37
*** psachin_ has quit IRC10:40
*** psachin has joined #openstack-nova10:40
*** jaosorior_lunch is now known as jaosorior10:41
*** longlq_ has quit IRC10:43
*** thorst_ has quit IRC10:45
*** psachin_ has joined #openstack-nova10:47
*** esberglu has joined #openstack-nova10:47
*** psachin has quit IRC10:48
*** yufei has quit IRC10:48
*** rfolco has joined #openstack-nova10:49
*** mvk has quit IRC10:50
*** kaisers has joined #openstack-nova10:51
*** esberglu has quit IRC10:52
*** gjayavelu has joined #openstack-nova10:52
*** lpetrut has joined #openstack-nova10:55
*** gjayavelu has quit IRC10:57
*** irenab has quit IRC10:58
*** kaisers has quit IRC10:59
*** sreeram_v has quit IRC11:00
openstackgerritAlex Szarka proposed openstack/nova: Transform instance.rebuild.error notification  https://review.openstack.org/39662111:00
*** lpetrut has quit IRC11:01
*** links has quit IRC11:07
*** links has joined #openstack-nova11:08
*** kaisers1 has quit IRC11:10
*** aloga_ has quit IRC11:10
openstackgerritHironori Shiina proposed openstack/nova: Ironic: Add soft reboot support to ironic driver  https://review.openstack.org/40374511:10
*** gabor_antal has quit IRC11:15
johnthetubaguycdent: can I ask about https://review.openstack.org/#/c/392891/2/nova/api/openstack/placement/deploy.py@5411:15
johnthetubaguycdent: I am not sure how that works, I am missing something I think11:16
cdenthi johnthetubaguy, looking11:16
cdentthere was a previous version which used conf._get_group('cors'), instead of catching the exceptions, but use of the private message was disliked (understandably), so the exception catching was done11:17
johnthetubaguycdent: so a better question, in production, how does register_opts get called11:17
johnthetubaguywho does, I mean11:17
cdentoh yeah, that is a different question, one sec, let me remind myself11:18
cdentjohnthetubaguy: the middleware itself, when you __init__ it11:20
*** lyarwood_ is now known as lyarwood11:20
cdentdoes that make a chicken egg problem?11:20
*** jamespage has joined #openstack-nova11:20
* cdent is constantly confused by oslo conf11:20
johnthetubaguycdent: hmm, possibly11:20
cdentone sec, I'll try a few things to see if I can figured it out (need to boot up my vm)11:21
johnthetubaguycdent: maybe we need a placement.enable_cors config?11:21
*** mvk has joined #openstack-nova11:21
cdentthat would be _horrible_11:21
jamespagemriedem, how would you feel about stable backporting the fixes for bug 1639239 to newton? I'd be happy to work through the commits but wanted to checkin on general agreement that's an OK thing first11:21
openstackbug 1639239 in python-os-brick (Ubuntu Yakkety) "ValueError for Invalid InitiatorConnector in s390" [Medium,Confirmed] https://launchpad.net/bugs/163923911:21
cdentadding config in order to be able to use config?11:21
johnthetubaguycdent: agreed, but its better than paste.ini11:21
*** sudipto has quit IRC11:23
*** sudipto_ has quit IRC11:23
johnthetubaguycdent: oh, that shouldn't be needed11:23
*** mvk has quit IRC11:23
* cdent is trying the tests without registering the opts11:23
*** mvk has joined #openstack-nova11:24
cdentjohnthetubaguy: ?11:24
*** kristia__ has quit IRC11:26
*** kaisers has joined #openstack-nova11:26
*** kristian__ has joined #openstack-nova11:26
*** sdague has joined #openstack-nova11:27
johnthetubaguycdent: I think its the conf.cors bit I don't get, does the comment I just added help?11:30
*** nherciu has joined #openstack-nova11:30
*** jlk has quit IRC11:31
cdentjohnthetubaguy: I've followed the link, and I know that stuff, but I'm not clear on what you're suggesting?11:31
*** jlk has joined #openstack-nova11:31
*** jlk has quit IRC11:31
*** jlk has joined #openstack-nova11:31
cdentthe docs are if you always want to run the middleware11:31
cdenti'm trying to make it so the middleware only runs if it has been configured11:32
johnthetubaguycdent: right, but what does "been configured" mean, there is there some enable/disable thing in the cors configuration already?11:32
johnthetubaguys/there is there/is there/11:32
cdentif there is a [cors] section in the conf file11:33
cdentthe middleware will then use that section, and validate it11:33
cdentif it is no good, the service won't start11:33
*** gjayavelu has joined #openstack-nova11:33
cdentif it not there, the middleware won't be used11:33
johnthetubaguycdent: OK, so by default it can't do anything, but in that case do we just add the middleware every time, and it just doesn't do anything?11:35
cdent"that case" is which case?11:35
johnthetubaguyerm, I mean when there is no config added in nova.conf11:35
*** irenab has joined #openstack-nova11:35
cdentwhat the code in deploy.py is doing is saying "if there is no [cors] in nova.conf, do not include the middleware in the application's stack"11:36
*** dave-mccowan has joined #openstack-nova11:37
*** josecastroleon has quit IRC11:37
johnthetubaguyright, I am asking, can't we just always added cors, and it just reads on the config to decide if it has to do anything11:37
johnthetubaguys/always added/always add/11:37
*** gjayavelu has quit IRC11:37
cdentthat's not the way the middleware works (as far as I can tell): if it gets passed a config that does not have an 'allowed_origin' value it raises a TypeError11:38
*** ftersin_ has joined #openstack-nova11:38
*** psachin_ has quit IRC11:39
johnthetubaguycdent: oh, sorry, I see, so you want to check the config to see if allowed_origin is set, ideally11:39
*** ijw has joined #openstack-nova11:39
johnthetubaguyyeah, I see that now: https://github.com/openstack/oslo.middleware/blob/36c35984a11667290e177e10c6fb67c6a50e1d9b/oslo_middleware/cors.py#L15211:39
johnthetubaguygrr....11:39
cdentyour third comment is effectively what the call to factory() is doing11:40
cdentall the oslo middlware's provide ways to use non global conf these days, which is a good thing, and the deploy.py is trying to stick to that within itself11:41
*** gszasz has joined #openstack-nova11:42
*** thorst_ has joined #openstack-nova11:43
*** tbachman has quit IRC11:43
*** psachin has joined #openstack-nova11:43
ftersin_mdbooth: hi. yesterday i missed your questions for me here by unknown reason. i removed -1 from https://review.openstack.org/#/c/337158 to do not block it11:45
mdboothftersin_: Thanks!11:45
pkholkinbauzas: hi Sylvain, are you here? I want to show you two patches related to scheduler11:46
ftersin_mdbooth: allthough i'm wondering what is a right way to handle such problems11:46
cdentjohnthetubaguy: I'm fairly certain you've identified a problem, though: without the options being registered, the code in deploy does not work correctly. I think.11:46
ftersin_mdbooth: i still think it requires a release note for the 1st problem, and a separate filed bug for the 2nd one11:47
*** bks has quit IRC11:47
mdboothftersin_: Yup, I'm intending to update that patch with a reno at the very least.11:47
*** thorst_ has quit IRC11:49
*** rtheis has joined #openstack-nova11:49
*** aloga_ has joined #openstack-nova11:50
*** hfu has joined #openstack-nova11:50
ftersin_mdbooth: ok. As for https://review.openstack.org/#/c/333244 and the next patch, i think it'd be better to improve them a bit11:50
*** ijw has quit IRC11:52
bhagyashriscdent: Hi, Can you guide me regarding the gabbi tests. I wrote gabbi test for the fix: https://review.openstack.org/#/c/398998/  and trying to test through the command  $ gabbi-run -x [localhost:[port]]/resource_providers < /[file_path]  but getting error: Failed to establish a new connection.  so where exactly i am going wrong.11:53
*** gjayavelu has joined #openstack-nova11:54
cdentbhagyashris: yeah, I can help with that, but I'll need to get back with you after I'm done with the thing I'm doing right now. For the gabbi tests that are in nova, you don't use gabbi-run to run them: they are a part of the existing functional tests. If you made changes to the existing yaml files, you can run them with `tox -efunctional placement`11:55
johnthetubaguycdent: yeah, the registering the opts, does cors.set_defaults() work by the way to register the options with oslo? I just found this: https://github.com/openstack/nova/blob/4f91ed3a547965ed96a22520edcfb783e7936e95/nova/common/config.py#L2311:55
*** kaisers_ has joined #openstack-nova11:56
cdentmy understanding is that is no longer needed11:56
cdent(that is, it sets reasonable defaults already)11:57
bhagyashriscdent:  ok. Thank you. sure will discuss latter once you will get free time.11:57
*** gjayavelu has quit IRC11:58
openstackgerritStephen Finucane proposed openstack/nova: conf: Add 'preallocate_images' to 'virt' group  https://review.openstack.org/39256712:00
openstackgerritStephen Finucane proposed openstack/nova: conf: Move virt options to a group  https://review.openstack.org/39256612:00
openstackgerritStephen Finucane proposed openstack/nova: conf: Detail the 'injected_network_template' opt  https://review.openstack.org/39256512:00
openstackgerritStephen Finucane proposed openstack/nova: conf: Standardize formatting of virt  https://review.openstack.org/39256412:00
johnthetubaguycdent: I was thinking that should register the options for us, indirectly12:01
openstackgerritAlex Szarka proposed openstack/nova: Transform instance.rebuild.error notification  https://review.openstack.org/39662112:03
*** ijw has joined #openstack-nova12:04
*** kaisers_ has quit IRC12:04
*** thorst_ has joined #openstack-nova12:07
mdboothftersin_: Thanks! I'm going to have to look into those in detail.12:08
*** lyarwood is now known as lyarwood_12:08
openstackgerritChris Dent proposed openstack/nova: Add rudimentary CORS support to placement API  https://review.openstack.org/39289112:09
*** ijw has quit IRC12:09
cdentjohnthetubaguy: the changes in that ^ may help to make sense of it. There's probably a bit more adjustments we can do but that actually allows it to work (in devstack) and shows the intent a bit more clearly (in the test fixture).12:10
*** kristian__ has quit IRC12:10
cdentbhagyashris: did `tox -efunctional placement` get you any further? I'm more free to answer questions now.12:11
*** kristian__ has joined #openstack-nova12:11
*** nicolasbock has joined #openstack-nova12:11
*** josecastroleon has joined #openstack-nova12:11
*** thorst_ has quit IRC12:11
*** gjayavelu has joined #openstack-nova12:14
*** ijw has joined #openstack-nova12:15
*** diogogmt has quit IRC12:15
*** kristian__ has quit IRC12:15
bhagyashriscdent: I have started and it's running.  waiting for the result.12:15
*** diogogmt has joined #openstack-nova12:15
*** kristian__ has joined #openstack-nova12:15
*** kristian__ has quit IRC12:16
*** kristian__ has joined #openstack-nova12:16
*** kristian__ has quit IRC12:17
*** slagle has joined #openstack-nova12:17
*** kristian__ has joined #openstack-nova12:17
*** gjayavelu has quit IRC12:19
*** aloga_ has quit IRC12:19
*** baoli_ has joined #openstack-nova12:20
*** baoli has quit IRC12:22
*** ijw has quit IRC12:25
*** prateek has quit IRC12:28
jaypipes*yawn*12:28
jaypipesmorning supernovas.12:28
pkholkinhi Jay12:29
*** catintheroof has joined #openstack-nova12:29
*** catinthe_ has joined #openstack-nova12:32
jaypipespkholkin: mornings are never good ;)12:33
*** diogogmt has quit IRC12:33
*** links has quit IRC12:34
*** catintheroof has quit IRC12:34
pkholkinjaypipes: that's true)12:34
*** bvanhav_ has joined #openstack-nova12:34
* cdent sings "oh what a beautiful morning oh what a beautiful day"12:37
cdentthis morning the frost glittered in the shade of the hedgerows while the sheep frolicked in the fields12:37
*** _gryf_ is now known as _gryf12:41
johnthetubaguycdent: it was a really beautiful frost this morning12:42
* johnthetubaguy goes for lunch12:43
*** claudiub has joined #openstack-nova12:46
alex_xunova api subteam meeting is in 14 mins at #openstack-meeting-412:47
jaypipescdent: what about the hedgehogs? were they frosty as well?12:48
cdentsleepin'12:48
jaypipes:)12:49
*** amoralej is now known as amoralej|lunch12:50
*** aloga_ has joined #openstack-nova12:52
*** hfu has quit IRC12:52
*** claudiub|2 has joined #openstack-nova12:53
*** thorst_ has joined #openstack-nova12:53
*** yufei has joined #openstack-nova12:54
*** lucasagomes is now known as lucas-hungry12:54
*** kylek3h has quit IRC12:54
*** claudiub has quit IRC12:54
*** ociuhandu has quit IRC12:55
*** yingjun has joined #openstack-nova12:56
*** gcb has quit IRC12:58
*** ociuhandu has joined #openstack-nova13:00
cdentjohnthetubaguy: you've opened up a very useful can of worms on this CORS review. it's identified a very weird problem with the 405 handling13:00
*** claudiub|2 is now known as claudiub13:01
*** kaisers_ has joined #openstack-nova13:01
*** tbachman has joined #openstack-nova13:03
*** lyarwood_ is now known as lyarwood13:03
*** jamesdenton has joined #openstack-nova13:04
lbeliveaujaypipes: quick question for you regarding RT, why do we have this check in drop_move_claim() https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L36313:04
lbeliveauI’m observing a different behavior on confirm resize, depending on if the periodic audit has executed or not13:04
*** ociuhandu has quit IRC13:07
jaypipeslbeliveau: on a call.. gimme a few :)13:07
lbeliveaujaypipes: sure, not urgent :)13:07
jaypipeslbeliveau: but cfriesen_ can answer that question :)13:07
*** ducttape_ has joined #openstack-nova13:07
*** lyarwood is now known as lyarwood_13:08
*** bkopilov has joined #openstack-nova13:08
*** kaisers_ has quit IRC13:09
*** jamesdenton has quit IRC13:09
lbeliveaujaypipes: some context for when you have some time ...13:10
lbeliveauIf the audit hasn’t been triggered and I do confirm, the resources won’t be dropped because the itype stored in the self.tracked_migrations correspond to new flavor13:10
lbeliveauBut if the audit got executed, it correspond to the old flavor, and resources gets dropped correctly13:10
_gryfjaypipes, cdent, hi. i'm finally get back from vacation, takes care about company things that was waitngo for me, and started looking at cdents last email http://lists.openstack.org/pipermail/openstack-dev/2016-November/107982.html13:11
_gryfis there any particular thing I can work on?13:12
*** tbachman has quit IRC13:12
*** tbachman has joined #openstack-nova13:12
*** catintheroof has joined #openstack-nova13:12
*** sahid has joined #openstack-nova13:12
_gryfi mean - around custom resource classes or nested resource providers?13:13
*** sahid has quit IRC13:13
*** sahid has joined #openstack-nova13:13
jaypipes_gryf: gimme a few, on phone13:13
_gryfjaypipes, k13:14
*** sridharg has quit IRC13:14
*** catinthe_ has quit IRC13:16
*** ljxiash has joined #openstack-nova13:17
*** gouthamr has quit IRC13:17
*** sudipto_ has joined #openstack-nova13:18
*** sudipto has joined #openstack-nova13:18
openstackgerritLee Yarwood proposed openstack/nova: Add a releasenote for bug#1633518  https://review.openstack.org/39621713:18
bhagyashriscdent:  is it ok to define the test data like 'name' paratmeter value as ('a' * 201) in nova.tests.functional.api.openstackplacemet.fixctures.py13:18
cdentbhagyashris: and set it in an environment variable? yes, if you plan to use it multiple times, that would be fine. otherwise it is also okay to just use strings in the yaml files13:20
*** hfu has joined #openstack-nova13:20
*** kristian__ has quit IRC13:20
bhagyashriscdent: ok. thank you.13:20
jaypipesbhagyashris: prefer one-off things like that to be just injected into the gabbit file.13:21
*** kristian__ has joined #openstack-nova13:21
jaypipesbhagyashris: meaning, don't put that into the environment context.13:22
jaypipesjust hard-code it into the individual gabbit test13:22
*** code-R_ has joined #openstack-nova13:22
*** kylek3h has joined #openstack-nova13:22
*** jheroux has joined #openstack-nova13:23
*** code-R has quit IRC13:24
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Ensure to_xml always returns an ASCII string  https://review.openstack.org/40238213:24
openstackgerritLee Yarwood proposed openstack/nova: libvirt: utils.get_disk return the target dev of the root device  https://review.openstack.org/40271613:24
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Use the mirror element to detect when to pivot  https://review.openstack.org/39973513:24
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Make guest.BlockDevice only use the target dev of a disk  https://review.openstack.org/40216313:24
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Wait for abort_job to complete when not resizing  https://review.openstack.org/38244913:24
*** sridharg has joined #openstack-nova13:25
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Move all extensions from contrib dir  https://review.openstack.org/28072613:26
*** karthiks has quit IRC13:29
*** xyang1 has joined #openstack-nova13:30
*** fandi has quit IRC13:31
bhagyashrisjaypipes: ok. can you please give me some example how to hard code it in gabbit test?13:31
jaypipesbhagyashris: just do:13:32
jaypipesdata:13:32
jaypipes    name: XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX13:32
jaypipesblah blah :)13:32
*** kaisers_ has joined #openstack-nova13:33
mriedemkashyap: can you jump in -qa re: "libguestfs installed but not usable (/usr/bin/supermin exited with error status 1."?13:34
kashyapmriedem: Just going to grap some quick lunch13:34
kashyapmriedem: Will join there13:34
* kashyap joined -qa; normally it's on auto-join13:35
*** links has joined #openstack-nova13:36
*** links has quit IRC13:36
*** edmondsw has joined #openstack-nova13:36
*** stevemar__ has joined #openstack-nova13:37
bhagyashrisjaypipes: ohh. ok. thanks.  :)13:37
*** ducttape_ has quit IRC13:37
*** kaisers_ has quit IRC13:39
*** sreeram_v has joined #openstack-nova13:39
*** kylek3h has quit IRC13:39
*** lamt has joined #openstack-nova13:41
*** jwcroppe has quit IRC13:41
*** sean-k-mooney has quit IRC13:43
*** oanson has quit IRC13:43
openstackgerritChris Dent proposed openstack/nova: Add rudimentary CORS support to placement API  https://review.openstack.org/39289113:44
*** karthiks has joined #openstack-nova13:45
cdentjohnthetubaguy: I think that ^ is much more correct now, thanks to you.13:45
cdentjaypipes: you might wanna look at that too as it is somewhat different from what you +2d earlier. The handling of config wasn't good enough in the earlier versions.13:46
cdentjaypipes: also I left a speculative comment on the start of the nested rps stack13:46
*** baoli has joined #openstack-nova13:48
*** kylek3h has joined #openstack-nova13:50
*** baoli_ has quit IRC13:50
*** lucas-hungry is now known as lucasagomes13:51
*** baoli_ has joined #openstack-nova13:54
*** baoli has quit IRC13:56
*** esberglu has joined #openstack-nova13:56
*** rodrigods is now known as rodrigods_13:58
alex_xujohnthetubaguy: so let me check more, and update the spec asap14:00
johnthetubaguyalex_xu: so its handled here: https://github.com/openstack/nova/blob/3cd2005523dc83b1cadc956c4f86385766aaf679/nova/compute/api.py#L246014:01
johnthetubaguyalex_xu: cool, thank you14:01
alex_xujohnthetubaguy: np14:01
*** karthiks has quit IRC14:02
*** esberglu has quit IRC14:03
amrithmriedem, dansmith ... trove's gate is now running on neutron. https://review.openstack.org/403855 and https://review.openstack.org/403853 are ok to merge. but they failed tests yesterday.14:03
*** esberglu has joined #openstack-nova14:03
alex_xujohnthetubaguy: that is bad, special for --all-tenant=114:03
johnthetubaguyalex_xu: what bit is bad?14:04
*** jwcroppe has joined #openstack-nova14:04
*** kaisers_ has joined #openstack-nova14:05
*** code-R has joined #openstack-nova14:05
alex_xujohnthetubaguy: it dump all the instances from the db without pagination14:05
mriedemamrith: hmm, gate-grenade-dsvm-ubuntu-xenial shouldn't be running on master14:05
alex_xujohnthetubaguy: https://github.com/openstack/nova/blob/3cd2005523dc83b1cadc956c4f86385766aaf679/nova/compute/api.py#L237314:06
amrithmriedem, sorry if I broke it but from the sounds of it, that isn't something I did :( breaking the trove CI, I do that everyday, thrice on Sundays. but not nova ...14:06
mriedemno there is probably a regression in project-config14:07
*** tongli has joined #openstack-nova14:07
*** esberglu has quit IRC14:07
*** code-R_ has quit IRC14:08
johnthetubaguyalex_xu: true, at least its admin only for the worst of it14:08
johnthetubaguyalex_xu: we should probably file a bug for that14:08
johnthetubaguyalex_xu: I should just page through extra results if none are found, or something14:08
johnthetubaguys/I/It/14:08
*** kaisers_ has quit IRC14:09
mriedemamrith: specifically this change https://review.openstack.org/#/c/398517/14:09
mriedemwas meant to restrict gate-grenade-dsvm-ubuntu-xenial to mitaka and newton14:09
alex_xujohnthetubaguy: sorry, I didn't get what is mean 'page through extra result'14:10
amrithyup14:10
amrithi see it14:10
*** nherciu has quit IRC14:10
johnthetubaguyalex_xu: sorry, I was meaning we could keep the limit param, and when we scan through code, if we run out of servers, we go get more from the DB until we run out, or we find our answer14:10
*** nherciu has joined #openstack-nova14:11
* mriedem git blames layout.yaml14:11
alex_xujohnthetubaguy: ah, i got it, that is better14:11
lyarwood_mriedem: thats failing on stable/newton btw I assume someone just made gate-grenade-dsvm-neutron-multinode-live-migration-nv a xenial job? ./me checks buffer14:11
amrithmriedem, in any event I'll try my best to be at your nova meeting tomorrow as dansmith suggested. but 9am is commute time and I may be either there at the very beginning, or at the very end. my update (if any) will only be that we're off nova networking so if it is ok, I'll skip it.14:11
*** amoralej|lunch is now known as amoralej14:11
mriedemamrith: skip it14:11
alex_xujohnthetubaguy: i will file a bug for it14:11
amriththx mriedem will take you up on that.14:12
mriedemlyarwood_: yeah someone changed the filters14:12
amriththx muchly14:12
mriedemlyarwood_: btw, why did you rebase this twice? https://review.openstack.org/#/c/404508/14:12
*** ducttape_ has joined #openstack-nova14:12
lyarwood_mriedem: gertty-- thought it was a new change the first time and stripped the cherry-picked line14:12
lyarwood_mriedem: the second was just an edit adding that back in14:12
mriedemok14:13
lyarwood_mriedem: sorry about that, I'll wait until it syncs fully next time or just use the webui14:13
mriedemi've never used gertty so no idea how that happens14:13
lyarwood_mriedem: it just syncs all reviews into a db locally, I just didn't wait until it had finished this morning before looking for things to cherry-pick14:14
*** hfu has quit IRC14:14
*** esberglu has joined #openstack-nova14:16
*** bvanhav_ has quit IRC14:16
*** mdrabe has joined #openstack-nova14:17
*** eharney has joined #openstack-nova14:17
*** ducttape_ has quit IRC14:17
kashyapAnd the local DB crashes regularly (causing you to reconstruct it from scratch - which is trivial, though), unfortunately, preventing many from adopting it.  But lately it's a bit more stable...14:18
*** karthiks has joined #openstack-nova14:19
*** bswartz has joined #openstack-nova14:20
*** jamesdenton has joined #openstack-nova14:20
*** donnalezyty has joined #openstack-nova14:22
*** shuquan has joined #openstack-nova14:24
*** pradk has quit IRC14:24
*** yingjun has quit IRC14:24
*** yingjun has joined #openstack-nova14:24
*** mriedem has quit IRC14:25
dansmithamrith: yeah, I wanted status _until_ it's fixed, so if it's fixed, no need14:26
*** ayoung has joined #openstack-nova14:29
amrithdansmith, g'morning. thx, yes now fixed'ish14:29
amrithnot out of woods yet14:29
dansmithamrith: well, we can re-land our thing right? that's what I mean14:30
*** victorhe has joined #openstack-nova14:30
*** victorhe has left #openstack-nova14:31
*** rodrigods_ is now known as rodrigods14:31
amrithdansmith, yes please. I've pushed up those changes. mriedem found an issue with layout.yaml and once that is fixed they can land.14:32
amriththe problems we have are now cluster/network related14:32
amrithand we've disabled those tests fornow.14:32
amrithsorry "disabled those tests for now (TM)"14:32
*** avolkov` has quit IRC14:33
*** READ10 has joined #openstack-nova14:33
*** gcb has joined #openstack-nova14:33
*** donnalezyty_ has joined #openstack-nova14:34
*** donnalezyty has quit IRC14:34
*** sudipto_ has quit IRC14:35
*** sudipto has quit IRC14:35
dansmithheh14:35
*** avolkov` has joined #openstack-nova14:35
*** bvanhav_ has joined #openstack-nova14:35
*** kaisers_ has joined #openstack-nova14:38
*** ducttape_ has joined #openstack-nova14:39
*** irenab has quit IRC14:40
dansmithalaski: you around by chance?14:41
*** mdnadeem has quit IRC14:42
johnthetubaguyjaypipes: do you think we need to skip calling save if there are no updates? its probably overkill I guess?: https://review.openstack.org/#/c/400448/314:43
*** mtanino has joined #openstack-nova14:43
*** kaisers_ has quit IRC14:44
*** tblakes_ has joined #openstack-nova14:45
*** Swami has joined #openstack-nova14:46
*** aloga_ has quit IRC14:47
*** sreeram_v has quit IRC14:47
*** tblakes_ is now known as tblakes14:49
*** krtaylor has joined #openstack-nova14:51
*** bvanhav_ is now known as bvanhav14:52
*** tbachman has quit IRC14:52
*** kristian__ has quit IRC14:52
*** kristian__ has joined #openstack-nova14:52
*** tbachman has joined #openstack-nova14:53
*** kristia__ has joined #openstack-nova14:54
*** unicell1 has quit IRC14:55
*** unicell has joined #openstack-nova14:55
*** burt has joined #openstack-nova14:56
*** mriedem has joined #openstack-nova14:56
*** kristian__ has quit IRC14:57
*** jaosorior has quit IRC14:57
*** tbachman has quit IRC14:57
*** kristia__ has quit IRC14:58
*** betherly has quit IRC14:59
*** jbernard has joined #openstack-nova14:59
*** prateek has joined #openstack-nova14:59
*** jaosorior has joined #openstack-nova14:59
*** anush has joined #openstack-nova15:00
*** tbachman has joined #openstack-nova15:03
*** eharney has quit IRC15:04
jaypipesjohnthetubaguy: it doesn't save if there are no updates, since updates dict would be empty, right?15:05
*** nherciu has quit IRC15:05
*** nherciu has joined #openstack-nova15:06
dansmithjaypipes: looks like id could be changed and attempted to save15:06
dansmithbut yeah, otherwise there are no other fields to dirty up an updates dict15:06
dansmithoh no wait, I have that backwards15:07
dansmithonly name15:07
*** tlian has quit IRC15:07
dansmithso, yeah, it won't save unless name is touched, but having confused two of us in five minutes probably means that logic could be cleaner15:07
jaypipesdansmith: id is a readonly field though.15:07
dansmithright, and the logic here prevents it from being saved, only name15:08
dansmithbut it's super confusing15:08
jaypipesdansmith: I mean, I'm happy to change things, but don't want to do it just for ResourceClass when pretty much no other objects do this AFAICT15:08
dansmithno other objects do what?15:09
dansmiththis patch has logic to ensure that only name is mutable, but it's very confusing logic15:09
jaypipesdansmith: do a manual check of whether updates dict is empty and raise some exception that nothing has changed.15:09
jaypipesdansmith: what johnthetubaguy was asking for above.15:10
dansmithjaypipes: this does do that15:10
jaypipesunless I've misunderstood what johnthetubaguy was asking for?15:10
dansmithit just does it super confusingly15:10
*** anush has quit IRC15:10
*** prateek has quit IRC15:10
jaypipesdansmith: it's copy-paste code from elsewhere in that file, unfortunately.15:10
jaypipesdansmith: one sec, lemme grab a link.15:10
pkholkinwho is good in scheduler? I want to discuss one issue15:11
dansmithwell, it's super confusing in that place too :)15:11
jaypipesdansmith: no disagreement. bad on me for copy/pasting.15:11
mriedempkholkin: edleafe-, bauzas, johnthetubaguy15:11
jaypipespkholkin: I can help too.15:11
johnthetubaguyjaypipes: I think instance does a skip for that one15:11
edleafe-pkholkin: go ahead15:11
dansmithjaypipes: that said, most objects (at least early ones I wrote) just quietly skip the db call if nothing has changed, which is probably what johnthetubaguy is really getting at15:12
dansmithjaypipes: this does that by virtue of checking for the only other field being changed, which is not allowable15:12
johnthetubaguydansmith: oh, I miss read that I guess15:12
dansmithjohnthetubaguy: it's confusing15:12
dansmithjohnthetubaguy: if we added another field to this, it would stop doing the thing you want15:13
pkholkinedleafe-: johnthetubaguy: jaypipes: I want to share two patches: 1) test fix (incorrect mock) https://review.openstack.org/#/c/402910/ 2) more interesting, I think aggregate_image_properties_isolation works incorrect and it is a bug  https://review.openstack.org/#/c/403640/15:13
dansmithjohnthetubaguy: regardless, it'd be nice to clean it up so it's less confusing15:13
*** oanson has joined #openstack-nova15:13
pkholkinI provided this fix to look at, it changes the behaviour of current filter, but I think old behaviour is incorrect15:14
pkholkinplease look at the commit message and tests there15:14
pkholkinwhat's your opinion?15:14
mriedemhere it comes15:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_shelve work with CONF.use_neutron=True by default  https://review.openstack.org/39736515:15
openstackgerritMatt Riedemann proposed openstack/nova: Make nova.tests.unit.virt pass with CONF.use_neutron=True by default  https://review.openstack.org/39694515:15
openstackgerritMatt Riedemann proposed openstack/nova: Make test_serversV21 tests work with neutron by default  https://review.openstack.org/39682415:15
openstackgerritMatt Riedemann proposed openstack/nova: Make test_compute_mgr work with CONF.use_neutron=True by default  https://review.openstack.org/39696515:15
openstackgerritMatt Riedemann proposed openstack/nova: Make libvirt unit tests work with CONF.use_neutron=True by default  https://review.openstack.org/39690415:15
openstackgerritMatt Riedemann proposed openstack/nova: Make xenapi tests work with CONF.use_neutron=True by default  https://review.openstack.org/39690515:15
openstackgerritMatt Riedemann proposed openstack/nova: Make vmware unit tests work with CONF.use_neutron=True  https://review.openstack.org/39690015:15
openstackgerritMatt Riedemann proposed openstack/nova: neutron: handle no_allocate in create_pci_requests_for_sriov_ports  https://review.openstack.org/39635915:15
openstackgerritMatt Riedemann proposed openstack/nova: Restrict test_compute_cells to nova-network  https://review.openstack.org/39733615:15
openstackgerritMatt Riedemann proposed openstack/nova: Explicitly use nova-network in nova-network network tests  https://review.openstack.org/39686415:15
openstackgerritMatt Riedemann proposed openstack/nova: Make test_compute pass with CONF.use_neutron=True by default  https://review.openstack.org/39997815:15
openstackgerritMatt Riedemann proposed openstack/nova: Make test_compute_api work with CONF.use_neutron=True by default  https://review.openstack.org/39695715:15
jaypipesdansmith: please point me at the example object you want me to use. I'm having trouble finding one that doesn't look like the ones in resource_provider.py15:16
openstackgerritLeehom Li proposed openstack/nova: Support detach interface with same MAC from instance  https://review.openstack.org/37224315:16
*** flip214 has joined #openstack-nova15:16
*** flip214 has joined #openstack-nova15:16
jaypipesdansmith: for instance... https://github.com/openstack/nova/blob/master/nova/objects/block_device.py#L18415:16
edleafe-pkholkin: I see your point on the defaultdict have sets as values15:17
jaypipesdansmith: is virtually the same as what we're doing there.15:17
*** tlian has joined #openstack-nova15:17
*** sudipto has joined #openstack-nova15:17
*** sudipto_ has joined #openstack-nova15:17
dansmithjaypipes: but totally not confusing :)15:17
*** flip214 has left #openstack-nova15:17
*** edleafe- is now known as edleafe15:17
*** armax has joined #openstack-nova15:17
*** sandanar has quit IRC15:18
*** stevemar__ has quit IRC15:18
jaypipesdansmith: what am I missing here... how is this confusing? https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L34015:18
wznoinskmriedem, highlighted the problem with timeouts, thanks, we've increased them last week but it looks like needs extending again, flavor is 14-35-1615:19
*** stevemar__ has joined #openstack-nova15:19
dansmithjaypipes: just nevermind, I should never have butted in15:19
jaypipesdansmith: no, I want to address this, I'm not being picky or anything. honestly trying to understand what changes to make.15:19
jaypipesdansmith: are you and johnthetubaguy asking to change the above to just pop 'uuid' off updates and ignore any changes to that "immutable" field?15:20
jaypipesis that what you're asking?15:20
jaypipesis it the raising of the ObjectActionError that is confusing?15:20
jaypipesor something else?15:20
johnthetubaguyjaypipes: I was thinking if you call save(), it would be nice if it was a noop if the name wasn't changed15:21
*** tbachman has quit IRC15:21
johnthetubaguyjaypipes: but thats more like a polish thing than a requirement thing, really15:22
jaypipesjohnthetubaguy: not sure why you have to bring Poland into this.15:22
jaypipeswhat did they do?15:22
jaypipes:P15:22
bauzassorry folks, had a private emergency issue15:22
pkholkinedleafe: yes, first patch is simple, I am more interested in the second15:22
johnthetubaguyjaypipes: heh15:22
jaypipesjohnthetubaguy: AFAIU, this would essentially be a no-op: https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L43415:22
bauzasmriedem: what do you want me to look ?15:22
jaypipesjohnthetubaguy: if nothing was updated on the model.15:22
mriedembauzas: pkholkin had some questions above15:23
jaypipesdansmith: does that match your understanding of SQLAlchemy model saving?15:23
pkholkinbauzas: hey, I need your opinion on two patches15:23
pkholkinyes15:23
johnthetubaguyjaypipes: yeah, when updates is empty15:23
bauzaspkholkin: sure, lemme know15:23
*** anush has joined #openstack-nova15:23
openstackgerritDanil Akhmetov proposed openstack/nova: Use proper user and tenant in the owner section of libvirt.xml.  https://review.openstack.org/39967915:23
johnthetubaguyjaypipes: yeah, its possible SQLAlchemy might just no op it anyways15:23
edleafepkholkin: looking at the second patch now15:23
*** ayogi has quit IRC15:23
pkholkinbauzas: the first patch related to tests https://review.openstack.org/#/c/402910/ (this is simple I think), the second more important https://review.openstack.org/#/c/403640/, I think this filter works incorrect and it is a bug15:24
pkholkinI made this patch to have smth to look at15:24
*** edand has quit IRC15:25
dansmithjaypipes: no idea, and I'm not talking about the no-op part15:26
bauzaspkholkin: the first one seems okay to me15:26
dansmithjaypipes: I'm talking about the "if updates and updates.keys() != ['name']" thing which is actually just trying to say "if 'id' in updates", but in a very confusing way15:27
bauzaspkholkin: about the second, I remember some discussion I had with some other people15:27
*** pradk has joined #openstack-nova15:27
*** eharney has joined #openstack-nova15:28
bauzaspkholkin: looking at https://review.openstack.org/#/c/403640/2/nova/scheduler/filters/aggregate_image_properties_isolation.py15:28
*** tbachman has joined #openstack-nova15:29
*** sridharg has quit IRC15:30
*** liverpooler has joined #openstack-nova15:30
*** yingjun has quit IRC15:30
pkholkinbauzas: ok, so what about the second? I understand that I change the behaviour of the filter, but I think it is incorrect. I found it working on spec that I wanted to be merged (about strict isolation)15:30
*** yingjun has joined #openstack-nova15:30
bauzaspkholkin: that's the problem I discussed previously with others15:30
jaypipesdansmith: got it. that is clear. will update shortly.15:30
*** ayoung has quit IRC15:30
*** kristian__ has joined #openstack-nova15:30
*** annegentle has joined #openstack-nova15:31
bauzaspkholkin: if we check by image properties instead of aggregate metadata keys, I want to make sure we're not modifying the behaviour15:31
bauzaspkholkin: because if not, that's terrible for operators15:31
pkholkinbauzas: unfortunately I changed it and I think this is correct15:32
pkholkinunderstand that it is not so good15:32
pkholkinfor existing clouds15:32
bauzaspkholkin: the real problem is that changing how a filter works means really a new situation15:33
pkholkinbut now docs and implementation differs(15:33
*** kristian__ has quit IRC15:34
*** kristian__ has joined #openstack-nova15:34
pkholkinbauzas: what we can do with it?15:34
bauzaspkholkin: honestly, while I can review your first change quickly, I need more time to think about the latter15:34
*** gouthamr has joined #openstack-nova15:35
*** yingjun has quit IRC15:35
bauzaspkholkin: what I need to consider is what could happen if we are not testing all aggregate keys like we did15:35
pkholkinbauzas: I think we should discuss it with other folks15:35
bauzasif we're sure that we're testing each of the possible image/agg keys, then okay15:35
pkholkinmy understanding is that this filter should works as AggregateInstanceExtraSpecsFilter15:36
bauzaspkholkin: probably, but I need to think again about this filter15:36
pkholkinbut now they differs15:36
openstackgerritStephen Finucane proposed openstack/nova: conf: Add 'preallocate_images' to 'virt' group  https://review.openstack.org/39256715:36
openstackgerritStephen Finucane proposed openstack/nova: conf: Move virt options to a group  https://review.openstack.org/39256615:36
bauzaspkholkin: well, that's not necessary15:36
pkholkinedleafe: jaypipes: please participate in this, please ^15:36
pkholkinbauzas: yes, but ideally they should imho15:37
bauzaspkholkin: I mean, your opinion that the both filters should work the same way is possibly understood, but that's not what's here now15:37
bauzaspkholkin: yeah, agreed, but changing an existing filter behaviour is not something we should change smoothly15:37
pkholkinbauzas: agree with it, but15:37
pkholkinbauzas: these docs http://docs.openstack.org/mitaka/config-reference/compute/scheduler.html and http://docs.openstack.org/developer/nova/filter_scheduler.html said that we should iterate over query I think15:38
pkholkinso, I think we have a mistake (15:38
bauzaspkholkin: to make it clear, my main concern is that if we modify this filter, I want to have it a backwards compatibilty15:38
pkholkinbauzas: that's true, do you have ideas how we can do this?15:39
*** ekuris has quit IRC15:40
*** gcb has quit IRC15:40
*** kaisers_ has joined #openstack-nova15:41
*** darrenc has quit IRC15:41
*** darrenc has joined #openstack-nova15:42
edleafepkholkin: it certainly does seem to change the existing behavior of the filter.15:42
*** hongbin has joined #openstack-nova15:42
pkholkinyes15:42
edleafepkholkin: my question is: are there bug reports about this? Are others complaining about its behavior?15:43
pkholkinedleafe: not sure, I found it during working on https://review.openstack.org/#/c/381912/15:43
edleafepkholkin: I don't know enough about the use of this filter in the field to have an opinion, other than don't break things15:43
pkholkinI supposed that those filters should work in the same way15:44
*** moshele has quit IRC15:45
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate all nova-net related opts  https://review.openstack.org/39568315:48
*** hieulq_ has joined #openstack-nova15:48
*** yufei has quit IRC15:48
*** jdurgin1 has joined #openstack-nova15:48
bauzaspkholkin: like I said, I need time to think of that15:50
bauzasbut first, I need to code back :)15:50
*** TravT has quit IRC15:50
*** mlavalle has joined #openstack-nova15:53
*** shuquan has quit IRC15:58
*** jaosorior has quit IRC15:58
*** Sukhdev_ has joined #openstack-nova15:58
*** psachin has quit IRC15:59
*** hshiina has quit IRC15:59
openstackgerritJay Pipes proposed openstack/nova: placement: REST API for resource classes  https://review.openstack.org/39195915:59
openstackgerritJay Pipes proposed openstack/nova: placement: adds ResourceClass.save()  https://review.openstack.org/40044815:59
openstackgerritJay Pipes proposed openstack/nova: placement: adds ResourceClass.destroy()  https://review.openstack.org/39191815:59
sfinucansahid: Could I get you to take a look at this when you have a chance. Having trouble understanding how libvirt "cap(abilitie)s" get translated into something the filters can see/parse https://review.openstack.org/#/c/39965316:00
sfinucanvladikr - likewise ^16:00
*** _ducttape_ has joined #openstack-nova16:00
vladikrsfinucan, sure16:01
sfinucanvladikr: Cheers :)16:01
*** thorst_ has quit IRC16:01
*** edmondsw_ has joined #openstack-nova16:02
*** rcernin has quit IRC16:02
*** gouthamr has quit IRC16:02
*** code-R has quit IRC16:02
*** edmondsw has quit IRC16:02
*** edmondsw_ has quit IRC16:02
*** edmondsw has joined #openstack-nova16:03
*** ducttape_ has quit IRC16:03
cdentjaypipes: another thing worth mentioning, should we do this at some point: https://review.openstack.org/#/c/404279/16:04
cdent(get rid of remotable in rp stuff)16:04
*** andreas_s has quit IRC16:04
*** Sukhdev_ has quit IRC16:09
*** tbachman has quit IRC16:09
*** jgriffith_away is now known as jgriffith16:09
jaypipescdent: ya.16:10
jaypipesdansmith, johnthetubaguy: addressed your concerns in above ResourceClass patches.16:10
edleafecdent: Yeah, I can't see this going over conductor at any point16:11
cdentDo we need to time the change with anything, or can we just do it?16:11
*** kaisers_ has quit IRC16:11
* cdent wants to be able to read * comprehensions as well as edleafe 16:12
bauzascdent: wait, why removing the possibility to have those objects remotable ?16:13
cdentedleafe: "unhelpful unit tests can be" s/unit tests/mocks/16:13
bauzascdent: because you definitely want to cut the line ?16:13
*** jdillaman has quit IRC16:13
cdentbauzas: because we don't _want_ them to be remotable, we want the data only accessed over the api16:13
*** ihrachys has quit IRC16:14
cdentand if we aren't going to use remotability, then having it in there is useless expense16:14
bauzascdent: well, if we don't have the indirection API equals to None, why it's a problem ?16:15
bauzasby default, objects are not indirecting to the conductor16:15
cdentyou still have to pass through the decorator code16:15
bauzasand what if sometimes later we could want to have those methods be remotable ?16:15
cdentdoes it make a big difference, no not at all, but if it is not required, may as well not be there16:15
bauzaslike if we were having another engine as a service ?16:16
cdentthen we could put it back when we actually need it16:16
*** mjura has quit IRC16:16
*** Bofu2U has joined #openstack-nova16:16
pkholkinedleafe: thanks for review16:16
cdentbut ideally the resources in the http api are the main interface16:16
bauzasI'm not sure I like removing that design pattern, because that means that those methods *could* be remotable16:16
*** mjura has joined #openstack-nova16:16
cdentbauzas: I'm not sure I'm following you, can you elaborate?16:17
bauzascdent: having those methods marked as remotable means they *could* be remotable16:17
cdentyes, that's what we want to prevent16:18
bauzaswhy should it be a problem to have those remotable ?16:18
pkholkinmriedem: do we have some features to help about API/DB areas? maybe smth else16:18
cdentbauzas: why should it be a problem to have them not remotable? If we make them remotable but don't need them to be, then we have a classic case of YAGNI16:19
pkholkinI think I will not write email about that my spec, so I can help to implement smth16:19
*** danpawlik has quit IRC16:21
cfriesen_so I've been poking around the "suspend" code, and it seems to be pretty fragile...there are no checks for whether there's enough space, and no resources get updated either on suspend or resume16:22
cfriesen_this seems odd when it could easily take many GB for the managed-save file.16:22
*** _ducttape_ has quit IRC16:23
*** ducttape_ has joined #openstack-nova16:24
sahidsfinucan: i will do it, thanks for those tests16:24
*** ducttape_ has quit IRC16:25
*** ducttape_ has joined #openstack-nova16:25
pkholkincfriesen_: hello Chris, I'm sorry, are you an operator?16:25
pkholkindo you see this my spec? https://review.openstack.org/#/c/381912/16:25
*** yamahata has joined #openstack-nova16:26
cfriesen_pkholkin: nope, I'm a dev16:26
jaypipesjohnthetubaguy: hopefully I did the whole oslo.concurrency.lockutils like you wanted.16:26
*** jdillaman has joined #openstack-nova16:27
*** code-R has joined #openstack-nova16:27
pkholkincfriesen_: ok, sorry)16:27
pkholkinsaw your some changes in scheduler, you are welcome to the spec if you are interested in)16:28
*** paw has quit IRC16:28
mriedempkholkin: api/db areas? i don't understand the question.16:29
*** jamesdenton has quit IRC16:29
*** ducttape_ has quit IRC16:29
*** ducttape_ has joined #openstack-nova16:30
pkholkinmriedem: these are areas that I like) just looking for some thing to implement16:30
pkholkinwill look approved blueprint list16:31
mriedempkholkin: yeah on a call, just a min16:31
*** hieulq_ has quit IRC16:31
mriedempkholkin: this hasn't started yet https://blueprints.launchpad.net/nova/+spec/deprecate-image-meta-proxy-api16:32
mriedemalex_xu: might be cool with you taking that16:32
*** jamesdenton has joined #openstack-nova16:32
cardeoisHello there ! If somebody has time to review this CORS fix that would really help me if it gets merged ! https://review.openstack.org/#/c/400785/16:33
pkholkinnp, will look16:34
*** code-R has quit IRC16:35
*** gjayavelu has joined #openstack-nova16:35
*** hieulq_ has joined #openstack-nova16:38
*** gjayavelu has quit IRC16:40
*** tbachman has joined #openstack-nova16:44
*** tbachman_ has joined #openstack-nova16:46
*** ljxiash has quit IRC16:47
*** ljxiash has joined #openstack-nova16:47
cardeoisthanks pkholkin !16:48
*** tbachman has quit IRC16:49
*** tbachman_ is now known as tbachman16:49
*** jamesdenton has quit IRC16:49
*** hieulq_ has quit IRC16:52
*** ljxiash has quit IRC16:52
*** mdrabe has quit IRC16:52
*** hieulq_ has joined #openstack-nova16:55
*** Zer0Byte__ has joined #openstack-nova16:55
*** thorst_ has joined #openstack-nova16:56
cdentedleafe, jaypipes is this still active? https://review.openstack.org/#/c/382000/16:57
*** rama_y has joined #openstack-nova16:59
jaypipescdent: no.17:00
jaypipescdent: shouldn't be.17:00
jaypipescdent: edleafe I think can abandon that one. he was listed on a similar patch that merged called "genericize on resource provider" (or somehting like that)17:00
*** dimtruck is now known as zz_dimtruck17:01
*** mdrabe has joined #openstack-nova17:01
*** mjura has quit IRC17:03
cdent✔17:03
edleafecdent: yeah, I think it's redundant.17:05
cdentkill it with fire!17:05
cdentscatter the ashes17:05
cdentetc17:05
edleafecdent: well, I was going to double-check first, but if you're in a rush...17:06
edleafecdent: :)17:06
cdentheh17:06
*** code-R has joined #openstack-nova17:06
*** matrohon has quit IRC17:07
openstackgerritSujitha proposed openstack/nova: Improve help text for network  options.  https://review.openstack.org/34120317:07
*** kaisers_ has joined #openstack-nova17:08
edleafecdent: done17:10
cdentYou are a credit to the universe. The less-noise league salutes you.17:10
*** zz_dimtruck is now known as dimtruck17:10
*** catintheroof has quit IRC17:11
*** slaweq_ has joined #openstack-nova17:11
*** catintheroof has joined #openstack-nova17:12
*** paw has joined #openstack-nova17:13
*** nic has joined #openstack-nova17:13
kashyap[OT] Hey folks, since we use QEMU, maybe I thought it's appropriate to 'spam' this announcement here:17:15
*** Kevin_Zheng has quit IRC17:15
kashyapTomorrow starts QEMU Advent Calendar -- https://lists.gnu.org/archive/html/qemu-devel/2016-09/msg03990.html17:15
*** kfarr has joined #openstack-nova17:15
*** kaisers_ has quit IRC17:15
kashyapIf you have any fun ideas for a disk image, feel free to contact on the email address mentioned there :-)17:16
kashyap</EOM>17:16
*** Kevin_Zheng has joined #openstack-nova17:17
*** code-R has quit IRC17:17
*** jgriffith is now known as jgriffith_away17:19
*** hieulq_ has quit IRC17:21
*** pcaruana has quit IRC17:23
*** slaweq_ has quit IRC17:23
*** slaweq_ has joined #openstack-nova17:24
*** code-R has joined #openstack-nova17:25
openstackgerritSylvain Bauza proposed openstack/nova: Extend get_all_by_filters to support resource criteria  https://review.openstack.org/38624217:25
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Expose a REST API method for a specific list of RPs  https://review.openstack.org/39256917:25
*** rmart04 has quit IRC17:29
*** tongli has quit IRC17:30
*** rodrigods is now known as rodrigods_17:31
*** woodster_ has joined #openstack-nova17:33
*** penick has joined #openstack-nova17:36
*** slaweq_ has quit IRC17:36
*** mvk has quit IRC17:40
*** kristian__ has quit IRC17:43
*** kristian__ has joined #openstack-nova17:44
pkholkinmriedem: question about https://blueprints.launchpad.net/nova/+spec/deprecate-image-meta-proxy-api, do we want to add new microversion and deprecate 'image-metadata' in it?17:45
openstackgerritJay Pipes proposed openstack/nova: placement: REST API for resource classes  https://review.openstack.org/39195917:45
openstackgerritJay Pipes proposed openstack/nova: placement: adds ResourceClass.save()  https://review.openstack.org/40044817:45
openstackgerritJay Pipes proposed openstack/nova: placement: adds ResourceClass.destroy()  https://review.openstack.org/39191817:45
*** edand has joined #openstack-nova17:46
mriedempkholkin: did you read the spec?17:46
mriedemhttps://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/deprecate-image-meta-proxy-api.html17:47
pkholkinvery very fast)17:47
*** derekh has quit IRC17:47
mriedempkholkin: so basically it's the same pattern as 2.36 http://docs.openstack.org/developer/nova/api_microversion_history.html#id3317:48
*** lucasagomes is now known as lucas-afk17:48
openstackgerritChris Dent proposed openstack/nova: Add rudimentary CORS support to placement API  https://review.openstack.org/39289117:48
*** kristian__ has quit IRC17:48
pkholkinyes, I looked it17:48
mriedempkholkin: but for this extension https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/image_metadata.py17:48
*** nherciu has quit IRC17:48
pkholkinlooks not so complicated17:48
mriedemso after a new microversion, all of the control methods in there will 40417:49
mriedemnope17:49
mriedempretty straight forward17:49
mriedemand then the similar changes in novaclient17:49
mriedemas we did for 2.3617:49
pkholkinI think I will work on this17:49
mriedemamrith: https://review.openstack.org/#/c/404895/ should get us going17:49
mriedempkholkin: cool, thanks17:49
mriedempkholkin: i really appreciate you stepping up to help out with an already approved but not yet started bp17:50
*** unicell has quit IRC17:50
*** dimtruck is now known as zz_dimtruck17:51
pkholkinmriedem: no problem, this often happens) I will look on 2.36 changes and will prepare some wip patch to look at17:51
*** yamahata has quit IRC17:51
amriththx mriedem17:52
*** Apoorva has joined #openstack-nova17:52
pkholkinalmost everything seems clear to me17:52
*** matrohon has joined #openstack-nova17:52
*** mriedem1 has joined #openstack-nova17:54
*** mriedem has quit IRC17:56
mriedem1jroll: have you seen this? https://wiki.openstack.org/wiki/Nimble17:59
*** mriedem1 is now known as mriedem17:59
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Move all extensions from contrib dir  https://review.openstack.org/28072618:00
*** jgriffith_away is now known as jgriffith18:00
*** markus_z has quit IRC18:02
*** ralonsoh has quit IRC18:04
*** penick has quit IRC18:05
*** gjayavelu has joined #openstack-nova18:06
*** sudipto_ has quit IRC18:06
*** sudipto has quit IRC18:06
*** gjayavelu has quit IRC18:07
paul-carlton2mriedem, can you take a look at https://review.openstack.org/#/q/status:open+topic:bp/detach-boot-volume please18:07
*** zz_dimtruck is now known as dimtruck18:10
*** penick has joined #openstack-nova18:11
*** mbirru has joined #openstack-nova18:11
*** sahid has quit IRC18:12
*** kaisers_ has joined #openstack-nova18:12
*** ihrachys has joined #openstack-nova18:15
*** ayoung has joined #openstack-nova18:16
*** kaisers_ has quit IRC18:19
*** ftersin_ has quit IRC18:20
*** edand has quit IRC18:20
*** dave-mcc_ has joined #openstack-nova18:23
*** unicell has joined #openstack-nova18:24
*** dave-mccowan has quit IRC18:26
*** mriedem has quit IRC18:27
*** kylek3h has quit IRC18:28
*** lyarwood_ is now known as lyarwood18:30
*** mriedem has joined #openstack-nova18:30
*** ihrachys has quit IRC18:31
*** tjones has joined #openstack-nova18:33
*** tjones has left #openstack-nova18:33
*** colby_ has joined #openstack-nova18:33
colby_Hey we just upgrade to Mitaka. We are having issues with the nova api for some of our projects. We have projects that the ids are the same as the name. This was to not break legacy swift swuath users. Does the new version of the api not work if the tenant/project id is not uuid?18:34
*** yamahata has joined #openstack-nova18:34
mriedemcolby_: see http://docs.openstack.org/releasenotes/nova/mitaka.html18:35
mriedemIn order to make project_id optional in urls, we must constrain the set  of allowed values for project_id in our urls. This defaults to a regex  of [0-9a-f\-]+, which will match hex uuids (with / without dashes), and integers. This covers all known project_id formats in the wild. If your site uses other values for project_id, you can set a site specific validation with project_id_regex config variable.18:35
*** paul-carlton2 has quit IRC18:35
*** ljxiash has joined #openstack-nova18:35
colby_should that regex not match an id like "cinergi"?18:35
*** Swami has quit IRC18:36
*** oanson has quit IRC18:36
mriedemregexpal.com is telling me it doesn't18:36
*** Swami has joined #openstack-nova18:37
mriedemoh a-f18:37
mriedemfor hex uuids18:37
mriedemso no that won't work18:37
mriedemyou'll need to set the project_id_regex config option18:37
colby_oh duh a-f not a-z18:37
colby_sorry for the confusion18:37
mriedemnp, i was confused too :)18:38
colby_I saw notes that the config was going to be deprecated. Will this be a breaking point in the future?18:38
mriedemlooks like yes,18:39
mriedemi imagine that option was added as an immediately deprecated backdoor option for people upgrading to mitaka, such as yourself18:39
mriedemsdague made those changes so he might remember better18:39
*** ljxiash has quit IRC18:39
mriedemcolby_: but as noted, we're working on making project_ids optional in the urls18:40
mriedemso not sure why you'd need the project name encoded in the project id18:40
mriedemexcept maybe convenience18:40
*** kylek3h has joined #openstack-nova18:41
colby_well horizon seems to do it along with the command line client. Thats how we found it it didn't work.18:42
colby_so hopefully they drop that in the future release too :)18:43
* robcresswell reading scrollback18:44
robcresswellWhat's Horizon need to change? Not entirely sure I follow18:44
colby_when you view instances it makes a call to /v2/cinergi/serers and gets a 404 becuase it puts the project id in the url18:45
*** dharinic is now known as dharinic|lunch18:47
robcresswellAh18:47
*** amoralej is now known as amoralej|off18:47
robcresswellcolby_: Could you open a bug against Horizon with more info?18:47
*** Bofu2U has quit IRC18:47
mriedemhmm, that's a v2 route, the project_id being optional is in v2.18 http://docs.openstack.org/developer/nova/api_microversion_history.html#id1618:48
jrollmriedem: choosing to ignore nimble as much as possible, honestly18:48
colby_the command line client does the same thing too18:48
*** rodrigods_ is now known as rodrigods18:49
mriedemcolby_: this was the change btw https://review.openstack.org/#/c/233076/18:49
mriedem2.18 doesn't help you if you're on v2 still18:50
mriedemthis is unrelated fun - tempest failed this patch during a shelve/unshelve test where in the process of unshelving, libvirt's lifecycle event callback stuff stops the server http://logs.openstack.org/08/404508/3/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/f2ef8c2/logs/screen-n-cpu.txt.gz#_2016-11-30_09_51_58_92118:52
mriedemso the unshelved server never goes active18:52
mriedemif the task_state is set though we shouldn't process that request18:52
*** jdurgin1 has quit IRC18:55
*** penick has quit IRC18:57
*** penick has joined #openstack-nova19:01
*** stewie925 has joined #openstack-nova19:03
mriedemfigured it out, nice....19:04
mriedemnic: cburgess: so do you guys support shelve?19:05
mriedema customer would probably never hit this19:05
nicIn theory, we do, but nobody seems to know what it is  ;)19:06
mriedemok,19:06
mriedemso there is a fun little race bug in the ceph job because of the fast snapshot stuff you guys added19:06
nicGo on…19:06
mriedemso snapshot taken during shelve offload is so fast, that between the time that shelve stops the instance, and we unshelve the instance, it falls within this 15 second window for https://github.com/openstack/nova/blob/14.0.0/nova/virt/libvirt/host.py#L10419:06
mriedemwhich calls back to the compute manager to stop the instance19:07
mriedemeven though it was just unshelved and is now active19:07
mriedemand hilarity ensues19:07
nicSo we should disable shelve/unshelve on Ceph deployments.  Got it.19:08
nic;)19:08
mriedemwel...19:08
mriedemwell,19:08
mriedemonly if your customer is tempest19:08
mriedemand you have fast rbd snapshots19:08
stewie925hello guys, after successfully doing a resize_confirm,  how do we change the server state back to 'active'?19:10
stewie925I assumed it was revertResize, but I think that's only for unsuccessful resize19:11
mriedemstewie925: it should go back automatically unless it fails19:11
mriedemrevert resize will also make it go back to active19:11
mriedemif it started out as active19:11
*** Swami has quit IRC19:11
mriedemyou can resize a stopped server too19:11
mriedemconfirm/revert are for kicking the tires while the vm is resized19:11
mriedemif there was some issue with it after the resize, you can revert the resize to go back19:12
stewie925oh I see, I am getting some weird errors, let me share the link to my log....19:12
mriedemjbernard: you asked for it https://bugs.launchpad.net/nova/+bug/164621219:12
openstackLaunchpad bug 1646212 in OpenStack Compute (nova) "unshelved offloaded instance is unexpectedly stopped" [Low,Triaged]19:12
*** mnestratov has quit IRC19:13
*** unicell has quit IRC19:14
jbernardmriedem: so what you're saying is that we're /too/ good19:14
mriedem:)19:15
jbernardim okay with that ;)19:15
mriedemi'm not too worried about this bug unless it has a high failure rate19:15
mriedemchecking logstash now19:16
jbernardmriedem: who normally works on nova rbd issues?19:16
jbernardmriedem: should i put this in my queue?19:16
mriedemit's not really an rbd issue per se, so no19:16
jbernardtrue19:16
mriedemjust tickled by the ceph job19:16
*** kaisers_ has joined #openstack-nova19:16
stewie925here is the result of my local testing (attempt to change server to 'active' status after successful confirmResize, but it failed):  http://paste.openstack.org/show/591004/19:17
*** unicell has joined #openstack-nova19:17
mriedemstewie925: does the api client in the functional tests not handle a 202?19:17
mriedemi think yo'uve hit this type of issue before19:17
stewie925ohhh19:17
*** gszasz has quit IRC19:18
stewie925yes I remember that one, i'll check, thanks for the guidance19:18
nicmriedem: are you thinking this merits a giant refactor of the workflow, or would papering over the problem with an adjustable timeout suffice?19:18
mriedemnic: no config options for sure19:18
stewie925but I remember the last time, my scenario showed the status code (204), this time no status code is shown?19:18
nic(e.g., if your backend is RBD, set the lifecycle timeout to, say, 5 seonds)19:19
mriedemnic: but then you could fail with the reboot thing19:19
mriedemas noted in that code that sets the delay to 15 sec19:19
nic*sigh*19:19
nicI hate computers.19:20
mriedemyou and me both19:20
mriedemstewie925: yeah i guess it's a 204 https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L80819:20
mriedemstewie925: that's not the issue though19:20
mriedemyour test is busted19:20
mriedemstewie925: it fails here: self.api.post_server_action(server['id'], {'revertResize': None})19:21
stewie925yeah - it failed at revertResize19:21
stewie925yes19:21
mriedemat that point you've already confirmed the resize on L52219:21
mriedemyou can't revert a resize of an active server19:21
mriedemas there is no resize pending19:21
stewie925ohhh19:21
mriedemso..you got a 40019:21
*** unicell has quit IRC19:22
stewie925should I do a verifyResize then? but i dont think that would be a valid request19:22
mriedemstewie925: no, L539 and below should be deleted19:23
stewie925oh just leave it out19:23
mriedemyes19:23
mriedemthe test is just, resize a server, confirm the resize, check notifications, fin.19:23
mriedem_test_resize_revert_server would be a separate test19:23
*** unicell has joined #openstack-nova19:24
*** Sukhdev has joined #openstack-nova19:24
stewie925mriedem: thank you so much, so after confirmResize, the server status should automatically set to active19:24
*** kaisers_ has quit IRC19:25
mriedemyes19:25
mriedemyou have to wait for active, but yes19:25
stewie925thank you so much, again :)19:26
*** dharinic|lunch is now known as dharinic19:28
jaypipescores looking for a bug patch to review... I've gone through this one from leehom li a few times and I think it's probably good enough at this point and fixes a relatively important (IMHO) bug: https://review.openstack.org/#/c/372243/19:29
mriedemwish danpb was back on that one, i see he was +2 one point19:31
mriedemi've had it starred so i need to get back on it19:31
mriedemi'd like sahid to look at that too19:32
openstackgerritSTEW TY proposed openstack/nova: Transform instance.resize_confirm notification  https://review.openstack.org/39065419:33
*** mkoderer has quit IRC19:33
*** betherly has joined #openstack-nova19:36
*** fandi has joined #openstack-nova19:38
openstackgerritMaciej Szankin proposed openstack/nova: conf: fix formatting in base  https://review.openstack.org/40493919:38
*** lunarlamp has joined #openstack-nova19:40
*** claudiub has quit IRC19:43
*** jeblair is now known as inserted_human19:45
*** inserted_human is now known as jeblair19:45
*** rajathagasthya has joined #openstack-nova19:46
mriedemjaypipes: found a problem19:47
mriedemso -119:47
mriedemthe number of random unrelated cleanups makes it hard to focus on that guy19:48
*** kylek3h_ has joined #openstack-nova19:51
jaypipesmriedem: yes, did you see my comment about that?19:53
mriedemyes, that wasn't my -119:53
*** kylek3h has quit IRC19:54
jaypipesmriedem: yeah, nice catch on the _LW thing.19:54
mriedemcopy/paste ftw19:54
*** catintheroof has quit IRC19:54
mriedemi also don't see why LibvirtConfigGuestDeviceAddressDrive needed to change19:55
jaypipesmriedem: the format_dom() thing is being tested in the test that verifies the assertXML() crap.19:55
mriedemi don't see that class used in here19:55
jaypipesmriedem: I think due to adding of the format_dom() change which actually is unrelated to the patch and I had said that in a previous patch. :(19:55
jaypipesle sigh19:55
mriedemyeah LibvirtConfigGuestDeviceAddressDrive.format_dom() isn't covered at all in tests19:56
mriedemjust ran coverage locally19:56
mriedemso it's not needed at all19:56
mriedemjaypipes: but heh, you asked for reviews :)19:57
mriedemthese are pretty simple things to fix19:57
mriedemoh i think i see what you're saying, the format_doms() aren't needed at all19:58
mriedemfor the actual bug fix19:58
mriedemhttps://review.openstack.org/#/c/372243/24/nova/virt/libvirt/guest.py@246 is the actual fix19:58
*** rajathagasthya has quit IRC20:00
jaypipesmriedem: right, I think that format-doms() stuff was in a separate patch at one point.. :(20:02
mriedemdansmith: https://review.openstack.org/#/c/403855/ is passing jobs again20:03
mriedemjust finishing up the placement job now20:03
jaypipesmriedem: honestly, I've kind of gotten worn out with this but it fixes the issue and don't see any issue fixing certain things in followup patches at this point.20:03
jaypipesmriedem: sorry20:03
mriedemjaypipes: well, LOG.debug(_LW()) isn't cool20:04
dansmithmriedem: kthx20:04
jaypipesmriedem: yes, I didn't catch that.20:04
jaypipesmriedem: I'm referring to the format_dom() and style concerns.20:04
mriedemdansmith: don't forget https://review.openstack.org/#/c/403853/1 under it20:04
mriedemamrith: i hope you're ready20:05
dansmithmriedem: is that one failed for another reason?20:05
mriedemyeah20:05
mriedemif it weren't the top patch wouldn't pass either20:06
mriedemtls timeout ftw http://logs.openstack.org/53/403853/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/45a4413/logs/screen-n-api.txt.gz?level=TRACE#_2016-11-28_19_36_12_75020:06
dansmithmriedem: um, this removes the warning, but should be putting back the exit right?20:06
mriedemthe exit is on top20:06
mriedemi know it's a bit weird20:06
mriedemthey should have probably been done in a single change20:07
dansmithyeah, okay20:07
mriedemi assume amrith reverted in order to avoid conflict20:07
*** gouthamr has joined #openstack-nova20:09
*** ijw has joined #openstack-nova20:12
*** lpetrut has joined #openstack-nova20:14
lbeliveaujaypipes: the bug/inconsistency in RT I told you about this morning, does it worth it to fix in your opinion ?20:15
*** ihrachys has joined #openstack-nova20:15
*** gjayavelu has joined #openstack-nova20:15
openstackgerritDan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method  https://review.openstack.org/36755720:17
openstackgerritDan Smith proposed openstack/nova: Make RPCFixture support multiple connections  https://review.openstack.org/39641720:17
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937920:17
openstackgerritDan Smith proposed openstack/nova: Add a CellDatabases test fixture  https://review.openstack.org/39971020:17
openstackgerritDan Smith proposed openstack/nova: Reset indirection_api for each test  https://review.openstack.org/40495020:17
*** paw has quit IRC20:17
*** ijw has quit IRC20:17
*** smatzek has joined #openstack-nova20:17
*** rfolco has quit IRC20:19
jaypipeslbeliveau: sorry can you repeat the point you made?20:19
*** lyarwood is now known as lyarwood_20:19
*** matrohon has quit IRC20:19
*** lyarwood_ is now known as lyarwood20:19
lbeliveaujaypipes: I’m observing a different behavior on confirm resize, depending on if the periodic audit has executed or not20:20
lbeliveauIf the audit hasn’t been triggered and I do confirm, the resources won’t be dropped because the itype stored in the self.tracked_migrations correspond to new flavor20:20
lbeliveauBut if the audit got executed, it correspond to the old flavor, and resources gets dropped correctly20:20
*** ijw has joined #openstack-nova20:20
*** kaisers_ has joined #openstack-nova20:22
*** Guest62009 has quit IRC20:22
openstackgerritSean Dague proposed openstack/nova: guestfs: Don't report exception if there's read access to kernel  https://review.openstack.org/40136620:25
*** matrohon has joined #openstack-nova20:25
sdaguemriedem / kashyap ^^^20:28
sdaguethat at least gets us to a more sane error message, then we can paper over the ubuntu issue with devstack. Though, I wonder if there is an upstream ubuntu bug that should be filed for it20:29
*** jamesdenton has joined #openstack-nova20:29
*** kaisers_ has quit IRC20:29
amrithmriedem, I did it as two changes so that I could get two ATC passes :) no, I wanted to git revert and if I did it as one it seemed to generate a conflict.20:30
mriedemsdague: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/75972520:30
openstackLaunchpad bug 759725 in linux (Ubuntu) "The kernel is no longer readable by non-root users" [Medium,Won't fix]20:30
*** ijw has quit IRC20:30
sdaguemriedem: ok, well so it is20:30
sdaguemriedem: well the new error message at least tells you what you need to do there20:31
mriedemjaypipes: i'm going to cleanup that change20:31
*** browne has joined #openstack-nova20:31
jaypipesmriedem: k, ty20:32
jaypipeslbeliveau: sec20:32
mriedemand remove the config format_dom stuff in the process20:32
*** ijw has joined #openstack-nova20:32
jaypipeslbeliveau: is this something you see in mitaka or newton vs. master? in other words, is this something you have noticed recently pop up?20:33
*** Zer0Byte__ has quit IRC20:34
lbeliveaujaypipes: on master, I ran into it when trying to fix something else20:34
lbeliveauwhen the claim is created, this is executed: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L75520:35
lbeliveaubut on audit, the tracked migrations are cleared and re-created, and this is executed: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L76420:35
lbeliveau(on resize on the same host)20:36
*** rcernin has joined #openstack-nova20:37
*** kaisers_ has joined #openstack-nova20:38
*** Zer0Byte__ has joined #openstack-nova20:39
openstackgerritSujitha proposed openstack/nova: Allow finding instance with invalid regex name  https://review.openstack.org/39230520:40
*** jamesdenton has quit IRC20:40
*** rodrigods is now known as rodrigods_20:41
*** READ10 has quit IRC20:41
jaypipeslbeliveau: hmm. seems indeed that is a bug. any chance you can push a test that demonstrates that issue?20:44
*** kristian__ has joined #openstack-nova20:44
*** kristian__ has quit IRC20:45
*** kristian__ has joined #openstack-nova20:45
*** ljxiash has joined #openstack-nova20:45
*** dtp has joined #openstack-nova20:45
lbeliveaujaypipes: I'll work on it, hopefully I'll get something tomorrow, bug is not trivial to unit test, took me a while to understand what was really happening20:45
lbeliveauI'll ping you when I have something20:46
lbeliveau(will need your help on how to fix it, not sure on some of the RT details)20:46
openstackgerritDiana Clarke proposed openstack/python-novaclient: Microversion 2.39 - Simple tenant usage pagination  https://review.openstack.org/39465320:47
openstackgerritMatt Riedemann proposed openstack/nova: Support detach interface with same MAC from instance  https://review.openstack.org/37224320:48
*** Swami has joined #openstack-nova20:48
mriedemjaypipes: ^ dropped all of the unrelated style stuff and config changes20:48
jaypipeslbeliveau: ty sir. looking forward to helping on your patch.20:49
dansmithPSA: cellsv2 meeting in ten minutes20:50
kristian__Hi, does someone have some experience with gaming on nvidia gpus on openstack?20:50
jaypipesmriedem: reviewed and approved. thanks again.20:50
*** ljxiash has quit IRC20:51
*** mnestratov has joined #openstack-nova20:51
openstackgerritSean Dague proposed openstack/nova: guestfs: Don't report exception if there's read access to kernel  https://review.openstack.org/40136620:53
sdaguemriedem: fixed the nit20:53
mriedemthanks20:54
*** catintheroof has joined #openstack-nova20:54
mriedemsdague: also https://review.openstack.org/#/c/336927/320:54
*** liverpooler has quit IRC20:54
mriedemthat was previously approved but was stuck on the depends-on20:54
mriedemwhich i've since dropped20:54
*** WillUK2010 has joined #openstack-nova20:58
*** annegentle has quit IRC21:00
*** annegentle has joined #openstack-nova21:01
openstackgerritLudovic Beliveau proposed openstack/nova: Release PCI devices on drop_move_claim()  https://review.openstack.org/37037421:02
*** bvanhav has quit IRC21:02
openstackgerritLudovic Beliveau proposed openstack/nova: Release PCI devices on drop_move_claim()  https://review.openstack.org/37037421:03
sdaguemriedem: +A21:04
*** larainema has quit IRC21:05
*** Apoorva has quit IRC21:05
mriedemthanks21:05
*** Zer0Byte__ has quit IRC21:05
*** Apoorva has joined #openstack-nova21:05
openstackgerritPushkar Umaranikar proposed openstack/nova: [WIP] Add service_token for nova-cinder interaction  https://review.openstack.org/39739921:06
openstackgerritSujitha proposed openstack/nova: Add post hook script to run tempest with old n-api  https://review.openstack.org/40139521:06
*** anush has quit IRC21:06
*** Zer0Byte__ has joined #openstack-nova21:07
openstackgerritDiana Clarke proposed openstack/python-novaclient: Microversion 2.39 - Simple tenant usage pagination  https://review.openstack.org/39465321:07
*** dimtruck is now known as zz_dimtruck21:09
*** rtheis has quit IRC21:10
*** ijw has quit IRC21:14
*** zz_dimtruck is now known as dimtruck21:14
*** ihrachys has quit IRC21:16
*** amoralej|off is now known as amoralej21:21
*** claudiub has joined #openstack-nova21:22
openstackgerritSujitha proposed openstack/nova: Add post hook script to run tempest with old n-api  https://review.openstack.org/40139521:24
*** Zer0Byte__ has quit IRC21:24
*** mvk has joined #openstack-nova21:25
*** Zer0Byte__ has joined #openstack-nova21:25
*** whenry has quit IRC21:27
*** owalsh has quit IRC21:32
*** owalsh_ has joined #openstack-nova21:33
*** smatzek has quit IRC21:34
*** Jeffrey4l_ has joined #openstack-nova21:34
*** Jeffrey4l has quit IRC21:35
*** owalsh__ has joined #openstack-nova21:37
*** ijw has joined #openstack-nova21:39
kashyapsdague: Thanks.  Obviously, please add yourself as the Co-Author to that21:39
*** owalsh__ is now known as owalsh21:39
* kashyap hits the hay21:39
*** owalsh_ has quit IRC21:39
openstackgerritSujitha proposed openstack/nova: Change os-server-tags default policy  https://review.openstack.org/39642021:40
*** Zer0Byte__ has quit IRC21:43
*** Zer0Byte__ has joined #openstack-nova21:48
*** gouthamr has quit IRC21:49
*** edmondsw has quit IRC21:50
*** annegentle has quit IRC21:51
*** matrohon has quit IRC21:51
*** annegentle has joined #openstack-nova21:52
*** kaisers_ has quit IRC21:52
*** kaisers_ has joined #openstack-nova21:54
*** hongbin has quit IRC22:01
*** Zer0Byte__ has quit IRC22:01
*** lpetrut has quit IRC22:01
*** ijw has quit IRC22:03
*** tbachman has quit IRC22:04
*** ijw has joined #openstack-nova22:08
*** ijw has quit IRC22:08
*** ijw has joined #openstack-nova22:08
*** WillUK2010 has quit IRC22:09
*** claudiub|2 has joined #openstack-nova22:09
*** dimtruck is now known as zz_dimtruck22:11
*** slaweq_ has joined #openstack-nova22:11
openstackgerritMatt Riedemann proposed openstack/nova-specs: Amend proposal to count resources to check quota in API for cells  https://review.openstack.org/39975022:11
*** ijw has quit IRC22:12
*** ijw has joined #openstack-nova22:12
*** penick has quit IRC22:12
*** claudiub has quit IRC22:13
*** ijw has quit IRC22:15
*** ijw has joined #openstack-nova22:16
*** ijw has quit IRC22:16
*** ijw has joined #openstack-nova22:16
openstackgerritJay Pipes proposed openstack/nova: placement: RT now adds proper Ironic inventory  https://review.openstack.org/40447222:17
openstackgerritJay Pipes proposed openstack/nova: placement: report client handle InventoryInUse  https://review.openstack.org/40499022:17
*** kylek3h_ has quit IRC22:17
openstackgerritChris Dent proposed openstack/nova: Some improvement to the process doc  https://review.openstack.org/40499122:19
cdentmriedem: ^ that's sort of a followup to the "when is a blueprint needed" stuff done a while ago22:19
*** tbachman has joined #openstack-nova22:20
mriedemcdent: cool, i know that most of that was all old and like kilo or liberty specific,22:20
mriedemit was literally a brain dump from a wiki put into docs22:20
cdentyeah, I addressed some but not all of that22:20
cdentthere's _loads_ more that could be done, but it'll take a few goes22:21
*** amoralej is now known as amoralej|off22:22
mriedemi thought about updating it once but it's exhausting22:23
cdentyeah, I think I need to go to bed22:24
melwittmriedem: should I mark this as superseded? https://blueprints.launchpad.net/nova/+spec/cells-quota-api-db I can't remember if the partial data move to API DB is okay to go under the counting resources bp22:25
openstackgerritAndrew Bogott proposed openstack/nova: Add disk_minimum_physical_free_space_ratio option to disk filter.  https://review.openstack.org/34119822:25
*** larainema has joined #openstack-nova22:25
*** ijw has quit IRC22:30
*** tbachman has quit IRC22:31
mriedemmelwitt: let me re-read the spec for that quick to see if i'm missing anything22:32
*** code-R_ has joined #openstack-nova22:33
*** code-R has quit IRC22:33
*** ijw has joined #openstack-nova22:33
melwittmriedem: okay. it's not urgent. I had been thinking the "move tables" has historically included object code that does the data migration22:34
melwittbut because of the second spec, we're not going to migrate everything, just part of it22:35
mriedemmelwitt: i didn't think we were doing migrations for quotas22:35
mriedembecause you can just refresh them22:35
melwittmriedem: I thought we would have to migrate any quota limits that are there22:35
*** ijw has quit IRC22:35
mriedemyeah probably22:35
mriedemjust not usages22:36
melwittif you use the API to update quota, it makes a limit entry in the DB that it uses from then on22:36
melwittright22:36
melwittthere won't be any more usages table22:36
melwitti.e. we won't put any data in there22:36
mriedemmelwitt: ok i think it makes sense to do the object code and limits migrations in that bp then22:38
mriedemand do the counting stuff in the counting spec22:38
melwittmriedem: k, thanks22:38
mriedemand now i'm off to make a turkey pot pie from all of the leftovers22:38
*** ijw has joined #openstack-nova22:38
melwittsounds good22:39
dansmith<-- jealous22:39
mriedemdon't be, first time, i'll likely ruin it22:39
* mriedem leaves22:39
*** mriedem has quit IRC22:39
dansmithI'm jealous you have leftovers22:39
*** ijw has quit IRC22:40
*** jheroux has quit IRC22:40
*** mnaser has quit IRC22:41
*** tblakes has quit IRC22:41
*** paw has joined #openstack-nova22:43
*** annegentle has quit IRC22:44
openstackgerritSujitha proposed openstack/nova: Add post hook script to run tempest with old n-api  https://review.openstack.org/40139522:44
*** slaweq_ has quit IRC22:44
*** ijw has joined #openstack-nova22:45
*** esberglu has quit IRC22:46
*** esberglu has joined #openstack-nova22:46
*** slaweq_ has joined #openstack-nova22:46
*** kylek3h has joined #openstack-nova22:47
*** ijw has quit IRC22:48
*** mnaser has joined #openstack-nova22:50
*** kristian__ has quit IRC22:51
*** ijw has joined #openstack-nova22:51
*** kristian__ has joined #openstack-nova22:51
*** esberglu has quit IRC22:51
*** dave-mcc_ has quit IRC22:53
*** masuberu has quit IRC22:54
*** masber has joined #openstack-nova22:54
*** ijw has quit IRC22:54
*** tbachman has joined #openstack-nova22:56
*** esberglu has joined #openstack-nova23:00
*** esberglu has quit IRC23:01
*** esberglu_ has joined #openstack-nova23:01
*** browne has quit IRC23:04
*** ljxiash has joined #openstack-nova23:05
*** _ducttape_ has joined #openstack-nova23:06
*** mdrabe has quit IRC23:07
*** thorst_ is now known as thorst_afk23:08
*** ljxiash has quit IRC23:09
*** ducttape_ has quit IRC23:10
*** ljxiash has joined #openstack-nova23:10
*** _ducttape_ has quit IRC23:11
*** Sukhdev has quit IRC23:12
*** catintheroof has quit IRC23:12
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements  https://review.openstack.org/39534723:13
*** ljxiash has quit IRC23:14
*** unicell has quit IRC23:15
*** burt has quit IRC23:15
*** unicell has joined #openstack-nova23:20
*** TravT has joined #openstack-nova23:22
openstackgerritLudovic Beliveau proposed openstack/nova: Race condition between audit and migrate/resize revert  https://review.openstack.org/40099523:24
dansmithmelwitt: so I hadn't looked that closely at your cell db fixture,23:24
*** jamesdenton has joined #openstack-nova23:24
dansmithbut it looks like you end up mocking the same thing a bunch of times23:24
dansmithget_context_manager for example23:24
dansmithmaybe just that one23:24
dansmithis that intentional?23:24
melwittdansmith: what do you mean by a bunch of times?23:24
dansmithmelwitt: you mock it in setup, and then again on each call to add_cell_db23:25
dansmithoh23:25
dansmithI see, maybe nevermind23:25
*** eharney has quit IRC23:25
dansmithadd_cell_db mocks temporarily,23:25
dansmiththen setup mocks for the test run23:25
melwittoh, that. yeah23:25
dansmithyeah I see now23:26
melwittI hope I put a note there. looking ...23:26
*** rcernin has quit IRC23:26
melwitttrying to get that thing to work was just weird, so there are likely better ways to do it too23:26
*** ducnc has quit IRC23:27
dansmithI'm trying to get it to work with functional now,23:27
*** ducnc has joined #openstack-nova23:27
dansmithand the services we start up are failing to do service record things23:27
dansmithI've been putting the "main cell" as the first add_cell_database() call,23:28
dansmithbut I'm wondering if that should be the last one, so that _last_ctxt_mgr will be "cell1" right off the bat..23:28
melwittyeah I think it should be last because of that hackiness23:28
dansmithhey, that makes it fail better23:29
dansmitha better wrong23:30
melwittdansmith: this is how I did it last time https://review.openstack.org/#/c/398645/2/nova/tests/functional/integrated_helpers.py23:30
melwittwhen I had tested it on top of your patch23:30
dansmithmelwitt: ah yeah, I read the comments and thought cell1 should be first23:30
dansmith# NOTE(melwitt): The first DB access through service start is23:31
dansmith        # local so this initializes _last_ctxt_mgr for that and needs23:31
dansmith        # to be a compute cell.23:31
dansmiththat23:31
dansmithbut my interpretation of that doesn't make sense for the code of course23:31
melwittoh, yeah. I guess my comment wasn't too good there23:32
melwittI'll improve it and also need to add a note about the temporary monkey patch thing too, assuming this ends up working23:32
*** jwcroppe has quit IRC23:34
dansmithwell, don't do it now or it'll mess up my linkage23:34
dansmithI'll try to remember to do that23:34
*** jwcroppe has joined #openstack-nova23:35
melwittno, I mean, later I'll make that stuff better if we can get all the tests working with it23:35
*** ijw has joined #openstack-nova23:36
melwittunless I missed something, the tests that derived from integrated_helpers were all passing in parallel and the remaining failures were tests that didn't, and needed CellDatabasesFixture set up for them23:36
*** claudiub|2 has quit IRC23:37
*** sdague has quit IRC23:38
*** penick has joined #openstack-nova23:38
dansmithmelwitt: my patch was never updated for the new fixture23:39
dansmithand just making the change in your patch above doesn't make them work23:39
*** ljxiash has joined #openstack-nova23:39
melwittdansmith: I know, in my experiment patch I modified what you did to use the new fixture and tried to run the tests in parallel23:39
*** jwcroppe has quit IRC23:39
melwittokay, I'll pull it down and see what it's doing23:40
dansmithwell23:40
dansmithlemme push up what I just did, which should be equivalent to what you had23:40
*** ijw has quit IRC23:41
melwittsure, that would be better23:41
dansmithbecause I took out the multicell listing patch23:41
openstackgerritDan Smith proposed openstack/nova: WIP Multicell support for instance listing  https://review.openstack.org/39677523:41
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937923:41
dansmithwith that, I'm getting no talky to conductor23:41
dansmithreversing the ordering of the add_db got me past the service create thing23:42
melwittk23:42
*** penick_ has joined #openstack-nova23:42
dansmithoh nm23:43
dansmiththat's compute23:43
*** pradk has quit IRC23:43
dansmiththat's the cascaded fail I hit last time23:43
*** penick has quit IRC23:43
*** penick_ is now known as penick23:43
*** lamt has quit IRC23:44
dansmithhmm23:45
*** ijw has joined #openstack-nova23:46
*** dtp has quit IRC23:46
dansmithyeaah23:47
dansmithI forgot23:47
dansmithI had a hack in there that I removed when I moved the rpc fixture out to another patch23:47
dansmithwhich is to default everything to a single bus23:47
melwittI hope you're about to tell me you already figured it out so I don't have to23:47
melwittoh, yeah23:47
dansmithbecause the things we start end up with the original config23:47
dansmithso yeah, I'm good23:47
melwittyeah you are23:47
dansmithnot sure what to do about that, but at least I have a thing23:47
dansmithI mean "good" as in "sorted"23:48
melwittoh. heh. well, I'm happy I didn't have to figure out how it wasn't talking to compute23:49
*** xyang1 has quit IRC23:50
melwittI've been confused about the same bus thing and how we could even do anything else since we can't give each service separate configs really23:50
dansmithholy frijoles, .*test_servers.* is working in parallel23:51
dansmithmelwitt: yeah I know, but I want to figure out a way23:51
*** ijw has quit IRC23:51
melwittdansmith: best I could think right now is something similar to the CellDatabasesFixture that intercepts and routes the calls somehow23:52
dansmithmelwitt: ah, and I got the connect string failure again23:52
dansmithmaybve23:52
dansmithdamn, such progress. I should take a week off to celebrate23:52
melwittheh. me too, that fixture was ... challenging to make work. I really hope it can handle all the functional tests23:53
*** Zer0Byte__ has joined #openstack-nova23:53
*** mriedem has joined #openstack-nova23:54
melwittdansmith: what's the connect string failure?23:54
dansmithmelwitt: remember the thing I commented on about having seen?23:54
dansmithbecause you weren't using url-ish urls for the connection string23:54
dansmithbut I dun' fixed that now23:54
dansmithfake://%(uuid)s23:55
melwittfor the database connection string? I didn't think that had to be formatted any way23:57
dansmithyeah something cares23:58

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!