Tuesday, 2017-06-20

*** ckopper has joined #openstack-nova00:00
*** ckopper has quit IRC00:00
*** ijw__ has quit IRC00:01
*** suresh12 has quit IRC00:02
*** suresh12 has joined #openstack-nova00:05
*** Apoorva_ has quit IRC00:05
*** gmann has quit IRC00:06
*** gmann has joined #openstack-nova00:06
*** Apoorva has joined #openstack-nova00:06
*** openstack has joined #openstack-nova00:17
*** hieulq has joined #openstack-nova00:17
*** sambetts_ has quit IRC00:17
*** rha has quit IRC00:17
*** patriciadomin_ has quit IRC00:17
*** aignatov has quit IRC00:17
*** breton has joined #openstack-nova00:19
*** edmondsw has quit IRC00:19
*** sambetts_ has joined #openstack-nova00:19
*** aignatov has joined #openstack-nova00:20
*** patriciadomin has joined #openstack-nova00:20
*** tbachman has quit IRC00:20
*** tbachman_ is now known as tbachman00:20
*** rha has joined #openstack-nova00:20
*** rha has quit IRC00:20
*** rha has joined #openstack-nova00:20
*** igordcard has joined #openstack-nova00:20
*** PramodJ has quit IRC00:22
*** inara has quit IRC00:23
*** john51 has joined #openstack-nova00:23
*** s1061123 has joined #openstack-nova00:23
*** strigazi has joined #openstack-nova00:24
*** 7JTABFHYX has joined #openstack-nova00:24
*** zigo has joined #openstack-nova00:24
*** gyee has joined #openstack-nova00:24
*** vipul has joined #openstack-nova00:24
*** 7JTABFHYX has quit IRC00:24
*** szaher has joined #openstack-nova00:25
*** thorst has joined #openstack-nova00:26
*** thorst has quit IRC00:27
*** tbachman_ has joined #openstack-nova00:29
*** inara has joined #openstack-nova00:30
*** yingjun has joined #openstack-nova00:32
*** tbachman has quit IRC00:33
*** tbachman_ is now known as tbachman00:33
*** ckopper has joined #openstack-nova00:33
*** Apoorva_ has joined #openstack-nova00:35
openstackgerritMerged openstack/nova master: api-ref: cleanup PUT /os-hypervisors/statistics docs  https://review.openstack.org/47425000:35
*** Apoorva has quit IRC00:37
*** jaosorior has joined #openstack-nova00:39
*** Apoorva_ has quit IRC00:39
*** ijw has joined #openstack-nova00:41
*** amotoki_away is now known as amotoki00:41
*** gbarros has quit IRC00:41
*** ijw has quit IRC00:43
*** ijw has joined #openstack-nova00:43
openstackgerritGhanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs  https://review.openstack.org/46793800:45
openstackgerritGhanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs  https://review.openstack.org/46793800:48
*** Apoorva has joined #openstack-nova00:49
*** claudiub has quit IRC00:52
*** acormier has quit IRC00:52
*** acormier has joined #openstack-nova00:52
*** Apoorva has quit IRC00:53
*** Apoorva has joined #openstack-nova00:54
*** hoonetorg has quit IRC00:54
*** amotoki is now known as amotoki_away00:55
*** mdnadeem has joined #openstack-nova00:56
*** tonyb_ is now known as tonyb00:57
*** dtp has quit IRC00:58
*** Apoorva has quit IRC00:58
alex_xumriedem: sorry, I commented at wrong place. We can implement that check in the python code.00:59
*** ijw has quit IRC01:02
*** gbarros has joined #openstack-nova01:03
*** harlowja has joined #openstack-nova01:04
*** ijw has joined #openstack-nova01:09
*** hoonetorg has joined #openstack-nova01:11
*** sree has joined #openstack-nova01:11
*** xyang1 has quit IRC01:12
*** liujiong has joined #openstack-nova01:13
*** ijw has quit IRC01:15
*** sree has quit IRC01:16
*** zzzeek has quit IRC01:18
*** zzzeek has joined #openstack-nova01:18
*** yangyapeng has joined #openstack-nova01:20
*** mingyu has joined #openstack-nova01:20
*** ijw has joined #openstack-nova01:21
*** StevenK_ is now known as StevenK01:22
*** ijw has quit IRC01:26
*** markvoelker has quit IRC01:27
*** Kevin_Zheng has joined #openstack-nova01:30
*** hoonetorg has quit IRC01:30
*** Swami has quit IRC01:31
*** xinliang has quit IRC01:31
*** zz_dimtruck is now known as dimtruck01:32
*** smatzek has joined #openstack-nova01:32
*** mriedem has quit IRC01:33
*** gbarros has quit IRC01:33
*** xinliang has joined #openstack-nova01:34
*** xinliang has quit IRC01:34
*** xinliang has joined #openstack-nova01:34
*** rajathag_ has quit IRC01:37
*** thorst has joined #openstack-nova01:38
*** coreywright has quit IRC01:38
*** rfolco has joined #openstack-nova01:39
*** ijw has joined #openstack-nova01:40
*** suresh12 has quit IRC01:44
*** xinliang_ has joined #openstack-nova01:44
*** yangyape_ has joined #openstack-nova01:48
*** ijw has quit IRC01:48
*** mingyu has quit IRC01:49
*** yangyapeng has quit IRC01:52
*** hoonetorg has joined #openstack-nova01:52
*** yamahata_ has quit IRC01:53
*** acormier has quit IRC01:54
*** acormier has joined #openstack-nova01:54
*** litao__ has joined #openstack-nova01:55
*** thorst has quit IRC01:56
*** yangyape_ has quit IRC01:56
*** yangyapeng has joined #openstack-nova01:56
*** thorst has joined #openstack-nova01:56
*** coreywright has joined #openstack-nova01:56
*** hoonetorg has quit IRC01:58
*** acormier has quit IRC01:59
*** esberglu has quit IRC02:00
*** thorst has quit IRC02:00
*** amotoki_away is now known as amotoki02:01
*** xinliang has quit IRC02:02
*** edmondsw has joined #openstack-nova02:03
*** tiendc has joined #openstack-nova02:05
*** edmondsw has quit IRC02:08
*** mingyu has joined #openstack-nova02:10
*** amotoki is now known as amotoki_away02:11
*** hoonetorg has joined #openstack-nova02:12
Kevin_Zhengping mriedem02:16
*** yamamoto has joined #openstack-nova02:17
*** armax has joined #openstack-nova02:19
*** acormier has joined #openstack-nova02:19
*** _ioni is now known as ioni02:21
*** gomarivera has joined #openstack-nova02:23
*** yassine has quit IRC02:24
*** yassine has joined #openstack-nova02:24
*** Guest8722 is now known as sdake02:29
*** sdake has joined #openstack-nova02:29
*** hongbin has joined #openstack-nova02:31
*** raginbajin has quit IRC02:33
*** masber has joined #openstack-nova02:34
*** amotoki_away is now known as amotoki02:36
*** masuberu has quit IRC02:37
*** mingyu has quit IRC02:37
*** mingyu has joined #openstack-nova02:37
*** masuberu has joined #openstack-nova02:38
*** raginbajin has joined #openstack-nova02:39
*** gyee has quit IRC02:41
*** masber has quit IRC02:42
openstackgerritGhanshyam Mann proposed openstack/nova master: Fix to return server groups quota in quota class set  https://review.openstack.org/46799902:44
openstackgerritGhanshyam Mann proposed openstack/nova master: Fix to return server groups quota in quota class set  https://review.openstack.org/46799902:50
*** gomarivera has quit IRC02:58
*** gongysh has joined #openstack-nova02:59
*** gomarivera has joined #openstack-nova03:01
*** armax has quit IRC03:04
*** smatzek has quit IRC03:05
*** armax has joined #openstack-nova03:10
*** wxy has joined #openstack-nova03:11
*** acormier has quit IRC03:12
openstackgerritMoshe Levi proposed openstack/os-vif master: hardware offload support for openvswitch  https://review.openstack.org/46027803:12
*** acormier has joined #openstack-nova03:12
*** rfolco has quit IRC03:13
*** mingyu has quit IRC03:14
*** moshele has joined #openstack-nova03:15
*** acormier has quit IRC03:17
*** moshele has quit IRC03:17
openstackgerritZhenyu Zheng proposed openstack/nova master: Don't delete neutron port when attach failed  https://review.openstack.org/40356803:18
*** mingyu has joined #openstack-nova03:23
*** amotoki is now known as amotoki_away03:24
*** nicolasbock has joined #openstack-nova03:27
*** markvoelker has joined #openstack-nova03:28
*** tristanC has quit IRC03:29
*** tristanC has joined #openstack-nova03:31
*** sridharg has joined #openstack-nova03:34
*** diga has joined #openstack-nova03:43
*** gomarivera has quit IRC03:48
*** gomarivera has joined #openstack-nova03:49
*** mingyu has quit IRC03:51
*** edmondsw has joined #openstack-nova03:51
*** ratailor has joined #openstack-nova03:52
*** ratailor has quit IRC03:52
*** ratailor has joined #openstack-nova03:55
*** edmondsw has quit IRC03:56
*** thorst has joined #openstack-nova03:57
*** links has joined #openstack-nova03:58
gmannalex_xu, oomichi updated quota class fix, check now - https://review.openstack.org/#/c/467999/04:00
*** catintheroof has quit IRC04:01
*** thorst has quit IRC04:01
*** gouthamr has quit IRC04:03
*** mingyu has joined #openstack-nova04:04
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add missing parameters in limits.inc  https://review.openstack.org/47564004:05
*** mingyu has quit IRC04:06
*** mingyu has joined #openstack-nova04:08
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/47556304:12
*** qwertyco has joined #openstack-nova04:13
*** mingyu has quit IRC04:13
*** hongbin has quit IRC04:17
*** gomarivera has quit IRC04:29
*** gomarivera has joined #openstack-nova04:30
*** takashin has left #openstack-nova04:30
*** gomarivera has quit IRC04:33
*** gomarivera has joined #openstack-nova04:33
*** adisky__ has joined #openstack-nova04:34
*** jerrygb has joined #openstack-nova04:36
*** mdrabe has quit IRC04:40
alex_xugmann: cool, will try to reach it asap04:43
gmannalex_xu, thanks04:43
alex_xugmann: np, thanks for the patch also04:43
*** amotoki_away is now known as amotoki04:48
*** salv-orlando has joined #openstack-nova04:48
*** salv-orlando has quit IRC04:49
*** salv-orlando has joined #openstack-nova04:49
*** salv-orlando has quit IRC04:51
*** salv-orlando has joined #openstack-nova04:51
*** gongysh has quit IRC04:56
*** yamahata_ has joined #openstack-nova04:58
*** amotoki is now known as amotoki_away05:00
*** jerrygb_ has joined #openstack-nova05:01
*** jerrygb has quit IRC05:03
*** psachin has joined #openstack-nova05:09
*** pcaruana has joined #openstack-nova05:14
*** salv-orl_ has joined #openstack-nova05:15
*** ltomasbo|away is now known as ltomasbo05:17
*** salv-orlando has quit IRC05:18
*** moshele has joined #openstack-nova05:20
*** gomarivera has quit IRC05:20
*** gomarivera has joined #openstack-nova05:21
*** prateek has joined #openstack-nova05:21
*** gomarivera has quit IRC05:25
*** sree has joined #openstack-nova05:29
*** pcaruana has quit IRC05:30
*** gongysh has joined #openstack-nova05:30
*** vks1 has joined #openstack-nova05:35
*** frickler_ is now known as frickler05:35
*** edmondsw has joined #openstack-nova05:39
*** edmondsw has quit IRC05:44
*** suresh12 has joined #openstack-nova05:46
*** ioni is now known as _ioni05:55
*** _ioni is now known as ioni05:55
*** thorst has joined #openstack-nova05:58
*** markvoelker has quit IRC06:01
*** thorst has quit IRC06:02
*** gaurangt has quit IRC06:05
*** gaurangt has joined #openstack-nova06:05
*** bkopilov has quit IRC06:07
*** armax has quit IRC06:10
*** Oku_OS-away is now known as Oku_OS06:14
*** markus_z has joined #openstack-nova06:23
*** d3n14l has joined #openstack-nova06:26
*** claudiub has joined #openstack-nova06:26
*** armax has joined #openstack-nova06:27
*** d3n14l has left #openstack-nova06:30
*** fnordahl_ is now known as fnordahl06:31
*** udesale has joined #openstack-nova06:31
*** udesale has quit IRC06:31
*** amotoki_away is now known as amotoki06:31
*** udesale has joined #openstack-nova06:32
*** abalutoiu_ has quit IRC06:34
*** artom has joined #openstack-nova06:35
*** tesseract has joined #openstack-nova06:40
*** gongysh has quit IRC06:41
*** qwertyco has quit IRC06:49
*** suresh12 has quit IRC06:49
*** amotoki is now known as amotoki_away06:49
*** links has quit IRC06:49
*** pcaruana has joined #openstack-nova06:49
*** rcernin_ has joined #openstack-nova06:53
*** cNilesh has joined #openstack-nova06:55
*** salv-orl_ has quit IRC07:01
*** salv-orlando has joined #openstack-nova07:02
*** gongysh has joined #openstack-nova07:04
*** links has joined #openstack-nova07:05
*** amotoki_away is now known as amotoki07:06
*** jgrassle1 is now known as jgrassler07:06
*** salv-orlando has quit IRC07:07
*** gaurangt has quit IRC07:08
*** salv-orlando has joined #openstack-nova07:09
openstackgerritZhenyu Zheng proposed openstack/nova master: Don't delete neutron port when attach failed  https://review.openstack.org/40356807:10
*** mingyu has joined #openstack-nova07:13
*** alexchadin has joined #openstack-nova07:13
*** mingyu has quit IRC07:14
*** gaurangt has joined #openstack-nova07:14
*** rmart04 has joined #openstack-nova07:15
openstackgerritGhanshyam Mann proposed openstack/nova master: Fix to return server groups quota in quota class set  https://review.openstack.org/46799907:17
*** yamahata_ has quit IRC07:18
*** ijw has joined #openstack-nova07:21
*** amotoki is now known as amotoki_away07:22
*** abalutoiu has joined #openstack-nova07:24
*** ijw has quit IRC07:25
*** edmondsw has joined #openstack-nova07:27
*** amotoki_away is now known as amotoki07:29
*** dtantsur|afk is now known as dtantsur07:30
*** dave-mccowan has quit IRC07:30
*** tbachman has quit IRC07:30
*** sbezverk has quit IRC07:31
*** markus_z has quit IRC07:31
*** huanxie has joined #openstack-nova07:32
*** edmondsw has quit IRC07:32
*** pcaruana|afk| has joined #openstack-nova07:35
*** pcaruana|afk| has quit IRC07:35
*** pcaruana has quit IRC07:36
*** dave-mccowan has joined #openstack-nova07:36
*** jpena|off is now known as jpena07:37
*** amotoki is now known as amotoki_away07:39
*** pcaruana has joined #openstack-nova07:40
*** markus_z has joined #openstack-nova07:50
*** ralonsoh has joined #openstack-nova07:50
*** amotoki_away is now known as amotoki07:53
*** tbachman has joined #openstack-nova07:53
*** links has quit IRC07:55
*** thorst has joined #openstack-nova07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** vvargaszte has joined #openstack-nova08:03
*** thorst has quit IRC08:03
*** alextricity25 has quit IRC08:04
*** alextricity25_ has joined #openstack-nova08:04
*** oomichi has quit IRC08:05
ralonsohHello fellow reviewers, can I take your attention to https://review.openstack.org/#/c/449257/?08:06
*** oomichi has joined #openstack-nova08:07
ralonsohThe spec was merged months ago, but I still have two out of four patches pending for reviews08:07
ralonsohthanks in advance08:07
*** links has joined #openstack-nova08:08
*** udesale has quit IRC08:08
*** danpawlik has quit IRC08:13
*** Guest39557 is now known as asettle08:16
*** bkopilov has joined #openstack-nova08:17
*** mvk has quit IRC08:24
*** priteau has joined #openstack-nova08:27
*** derekh has joined #openstack-nova08:32
*** lucas-afk is now known as lucasagomes08:34
*** klipka_ is now known as klipka08:34
*** danpawlik has joined #openstack-nova08:35
*** efoley has joined #openstack-nova08:39
*** udesale has joined #openstack-nova08:41
*** markus_z has quit IRC08:46
*** openstackgerrit has quit IRC08:48
*** suresh12 has joined #openstack-nova08:50
*** mvk has joined #openstack-nova08:53
*** suresh12 has quit IRC08:54
*** openstackgerrit has joined #openstack-nova08:55
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.soft_delete notifications  https://review.openstack.org/46002908:55
*** danpawlik has quit IRC08:58
*** sdatko has quit IRC08:58
*** markus_z has joined #openstack-nova08:58
*** karimb has joined #openstack-nova08:58
*** mvk has quit IRC08:59
*** sree has quit IRC09:03
*** sree has joined #openstack-nova09:04
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.soft_delete notifications  https://review.openstack.org/46002909:07
*** sree has quit IRC09:08
gabor_antalsfinucan, bauzas: would you please give back +2s on my patchset https://review.openstack.org/460029 ? I had to modify the json because https://review.openstack.org/#/c/451789/ was merged a few days ago.09:08
*** salv-orlando has quit IRC09:09
*** salv-orlando has joined #openstack-nova09:09
*** mvk has joined #openstack-nova09:11
*** udesale has quit IRC09:13
*** salv-orlando has quit IRC09:13
openstackgerritMerged openstack/nova master: Provide error message in MismatchError for api-samples tests  https://review.openstack.org/47548909:15
*** edmondsw has joined #openstack-nova09:15
*** udesale has joined #openstack-nova09:16
*** sambetts_ is now known as sambetts09:19
*** edmondsw has quit IRC09:20
*** sdatko has joined #openstack-nova09:21
openstackgerritGhanshyam Mann proposed openstack/nova master: Fix to return server groups quota in quota class set  https://review.openstack.org/46799909:22
*** danpawlik has joined #openstack-nova09:23
*** vvargaszte has quit IRC09:25
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add datapath type information to OVS vif objects  https://review.openstack.org/47489209:30
*** openstackgerrit has quit IRC09:33
*** openstackgerrit has joined #openstack-nova09:34
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP traits  https://review.openstack.org/47455009:34
*** yingjun has quit IRC09:38
gabor_antalsfinucan: thanks!09:39
*** yamamoto has quit IRC09:41
openstackgerritAndrey Volkov proposed openstack/nova master: Add ability to view quota details for project owner  https://review.openstack.org/38600809:41
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.resize_prep notification  https://review.openstack.org/46508109:44
*** ijw has joined #openstack-nova09:44
*** mdnadeem has quit IRC09:46
*** sdatko has quit IRC09:47
*** danpawlik has quit IRC09:48
*** ftersin has joined #openstack-nova09:49
*** mingyu has joined #openstack-nova09:49
*** thorst has joined #openstack-nova09:49
*** ijw has quit IRC09:49
*** sdatko has joined #openstack-nova09:49
*** danpawlik has joined #openstack-nova09:50
*** gjayavelu has joined #openstack-nova09:52
*** alexchadin has quit IRC09:53
*** alexchadin has joined #openstack-nova09:53
*** vladikr has joined #openstack-nova09:55
*** dtantsur is now known as dtantsur|bbl09:55
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Read datapath_type from VIF object  https://review.openstack.org/47491409:56
*** yamamoto has joined #openstack-nova09:56
*** mnestratov has joined #openstack-nova09:56
*** thorst has quit IRC09:59
*** udesale has quit IRC10:02
*** liujiong has quit IRC10:02
*** jerrygb_ has quit IRC10:03
*** udesale has joined #openstack-nova10:03
*** yamamoto has quit IRC10:06
*** alexchadin has quit IRC10:07
openstackgerritMoshe Levi proposed openstack/os-vif master: hardware offload support for openvswitch  https://review.openstack.org/46027810:13
*** gszasz has joined #openstack-nova10:15
*** dims has quit IRC10:16
*** yangyapeng has quit IRC10:16
*** yangyapeng has joined #openstack-nova10:17
*** thorst has joined #openstack-nova10:17
*** gongysh has quit IRC10:18
*** dims has joined #openstack-nova10:21
*** yangyapeng has quit IRC10:21
*** gomarivera has joined #openstack-nova10:22
*** yangyapeng has joined #openstack-nova10:22
*** yamamoto has joined #openstack-nova10:25
*** mingyu has quit IRC10:25
*** gomarivera has quit IRC10:26
*** yangyapeng has quit IRC10:27
*** jpena is now known as jpena|off10:27
*** jpena|off is now known as jpena10:31
*** pewp has quit IRC10:34
*** moshele has quit IRC10:36
*** andreas_s has joined #openstack-nova10:36
*** pewp has joined #openstack-nova10:36
*** mingyu has joined #openstack-nova10:38
*** dtantsur|bbl is now known as dtantsur10:45
*** vvargaszte has joined #openstack-nova10:46
openstackgerritMerged openstack/nova master: Consolidate index and detail methods in HypervisorsController  https://review.openstack.org/47545810:47
*** karimb has quit IRC10:50
*** vks1 has quit IRC10:51
*** karimb has joined #openstack-nova10:51
*** qwertyco has joined #openstack-nova10:54
*** smatzek has joined #openstack-nova10:55
*** vvargaszte has quit IRC11:03
*** edmondsw has joined #openstack-nova11:03
*** edmondsw has quit IRC11:08
*** mingyu has quit IRC11:08
openstackgerritGábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification  https://review.openstack.org/46257611:10
*** bkopilov has quit IRC11:12
*** gjayavelu has quit IRC11:12
*** huanxie has quit IRC11:15
openstackgerritBéla Vancsics proposed openstack/nova master: Transform aggregate.update_metadata notification  https://review.openstack.org/46062511:16
*** thorst has quit IRC11:16
*** tbachman has quit IRC11:17
*** vks1 has joined #openstack-nova11:18
*** vladikr has quit IRC11:21
*** eric_young has quit IRC11:23
*** lucasagomes is now known as lucas-hungry11:24
*** moshele has joined #openstack-nova11:24
*** boolman has joined #openstack-nova11:25
boolmanHi guys, I'm having trouble with a new deployment.. I'm using juju to deploy ocata. and I seem to have issues with my compute-nodes/placement API, http://ix.io/xHP http://ix.io/xHO11:25
openstackgerritBéla Vancsics proposed openstack/nova master: Use more specific asserts in tests  https://review.openstack.org/46660911:27
*** yangyapeng has joined #openstack-nova11:28
*** karimb has quit IRC11:31
*** acormier has joined #openstack-nova11:33
*** ijw has joined #openstack-nova11:37
*** diga has quit IRC11:39
*** ijw has quit IRC11:42
*** vks1 has quit IRC11:44
*** thorst has joined #openstack-nova11:47
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/47556311:49
*** suresh12 has joined #openstack-nova11:51
*** thorst has quit IRC11:52
openstackgerritMoshe Levi proposed openstack/os-vif master: hardware offload support for openvswitch  https://review.openstack.org/46027811:53
*** jpena is now known as jpena|lunch11:54
*** suresh12 has quit IRC11:55
*** udesale has quit IRC11:55
*** udesale has joined #openstack-nova11:55
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification  https://review.openstack.org/47392911:56
*** acormier has quit IRC12:01
*** acormier has joined #openstack-nova12:02
*** acormier has quit IRC12:06
*** thorst has joined #openstack-nova12:08
*** tbachman has joined #openstack-nova12:09
*** rfolco has joined #openstack-nova12:14
*** edmondsw has joined #openstack-nova12:15
*** salv-orlando has joined #openstack-nova12:19
*** efried has joined #openstack-nova12:21
*** gomarivera has joined #openstack-nova12:27
*** jaypipes has joined #openstack-nova12:30
openstackgerritJay Pipes proposed openstack/nova master: placement: separate normalize_resources_qs_param  https://review.openstack.org/47559912:30
openstackgerritJay Pipes proposed openstack/nova master: placement: support GET /allocation_candidates  https://review.openstack.org/47544812:30
openstackgerritJay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates  https://review.openstack.org/47560012:30
*** yingjun has joined #openstack-nova12:30
*** kylek3h has joined #openstack-nova12:31
*** lucas-hungry is now known as lucasagomes12:32
*** efried has quit IRC12:34
*** tiendc has quit IRC12:34
bhagyashrisboolman: Hi, you can make sure that the [placement] section is there in the /etc/nova/nova.conf if its not then you should add this and restart the nova compute service.12:36
*** esberglu has joined #openstack-nova12:37
*** cNilesh has quit IRC12:37
*** mriedem has joined #openstack-nova12:38
*** esberglu has quit IRC12:39
*** esberglu has joined #openstack-nova12:39
mriedemsdague: when you get some time, can you go through the bottom set of +2ed changes in melwitt's quotas series? starts here: https://review.openstack.org/#/c/446241/ - they are mostly mechanical conversions from old style to new style quota checks.12:40
sfinucanmoshele, jaypipes: Am I missing something with https://review.openstack.org/#/c/460278/?12:40
rfolcojaypipes, could you please review https://review.openstack.org/#/c/460147/ ?12:40
*** acormier has joined #openstack-nova12:40
sfinucanfwict, you have nested try-excepts for the same exception. Surely there's no reason for that and the whole thing could be flattened?12:40
sfinucan...but I can't seem to get that point across in Gerrit, unfortunately :)12:41
*** acormier has quit IRC12:41
sdaguemriedem: yeh, probably an hour from now12:41
*** mdrabe has joined #openstack-nova12:42
*** bkopilov has joined #openstack-nova12:42
*** gomarivera has quit IRC12:43
*** vks1 has joined #openstack-nova12:43
*** esberglu has quit IRC12:44
ftersinmdbooth: hi. any news?12:44
*** pchavva has joined #openstack-nova12:49
*** lbragstad_ has quit IRC12:49
*** gomarivera has joined #openstack-nova12:49
*** gomarivera has quit IRC12:50
dtantsurdansmith: hi! remember I was talking about nova-api traceback on adding an ironic nova-compute host to an aggregate? here it is https://paste.fedoraproject.org/paste/giaZIJ~cymkC3nNXO1spaA12:51
*** esberglu has joined #openstack-nova12:51
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.resize_prep notification  https://review.openstack.org/46508112:51
*** salv-orlando has quit IRC12:52
*** salv-orlando has joined #openstack-nova12:52
*** efried has joined #openstack-nova12:53
*** alexchadin has joined #openstack-nova12:54
*** jwcroppe has quit IRC12:55
-openstackstatus- NOTICE: restarting gerrit to address slowdown issues12:56
*** salv-orlando has quit IRC12:56
*** jpena|lunch is now known as jpena12:57
*** efoley has quit IRC12:58
*** efoley has joined #openstack-nova12:58
*** lucasxu has joined #openstack-nova13:00
moshelesfinucan: to me it just a style issue, but I will do the change13:00
moshelesfinucan: sure I will put you as Co-Authored-by :) I will put the patch workflow -1 for now13:01
*** lyan has joined #openstack-nova13:02
*** mingyu has joined #openstack-nova13:02
*** awaugama has joined #openstack-nova13:06
*** mingyu_ has joined #openstack-nova13:09
*** mingyu has quit IRC13:09
alex_xujaypipes: bauzas edleafe, would you remind me about why we put the resource-classes in the extra_specs? I'm thinking of the same question for required_traits in the flavor. Put something new in the extra specs, that means something undiscoverable for the API user, so I didn't understand why.13:09
*** lpetrut has joined #openstack-nova13:09
*** karimb has joined #openstack-nova13:10
*** yamamoto has quit IRC13:10
*** ratailor has quit IRC13:10
*** yamamoto has joined #openstack-nova13:11
*** salv-orlando has joined #openstack-nova13:12
*** gouthamr has joined #openstack-nova13:12
*** alexchadin has quit IRC13:13
dtantsurdansmith (or anyone): isn't it doomed to fail: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4383 ? even my basic deployment has 2 cells (cell0 and default), and the service can be found in both (for whatever reason..)13:13
*** zz9pzza has quit IRC13:14
*** zz9pzza has joined #openstack-nova13:14
*** zz9pzza has quit IRC13:15
*** zz9pzza has joined #openstack-nova13:15
*** yamamoto has quit IRC13:15
*** jerrygb has joined #openstack-nova13:15
*** njohnston has left #openstack-nova13:17
*** links has quit IRC13:18
edleafealex_xu: IIRC, it's simply an interim solution until we can overhaul the handling of flavors.13:18
*** jerrygb_ has joined #openstack-nova13:20
*** baoli has joined #openstack-nova13:20
*** jerrygb has quit IRC13:20
*** eharney has joined #openstack-nova13:23
sfinucanedleafe: Remind me, most of the operations you refer to in the "Trying to understand the proposed direction" occur in, what, cell 0?13:26
sfinucanexcept for the actual booting of VM, which occurs in the target cell, of course13:26
*** mingyu_ has quit IRC13:27
sfinucanor the super cell, even13:28
*** mingyu has joined #openstack-nova13:28
*** yamamoto has joined #openstack-nova13:29
*** yamamoto has quit IRC13:29
*** gszasz has quit IRC13:30
edleafesfinucan: they occur in placement/scheduler. It is the super conductor that moderates the flow, and eventually dispatches the build to the cell chosen by the scheduler13:31
*** yamamoto has joined #openstack-nova13:31
sfinucanand the scheduler is running outside of the cells?13:32
sfinucani.e. there isn't a super-scheduler?13:32
edleafesfinucan: yeah, just one. No per-cell schedulers13:32
*** jwcroppe has joined #openstack-nova13:32
sfinucanGotcha. Thank you :)13:32
edleafenp13:32
* sfinucan is trying to formalize what you said in that mail into a more permanent doc13:32
dtantsurspeaking of which, is there something like "nova cells 101" for absolute beginners?13:33
sfinucandtantsur: dansmith's talk from the ATL summit it excellent13:33
edleafesfinucan: remember that what I called the "current flow" isn't what has merged into master; it was the design agreed to before the recent changes13:34
kashyapYeah, was about to suggest that13:34
dtantsursfinucan: cool! I've found something from ATX right now13:34
*** yingjun has quit IRC13:34
*** yamahata_ has joined #openstack-nova13:34
sfinucanedleafe: Yup, I haven't bothered including the "current flow" - just the new one13:34
sfinucanand can -WIP it til code is merged13:34
mriedemdtantsur: those are also linked here https://docs.openstack.org/developer/nova/cells.html13:34
edleafesfinucan: ok cool13:34
*** efried has quit IRC13:35
dtantsursweet. thanks mriedem13:35
boolmanbhagyashris: the [placment] section is there, configured and all. ( if missing or incorrect the nova-compute wont start )13:35
dansmithdtantsur: oh you probably have some control services in your cell0 and some computes in cell1, with the same id13:35
dansmithdtantsur: you need mriedem13:35
dansmither13:35
dtantsurdansmith: this is definitely the case13:35
dtantsuror wait13:35
dansmithdtantsur: you need mriedem's set which is converting that API to uuids13:35
dtantsurwhat is cell1?13:35
dansmithdtantsur: I just mean "the other cell that is not cell0, the real one"13:36
dtantsuraha13:36
dtantsurso, sorry, what do I need? is there some patch up?13:36
mriedemdtantsur: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/service-hyper-uuid-in-api13:36
mriedemis the wip for converting os-services and os-hypervisors APIs to use uuids13:37
dansmithmriedem: the -2 on that is for microversion sequencing I assume?13:37
mriedemdansmith: yes13:37
mriedemthose need to go together13:37
*** damien_r has quit IRC13:37
mriedemwell, they do'nt need to13:37
mriedembut we said they would13:37
dansmithoh I see the one behind it?13:37
mriedemonly thing left in os-hypervisors at the top is docs and paging13:37
*** zz9pzza has quit IRC13:38
dtantsurdoes it mean that I'll have to stop referring to host names when adding them to aggregates? please pardon my ignorance, I'm not sure how it solves the problem in question13:38
dansmithhe needs services13:38
*** yingjun has joined #openstack-nova13:38
dansmithdtantsur: oh are you getting that while querying for a service by hostname?13:38
dtantsurdansmith: I'm doing $ openstack aggregate add host baremetal-hosts overcloud-controller-0.localdomain13:39
dtantsurdo add the ironic host to an aggregate13:39
*** cleong has joined #openstack-nova13:39
dansmithI see13:39
dtantsurwhich may be not a perfect approach to a problem of hybrid bm-vm environment, but that's what we do now13:39
*** pchavva has quit IRC13:39
dansmithlet me look a bit13:39
* dtantsur gets back to cell videos meanwhile13:40
*** gomarivera has joined #openstack-nova13:40
dansmithohhh13:40
dtantsurI don't like this "ohhh" :)13:40
dansmithbecause you have multiple per host maybe13:40
dansmithhang on, I have no coffee in my system. this is not good.13:40
dtantsurmultiple what? I definitely have only one nova-compute process per host..13:41
*** zz9pzza has joined #openstack-nova13:41
boolmanHi guys, I'm having trouble with a new deployment.. I'm using juju to deploy ocata, so all the configuration is automated, and yes I have the [placement] section. and I seem to have issues with my compute-nodes/placement API, http://ix.io/xHP http://ix.io/xHO13:41
dtantsurdansmith: sure, lack of coffee is an urgent priority bug :)13:41
dansmithdtantsur: hmm, no this should work.. are you running control services on the same node as a compute?13:42
dtantsurdansmith: I am13:42
dansmithdtantsur: with different config files I assume?13:42
*** efried has joined #openstack-nova13:42
*** liverpooler has joined #openstack-nova13:42
dtantsurdansmith: I don't think so actually13:42
dansmithdtantsur: if not then they wouldn't be in different cells...13:43
dtantsurare cells assigned via nova.conf?13:43
*** gbarros has joined #openstack-nova13:43
dansmithno, but which cell you check into is determined by which database url you have configured13:43
dtantsurhmm, ok. these hosts may indeed end up in a wrong cell.. however, it does not quite explain to me why we have duplicate hosts exception13:44
dansmithwell,13:44
dansmithif they13:44
dansmithare in different cells,13:44
dansmithand they both report the same hostname, then there are multiple services with the same hostname but in different cells13:45
dtantsurthey = who? for now it seems that I have nova-api, scheduler and ironic's computes in one cell, then real compute in another13:46
dansmithright13:46
dansmiththey = any service (n-api, n-cpu, etc)13:46
dansmithsince they have the same hostname as what n-cpu is reporting,13:46
*** gomarivera has quit IRC13:46
dansmithwe could look that service up to find the cell, but end up getting the wrong one13:47
dansmithwhich is why we now check for dupes and fail if the result is ambiguous13:47
dansmithdtantsur: can we check to see if they are using the same config and thus reporting into the same cell for the moment?13:47
dtantsurdansmith: I have only one nova.conf (provided by puppet, I guess), so I think that's the case13:49
*** mingyu has quit IRC13:49
dtantsurI still don't quite understand why the result is ambiguos13:49
dansmithactually, that query should be looking specifically for a nova-compute service, which shouldn't be duplicated if you only have one compute13:49
dansmithdtantsur: that's what we're trying to find out13:49
dtantsurright13:49
dansmithdtantsur: can you dump the services table for your cell0 and other cell databases for me?13:49
dtantsurlemme try13:49
*** sree has joined #openstack-nova13:51
dtantsurdansmith: there are 3 databases, I guess I need one that is "nova", not "nova_api" or "nova_placement". right?13:51
*** gszasz has joined #openstack-nova13:51
*** pchavva has joined #openstack-nova13:51
*** crushil has joined #openstack-nova13:51
alex_xuedleafe: not sure what I should do for required traits13:52
*** suresh12 has joined #openstack-nova13:52
dansmithdtantsur: you should have nova and nova_cell013:53
dtantsurok, lemme see13:54
*** udesale has quit IRC13:56
*** qwertyco has quit IRC13:58
*** karimb has quit IRC13:58
*** gouthamr has quit IRC13:58
*** suresh12 has quit IRC14:00
dtantsurdansmith: ok, cell0 is empty. lemme fetch another one for you14:01
dansmithcell0.services you mean right?14:02
dansmiththat's what I would expect if you're all pointing at the other cell14:02
dansmithbut I'm not sure how we're hitting that not unique error then14:02
dansmithoh,14:02
*** gomarivera has joined #openstack-nova14:02
edleafealex_xu: I would mimic what was done for custom resources, but using the 'traits:' prefix. Not too sure how to distinguish required vs. preferred14:03
dansmithunless you have two cell records for the same cell?14:03
dansmithdtantsur: also in nova_api, dump out the cell_mapping table please14:03
dtantsursure, gimme a second14:03
dtantsurI need to figure out how to escape column names in mysql (binary is a reserved word, apparently)14:03
*** gomarivera has quit IRC14:04
*** suresh12 has joined #openstack-nova14:04
alex_xuedleafe: but really want to understand the reason why we continue put something in the extra spec first14:04
dansmithdtantsur: heh14:04
*** gomarivera has joined #openstack-nova14:04
dansmithdtantsur: I think if you use the full path it'll work, i.e.: SELECT services.binary FROM services14:04
*** prateek has quit IRC14:05
dtantsurdansmith: thnx! ok, so here are services: https://paste.fedoraproject.org/paste/ZK~YdTqfsWnpIImEF6TxLQ14:05
*** gbarros has quit IRC14:05
openstackgerritStephen Finucane proposed openstack/nova master: docs: Document the scheduler workflow  https://review.openstack.org/47581014:06
sfinucanedleafe, jaypipes: ^ probably not suitable for merge yet, but there you go14:06
jaypipessfinucan: hey, cool, thanks for working on that14:06
dtantsurdansmith: cell_mappings has 2 cells14:07
dansmithdtantsur: do they both point to the same database_connection ?14:08
dtantsurdansmith: nope14:08
*** felipemonteiro has joined #openstack-nova14:09
dansmithwell, then I'm not sure how you're getting that exception14:09
*** acormier has joined #openstack-nova14:09
* dtantsur bangs his head on the table14:09
openstackgerritMaciej Kucia proposed openstack/nova master: [doc] Updated sqlalchemy URL in migrate readme  https://review.openstack.org/47254914:09
dansmithdtantsur: is this a machine I can get on to add some debug? or can you just sprinkle some debug statements in that _find_service_in_cell() function?14:09
*** lyan has quit IRC14:10
*** karimb has joined #openstack-nova14:10
dtantsurdansmith: I can do both, the latter is easier, I guess14:10
dtantsurjust after the current meeting14:10
*** lyan has joined #openstack-nova14:10
dansmithhmm14:12
edleafealex_xu: because right now flavors are the only means to specify what you want Nova to build. Until we can clean up that technical debt, we are stuck using flavors, which means cramming more and more stuff into extra_specs14:13
*** jdillaman has joined #openstack-nova14:14
*** eharney has quit IRC14:14
dtantsurdansmith: what exactly would you like to see printed? I think I have an idea, but just in case you have something else in mind14:15
dansmithdtantsur: that for loop should iterate twice.. once over each cell. if cell0 is empty, it should never set "service" to something, and thus in the second/last iteration, it should not hit the ServiceNotUnique raise, because service hasn't been set yet14:16
dansmithdtantsur: so just anything around service getting set, what it's set to, etc14:17
dtantsuryep, this is my assumption too. I'll try to check it.14:17
*** gbarros has joined #openstack-nova14:20
*** tidwellr has joined #openstack-nova14:22
openstackgerritAndrey Kurilin proposed openstack/python-novaclient master: Allow tuple as for nics value  https://review.openstack.org/47581614:23
*** yamahata_ has quit IRC14:23
*** salv-orlando has quit IRC14:24
sdaguemelwitt / mriedem - in https://review.openstack.org/#/c/446247/23 ... auto_allocated floating ips are excluded from quota?14:24
sdagueis that they way it's always been?14:24
*** moshele has quit IRC14:25
dtantsurdansmith: it looks like both Cell objects (cell0 and default) have the same database connection (poining to "nova") Oo14:26
*** suresh12 has quit IRC14:26
dansmithdtantsur: okay, that makes sense for getting that not unique error, so now we need to figure out why the db is different14:27
dtantsurwait, no14:27
sdagueah, I found other code that indicates yes14:27
sdaguemriedem: ok, I went up until the first jenkins failing patch14:27
mriedemsdague: that's the way it's always been14:27
sdagueand they look good14:27
*** salv-orlando has joined #openstack-nova14:28
sdaguemriedem: yeh, I didn't realize, but the follow on patch touched a line where it was more obvious14:28
dtantsurdansmith: sorry for confusing. the cells are ok. I'm just getting the same host out of both..14:29
mriedemthanks for hitting those14:29
dansmithdtantsur: okay14:29
*** gouthamr has joined #openstack-nova14:30
*** salv-orlando has quit IRC14:30
*** yangyape_ has joined #openstack-nova14:30
*** salv-orlando has joined #openstack-nova14:30
dtantsurbut 'services' table is empty on nova_cell0, I've just double-checked >_<14:31
sdaguemriedem: my guess is that got caught up in the 6 way octopus merge limits with git14:31
sdaguemriedem: so honestly rebasing everything once the base set lands is probably in order anyway14:32
dansmithdtantsur: and just to be clear, the api_connection is pointed at nova_api and not nova_placement right?14:32
dtantsurlemme check too14:32
*** eharney has joined #openstack-nova14:33
sdaguemriedem: my only wish in those patches (which is long past as it landed in march) is that the config option was strict_quotas instead of recheck_quotas, just from a nice name perspective14:33
*** yangyapeng has quit IRC14:33
*** udesale has joined #openstack-nova14:34
*** knikolla_phone has joined #openstack-nova14:34
dtantsurdansmith: [api_database]connection points to nova_api14:34
dansmithdtantsur: okay so in that for loop, is it finding/setting14:34
dansmithservice to anything on the cell0 pass?14:35
dtantsurdansmith: yep. lemme show you the output of my added logging14:35
openstackgerritBalazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref  https://review.openstack.org/45366714:35
openstackgerritBalazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications  https://review.openstack.org/45307714:35
openstackgerritAndrey Kurilin proposed openstack/python-novaclient master: Allow tuple as for nics value  https://review.openstack.org/47581614:36
knikolla_phoneHow does nova calculate quota usage? After manually setting an instance as deleted in the database 'instances' table the resources still show up as used.14:36
dansmithheh14:37
dtantsurdansmith: https://paste.fedoraproject.org/paste/r26SqCAGrk8dXPlOk-sfjg14:37
*** thorst has quit IRC14:37
*** annegentle has joined #openstack-nova14:38
*** lbragstad_ has joined #openstack-nova14:38
*** jdwidari has quit IRC14:39
dansmithdtantsur: that's kinda hard to make sense of, especially without seeing the code, but.. why the HostMappingNotFound messages interleaved with the iteration of that for loop?14:39
dtantsurdansmith: I think it was a bug in oslo.log14:39
dansmithoh eff me14:39
dansmithdtantsur: I see the bug14:40
*** yingjun has quit IRC14:40
openstackgerritDan Smith proposed openstack/nova master: Fix using targeted cell context when finding services in cells  https://review.openstack.org/47583014:41
dansmithdtantsur: try that ^ :)14:41
*** felipemonteiro has quit IRC14:41
dtantsurthanks!14:41
*** suresh12 has joined #openstack-nova14:41
*** lbragstad_ is now known as lbragstad14:42
dansmithdtantsur: it was iterating but not actually looking in the other database :P14:42
dtantsurI see14:42
*** yamahata_ has joined #openstack-nova14:42
*** thorst has joined #openstack-nova14:42
*** lyan has quit IRC14:43
*** lyan has joined #openstack-nova14:43
*** felipemonteiro has joined #openstack-nova14:44
*** hongbin has joined #openstack-nova14:46
dansmithdtantsur: if this fixes you, then: sorry for wasting your time14:46
*** udesale has quit IRC14:46
*** thorst has quit IRC14:47
openstackgerritJay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates  https://review.openstack.org/47560014:47
*** rcernin_ has quit IRC14:47
dtantsurdansmith: w00t, it works with this patch :) thanks!14:48
dansmithdtantsur: cool, I shall write a test and then try to get that in asap14:48
dtantsurgreat14:48
jaypipesknikolla_phone: you need to run the nova-manage quota refresh command.14:48
dansmithcc: mriedem14:48
jaypipesknikolla_phone: unfortunately, quotas uses different tables than instances to store usage.14:49
dansmithjaypipes: yesterday that was true14:49
dansmithjaypipes: today it may not be14:49
jaypipeswait, what?14:49
*** markvoelker has joined #openstack-nova14:49
jaypipesoh, yeah.14:49
* dansmith looks at all the counting quotas patches in the gate14:49
jaypipesbut pretty sure knikolla_phone doesn't have yesterday's code deployed ;)14:49
dansmithGASP14:49
mriedemhells bells https://review.openstack.org/#/c/475830/1/nova/compute/api.py14:50
dansmithjaypipes: I know, but it's amusing timing given the flurry of them in the gate atm14:50
knikolla_phoneJaypipes: we're running ocata14:50
jaypipesknikolla_phone: yup, so you need to do the nova-manage quota refresh thing :)14:50
jaypipesdansmith: ack14:50
knikolla_phoneJaypipes: thanks. Will give it a try now.14:51
*** annegentle has quit IRC14:53
mriedemmgagne: sorry it took awhile, but i finally got another thorough review on https://review.openstack.org/#/c/454322/14:53
mriedemmgagne: lots of edge cases in there that need to be dealt with14:53
mriedemdansmith: so,14:53
mgagneok, will read and address them asap14:53
mriedemsomething i've been meaning to pester other cores about for their opinions on https://review.openstack.org/#/c/454322/14:53
*** yamamoto has quit IRC14:53
mriedemdansmith: ^ is a new compute driver capability,14:53
mriedemcalled by cinder when a volume is resized14:54
mriedemif you're doing this on an instance running on an ocata compute, that's going to be ignored14:54
mriedemwith no indication to the user/admin14:54
*** yamamoto has joined #openstack-nova14:54
*** yamamoto has quit IRC14:54
knikolla_phonejaypipes: that didn't work. Still ghost resources.14:55
mriedemdansmith: well, take that back, you'll get a warning  in the compute logs on the ocata node for "Received unexpected event %(event)s for instance"14:55
openstackgerritGábor Antal proposed openstack/nova master: Transform keypair.delete notification  https://review.openstack.org/46394614:55
knikolla_phoneI know i asked this once and mriedem answered. But what does the deleted column in the table need to be set for a deleted instance? I can't remember what it was, sorry to ask again :(14:56
mriedemdansmith: i was pondering doing a minimum compute service version check in the api for these types of events, since we know that you need a minimum of pike to support those events, and we could return a 400 to cinder otherwise,14:56
mriedemand they could rollback the volume resize on their side14:56
jaypipesknikolla_phone: to its ID value.14:56
mriedemknikolla_phone: instances.deleted = instances.id14:56
mriedemif it's deleted14:56
jaypipesknikolla_phone: but honestly, you shouldn't be setting stuff manually in the DB. Dragons be there.14:56
mriedemdansmith: but i'm wondering if that's too heavy handed, doing the service version check in the api14:56
dansmithmriedem: yeah, I mean, it's a warning and the assumption was that we'd add events over time, so the warning isn't terrible14:57
dansmithmriedem: just depends on whether or not we have different behavior before/after14:57
dansmithif it's just an optimization that will continue to work but slower, then no need to be heavy-handed I think14:57
mriedemmy thinking was as a user i do this volume resize thing, and it all says it's fine,14:57
mriedembut the block device in my guest isn't resized to the new size, so i report a support ticket14:57
mriedemthe admin has to go figure out through the logs that the event was basically ignored on the ocata compute because it didn't recognize it14:58
knikolla_phonejaypipes: i like dragons.14:58
dansmithah, yeah, but.. they triggered the resize from the cinder side right?14:58
mriedemwe also don't have any policy check for this to effectively disable it per event until your computes are all upgraded14:58
mriedemdansmith: yes14:58
dansmithmriedem: blocking the event from cinder isn't going to tell the user anything right?14:58
mriedembut cinder is pike and the instance is runnign on an ocata compute14:59
knikolla_phoneWe were left with an inconsistent db after upgrade. Some instances failed to create and delete. But everything else was fine.14:59
mriedemdansmith: i was thinking that if we could detect in the api that this isn't going to work based on service version, then we return a 400 to cinder and cinder rolls back the volume resize14:59
mriedemwhich is the indication to the user14:59
dansmithmriedem: that's super expensive14:59
dansmithmriedem: and.. can you even roll it back? often times block device resizes are one-way15:00
mriedemalternatively, if cinder has a policy rule for extending an attached volume, then the admin can disable this from the cinder side until they know they can handle it15:00
mriedemmgagne: does the cinder side of this have a unique policy rule for extending attached volumes?15:00
mriedemor is it just the same as os-extend for 'available' non-attached volumes?15:00
jaypipesknikolla_phone: here's a dragon for you: http://bit.ly/2sLVKtR15:01
mriedemmgagne: because if i were a xen or vmware or hyperv shop, and this just will never work, i'd want to disable the cinder api behavior globally15:01
*** takedakn has joined #openstack-nova15:01
*** gomarivera has quit IRC15:01
*** ckopper has quit IRC15:02
knikolla_phonejaypipes: the nova-manage project quota_refresh_usage command didn't do anything.15:02
*** gomarivera has joined #openstack-nova15:02
knikolla_phoneNewton -> ocata was the only problematic upgrade so far for us. :(15:02
*** mingyu has joined #openstack-nova15:03
*** yamamoto has joined #openstack-nova15:04
jaypipesknikolla_phone: other than manually going through your DB, I'm not entirely sure I can help you on this channel. There's some mismatch somewhere between the project_quota_usages or quota_usages table and the "true" resource tables like instances, instance_extra, pci_devices, etc15:04
*** READ10 has joined #openstack-nova15:04
jaypipesknikolla_phone: is this isolated to a particular tenant? or general?15:04
*** annegentle has joined #openstack-nova15:05
knikolla_phonejaypipes: no, that's okay, you've helped more than enough. This is only limited to one tenant.15:06
jaypipesknikolla_phone: and only for their instances quota?15:06
*** gbarros has quit IRC15:06
*** gomarivera has quit IRC15:06
*** suresh12 has quit IRC15:07
knikolla_phonejaypipes: whatever resources two of the instances we manually deleted used (instances, vcpu, ram, etc)15:07
jaypipeshmm15:08
mriedemdansmith: ok, so i think policy blocker on the cinder side is key, i've put that in the cinder change https://review.openstack.org/#/c/454287/15:08
jaypipesknikolla_phone: but does the instances usage show correct? meaning, does it show the tenant using instances that were deleted?15:08
dansmithmriedem: ack15:08
mriedemdansmith: and then we just report the shit out of when it fails on the nova compute side in pike15:08
*** sbezverk has joined #openstack-nova15:08
mriedemusing action events and faults15:08
*** yamamoto has quit IRC15:11
knikolla_phonejaypipes: yes. The usage is correct to what it would be if those two instances weren't deleted (which is weird because we manually deleted five of them). Quota_usages table reflects that amount, and updating it to random values and then using quota_refresh_usage command, it preserves the random amount.15:11
*** yamamoto has joined #openstack-nova15:11
jaypipesknikolla_phone: what's in the reservations table for that project?15:13
*** rmart04 has quit IRC15:14
*** sree has quit IRC15:14
*** sree has joined #openstack-nova15:15
*** yamamoto has quit IRC15:15
*** sree has quit IRC15:18
*** jerrygb_ has quit IRC15:20
*** gszasz has quit IRC15:20
knikolla_phonejaypipes: what does uuid in that table (reservations) map to? Is it an instance uuid?15:21
jaypipesknikolla_phone: yep15:21
*** jerrygb has joined #openstack-nova15:21
*** ckopper has joined #openstack-nova15:21
*** Oku_OS is now known as Oku_OS-away15:22
*** takedakn has quit IRC15:23
knikolla_phonejaypipes: okay, i see a deleted row. Going to try setting that too.15:23
knikolla_phoneThanks for all your help.15:23
mnaseri've been doing some digging around and i cannot seem to find where root_device_name gets assigned for vms.. can someone give a hint :(15:23
jaypipesknikolla_phone: hey, no prob :)15:23
mnaseri've discovered a bug with when using virtio-scsi with nova15:23
jaypipesmnaser: depends on the hypervisor.15:24
jaypipesmnaser: you talking about libvirt?15:24
mnaserroot_device_name gets set to /dev/vda .. but when attaching cinder volumes, it tries to attach them at /dev/sdX, and tries to assign the same pci address15:24
mnaseryep15:24
jaypipesmnaser: hmm...15:24
jaypipesmnaser: might need to bring in mdbooth on that one.15:24
mnaseri turned off the vm, attached the cinder block device and then started it15:24
jaypipesmdbooth: Matt, any ideas?15:24
mnaserand it actually generated invalid libvirt xml15:24
mnaseri have a copy of it, let me paste15:25
* mdbooth reads back15:25
mnaserwell looks like paste.openstack.org isn't loading15:26
mriedemmnaser: _validate_and_build_base_options in nova.compute.api.API15:26
*** knikolla_phone has quit IRC15:26
mnaserhttps://gist.github.com/mnaser/8b000d20590e4343e4d7a73459d8521015:26
mriedemsets it in the base_options dict which eventually gets set on the instance (really the BuildRequest now)15:26
mnasernotice the address of the first 2 disks15:26
mnaseris the same15:26
mnaserso qemu never boots the vm15:26
*** claudiub has quit IRC15:27
mnaseri added a LOG.error(disk_info) in the libvirt tooling15:27
mnaserand this is what came out of it too - https://gist.github.com/mnaser/aedd44b9df018786a144f7c719ec5a1315:27
*** jerrygb_ has joined #openstack-nova15:27
mnasernotice how 'dev': 'vda' (which is ephermal) and the 'dev': 'sda' (which is attached cinder block device)15:27
mnaseri am guessing that's where the issue is stemming, the first one should be sda too (because using virtio-scsi with the image hw_ parameters)15:28
mriedemmnaser: https://github.com/openstack/nova/blob/9045a98145f0834446d668e0282ddb1c6f2d136c/nova/compute/api.py#L921 is eventually what gets set in the instance.root_device_name field15:28
*** jerrygb has quit IRC15:29
mnaseri've confirmed with two different installs that booting an image with hw_scsi_model=virtio-scsi and hw_disk_bus=scsi will work but no cinder volumes will successfully attach: libvirtError: internal error: unable to execute QEMU command '__com.redhat_drive_add': Duplicate ID 'drive-scsi0-0-0-0' for drive15:29
mdboothmnaser: what's the symptom of the weird xml?15:29
mdboothDoes it barf?15:29
mnaserso, if you try to attach it to an already running vm, you get the libvirt error above (duplicate id)15:29
*** claudiub has joined #openstack-nova15:30
mnaserif you turn the vm off, attach (really a no-op if its offline), start it, it'll generate that invalid xml and fail15:30
*** tidwellr has quit IRC15:30
mnaserso im not sure if libvirt or nova is to blame in putting the duplicate address ther15:30
mdboothWe really shouldn't be looking at root_device_name15:30
*** tidwellr has joined #openstack-nova15:31
*** sapcc-bot has quit IRC15:31
*** david_3 has quit IRC15:31
*** databus23_ has quit IRC15:31
*** carthaca_ has quit IRC15:31
mnaseri've just been kinda trying to understand the code and i dont know if i can point the finger at nova or libvirt yet15:31
*** jerrygb has joined #openstack-nova15:31
*** mingyu has quit IRC15:31
*** david_4 has joined #openstack-nova15:31
*** sapcc-bot has joined #openstack-nova15:31
*** carthaca_ has joined #openstack-nova15:31
*** databus23_ has joined #openstack-nova15:31
mnaserbut it's obviously just an issue when using virtio-scsi15:32
*** jerrygb_ has quit IRC15:32
*** karimb has quit IRC15:33
*** thorst has joined #openstack-nova15:33
*** thorst has quit IRC15:34
*** ltomasbo is now known as ltomasbo|away15:34
*** thorst has joined #openstack-nova15:34
*** markus_z has quit IRC15:36
*** jerrygb_ has joined #openstack-nova15:36
*** jerrygb has quit IRC15:37
mgagnemriedem: supports_volume_extend vs supports_extend_volume ? RE https://review.openstack.org/#/c/454322/18/nova/virt/driver.py15:37
*** gyee has joined #openstack-nova15:38
openstackgerritDan Smith proposed openstack/nova master: Fix using targeted cell context when finding services in cells  https://review.openstack.org/47583015:38
dansmithdtantsur: mriedem ^15:38
dtantsurnice15:39
openstackgerritKaitlin Farr proposed openstack/nova master: Remove deprecated keymgr code  https://review.openstack.org/43985515:40
mriedemmelwitt: i think you might have gotten delete happy in the tests here https://review.openstack.org/#/c/457862/15:41
mriedemmgagne: i'm ok with either - i suppose supports_extend_volume to match the driver's method name15:41
mgagnethat's my opinion15:41
*** crushil has quit IRC15:42
*** jerrygb_ has quit IRC15:42
*** jerrygb has joined #openstack-nova15:43
*** gomarivera has joined #openstack-nova15:47
mriedemmdbooth: i think that root_device_name thing is a legacy bdm v1 thing, but not totally sure15:50
*** tidwellr has quit IRC15:50
mriedemsome kind of ec2 carry over15:50
*** mriedem is now known as mriedem_bowlcut15:51
mdboothmriedem: I haven't followed the code all the way down, but I suspect we could ignore it15:51
*** sridharg has quit IRC15:52
openstackgerritJay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates  https://review.openstack.org/47560015:54
*** tbachman has quit IRC15:56
*** efried has quit IRC15:56
*** andreas_s has quit IRC15:56
dansmithjaypipes: quick +W bait: https://review.openstack.org/#/c/475830/15:56
*** tbachman has joined #openstack-nova15:56
dansmithjaypipes: fixes the ironic people15:57
jaypipesdansmith: wallaby'd15:57
dansmiththanks15:57
dtantsurthanks all!15:57
dansmithI guess cdent still isn't around today?15:57
* jaypipes trying to figure out gibi's faults bug...15:57
jaypipesdansmith: he's on PTO for next two weeksd.15:58
dansmithoh my15:58
jaypipesdansmith: after which he will be a VMWare employee.15:58
*** vks1 has quit IRC15:59
dansmithoh15:59
dansmithmy15:59
dansmithGAWD15:59
*** nic has joined #openstack-nova16:00
*** tidwellr has joined #openstack-nova16:00
openstackgerritMerged openstack/nova master: Count networks to check quota  https://review.openstack.org/44624116:01
gibi_jaypipes: good luck with the api.fault bug16:02
*** eharney has quit IRC16:02
jaypipesgibi_: yeah, I might need to pass on that :(16:02
jaypipesgibi_: that's some old-ass code in there :)16:02
*** vks1 has joined #openstack-nova16:03
*** armax has quit IRC16:03
gibi_jaypipes: I hope somebody can say 'delete that code!'16:03
*** eharney has joined #openstack-nova16:03
jaypipesgibi_: it's possible. some day. https://twitter.com/dims/status/54200201897052979216:04
*** kaisers_ has joined #openstack-nova16:04
*** artom has quit IRC16:05
gibi_:)16:06
openstackgerritMerged openstack/nova master: Add SecurityGroupList.get_counts()  https://review.openstack.org/44624516:07
*** dtantsur is now known as dtantsur|afk16:08
openstackgerritMerged openstack/nova master: Count security groups to check quota  https://review.openstack.org/45786016:08
openstackgerritMerged openstack/nova master: Add FixedIPList.get_count_by_project()  https://review.openstack.org/44624616:09
*** thorst has quit IRC16:09
*** mvk has quit IRC16:09
openstackgerritMerged openstack/nova master: Count fixed ips to check quota  https://review.openstack.org/45786116:09
openstackgerritMerged openstack/nova master: Add FloatingIPList.get_count_by_project()  https://review.openstack.org/44624716:10
dansmithwow, just count those counting quotas merges16:10
*** Apoorva has joined #openstack-nova16:11
*** psachin has quit IRC16:13
*** mdrabe has quit IRC16:13
*** ltomasbo|away is now known as ltomasbo16:13
*** sudipto has joined #openstack-nova16:17
*** sudipto_ has joined #openstack-nova16:17
*** yamamoto has joined #openstack-nova16:17
*** mdrabe has joined #openstack-nova16:19
*** pcaruana has quit IRC16:19
openstackgerritBalazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (3)  https://review.openstack.org/45282016:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data  https://review.openstack.org/45281816:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (2)  https://review.openstack.org/45281916:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data  https://review.openstack.org/47586016:21
*** dtp has joined #openstack-nova16:22
openstackgerritGábor Antal proposed openstack/nova master: Transform keypair.delete notification  https://review.openstack.org/46394616:25
*** thorst has joined #openstack-nova16:25
*** suresh12 has joined #openstack-nova16:27
*** yamamoto has quit IRC16:28
openstackgerritAlex Szarka proposed openstack/nova master: [WIP]Transform instance-live_migration_abort notification  https://review.openstack.org/42045316:32
*** jwcroppe has quit IRC16:32
*** thorst has quit IRC16:35
openstackgerritAlex Szarka proposed openstack/nova master: [WIP]Transform instance-live_migration_abort notification  https://review.openstack.org/42045316:36
openstackgerritAlex Szarka proposed openstack/nova master: [WIP]Transform instance-live_migration_abort notification  https://review.openstack.org/42045316:37
openstackgerritAlex Szarka proposed openstack/nova master: [WIP]Transform instance-live_migration_abort notification  https://review.openstack.org/42045316:40
*** lpetrut has quit IRC16:40
*** ltomasbo is now known as ltomasbo|away16:42
*** tbachman has quit IRC16:44
*** sambetts is now known as sambetts|afk16:45
*** suresh12 has quit IRC16:45
*** kfarr has joined #openstack-nova16:46
openstackgerritGábor Antal proposed openstack/nova master: Add json style checking for sample notifications  https://review.openstack.org/44367716:47
*** rajathagasthya has joined #openstack-nova16:47
*** sbezverk has quit IRC16:51
*** thorst has joined #openstack-nova16:52
*** itlinux_ has joined #openstack-nova16:52
*** crushil has joined #openstack-nova16:54
*** zzzeek has quit IRC16:55
*** thorst has quit IRC16:55
*** moshele has joined #openstack-nova16:55
*** derekh has quit IRC16:56
*** yamamoto has joined #openstack-nova16:57
*** ralonsoh has quit IRC16:58
*** thorst has joined #openstack-nova17:00
*** tesseract has quit IRC17:00
*** moshele has quit IRC17:01
*** efried has joined #openstack-nova17:02
*** jwcroppe has joined #openstack-nova17:03
*** lucasagomes is now known as lucas-afk17:03
*** yamamoto has quit IRC17:03
*** rajathag_ has joined #openstack-nova17:03
*** liangy has joined #openstack-nova17:04
*** nic has quit IRC17:04
*** rajathagasthya has quit IRC17:04
*** moshele has joined #openstack-nova17:06
*** Swami has joined #openstack-nova17:09
*** nic1 has joined #openstack-nova17:12
*** felipemonteiro has quit IRC17:13
*** vks1 has quit IRC17:16
*** vks1 has joined #openstack-nova17:18
*** suresh12 has joined #openstack-nova17:19
*** qwertyco has joined #openstack-nova17:19
*** lyan has quit IRC17:19
*** tbachman has joined #openstack-nova17:20
*** slaweq has joined #openstack-nova17:20
openstackgerritMerged openstack/nova master: Updated from global requirements  https://review.openstack.org/47556317:22
*** suresh12 has quit IRC17:23
*** amotoki is now known as amotoki_away17:23
*** suresh12 has joined #openstack-nova17:24
*** catintheroof has joined #openstack-nova17:24
*** slaweq has quit IRC17:25
*** suresh12_ has joined #openstack-nova17:25
*** slaweq has joined #openstack-nova17:25
*** suresh12 has quit IRC17:29
*** slaweq has quit IRC17:30
*** artom has joined #openstack-nova17:30
*** READ10 has quit IRC17:31
*** suresh12_ has quit IRC17:32
*** rajathag_ has quit IRC17:32
*** vks1 has quit IRC17:35
*** jwcroppe_ has joined #openstack-nova17:35
*** jwcroppe has quit IRC17:37
*** suresh12 has joined #openstack-nova17:38
*** sudipto has quit IRC17:40
*** sudipto_ has quit IRC17:40
*** priteau has quit IRC17:42
*** priteau has joined #openstack-nova17:42
*** jwcroppe_ has quit IRC17:43
*** claudiub has quit IRC17:45
*** priteau has quit IRC17:47
*** zzzeek has joined #openstack-nova17:47
*** efried has quit IRC17:47
*** rajathagasthya has joined #openstack-nova17:49
*** priteau has joined #openstack-nova17:49
*** priteau has quit IRC17:53
*** priteau has joined #openstack-nova17:54
*** priteau has quit IRC17:54
*** jpena is now known as jpena|off17:54
*** priteau has joined #openstack-nova17:54
*** artom has quit IRC17:56
*** tbachman has quit IRC17:56
*** zzzeek has quit IRC17:57
*** qwertyco has quit IRC17:57
*** kfarr has quit IRC17:58
*** suresh12 has quit IRC17:59
*** suresh12 has joined #openstack-nova17:59
*** mriedem_bowlcut is now known as mriedem17:59
*** priteau has quit IRC17:59
*** moshele has quit IRC17:59
*** priteau has joined #openstack-nova18:02
*** zzzeek has joined #openstack-nova18:03
*** catinthe_ has joined #openstack-nova18:03
*** thorst has quit IRC18:05
*** catintheroof has quit IRC18:05
*** MasterOfBugs has joined #openstack-nova18:06
*** suresh12 has quit IRC18:10
*** efoley has quit IRC18:11
*** nic1 has quit IRC18:12
*** winston-d_ has quit IRC18:12
*** suresh12 has joined #openstack-nova18:18
*** jwcroppe has joined #openstack-nova18:20
*** priteau has quit IRC18:23
*** priteau has joined #openstack-nova18:23
*** jwcroppe has quit IRC18:25
*** tbachman has joined #openstack-nova18:26
mriedemgibi_: i know this was already approved, but needed to be rebased and while it's sitting there i had a question inline https://review.openstack.org/#/c/460029/8/doc/notification_samples/instance-soft_delete-start.json18:26
mriedemabout transforming soft delete notifications to versioned payloads18:27
*** nic has joined #openstack-nova18:27
*** priteau has quit IRC18:28
*** rajathagasthya has quit IRC18:29
*** suresh12 has quit IRC18:32
*** sbezverk has joined #openstack-nova18:33
melwittmriedem: I deleted those tests on purpose because after changing floating_ips to a CountableResource, there are no more "project scoped quotas" that are refreshable or syncable. the majority of the tests in nova/tests/unit/test_quota.py rely on the global QUOTAS engine and once there are no more real examples of a certain type of quota, we would have to synthesize fake resources for the tests to still work18:36
melwittinstead, I figured delete them since they're no longer applicable to anything real. the quota refresh command only refreshes ReservableResources18:37
mriedemi thought the refresh command was using the _sync_*_quotas stuff still, which is in the db api, but would have to dig18:38
*** salv-orlando has quit IRC18:38
mriedemif it's only using ReservableResources, and we don't have any of those, that's also grounds to remove it - but i need to check18:38
*** jwcroppe has joined #openstack-nova18:38
*** salv-orlando has joined #openstack-nova18:39
melwittmriedem: it does, but the _sync_* functions are only used on ReservableResources. the connection is made in nova/quota.py and each patch that removes the connection stops it from working that way18:39
melwittmriedem: we still have ReservableResources, they are just only user scoped and those unit tests are testing project scoped. it's confusing :(18:39
openstackgerritKaitlin Farr proposed openstack/nova master: Remove deprecated keymgr code  https://review.openstack.org/43985518:39
*** eharney has quit IRC18:40
mriedemi'll dig a bit18:40
melwittthe patch that removes the last ReservableResource is "Count instances" and we're not there yet. k. the "user scoped quotas" are the ones in the "project_user_quotas_table" and the "project scoped quotas" are in the "quotas" table (floating_ips, fixed_ips, networks). that's what the tests refer to when it says "project key" it means one of: 'floating_ips', 'fixed_ips', or 'networks'18:42
mriedemmelwitt: ok so https://github.com/openstack/nova/blob/master/nova/quota.py#L900 checks https://github.com/openstack/nova/blob/master/nova/quota.py#L41618:43
melwittthere's also the global var in nova/db/sqlalchemy/api.py PER_PROJECT_QUOTAS that show the project keys18:43
mriedemand since per_project_resources is fixedips, floatingips and networks,18:43
mriedemand none of those are ReservableResources18:43
melwittyes18:44
mriedembut we could still have user-scoped resources,18:44
mriedemlike server groups18:44
*** jwcroppe has quit IRC18:44
melwittyes, and we still do. we still have 'instances' 'cores' 'ram' as of that patch I think18:44
*** ericyoung has joined #openstack-nova18:45
mriedemlet me check the tests being removed again18:45
melwittso those will still use reserve/commit/rollback and be synced/refreshed18:45
melwittk18:45
*** eharney has joined #openstack-nova18:46
mriedemrfolco: some issues in here https://review.openstack.org/#/c/460147/18:46
openstackgerritMerged openstack/nova master: Fix using targeted cell context when finding services in cells  https://review.openstack.org/47583018:46
mriedemrfolco: i'd really appreciate to leave the translation marker removal to a separate patch devoted to that18:46
melwittthe only one we could argue is test_usage_refresh_invalid_user_key. I could do s/floating_ips/fake_resource_key/ there and the test would still work. the point of it had been, "verify you get Invalid if you try to refresh a valid key that isn't a valid user key"18:47
melwittand since there are no more project keys, we can't choose any valid keys. the test on L3270 is sort of identical18:48
rfolcomriedem, looking18:48
openstackgerritMerged openstack/nova master: api-ref: fix hypervisor_hostname description for Ironic  https://review.openstack.org/47472318:51
*** mnestratov has quit IRC18:51
openstackgerritMerged openstack/nova master: [doc] Updated sqlalchemy URL in migrate readme  https://review.openstack.org/47254918:51
mriedemmelwitt: removed my -1 for that - are we ok with https://review.openstack.org/#/c/457862/22/nova/network/floating_ips.py@225 ?18:52
mriedemi think i talked myself into being ok with it18:52
mriedembut i need you to close the deal18:52
melwittmriedem: thanks. the test on L3270 is the same test really anyway. looking at the next question ...18:55
*** thorst has joined #openstack-nova18:58
*** mdrabe_ has joined #openstack-nova18:59
*** mdrabe has quit IRC19:00
melwittmriedem: looks like that's similar to the fixed ip one. I didn't think to call the self.deallocate* methods bc I thought to just use the inverse object functions to undo the create. but, I think technically maybe someone could create a floating ip and a dns domain in parallel and maybe could end up with orphaned dns domains that way. so it's probably most correct to call self.deallocate*19:00
melwittI'm not sure if associating the dns domain requires anything from the floating ip create response maybe. but either way it doesn't hurt to call the self.deallocate function19:01
mriedemyeah i'm having a hard time figuring out how/where the dns domains and floating ips are correlated19:02
mriedemjust looking at https://developer.openstack.org/api-ref/compute/?expanded=create-allocate-floating-ip-address-detail#floating-ip-dns-records-os-floating-ip-dns-deprecated isn't helping me19:03
melwittyeah, I just cracked open the api-ref19:03
melwittheh "Floating IP DNS records (os-floating-ip-dns) (DEPRECATED)"19:03
*** jwcroppe has joined #openstack-nova19:03
melwittmaybe it's this one, you provide an IP and it creates an entry Floating IP DNS records (os-floating-ip-dns) (DEPRECATED)19:04
melwittgah19:05
melwitthttps://developer.openstack.org/api-ref/compute/?expanded=create-or-update-dns-domain-detail,create-or-update-dns-entry-detail#create-or-update-dns-entry19:05
*** jwcroppe has quit IRC19:06
melwittI would guess that you would have to receive the address back from the API after creating the floating ip before you could create dns entries19:06
mriedemwow https://developer.openstack.org/api-ref/compute/?expanded=create-or-update-dns-entry-detail#create-or-update-dns-entry19:07
mriedemthe request params table there is all sorts of wrong19:08
melwittoh, well that's the stuff from the url looks like19:08
mgagnemriedem: actively working on the online volume extend change, adding all relevant unit tests atm. will complete tomorrow. I addressed most if not all comments.19:08
*** suresh12 has joined #openstack-nova19:08
melwittbut the other stuff isn't in the table. yeah19:09
mriedemmgagne: sounds good19:09
*** slaweq has joined #openstack-nova19:10
mriedemmgagne: i imagine you may be a bit sore19:10
mgagneall fine, just a bit out of time for today ;)19:10
mriedemmelwitt: yeah it's this i think https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/floating_ip_dns.py#L22519:11
melwittah, yep, that looks like it19:11
mriedemmelwitt: so you can't add a floating ip address dns entry to the dns domain if you don't already have the floating ip address,19:12
mriedemso,19:12
mriedemwe can probably just delete the floating ip if we go over-quota, rather than call deallocate_floating_ip19:12
mriedemlike you're doing19:12
melwittyeah, so it just looks like the same deal as the fixed ip alloc/dealloc thing, you can't have made other associations before the REST call returns something to the user for the user to call back and make associations19:12
mriedemok, and if we mess this up, the .002% of people that will likely hit it can yell at us19:13
melwittso I probably lucked out on that. and yes, that is also true heh19:13
mriedemwe'll just tell them dansmith told us it was ok19:13
*** itlinux_ has quit IRC19:13
melwitt:D19:13
melwittsdague had a good point about calling that option "strict_quotas" instead of "recheck_quota." now that's gonna bug me19:13
melwittI know I could technically go ahead and deprecate "recheck_quota" and add strict_quotas and have both work for awhile19:14
*** thorst has quit IRC19:15
*** gbarros has joined #openstack-nova19:15
sdaguemelwitt: ++19:15
*** thorst has joined #openstack-nova19:15
sdagueit's not in a stable yet, so now would be the time to do it19:15
mriedemmelwitt: +W on the change,19:15
mriedemyou're getting eerily close to seeing light at the end of this quota tunnel19:15
*** catinthe_ has quit IRC19:16
melwittsdague: yeah, I think I'll do it then. that name is so much better. I just couldn't think of anything good19:16
*** slaweq has quit IRC19:17
melwittmriedem: \o/ can't believe it! also, I need to add all of the recheck quota test cases for instances now, that's gonna be a pain19:17
*** jwcroppe has joined #openstack-nova19:18
melwittI meant to have done it already but was busy and sort of avoiding it :(19:18
*** thorst has quit IRC19:20
*** nicolasbock has quit IRC19:21
*** priteau has joined #openstack-nova19:25
mriedemthat's ok, i'm putting off supporting compute node paging across multiple cells19:26
*** priteau has joined #openstack-nova19:26
*** gbarros_ has joined #openstack-nova19:28
*** ericyoung has quit IRC19:29
*** edmondsw has quit IRC19:29
*** gbarros has quit IRC19:30
sdaguemelwitt: so, given the bug dan was debugging this morning, is this bit right - https://review.openstack.org/#/c/416521/46/nova/compute/api.py@302219:33
sdagueI also kind of wonder if there is going to need to be some standard scatter/gather callers here19:33
*** hoonetorg has quit IRC19:34
dansmithsdague: I just replied to both19:34
melwittsdague: it is now wrong since dansmith's change to how target_cell works. the change is really old19:35
melwittso, good catch19:35
*** gomarivera has quit IRC19:36
*** gomarivera has joined #openstack-nova19:36
*** liangy has quit IRC19:37
*** jwcroppe has quit IRC19:37
*** rajathagasthya has joined #openstack-nova19:38
*** dimtruck is now known as zz_dimtruck19:39
openstackgerritMerged openstack/nova master: placement: separate normalize_resources_qs_param  https://review.openstack.org/47559919:40
*** gomarivera has quit IRC19:41
openstackgerritMerged openstack/nova master: Use more specific asserts in tests  https://review.openstack.org/46660919:41
*** qwertyco has joined #openstack-nova19:43
*** gbarros_ has quit IRC19:48
*** qwertyco has quit IRC19:50
*** hoonetorg has joined #openstack-nova19:50
*** gomarivera has joined #openstack-nova19:53
*** gomarivera has quit IRC19:53
*** zzzeek has quit IRC19:53
*** gomarivera has joined #openstack-nova19:54
openstackgerritIldiko Vancsa proposed openstack/nova master: WIP: Add Cinder v3 detach to swap_volume  https://review.openstack.org/45697119:57
*** edmondsw has joined #openstack-nova20:00
*** jwcroppe has joined #openstack-nova20:04
*** Apoorva_ has joined #openstack-nova20:05
*** Apoorva has quit IRC20:08
*** smatzek has quit IRC20:09
*** slaweq has joined #openstack-nova20:11
*** jwcroppe has quit IRC20:12
*** jwcroppe has joined #openstack-nova20:13
*** hoonetorg has quit IRC20:15
*** kylek3h has quit IRC20:16
*** jwcroppe has quit IRC20:17
*** mdrabe_ is now known as mdrabe20:18
*** jerrygb has quit IRC20:19
*** efried has joined #openstack-nova20:20
*** amotoki_away is now known as amotoki20:24
*** liverpooler has quit IRC20:24
*** mnestratov has joined #openstack-nova20:27
*** suresh12 has quit IRC20:28
*** suresh12 has joined #openstack-nova20:29
*** pchavva has quit IRC20:31
cfriesenhas anyone looked at what the qemu logs look like when block live-migrating under heavy localdisk IO?  does it show up in the "Migration running for..." log from the live migration monitor?20:31
*** amotoki is now known as amotoki_away20:33
*** xyang1 has joined #openstack-nova20:33
*** hoonetorg has joined #openstack-nova20:36
*** smatzek has joined #openstack-nova20:38
*** suresh12 has quit IRC20:39
*** zzzeek has joined #openstack-nova20:42
*** awaugama has quit IRC20:42
*** suresh12 has joined #openstack-nova20:44
*** ijw has joined #openstack-nova20:45
*** jwcroppe has joined #openstack-nova20:46
*** suresh12 has quit IRC20:46
*** suresh12 has joined #openstack-nova20:46
*** eharney has quit IRC20:46
mriedemmelwitt: were you looking for a release note on this? https://review.openstack.org/#/c/393695/20:48
melwittmriedem: not really, just wanted a sanity check that we can be assured adding a minimum wouldn't break anyone. maybe a release note is the right thing?20:49
melwittI don't know why anyone would use -1 but what if they did. I didn't check yet what a -1 would do for that interval, maybe it just wouldn't even work20:50
mriedemmelwitt: don't know, but i left a reply in there too - -1 has special meanings in things20:50
mriedemlike retry forever,20:50
mriedemor disable20:50
melwittyeah. so maybe the question is, does -1 work presently or not. if not, then the point is moot20:51
*** jwcroppe has quit IRC20:51
*** jwcroppe has joined #openstack-nova20:51
*** cleong has quit IRC20:53
*** jwcroppe has quit IRC20:56
openstackgerritMerged openstack/nova master: Count floating ips to check quota  https://review.openstack.org/45786220:57
openstackgerritEric Fried proposed openstack/nova master: WIP: hypervisor_hostname must match get_available_nodes  https://review.openstack.org/47592621:03
*** lucasxu has quit IRC21:05
*** gbarros has joined #openstack-nova21:07
*** tidwellr has quit IRC21:08
*** gbarros has quit IRC21:08
openstackgerritRick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API  https://review.openstack.org/44928821:09
*** owalsh_ has joined #openstack-nova21:10
*** gbarros has joined #openstack-nova21:12
*** owalsh has quit IRC21:12
*** gbarros has quit IRC21:14
*** gbarros has joined #openstack-nova21:15
openstackgerritMerged openstack/nova master: Remove translation of log messages  https://review.openstack.org/46794821:16
*** salv-orl_ has joined #openstack-nova21:17
openstackgerritMerged openstack/nova master: [placement] Update placement devref to modern features  https://review.openstack.org/46904721:17
*** smatzek has quit IRC21:17
*** efried has quit IRC21:18
*** salv-orlando has quit IRC21:20
*** armax has joined #openstack-nova21:22
*** jwcroppe has joined #openstack-nova21:22
*** crushil has quit IRC21:22
*** efried has joined #openstack-nova21:22
*** crushil has joined #openstack-nova21:22
*** catintheroof has joined #openstack-nova21:25
*** crushil has quit IRC21:27
*** gouthamr has quit IRC21:31
*** smatzek has joined #openstack-nova21:32
*** thorst has joined #openstack-nova21:32
*** dave-mccowan has quit IRC21:35
*** pramodrj07 has joined #openstack-nova21:37
*** kylek3h has joined #openstack-nova21:38
*** gbarros has quit IRC21:40
*** baoli has quit IRC21:40
*** MasterOfBugs has quit IRC21:40
*** rfolco has quit IRC21:42
*** rajathagasthya has quit IRC21:45
openstackgerritDan Smith proposed openstack/nova master: Fix and optimize external_events for multiple cells  https://review.openstack.org/44514221:47
*** thorst has quit IRC21:47
*** gouthamr has joined #openstack-nova21:47
*** mingyu has joined #openstack-nova21:49
openstackgerritEric Fried proposed openstack/nova master: hypervisor_hostname must match get_available_nodes  https://review.openstack.org/47592621:52
*** gbarros has joined #openstack-nova21:53
*** mingyu has quit IRC21:53
*** armax has quit IRC21:54
*** slaweq has quit IRC21:55
*** slaweq has joined #openstack-nova21:56
*** markvoelker has quit IRC21:56
*** slaweq has quit IRC21:56
*** slaweq has joined #openstack-nova21:57
*** thorst has joined #openstack-nova21:57
*** smatzek has quit IRC21:59
*** slaweq has quit IRC21:59
*** priteau has quit IRC22:00
*** slaweq has joined #openstack-nova22:00
*** gbarros has quit IRC22:00
mriedemdansmith: we need to talk about https://review.openstack.org/#/c/391941/22:03
mriedemthe easy one first https://review.openstack.org/#/c/391941/50/nova/compute/cells_api.py22:03
*** armax has joined #openstack-nova22:03
mriedemi'm thinking we don't need to be passing that tag kwarg through for cells v122:03
dansmithtotes agree22:04
*** markvoelker has joined #openstack-nova22:04
mriedemok now the hard one https://review.openstack.org/#/c/391941/50/nova/compute/api.py22:04
mriedemthe way artom has this written,22:04
dansmithif we're checking service version and failing,22:04
mriedemif you try to attach a volume with a tag to a shelved offloaded instance, he fails with a 40022:04
dansmithwe should just always fail in cells_api if you pass a tag22:04
*** thorst has quit IRC22:05
mriedemgood point22:05
mriedemmayhap you'd like to leave that comment22:05
*** thorst has joined #openstack-nova22:05
dansmithmayhap I already did22:05
mriedemok, so the shelved offloaded case22:05
*** slaweq has quit IRC22:05
mriedemhe's got this failing for a shelved offloaded instance since you can't call reserve_block_device_name on the compute22:06
mriedemwhich is what's checking the driver flag22:06
mriedemhere https://review.openstack.org/#/c/391941/50/nova/compute/manager.py22:06
*** mdrabe has quit IRC22:06
dansmithugh, we support regular volume attach for offloaded instances?22:07
mriedemi'm trying to decide if we should support attaching a volume with a tag to a shelved offloaded instance. his argument was it's not good to mysteriously fail to apply the tag when unshelving22:07
mriedemeff yes we do22:07
mriedemHPE MFer22:07
*** thorst has quit IRC22:07
dansmithyeah, I understand his point, but.. oye22:07
*** esberglu has quit IRC22:07
mriedemso with attach to a shelve offloaded instance, we prep the bdms (attach them) to the guest when unshelving22:07
*** gbarros has joined #openstack-nova22:08
mriedembasically server create, but you know22:08
mriedemlike bfv for an already booted instance - it's the best22:08
*** esberglu has joined #openstack-nova22:08
*** efried has quit IRC22:08
mriedemanyway, my thinking was we could support attaching volumes to a shelved offloaded instance in basically the same way as we treat normal create22:08
dansmithright, but it would suck to have destined your instance to certain doom while offloaded,22:08
dansmithand then be unable to ever boot it ever because you have no supporting hypervisors22:08
mriedemit wouldn't be certain doom22:09
mriedemand the same can be said for creating a server with a tagged bdm22:09
mriedemif it ends up on a compute that doesn't support tagged devices, it's a BuildAbortException22:09
mriedemremember me freaking out about this earlier in the week?22:09
*** david_4 has quit IRC22:10
*** sapcc-bot has quit IRC22:10
*** carthaca_ has quit IRC22:10
*** databus23_ has quit IRC22:10
mriedemon normal server create we check here https://github.com/openstack/nova/blob/4e6a7c18b1c8340b9fdc93ee02e56b5a29a99648/nova/compute/manager.py#L193022:10
mriedemif that fails (it's a vmware host), we abort the build22:11
mriedemfin22:11
mriedemi was thinking we could allow attaching a tagged bdm to a shelved offloaded instance, and during unshelve we basically have the same flag check, and if the compute doesn't support it then we fail the unshelve22:12
*** openstack has joined #openstack-nova22:16
*** david_ has joined #openstack-nova22:16
*** Apoorva_ has quit IRC22:17
*** dansmith_ is now known as dansmith22:17
dansmithmriedem: well, the options are:22:17
dansmith1. refuse to attach with a tag to offloaded instances22:17
dansmith2. allow it and fail on unshelve22:17
*** oomichi has quit IRC22:17
*** Apoorva has joined #openstack-nova22:17
*** dansmith is now known as Guest9625922:17
Guest96259...I guess that's it, right?22:17
mriedemGuest96259: basically22:18
*** david_ has quit IRC22:18
*** databus23_2 has quit IRC22:18
*** carthaca_2 has quit IRC22:18
*** sapcc-bot3 has quit IRC22:18
*** owalsh_ is now known as owalsh22:18
*** databus23_ has joined #openstack-nova22:19
*** sapcc-bot has joined #openstack-nova22:19
*** david_ has joined #openstack-nova22:19
*** carthaca_ has joined #openstack-nova22:19
Guest96259I dunno, #1 seems not that terrible to me, given how much stuff doesn't really work with shelve22:19
*** david_ has quit IRC22:19
*** databus23_ has quit IRC22:19
*** carthaca_ has quit IRC22:19
mriedemGuest96259: failing in unshelve seems like a can of worms given we don't do anything with the instance if spawn fails22:19
*** oomichi has joined #openstack-nova22:19
*** sapcc-bot has quit IRC22:20
Guest96259yeah22:20
*** dosaboy has joined #openstack-nova22:20
mriedemwe'll revert the task_state,22:20
mriedemrecord a fault22:20
mriedembut we don't clean anything up, i.e. https://bugs.launchpad.net/nova/+bug/162769422:20
mriedemi guess you don't want the instance in ERROR state b/c then you can't attempt to unshelve it again22:20
*** david_ has joined #openstack-nova22:20
*** sapcc-bot has joined #openstack-nova22:20
openstackLaunchpad bug 1627694 in OpenStack Compute (nova) "unshelving an instance doesn't rollback volumes connections on failure" [Low,In progress] - Assigned to Shoham Peller (shoham-peller)22:21
*** edmondsw has quit IRC22:23
*** gomarivera has joined #openstack-nova22:23
mriedemmordred: is your new api user ML a place i can ask if people are actually using some of the compute APIs? like shelve?22:24
*** StevenK has joined #openstack-nova22:25
*** suresh12 has quit IRC22:28
*** gomarivera has quit IRC22:29
*** slaweq has joined #openstack-nova22:30
*** esberglu has joined #openstack-nova22:32
*** Apoorva_ has joined #openstack-nova22:33
*** Apoorva has quit IRC22:33
*** slaweq has quit IRC22:37
*** esberglu has quit IRC22:37
openstackgerritMerged openstack/nova master: libvirt: Extract method _guest_add_video_device  https://review.openstack.org/47078322:37
openstackgerritMerged openstack/nova master: libvirt: Extract method _guest_add_pci_devices  https://review.openstack.org/47078422:37
*** slaweq has joined #openstack-nova22:38
openstackgerritMerged openstack/nova master: libvirt: Extract method _guest_add_watchdog_action  https://review.openstack.org/47078522:41
*** acormier has quit IRC22:41
openstackgerritMerged openstack/nova master: libvirt: Extract method _guest_add_memory_balloon  https://review.openstack.org/47078622:42
*** sbezverk has quit IRC22:42
*** acormier has joined #openstack-nova22:42
*** jwcroppe has joined #openstack-nova22:43
*** slaweq has quit IRC22:43
*** jwcroppe has quit IRC22:43
*** suresh12_ has joined #openstack-nova22:44
openstackgerritMerged openstack/nova master: libvirt: Extract method _guest_add_spice_channel  https://review.openstack.org/47078722:44
*** acormier has quit IRC22:46
*** mriedem1 has joined #openstack-nova22:48
*** gbarros has quit IRC22:49
mordredmriedem: sure!22:49
*** mriedem has quit IRC22:50
*** pramodrj07 has quit IRC22:52
*** jwcroppe has joined #openstack-nova22:52
*** pramodrj07 has joined #openstack-nova22:53
openstackgerritMerged openstack/nova master: remove null_safe_int from module scope  https://review.openstack.org/45577822:54
*** rajathagasthya has quit IRC22:54
*** xyang1 has quit IRC22:55
*** rajathagasthya has joined #openstack-nova22:55
* Guest96259 rages22:56
Guest96259oh double rage22:56
*** Guest96259 is now known as dansmith22:56
*** gbarros has joined #openstack-nova22:57
*** annegentle has quit IRC22:58
*** salv-orl_ has quit IRC22:59
*** salv-orlando has joined #openstack-nova22:59
*** gbarros has quit IRC23:00
*** Apoorva has joined #openstack-nova23:02
*** gbarros has joined #openstack-nova23:02
*** jwcroppe has quit IRC23:03
*** salv-orlando has quit IRC23:03
*** gomarivera has joined #openstack-nova23:03
*** jwcroppe has joined #openstack-nova23:04
*** Apoorva_ has quit IRC23:05
*** jwcroppe has quit IRC23:09
*** gomarivera has quit IRC23:09
openstackgerritIldiko Vancsa proposed openstack/nova master: WIP: Add Cinder v3 detach to swap_volume  https://review.openstack.org/45697123:09
*** slaweq has joined #openstack-nova23:14
*** catintheroof has quit IRC23:18
*** slaweq has quit IRC23:19
*** harlowja has quit IRC23:19
*** annegentle has joined #openstack-nova23:19
*** annegentle has quit IRC23:22
*** Apoorva has quit IRC23:24
*** armax has quit IRC23:26
*** mvk has joined #openstack-nova23:27
*** gomarivera has joined #openstack-nova23:27
openstackgerritMerged openstack/nova master: Use plain routes list for os-remote-consoles instead of stevedore  https://review.openstack.org/47493023:27
gmannalex_xu, mriedem1 its ready now - https://review.openstack.org/#/c/467999/23:28
mriedem1ok23:31
*** mriedem1 is now known as mriedem23:31
*** ijw has quit IRC23:33
*** slaweq has joined #openstack-nova23:35
mriedemalex_xu: i'm not sure if i'll be in the next api meeting but i added something to open discussion in the agenda https://wiki.openstack.org/wiki/Meetings/NovaAPI#Agenda23:36
*** gbarros has quit IRC23:37
*** gbarros has joined #openstack-nova23:38
*** slaweq has quit IRC23:40
openstackgerritmelanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object  https://review.openstack.org/44624423:40
openstackgerritmelanie witt proposed openstack/nova master: Remove 'reserved' count from used limits  https://review.openstack.org/44624223:40
openstackgerritmelanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage  https://review.openstack.org/44624323:40
openstackgerritmelanie witt proposed openstack/nova master: Count instances to check quota  https://review.openstack.org/41652123:40
openstackgerritmelanie witt proposed openstack/nova master: Make Quotas object favor the API database  https://review.openstack.org/41094523:40
openstackgerritmelanie witt proposed openstack/nova master: Add online migration to move quotas to API database  https://review.openstack.org/41094623:40
openstackgerritmelanie witt proposed openstack/nova master: Add scatter_gather_all_cells() utility  https://review.openstack.org/47595723:40
*** annegentle has joined #openstack-nova23:42
*** sbezverk has joined #openstack-nova23:44
*** gomarivera has quit IRC23:46
*** annegentle has quit IRC23:47
*** acormier has joined #openstack-nova23:51
*** slaweq has joined #openstack-nova23:56
*** mnestratov has quit IRC23:59

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