Friday, 2017-03-10

efriedNow if he could just do that for pyXX...00:00
mriedemyou don't want it for py2700:01
efriedmriedem It would have to be really, really smart.00:01
mriedemthis just cycled in https://www.youtube.com/watch?v=Lrle0x_DHBM so things are looking up00:02
efriedIn my case, I'm getting away with -- nova.tests.unit.virt.powervm00:02
*** rfolco has joined #openstack-nova00:02
cburgessOh wow thats soo cool.00:05
*** yingjun has joined #openstack-nova00:07
mriedemcburgess: -HEAD or earth, wind and fire? :)00:08
cburgesshead00:08
melwittboth, duh00:08
mriedemhere come dem fixes00:11
openstackgerritMatt Riedemann proposed openstack/nova master: Decrement quota when deleting an instance in cell0  https://review.openstack.org/44339500:11
openstackgerritMatt Riedemann proposed openstack/nova master: Fix missing instance.delete notification  https://review.openstack.org/43722200:11
openstackgerritMatt Riedemann proposed openstack/nova master: Add regression test for bug 1670627  https://review.openstack.org/44374500:11
openstackbug 1670627 in OpenStack Compute (nova) "quota is always in-use after delete the ERROR instances " [Critical,In progress] https://launchpad.net/bugs/1670627 - Assigned to Matt Riedemann (mriedem)00:11
openstackgerritMatt Riedemann proposed openstack/nova master: Temporarily untarget context when deleting from cell0  https://review.openstack.org/44340300:11
*** Fdaisuke has quit IRC00:14
*** markvoelker has quit IRC00:15
*** jose-phillips has quit IRC00:17
openstackgerritEric Fried proposed openstack/nova master: WIP: PowerVM Driver: SSP emphemeral disk support  https://review.openstack.org/44318900:17
*** tblakes has joined #openstack-nova00:18
*** smatzek has joined #openstack-nova00:18
*** smatzek has quit IRC00:19
*** smatzek has joined #openstack-nova00:19
*** gfhellma_ has quit IRC00:22
*** hongbin has quit IRC00:22
*** smatzek has quit IRC00:24
*** marst has quit IRC00:24
*** yamamoto_ has joined #openstack-nova00:25
*** crushil has joined #openstack-nova00:26
*** smatzek has joined #openstack-nova00:27
openstackgerritMatt Riedemann proposed openstack/nova master: Decrement quota when deleting an instance in cell0  https://review.openstack.org/44339500:28
openstackgerritMatt Riedemann proposed openstack/nova master: Temporarily untarget context when deleting from cell0  https://review.openstack.org/44340300:28
*** jose-phillips has joined #openstack-nova00:29
*** jaypipes has quit IRC00:29
*** catintheroof has quit IRC00:29
*** mriedem has quit IRC00:29
*** catintheroof has joined #openstack-nova00:30
*** catintheroof has quit IRC00:30
*** salv-orlando has quit IRC00:30
*** crushil has quit IRC00:32
melwittdansmith: I added a question on the consoleauth upcall patch https://review.openstack.org/#/c/415922/7/nova/compute/api.py@3469 if you have any thoughts00:33
*** ducttape_ has quit IRC00:34
*** nic has quit IRC00:35
*** jamielennox is now known as jamielennox|away00:36
*** jamielennox|away is now known as jamielennox00:37
*** tovin07_ has joined #openstack-nova00:38
*** eglynn has joined #openstack-nova00:38
*** tuanluong has joined #openstack-nova00:41
*** catintheroof has joined #openstack-nova00:42
*** hamzy has joined #openstack-nova00:43
*** gcb has quit IRC00:44
*** jose-phillips has quit IRC00:44
*** sree has joined #openstack-nova00:45
*** adrian_otto has joined #openstack-nova00:46
*** dimtruck is now known as zz_dimtruck00:46
*** jamesdenton has joined #openstack-nova00:49
*** sree has quit IRC00:49
*** jose-phillips has joined #openstack-nova00:53
*** iceyao has joined #openstack-nova00:53
*** thorst has joined #openstack-nova00:53
*** jose-phillips has quit IRC00:53
*** jose-phillips has joined #openstack-nova00:54
*** zhurong has joined #openstack-nova00:56
*** priteau has joined #openstack-nova00:56
*** tovin07 has joined #openstack-nova00:58
*** jamesdenton has quit IRC00:59
*** priteau has quit IRC01:01
*** eglynn has quit IRC01:06
*** crushil has joined #openstack-nova01:09
*** jamesdenton has joined #openstack-nova01:09
*** browne has quit IRC01:10
*** jamesdenton has quit IRC01:12
*** eglynn has joined #openstack-nova01:15
*** markvoelker has joined #openstack-nova01:15
*** dtp has quit IRC01:17
*** phuongnh has joined #openstack-nova01:18
*** markvoelker has quit IRC01:20
*** mlavalle has quit IRC01:21
*** eglynn has quit IRC01:22
*** mriedem has joined #openstack-nova01:23
*** armax has joined #openstack-nova01:27
*** yamamoto_ has quit IRC01:32
*** ducttape_ has joined #openstack-nova01:34
*** tblakes has quit IRC01:35
*** ducttape_ has quit IRC01:40
openstackgerritgengchc2 proposed openstack/nova-specs master: Fix typos in server-count-api.rst,etc  https://review.openstack.org/41391601:41
*** sree has joined #openstack-nova01:43
*** adrian_otto has quit IRC01:44
*** thorst has quit IRC01:45
*** kevinz has joined #openstack-nova01:46
*** sree has quit IRC01:47
*** gouthamr has quit IRC01:48
*** gouthamr has joined #openstack-nova01:49
*** claudiub has quit IRC01:54
*** kevinz has quit IRC01:57
*** kevinz has joined #openstack-nova01:57
*** crushil has quit IRC01:58
*** cloud-liang has joined #openstack-nova02:03
*** cloud-liang has quit IRC02:04
*** ducttape_ has joined #openstack-nova02:05
*** browne has joined #openstack-nova02:08
*** rfolco has quit IRC02:09
*** kevinz has quit IRC02:10
*** kevinz has joined #openstack-nova02:11
*** smatzek has quit IRC02:12
*** cloud-liang has joined #openstack-nova02:13
*** yamamoto_ has joined #openstack-nova02:14
*** gongysh has joined #openstack-nova02:16
*** markvoelker has joined #openstack-nova02:17
*** kaisers_ has joined #openstack-nova02:17
*** jamesdenton has joined #openstack-nova02:19
*** adrian_otto has joined #openstack-nova02:20
*** kaisers_ has quit IRC02:22
*** markvoelker has quit IRC02:23
*** kevinz has quit IRC02:23
*** unicell has quit IRC02:24
*** kevinz has joined #openstack-nova02:24
*** diga has quit IRC02:25
*** _ducttape_ has joined #openstack-nova02:28
openstackgerritAlex Xu proposed openstack/nova-specs master: Fix the typo in the resource provider traits spec  https://review.openstack.org/44406502:28
*** tbachman has quit IRC02:28
alex_xudansmith: thanks for the review, and the typo fix ^02:29
*** adrian_otto has quit IRC02:29
*** ducttape_ has quit IRC02:30
*** kevinz has quit IRC02:36
*** raunak has quit IRC02:37
*** kevinz has joined #openstack-nova02:37
*** fragatin_ has joined #openstack-nova02:38
*** CristinaPauna has quit IRC02:40
*** sudipto has joined #openstack-nova02:41
*** youzp has joined #openstack-nova02:41
*** fragatina has quit IRC02:42
*** fragatin_ has quit IRC02:42
*** browne has quit IRC02:50
Qimingalex_xu, morning02:51
alex_xuQiming: morning too02:51
Qimingalex_xu, about the returning of user_data in OS-EXT-SERV-ATTR:user_data02:52
alex_xuQiming: yea02:52
Qimingalex_xu, why didn't we just return it in user_data?02:52
alex_xuQiming: you mean the prefix?02:53
Qimingthe API doc says this: OS-EXT-SERV-ATTR:user_databodystring02:53
QimingThe user_data the instance was created with.02:53
QimingNew in version 2.302:53
Qimingalex_xu, it is creating some problems when we try to deal with it in openstacksdk02:53
*** yingjun has quit IRC02:54
Qimingalex_xu, when we are creating an instance, we pass in user_data, when we receive a response from CREATE or GET, we have to parse it with a prefex02:54
Qimingalex_xu, it is not like the other attributes, kernel_id, root_device_name, ramdisk_id, reservation_id, launch_index, and hostname02:55
alex_xuQiming: yea, that is really a old style. for now, any new added field without prefix any more02:55
Qimingall the others can be treated as extended server attributes, except for user_data02:55
alex_xubut for back-compatible, we can't remove the prefix directly. at least, it needs a microversion02:56
gmannQiming:  yea as alex_xu mentioned, they were old notation which we have to keep supporting02:56
Qimingso, a microversion to change it (user_data) back is acceptable?02:56
gmannbut can you access kernel_id without prefix in server GET ?02:56
Qiminggmann, no02:57
gmannok02:57
Qimingkernel_id didn't come from user request, users won't expect that02:57
*** priteau has joined #openstack-nova02:57
*** Apoorva has quit IRC02:57
alex_xuQiming: if just change a prefix for a field, i'm not sure it is worth to do that02:57
gmannQiming: ok on create things you  mean, got it02:57
gmannbecause GET server has all those with prefix like OS-EXT-SERV-ATTR:kernel_id02:58
Qimingsame question for 'personality', which is also an "input only" attribute02:58
Qimingonce provided to create/POST, we get no chance to see it again, :)02:58
alex_xuQiming: personality is a field plan to remove02:58
Qiminggmann, for attributes "added" at server side, it is okay to add a prefix, but for 'user_data', it is weird that its name has changed when I'm retrieving it02:59
*** jamesdenton has quit IRC03:00
gmannalex_xu:  yea and that was the reason to plan remove03:00
Qimingya, removal might be the right thing to do03:00
Qiminganyway, 'personality' is not heavily used like 'user_data' AFAICT03:01
*** priteau has quit IRC03:02
Qimingin the same spirit, maybe 'host_status' for example should have a prefix, :)03:02
*** _ducttape_ has quit IRC03:02
*** sudipto has quit IRC03:03
gmannQiming: yea those are not consistent actually due to legacy v2 way was like that and in v2.1 we try to do without prefix as no more extensions03:04
*** yuntongjin has joined #openstack-nova03:04
alex_xuQiming: for any new field, we won't add prefix03:04
alex_xugmann: yea03:04
Qimingalex_xu, I don't think we should make the prefix decision based on whether it is 'new' or not. Instead, we may want to make the server attributes naming consistent.03:05
alex_xuQiming: yes, the 'new' one, i mean new in the request also03:06
Qimingya, that makes sense. I am just trying to raise some awareness about consistency03:06
alex_xuQiming: yea, agree with you, that should consistent03:07
*** asettle has quit IRC03:07
Qimingwould it be on your agenda to remove the prefix from user_data in the response body?03:08
*** andymccr has quit IRC03:08
gmann but things is we cannot change old one (which are with prefix) without microversion and still for older version they will be with prefix03:08
gmannalex_xu: we can do for all those in single version bump? what you say, worthy to microversion that?03:08
Qimingyes, I'm 100% with you on the requirement to bump microversion03:08
alex_xugmann: yea, i think that also03:09
Qiminggreat, if there is a plan to "fix" it at server side, I will abandon the fix in SDK -- https://review.openstack.org/#/c/443036/03:09
*** adrian_otto has joined #openstack-nova03:10
gmannQiming: still we fix on nova side you need that fix on sdk to support the older version request03:10
*** esberglu has joined #openstack-nova03:10
*** lucasxu has joined #openstack-nova03:10
alex_xuyes, so you still have same problem on the old version03:11
Qiminggmann, SDK has its own problems to fix, but I hope a full-fleged microversion support will be in place soon03:11
alex_xutry to know what kind of try you got in the openstacksdk03:11
alex_xuQiming: so SDK still need to support older version of nova api, right? so it still need to care about the user_data field with prefix03:12
gmannyea03:12
*** gjayavelu has quit IRC03:12
*** vladikr has quit IRC03:13
gmannsdk fix is anyways needed irrespective of nova fix that in new version or not03:13
Qimingyes, but that will become a different topic. there are still other fields that are "request-only", "response-only", "valid-in-both-direction-but-RENAMED", :_03:13
*** thorst has joined #openstack-nova03:13
Qiminggmann, exactly03:13
gmannalex_xu: i think its little bit overhead to microversion that as we have lot of other inconsistency like CamelCase things etc but if we do we should do in one shot but again that can change lot of APIs :)03:14
alex_xugmann: yes, but we said we won't fix the CamelCase things03:15
gmannyeaaa03:15
*** esberglu has quit IRC03:15
gmann+1 on that, it unnecessary and overhead to fix those03:16
*** raunak has joined #openstack-nova03:16
*** wangqun has joined #openstack-nova03:16
alex_xuso...remove the prefix, Qiming will get benefit, and also have people already make their code aware the prefix, will get overhead to fix that.03:17
*** wangqun has quit IRC03:17
*** thorst has quit IRC03:18
alex_xuso...the question will be, how much benefit we got from removing prefix, is that enough for other people change their client code to remove fix?03:18
*** wangqun has joined #openstack-nova03:18
gmannyea thats true, its overhead for existing users and benefit for new03:19
*** gjayavelu has joined #openstack-nova03:19
alex_xuQiming: anyway, my first todo is getting clear what kind of trouble you got in the sdk03:21
Qimingokay03:22
*** jamesdenton has joined #openstack-nova03:22
*** sdague has joined #openstack-nova03:23
*** gjayavelu has quit IRC03:23
*** yingjun has joined #openstack-nova03:26
*** marst has joined #openstack-nova03:27
*** ducttape_ has joined #openstack-nova03:30
*** lucasxu has quit IRC03:31
*** jamesdenton has quit IRC03:31
gmannyea +103:32
openstackgerritDan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db  https://review.openstack.org/43975403:33
openstackgerritDan Smith proposed openstack/nova master: Teach HostAPI about cells  https://review.openstack.org/44216203:33
openstackgerritDan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info  https://review.openstack.org/43989103:33
openstackgerritDan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety  https://review.openstack.org/44317403:33
openstackgerritDan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList  https://review.openstack.org/44329203:33
openstackgerritDan Smith proposed openstack/nova master: Make server_groups determine deleted-ness from InstanceMappingList  https://review.openstack.org/44329303:33
openstackgerritDan Smith proposed openstack/nova master: Remove Mitaka-era service version check  https://review.openstack.org/44286103:33
openstackgerritDan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall  https://review.openstack.org/44273603:33
*** takashin has joined #openstack-nova03:34
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/44335703:35
*** jamesdenton has joined #openstack-nova03:37
*** links has joined #openstack-nova03:38
*** fragatina has joined #openstack-nova03:38
*** lucasxu has joined #openstack-nova03:40
*** adrian_otto has quit IRC03:40
*** crushil has joined #openstack-nova03:41
*** fragatina has quit IRC03:41
*** fragatina has joined #openstack-nova03:41
*** fragatin_ has joined #openstack-nova03:42
*** adrian_otto has joined #openstack-nova03:42
*** adrian_otto has quit IRC03:43
*** yamahata has quit IRC03:45
*** fragatina has quit IRC03:46
*** ducttape_ has quit IRC03:49
*** guchihiro has joined #openstack-nova03:49
*** iceyao has quit IRC03:50
*** iceyao has joined #openstack-nova03:50
*** ssurana has quit IRC03:52
*** nmathew has joined #openstack-nova03:57
*** catintheroof has quit IRC03:57
*** catintheroof has joined #openstack-nova03:58
*** catintheroof has quit IRC03:59
*** ducttape_ has joined #openstack-nova03:59
*** salv-orlando has joined #openstack-nova04:05
*** iceyao has quit IRC04:06
*** yuntongjin has quit IRC04:06
*** nicolasbock has quit IRC04:07
*** salv-orlando has quit IRC04:09
*** armax has quit IRC04:10
*** esberglu has joined #openstack-nova04:10
*** adrian_otto has joined #openstack-nova04:11
*** ducttape_ has quit IRC04:13
*** Sukhdev has joined #openstack-nova04:14
*** thorst has joined #openstack-nova04:14
*** sudipto has joined #openstack-nova04:14
*** esberglu has quit IRC04:15
*** david-lyle has quit IRC04:16
*** thorst has quit IRC04:18
*** gcb has joined #openstack-nova04:19
*** lucasxu has quit IRC04:24
*** iceyao has joined #openstack-nova04:26
*** ssurana has joined #openstack-nova04:26
*** udesale has joined #openstack-nova04:29
*** iceyao has quit IRC04:30
*** kaisers_ has joined #openstack-nova04:33
*** sdague has quit IRC04:35
*** mdnadeem has joined #openstack-nova04:36
takashintox -e pep804:36
*** adrian_otto1 has joined #openstack-nova04:38
*** kaisers_ has quit IRC04:38
*** crushil has quit IRC04:38
*** adrian_otto has quit IRC04:40
*** fragatin_ has quit IRC04:42
*** psachin has joined #openstack-nova04:42
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895504:43
*** wangqun has quit IRC04:45
*** wangqun has joined #openstack-nova04:46
*** fragatina has joined #openstack-nova04:48
*** zz_dimtruck is now known as dimtruck04:49
*** iceyao has joined #openstack-nova04:57
*** adrian_otto1 has quit IRC04:57
*** priteau has joined #openstack-nova04:58
*** priteau has quit IRC05:03
*** jamesdenton has quit IRC05:03
*** Sukhdev has quit IRC05:05
*** salv-orlando has joined #openstack-nova05:05
*** ayogi has joined #openstack-nova05:09
*** yassine has quit IRC05:10
*** salv-orlando has quit IRC05:10
*** yassine has joined #openstack-nova05:12
*** amotoki has joined #openstack-nova05:13
*** thorst has joined #openstack-nova05:15
*** thorst has quit IRC05:19
*** markvoelker has joined #openstack-nova05:20
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896405:22
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004205:23
openstackgerritTakashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host  https://review.openstack.org/41492605:23
*** kaisers_ has joined #openstack-nova05:23
*** yuntongjin has joined #openstack-nova05:23
*** markvoelker has quit IRC05:25
*** kaisers_ has quit IRC05:25
*** iceyao has quit IRC05:25
*** gouthamr has quit IRC05:29
*** ratailor has joined #openstack-nova05:32
*** diga has joined #openstack-nova05:32
*** yingjun has quit IRC05:33
*** prateek has joined #openstack-nova05:39
*** iceyao has joined #openstack-nova05:40
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: List/show all server migration types  https://review.openstack.org/40723705:40
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: List/show all server migration types  https://review.openstack.org/40723705:40
*** mriedem has quit IRC05:44
*** sandanar has joined #openstack-nova05:48
*** esberglu has joined #openstack-nova05:55
*** yuntongjin has quit IRC05:55
*** yuntongjin has joined #openstack-nova05:56
*** yamahata has joined #openstack-nova05:56
*** mtanino has joined #openstack-nova05:57
*** sridharg has joined #openstack-nova05:58
*** esberglu has quit IRC05:59
*** udesale__ has joined #openstack-nova06:00
*** udesale has quit IRC06:02
*** cloud-liang has quit IRC06:02
*** salv-orlando has joined #openstack-nova06:06
*** unicell has joined #openstack-nova06:08
*** jaosorior has joined #openstack-nova06:09
*** salv-orlando has quit IRC06:11
*** Shunli has joined #openstack-nova06:12
openstackgerritShunli Zhou proposed openstack/nova master: Add populate_retry to schedule_and_build_instances  https://review.openstack.org/44410606:12
*** unicell has quit IRC06:13
*** unicell1 has joined #openstack-nova06:13
*** crushil has joined #openstack-nova06:14
*** ducttape_ has joined #openstack-nova06:14
*** ducttape_ has quit IRC06:19
*** udesale has joined #openstack-nova06:19
*** markvoelker has joined #openstack-nova06:21
*** udesale__ has quit IRC06:21
*** markvoelker has quit IRC06:25
*** ltomasbo|away is now known as ltomasbo06:32
*** phuongnh has quit IRC06:32
*** phuongnh has joined #openstack-nova06:33
*** diga has quit IRC06:34
*** rcernin has joined #openstack-nova06:35
*** nkrinner_afk is now known as nkrinner06:37
*** markus_z has joined #openstack-nova06:39
*** markvoelker has joined #openstack-nova06:44
*** satyar has joined #openstack-nova06:44
*** amotoki has quit IRC06:50
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix keypair API parameters  https://review.openstack.org/42062606:54
*** markvoelker has quit IRC06:54
*** arne_r has joined #openstack-nova06:55
*** esberglu has joined #openstack-nova06:55
openstackgerritWei Cao proposed openstack/nova master: vmware: Change the VM Create spec  https://review.openstack.org/44411906:55
*** slaweq has quit IRC06:57
*** yingjun has joined #openstack-nova06:58
*** priteau has joined #openstack-nova06:59
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix keypair API parameters  https://review.openstack.org/42062606:59
*** esberglu has quit IRC07:00
*** priteau has quit IRC07:03
*** slaweq has joined #openstack-nova07:05
*** salv-orlando has joined #openstack-nova07:07
*** gjayavelu has joined #openstack-nova07:07
*** andreas_s has joined #openstack-nova07:08
*** raunak has quit IRC07:09
*** raunak has joined #openstack-nova07:10
*** raunak has quit IRC07:10
*** raunak has joined #openstack-nova07:11
*** raunak has quit IRC07:11
*** raunak has joined #openstack-nova07:12
*** salv-orlando has quit IRC07:12
*** raunak has quit IRC07:12
*** salv-orlando has joined #openstack-nova07:12
*** raunak has joined #openstack-nova07:12
*** raunak has quit IRC07:13
*** moshele has joined #openstack-nova07:13
*** raunak has joined #openstack-nova07:13
*** raunak has quit IRC07:13
*** thorst has joined #openstack-nova07:16
openstackgerritTakashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases  https://review.openstack.org/39543607:21
openstackgerritTakashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases  https://review.openstack.org/39473807:21
*** thorst has quit IRC07:21
*** amotoki has joined #openstack-nova07:23
*** salv-orl_ has joined #openstack-nova07:23
*** salv-orlando has quit IRC07:26
*** kaisers_ has joined #openstack-nova07:26
*** karimb has joined #openstack-nova07:29
*** kaisers_ has quit IRC07:30
*** abalutoiu_ has quit IRC07:30
*** guchihiro has quit IRC07:32
*** fragatina has quit IRC07:36
*** xek has quit IRC07:37
*** xek has joined #openstack-nova07:37
*** xek has left #openstack-nova07:38
*** edmondsw has joined #openstack-nova07:39
*** tesseract has joined #openstack-nova07:42
*** edmondsw has quit IRC07:44
*** diga has joined #openstack-nova07:49
*** esberglu has joined #openstack-nova07:50
*** gszasz has joined #openstack-nova07:51
*** Guest27337 has joined #openstack-nova07:51
*** mlakat has quit IRC07:51
*** mlakat has joined #openstack-nova07:53
*** mlakat has quit IRC07:54
*** karimb has quit IRC07:54
*** esberglu has quit IRC07:54
*** gjayavelu has quit IRC07:55
*** takashin has quit IRC07:57
*** sudipto has quit IRC08:00
*** mlakat has joined #openstack-nova08:00
*** ssurana has quit IRC08:01
*** yuntongjin has quit IRC08:01
*** gszasz has quit IRC08:03
*** lpetrut has joined #openstack-nova08:03
*** yuntongjin has joined #openstack-nova08:06
openstackgerritMichael  Henkel proposed openstack/nova master: Implements: blueprint opencontrail-dpdk-support  https://review.openstack.org/44118308:07
*** phuongnh has quit IRC08:12
*** lpetrut has quit IRC08:12
*** nmathew- has joined #openstack-nova08:13
*** nmathew has quit IRC08:16
*** moshele has quit IRC08:17
*** thorst has joined #openstack-nova08:17
*** nmathew- has quit IRC08:18
*** eglynn has joined #openstack-nova08:19
*** nmathew- has joined #openstack-nova08:19
*** salv-orl_ has quit IRC08:20
*** thorst has quit IRC08:22
*** eglynn has quit IRC08:23
*** eglynn has joined #openstack-nova08:23
openstackgerritTomi Juvonen proposed openstack/nova-specs master: Planned host maintenance  https://review.openstack.org/31051008:23
*** nmathew- has quit IRC08:24
*** aarefiev_afk is now known as aarefiev08:24
*** bauzas is now known as bauwser08:26
bauwsergood morning Nova08:26
*** haplo37 has quit IRC08:27
arne_rgood morning bauwser :)08:27
*** eglynn has quit IRC08:28
*** yingjun has quit IRC08:33
*** jaosorior is now known as jaosorior_breakf08:33
*** bkopilov has quit IRC08:36
*** haplo37 has joined #openstack-nova08:36
*** salv-orlando has joined #openstack-nova08:36
*** nmathew has joined #openstack-nova08:37
*** jpena|off is now known as jpena08:43
*** gszasz has joined #openstack-nova08:45
*** karimb has joined #openstack-nova08:46
*** crushil has quit IRC08:48
*** jaosorior_breakf is now known as jaosorior08:48
*** gszasz has quit IRC08:50
*** amoralej|off is now known as amoralej08:51
*** zioproto has joined #openstack-nova08:52
*** jpena is now known as jpena|off08:56
openstackgerritHuan Xie proposed openstack/nova master: WIP: add linux bridge qbr for live migration on XenServer  https://review.openstack.org/44416308:59
*** kevinz has quit IRC08:59
*** priteau has joined #openstack-nova08:59
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-nova09:00
*** jpena|off is now known as jpena09:03
*** priteau has quit IRC09:04
*** yamamoto_ has quit IRC09:04
*** diga has quit IRC09:10
*** diga has joined #openstack-nova09:11
*** sandanar has quit IRC09:17
*** thorst has joined #openstack-nova09:18
*** ralonsoh has joined #openstack-nova09:22
*** priteau has joined #openstack-nova09:22
*** thorst has quit IRC09:23
*** nmathew has quit IRC09:27
*** Shunli has quit IRC09:27
*** haplo37 has quit IRC09:33
*** diga has quit IRC09:34
*** lucas-afk is now known as lucasagomes09:34
*** efoley__ has joined #openstack-nova09:34
*** esberglu has joined #openstack-nova09:35
*** efoley_ has joined #openstack-nova09:36
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery  https://review.openstack.org/43595409:37
*** yuntongjin has quit IRC09:38
*** esberglu has quit IRC09:39
*** dimtruck is now known as zz_dimtruck09:39
*** zz_dimtruck is now known as dimtruck09:39
*** efoley__ has quit IRC09:39
*** udesale has quit IRC09:41
*** haplo37 has joined #openstack-nova09:41
*** rdo has quit IRC09:46
*** sudipto has joined #openstack-nova09:47
*** derekh has joined #openstack-nova09:47
*** rdo has joined #openstack-nova09:48
*** youzp has quit IRC09:49
*** dimtruck is now known as zz_dimtruck09:49
openstackgerritMichael  Henkel proposed openstack/nova master: Implements: blueprint opencontrail-dpdk-support  https://review.openstack.org/44118309:49
*** awaugama has joined #openstack-nova09:50
*** ducnc has quit IRC09:54
*** ducnc has joined #openstack-nova09:55
*** rdo has quit IRC09:56
*** wangqun has quit IRC09:58
*** moshele has joined #openstack-nova09:58
*** rdo has joined #openstack-nova09:58
*** andymccr has joined #openstack-nova10:03
*** asettle has joined #openstack-nova10:03
*** edmondsw has joined #openstack-nova10:04
*** efoley__ has joined #openstack-nova10:06
*** aloga_ has joined #openstack-nova10:06
*** kevinz has joined #openstack-nova10:07
*** claudiub has joined #openstack-nova10:08
*** edmondsw has quit IRC10:09
*** zhurong has quit IRC10:10
*** efoley_ has quit IRC10:10
*** moshele has quit IRC10:15
*** yuntongjin has joined #openstack-nova10:15
*** thorst has joined #openstack-nova10:19
*** kevinz has quit IRC10:20
*** ducnc has quit IRC10:21
*** abalutoiu_ has joined #openstack-nova10:22
*** yamamoto has joined #openstack-nova10:22
*** kaisers_ has joined #openstack-nova10:23
*** thorst has quit IRC10:23
*** kaisers_ has quit IRC10:23
*** kaisers_ has joined #openstack-nova10:24
*** sambetts|afk is now known as sambetts10:25
*** yamamoto has quit IRC10:26
*** gongysh has quit IRC10:26
openstackgerritGhanshyam Mann proposed openstack/nova master: Remove duplicate attributes in sample files  https://review.openstack.org/44419910:26
gmannalex_xu: oomichi ^^ easy one10:27
*** tovin07_ has quit IRC10:27
*** tovin07 has quit IRC10:27
*** satyar has quit IRC10:31
*** openstackgerrit has quit IRC10:33
*** iceyao has quit IRC10:33
*** cdent has joined #openstack-nova10:33
*** esberglu has joined #openstack-nova10:34
*** kaisers_ has quit IRC10:38
kashyapmdbooth: Hello there, wonder if you have time / energy today to make any comment here -- https://review.openstack.org/#/c/246910/10:38
kashyapI know this involves Neutron voodoo, so if you shy away from it with a "meh", can understand :-)10:38
kashyapJust this is causing a nasty interaction bug between Nova, Neutron, libvirt, QEMU and Virtio code10:39
*** esberglu has quit IRC10:39
*** zz_dimtruck is now known as dimtruck10:40
*** yamahata has quit IRC10:40
mdboothkashyap: Eww10:43
kashyap:-)10:43
kashyapSorry10:43
mdboothTo me that just says the flow is wrong and we should be casting10:43
mdboothWith callbacks10:43
mdboothif necessary10:43
mdboothkashyap: I don't feel I have the bandwidth to jump into that meaningfully right now, I'm afraid10:44
*** karimb has quit IRC10:44
kashyapmdbooth: No worries at all10:44
mdboothAt first glance I'm not a fan, but I don't have an alternative10:44
kashyapYeah, I hear you.  It's just feels (to me) impenetrable sometimes, with so many moving parts10:45
markus_zWhat's our current policy for images in our docs? I see different types of image sources (*.dia, *.graphml, *odp) and also image generators like "graphviz".10:46
mdboothThere are non-neutron reasons for not using a call in that flow10:46
*** dimtruck is now known as zz_dimtruck10:50
*** zz_dimtruck is now known as dimtruck10:53
*** yuntongjin has quit IRC10:56
*** salv-orlando has quit IRC11:02
*** dimtruck is now known as zz_dimtruck11:02
*** salv-orlando has joined #openstack-nova11:02
*** salv-orlando has quit IRC11:02
gmannsfinucan: if you can fix missing 's' on this - https://review.openstack.org/#/c/430914/411:03
*** cdent has quit IRC11:05
*** cdent has joined #openstack-nova11:05
*** efoley_ has joined #openstack-nova11:06
*** youzp has joined #openstack-nova11:06
*** nicolasbock has joined #openstack-nova11:07
*** karimb has joined #openstack-nova11:08
*** efoley__ has quit IRC11:09
*** youzp has quit IRC11:10
*** purplerbot has quit IRC11:10
*** purplerbot has joined #openstack-nova11:10
*** salv-orlando has joined #openstack-nova11:11
*** thorst has joined #openstack-nova11:20
*** thorst has quit IRC11:24
*** yamamoto has joined #openstack-nova11:26
*** esberglu has joined #openstack-nova11:30
*** efoley__ has joined #openstack-nova11:32
*** openstackgerrit has joined #openstack-nova11:34
openstackgerritHuan Xie proposed openstack/nova master: WIP: Create ovs port with other params together  https://review.openstack.org/44423011:34
*** yamamoto has quit IRC11:35
*** efoley_ has quit IRC11:35
*** links has quit IRC11:37
*** esberglu has quit IRC11:38
*** gongysh has joined #openstack-nova11:41
*** cdent has quit IRC11:43
*** rmart04 has joined #openstack-nova11:47
*** abhishek_k has quit IRC11:48
*** ratailor has quit IRC11:53
*** zz_dimtruck is now known as dimtruck11:53
*** efoley_ has joined #openstack-nova11:53
*** links has joined #openstack-nova11:54
*** youzp has joined #openstack-nova11:56
*** iceyao has joined #openstack-nova11:56
*** efoley__ has quit IRC11:57
*** smatzek has joined #openstack-nova11:58
*** dimtruck is now known as zz_dimtruck12:03
*** aysyd has joined #openstack-nova12:03
*** fengzhr has quit IRC12:03
openstackgerritIldiko Vancsa proposed openstack/nova master: Deprecate the Cinder API v2 support  https://review.openstack.org/44425312:04
andreafhello nova folks, https://bugs.launchpad.net/nova/+bug/1532809 pop up again in the gate, it's now the top recheck issue: http://status.openstack.org/elastic-recheck/12:04
openstackLaunchpad bug 1532809 in OpenStack Compute (nova) liberty "Gate failures when DHCP lease cannot be acquired" [High,In progress] - Assigned to Sean Dague (sdague)12:04
andreafuhm - it's only on openstack/networking-odl12:07
*** hshiina has joined #openstack-nova12:09
*** phuongnh has joined #openstack-nova12:13
*** sileht has quit IRC12:17
alex_xugmann: thanks, just done12:17
*** sileht has joined #openstack-nova12:18
*** sileht has quit IRC12:18
*** sileht has joined #openstack-nova12:18
*** bvanhav_ has joined #openstack-nova12:18
*** satyar has joined #openstack-nova12:18
*** ducttape_ has joined #openstack-nova12:19
*** lucasagomes is now known as lucas-hungry12:21
*** rfolco has joined #openstack-nova12:22
*** cdent has joined #openstack-nova12:22
*** steveq has joined #openstack-nova12:23
*** ducttape_ has quit IRC12:24
*** yamamoto has joined #openstack-nova12:24
*** Muranishi has joined #openstack-nova12:25
*** CristinaPauna has joined #openstack-nova12:26
*** Muranishi has quit IRC12:26
*** yamamoto has quit IRC12:26
*** yamamoto has joined #openstack-nova12:27
*** yamamoto has quit IRC12:27
steveqhi all, I want to test reproducethis bug, https://bugs.launchpad.net/nova/+bug/1481262, is there any way where I can create 100K instances without invoking boot api?12:27
openstackLaunchpad bug 1481262 in OpenStack Compute (nova) ""nova usage" taking too much time with many VMs in database" [Medium,In progress] - Assigned to Guillaume Espanel (guillaume-espanel)12:27
steveqs/test reproducethis/reproduce this12:27
*** youzp has quit IRC12:30
*** catintheroof has joined #openstack-nova12:34
*** kaisers_ has joined #openstack-nova12:38
*** ayogi has quit IRC12:39
*** thorst has joined #openstack-nova12:43
*** kaisers_ has quit IRC12:43
*** prateek has quit IRC12:45
*** fragatina has joined #openstack-nova12:46
*** salv-orlando has quit IRC12:48
*** artom has quit IRC12:51
*** databus23_ has joined #openstack-nova12:53
*** zz_dimtruck is now known as dimtruck12:54
avolkovhttps://docs.openstack.org/ocata/config-reference/compute/config-options.html looks outdated for live_migration_uri option12:55
avolkovcould somebody point me from where it's generated?12:56
*** bhagyashri_s has quit IRC12:58
*** bhagyashris has quit IRC12:59
*** salv-orlando has joined #openstack-nova13:00
*** liverpooler has joined #openstack-nova13:00
*** ducttape_ has joined #openstack-nova13:00
markus_zavolkov: It gets pulled from these string here. https://github.com/openstack/nova/blob/master/nova/conf/libvirt.py13:00
*** andreas_s has quit IRC13:00
*** markus_z has quit IRC13:01
*** sileht has quit IRC13:03
*** sileht has joined #openstack-nova13:03
*** sileht has quit IRC13:03
*** sileht has joined #openstack-nova13:03
*** sdague has joined #openstack-nova13:03
*** dimtruck is now known as zz_dimtruck13:04
avolkovmarkus_z: ok, maybe it takes only first paragraph for description, but live_migration_scheme is missed in docs but exists in libvirt.py, do you know why?13:04
*** sileht has quit IRC13:08
*** sileht has joined #openstack-nova13:08
*** sileht has quit IRC13:08
*** gcb has quit IRC13:08
*** sileht has joined #openstack-nova13:10
*** lucas-hungry is now known as lucasagomes13:10
*** esberglu has joined #openstack-nova13:12
*** esberglu has quit IRC13:13
*** yuntongjin has joined #openstack-nova13:13
*** esberglu has joined #openstack-nova13:13
*** des_consolado has quit IRC13:14
*** edleafe is now known as figleaf13:15
*** sudipto has quit IRC13:16
*** links has quit IRC13:17
*** esberglu has quit IRC13:17
*** READ10 has joined #openstack-nova13:18
*** lpetrut has joined #openstack-nova13:19
*** salv-orl_ has joined #openstack-nova13:23
*** edmondsw has joined #openstack-nova13:23
*** salv-orl_ has quit IRC13:24
*** kfarr has quit IRC13:24
*** moshele has joined #openstack-nova13:24
*** salv-orl_ has joined #openstack-nova13:25
*** phuongnh has quit IRC13:25
*** salv-orlando has quit IRC13:26
*** kaisers_ has joined #openstack-nova13:27
*** yamamoto has joined #openstack-nova13:27
*** links has joined #openstack-nova13:31
*** gszasz has joined #openstack-nova13:31
*** kevinz has joined #openstack-nova13:32
*** gongysh has quit IRC13:32
*** moshele has quit IRC13:32
*** dave-mccowan has joined #openstack-nova13:33
*** yamamoto has quit IRC13:33
*** yamamoto has joined #openstack-nova13:34
*** READ10 has quit IRC13:34
*** READ10 has joined #openstack-nova13:37
*** ujjain has quit IRC13:39
*** links has quit IRC13:39
*** kaisers_ has quit IRC13:46
*** tblakes has joined #openstack-nova13:50
*** ujjain has joined #openstack-nova13:50
*** ujjain has quit IRC13:50
*** ujjain has joined #openstack-nova13:50
*** zz_dimtruck is now known as dimtruck13:55
openstackgerritIldiko Vancsa proposed openstack/nova master: Deprecate the Cinder API v2 support  https://review.openstack.org/44425313:57
*** bvanhav__ has joined #openstack-nova13:58
*** bvanhav__ is now known as bvanhav13:59
*** aloga_ has quit IRC13:59
*** rmart04 has quit IRC13:59
*** tuanluong has quit IRC14:01
*** bvanhav_ has quit IRC14:01
*** dimtruck is now known as zz_dimtruck14:05
*** salv-orl_ has quit IRC14:08
*** jamesdenton has joined #openstack-nova14:11
*** yuntongjin has quit IRC14:11
*** yuntongjin1 has joined #openstack-nova14:11
*** mriedem has joined #openstack-nova14:11
*** gszasz has quit IRC14:12
*** zz_dimtruck is now known as dimtruck14:12
*** xyang1 has joined #openstack-nova14:12
*** baoli has joined #openstack-nova14:13
*** yuntongjin1 has quit IRC14:14
mriedemo/14:15
*** baoli has quit IRC14:16
*** Cristina_ has joined #openstack-nova14:16
*** cdent_ has joined #openstack-nova14:17
*** nkrinner is now known as nkrinner_afk14:17
*** cdent has quit IRC14:17
*** cdent_ is now known as cdent14:17
Dinesh_Bhormriedem: Hi, if you get time could you please check this https://review.openstack.org/#/c/441750/?14:18
*** ducttape_ has quit IRC14:18
*** salv-orlando has joined #openstack-nova14:18
*** hshiina has quit IRC14:18
*** salv-orlando has quit IRC14:20
*** kaisers_ has joined #openstack-nova14:20
*** CristinaPauna has quit IRC14:20
*** sudipto has joined #openstack-nova14:21
*** amoralej is now known as amoralej|lunch14:21
*** abalutoiu_ has quit IRC14:22
*** Dinesh_Bhor has quit IRC14:23
*** baoli has joined #openstack-nova14:24
*** sdague has quit IRC14:24
*** tbachman has joined #openstack-nova14:24
*** kaisers_ has quit IRC14:25
*** gouthamr has joined #openstack-nova14:26
*** mdnadeem has quit IRC14:27
*** esberglu has joined #openstack-nova14:27
*** smatzek has quit IRC14:29
*** jdillaman has joined #openstack-nova14:29
mriedemdansmith: i think we're happy with this now https://review.openstack.org/#/c/437222/ - the notification delete patch14:35
*** jaypipes has joined #openstack-nova14:38
*** jaypipes is now known as leakypipes14:39
*** Andrew_jedi has joined #openstack-nova14:39
*** Andrew_jedi has left #openstack-nova14:39
*** ioni has joined #openstack-nova14:39
*** esberglu_ has joined #openstack-nova14:39
*** kevinz has quit IRC14:40
ionihello guys14:40
ionisorry for bothering you, i have noticed something and I don't know if is a bug in nova or a bug in documentation14:41
ionihttps://docs.openstack.org/ocata/config-reference/compute/config-options.html14:41
ioniis mentioned that notify_on_state_change is in [DEFAULT]14:41
ioniwhile looking at the logs, i have noticed that notifications.notify_on_state_change =  None14:42
ionithe only way to set this up is to create a new [notifications] section, having notify_on_state_change = vm_and_task_state14:42
ionibug in nova or bug in documentation?14:42
*** esberglu has quit IRC14:42
*** Andrew_jedi has joined #openstack-nova14:43
*** crushil has joined #openstack-nova14:44
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery  https://review.openstack.org/43595414:45
*** kaisers_ has joined #openstack-nova14:46
*** arne_r has quit IRC14:46
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Websockify security proxy framework  https://review.openstack.org/44323414:47
*** leakypipes has quit IRC14:49
*** yamamoto has quit IRC14:50
*** hongbin has joined #openstack-nova14:50
*** yamamoto has joined #openstack-nova14:50
*** Andrew_jedi has quit IRC14:51
*** smatzek has joined #openstack-nova14:52
*** jaypipes has joined #openstack-nova14:54
*** jaypipes is now known as leakypipes14:54
figleafioni: that's a doc bug14:54
figleafare you running ocata?14:55
*** yamamoto has quit IRC14:55
*** kaisers_ has quit IRC14:55
ionifigleaf, yes14:55
*** ducttape_ has joined #openstack-nova14:56
figleafioni: https://github.com/openstack/nova/blob/master/nova/conf/notifications.py#L108-L11014:57
figleafioni: that shows the opts defined above it, including notify_state_on_change, as being part of the notifications_group14:57
mriedemioni: are you sure that it wasn't deprecated in the DEFAULT group in ocata and still pass through to [notifications]?14:57
mriedemwe generally deprecate the group for a release when moving them14:57
mriedemso the old config will work14:58
mriedemso it depends on when using it in DEFAULT was deprecated14:58
ionimriedem, i'm looking at: https://docs.openstack.org/ocata/config-reference/compute/config-options.html14:58
mriedemhttps://docs.openstack.org/developer/nova/ocata/sample_config.html14:58
mriedemyeah, the docs.o.o might be wrong14:58
mriedem^ is generated from the nova repo14:58
ionii don't know what's in the code, i'm just looking at them :D14:58
mriedemDeprecated group/name - [default]/notify_on_state_change14:59
ioniok14:59
mriedemhttps://github.com/openstack/nova/blob/stable/ocata/nova/conf/notifications.py#L3314:59
mriedemso if you have it in [DEFAULT] in ocata it should still work14:59
mriedemyou might get a warning14:59
ioniit doesn't work15:00
mriedemsfinucan: does deprecated_group='default' work for [DEFAULT]?15:00
ionino warning, it defaults to None15:00
mriedemdims_: ^?15:00
openstackgerritDan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db  https://review.openstack.org/43975415:00
openstackgerritDan Smith proposed openstack/nova master: Make os-availability-zones know about cells  https://review.openstack.org/44216315:00
openstackgerritDan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context  https://review.openstack.org/43802315:00
openstackgerritDan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info  https://review.openstack.org/43989115:00
openstackgerritDan Smith proposed openstack/nova master: Add identity helper property to CellMapping  https://review.openstack.org/44403815:00
*** catintheroof has quit IRC15:00
openstackgerritDan Smith proposed openstack/nova master: Add cell field to Destination object  https://review.openstack.org/43802415:00
openstackgerritDan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell  https://review.openstack.org/43802515:00
mriedemioni: let me try something quick in ocata15:01
*** catintheroof has joined #openstack-nova15:01
sfinucanmriedem: I think macsz tried that and it does15:01
mriedemok i'm changing a unit test in ocata that sets it into the 'notifications' group, putting it back in DEFAULT and seeing if that passes15:02
mriedemoslo_config.cfg.NoSuchOptError: no such option notify_on_state_change in group [DEFAULT]15:03
*** Cristina_ is now known as CristinaPauna15:03
*** artom has joined #openstack-nova15:04
mriedemchanging deprecated_group="DEFAULT"15:04
*** david-lyle has joined #openstack-nova15:05
*** dimtruck is now known as zz_dimtruck15:05
ionimriedem, https://paste.xinu.at/VmpI/15:05
ionimriedem, so in the first two lines, i had notify_on_state_change in  [DEFAULT]15:05
*** ayogi has joined #openstack-nova15:06
ionino warning, nothing related to that15:06
*** mlavalle has joined #openstack-nova15:06
mriedemthis doesn't work for me either https://gist.github.com/mriedem/d20978819dbbc28c77f89e118fd4f52f15:06
mriedemi get oslo_config.cfg.NoSuchOptError: no such option notify_on_state_change in group [DEFAULT]15:06
sfinucanmriedem: AFAIK the deprecated_group and deprecated_name properties only affects reading from config files. When accessing it programatically (cfg.CONF) you need to use the current group and name15:06
mriedemmight be a problem in the unit tests15:06
mriedemsfinucan: ah15:07
*** vladikr has joined #openstack-nova15:07
*** Oku_OS is now known as Oku_OS-away15:07
sfinucanwhich, imo, is the correct behavior15:07
* dims_ catching up15:07
mriedemioni: are you able to change nova/conf/notifications.py and change deprecated_group='DEFAULT', restart and see if that works?15:07
mriedemi don't have a devstack in front of me to recreate15:07
ioniok, wait15:08
dims_mriedem : sfinucan : we just merged something that may help in the future - https://review.openstack.org/#/c/357987/15:08
*** david-lyle_ has joined #openstack-nova15:08
*** david-lyle_ has quit IRC15:08
*** david-lyle has quit IRC15:08
*** lucasxu has joined #openstack-nova15:09
*** felipemonteiro_ has joined #openstack-nova15:09
ionimriedem, deprecated_group='default' this line?15:10
mriedemioni: yeah15:10
ioniwith capital DEFAULT15:10
ioniright?15:10
mriedemyes15:10
*** yamahata has joined #openstack-nova15:11
ioni2017-03-10 17:11:26.849 14528 WARNING oslo_config.cfg [req-13263cac-0dca-42bd-9c0a-d2e3cbab71a6 - - - - -] Option "notify_on_state_change" from group "DEFAULT" is deprecated. Use option "notify_on_state_change" from group "notifications".15:11
ioninow it works15:11
*** Oku_OS-away is now known as Oku_OS15:11
mriedemioni: ok, crap, sorry - can you report a bug against nova?15:12
ionishould i submit a patch ?15:12
mriedemhttps://bugs.launchpad.net/nova15:12
mriedemioni: sure, i see we have the same mistake for some console options15:12
mriedemioni: report a bug, submit a patch and then we can work on reviewing and backporting to ocata15:12
*** mtanino has quit IRC15:12
mriedemi'm going to do a stable/ocata release next week, we can get this fix in there15:13
mriedemdims_: ^ seems like a bug in oslo.config if you can specify a deprecated_group that doesn't exist15:13
*** eharney has joined #openstack-nova15:13
mriedemmaybe that's a chicken and egg,15:13
mriedemsince the deprecated group is no longer registered (except the DEFAULT case)15:13
mriedemjohnthetubaguy: were you going to re-propose a spec for pike for this? https://review.openstack.org/#/c/393888/15:14
dims_mriedem : a bit distracted at the moment. bug please?15:15
mriedemforthcoming, i'll point it out when it's up15:15
sfinucanmriedem: Yeah, I reviewed that patch and missed that. Sorry :(15:16
dims_thanks mriedem !15:16
sfinucanI'll try see if we can fix this in oslo.config, if that's ok with you dims_15:16
ionimriedem, report here: https://bugs.launchpad.net/nova/+bug/167184715:17
openstackLaunchpad bug 1671847 in OpenStack Compute (nova) "Incorect set deprecated flag for notify_on_state_change" [Undecided,New]15:17
ionimriedem, patch coming up15:17
sfinucantonyb, lyarwood: Could ye stick this on your review queues? https://review.openstack.org/#/c/444306/15:18
sfinucanI'm seeing the issue in openstack-manuals for stable/ocata still. Not a huge problem but it's very annoying15:18
cdentleakypipes: is https://review.openstack.org/#/c/442718/ -w cuz the tests aren't done, or are you reconsidering the idea? (I'm gathering things for this week's updated)15:18
*** mlavalle has quit IRC15:19
*** mlavalle has joined #openstack-nova15:19
cdentcounting number of available mempages seems ... odd?15:19
mriedemioni: thanks15:19
*** lucasxu has quit IRC15:20
mriedemcdent: also, someone needs to re-propose https://review.openstack.org/#/c/386710/ for pike and update it to mention the discussion at the ptg about not supporting multiple parents,15:21
cdentmriedem: ✔15:21
mriedemdoesn't need great detail, but that seemed like a long whiteboarding session and if that was the outcome we should probably capture in the spec with the reason why15:21
*** shaner has quit IRC15:21
lyarwoodsfinucan: ack, looking15:22
*** ayogi has quit IRC15:22
sfinucanlyarwood: (y)15:23
*** jaosorior has quit IRC15:24
*** shaner has joined #openstack-nova15:25
openstackgerritIonuÈ› BĂ®ru proposed openstack/nova master: Fix notify_on_state_change deprecation  https://review.openstack.org/44435015:25
ionimriedem, ^15:25
dims_ioni : does it work if you just delete the line 33? (https://review.openstack.org/#/c/444350/1/nova/conf/notifications.py)15:27
ionitesting now15:28
dims_thanks!15:28
mriedemdims_: removing the deprecated_group line would break the alias right?15:28
mriedemioni: we should fix the other similar issues15:28
dims_should end up as 'DEFAULT' i think mriedem (need to check)15:28
mriedemi pointed those out in the review (as did sfinucan)15:28
ionidims_, no, it doesn't work15:29
mriedemdims_: if you don't have the deprecated_group kwarg, oslo.config doesn't know it came from a deprecated group and won't alias15:29
ionimriedem, i just saw your comment15:29
lyarwoodsfinucan: stupid question but where is the stable/ocata openstack-docs-tools build listed?15:29
ionimriedem, i'll resend15:29
mriedemlyarwood: there isn't a stable branch for that http://git.openstack.org/cgit/openstack/openstack-doc-tools/15:29
dims_ioni : mriedem : ack thanks15:29
sfinucanlyarwood: The tools are found in openstack-docs-tools, but the output is kept in openstack-manuals15:30
sfinucanhttps://review.openstack.org/#/c/436474/15:30
lyarwoodmriedem / sfinucan ; right, so do we need https://review.openstack.org/#/c/444306/ in stable/ocata ? I assume not15:30
*** Jack_I has joined #openstack-nova15:31
sfinucanlyarwood: That tool checks out the nova code and uses a combo of oslo.config and Jinja2 templates to build those table files15:31
lyarwoodsfinucan: ah gotcha15:31
*** lucasxu has joined #openstack-nova15:32
sfinucanthen dumps it into openstack-manuals project, which is where the stable/ocata branch will be (not openstack-doc-tools)15:32
sfinucanyes, it *is* a somewhat convoluted tool chain :)15:32
*** mriedem1 has joined #openstack-nova15:33
lyarwoodsfinucan: ha, I was just looking for proof that we needed it, I'll add a note in the review, LGTM otherwise.15:34
*** ducttape_ has quit IRC15:34
*** rdo has quit IRC15:35
*** artom has quit IRC15:35
*** zz_dimtruck is now known as dimtruck15:36
ionimriedem can you help me a bit?15:36
*** artom has joined #openstack-nova15:36
*** mriedem has quit IRC15:36
*** rdo has joined #openstack-nova15:36
ionii don't know how to send a new patch in the same review :D15:37
*** Andrew_jedi has joined #openstack-nova15:37
*** marst has quit IRC15:38
*** amoralej|lunch is now known as amoralej15:39
openstackgerritIonuÈ› BĂ®ru proposed openstack/nova master: Correctly set up deprecation warning  https://review.openstack.org/44435715:41
ionisorry for that :(15:42
ionii didn't know how to send a new patch set15:42
sfinucanioni: No problem :) If you keep the 'Change-ID' line the same, it will submit a new patchset15:42
*** liangy has joined #openstack-nova15:43
ionidamn it15:43
sfinucanit's fine though. Don't worry about it15:43
*** kornicameister has quit IRC15:43
siva_krishnanioni: git review --amend should help as well15:44
sfinucanioni: +2d15:44
siva_krishnansorry git  commit --amend15:44
ioniwell, i did a rebase15:44
ionifrom two commits that i made locally15:45
ioniand i messed up i guess15:45
mriedem1artom: replied in https://review.openstack.org/#/c/393930/ - i think we talked about that at one point15:45
mriedem1artom: basically the various interface APIs in nova are a mess15:45
mriedem1and that's why i omitted nic tags from the spec15:45
*** mriedem1 is now known as mriedem15:46
mriedemthat's lazy on my part, i just didn't want to sort out the interface APIs first15:46
*** rfolco has quit IRC15:46
*** rfolco has joined #openstack-nova15:46
mriedemioni: i do git commit -a --amend15:46
mriedemthen git review to push it up15:46
ionimriedem, cool. i'll keep in mind15:46
*** marst has joined #openstack-nova15:47
leakypipescdent: oh, that's -w because I still need to submit a mini-spec or blueprint.15:47
cdentleakypipes: ended up leaving it off this week's update15:48
sfinucanjohnthetubaguy, mriedem: noddy stuff, but I'd like to fix this in stable/ocata before you make the point release next week https://review.openstack.org/#/c/443086/15:48
sfinucan(AJaeger asked me to do so for openstack-manuals)15:48
artommriedem, I don't remember talking about it, but OK, your explanation makes sense15:48
*** tbachman has quit IRC15:49
mriedemsfinucan: can you show me where those are broken today?15:49
mriedema bug would have been nice15:50
mriedemif we're backporting15:50
leakypipescdent: totally fine :) mostly was just pushing it off my lappie so I could pull to my workstation ;)15:50
mriedemplus links to see what the bug is would be nice15:50
*** salv-orlando has joined #openstack-nova15:50
cdentleakypipes: I know how that can be. I left a comment on it as a sort of note to self to remember my confusion and sadness15:50
leakypipes:)15:51
cdentand with that I'm going for a walk, biab15:51
*** cdent has quit IRC15:51
sfinucanmriedem: https://docs.openstack.org/newton/config-reference/compute/config-options.html look for 'flat_network_bridge'15:51
sfinucanor any of the giant text with grey bars on the left15:51
*** liusheng has quit IRC15:52
sfinucanI didn't think formatting issues like that warranted a bug but I'll draw up one shortly15:52
mriedemok i see15:52
mriedemsfinucan: well i just didn't see any examples in the commit message15:52
*** liusheng has joined #openstack-nova15:52
mriedemalso, this doesn't seem like a huge issue15:52
*** iceyao has quit IRC15:52
mriedemand it's been like this for awhile already, like all of ocata15:53
sfinucanmriedem: I can't include images in commit messages, and the issues won't exist once they're fixed15:53
sfinucanopenstack-manuals hasn't released the ocata docs yet, hence why I'm eager to fix it now15:54
mriedemok i guess i just won't ask for any context or examples15:54
mriedemanyway, +215:54
sfinucanmriedem: ta15:55
*** salv-orlando has quit IRC15:55
*** rcernin has quit IRC15:59
mriedemioni: https://review.openstack.org/#/c/444374/15:59
ionimriedem, do i have to do something? :D16:01
openstackgerritStephen Finucane proposed openstack/nova master: Always set MTU when plugging devices  https://review.openstack.org/40878116:02
*** eharney has quit IRC16:03
mriedemioni: nope16:03
*** Oku_OS is now known as Oku_OS-away16:06
openstackgerritStephen Finucane proposed openstack/nova master: hardware: Flatten functions  https://review.openstack.org/36747016:07
openstackgerritStephen Finucane proposed openstack/nova master: Standardize '_get_XXX_constraints' functions  https://review.openstack.org/38507116:07
openstackgerritStephen Finucane proposed openstack/nova master: Rename '_numa_get_constraints_XXX' functions  https://review.openstack.org/38507216:07
openstackgerritStephen Finucane proposed openstack/nova master: De-duplicate _numa_get_flavor_XXX_map_list  https://review.openstack.org/38507416:07
*** sneti_ has joined #openstack-nova16:07
leakypipesrfolco: morning!16:08
rfolcoleakypipes, afternoon!16:08
leakypipesrfolco: so, I can support cdent's review suggestions on your patch. Go with KeyError raise.16:08
*** ducttape_ has joined #openstack-nova16:09
rfolcoleakypipes, nice. I can do something like: http_except = webob.exc.status_map.get(req) .... if not None, then raise http_except16:10
rfolcoinstead of status_map[req]16:10
rfolcothis way I avoid keyerror and get rid of dict16:10
*** unicell has joined #openstack-nova16:11
leakypipesrfolco: well, I think cdent actually prefers raising a KeyError there, since this is a code path that is dev-only -- i.e. users cannot influence this code path and we want to bomb out early if a developer adds an call to raise_http_code_if_microversion() is passed an improper status code.16:11
*** gabor_antal has quit IRC16:13
rfolcohmm makes sense. If keyerror is fine, easier for me :)16:13
rfolcoleakypipes, thx++16:13
leakypipesrfolco: go for it :)16:13
*** unicell1 has quit IRC16:14
*** dtp has joined #openstack-nova16:14
*** eharney has joined #openstack-nova16:17
*** Andrew_jedi has quit IRC16:18
*** bvanhav has quit IRC16:19
openstackgerritEric Fried proposed openstack/nova master: WIP: PowerVM Driver: SSP emphemeral disk support  https://review.openstack.org/44318916:22
*** felipemonteiro_ has quit IRC16:22
*** liverpooler has quit IRC16:22
*** ducttape_ has quit IRC16:23
*** liverpooler has joined #openstack-nova16:26
*** sridharg has quit IRC16:27
*** thorst is now known as thorst_afk16:27
*** bvanhav has joined #openstack-nova16:28
*** mriedem1 has joined #openstack-nova16:29
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce basic framework for security proxying  https://review.openstack.org/34539616:30
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication  https://review.openstack.org/34539716:30
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme  https://review.openstack.org/34539816:30
openstackgerritStephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation  https://review.openstack.org/34539916:30
*** mriedem has quit IRC16:31
*** aarefiev is now known as aarefiev_afk16:32
*** jaosorior has joined #openstack-nova16:34
*** fandi has joined #openstack-nova16:36
*** sneti_ has quit IRC16:40
*** felipemonteiro_ has joined #openstack-nova16:42
*** jaosorior has quit IRC16:42
*** alexpilotti has quit IRC16:42
*** alexpilotti has joined #openstack-nova16:43
*** derekh has quit IRC16:43
*** dims_ has quit IRC16:45
*** alexpilotti has quit IRC16:47
*** fandi has quit IRC16:49
*** tbachman has joined #openstack-nova16:51
*** ltomasbo is now known as ltomasbo|away16:51
*** yamamoto has joined #openstack-nova16:51
*** nic has joined #openstack-nova16:56
*** lucasxu has quit IRC16:56
*** logan_ has joined #openstack-nova16:56
*** gfhellma has joined #openstack-nova16:56
*** logan_ is now known as Guest2958916:57
*** lucasxu has joined #openstack-nova16:58
*** logan- has quit IRC16:59
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: [WIP] Read NIC features in libvirt  https://review.openstack.org/44441517:02
*** lucasxu has quit IRC17:02
*** ralonsoh has quit IRC17:03
*** Guest29589 is now known as logan-17:03
*** alexpilotti has joined #openstack-nova17:04
mriedem1dansmith: so no obfuscation for https://review.openstack.org/#/c/438023/25/nova/context.py ?17:06
*** kaisers_ has joined #openstack-nova17:07
*** catintheroof has quit IRC17:07
*** mriedem1 is now known as mriedem17:07
*** catintheroof has joined #openstack-nova17:08
melwittmriedem: it's not storing the url in the context anymore17:08
mriedemoh cool17:09
mriedemi'm going to have to rebase my quotas fix now.. :(17:09
*** yamamoto has quit IRC17:11
leakypipesdansmith: your once over on gibi's patch here pls: https://review.openstack.org/#/c/44034717:11
*** kaisers_ has quit IRC17:12
*** ayogi has joined #openstack-nova17:14
melwittleakypipes: not sure I understand the comment on that one, that instance.flavor isn't connected to data in the db. it's connected to instance_extra and can be lazy-loaded on an instance that was queried without joining with instance_extra earlier17:14
mriedemmelwitt: leakypipes: bauwser: also need to get these in https://review.openstack.org/#/q/topic:bug/1670627 - that's a major regression in ocata and need to get that backported for a 13.0.1 next week17:15
mriedemfwiw dan was +2 on the bottom change yesterday, and had skimmed the rest of the series already17:16
*** cdent has joined #openstack-nova17:16
leakypipesmelwitt: yeah, it's weird. basically, what we're attempting to prevent is the lazy-loading of flavor *projects* -- i.e. the second-stage lazy-loading of Flavor attributes.17:16
mriedemi'm going to be basically MIA the rest of the afternoon, it's my last day here so lunch, exit interview and then they kick my ass out17:16
mriedemand my new laptop hasn't shown up yet17:16
leakypipesmelwitt: so the instance.Flavor is populated when expected_attributes=['flavor'] and that's fine. But by orphaning the Flavor in instance.flavor, we prevent the instance.flavor.projects attribute from being lazy-loaded.17:17
melwittleakypipes: so like a cascading lazy-load17:17
leakypipesmelwitt: weird, I know :)17:17
leakypipesmelwitt: yeah.17:17
melwittah, I see17:17
*** sudipto has quit IRC17:17
melwittmriedem: ack, will review those today17:17
leakypipesdansmith: time for you to enter the phone booth and emerge as superdan.17:18
*** ducttape_ has joined #openstack-nova17:20
*** mriedem is now known as mriedem_afk17:21
artomIs Phone Booth is mdbooth's rapper name?17:21
*** lucasagomes is now known as lucas-afk17:22
melwittheh17:22
*** Apoorva has joined #openstack-nova17:23
*** browne has joined #openstack-nova17:24
*** Apoorva has quit IRC17:24
*** Apoorva has joined #openstack-nova17:25
*** salv-orlando has joined #openstack-nova17:26
*** salv-orlando has quit IRC17:27
*** salv-orlando has joined #openstack-nova17:27
*** david-lyle has joined #openstack-nova17:28
*** raunak has joined #openstack-nova17:29
*** lucasxu has joined #openstack-nova17:29
*** Jack_I has quit IRC17:30
*** JoseMello has joined #openstack-nova17:31
*** Apoorva has quit IRC17:31
*** alexpilo_ has joined #openstack-nova17:33
*** claudiub|2 has joined #openstack-nova17:34
*** alexpilotti has quit IRC17:37
*** vishwanathj has quit IRC17:37
*** claudiub has quit IRC17:37
*** vishwanathj has joined #openstack-nova17:37
*** armax has joined #openstack-nova17:37
*** ssurana has joined #openstack-nova17:40
andrewbogottIs 'Host xxx has more disk space than database expected' ever anything important?  That message shows up in several bug reports but best I can tell the message is a red herring in all cases.17:44
andrewbogott(I have 4 out of 13 compute nodes that throw that warning regularly.)17:44
*** unicell has quit IRC17:44
*** lpetrut has quit IRC17:45
openstackgerritEric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support  https://review.openstack.org/44318917:47
efriedthorst_afk adreznec esberglu_ ^^ live tested, works, ready for subteam review.17:47
*** psachin has quit IRC17:48
*** Apoorva has joined #openstack-nova17:49
*** ducttape_ has quit IRC17:50
*** avolkov has quit IRC17:59
openstackgerritDan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db  https://review.openstack.org/43975417:59
openstackgerritDan Smith proposed openstack/nova master: Make os-availability-zones know about cells  https://review.openstack.org/44216317:59
openstackgerritDan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info  https://review.openstack.org/43989117:59
openstackgerritDan Smith proposed openstack/nova master: Add identity helper property to CellMapping  https://review.openstack.org/44403817:59
openstackgerritDan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell  https://review.openstack.org/43802517:59
*** felipemonteiro_ has quit IRC18:03
*** efoley_ has quit IRC18:08
*** lucasxu has quit IRC18:08
*** sapcc-bot2 has quit IRC18:10
*** sapcc-bot has joined #openstack-nova18:10
*** zioproto has quit IRC18:11
*** lucasxu has joined #openstack-nova18:15
*** mnestratov has quit IRC18:17
*** dharinic is now known as dharinic|lunch18:17
*** hieulq has quit IRC18:21
*** jpena is now known as jpena|off18:24
*** unicell has joined #openstack-nova18:28
*** ayogi has quit IRC18:32
*** ducttape_ has joined #openstack-nova18:32
*** eharney has quit IRC18:35
*** hieulq has joined #openstack-nova18:36
*** fragatina has quit IRC18:39
*** Jack_I has joined #openstack-nova18:40
*** sambetts is now known as sambetts|afk18:41
*** Jack_V has joined #openstack-nova18:43
*** Jack_I has quit IRC18:46
*** satyar has quit IRC18:48
*** yamahata has quit IRC18:50
*** eharney has joined #openstack-nova18:51
*** Jack_I has joined #openstack-nova18:52
*** Jack_V has quit IRC18:54
*** awaugama has quit IRC18:54
*** eharney_ has joined #openstack-nova18:59
*** kevinz has joined #openstack-nova19:00
*** eharney has quit IRC19:01
*** eharney_ is now known as eharney19:02
cfriesensomeone pointed out to me that InstanceActionEvent.pack_action_event_finish() uses a "message" field, but the InstanceActionEvent model in the DB uses "details".  Is this a bug?19:04
*** dharinic|lunch is now known as dharinic19:04
mriedem_afkandrewbogott: this is the comment by that warning19:07
mriedem_afk# can occur when an instance in database is not on host19:07
*** mriedem_afk is now known as mriedem19:07
mriedemandrewbogott: which is similar to the _sync_power_states periodic task in the compute manager, but that might not be a warning19:08
mriedemwould have to check19:08
andrewbogottmriedem: thanks.  I did a side-by-side comparison on one of the hosts already but maybe I missed a discrepancy.19:08
*** kaisers_ has joined #openstack-nova19:08
andrewbogottAt least… you'd expect 'nova list —host <hostname>' to show the db contents and not the disk contents, right?19:08
openstackgerritJohn Griffith proposed openstack/nova master: Add Cinder API version detection  https://review.openstack.org/44446519:10
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/44335719:10
*** mriedem has quit IRC19:11
jgriffithildikov mriedem ^^ don't know if that's more along the lines of what you'd like to see or not19:11
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/39534719:12
*** slaweq_ has joined #openstack-nova19:12
*** kaisers_ has quit IRC19:12
*** felipemonteiro_ has joined #openstack-nova19:14
openstackgerritJay Pipes proposed openstack/nova master: virt: implement get_inventory() for Ironic  https://review.openstack.org/44154419:18
leakypipescdent: hi. I'm getting a single unit test failure for nova.tests.unit.test_fixtures.TestPlacementFixture.test_responds_to_version locally on ^^ patch. it looks like this: http://paste.openstack.org/show/602305/19:19
* cdent clicks19:19
leakypipescdent: nothing in the change to that patch should affect the placement API as far as I can tell. have you seen that error before?19:20
cdenti've not, one sec19:20
*** yamahata has joined #openstack-nova19:23
cdentleakypipes: are the functional tests running okay?19:23
leakypipescdent: yes.19:23
*** salv-orl_ has joined #openstack-nova19:23
* cdent looks at the patch closer19:24
*** sdague has joined #openstack-nova19:24
*** bvanhav has quit IRC19:25
cdentimma need to try this stuff myself leakypipes19:26
*** salv-orlando has quit IRC19:26
leakypipescdent: np19:26
*** bvanhav has joined #openstack-nova19:27
*** salv-orl_ has quit IRC19:28
*** sdague has quit IRC19:29
*** raunak has quit IRC19:32
*** READ10 has quit IRC19:32
*** raunak has joined #openstack-nova19:33
*** bnemec is now known as beekneemech19:34
leakypipesJayF: thx for the reviews on the ironic patch. responded to your query on the latest revision.19:36
JayFleakypipes: it's fine, "valid -1 on a jaypipes nova patch" will make a fine line on my resume ;)19:37
*** jamesdenton has quit IRC19:37
leakypipesJayF: lmao19:37
* cdent updates resume19:37
*** esberglu_ has quit IRC19:39
JayFleakypipes: re: your comment, that's exactly what I suspected, but wanted to be sure. Also get_inventory() will only ever be called on nodes that pass get_available_resources() which guarantees all our logic guarding nodes in maintenance/wrong states/etc will be intact19:39
*** esberglu has joined #openstack-nova19:39
leakypipesJayF: correct. node_is_available() gates any call to get_inventory()19:39
*** esberglu has quit IRC19:44
cdentleakypipes: does it fail consistently? does it only fail when you run all the tests?19:44
leakypipescdent: fails consistently when I run all unit tests. lemme try running it singularly.19:44
cdentin other words, works for me when I test just that file and also a) make sure I don't have an /etc/nova/nova.conf and -r my tox19:44
* cdent tests them all19:45
cdentI resisted this fixture. I figured we should just keep on doing the wsgi-intercept thing everywhere.19:45
*** dims has joined #openstack-nova19:49
*** abalutoiu has joined #openstack-nova19:50
leakypipescdent: yeah, fails when I just run tox -epy27 -- test_fixtures19:50
cdentinteresting, wfm19:50
cdenti'm running all of them now19:50
openstackgerritDan Smith proposed openstack/nova master: Teach HostAPI about cells  https://review.openstack.org/44216219:51
openstackgerritDan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety  https://review.openstack.org/44317419:51
openstackgerritDan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList  https://review.openstack.org/44329219:51
openstackgerritDan Smith proposed openstack/nova master: Make server_groups determine deleted-ness from InstanceMappingList  https://review.openstack.org/44329319:51
openstackgerritDan Smith proposed openstack/nova master: Remove Mitaka-era service version check  https://review.openstack.org/44286119:51
openstackgerritDan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall  https://review.openstack.org/44273619:51
openstackgerritDan Smith proposed openstack/nova master: Clean up ClientRouter debt  https://review.openstack.org/44448719:51
*** esberglu has joined #openstack-nova19:51
*** fragatina has joined #openstack-nova19:55
* cdent NEEDS MORE CPUs19:59
cdentGET /resource_providers?resources=VCPU:2420:01
*** gouthamr has quit IRC20:04
*** JoseMello has quit IRC20:05
*** sambetts|afk has quit IRC20:06
cdentleakypipes: works when running all too20:07
*** sambetts_ has joined #openstack-nova20:08
cdentdo you have any evidence from the fixture of the internal failure (the exception within the 500)20:08
*** artom has quit IRC20:09
*** crushil has quit IRC20:10
*** ducttape_ has quit IRC20:13
*** gouthamr has joined #openstack-nova20:16
leakypipescdent: none :(20:19
cdentyou done a fresh tox?20:20
cdent:( I dunno.20:21
*** markvoelker has joined #openstack-nova20:22
*** vladikr has quit IRC20:23
*** vishwanathj has quit IRC20:25
*** vishwanathj has joined #openstack-nova20:26
*** ducttape_ has joined #openstack-nova20:26
*** salv-orlando has joined #openstack-nova20:29
*** Apoorva has quit IRC20:31
cdentleakypipes: I gotta go, if/when you figure it out I'm curious to know. I think it's just you though: unit tests passed in the gate for that commit20:31
leakypipesyeah...20:31
leakypipescdent: I'll let ya know what I discover.20:31
*** ssurana has quit IRC20:32
*** ssurana has joined #openstack-nova20:33
*** salv-orlando has quit IRC20:34
*** felipemonteiro_ has quit IRC20:34
tonybsfinucan: np20:34
*** cdent has quit IRC20:34
openstackgerritRafael Folco proposed openstack/nova master: DELETE all inventory for a resource provider  https://review.openstack.org/41666920:37
openstackgerritRafael Folco proposed openstack/nova master: Raising http codes on old microversion  https://review.openstack.org/44449720:37
*** zul is now known as zulVacation20:37
*** bvanhav has quit IRC20:38
*** marst has quit IRC20:39
*** jose-phillips has quit IRC20:43
*** kaisers_ has joined #openstack-nova20:44
*** amoralej is now known as amoralej|off20:44
*** READ10 has joined #openstack-nova20:45
*** jose-phillips has joined #openstack-nova20:45
*** abalutoiu has quit IRC20:46
*** kaisers_ has quit IRC20:49
*** marst has joined #openstack-nova20:51
openstackgerritJohn Griffith proposed openstack/nova master: Add Cinder API version detection  https://review.openstack.org/44446520:51
*** lpetrut has joined #openstack-nova20:53
*** dtp is now known as dtp-afk20:55
*** lucasxu has quit IRC20:57
*** lucasxu has joined #openstack-nova20:57
*** eharney has quit IRC21:07
*** ducttape_ has quit IRC21:08
openstackgerritEric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support  https://review.openstack.org/44318921:19
*** rfolco has quit IRC21:20
efriedthorst_afk ^^ addressed comments.  Hoping for that +1 before you disappear into babyland.21:20
thorst_afkefried: k21:20
*** thorst_afk has quit IRC21:26
*** salv-orlando has joined #openstack-nova21:30
efriedharlowja Another TaskFlow-havin' change set, if you're interested ^^21:33
harlowjacools21:33
efriedharlowja Be sure to read thorst's commentary ;-)21:34
harlowjakk21:34
*** Jeffrey4l__ has joined #openstack-nova21:35
harlowjawoah, lots of commentary21:35
harlowjalol21:35
*** Jeffrey4l_ has quit IRC21:35
harlowjahopefully taskflow has helped some :-P21:36
*** smatzek has quit IRC21:36
efriedharlowja As you can see, we definitely dig it.21:36
harlowjaya, i mean i don't like having all the LinearFlow and such objects21:36
harlowjai was going to try to make it more natural, but haven't quite figured that out21:37
harlowjaha21:37
harlowjait'd be interesting to do something like21:37
efriedharlowja I was telling thorst yesterday - I've had a TODO for a long time to rework our TaskFlow usage into a much more complicated unordered parallel flow with appropriate interdependencies.21:37
*** liangy has quit IRC21:37
*** aysyd has quit IRC21:37
efriedHe basically said, "You had my veto at 'complicated'"21:37
harlowjalol21:38
harlowjaya, if i could i'd somehow plug what taskflow in at more of the language level21:38
efriedharlowja FYI, this is the flow I was talking about: https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/driver.py#L38421:39
*** jose-phillips has quit IRC21:39
harlowjaso that somehow u could give say a set of interconnected functions to some kind of object and it would try to extrapolate the flow(s)21:39
efriedharlowja You basically have that, with provides/requires.  And I'm sure it works perfectly with no bugs.21:40
harlowjaya, it'd be interesting to try to make such kind of functions more elegant21:40
*** Apoorva has joined #openstack-nova21:40
harlowjawell i was thinking of something like21:40
efriedBut I've fiddled with it a little bit and remember finding it a bit complicated to figure out how to set up the right engine/flow so it would actually be useful and run stuff in parallel.21:40
harlowja`def a():21:41
harlowja   b()21:41
harlowjadef b():21:41
harlowja    c()`21:41
harlowjanow given a, is there a way to 'decompile' that into a corresponding set of linear flows (for example)21:41
efriedharlowja You're talking about static analysis of the code, though?21:41
harlowjasort of21:41
*** rfolco has joined #openstack-nova21:41
efriedMy knee-jerk is that you'd be passing the 80/20 point there ;-)21:42
harlowjafair enough21:42
harlowjaha21:42
harlowjaafaik the parallelism, basiclaly if u have things that can run in parallel (determined by ordering relationships); then changing the engine will run them in parallel (if there is no paralleism possible, then changing the engine won't do much)21:43
*** thorst_afk has joined #openstack-nova21:44
efriedharlowja Well, we did do some of that, down in pypowervm.  You may actually get a kick out of this: https://github.com/powervm/pypowervm/blob/develop/pypowervm/utils/transaction.py#L49821:44
efried(thorst_afk we were just talkin about ya)21:45
harlowjainteresting21:45
harlowjaanything in taskflow that may make such stuff easier/more pretty/other?21:45
harlowjaseems like that FeedTask could just be a nestedflow?21:47
efriedharlowja To be honest, given what it does, I don't see a whole lot of improvement.21:47
harlowjak21:47
efriedWhat's a nestedflow?21:47
harlowjaroot_flow = flow("root")21:47
harlowjaroot_flow.add(task1, task2...)21:47
*** jamesdenton has joined #openstack-nova21:48
harlowjaroot_flow.add(nested_flow)21:48
harlowjawhere nested_flow is yet another flow21:48
efriedharlowja Yeah, that's how we use it.21:48
*** rfolco has quit IRC21:48
harlowjaoh, k21:48
efriedhttps://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/driver.py#L45421:49
*** jose-phillips has joined #openstack-nova21:49
efriedThe aforementioned21:49
*** rebase has joined #openstack-nova21:49
*** vishwanathj has quit IRC21:49
harlowjakk21:50
*** vishwanathj has joined #openstack-nova21:50
*** jamesden_ has joined #openstack-nova21:52
*** lucasxu has quit IRC21:52
*** jamesdenton has quit IRC21:52
*** lucasxu has joined #openstack-nova21:52
figleafleakypipes: just catching up on scrollback. I ran tests on that patch and didn't get any errors21:53
leakypipesfigleaf: yeah, it's weird.. :)21:54
figleafleakypipes: well, Gerrit like is, so... :)21:54
figleafs/like is/likes it21:55
leakypipesyeah, I saw that :)21:55
openstackgerritDan Smith proposed openstack/nova master: Clean up ClientRouter debt  https://review.openstack.org/44448721:56
*** dave-mccowan has quit IRC21:57
*** gouthamr has quit IRC22:03
*** lpetrut has quit IRC22:06
cfriesenhas anyone run into an issue with libvirt (version 1.2.12) reporting "operation failed: domain is no longer running" if you pause the instance while the migration is running (to force completion)?   It seems to be fixed in libvirt 1.2.17.   I'm looking at the libvirt code, but I figured I'd ask...22:06
*** lpetrut has joined #openstack-nova22:07
*** bauwser is now known as bauzas22:08
*** ducttape_ has joined #openstack-nova22:09
*** lpetrut has quit IRC22:11
*** jamesden_ has quit IRC22:12
*** Jack_I has quit IRC22:14
*** priteau has quit IRC22:14
*** ducttape_ has quit IRC22:14
*** jamesdenton has joined #openstack-nova22:15
*** eharney has joined #openstack-nova22:16
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova master: Allow live migrate paused instance when post copy is enabled  https://review.openstack.org/44451722:17
*** catintheroof has quit IRC22:23
*** dimtruck is now known as zz_dimtruck22:26
*** READ10 has quit IRC22:26
*** fragatin_ has joined #openstack-nova22:28
melwittdansmith: your thoughts would be appreciated on the consoleauth token upcall patch https://review.openstack.org/#/c/415922/ I was wondering whether it makes sense to also reject requests for connect_info too since it seemed possible it wouldn't be usable22:29
*** gouthamr has joined #openstack-nova22:29
melwittor if I'm not understanding that right22:29
*** knikolla has left #openstack-nova22:31
*** fragatina has quit IRC22:32
*** jamesdenton has quit IRC22:33
*** lucasxu has quit IRC22:35
*** sdague has joined #openstack-nova22:35
*** dtp-afk is now known as dtp22:36
openstackgerritEd Leafe proposed openstack/nova master: WIP - Combine the two auth.py files  https://review.openstack.org/44452122:39
openstackgerritEd Leafe proposed openstack/nova master: WIP - Move __init__.py  https://review.openstack.org/44452222:39
openstackgerritEd Leafe proposed openstack/nova master: WIP - Move wsgi.py  https://review.openstack.org/44452322:39
openstackgerritEd Leafe proposed openstack/nova master: WIP - move urlmap.py  https://review.openstack.org/44452422:39
*** kaisers_ has joined #openstack-nova22:40
*** rebase has left #openstack-nova22:40
*** sdague has quit IRC22:42
*** kaisers_ has quit IRC22:44
efriedmelwitt Seems like you still have reservations about https://review.openstack.org/#/c/443745/.  Not that you want to rollback the whole change; maybe just a reword before you commit.  (I'm paraphrasing - that's not a direct quota [sic].)22:52
*** tbachman has quit IRC22:53
*** Kevin_Zheng has quit IRC22:53
*** karimb has quit IRC22:55
melwittefried: yeah, I think it's probably worth fixing the docstring and comments so as not to possibly confuse others down the line. I considered adjusting the wording myself and +2 it, but mriedem has a merge conflict at the top of the stack that I thought he might want to handle himself22:56
* efried 's attempts at humor fall flat.22:56
melwittheh, no, was a good quota joke22:58
efriedwhew22:59
efriedThere are so precious few of those, as we all know.22:59
melwittthat's true, we don't get a lot of quota jokes around here. mostly just quota lamentations23:00
*** fragatin_ has quit IRC23:02
*** fragatina has joined #openstack-nova23:03
*** hongbin has quit IRC23:05
*** jamesdenton has joined #openstack-nova23:06
*** tbachman has joined #openstack-nova23:10
*** tblakes has quit IRC23:12
*** salv-orlando has quit IRC23:18
*** zz_dimtruck is now known as dimtruck23:20
*** nicolasbock has quit IRC23:23
*** nicolasbock has joined #openstack-nova23:27
*** thorst_afk has quit IRC23:27
*** tesseract has quit IRC23:29
*** xyang1 has quit IRC23:33
*** dtp has quit IRC23:35
*** edmondsw has quit IRC23:35
*** edmondsw has joined #openstack-nova23:37
*** edmondsw has quit IRC23:42
*** baoli has quit IRC23:43
*** baoli has joined #openstack-nova23:45
*** dimtruck is now known as zz_dimtruck23:46
*** baoli has quit IRC23:48
*** jamesdenton has quit IRC23:49
*** ssurana has quit IRC23:53
*** ssurana has joined #openstack-nova23:53
*** tblakes has joined #openstack-nova23:57
*** ssurana has quit IRC23:58

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