Friday, 2016-12-09

*** ayoung has quit IRC00:05
*** ihrachys has quit IRC00:05
*** efried has quit IRC00:07
*** jamielennox|away is now known as jamielennox00:08
*** efried has joined #openstack-nova00:08
*** smatzek__ has quit IRC00:14
*** rama_y has joined #openstack-nova00:19
*** mlavalle has quit IRC00:21
*** ljxiash has joined #openstack-nova00:21
*** yamahata has joined #openstack-nova00:22
*** ljxiash has quit IRC00:26
*** rama_y has quit IRC00:27
*** baoli has joined #openstack-nova00:32
*** harlowja_ has quit IRC00:32
*** baoli has quit IRC00:38
*** tbachman has quit IRC00:45
*** nic has quit IRC00:49
*** jgriffith is now known as jgriffith_AutoAw00:49
*** harlowja has joined #openstack-nova00:51
*** hoangcx has joined #openstack-nova00:57
*** ayoung has joined #openstack-nova00:58
openstackgerritLi Yingjun proposed openstack/nova: Followup patch for minor fix for flavor-notifications  https://review.openstack.org/40886900:59
*** tbachman has joined #openstack-nova01:00
*** ducnc has joined #openstack-nova01:01
*** ishant|away has quit IRC01:03
*** jamielennox is now known as jamielennox|away01:04
*** ishant|away has joined #openstack-nova01:04
*** jamielennox|away is now known as jamielennox01:20
*** markvoelker has quit IRC01:22
*** markvoelker has joined #openstack-nova01:27
*** tbachman_ has joined #openstack-nova01:28
*** tbachman has quit IRC01:30
*** tbachman_ is now known as tbachman01:30
openstackgerritGhanshyam Mann proposed openstack/nova: Stop allowing tags as empty string  https://review.openstack.org/40887901:35
*** WillUK2010 has quit IRC01:38
*** mtanino has quit IRC01:39
*** Sukhdev has quit IRC01:39
*** jamesdenton has quit IRC01:39
*** raunak has joined #openstack-nova01:40
*** sacharya has joined #openstack-nova01:43
*** yufei has joined #openstack-nova01:45
*** sacharya has quit IRC01:47
*** yuntongjin has joined #openstack-nova01:50
*** browne1 has quit IRC01:51
*** raunak has quit IRC01:57
*** smatzek__ has joined #openstack-nova02:12
*** ljxiash has joined #openstack-nova02:13
*** ljxiash has quit IRC02:17
*** ducnc has quit IRC02:19
*** zhangjl has joined #openstack-nova02:28
*** int32bit_ has joined #openstack-nova02:30
*** esberglu has joined #openstack-nova02:32
*** esberglu has quit IRC02:39
*** kaisers__ has joined #openstack-nova02:42
openstackgerritgengchc2 proposed openstack/python-novaclient: Replace six.iteritems() with .items()  https://review.openstack.org/40890902:43
*** kaisers_ has quit IRC02:45
*** ducttape_ has joined #openstack-nova02:48
*** ducttape_ has quit IRC02:49
*** ducttape_ has joined #openstack-nova02:49
*** jwcroppe has joined #openstack-nova02:53
*** harlowja has quit IRC02:55
*** kevinz has joined #openstack-nova02:56
*** sam_nowitzki has joined #openstack-nova02:58
*** kevinz has quit IRC03:04
*** kevinz has joined #openstack-nova03:05
*** raunak has joined #openstack-nova03:08
*** thorst_ has joined #openstack-nova03:09
*** cdelatte has quit IRC03:15
*** kevinz has quit IRC03:24
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port libvirt.test_vif to Python 3  https://review.openstack.org/40341903:24
*** tuanluong has joined #openstack-nova03:25
*** yuntongjin has quit IRC03:25
*** dave-mccowan has quit IRC03:25
*** rnoriega has joined #openstack-nova03:28
*** smatzek__ has quit IRC03:28
*** links has joined #openstack-nova03:29
*** slagle has joined #openstack-nova03:29
*** Prophidys has joined #openstack-nova03:30
*** rdo has quit IRC03:30
*** rdo has joined #openstack-nova03:33
*** gjayavelu has quit IRC03:42
*** sacharya has joined #openstack-nova03:44
*** nandal_in has joined #openstack-nova03:46
*** ducttape_ has quit IRC03:48
*** sacharya has quit IRC03:48
*** pbandark has joined #openstack-nova03:53
*** yamahata has quit IRC03:54
*** tovin07 has quit IRC03:55
*** tovin07 has joined #openstack-nova03:56
*** claudiub has joined #openstack-nova03:57
openstackgerritLi Yingjun proposed openstack/nova: Check that all JSON files don't have \r\n in line  https://review.openstack.org/40894404:01
*** bks has joined #openstack-nova04:04
*** gjayavelu has joined #openstack-nova04:05
*** nicolasbock has quit IRC04:08
*** tbachman has quit IRC04:08
*** gjayavelu has quit IRC04:09
*** mdrabe has quit IRC04:13
*** psachin has joined #openstack-nova04:14
*** claudiub has quit IRC04:19
*** ijw has quit IRC04:19
*** ijw has joined #openstack-nova04:20
*** esberglu has joined #openstack-nova04:21
*** jwcroppe has quit IRC04:24
*** sacharya has joined #openstack-nova04:24
*** ijw has quit IRC04:24
*** esberglu has quit IRC04:26
*** tovin07 has quit IRC04:31
*** tovin07 has joined #openstack-nova04:32
*** int32bit_ has quit IRC04:34
*** ijw has joined #openstack-nova04:35
*** ayogi has joined #openstack-nova04:40
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/3)  https://review.openstack.org/35710504:40
*** ijw has quit IRC04:40
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/3)  https://review.openstack.org/35710504:42
*** kaisers__ has quit IRC04:46
*** tojuvone has joined #openstack-nova04:47
*** ducttape_ has joined #openstack-nova04:48
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port libvirt.storage.test_rbd to Python 3  https://review.openstack.org/40342504:52
*** ducttape_ has quit IRC04:53
*** links has quit IRC04:54
*** jschlueter has joined #openstack-nova04:56
*** yuntongjin has joined #openstack-nova04:56
*** zhenq has quit IRC04:56
*** sudipto has joined #openstack-nova04:58
*** sudipto_ has joined #openstack-nova04:58
*** jamespage has quit IRC05:00
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/3)  https://review.openstack.org/40895505:00
*** mdnadeem has joined #openstack-nova05:01
*** jamespag` has joined #openstack-nova05:01
*** mingyu has joined #openstack-nova05:02
*** mingyu has quit IRC05:04
*** cfriesen has quit IRC05:05
*** hamzy has joined #openstack-nova05:05
*** ijw_ has joined #openstack-nova05:05
*** yufei has quit IRC05:08
*** yufei has joined #openstack-nova05:08
openstackgerritDrew Thorstensen (thorst) proposed openstack/nova: WIP: Introduce base PowerVM driver  https://review.openstack.org/39128805:09
*** yufei has quit IRC05:09
*** links has joined #openstack-nova05:09
*** sc68cal has quit IRC05:12
*** prateek has joined #openstack-nova05:12
*** ratailor has joined #openstack-nova05:16
*** esberglu has joined #openstack-nova05:16
*** esberglu has quit IRC05:21
*** esberglu has joined #openstack-nova05:21
*** sc68cal has joined #openstack-nova05:24
*** esberglu has quit IRC05:25
*** jaypipes has joined #openstack-nova05:29
*** klkumar has joined #openstack-nova05:42
*** yamahata has joined #openstack-nova05:45
*** thorst_ has quit IRC05:45
*** thorst_ has joined #openstack-nova05:46
*** kaisers_ has joined #openstack-nova05:47
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(3/3)  https://review.openstack.org/40896405:47
*** takashin has left #openstack-nova05:50
*** sandanar has joined #openstack-nova05:51
*** kaisers_ has quit IRC05:52
*** GB21 has joined #openstack-nova05:52
*** kaisers_ has joined #openstack-nova05:54
*** kevinz has joined #openstack-nova05:54
*** thorst_ has quit IRC05:54
*** kaisers_ has quit IRC05:54
*** mingyu has joined #openstack-nova05:56
*** kaisers_ has joined #openstack-nova05:57
*** kaisers_ has quit IRC05:58
*** kristian__ has joined #openstack-nova06:01
*** raunak has quit IRC06:03
*** kristian__ has quit IRC06:05
*** esberglu has joined #openstack-nova06:11
*** klkumar has quit IRC06:14
*** klkumar has joined #openstack-nova06:15
*** esberglu has quit IRC06:16
*** esberglu has joined #openstack-nova06:16
*** ducttape_ has joined #openstack-nova06:19
*** esberglu has quit IRC06:21
*** ducttape_ has quit IRC06:24
*** Cristina_ has joined #openstack-nova06:30
*** CristinaPauna has quit IRC06:33
*** oanson has joined #openstack-nova06:45
*** kristian__ has joined #openstack-nova06:50
*** ijw_ has quit IRC06:51
*** ijw has joined #openstack-nova06:51
*** thorst_ has joined #openstack-nova06:53
*** yufei has joined #openstack-nova06:54
*** kristian__ has quit IRC06:55
*** nkrinner_afk is now known as nkrinner06:55
openstackgerritZhenyu Zheng proposed openstack/nova: Support tag instances when boot  https://review.openstack.org/39432106:55
*** ijw has quit IRC06:56
*** mkoderer has joined #openstack-nova06:58
*** kaisers_ has joined #openstack-nova06:59
*** thorst_ has quit IRC06:59
*** jaosorior has joined #openstack-nova07:00
*** longlq_ has joined #openstack-nova07:03
*** sam_nowitzki has quit IRC07:03
*** mjura has joined #openstack-nova07:03
*** kaisers_ has quit IRC07:03
*** esberglu has joined #openstack-nova07:06
*** sacharya has quit IRC07:07
*** tesseract has joined #openstack-nova07:07
*** tesseract is now known as Guest9999707:07
*** sacharya has joined #openstack-nova07:08
*** nandal has joined #openstack-nova07:09
*** GB21 has quit IRC07:09
*** ecelik has joined #openstack-nova07:10
*** ecelik has left #openstack-nova07:10
*** kristian__ has joined #openstack-nova07:11
*** nandal_in has quit IRC07:12
*** sacharya has quit IRC07:13
*** ihrachys has joined #openstack-nova07:13
*** Guest99997 is now known as tesseract-07:13
*** armax has quit IRC07:14
*** esberglu has quit IRC07:15
*** kristian__ has quit IRC07:15
*** lpetrut has joined #openstack-nova07:16
*** markus_z has joined #openstack-nova07:21
*** GB21 has joined #openstack-nova07:21
*** nherciu has joined #openstack-nova07:25
*** rcernin has joined #openstack-nova07:28
*** longlq__ has joined #openstack-nova07:28
*** longlq_ has quit IRC07:29
*** mingyu_ has joined #openstack-nova07:31
*** mingyu has quit IRC07:33
*** oanson has quit IRC07:33
*** GB21 has quit IRC07:34
*** sahid has joined #openstack-nova07:36
*** andreas_s has joined #openstack-nova07:38
*** kristian__ has joined #openstack-nova07:41
*** GB21 has joined #openstack-nova07:45
*** kristian__ has quit IRC07:45
*** gszasz has joined #openstack-nova07:47
openstackgerritZhenyu Zheng proposed openstack/nova: Add index on instances table across vm_state, task_state & project_id  https://review.openstack.org/40899907:47
*** WillUK2010 has joined #openstack-nova07:48
*** Dinesh_Bhor has quit IRC07:49
*** ducttape_ has joined #openstack-nova07:50
*** mingyu_ has quit IRC07:51
*** kristian__ has joined #openstack-nova07:51
tdurakovalex_xu: hi, could you please take a look https://review.openstack.org/#/c/408964/107:52
*** Dinesh_Bhor has joined #openstack-nova07:52
alex_xutdurakov: will try to reach that one later07:53
tdurakovalex_xu: cool, it's about api changes:)07:53
*** WillUK2010 has quit IRC07:53
alex_xutdurakov: yea07:53
*** ducttape_ has quit IRC07:54
*** GB21 has quit IRC07:55
*** jaosorior has quit IRC07:55
*** jaosorior has joined #openstack-nova07:55
*** kristian__ has quit IRC07:56
*** thorst_ has joined #openstack-nova07:57
*** mingyu has joined #openstack-nova07:58
*** kaisers_ has joined #openstack-nova08:00
*** yamahata has quit IRC08:00
*** jaosorior has quit IRC08:00
*** sacharya has joined #openstack-nova08:00
openstackgerritBhagyashri Shewale proposed openstack/nova: Return 400 when name is more than 255 characters  https://review.openstack.org/40900208:01
*** jaosorior has joined #openstack-nova08:02
openstackgerritTimofey Durakov proposed openstack/nova: fix for auth during live-migration  https://review.openstack.org/40714708:02
*** fragatin_ has quit IRC08:03
*** ihrachys has quit IRC08:03
*** thorst_ has quit IRC08:05
*** sacharya has quit IRC08:05
*** kaisers_ has quit IRC08:05
*** esberglu has joined #openstack-nova08:06
*** fragatina has joined #openstack-nova08:08
*** yuntongjin has quit IRC08:11
*** esberglu has quit IRC08:11
*** pcaruana has joined #openstack-nova08:16
*** lpetrut has quit IRC08:18
openstackgerritAlex Xu proposed openstack/nova: Add query parameters white list for server list/detail  https://review.openstack.org/40857108:18
*** stvnoyes has quit IRC08:20
*** stvnoyes has joined #openstack-nova08:20
openstackgerritsahid proposed openstack/nova: objects: remove related pining from __init__ of InstanceNUMATopology  https://review.openstack.org/40824208:23
openstackgerritsahid proposed openstack/nova: objects: remove pagesize from __init__ of InstanceNUMATopology  https://review.openstack.org/40824108:23
openstackgerritsahid proposed openstack/nova: objects: remove cpu_topology from __init__ of InstanceNUMATopology  https://review.openstack.org/40824008:23
*** fragatina has quit IRC08:27
*** zenoway has joined #openstack-nova08:31
*** avolkov has quit IRC08:31
*** lpetrut has joined #openstack-nova08:32
*** zenoway has quit IRC08:33
*** zenoway has joined #openstack-nova08:33
tdurakovraj_singh: hi, are you around?08:33
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.rescue notifications  https://review.openstack.org/38564408:34
*** matrohon has joined #openstack-nova08:36
*** Jeffrey4l has quit IRC08:38
*** yuntongjin has joined #openstack-nova08:43
gibibauzas: hi! Li proposed a subsequent patch for https://review.openstack.org/#/c/403989 with the automatic check. So I think https://review.openstack.org/#/c/403989 is good to go in08:45
bauzasgibi: okay cool :)08:45
*** mingyu has quit IRC08:52
*** sdatko has quit IRC08:53
bauzasgibi: do you know Li's IRC ?08:53
bauzasgibi: I have a question for him about the follow-up change08:54
gibibauzas: let me check my logs08:57
*** yassine has joined #openstack-nova08:59
*** zzzeek has quit IRC09:00
*** yassine is now known as Guest2371809:00
gibibauzas: Li's IRC is yingjun09:00
*** zzzeek has joined #openstack-nova09:00
*** ihrachys has joined #openstack-nova09:00
*** ralonsoh has joined #openstack-nova09:00
*** lpetrut has quit IRC09:00
*** yuntongjin has quit IRC09:01
*** kaisers_ has joined #openstack-nova09:01
*** esberglu has joined #openstack-nova09:01
*** gszasz has quit IRC09:01
*** yuntongjin has joined #openstack-nova09:02
*** thorst_ has joined #openstack-nova09:02
*** gszasz has joined #openstack-nova09:04
*** namnh has joined #openstack-nova09:05
*** kaisers_ has quit IRC09:06
*** esberglu has quit IRC09:06
bauzasgibi: saw my comment on the change ?09:08
bauzasoh yes09:10
*** thorst_ has quit IRC09:10
openstackgerritZhenyu Zheng proposed openstack/nova: Support tag instances when boot  https://review.openstack.org/39432109:13
*** gszasz has quit IRC09:15
*** ducttape_ has joined #openstack-nova09:20
*** sahid has quit IRC09:20
gibibauzas: I agree with your comment09:20
bauzasI think he probably just copied the below line09:21
*** int32bit has joined #openstack-nova09:21
gibiyeah09:22
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Enable test_bdm.BlockDeviceMappingEc2CloudTestCase on Python 3  https://review.openstack.org/40841509:22
*** yuntongjin has quit IRC09:23
*** ducttape_ has quit IRC09:25
*** kristian__ has joined #openstack-nova09:25
*** cdent has joined #openstack-nova09:26
*** nandal_in has joined #openstack-nova09:27
openstackgerritLi Yingjun proposed openstack/nova: Notifications on flavor operations  https://review.openstack.org/39817109:30
*** kristian__ has quit IRC09:30
*** nandal has quit IRC09:30
openstackgerritLi Yingjun proposed openstack/nova: Check that all JSON files don't have \r\n in line  https://review.openstack.org/40894409:34
*** gszasz has joined #openstack-nova09:42
*** longlq_ has joined #openstack-nova09:43
*** kristian__ has joined #openstack-nova09:45
*** longlq__ has quit IRC09:46
*** kevinz has quit IRC09:46
*** kaisers_ has joined #openstack-nova09:48
*** GB21 has joined #openstack-nova09:49
*** kristian__ has quit IRC09:49
*** betherly_ has joined #openstack-nova09:51
betherly_hi all09:51
*** yingjun has joined #openstack-nova09:51
*** claudiub has joined #openstack-nova09:51
betherly_wanting to spin up some fake/dummy instances and drivers to test pagination work im doing in horizon09:51
*** esberglu has joined #openstack-nova09:51
betherly_i have got as far as http://docs.openstack.org/developer/nova/icehouse/devref/fakes.html09:51
betherly_can someone help me with where and how to configure this?09:51
*** kaisers_ has quit IRC09:52
*** yingjun has left #openstack-nova09:52
*** yingjun has quit IRC09:52
robcresswellbetherly_: The easiest way to test is to set Horizons page limit to 2 or 3 then create a dozen tiny instances :)09:53
*** int32bit has quit IRC09:55
*** mingyu has joined #openstack-nova09:56
openstackgerritAlex Xu proposed openstack/nova: Add query parameters white list for server list/detail  https://review.openstack.org/40857110:00
*** esberglu has quit IRC10:01
*** mingyu has quit IRC10:01
betherly_robcresswell: ah perfect thank you!10:02
*** eandersson has quit IRC10:03
*** thorst_ has joined #openstack-nova10:09
*** avolkov has joined #openstack-nova10:12
openstackgerritBhagyashri Shewale proposed openstack/nova: Return 400 when name is more than 255 characters  https://review.openstack.org/40900210:13
*** thorst_ has quit IRC10:15
*** sahid has joined #openstack-nova10:16
*** longlq_ has quit IRC10:16
*** yufei has quit IRC10:18
*** zhangjl has left #openstack-nova10:21
*** tovin07 has quit IRC10:22
openstackgerritAndrea Frittoli proposed openstack/nova: Guestfs handle no passwd or group in image  https://review.openstack.org/40703710:23
*** sudipto_ has quit IRC10:27
*** sudipto has quit IRC10:27
*** erlon-airlong has joined #openstack-nova10:28
*** kristian__ has joined #openstack-nova10:30
*** hoangcx has quit IRC10:30
*** lpetrut has joined #openstack-nova10:31
*** nherciu_ has joined #openstack-nova10:34
*** kristian__ has quit IRC10:34
*** nherciu has quit IRC10:38
*** GB21 has quit IRC10:39
*** kristian__ has joined #openstack-nova10:43
*** Jeffrey4l has joined #openstack-nova10:47
*** kristian__ has quit IRC10:47
*** kaisers_ has joined #openstack-nova10:48
*** ratailor has quit IRC10:50
*** ducttape_ has joined #openstack-nova10:51
*** timello has joined #openstack-nova10:51
*** esberglu has joined #openstack-nova10:51
*** kaisers_ has quit IRC10:53
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Use min parameter to restrict live-migration config options  https://review.openstack.org/32331310:55
*** ducttape_ has quit IRC10:55
*** esberglu has quit IRC10:55
*** betherly_ has quit IRC10:56
openstackgerritDirk Mueller proposed openstack/nova: [WIP] Testing latest u-c  https://review.openstack.org/40907710:58
*** jaosorior is now known as jaosorior_lunch11:00
BlackDexHello there, when i deploy an instance i always get .novalocal appened to the hostname. This happens with the debian 8.6.x images for instance.11:01
BlackDexEven though i have set dhcp_domain and dns_domain to '' so it should be empty11:01
BlackDexi set it in the nova.conf of the nova-cloud-controller and nova-compute nodes11:01
*** GB21 has joined #openstack-nova11:01
*** derekh has joined #openstack-nova11:06
*** claudiub has quit IRC11:08
*** sdague_ has joined #openstack-nova11:09
*** tbachman has joined #openstack-nova11:09
*** thorst_ has joined #openstack-nova11:13
tdurakovbauzas: hi, could you take a look https://review.openstack.org/#/c/407147/11:15
*** thorst_ has quit IRC11:20
*** mdnadeem has quit IRC11:22
*** claudiub has joined #openstack-nova11:24
*** tbachman_ has joined #openstack-nova11:28
*** tbachman has quit IRC11:30
*** tbachman_ is now known as tbachman11:30
openstackgerritStephen Finucane proposed openstack/os-vif: vif_plug_ovs: Always set MTU when plugging devices  https://review.openstack.org/40877911:32
*** tuanluong has quit IRC11:34
*** nicolasbock has joined #openstack-nova11:35
*** david-lyle_ has joined #openstack-nova11:35
*** david-lyle has quit IRC11:37
openstackgerritStephen Finucane proposed openstack/nova: Always set MTU when plugging devices  https://review.openstack.org/40878111:38
*** klkumar has quit IRC11:45
openstackgerritStephen Finucane proposed openstack/os-vif: vif_plug_ovs: Always set MTU when plugging devices  https://review.openstack.org/40877911:47
*** kaisers_ has joined #openstack-nova11:49
*** kaisers_ has quit IRC11:54
*** klkumar has joined #openstack-nova11:55
*** jaosorior_lunch is now known as jaosorior11:55
*** smatzek__ has joined #openstack-nova11:57
*** nherciu__ has joined #openstack-nova11:57
*** smatzek__ has quit IRC11:58
*** smatzek has joined #openstack-nova11:59
*** bauzas is now known as bauwseer12:00
*** bauwseer is now known as bauwser12:00
sdague_jaypipes: when you wake, I've got a bug that needs a complicated sql solution, and my brain can't quite generate it - https://bugs.launchpad.net/nova/+bug/163224712:00
openstackLaunchpad bug 1632247 in OpenStack Compute (nova) "We fetch all instance faults, then throw away all but the last ones on "nova list --all-tenants", potentinally making the command slow" [Medium,Confirmed] - Assigned to Sivasathurappan Radhakrishnan (siva-radhakrishnan)12:00
wznoinsksfinucan, around?12:00
sfinucanwznoinsk: Sure am12:00
*** nherciu_ has quit IRC12:00
wznoinskgreat, intel nfv ci coming back today, it's currently silently testing all nova changes anyways, spotted an issue with numa (or test of it) - http://intel-openstack-ci-logs.ovh/31/403431/3/check/tempest-dsvm-intel-nfv-xenial/13a2872/ , would you have a moment to have a look?12:01
*** edmondsw_ has joined #openstack-nova12:02
*** nherciu_ has joined #openstack-nova12:02
wznoinsksfinucan, ^12:02
sfinucanwznoinsk: Sure12:03
*** edmondsw_ has quit IRC12:03
wznoinskcheers12:03
*** nherciu__ has quit IRC12:05
*** dikonoor has joined #openstack-nova12:09
*** dave-mccowan has joined #openstack-nova12:09
*** kristian__ has joined #openstack-nova12:09
*** namnh has quit IRC12:12
BlackDexI just stumbled upon an other charm which doesn't allow config change but does have this config in its tempaltes. the neutron-gateway, it has a nova.conf which i need to change also because of the .novalocal stuff. I can't use the charm to change the nova.conf. So i needed to update the file my self and also the templates to prevent a los of the configuration.12:12
*** oanson has joined #openstack-nova12:13
*** kristian__ has quit IRC12:13
*** zhangjl has joined #openstack-nova12:14
*** tbachman has quit IRC12:14
* mdbooth wonders what a charm is12:15
kashyapmdbooth: "Charms are sets of scripts for deploying and managing services. With event handling built in, they can declare interfaces that fit charms for other services, so relationships can be formed." -- https://jujucharms.com/12:16
kashyap(From Ubuntu)12:16
mdboothAh....12:17
mdboothI have heard of those12:17
*** kristian__ has joined #openstack-nova12:17
*** thorst_ has joined #openstack-nova12:18
mdboothBlackDex: We don't make charms here. I think you want to report that to Canonical.12:18
BlackDexoeps12:18
BlackDexsorru wrong channel :)12:18
BlackDexi thought i still was in openstack-charms12:18
openstackgerritAlex Xu proposed openstack/nova: Add query parameters white list for server list/detail  https://review.openstack.org/40857112:19
*** bvanhav has joined #openstack-nova12:19
BlackDexbut still, al those nova.conf location ;)12:19
*** fragatina has joined #openstack-nova12:20
*** ducttape_ has joined #openstack-nova12:21
*** romainh has joined #openstack-nova12:23
wznoinsksfinucan, a few more examples: http://paste.openstack.org/show/591931/12:24
*** thorst_ has quit IRC12:24
tdurakovsdague_: hi, are you around?12:26
*** ducttape_ has quit IRC12:26
*** nherciu_ has quit IRC12:27
*** nherciu has joined #openstack-nova12:27
openstackgerritAlex Xu proposed openstack/nova-specs: Add whitelist for filter and sort query parameters for server list API  https://review.openstack.org/39320512:29
cdentgibi: thanks for the explanation, makes sense. the visibility is a good thing, so a fair price to pay12:29
gibicdent: It was good that you brought this up as finally I managed to explain myself why we have this in the code :)12:32
cdent:)12:32
alex_xusdague_: just found both filters 'name' and 'display_name' works for admin user. I decided to ignore the 'display_name', due to 'display_name' isn't claim supported before. 'name' is works for non-admin user and is also claimed by python-novaclient. let me know if you don't think it is right choice.12:32
gibicdent: it was an emergent thing12:32
cdentas so many things12:32
*** jamielennox is now known as jamielennox|away12:33
sdague_alex_xu: is there a reason not to allow both? or did we want to deprecate one?12:35
sdague_honestly, I kind of feel like the overhead of additional filter aliases is pretty low12:35
*** GB21 has quit IRC12:40
sfinucanwznoinsk: So all those errors seem to be unrelated to the tests themselves12:42
sfinucanInstead, it appears to be something to do with...deleting floating IPs?12:43
*** tbachman has joined #openstack-nova12:44
sfinucanwznoinsk: This is the line that's breaking https://github.com/openstack/tempest/blob/master/tempest/lib/services/compute/floating_ips_client.py#L77-L7812:44
sfinucanI'll have a look at the tests, but I would suspect (based on wild guesses and accusations) a latent bug somewhere12:44
sfinucan...but I'll have a look /after/ lunch :)12:45
*** pbandark has quit IRC12:45
*** nherciu_ has joined #openstack-nova12:45
*** kaisers_ has joined #openstack-nova12:46
sfinucanwznoinsk: I do note that that test, unlike the others, is based off of 'NetworkScenarioTest' rather than 'BaseV2ComputeAdminTest'. Must be doing something different12:47
*** ayogi has quit IRC12:48
*** nherciu has quit IRC12:48
openstackgerritAndrea Frittoli proposed openstack/nova: Guestfs handle no passwd or group in image  https://review.openstack.org/40703712:48
*** gszasz has quit IRC12:49
*** kaisers_ has quit IRC12:50
*** zhangjl has quit IRC12:50
*** tbachman has quit IRC12:53
*** gcb has quit IRC12:54
*** gszasz has joined #openstack-nova12:55
*** psachin has quit IRC12:55
*** tblakes has joined #openstack-nova12:55
*** cdelatte has joined #openstack-nova12:57
*** pradk has joined #openstack-nova13:01
*** fyxim has quit IRC13:02
*** fyxim has joined #openstack-nova13:02
*** pcaruana has quit IRC13:05
*** pcaruana has joined #openstack-nova13:08
*** thorst_ has joined #openstack-nova13:08
*** ducttape_ has joined #openstack-nova13:09
*** sudipto_ has joined #openstack-nova13:12
*** sudipto has joined #openstack-nova13:12
*** nherciu_ has quit IRC13:14
*** nherciu has joined #openstack-nova13:15
bauwsertdurakov: hola13:15
bauwsertdurakov: I'm maybe not the best core for reviewing https://review.openstack.org/#/c/407147/13:15
bauwsertdurakov: but I'll look at it13:16
tdurakovbauwser, no worries, thanks anyway13:16
*** ljxiash has joined #openstack-nova13:18
*** asettle has quit IRC13:18
*** asettle has joined #openstack-nova13:19
*** kylek3h_away has joined #openstack-nova13:26
cdentjaypipes: i'm going to fix the pep8 change in this, so you're aware: https://review.openstack.org/#/c/407309/13:26
*** tbachman has joined #openstack-nova13:27
*** romainh1 has joined #openstack-nova13:28
*** mdrabe has joined #openstack-nova13:30
*** romainh has quit IRC13:30
*** maestropandy has joined #openstack-nova13:31
*** maestropandy has left #openstack-nova13:31
*** romainh has joined #openstack-nova13:33
openstackgerritChris Dent proposed openstack/nova: HTTP interface for resource providers by aggregates  https://review.openstack.org/40774213:34
openstackgerritChris Dent proposed openstack/nova: placement: create aggregate map in report client  https://review.openstack.org/40730913:34
openstackgerritChris Dent proposed openstack/nova: Support filtering resource providers by aggregate membership  https://review.openstack.org/40762913:34
*** dougwig has quit IRC13:34
*** romainh1 has quit IRC13:34
*** dougwig has joined #openstack-nova13:35
*** tbachman_ has joined #openstack-nova13:37
*** tbachman has quit IRC13:37
*** tbachman_ is now known as tbachman13:37
openstackgerritChris Dent proposed openstack/nova: HTTP interface for resource providers by aggregates  https://review.openstack.org/40774213:37
*** maestropandy has joined #openstack-nova13:39
*** maestropandy has left #openstack-nova13:39
*** andreas_s has quit IRC13:41
*** ducttape_ has quit IRC13:41
*** ljxiash_ has joined #openstack-nova13:41
*** ljxiash__ has joined #openstack-nova13:43
openstackgerritSergey Nikitin proposed openstack/nova: Mark sibling CPUs as 'used' for cpu_thread_policy = 'isolated'  https://review.openstack.org/39141613:43
*** ljxiash has quit IRC13:44
*** nherciu_ has joined #openstack-nova13:46
*** ljxiash_ has quit IRC13:46
*** kaisers_ has joined #openstack-nova13:47
*** nherciu has quit IRC13:47
*** boris-42 has quit IRC13:51
*** kaisers_ has quit IRC13:51
*** boris-42 has joined #openstack-nova13:51
*** edleafe is now known as figleaf13:52
*** andymaier has joined #openstack-nova13:53
sdague_bauwser: https://review.openstack.org/#/c/392569/15 - question in there about unexpected values13:54
* bauwser looking13:54
bauwsersdague_: yeah, that seems to me a good point13:55
bauwsersdague_: that was written by jaypipes but I can change it13:55
sdague_bauwser: cool13:56
sdague_yeh, I think just doing some stronger validation there saves us a lot of grief and bugs later13:56
bauwsersdague_: just making sure to understand you, what do you mean by "fuzzing" ?13:56
*** dinobot_ has joined #openstack-nova13:56
bauwserokay13:56
sdague_unexpected values in parameters13:56
bauwseroops13:56
pkholkindansmith: hello Dan, could you please look at this patch https://review.openstack.org/#/c/246910/, obondarev asks you to unblock it, thanks!13:56
sdague_like CPU=-113:56
bauwsersdague_: okay, lemme try to check that13:57
pkholkinbauzas: are you here? I want to return to this https://review.openstack.org/#/c/403640/, thanks!13:57
*** nherciu has joined #openstack-nova13:58
bauwserthe last time I heard "fuzzing" was with OWASP :)13:58
bauwserpkholkin: \o13:58
*** nherciu_ has quit IRC13:58
bauwserpkholkin: it's Friday :)13:59
mdbooth\o/13:59
*** daemontool has joined #openstack-nova14:00
*** bvanhav has quit IRC14:00
*** kristia__ has joined #openstack-nova14:01
*** bvanhav has joined #openstack-nova14:01
*** kristian__ has quit IRC14:01
*** kristia__ has quit IRC14:02
*** mariusv has quit IRC14:02
*** lunarlamp is now known as mariusv14:02
*** mjura has quit IRC14:02
*** xyang1 has joined #openstack-nova14:03
*** mariusv has joined #openstack-nova14:03
*** mariusv has quit IRC14:03
*** mariusv has joined #openstack-nova14:03
*** tlian has joined #openstack-nova14:03
*** kristian__ has joined #openstack-nova14:03
pkholkinbauwser: yeah, I know)14:05
pkholkinbut I still want to dusciss it with you)14:05
pkholkine.g.g on next week)14:05
bauwserpkholkin: heh, no worries, I'm here :)14:05
*** liverpooler has joined #openstack-nova14:06
*** paul-carlton2 has joined #openstack-nova14:06
pkholkinbauwser: I wanted to do smth with this issue because after this I want to implement my spec https://review.openstack.org/38191214:06
pkholkinbut this "bug" is related to it14:06
pkholkinbauwser: me ideal plan is to do smth with this issue in Ocata and implement the spec in P14:07
pkholkinmy*14:07
*** kristian__ has quit IRC14:07
bauwserokay lemme understand14:07
bauwserso, your spec isn't merged14:07
*** kristian__ has joined #openstack-nova14:07
*** jamesdenton has joined #openstack-nova14:07
bauwserbut you think that for Ocata, we could just merge a bugfix first, right?14:07
bauwserie. https://batiment.imag.fr/files/eco-conception-v2.pdf14:08
bauwseroops14:08
bauwsersorry14:08
bauwserbad copy/paste14:08
*** mlavalle has joined #openstack-nova14:08
bauwser(if people want to attend SnowCamp 2017 conf, here is the venue ^ :p )14:08
pkholkinbauwser: yeah, I want to solve the issue in some way, and to implement the spec in P (it is not approved for O :( )14:09
bauwserokay I see14:09
pkholkingreat!14:09
bauwserso, about https://review.openstack.org/#/c/403640/, how can I help ?14:09
pkholkinthis changes current behaviour of the filter14:10
pkholkinbut to correct way14:10
pkholkinwe should think how to do it in the best way14:11
alex_xusdague_: I didn't have very strong reason for not allow both. just think it is admin-api, and it isn't the one we explictly say supported.14:11
pkholkinnot to affect users14:11
alex_xusdague_: yea, there isn't any overhead for few more alias14:11
pkholkinbut to fix the filter (I think it should work as aggregate_extra_specs)14:11
*** kevinz has joined #openstack-nova14:11
pkholkinbauwser: ^14:12
*** tbachman has quit IRC14:14
syjulianhi all, would anyone be available to do a quick notifications transformation review? https://review.openstack.org/#/c/330128/14:14
*** baoli has joined #openstack-nova14:15
openstackgerritAlex Xu proposed openstack/nova-specs: Add whitelist for filter and sort query parameters for server list API  https://review.openstack.org/39320514:16
*** esberglu has joined #openstack-nova14:17
*** nkrinner is now known as nkrinner_afk14:19
*** mtanino has joined #openstack-nova14:19
*** nkrinner_afk is now known as nkrinner14:19
*** ducttape_ has joined #openstack-nova14:19
*** dimtruck has quit IRC14:22
bauwserpkholkin: mmmm, okay, I'll review that WIP14:22
bauwserpkholkin: I'm honestly not sure we would want to modify a filter behaviour, but lemme see your change14:23
*** andrewbogott has quit IRC14:23
pkholkinbauwser: ok, thanks14:23
*** andrewbogott has joined #openstack-nova14:23
*** andrewbogott has joined #openstack-nova14:23
openstackgerritChris Dent proposed openstack/nova-specs: Update the generic resource pools spec to reflect reality  https://review.openstack.org/40756214:24
*** nherciu_ has joined #openstack-nova14:25
*** bks has quit IRC14:25
cdentfigleaf: you around? if not I'll rebase this: https://review.openstack.org/#/c/407180/14:25
openstackgerritAlex Xu proposed openstack/nova: Add query parameters white list for server list/detail  https://review.openstack.org/40857114:26
*** kristian__ has quit IRC14:27
*** nkrinner is now known as nkrinner_afk14:28
*** nherciu__ has joined #openstack-nova14:28
*** kristian__ has joined #openstack-nova14:28
*** nherciu has quit IRC14:28
*** bhagyashris has quit IRC14:30
*** nherciu_ has quit IRC14:30
*** tbachman has joined #openstack-nova14:30
*** kristian__ has quit IRC14:32
*** kristian__ has joined #openstack-nova14:32
*** smatzek has quit IRC14:32
*** nherciu_ has joined #openstack-nova14:36
*** gszasz has quit IRC14:37
*** nherciu__ has quit IRC14:37
*** patrickeast has quit IRC14:38
openstackgerritChris Dent proposed openstack/nova: Do not post allocations that are zero  https://review.openstack.org/40718014:38
*** patrickeast has joined #openstack-nova14:38
tdurakovjaypipes: hi, could you please re +2 once again https://review.openstack.org/#/c/407147/ - fixed typos in commit message14:39
*** dikonoor has quit IRC14:39
*** pcaruana has quit IRC14:42
*** GB21 has joined #openstack-nova14:45
*** markus_z has quit IRC14:45
*** kristian__ has quit IRC14:46
*** ljxiash__ has quit IRC14:46
*** kristian__ has joined #openstack-nova14:46
*** kaisers_ has joined #openstack-nova14:47
*** Dinesh_Bhor has quit IRC14:50
*** cargonza has quit IRC14:50
*** cargonza has joined #openstack-nova14:50
*** macsz has quit IRC14:51
*** macsz has joined #openstack-nova14:52
*** kaisers_ has quit IRC14:53
*** READ10 has joined #openstack-nova14:54
*** ediardo has quit IRC14:56
*** tbachman has quit IRC14:56
*** ediardo has joined #openstack-nova14:56
*** smatzek has joined #openstack-nova14:56
*** pcaruana has joined #openstack-nova14:56
*** kaisers_ has joined #openstack-nova14:58
*** mriedem has joined #openstack-nova14:58
*** links has quit IRC14:59
sdague_bauwser / alex_xu - this should actually be a pretty straight forward bug fix - https://review.openstack.org/#/c/408668/  - just needs a final +215:01
*** kaisers_ has quit IRC15:02
*** cfriesen has joined #openstack-nova15:02
jrollcdent: ewwwww extra_spec-like thing for now :/15:03
cdentjroll: yeah, eww15:03
cdentpotential emphasis on the '-like'15:04
*** gouthamr has joined #openstack-nova15:04
openstackgerritDan Smith proposed openstack/nova: Cleanup after any failed libvirt spawn  https://review.openstack.org/40880615:04
bauwsersdague_: yeah I saw that one15:04
*** dansmith is now known as superdan15:04
jrollcdent: I wonder if we should just leave cpu/ram/disk scheduling until we have the requires/prefers thing, so people don't have to do two flavor transitions in such a short time15:05
*** csatari has quit IRC15:05
*** csatari has joined #openstack-nova15:05
bauwsersdague_: Wipped15:05
cdentjroll: I reckon the requires/prefers expression in the future requestspec-like thing is quite a ways off, when flavors mostly evaporate (at least in communication amongst the services, if not with humans)15:07
jrollcdent: ah, I thought the requires/prefers thing was part of (all of?) a flavor definition15:08
jroll(or could be)15:08
cdentit expresses sort of the same thing as a flavor, but is dynamic (as in changeable in any detail)15:08
jrollright, so it's specified by the user?15:09
cdentnot necessarily. I think how it is going to be used is still up in the air. And the details of how it all work are fuzzy (at least to me)15:09
jrollmmm. ok.15:09
sfinucanwznoinsk: So I can't test this locally (I finally have a physical machine again but it's not configured yet) but https://review.openstack.org/#/q/topic:feat/resolve-floatingip-issues15:10
*** tomhambleton has quit IRC15:10
mriedemsuperdan: left a question in that change of yours - the guest isn't always started in there15:10
mriedemyou can resize/migrate a stopped guest15:10
mriedemi'm not sure if it will make any difference though15:10
*** tomhambleton has joined #openstack-nova15:10
jrollcdent: so the one other downside of "extra specs like thing" is 'nova flavor-list' is going to feel very empty :P15:10
*** tbachman has joined #openstack-nova15:10
openstackgerritAndrey Volkov proposed openstack/nova: PoC: add hierarchy quota driver  https://review.openstack.org/39107215:11
cdentjroll: these things are inscrutable to me15:11
jrollcdent: fair enough :) thanks15:11
cdentIf I knew what was going on, I would be writing these emails so people can correct me or questions. I'd be writing ThePlanâ„¢15:12
cdents/would/wouldn't15:12
*** jwcroppe has joined #openstack-nova15:12
*** jwcroppe has quit IRC15:12
*** jwcroppe has joined #openstack-nova15:13
jrollheh15:13
jrollI'm just bouncing thoughts around, I know even less of what's going on :)15:13
*** derekh has quit IRC15:13
cdentyeah, is good, that's totally the reason for the emails: get more thoughts bouncing15:14
openstackgerritMaciej Szankin proposed openstack/nova: conf: added notifications group  https://review.openstack.org/39214515:14
jroll++15:14
*** derekh has joined #openstack-nova15:15
wznoinsksfinucan, cool, thanks, the 4 are in the works in our CI already, ETD of results 1h15:16
sfinucanwznoinsk: Sweet. Out of curiosity, intel-nfv-ci uses zuul and will drop older patches if I resubmit new ones, right?15:17
wznoinsksfinucan, correct, will abort and not report if newer patchers are submitted15:17
sfinucanwznoinsk: Excellent15:17
wznoinsksfinucan, btw you never had shortage of physical machines in your last work place ;-)15:18
sfinucanwznoinsk: That I did not. Never thought I'd miss a /machine/ :P15:18
*** mwhahaha has quit IRC15:18
*** mwhahaha has joined #openstack-nova15:18
*** kevinz has quit IRC15:19
mriedemsahid: any idea what happens if you call guest.poweroff() on an instance that isn't running? is it just a noop or does it fail?15:19
*** tbachman has quit IRC15:20
mdboothmriedem: I just tried it locally15:20
mdboothWith virsh, not python-libvirt15:20
mdboothIt raises an exception15:20
mriedemew15:21
mdboothSo, that's probably a bug in the other 2 code paths, too15:21
superdanmdbooth: I can't find in libvirt-python where it would raise though, even if automatic15:21
*** efried has quit IRC15:21
mdboothsuperdan: I'm just firing up a test env to do it with python-libvirt15:21
*** efried has joined #openstack-nova15:21
superdanokaay15:22
mdboothsuperdan: It's pretty simple to hand, though15:22
*** jwcroppe has quit IRC15:22
mdboothLogic is:15:22
superdansure, I can just catch and release, but also,15:22
mdboothif guest is not None and power_on:15:22
superdanif create_domain raises, we won't get guest in the handler anyway15:22
mdboothRight15:22
superdanoh, you guys are talking about poweron15:22
mdboothAnd we assume it handled that15:22
mriedemwell, or, if guest and guest.is_active():15:22
*** jwcroppe has joined #openstack-nova15:22
mriedemguest.poweroff()15:22
superdanyeah, yeah15:22
superdanis_active means not having to pass yet another thing to that helper15:23
mriedemsuperdan: which you'd want right?15:24
superdanmriedem: which I'd who?15:24
mriedemnvm15:24
mriedemyou don't need to pass power_on to your helper if you just use if guest and guest.is_active()15:24
*** robcresswell has quit IRC15:24
superdanthat's what I'm saying yes15:25
*** robcresswell has joined #openstack-nova15:25
*** yamahata has joined #openstack-nova15:25
*** TravT has joined #openstack-nova15:25
*** lamt has joined #openstack-nova15:25
mdboothThat's an additional libvirt call for context you've already got, but in the grand scheme probably not an issue :)15:26
*** jwcroppe has quit IRC15:26
mdboothsuperdan: Probably better to catch the exception and ignore it, tbh15:27
mdboothIdempotent that sucker15:27
superdanmdbooth: yeah already typed those things :)15:27
* mdbooth will try to determine exactly what exception it raises in that circumstance15:28
mriedemlibvirtError15:29
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: remove extraneous retry assignment in cleanup method  https://review.openstack.org/40919915:29
mriedemis what you'll get from the guest15:29
mriedemwith some msg and error codes, but you probably don't care about them it sounds like15:29
*** mriedem is now known as mriedem_meeting15:29
openstackgerritDan Smith proposed openstack/nova: Cleanup after any failed libvirt spawn  https://review.openstack.org/40880615:30
superdanthere, the most well-tested trivial change evar ^15:30
*** daemontool has quit IRC15:33
mdboothsuperdan: http://paste.openstack.org/show/591956/15:34
superdanmdbooth: ack, but I think I'm good with that latest change right?15:34
* mdbooth hasn't looked yet15:34
superdanokay15:34
*** jaosorior has quit IRC15:35
*** tobasco has joined #openstack-nova15:36
*** nandal_in has quit IRC15:36
openstackgerritChris Dent proposed openstack/nova: rt: pass the nodename to public methods  https://review.openstack.org/39847015:36
openstackgerritChris Dent proposed openstack/nova: HTTP interface for resource providers by aggregates  https://review.openstack.org/40774215:36
openstackgerritChris Dent proposed openstack/nova: rt: remove fluff from test_resource_tracker  https://review.openstack.org/39847115:36
openstackgerritChris Dent proposed openstack/nova: rt: pass nodename to internal methods  https://review.openstack.org/39846915:36
openstackgerritChris Dent proposed openstack/nova: placement: create aggregate map in report client  https://review.openstack.org/40730915:36
openstackgerritChris Dent proposed openstack/nova: Support filtering resource providers by aggregate membership  https://review.openstack.org/40762915:36
openstackgerritChris Dent proposed openstack/nova: placement: correct improper test case inheritance  https://review.openstack.org/40636315:36
openstackgerritChris Dent proposed openstack/nova: rt: explicitly pass compute node to _update()  https://review.openstack.org/39847215:36
openstackgerritChris Dent proposed openstack/nova: rt: use a single ResourceTracker object instance  https://review.openstack.org/39847315:36
tobascowhat service executed the service creation? should be api or conductor right? https://github.com/openstack/nova/blob/d49f901b43fb06a041ff0a54666c0df37273878b/nova/db/sqlalchemy/api.py#L56615:37
tobasco*executes15:37
mdboothsuperdan: Yeah, finally block is safest for that call.15:38
superdancool15:38
*** ducttape_ has quit IRC15:39
*** paul-carlton2 has quit IRC15:44
*** eharney has joined #openstack-nova15:44
kashyapsuperdan: Is passing five things a bit excessive?15:46
kashyapI now see you've refactored it into a common method15:46
*** krtaylor has joined #openstack-nova15:47
superdankashyap: not my preference :)15:48
* kashyap nods15:48
*** tbachman has joined #openstack-nova15:48
*** pradk- has joined #openstack-nova15:49
*** kristian__ has quit IRC15:49
superdanalthough given that they made me make it "more robust" there's more justification I guess15:49
*** pradk has quit IRC15:49
mdboothsuperdan: I'm assuming save_and_reraise() logs any exception it swallows, right?15:49
* mdbooth shouldn't assume things15:49
superdanmdbooth: no I don't think so15:50
superdanthat's kindof the point thought,15:50
superdanwhich is to assume any other exceptions are incidental15:50
mdboothThat's evil15:50
superdanum, okay :)15:51
mdboothI understand wanting to re-raise the original exception, but not to log the swallowed exception seems... harmful15:52
*** prateek has quit IRC15:53
superdanso, if libvirt started the instance, it failed immediately, and then the next thing we did failed, we should log that we couldn't stop the domain because it's not running along with the real error?15:53
superdanI think the point is to keep the noise in the logs down to as much of the original case as possible,15:53
superdanwhich I surely appreciate whilst trying to find the root cause of a reported issue15:53
*** prateek has joined #openstack-nova15:54
*** prateek has quit IRC15:54
mdboothMeh, I've cursed a lack of logs far more often than an abundance of logs :)15:54
*** pece has joined #openstack-nova15:54
superdanyeah, ask the ops how they feel about that :)15:54
mriedem_meetingi agree with dan, see https://review.openstack.org/#/c/408771/15:54
*** mriedem_meeting is now known as mriedem15:54
*** catintheroof has joined #openstack-nova15:54
mriedemwas debugging a gate failure yesterday and was getting tripped up in the logs by ^15:54
mriedemwhich was incidental cleanup noise15:54
sdague_mriedem: we even got a bug registered today for that same issue15:55
mriedemsdague_: no shit15:55
mriedemsdague_: open bugs down to 15, i see you've been busy15:55
sdague_mriedem: https://bugs.launchpad.net/nova/+bug/164876715:56
openstackLaunchpad bug 1648767 in OpenStack Compute (nova) "Stack trace when tring to delete an inexisting img" [Undecided,Fix released]15:56
mriedemwow coincidental15:56
*** tesseract- has quit IRC15:56
sdague_mriedem: yeh, it took me a second, because I was like "wait, I thought I just +2ed that patch"15:56
sdague_and... turns out I did :)15:56
*** zenoway has quit IRC15:57
*** pradk- is now known as pradk15:58
*** zenoway has joined #openstack-nova15:58
*** paul-carlton2 has joined #openstack-nova15:58
*** kaisers_ has joined #openstack-nova15:58
*** hongbin has joined #openstack-nova15:59
*** jwcroppe has joined #openstack-nova16:00
mriedemsuperdan: +2, nits inline if you respin16:00
superdanmriedem: thanks16:01
mriedemsuperdan: no bug for this?16:01
mriedemwe're going to want to track this against stable and backport it right?16:01
superdanmriedem: it was a downstream bug and very confused from start to finish16:02
superdanmriedem: yeah, I want to backport it,16:02
superdanbut at this point the bug report will be "this could happen"16:02
superdanbut I can surely create one16:02
mriedemconsider it my xmas gift16:02
*** zenoway has quit IRC16:02
superdanI know how you love your paperwork16:02
mriedemfrom you to me i mean16:02
mriedemi do16:02
*** kaisers_ has quit IRC16:02
mdboothWhile we're riffing on small fixes in the libvirt driver: https://review.openstack.org/#/c/408152/ already has +216:03
mdboothftersin would be happy about that too :)16:03
mriedemthis isn't jazz16:03
mdboothIt might be16:04
sdague_superdan: can you comment in on this bug - https://bugs.launchpad.net/nova/+bug/1632173 - it's description is an email dump between you and the reporter - and trying to sift through whether you think there is an issue to be fixed here16:05
openstackLaunchpad bug 1632173 in OpenStack Compute (nova) "migrate_flavor_data doesn't flavor migrate meta data of VMs spawned during upgrade" [Undecided,Incomplete]16:05
*** rcernin has quit IRC16:07
sdague_mriedem: is this still a thing - https://bugs.launchpad.net/nova/+bug/1175464 ?16:08
openstackLaunchpad bug 1175464 in OpenStack Compute (nova) "should not add default security group to quantum unless api-request had it" [High,In progress] - Assigned to sahid (sahid-ferdjaoui)16:08
mriedemlyarwood: would be good to get this into newton - cellsv2 will fail on a marker query always w/o it https://review.openstack.org/#/c/407791/16:08
sdague_actually sahid is assigned, sahid is that still something we're trying to fix?16:08
mriedemsdague_: that's fixed16:08
mriedemlet me dig up the details16:08
sdague_ok, well still marked as in progress16:09
mriedemb/c https://review.openstack.org/#/c/306470/ was using related-bug16:09
kashyapsuperdan: Are you writing a bug for this, as mriedem requested?  https://review.openstack.org/#/c/408806/  If not, can do that.16:09
*** ralonsoh has quit IRC16:09
mriedemkashyap: i think he's on it16:09
kashyapOkido16:10
sdague_mriedem: ok, fixed released it is16:10
mriedemsdague_: i just updated it too :)16:10
superdankashyap: just did16:10
*** armax has joined #openstack-nova16:11
mriedemsdague_: you too https://review.openstack.org/#/c/407791/16:11
kashyapsuperdan: You suauve up vote16:11
kashyap316:11
kashyapdown vote16:11
kashyap"Pacesetter"16:11
kashyapErr, what was that gibberish16:11
*** danpawlik has quit IRC16:11
sdague_lyarwood: what's the state of - https://bugs.launchpad.net/nova/+bug/1419577 - as the patch you owned is now abandoned16:11
openstackLaunchpad bug 1419577 in OpenStack Compute (nova) "when live-migrate failed, lun-id couldn't be rollback in havana" [High,In progress] - Assigned to Lee Yarwood (lyarwood)16:11
kashyapSorry, spurious copy / paste from elsewhere16:11
* kashyap should stop switching windows too fast on a Friday evening16:12
mriedemsdague_: i just removed lee from that one and marked it confirmed16:12
sdague_mriedem: do we have any more robust testing for marker going across all these databases?16:12
openstackgerritSTEW TY proposed openstack/nova: Transform instance.resize_confirm notification  https://review.openstack.org/39065416:12
mriedemsdague_: that might just eventually get fixed with the new vol attach flow at some point16:12
*** ihrachys has quit IRC16:12
mriedemsdague_: i was going to write a functional test but didn't have all of the new fixtures yet for setting up a cells v2 env16:13
sdague_mriedem: ok16:13
mriedemsdague_: and that was blocking novaclient in the gate, so i wanted to get it in16:13
*** FL1SK has quit IRC16:13
sdague_mriedem: ok, sure, that's fine. I was just thinking about stressing that marker logic now that it's not a simple single db16:13
mriedemsdague_: i agree, plus with build requests16:14
mriedemb/c if we have the build request but not the instance yet, the marker has to come from there when paging instances16:14
sdague_ok on - https://bugs.launchpad.net/nova/+bug/1419577 - high/in progress is probably the wrong answer if this is "might eventually get fixed with unrelated code"16:14
openstackLaunchpad bug 1419577 in OpenStack Compute (nova) "when live-migrate failed, lun-id couldn't be rollback in havana" [High,Confirmed]16:14
*** annegentle has joined #openstack-nova16:14
*** ijw has joined #openstack-nova16:14
mriedemi marked it confirmed, but it doesn't need to be high16:15
*** thorst__ has joined #openstack-nova16:15
mriedemit's super latent and has been around forever16:15
toabctlmriedem, could you have another look at https://review.openstack.org/#/c/402318/ please? anthonyper already had a look16:16
*** FL1SK has joined #openstack-nova16:16
cfriesenquick question...why is VIR_MIGRATE_TUNNELLED not compatible with block migration?16:16
*** thorst_ has quit IRC16:17
lyarwoodsdague_: https://review.openstack.org/#/c/338929/ is the WIP I currently have for bug 1419577, iirc it ended up being blocked by some work paul-carlton2 was doing. I'll circle back to it on Monday as I'm on the road at the moment.16:17
openstackbug 1419577 in OpenStack Compute (nova) "when live-migrate failed, lun-id couldn't be rollback in havana" [Medium,Confirmed] https://launchpad.net/bugs/141957716:17
kashyapcfriesen: Nope. (It has a long, but fraught history)16:17
openstackgerritDan Smith proposed openstack/nova: Cleanup after any failed libvirt spawn  https://review.openstack.org/40880616:18
cfriesenkashyap: okay, so I'm using VIR_MIGRATE_TUNNELLED with block migration in Mitaka....are there docs saying what I need to set up for Newton?16:18
kashyapcfriesen: Sure, there was a release note16:18
kashyapLet me find it16:18
kashyapcfriesen: Meanwhile, you can read the 'fun' history here (in the comments of this now-abandoned review) - https://review.openstack.org/#/c/171098/16:19
mriedemcfriesen: docs.openstack.org/releasenotes/nova/unreleased.html16:19
mriedemThe libvirt driver now has a live_migration_tunnelled configuration  option which should be used where the VIR_MIGRATE_TUNNELLED flag would  previously have been set or unset in the live_migration_flag and  block_migration_flag configuration options.16:19
kashyapYep16:19
*** sudipto_ has quit IRC16:19
*** sudipto has quit IRC16:19
kashyapMatt is much faster :-)16:19
mriedemcfriesen: grep for live_migration_tunnelled in there, lots of hits16:19
*** mnestratov has quit IRC16:19
*** sudipto has joined #openstack-nova16:19
cfriesenmriedem: kashyap: thanks, will take a look16:20
*** sudipto_ has joined #openstack-nova16:20
*** mnestratov has joined #openstack-nova16:20
*** matrohon has quit IRC16:21
*** ralonsoh has joined #openstack-nova16:21
sdague_bauwser / cdent / jaypipes - https://review.openstack.org/#/c/390984/5 - MemFilter is apparently not working right atm?16:22
kashyapcfriesen: That's the new tri-state flag that handles three cases (QEMU native encryption; libvirt-tunnelled; none - 'if not specified, choose the best of the previous two) -- https://review.openstack.org/#/c/263434/16:23
sfinucanwznoinsk: umm, why did this +1 the patch when the test failed? https://review.openstack.org/#/c/409185/16:24
wznoinskresult is assumed SUCCESS unless any of the voting jobs fail16:25
cdentsdague_: I've not seen that before, catching up.16:25
wznoinsksfinucan, ^16:25
mriedemsfinucan: it's non-voting16:25
mriedembut that is confusing...16:25
sdague_cdent: it looks like a cells v1 fix caused some fallout with non cells16:25
*** bvanhav_ has joined #openstack-nova16:25
sfinucanwznoinsk, mriedem: I would have expected 0 for non-voting, myself16:25
mriedemyeah i'd prefer no vote on that16:26
cfriesenkashyap: mriedem: based on the final comments at https://review.openstack.org/#/c/171098/ it seems like libvirt falls back to legacy mode when using TUNNELLED with block migration...so why does the config option comment say it's not compatible?  is it deprecated in libvirt or something?16:26
*** bvanhav has quit IRC16:27
sdague_lyarwood: this fixed in Master - https://bugs.launchpad.net/nova/+bug/1633518 ?16:27
openstackLaunchpad bug 1633518 in os-brick "The passphrase used to encrypt or decrypt volumes was mangled prior to Newton" [High,In progress] - Assigned to Lee Yarwood (lyarwood)16:27
mriedemsdague_: partial https://review.openstack.org/#/c/386670/16:28
kashyapcfriesen: That's only the case _if_ the native QEMU encryption is not available16:28
kashyapcfriesen: Look at this comment from danpb (from the merged patch) - https://review.openstack.org/#/c/263434/4/nova/virt/libvirt/driver.py16:28
mriedemkashyap: qemu native tls might be close to libvirt/qemu release, but that's going to be years out for nova/distros16:29
mriedemi remember digging into some of this tunneled stuff at one point and got a bit lost, and it seemed some of it was half baked16:30
mriedemwrt how the new config options worked16:30
mriedemi've since lost all of that context16:30
lyarwoodsdague_ / mriedem ; yup that was only partial as it lacked a releasenote and John wanted to get the fix in first16:30
cfriesenso the qemu native tls isn't generally available yet?16:31
mriedemcfriesen: kashyap: i was looking at something related to this thread http://lists.openstack.org/pipermail/openstack-dev/2016-January/083275.html16:31
mriedemcfriesen: don't know, but it's certainly not in what we test with ubuntu trusty16:31
kashyapYeah, that was the thread I wanted to pint, too16:31
*** kragniz has quit IRC16:32
cfriesenlooks like maybe it's in qemu 2.7.0 based on dpb's post at https://www.berrange.com/posts/2016/08/16/improving-qemu-security-part-7-tls-support-for-migration/16:33
*** sandanar has quit IRC16:33
*** kragniz has joined #openstack-nova16:33
kashyapYep, was looking for that16:34
cdentsdague_: so the memory_mb_used thing makes sense (and also more sense than the ambiguous free_ram_mb thing) but a) yet another field b) it is difficult to comment on the implications of this in the bigger picture. I think jaypipes and bauwser will be able to say more.16:35
*** ljxiash has joined #openstack-nova16:37
sfinucanwznoinsk: Looks like you need to change '[validation] run_validation=True' in tempest.conf to enable that rework16:37
cfriesenmriedem: kashyap: looks like the config comment may be a bit simplistic...live_migration_tunnelled=True seems to still be okay with block migration as long as you don't also have cinder volumes.16:37
*** siva_krish has joined #openstack-nova16:38
kashyapcfriesen: Haven't tested w/ Cinder volmes myself -- https://bugs.launchpad.net/nova/+bug/139899916:39
openstackLaunchpad bug 1398999 in nova (Ubuntu Trusty) "Block migrate with attached volumes copies volumes to themselves" [High,Triaged]16:39
sdague_lyarwood: didn't we land the release note?16:39
sdague_sfinucan: is this still in progress for real - https://bugs.launchpad.net/nova/+bug/1607996 ?16:39
openstackLaunchpad bug 1607996 in OpenStack Compute (nova) "Live migration does not update numa hugepages info in xml" [High,In progress] - Assigned to Stephen Finucane (stephenfinucane)16:39
cfriesenkashyap: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L567916:40
lyarwoodsdague_: we did but outside of that review, iirc that's why John asked if I could mark it as partial16:40
cfriesensdague_: I hope so, it's a real issue16:40
wznoinsksfinucan, I usually see a skip when test can't run because of the validation is false in tempest.conf, or is it a diff patchset you're referring to?16:40
kashyapcfriesen: Bingo16:40
sfinucansdague_: fwict it's a variation of https://bugs.launchpad.net/nova/+bug/141766716:41
openstackLaunchpad bug 1417667 in OpenStack Compute (nova) "migration/evacuation/rebuild/resize of instance with dedicated cpus needs to recalculate cpus on destination" [Medium,In progress] - Assigned to Pawel Koniszewski (pawel-koniszewski)16:41
sdague_sfinucan: should we merge the bugs?16:41
sfinucansdague_: Yeah, I think so16:42
sdague_sfinucan: ok, which one should we keep, and which is the dup?16:42
*** ljxiash has quit IRC16:42
sfinucansdague_: The 141 one one. Needs to be renamed too. I'll do it now16:42
sdague_sfinucan: thank you16:42
cfriesensfinucan: sdague_: going to change it to "needs to recalculate numa_topology" or something like that?16:43
*** jwcroppe has quit IRC16:43
sdague_lyarwood: ok, so the release note is actually merged now though, right, which means - https://bugs.launchpad.net/nova/+bug/1607996 is fix released?16:43
openstackLaunchpad bug 1417667 in OpenStack Compute (nova) "duplicate for #1607996 migration/evacuation/rebuild/resize of instance with NUMA topology needs to recalculate NUMA topology" [Medium,In progress] - Assigned to Pawel Koniszewski (pawel-koniszewski)16:43
sfinucancfriesen: took the words out of my mouth ;)16:43
*** ijw has quit IRC16:44
cfriesensdague_: as far as I know the live migration case is still in review16:44
*** cmartin has joined #openstack-nova16:44
lyarwoodsdague_: correct16:44
*** raunak has joined #openstack-nova16:45
cfriesensdague_: https://review.openstack.org/#/c/28674416:45
lyarwoodsdague_: wait, that's a different bug :)16:45
mriedemsfinucan: while you're around, you were +1 on all of these i think https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/use-neutron-by-default16:45
mriedemsfinucan: now that you have new +2 powers, want to upgrade the +1 to +2?16:45
mriedemthe -1s are all busted 3rd party ci16:45
sfinucanmriedem: Sure, why not16:45
lyarwoodsdague_: https://bugs.launchpad.net/nova/+bug/1633518 is fixed16:45
openstackLaunchpad bug 1633518 in os-brick "The passphrase used to encrypt or decrypt volumes was mangled prior to Newton" [High,In progress] - Assigned to Lee Yarwood (lyarwood)16:45
sfinucanI was going to say - intel-nfv-ci was not happy16:45
sdague_lyarwood: can you make it Fix Released then, instead of In Progress16:46
cfriesensfinucan: sdague_: I'd love to see the live-migration stuff go in, we've got a bunch of fixes to those internally that we'd like to push.16:46
lyarwoodsdague_: yup doing that now, thanks for the os-brick stable/newton review btw16:46
sdague_lyarwood: no prob16:46
sfinucanwznoinsk: I didn't include the flag in the tests. Added now and pushed, so if it skips that's the issues16:46
sfinucan*issue16:46
wznoinsksfinucan, roger that16:47
sfinucan(or so I hope - it's practically identical to the upstream tempest test otherwise :/)16:47
*** browne has joined #openstack-nova16:47
sfinucancfriesen: Aye, but that patch is a nightmare to review/rework and I've no idea how to break it up :/ Was hoping sahid would take a shot at it again, heh16:48
cmartinmdbooth: Hey we had a conversation about this bug ( https://bugs.launchpad.net/nova/+bug/1501808 ) a long time ago was wondering if you were around16:48
openstackLaunchpad bug 1501808 in OpenStack Compute (nova) "Enabling soft-deletes opens a DOS on compute hosts" [High,In progress] - Assigned to Chris Martin (cm876n)16:48
mdboothcmartin: Yeah, wassup?16:48
*** clenimar has joined #openstack-nova16:49
cfriesensfinucan: understood.  it's frustrating though since it's so close...we've been complaining about it since cpu pinning first went in. :)16:49
cmartinmdbooth: I pushed a patch up for it, per our discussion which was to just remove rolling back of quotas from soft delete and save it until the VMs are actually deleted. However I haven't got any reviews on it in like 2 months and was wondering if maybe you could take a look?16:50
mdboothcmartin: Sure.16:50
cmartinmdbooth: thanks!16:50
wznoinsksfinucan, re intel-nfv-ci multinode not happy, it doesn't look like the CI itself problem so maybe codes16:50
wznoinsksfinucan, on those patches from https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/use-neutron-by-default I mean16:51
mriedemwznoinsk: those are all unit test changes16:51
mriedemso not sure why intel ci would care16:51
*** paul-carlton2 has quit IRC16:52
wznoinskmriedem, ah yes, ignore me so16:52
mriedemexcept i guess the bottom change https://review.openstack.org/#/c/396359/16:52
mriedembut not sure how that would cause issues16:52
mriedemsince i'm assuming the tests aren't passing in a networks='none16:52
mriedemwith microversion 2.3716:52
cmartinmriedem: If I wanted to work on this https://review.openstack.org/#/c/340660/ would you suggest uploading a new patch/commit or unabandoning it and pushing up to that one?16:55
mriedemcmartin: probably that one, the bp reference needs to be updated though16:55
mriedemi can restore if you want16:55
cmartinmriedem: yea restore that one and this one https://review.openstack.org/#/c/334841/16:56
cmartinmriedem: i'll update the topic16:56
mriedemdone and done16:56
cmartinmriedem: thanks16:56
sahidmriedem: sdague_, sorry just noticed your pings16:56
sahidsfinucan: which one ?16:56
sdague_sahid: no worries, just trying to get a handle on the bug tracker16:56
sahid:)16:56
sfinucansahid: This beauty :) https://review.openstack.org/#/c/28674416:58
openstackgerritSarafraj Singh proposed openstack/nova: conf: add warning for vm's max delete attempts  https://review.openstack.org/40827416:59
*** rcernin has joined #openstack-nova16:59
*** kaisers_ has joined #openstack-nova16:59
sdague_mriedem et al. I thought https://bugs.launchpad.net/nova/+bug/1501808 was basically Working as Designed17:00
openstackLaunchpad bug 1501808 in OpenStack Compute (nova) "Enabling soft-deletes opens a DOS on compute hosts" [High,In progress] - Assigned to Chris Martin (cm876n)17:00
sdague_I'm tempted to just mark that as Won't Fix17:00
*** claudiub|2 has joined #openstack-nova17:01
openstackgerritChristopher Martin proposed openstack/nova: Remove mox from unit/compute/test_compute_api.py (1)  https://review.openstack.org/33484117:01
*** Oku_OS is now known as Oku_OS-away17:01
sahidsfinucan: ah yes... i'm afraid but i can't really review it tonight and next week i'm in PTO :/17:02
*** asettle has quit IRC17:02
sfinucansahid: That's fine :) I was hoping to avoid having to dive in myself but it looks inevitable now that I've a machine again :'(17:02
*** fragatina has quit IRC17:02
openstackgerritsahid proposed openstack/nova: libvirt: update the active migrations ds to support migration result  https://review.openstack.org/40926017:02
openstackgerritsahid proposed openstack/nova: (WIP) libvirt: pass to live migration monitor the API result  https://review.openstack.org/40926117:02
*** asettle has joined #openstack-nova17:03
*** fragatina has joined #openstack-nova17:03
*** pcaruana has quit IRC17:03
raj_singhjroll: Can you plz take a look at https://review.openstack.org/#/c/405597/ to make sure we are doing the right thing.17:03
*** kaisers_ has quit IRC17:03
openstackgerritChristopher Martin proposed openstack/nova: Remove mox from unit/compute/test_compute_api.py(2)  https://review.openstack.org/34066017:04
*** claudiub has quit IRC17:04
mriedembauwser: cdent: what's the use case for listing resource providers with in:uuid=1,uuid=2,uuid=3, etc?17:05
mriedemsame for name17:05
mriedemas uuid/name are unique17:05
*** asettle__ has joined #openstack-nova17:05
mriedemif you know the uuid, you can just GET the RP via the uuid17:05
cdentmriedem: I don't know. I had that question when uuid= (without the in:) was requested17:05
mriedemyeah.... GET /resource_providers/?uuid=unique thing...17:05
cdentwith in: at least you have the option of a getting a list of them17:06
cdentwhich is potentially an optimization of some kind (instead of doing many GETs)17:06
* cdent shrugs?17:06
mriedemyeah i guess17:06
*** whenry has quit IRC17:06
cfriesensfinucan: are the host CPUs that are "reserved" by the ISOLATE policy tracked anywhere?  I'm wondering if they shouldn't count towards "vcpus_used".17:07
*** asettle has quit IRC17:07
openstackgerritRoman Podoliaka proposed openstack/nova-specs: WIP: Ironic: add support for instance RAID configuration  https://review.openstack.org/40815117:07
openstackgerritMaciej Szankin proposed openstack/nova: conf: remove deprecated nova quota options  https://review.openstack.org/39570617:08
mriedemoh i guess it's name=in:foo,bar,baz17:08
sfinucancfriesen: They are17:08
sfinucanSee here https://github.com/openstack/nova/blob/5158ca7d/nova/objects/numa.py#L98-L12017:09
*** derekh has quit IRC17:09
*** Zer0Byte__ has joined #openstack-nova17:09
*** Zer0Byte__ has quit IRC17:09
*** asettle__ has quit IRC17:09
cfriesensfinucan: right, but do they count towards the "used cpus" count on the hypervisor?17:10
cfriesenoh, I see what you mean.  they show up as pinned17:10
mriedemcdent: i guess it's at least similar to how you can filter and list images in glance v2 with in: http://developer.openstack.org/api-ref/image/v2/?expanded=show-images-detail#show-images17:10
mriedemso consistency is good i guess17:10
mriedemGET /v2/images?status=in:saving,queued17:10
cdentmriedem: If I recall correctly the original impetus that bauwser had for wanting uuid=in:<uuid1>,<uuid2> was because it was going to be used in rel links in the representation returned from /claim or /request17:11
cdents/the original/an/17:12
mriedemwhich isn't a thing right now...17:12
mriedemwhat if my rp name has commas in it17:12
mriedem?17:12
mriedemthis will be hosted17:12
mriedem*hosed17:12
openstackgerritMaciej Szankin proposed openstack/nova: config options: improve libvirt imagecache  https://review.openstack.org/37849717:13
*** ducttape_ has joined #openstack-nova17:13
cdentmriedem: I think that's one of the reasons for sdague_'s comments about validating the query values17:14
cdentand hosed is a bit strong. it's more like: you won't be able to get it this way, so don't do that17:15
cdentwhich is still not ideal, but not quite the same17:15
mriedemgranted putting a comma in an rp name seems dumb, but we don't restrict that17:16
mriedemPOST_RESOURCE_PROVIDER_SCHEMA just allows 'string' for the name17:16
*** GB21 has quit IRC17:16
mriedemrp.name = 'this is my shared storage cluster, dawg!'17:16
mriedemGET /resource_providers?name=in:"this is my shared storage cluster, dawg!"17:17
figleafmriedem: little bobby tables is a resource provider17:17
mriedemwon't return anything17:17
cdentmriedem: yeah, that's what I mean by not ideal but not hosed17:17
cdenti'm totally in agreement with you17:17
*** nic has joined #openstack-nova17:18
figleafan ampersand in the name would be fun17:18
mriedemso i guess in that case, you just can't use in with names that have commas in them17:18
mriedemyou'd have to 'list' those one by one17:18
*** ljxiash has joined #openstack-nova17:18
mriedemi also want to complain about not having api ref for this query language17:19
cdentthere's code somewhere that allows quoted comma separate multi-values in query params (I vaguely recall something from some api-wg discussion)17:19
cdentbut we've gone for simplest thing, for now, I guess17:19
cdentmriedem: yeah, totes on the docs, but there's only so much time in the day, week year17:20
openstackgerritChristopher Martin proposed openstack/nova: Remove mox from unit/compute/test_compute_api.py(2)  https://review.openstack.org/34066017:20
mriedemwhat about the month?17:20
mriedemplenty of time in the month17:20
* cdent shakes tiny fist17:20
mriedemwhy do we hardcode uuid and name in there when we have objects.ResourceProviderList.allowed_filters ?17:20
mdboothcmartin: I expect you have more context than me on quotas, so please correct my misunderstandings.17:20
cdentallowed_filters is removed somewhere in that stack17:21
mriedemyet it's used up top invalid_filters = passed_filters - allowed_filters17:21
mdboothcmartin: My other concern, btw, is that this is a behaviour change. It feels like it would need a release note.17:21
mriedemcdent: yeah i guess here https://review.openstack.org/#/c/392569/15/nova/objects/resource_provider.py17:21
mdboothcmartin: However, to the best of my limited understanding, I think that's the right approach.17:21
cdent but with regards to why any of it was done the way it was, I'm not sure I can answer any of that. that would be some combo of jaypipes and bauwser17:21
*** yamahata has quit IRC17:21
cmartinmdbooth: yea it would, I just have never done anything that required a release note before so I was unsure exactly how to do it17:21
cdentas much as I wish we were all working from the same page, it's not really the case17:22
mdboothcmartin: http://docs.openstack.org/developer/reno/17:22
*** ihrachys has joined #openstack-nova17:22
mdboothIt's pretty easy these days17:22
cmartinmdbooth: thanks for the link I'll add that in, and thanks for the review :)17:22
mriedemwell, i'm going to just let myself get distracted with sandwiches for awhile17:22
cdentmriedem: now you've made me hungry17:23
*** ljxiash has quit IRC17:23
cdentand also worried17:23
*** nicolasbock has quit IRC17:23
*** nicolasbock has joined #openstack-nova17:23
* mdbooth has a Christmas party to attend. Happy weekend, all!17:23
cdentmriedem: before 'in:' came along the intent was going to be to use much more robust jsonschema (not just string) validation of the query params, but 'in:' blows that out of the water17:24
*** klkumar has quit IRC17:25
cdentwe could kill that (but we still need it for member_of: https://review.openstack.org/#/c/407742/ ) or we could write some utility methods that dealt with things in a more tidy fashion. there's room to do so.17:25
*** nherciu_ has quit IRC17:25
* cdent makes a note17:25
figleafmriedem: the API WG stuff on in: is here - https://specs.openstack.org/openstack/api-wg/guidelines/pagination_filter_sort.html17:26
cdentoh her dere figleaf,did you see I rebased your thing?17:28
figleafyeah - thanks17:29
*** Swami has joined #openstack-nova17:29
*** nherciu has joined #openstack-nova17:31
*** cmartin has quit IRC17:32
*** nherciu_ has joined #openstack-nova17:32
mriedemi think i've just lost all motivation to really do anything else today despite the long list of things i should be doing17:33
*** jwcroppe has joined #openstack-nova17:34
*** nherciu has quit IRC17:36
cdentno sandwiches for you17:36
figleafmriedem: https://s-media-cache-ak0.pinimg.com/736x/ba/ba/60/baba60c6c326a7b1338a798b6509a445.jpg17:36
figleafs/night/afternoon17:36
mriedemnot applicable17:37
openstackgerritjichenjc proposed openstack/nova: network id is uuid instead of id  https://review.openstack.org/40927417:39
*** dinobot_ has quit IRC17:39
mriedemsuperdan: so on this build request busted data thing when upgrading from mitaka https://review.openstack.org/#/c/408727/17:41
mriedemi know how we got there, and why it fails,17:41
mriedemi'm trying to decide if we do a db migration that just queries for these busted build requests and deletes them (with the schema migrations), or do an online data migration thing with nova-manage17:42
mriedembasically (1) query all build requests where instance_uuid=None, (2) delete those mofos17:42
superdanmriedem: build requests are intended to be short-lived, so just nuking them seems like a reasonable plan, but doing it on/inline also seems nice17:43
superdanbut either would be cool with me I think17:43
mriedemok yeah i don't have a strong preference either way, and both should be a small data set17:43
*** bvanhav_ has quit IRC17:44
mriedembackporting the online migration might be less of a hassle17:44
*** bvanhav_ has joined #openstack-nova17:44
superdandefinitely17:45
superdanyeah, good point17:45
*** Jeffrey4l has quit IRC17:45
mriedemalright, looks like i found my muse for this afternoon17:46
openstackgerritJulian proposed openstack/nova: Transform keypair.create notification  https://review.openstack.org/40868917:46
*** alaski is now known as lascii17:46
*** mnestratov has quit IRC17:47
mriedemlascii: o/17:47
lasciio/17:47
*** romainh has quit IRC17:48
lasciiI like the online migration solution fwiw17:49
*** Zer0Byte__ has joined #openstack-nova17:50
mriedemlascii: ok, i'll do it for you :)17:50
lasciiheh17:50
mriedemi have yet to write something that uses the wonky engine facade reader context stuff anyway so i need to brush up17:50
*** siva_krish has quit IRC17:51
*** rcernin has quit IRC17:52
*** rcernin has joined #openstack-nova17:52
superdanmriedem: I will totally sit on my lazy ass while you do that and then review it when you're done17:54
*** ijw has joined #openstack-nova17:55
*** sahid has quit IRC17:55
mriedemsuperdan: you could check out my comments on https://review.openstack.org/#/c/406380/ instead :)17:55
*** slaweq_ has quit IRC17:55
mriedemyou said you would yesterday morning but decided customer paying your salary were more important17:55
superdanyeah I need to get back to those comments17:55
superdanheh17:55
mriedem*customers17:55
*** larainema has quit IRC17:56
*** slaweq_ has joined #openstack-nova17:57
*** prateek has joined #openstack-nova17:58
*** prateek has quit IRC17:58
*** bvanhav_ has quit IRC17:59
openstackgerritint32bit proposed openstack/python-novaclient: Add paging support in server groups list  https://review.openstack.org/40928217:59
*** ijw has quit IRC17:59
*** kaisers_ has joined #openstack-nova18:00
*** yamahata has joined #openstack-nova18:04
*** kaisers_ has quit IRC18:04
*** GB21 has joined #openstack-nova18:05
*** andymaier has quit IRC18:09
*** stvnoyes1 has joined #openstack-nova18:10
*** ijw has joined #openstack-nova18:11
bauwserp!spy18:12
*** slaweq_ has quit IRC18:13
*** stvnoyes has quit IRC18:13
bauwsersorry folks, I'm AFK till tonight around 8pmUTC (Christmas tree relationship)18:13
*** jgriffith_AutoAw is now known as jgriffith18:14
*** lpetrut has quit IRC18:15
superdantree hugger?18:15
*** tbachman_ has joined #openstack-nova18:16
*** tbachman has quit IRC18:16
*** tbachman_ is now known as tbachman18:16
*** sudipto has quit IRC18:17
*** sudipto_ has quit IRC18:17
*** rcernin has quit IRC18:17
*** bvanhav_ has joined #openstack-nova18:17
openstackgerritSujitha proposed openstack/nova: Adding tags to InstancePayload  https://review.openstack.org/40722818:21
*** asettle has joined #openstack-nova18:24
openstackgerritDan Smith proposed openstack/nova: Make servers api view load instance fault from proper cell  https://review.openstack.org/40723818:25
openstackgerritDan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method  https://review.openstack.org/36755718:25
openstackgerritDan Smith proposed openstack/nova: WIP Multicell support for instance listing  https://review.openstack.org/39677518:25
openstackgerritDan Smith proposed openstack/nova: Setup CellsV2 environment in base test  https://review.openstack.org/40638018:25
openstackgerritDan Smith proposed openstack/nova: Add SecurityGroup.identifier to prefer uuid over name  https://review.openstack.org/40626018:25
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937918:25
*** asettle has quit IRC18:26
*** asettle has joined #openstack-nova18:26
openstackgerritSujitha proposed openstack/nova: Do not merge: Add post hook script to run tempest with old n-api  https://review.openstack.org/40139518:30
*** dharinic is now known as dharinic|lunch18:31
*** harlowja has joined #openstack-nova18:32
*** kaisers_ has joined #openstack-nova18:34
*** harlowja_ has joined #openstack-nova18:34
*** harlowja has quit IRC18:35
*** GB21 has quit IRC18:36
mriedemmelwitt: +2 on https://review.openstack.org/#/c/406380/ if you want to get it18:37
mriedemriddle me this (someone):18:37
mriedemfilter_by(api_models.BuildRequest.instance_uuid==null()).\18:37
mriedemTypeError: filter_by() takes exactly 1 argument (2 given)18:37
superdanfilter_by takes kwargs right?18:38
*** Guest23718 has quit IRC18:38
superdanvs. filter18:38
mriedemsob18:38
openstackgerritChristopher Martin proposed openstack/nova: Remove mox from unit/compute/test_compute_api.py (1)  https://review.openstack.org/33484118:40
*** slaweq_ has joined #openstack-nova18:40
mriedemcool, it's working like a dream now18:42
*** whenry has joined #openstack-nova18:42
*** ihrachys has quit IRC18:43
superdanyour dreams are sad18:44
openstackgerritMatt Riedemann proposed openstack/nova: Provide an online data migration to cleanup orphaned build requests  https://review.openstack.org/40872718:46
mriedemlascii: superdan: melwitt: ^18:46
mriedemsuperdan: i did have a question about this a bit, i was basing it off the instance_group_obj.migrate_instance_groups_to_api_db migration,18:48
mriedemsuperdan: the migration returns total, deleted18:48
mriedembut it's limiting how many to get back based on count18:48
mriedemi thought this was supposed to be more like, query all (total unlimited), then delete up to count18:49
mriedemso you can say how many more you have to processs18:49
mriedeme.g. i start with 100, first patch returns 100, 50, then 50, 50, then 0, 0 (if you run the command with no args)18:50
mriedems/patch/batch/18:50
*** slaweq_ has quit IRC18:51
lasciiI would ignore count for this and just delete all matching rows. Then you aren't pulling any rows from the db.18:51
mriedemwell, i could change all() to delete() in _get_build_requests_with_no_instance_uuid18:52
mriedemand it will delete up to limit18:52
lasciiand there seems to be little chance of this affecting db performance while running, which is what the count parameter is intended to help with18:52
mriedemwhich is really all this is doing anyway, but less efficiently18:52
lasciiyeah, that would be my preference18:52
mriedemi think you can also get the result from delete() and return result.rowcount18:53
*** kaisers_ has quit IRC18:53
mriedemso the output is always going to be <= count18:53
mriedembut the items in the tuple would be the same value always18:53
*** haplo37_ has quit IRC18:55
*** haplo37_ has joined #openstack-nova18:56
mriedemah you can't use limit with bulk delete18:57
mriedemi think in that case we want to do it this way with honoring the limit18:58
mriedembecause if someone passes in max_count=5 and we delete more than that, they might consider it a bug18:58
*** haplo37_ has quit IRC18:59
*** haplo37_ has joined #openstack-nova18:59
mriedemlet's let someone else break the tie19:00
mriedemi'm gonna run some errands so will be out for awhile, check back on later19:02
*** mriedem has quit IRC19:03
openstackgerritSujitha proposed openstack/nova: Adding tags to InstancePayload  https://review.openstack.org/40722819:03
*** ijw has quit IRC19:04
*** eharney has quit IRC19:04
*** ijw has joined #openstack-nova19:05
*** Sukhdev has joined #openstack-nova19:09
*** ijw has quit IRC19:10
*** david-lyle_ is now known as david-lyle19:13
*** asettle has quit IRC19:16
*** asettle has joined #openstack-nova19:16
*** ralonsoh has quit IRC19:16
*** lpetrut has joined #openstack-nova19:19
*** asettle has quit IRC19:21
*** jgriffith is now known as jgriffith_AutoAw19:24
openstackgerritEd Leafe proposed openstack/nova: rt: use a single ResourceTracker object instance  https://review.openstack.org/39847319:24
figleafcdent: jaypipes: ^^ Fixed the formatting problem19:24
*** lpetrut has quit IRC19:25
cdentnice19:26
*** lpetrut has joined #openstack-nova19:26
*** ijw has joined #openstack-nova19:32
*** haplo37 has quit IRC19:37
*** jgriffith_AutoAw is now known as jgriffith19:40
*** haplo37 has joined #openstack-nova19:41
*** dharinic|lunch is now known as dharinic19:43
*** openstack has joined #openstack-nova19:45
*** fragatina has quit IRC19:46
*** ijw has quit IRC19:48
*** ijw has joined #openstack-nova19:49
*** ijw has quit IRC19:49
*** bkopilov has quit IRC19:51
*** fragatina has joined #openstack-nova19:53
*** tbachman has quit IRC19:53
*** ihrachys has joined #openstack-nova19:54
*** fragatina has quit IRC19:55
*** sdague__ has joined #openstack-nova19:56
*** lpetrut has quit IRC19:57
*** esberglu has quit IRC19:59
*** sdague__ has quit IRC19:59
*** esberglu has joined #openstack-nova19:59
*** esberglu has quit IRC20:04
*** masber has quit IRC20:07
*** masber has joined #openstack-nova20:07
*** esberglu_ has joined #openstack-nova20:09
*** erlon-airlong has quit IRC20:10
*** READ10 has quit IRC20:13
*** bvanhav_ has quit IRC20:15
*** eharney has joined #openstack-nova20:16
*** raunak has quit IRC20:16
*** ljxiash has joined #openstack-nova20:17
*** bvanhav_ has joined #openstack-nova20:17
*** thorst__ has quit IRC20:18
*** hwoarang has quit IRC20:18
*** thorst has joined #openstack-nova20:19
*** ljxiash has quit IRC20:21
*** hwoarang has joined #openstack-nova20:24
*** bvanhav_ has quit IRC20:26
*** thorst has quit IRC20:27
*** mriedem has joined #openstack-nova20:29
openstackgerritMatt Riedemann proposed openstack/nova: Provide an online data migration to cleanup orphaned build requests  https://review.openstack.org/40872720:33
*** rcernin has joined #openstack-nova20:38
*** rcernin has quit IRC20:38
*** rcernin has joined #openstack-nova20:38
*** kaisers_ has joined #openstack-nova20:41
*** thorst has joined #openstack-nova20:44
mriedemsdague_: here is a newish ceph failure https://bugs.launchpad.net/cinder/+bug/164888520:45
openstackLaunchpad bug 1648885 in Cinder "ceph: TestVolumeBootPattern.test_create_ebs_image_and_check_boot times out failing to delete volume due to snapshot is busy" [Undecided,New]20:45
*** kaisers_ has quit IRC20:45
mriedemmight not be new, just doesn't hit that often20:45
smcginnisBeen around for awhile, actually.20:46
smcginnisDue to the way ceph handles cleaning up on the backend.20:46
*** thorst has quit IRC20:47
*** ljxiash has joined #openstack-nova20:48
mriedemsmcginnis: is it worth doing a backoff retry on the snapshot delete?20:48
*** avolkov has quit IRC20:48
mriedemlike for 30 seconds or something?20:49
smcginnismriedem: From my understanding of how that works, that may be enough to get around it.20:49
mriedemonly retrying if it's busy really20:49
mriedembizzay20:49
*** ijw has joined #openstack-nova20:51
*** ljxiash has quit IRC20:52
sdague_mriedem: so, we currently have 141 open patches in gerrit that are Jenkins -1 and have not been touched in > 6 weeks20:56
sdague_you good with me doing abandon with description on those?20:56
mriedemsdague_: yeah20:57
*** masuberu has joined #openstack-nova20:58
mriedemmikal has been doing some of that sporadically too20:58
mriedemyeah == go ahead20:58
*** jamielennox|away is now known as jamielennox20:58
*** egarbade- has quit IRC20:58
*** ekhugen has quit IRC20:58
snetiHi all, Can some one help me with this: I'm working on multi api setup for zero downtime upgrade. I have n-api on both controller and compute nodes. Database is only on controller node. When I change the nova api endpoint to compute node, I get db connection errors in n-api: "Can't connect to MySQL server on u'127.0.0.1' ([Errno 111] ECONNREFUSED)")20:59
snetiI checked nova.conf on compute node. database and api_database connections both point to controller node. I'm not sure why it is trying to connect to MYSQL server on local host.20:59
*** masber has quit IRC20:59
sdague_mriedem: ok, away we go21:00
sdague_mriedem: I was noticing mikal doing that when I was going through the In Progress bug reports and finding a ton of them are dead patches21:01
*** egarbade- has joined #openstack-nova21:02
*** ijw has quit IRC21:02
*** ekhugen has joined #openstack-nova21:02
* mriedem prepares to curse you both for abandoning my precious changes21:03
*** ijw has joined #openstack-nova21:03
sdague_sfinucan: https://review.openstack.org/#/c/290706/ ... are you going to come back around to that issue?21:03
sdague_it's been a while, it still has your intel address on it :)21:04
*** raunak has joined #openstack-nova21:05
sdague_jaypipes: you still about?21:06
sdague_I had a complicated SQL question for you21:06
*** slaweq_ has joined #openstack-nova21:07
mriedemsuperdan: do you have any tricks up your sleeve to enforce that something isn't remotable? i.e. https://review.openstack.org/#/c/404279/321:10
superdanmriedem: not sure why we need to do that21:11
openstackgerritDiana Clarke proposed openstack/nova: Simple tenant usage pagination  https://review.openstack.org/38609321:11
mriedemsuperdan: b/c bauwser was asking for it21:11
superdanmriedem: yeah, I've already said to bauwser that I think this is much less of a deal than he's making it out to be :)21:11
superdanI'll comment to that effect21:11
mriedemok21:11
bauwsermriedem: superdan: I don't honestly think it's a problem21:12
bauwsermriedem: superdan: I just thought that given it looked like not an important change, that we could leave cdent adding the test21:13
mriedembauwser: you're supposed to be making sweet love to a fir tree21:13
superdanbauwser: yeah, just not worth the hassle, IMHO21:13
*** ijw has quit IRC21:13
bauwsermriedem: I'm done with that :p21:14
bauwsermriedem: just needed 2 hours for that21:14
mriedembauwser: wow21:14
mriedemimpressive21:14
mriedemi hope you were gentle21:14
*** thorst_ has joined #openstack-nova21:14
bauwseryeah, because I had to go finding light strings :p21:15
mriedemi should have shipped over all of our random boxes of lights21:16
mriedemthey seem to multiply21:16
bauwserI still need 1 hour for putting some of them outside :)21:17
bauwseranyway21:18
bauwsersuperdan: +1d21:18
*** ijw has joined #openstack-nova21:18
openstackgerritDan Smith proposed openstack/nova: Fix crashing during guest config with pci_devices=None  https://review.openstack.org/40933821:18
*** esberglu_ has quit IRC21:20
*** esberglu has joined #openstack-nova21:21
*** ijw has quit IRC21:21
*** liverpooler has quit IRC21:22
*** ijw has joined #openstack-nova21:23
*** esberglu has quit IRC21:25
*** slaweq_ has quit IRC21:27
*** slaweq_ has joined #openstack-nova21:27
*** esberglu has joined #openstack-nova21:28
openstackgerritDiana Clarke proposed openstack/nova: Simple tenant usage pagination  https://review.openstack.org/38609321:30
*** esberglu_ has joined #openstack-nova21:31
*** kaisers_ has joined #openstack-nova21:32
*** esberglu has quit IRC21:33
*** smatzek has quit IRC21:33
*** esberglu_ has quit IRC21:36
*** esberglu has joined #openstack-nova21:37
openstackgerritSean Dague proposed openstack/nova: Refactor REGEX filters to eliminate 500 errors  https://review.openstack.org/28219021:37
*** esberglu has quit IRC21:42
*** slaweq_ has quit IRC21:43
*** ihrachys has quit IRC21:44
*** mdrabe has quit IRC21:46
*** claudiub|2 has quit IRC21:49
*** slaweq_ has joined #openstack-nova21:50
*** dave-mccowan has quit IRC21:50
openstackgerritChris Dent proposed openstack/nova: [WIP] Structure for simply managing placement-api-ref  https://review.openstack.org/40934021:52
*** esberglu has joined #openstack-nova21:52
cdentmriedem: ^ that doesn't have any real content yet, but has the tooling to make the content easier to make21:53
*** ijw has quit IRC21:56
*** ihrachys has joined #openstack-nova21:58
*** ijw has joined #openstack-nova21:58
*** kylek3h_away has quit IRC21:59
*** fragatina has joined #openstack-nova22:01
*** slaweq_ has quit IRC22:01
*** rcernin has quit IRC22:04
*** thorst_ has quit IRC22:04
*** jamielennox is now known as jamielennox|away22:04
*** thorst_ has joined #openstack-nova22:04
*** jamielennox|away is now known as jamielennox22:11
*** tbachman has joined #openstack-nova22:12
*** thorst_ has quit IRC22:13
*** ijw has quit IRC22:13
mriedemcdent: cool22:13
*** jwcroppe has quit IRC22:14
*** asettle has joined #openstack-nova22:19
*** tblakes has quit IRC22:20
*** gouthamr has quit IRC22:23
*** ihrachys has quit IRC22:23
*** asettle has quit IRC22:24
*** pradk has quit IRC22:28
*** jamesdenton has quit IRC22:28
*** annegentle has quit IRC22:29
*** tbachman has quit IRC22:33
*** jwcroppe has joined #openstack-nova22:34
*** jwcroppe has quit IRC22:35
*** jwcroppe has joined #openstack-nova22:35
*** jwcroppe_ has joined #openstack-nova22:36
*** jwcroppe has quit IRC22:39
*** cdent has quit IRC22:41
*** jwcroppe_ has quit IRC22:41
*** kylek3h_away has joined #openstack-nova22:41
*** mbirru has quit IRC22:42
*** unicell1 has quit IRC22:43
*** fragatina has quit IRC22:43
*** browne has quit IRC22:44
*** armax has quit IRC22:44
*** jogo has quit IRC22:44
*** fragatina has joined #openstack-nova22:44
*** Swami has quit IRC22:45
*** fragatina has quit IRC22:45
*** unicell has joined #openstack-nova22:45
*** mbirru has joined #openstack-nova22:46
*** fragatina has joined #openstack-nova22:46
*** armax has joined #openstack-nova22:46
*** browne has joined #openstack-nova22:47
*** jogo has joined #openstack-nova22:51
*** devkulkarni has joined #openstack-nova22:52
*** raunak has quit IRC22:53
*** xyang1 has quit IRC22:55
*** baoli has quit IRC22:56
*** mtanino has quit IRC22:59
devkulkarnihey nova team, solum's dsvm-ubuntu-xenial gate is currently failing with "No hosts found to map to cell, exiting" error arising from lib/nova:create_cell. here is the failure log: http://logs.openstack.org/09/408409/4/check/gate-solum-devstack-dsvm-ubuntu-xenial/a41ada3/logs/devstacklog.txt.gz#_2016-12-09_07_04_07_953 . looking through recent devstack commits, I found that the check on create_cell has been removed https:/22:59
*** burt has quit IRC23:02
*** _ducttape_ has joined #openstack-nova23:04
*** ducttape_ has quit IRC23:07
*** _ducttape_ has quit IRC23:09
*** thorst has joined #openstack-nova23:11
*** tbachman has joined #openstack-nova23:13
*** thorst has quit IRC23:18
*** ljxiash has joined #openstack-nova23:23
*** tbachman has quit IRC23:24
*** tbachman has joined #openstack-nova23:26
*** ljxiash has quit IRC23:28
*** jwcroppe has joined #openstack-nova23:42
*** esberglu has quit IRC23:49
*** jgriffith is now known as jgriffith_AutoAw23:52
*** ducttape_ has joined #openstack-nova23:53
*** lascii is now known as alaski23:57

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