Friday, 2015-04-17

« Thursday, 2015-04-16 Index Saturday, 2015-04-18 »
*** emagana has joined #openstack-nova00:02
*** dims__ has joined #openstack-nova00:04
*** ZZelle_ has quit IRC00:04
*** dims_ has joined #openstack-nova00:05
*** dims__ has quit IRC00:09
*** markmcclain has joined #openstack-nova00:14
openstackgerritNha Pham proposed openstack/nova: Encrypted iSCSI volume has name too long  https://review.openstack.org/17465600:16
*** sdake has joined #openstack-nova00:20
*** sdake_ has quit IRC00:24
*** marun has quit IRC00:31
*** VW_ has joined #openstack-nova00:35
*** VW_ has quit IRC00:36
*** Marga_ has joined #openstack-nova00:36
*** VW_ has joined #openstack-nova00:37
*** zhenguo has joined #openstack-nova00:38
*** Marga_ has quit IRC00:41
*** yingjun has joined #openstack-nova00:47
*** patrickeast has quit IRC00:50
*** achanda has quit IRC00:51
*** yingjun has quit IRC00:51
*** yingjun has joined #openstack-nova00:52
*** yingjun has quit IRC00:56
*** achanda has joined #openstack-nova00:59
*** yingjun has joined #openstack-nova01:00
*** mwagner_lap has joined #openstack-nova01:00
*** vilobhmm11 has quit IRC01:01
*** emagana has quit IRC01:02
*** emagana has joined #openstack-nova01:03
*** emagana has quit IRC01:03
*** ozialien has joined #openstack-nova01:03
*** baoli has joined #openstack-nova01:08
*** ozialien has quit IRC01:12
*** gyee_ has quit IRC01:13
*** dsanders has quit IRC01:13
*** krtaylor has joined #openstack-nova01:13
*** popw has joined #openstack-nova01:14
openstackgerritGhanshyam Mann proposed openstack/nova: Merge V2 and V2.1 hypervisor functional tests  https://review.openstack.org/16708201:16
openstackgerritGhanshyam Mann proposed openstack/nova: Merge V2 and V2.1 cloudpipe functional tests  https://review.openstack.org/16748101:21
openstackgerritGhanshyam Mann proposed openstack/nova: Share V2 and V2.1 images functional tests  https://review.openstack.org/16785401:21
openstackgerritTracy Jones proposed openstack/nova: Cleanup of compute api reboot method  https://review.openstack.org/12742701:21
openstackgerritGhanshyam Mann proposed openstack/nova: Merge V2 and V2.1 networks functional tests  https://review.openstack.org/16824201:21
*** jecarey has joined #openstack-nova01:25
openstackgerritGhanshyam Mann proposed openstack/nova: Merge V2 and V2.1 services functional tests  https://review.openstack.org/16880801:32
openstackgerritGhanshyam Mann proposed openstack/nova: Merge test_admin_actions between v2 and v2.1  https://review.openstack.org/17075401:34
*** Sukhdev has quit IRC01:34
openstackgerritGhanshyam Mann proposed openstack/nova: Share admin password func test between v2 and v2.1  https://review.openstack.org/17075601:34
*** Marga_ has joined #openstack-nova01:37
*** vladikr has quit IRC01:38
openstackgerritGhanshyam Mann proposed openstack/nova: Share assisted vol snapshots test between v2 and v2.1  https://review.openstack.org/17075801:41
*** claudiub has quit IRC01:41
*** Marga_ has quit IRC01:42
*** erkules has joined #openstack-nova01:43
openstackgerritLianhao Lu proposed openstack/nova: Convert bandwidth_usage related timestamp to UTC native datetime  https://review.openstack.org/17321301:43
openstackgerritGhanshyam Mann proposed openstack/nova: Merge attach interfaces func test between v2 and v2.1  https://review.openstack.org/17076301:43
openstackgerritGhanshyam Mann proposed openstack/nova: Merge availability_zone tests between v2 and v2.1  https://review.openstack.org/17101701:44
openstackgerritGhanshyam Mann proposed openstack/nova: Merge config_drive functional tests between v2 and v2.1  https://review.openstack.org/17102901:44
*** erkules_ has quit IRC01:46
*** rfolco has quit IRC01:46
*** harlowja is now known as harlowja_away01:47
*** shaohe_feng has joined #openstack-nova01:47
openstackgerritGhanshyam Mann proposed openstack/nova: Merge console_auth_tokens tests between v2 and v2.1  https://review.openstack.org/17104301:52
*** Marga_ has joined #openstack-nova01:53
openstackgerritGhanshyam Mann proposed openstack/nova: Share deferred_delete func tests between v2 and v2.1  https://review.openstack.org/17144701:53
openstackgerritGhanshyam Mann proposed openstack/nova: Merge disk-config func test between v2 and v2.1  https://review.openstack.org/17145301:54
*** dims_ has quit IRC01:54
openstackgerritGhanshyam Mann proposed openstack/nova: Merge extended server status tests between v2 and v2.1  https://review.openstack.org/17197001:54
openstackgerritGhanshyam Mann proposed openstack/nova: Share hide server add tests between v2 and v2.1  https://review.openstack.org/17230201:55
openstackgerritGhanshyam Mann proposed openstack/nova: Merge lock/pause server tests between v2 and v2.1  https://review.openstack.org/17230801:55
openstackgerritGhanshyam Mann proposed openstack/nova: Merge migrate server tests between v2 and v2.1  https://review.openstack.org/17279201:55
openstackgerritGhanshyam Mann proposed openstack/nova: Merge server personality tests between v2 and v2.1  https://review.openstack.org/17319601:56
openstackgerritGhanshyam Mann proposed openstack/nova: Merge preserve_ephemeral_rebuild tests between v2 & v2.1  https://review.openstack.org/17328401:56
openstackgerritGhanshyam Mann proposed openstack/nova: Merge server rescue tests between v2 and v2.1  https://review.openstack.org/17365101:57
openstackgerritGhanshyam Mann proposed openstack/nova: Merge security groups tests between v2 and v2.1  https://review.openstack.org/17367101:58
openstackgerritGhanshyam Mann proposed openstack/nova: Merge server diagnostics & ext events tests for v2 & v2.1  https://review.openstack.org/17417601:58
*** axpBen has joined #openstack-nova01:59
*** VW_ has quit IRC02:00
*** Nikolay_St has joined #openstack-nova02:01
*** tjones1 has joined #openstack-nova02:02
*** Kevin_Zheng has quit IRC02:06
*** tjones1 has quit IRC02:07
*** tjones1 has joined #openstack-nova02:07
*** VW_ has joined #openstack-nova02:09
*** isd has joined #openstack-nova02:10
*** axpBen has left #openstack-nova02:10
*** tjones1 has quit IRC02:12
*** RuiChen has joined #openstack-nova02:12
*** tjones1 has joined #openstack-nova02:12
openstackgerritGhanshyam Mann proposed openstack/nova: Merge server groups tests between v2 and v2.1  https://review.openstack.org/17418102:14
*** Marga_ has quit IRC02:15
*** tjones1 has quit IRC02:17
openstackgerritGhanshyam Mann proposed openstack/nova: Merge multinic functional tests between v2 and v2.1  https://review.openstack.org/17279602:19
*** achanda has quit IRC02:20
*** yingjun has quit IRC02:21
*** achanda has joined #openstack-nova02:22
openstackgerritGhanshyam Mann proposed openstack/nova: Merge fping functional tests between v2 and v2.1  https://review.openstack.org/17229402:24
*** jwang_ has quit IRC02:26
*** tjones1 has joined #openstack-nova02:27
openstackgerritNha Pham proposed openstack/nova: Encrypted iSCSI volume has name too long  https://review.openstack.org/17465602:28
openstackgerritGhanshyam Mann proposed openstack/nova: Merge extended server Att tests between v2 and v2.1  https://review.openstack.org/17196102:30
*** VW_ has quit IRC02:33
*** Kevin_Zheng has joined #openstack-nova02:37
*** jwang_ has joined #openstack-nova02:37
openstackgerritGhanshyam Mann proposed openstack/nova: Merge evacuate functional tests between v2 and v2.1  https://review.openstack.org/17190202:42
*** Nikolay_St has quit IRC02:45
*** liusheng has joined #openstack-nova02:46
*** baoli has quit IRC02:46
*** jecarey has quit IRC02:47
*** achanda has quit IRC02:49
*** deepthi has joined #openstack-nova02:51
tjones1mikal: finally got to finishing this patch you asked for https://review.openstack.org/#/c/127427/02:52
*** unicell has quit IRC02:54
*** yamahata has quit IRC02:56
*** dsanders has joined #openstack-nova02:56
openstackgerritGhanshyam Mann proposed openstack/nova: Merge attach interfaces func test between v2 and v2.1  https://review.openstack.org/17076302:57
*** baoli has joined #openstack-nova02:57
*** Longgeek has joined #openstack-nova02:59
openstackgerritVincent Hou proposed openstack/nova: Correct the order of the parameters in swap_volume  https://review.openstack.org/17430703:01
*** VW_ has joined #openstack-nova03:01
*** pixelb has quit IRC03:02
*** Longgeek has quit IRC03:04
*** Longgeek has joined #openstack-nova03:06
*** liusheng has quit IRC03:07
*** liusheng has joined #openstack-nova03:07
*** otter768 has quit IRC03:08
openstackgerritVincent Hou proposed openstack/nova: Correct the order of the parameters to call swap_volume  https://review.openstack.org/17430703:09
*** baoli has quit IRC03:17
*** achanda has joined #openstack-nova03:20
*** vilobhmm1 has joined #openstack-nova03:20
*** dsanders has quit IRC03:21
*** VW__ has joined #openstack-nova03:21
*** dave-mccowan has joined #openstack-nova03:23
*** tojuvone has joined #openstack-nova03:23
*** VW_ has quit IRC03:23
*** VW__ has quit IRC03:31
*** isd has quit IRC03:34
*** sdake_ has joined #openstack-nova03:34
*** neelashah has joined #openstack-nova03:35
*** wwwBUKOLAYcom has quit IRC03:36
*** sdake has quit IRC03:38
*** Sukhdev has joined #openstack-nova03:44
*** ishant has joined #openstack-nova03:45
*** gyee has quit IRC03:52
*** achanda has quit IRC03:52
*** n0ano has quit IRC03:52
*** gyee_ has joined #openstack-nova03:52
*** gyee_ has quit IRC03:54
*** Nikolay_St has joined #openstack-nova03:55
openstackgerritNha Pham proposed openstack/nova: Encrypted iSCSI volume has name too long  https://review.openstack.org/17465603:58
openstackgerritwangxiyuan proposed openstack/nova: Spelling mistakes in nova/compute/api.py  https://review.openstack.org/17315103:58
*** dsanders has joined #openstack-nova04:02
*** shaohe_feng has quit IRC04:04
*** achanda has joined #openstack-nova04:07
*** Nikolay_St has quit IRC04:09
*** achanda has quit IRC04:11
*** arnaud___ has joined #openstack-nova04:15
*** achanda has joined #openstack-nova04:16
*** achanda has quit IRC04:18
*** rajesht has joined #openstack-nova04:20
*** achanda has joined #openstack-nova04:21
*** achanda has quit IRC04:29
openstackgerritGhanshyam Mann proposed openstack/nova: Merge server metadata tests between v2 and v2.1  https://review.openstack.org/17469604:32
*** kashyap` is now known as kashyap04:36
*** IanGovett1 has joined #openstack-nova04:39
*** IanGovett has quit IRC04:40
*** achanda has joined #openstack-nova04:42
*** haomaiwa_ has joined #openstack-nova04:43
*** shaohe_feng has joined #openstack-nova04:43
*** mmedvede has quit IRC04:46
*** shaohe_robot has joined #openstack-nova04:50
*** gtt116__ has joined #openstack-nova04:51
*** mmedvede has joined #openstack-nova04:57
*** flwang1 has quit IRC05:04
*** Kevin_Zheng has quit IRC05:05
openstackgerritNha Pham proposed openstack/nova: Encrypted iSCSI volume has name too long  https://review.openstack.org/17465605:06
*** ajayaa has joined #openstack-nova05:06
*** xyang1 has quit IRC05:07
*** oomichi has joined #openstack-nova05:08
*** otter768 has joined #openstack-nova05:09
*** otter768 has quit IRC05:13
*** ajayaa has quit IRC05:14
*** dikonoor has joined #openstack-nova05:17
*** divya_ has joined #openstack-nova05:17
*** dikonoor has quit IRC05:17
divya_alex_xu : this is about https://review.openstack.org/#/c/150352/05:18
*** arnaud___ has quit IRC05:20
divya_alex_xu : the changes for this were commited by you. I am looking for info on why REST apis like create mandatorily needs admin permissions instead of the permissions being picked up from policy.json05:20
*** openstackgerrit has quit IRC05:21
*** openstackgerrit has joined #openstack-nova05:21
*** neelashah has quit IRC05:23
*** divya_ has quit IRC05:24
*** dikonoor has joined #openstack-nova05:25
dikonoorsdague: this is about https://review.openstack.org/#/c/150352/ .05:27
dikonoorsdague: the changes were reviewed by you.05:28
alex_xudikonoor: you want to ask why the admin permission enforced at db layer before?05:28
dikonoorsdague: as part of these changes, the check for admin was moved from the nova/db/sqlalchemy/api layer to the api layer.05:28
dikonooralex_xu : I notice that before the changes were made a non-admin user is able to create a flavor. But after the changes a non-admin user is not able to..05:29
dikonooralex_xu : I checked and find that as soon as the request reaches the api layer (where you now have a check for whether the user is admin) , the context has is_admin set to False for a non-user (which is expected) ..However, by the time the control reaches the flavor_create method in sqlalchemy/api layer, is_admin for the same non-admin user is set to True05:31
alex_xudikonoor: strange, the db layer hard-code the admin permission checks before05:31
alex_xudikonoor: I think you are right, I know what happened05:33
dikonooralex_xu : In the create flavor flow , the request goes from nova/api layer to nova/compute/flavors to nova/objects/flavor.. Inside the create method of nova/compute/flavors , I see that the context passed to the nova.objects.flavor looks like this05:33
alex_xudikonoor: thanks for the report, do you have file a bug for it, or I can let me fix it?05:33
dikonoor    flavor = objects.Flavor(context=context.get_admin_context(), **kwargs)05:33
dikonoor    flavor.create()05:33
dikonoor    return flavor05:33
alex_xudikonoor: yea, I missed that before :(05:34
dikonooralex_xu : I will file a bug in a few min05:34
alex_xudikonoor: thanks05:34
dikonooralex_xu : but I also want to understand why the permission required for create_flavor is mandated as admin and not picked up from the policy.json05:35
alex_xudikonoor: the fix will be easy, and the fix should be backport to kilo rc05:35
openstackgerritKen'ichi Ohmichi proposed openstack/python-novaclient: Fix typo on class Client sample  https://review.openstack.org/17470305:36
*** nkrinner has joined #openstack-nova05:36
alex_xudikonoor: the orginal goal is remove all the db layer admin permission checks. only make the permission checked by policy.json05:36
alex_xudikonoor: and we want to keep the v2 api behavior as before05:36
alex_xudikonoor: but in this https://review.openstack.org/#/c/150352, I missed the user's request didn't pass down to the db layer05:37
dikonooralex_xu : ok.. but with the latest changes ,  if I add a rule in policy.json that says that create flavor should be permitted for all users , it fails05:37
alex_xudikonoor: sorry, I didn't describe clear. That goal is for v2.1 API05:38
alex_xudikonoor: for v2.1 API the permission checks is only done by policy.json. for v2 API just keep the behavior as before05:38
*** rushiagr_away is now known as rushiagr05:38
dikonooralex_xu : it fails because as soon as the rest call is made , the nova_context.require_admin_context(context) checks if the user is admin or not05:39
alex_xudikonoor: yes, that line is used for keep the v2 api behavor as before (the behavior I miss understood....)05:39
dikonooralex_xu: I tried only create flavor but I see the behavior in others like destory flavor as well.05:39
alex_xudikonoor: yes, it is same mistake with create05:41
dikonooralex_xu : I am a lil confused....so the old behavior mandates that onlly an admin user can create a flavor?05:41
alex_xudikonoor: yes, some of api like that, for example, service05:41
tojuvoneThere has been lively discussion over BP I posted: https://review.openstack.org/#/c/169836/05:42
dikonooralex_xu : but with the old behavior, non-admins could create flavors (and I guess that was probably a bug because the correct context was not propagated through all the layers)05:42
tojuvoneAs result of that the scope of BP is narrowed down and topic if no more good05:42
alex_xudikonoor: yes, for flavor-manage api, that is a bug05:42
*** Sukhdev has quit IRC05:43
dikonooralex_xu : ok.. so the expected behavior should be : 1) only admins should be able to create and delete flavors 2) correct context should be propagated.05:43
tojuvoneTo keep the discussion that lead to that, should I keep the current BP (.rst nem, launchpad url....) or should there just be new BP with correct topic and naming?05:44
*** dsanders has quit IRC05:44
*** Murali has joined #openstack-nova05:45
tojuvonebauzas, cfriesen?05:45
alex_xudikonoor: the old behavior for create and delete flavor is the permission can be controlled by policy.json, right?05:45
dikonooralex_xu : no..the old behavior overides the permissions defined in policy.json by the use of the decorator @require_admin_context against def flavor_create in the db api layer05:47
dikonooralex_xu : ideally the permission should be controlled by policy.json. But both in the old and new behavior , the permission defined in policy.json is overriden05:48
tojuvoneor am I able to change all these names while keeping it under same review?05:48
*** abhishekk has joined #openstack-nova05:48
alex_xudikonoor: but we also pass the admin context to the db call, so that decorator doesn't works05:49
alex_xudikonoor: yes, but we don't want to change the v2 API behavior, only change the v2.1 API behavior05:49
*** ajayaa has joined #openstack-nova05:49
*** Sukhdev has joined #openstack-nova05:50
*** oro has joined #openstack-nova05:50
dikonooralex_xu : because the admin context gets passed instead of the user context  (in the db layer with the old changes) , the call for all users used to go through05:50
dikonooralex_xu : so the primary question is - why is permission not picked up from policy.json and hardcoded05:51
dikonooralex_xu : If we fix the context to pass the right context and remove the hardcoded check ( i e nova_context.require_admin_context(context)) , the permission will be picked up from policy.json05:53
alex_xudikonoor: yes05:53
dikonooralex_xu : and that should apply for all i e delete and create . The old behavior was incorrect and I am not sure if that needs to be supported ?05:55
alex_xudikonoor: yes, fix both for delete and create05:56
alex_xudikonoor: although the user context didn't pass down, but the old behavior is right....05:56
alex_xudikonoor: the admin context is pass down. that means any request can be successful. So the policy.json is used for control permission05:57
*** oomichi has quit IRC05:59
alex_xudikonoor: let me know when you filed bug06:00
dikonooralex_xu : Let's say with the old changes , I put a rule in policy.json that says that create flavor can be done only by admin role, that permission rule will never work. This is because the context always set admin as true . So a non-admin (with the old changes) will always be able to create a flavor06:00
alex_xudikonoor: it's not, the policy enforcement will stop that. The policy enforcement is at very beginning of rest api call https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/contrib/flavormanage.py#L5606:02
dikonooralex_xu : ok.I didn't realize that authorize(context) makes the call for policy enforcement06:04
dikonooralex_xu : Then I think it picks up from policy.json06:05
alex_xudikonoor: we can just fix it now by remove this line https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/contrib/flavormanage.py#L6006:06
*** vilobhmm1 has quit IRC06:06
*** Nikolay_St has joined #openstack-nova06:06
*** Maike has joined #openstack-nova06:07
*** Maike_ has joined #openstack-nova06:08
*** Maike has quit IRC06:12
dikonooralex_xu: I have opened bug https://bugs.launchpad.net/nova/+bug/144533506:13
openstackLaunchpad bug 1445335 in OpenStack Compute (nova) "create/delete flavor permissions should be controlled by policy.json" [Undecided,Confirmed]06:13
alex_xudikonoor: thanks06:14
*** oro has quit IRC06:14
*** emagana has joined #openstack-nova06:16
*** oro has joined #openstack-nova06:16
openstackgerritGhanshyam Mann proposed openstack/nova: Merge server usage tests between v2 and v2.1  https://review.openstack.org/17471906:17
*** Nikolay_St has quit IRC06:17
alex_xudikonoor: do you want to take it? or I can take it?06:17
dikonooralex_xu : I can take it06:18
alex_xudikonoor: ok thanks06:19
*** ankit_ag has joined #openstack-nova06:19
*** Kevin_Zheng has joined #openstack-nova06:19
*** Administrator_ has joined #openstack-nova06:19
*** ajayaa has quit IRC06:19
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex  https://review.openstack.org/17194306:19
openstackgerritAnkit Agrawal proposed openstack/nova: Detach volume after deleting the instance  https://review.openstack.org/14573806:21
*** armax has quit IRC06:22
*** oro has quit IRC06:22
*** Nikolay_St has joined #openstack-nova06:23
*** tjones1 has left #openstack-nova06:24
*** IanGovett has joined #openstack-nova06:26
*** IanGovett1 has quit IRC06:27
*** TobiasE has joined #openstack-nova06:29
*** bnemec has quit IRC06:32
*** hdd has joined #openstack-nova06:32
openstackgerritAnkit Agrawal proposed openstack/nova: 'deleted' filter does not work properly  https://review.openstack.org/17329606:33
*** lpetrut has joined #openstack-nova06:34
*** cfriesen has quit IRC06:34
*** IanGovett1 has joined #openstack-nova06:34
*** IanGovett has quit IRC06:36
*** Longgeek_ has joined #openstack-nova06:37
*** yamahata has joined #openstack-nova06:38
*** Longgeek has quit IRC06:39
*** jyuso1 has quit IRC06:40
*** romainh has joined #openstack-nova06:41
*** tjones1 has joined #openstack-nova06:42
tjones1mik06:42
tjones1oops06:42
tjones1wrong window06:42
*** IanGovett1 has quit IRC06:43
*** vokhrimenko has quit IRC06:44
*** IanGovett has joined #openstack-nova06:44
*** mhorban has quit IRC06:45
*** Sukhdev has quit IRC06:45
*** dulek has joined #openstack-nova06:48
*** IanGovett has quit IRC06:48
*** IanGovett has joined #openstack-nova06:48
*** dave-mccowan has quit IRC06:51
*** achanda has quit IRC06:53
*** IanGovett has quit IRC06:53
*** IanGovett has joined #openstack-nova06:55
*** achanda has joined #openstack-nova06:56
*** sahid has joined #openstack-nova07:00
*** Nikolay_St has quit IRC07:02
*** dsanders has joined #openstack-nova07:02
gilliardHappy Friday!07:03
*** gilliard is now known as gillllliard07:03
*** gillllliard is now known as gillllllliard07:03
*** mpavone has joined #openstack-nova07:03
*** sdake has joined #openstack-nova07:06
*** sdake_ has quit IRC07:09
*** otter768 has joined #openstack-nova07:10
*** jcoufal has joined #openstack-nova07:10
*** markus_z has joined #openstack-nova07:14
*** otter768 has quit IRC07:14
*** Mike_D_wk has quit IRC07:15
tjones1ah gillllllliard - can you take a look at https://review.openstack.org/#/c/127427/9.  mikal asked me to finish it and i believe i have addressed your comments from a while ago07:15
tjones1and happy friday07:16
*** jichen has joined #openstack-nova07:17
gillllllliardno problem tjones107:19
tjones1thanks07:20
gillllllliardoh yes I remember this one. Thanks07:20
*** dsanders has quit IRC07:26
*** asettle has quit IRC07:28
*** matrohon has joined #openstack-nova07:28
*** achanda has quit IRC07:29
openstackgerritsahid proposed openstack/nova: compute: make re-schedule working when resize fails  https://review.openstack.org/17204407:30
*** alexpilotti has joined #openstack-nova07:30
*** haomaiw__ has joined #openstack-nova07:32
*** andreykurilin__ has joined #openstack-nova07:32
*** haomaiwa_ has quit IRC07:32
*** achanda_ has joined #openstack-nova07:32
*** dims__ has joined #openstack-nova07:33
*** yamahata has quit IRC07:34
*** dims__ has quit IRC07:38
openstackgerritNha Pham proposed openstack/nova: Encrypted iSCSI volume has name too long  https://review.openstack.org/17465607:39
*** dsanders has joined #openstack-nova07:39
*** jlanoux has joined #openstack-nova07:40
*** achanda_ has quit IRC07:40
*** asettle has joined #openstack-nova07:41
*** ndipanov has quit IRC07:44
*** yuntong has quit IRC07:45
*** shaohe_feng has quit IRC07:45
*** shaohe_feng has joined #openstack-nova07:46
*** yuntong has joined #openstack-nova07:46
*** ankit_ag has quit IRC07:46
*** rajesht has quit IRC07:46
*** alex_xu has quit IRC07:46
*** zhangtralon has joined #openstack-nova07:47
*** gzhai1 has quit IRC07:47
*** tdurakov has joined #openstack-nova07:47
*** heyongli has quit IRC07:47
*** foexle has joined #openstack-nova07:48
*** atuvenie has joined #openstack-nova07:48
*** heyongli has joined #openstack-nova07:48
openstackgerritzhangtralon proposed openstack/nova: change the reboot rpc call to local reboot  https://review.openstack.org/17011007:48
*** alex_xu has joined #openstack-nova07:48
*** gzhai1 has joined #openstack-nova07:49
*** jistr has joined #openstack-nova07:50
*** andreykurilin___ has joined #openstack-nova07:53
*** andreykurilin__ has quit IRC07:53
*** erkules has quit IRC07:53
*** erkules has joined #openstack-nova07:53
*** yuntong1 has joined #openstack-nova07:54
*** yuntong has quit IRC07:55
openstackgerritPetrut Lucian proposed openstack/nova: Hyper-V: Implement nova rescue  https://review.openstack.org/12715907:55
tdurakovsahid, ping07:57
sahidtdurakov: i will review your patch, but actually since your are updating the behavior i would expect that you explain on the commit message what will happen07:58
tdurakovsahid, here is the bug descripition: https://bugs.launchpad.net/nova/+bug/128471907:59
openstackLaunchpad bug 1284719 in OpenStack Compute (nova) "buggy live migration rollback when using shared storage" [Medium,In progress] - Assigned to Timofey Durakov (tdurakov)07:59
sahidyes but your fix also update the behavior of post_live_migration and you don't speak about it on your commit message07:59
tdurakovsahid, you are right, foget to notice about it in commit message08:00
tdurakovsahid, you are right, forget to notice about it in commit message08:00
sahidtdurakov: i will review it an other time and share with you more detail08:01
*** Hosam has joined #openstack-nova08:02
*** atuvenie has quit IRC08:03
*** dsanders has quit IRC08:04
*** rushiagr is now known as rushiagr_away08:04
tdurakovsahid, about post_live_migration, checked on env, it just cleans up resources on source node. So will be waiting for details from you.08:05
sahidtdurakov: ok cool let me do that before the end of this morning08:05
*** achanda has joined #openstack-nova08:06
sahidi did not verify it but since you did not speak at all about that i wanted to notify this to you08:06
*** apuimedo has joined #openstack-nova08:09
*** Nikolay_St has joined #openstack-nova08:11
*** emagana has quit IRC08:12
*** ajayaa has joined #openstack-nova08:15
*** lucasagomes has joined #openstack-nova08:19
*** unicell has joined #openstack-nova08:19
*** achanda has quit IRC08:20
*** derekh has joined #openstack-nova08:22
*** achanda_ has joined #openstack-nova08:22
*** zz_johnthetubagu is now known as johnthetubaguy08:22
openstackgerritNha Pham proposed openstack/nova: Encrypted iSCSI volume has name too long  https://review.openstack.org/17465608:23
*** sdake has quit IRC08:28
openstackgerritNoel Nelson Dsouza proposed openstack/nova: Nova can stop instance when Fake/Actual compute driver used  https://review.openstack.org/17474608:29
openstackgerritzhangtralon proposed openstack/nova: DB:support for migration objects multi-statues  https://review.openstack.org/16960408:31
openstackgerritGhanshyam Mann proposed openstack/nova: Merge V2 and V2.1 hypervisor functional tests  https://review.openstack.org/16708208:32
*** c0m0 has joined #openstack-nova08:35
*** achanda_ has quit IRC08:35
openstackgerritGhanshyam Mann proposed openstack/nova: Share V2 and V2.1 images functional tests  https://review.openstack.org/16785408:38
*** tjones1 has left #openstack-nova08:40
*** ndipanov has joined #openstack-nova08:40
ndipanovpczesno, there?08:40
*** ritesh has joined #openstack-nova08:41
*** bauzas is now known as bauwser08:41
riteshHi,08:42
riteshAll08:42
riteshWhen I eexecte nova absolute-limits08:42
riteshIt gives me Max RAM of 5120008:43
riteshBut machine ram is 16 GB only08:43
*** IanGovett1 has joined #openstack-nova08:43
openstackgerritGhanshyam Mann proposed openstack/nova: Merge availability_zone tests between v2 and v2.1  https://review.openstack.org/17101708:43
*** andreykurilin___ has quit IRC08:43
riteshhow do I make my abosulte limit of  Max RAM to 16 GB08:43
*** IanGovett has quit IRC08:44
*** atuvenie has joined #openstack-nova08:44
openstackgerritzhangtralon proposed openstack/nova: add support for muti_statuses_query from migration  https://review.openstack.org/16963108:45
bauwseris it me or are we spammed by a company asking us to create accounts for testing N08:46
alex_xundipanov: hi, would you like give me some suggestion on https://review.openstack.org/170396 when you have time? Really hope I can get some suggestion from you :)08:46
bauwser?08:46
bauwserdamn hell.08:46
*** xylan has left #openstack-nova08:47
alex_xubauwser: good morning, a question about can I just free to add something in https://etherpad.openstack.org/p/liberty-nova-summit-ideas ? Then later the core team will figure whether my item is worth discussion or not? or at least I need get some suggestion from core first?08:49
*** flwang1 has joined #openstack-nova08:49
bauwseralex_xu: afternoon08:50
*** pcaruana has joined #openstack-nova08:50
bauwseralex_xu: you mean adding an idea to the etherpad ?08:50
alex_xubauwser: yea08:50
*** mpaolino has joined #openstack-nova08:50
bauwseralex_xu: oh, go for it, that's what etherpads are used for :)08:50
alex_xubauwser: ok thanks :)08:51
ndipanovalex_xu, yeah it's on my list - sadly, still fixing kilo bugs :(08:51
bauwseralex_xu: that said, I have the same voice as you - not a core y'know ;)08:51
openstackgerritGhanshyam Mann proposed openstack/nova: Share hide server add tests between v2 and v2.1  https://review.openstack.org/17230208:51
bauwseralex_xu: but the etherpad is really useful to see ideas08:52
alex_xundipanov: yea, the kilo bug is priority, thanks in advance :)08:52
alex_xubauwser: yea, but i know you are helpful :)08:53
openstackgerritGhanshyam Mann proposed openstack/nova: Merge server usage tests between v2 and v2.1  https://review.openstack.org/17471908:53
pczesnondipanov: hi08:54
openstackgerritAndrzej Kuriata proposed openstack/nova-specs: Scheduling VMs to servers taking into account the health of the platform.  https://review.openstack.org/17476408:56
ndipanovpczesno, just to confirm - the pci_requests db field bug you fixed - this was triggered on instance.create()08:57
ndipanovand looking at Git it's been there for at least 6 months08:57
pczesnondipanov: yes08:57
pczesnondipanov: actually on _from_db_object08:58
ndipanovcalled from create()08:58
pczesnondipanov: right08:58
ndipanovbah08:58
*** claudiub has joined #openstack-nova09:01
claudiubjohnthetubaguy: hello sir09:02
johnthetubaguyclaudiub: hello hello09:02
johnthetubaguyclaudiub: hows things?09:02
claudiubjohnthetubaguy: it appears I have missed a bp that needs to be reapproved for L. :) https://blueprints.launchpad.net/nova/+spec/hyper-v-remotefx was approved for Kilo, or earlier09:02
claudiubjohnthetubaguy: fine I guess. Later, could you take a look at a spec I've submitted? it is about live resize09:03
*** ajayaa has quit IRC09:03
*** arnaud___ has joined #openstack-nova09:03
claudiubjohnthetubeguy: I would like your opinion on it. :)09:04
johnthetubaguyclaudiub: ah live resize, so you get a prize for including the word resize in there, the previous specs I disliked because they were trying to hotplug outside of the flavor concept09:04
claudiubjohnthetubaguy: yeay! :D when / how can I claim my prize? :D09:05
claudiubjohnthetubaguy: spec: https://review.openstack.org/#/c/141219/09:05
*** Hosam has quit IRC09:05
johnthetubaguyclaudiub: the prize might be a spec review :p09:05
claudiubjohnthetubaguy: awesome. :D09:06
openstackgerritzhangtralon proposed openstack/nova: resize-revert can break anti-affinity policy  https://review.openstack.org/16011009:09
openstackgerritMh Raies proposed openstack/python-novaclient: Adding missing nova read only CLI test  https://review.openstack.org/17114809:10
*** rushiagr_away is now known as rushiagr09:10
*** otter768 has joined #openstack-nova09:11
*** otter768 has quit IRC09:15
*** lpetrut has quit IRC09:16
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: WIP: Update hypervisor support matrix with column for kvm on system z  https://review.openstack.org/17239109:17
*** gszasz has joined #openstack-nova09:18
*** shaohe_feng has quit IRC09:24
*** rajesht has joined #openstack-nova09:32
*** obondarev has quit IRC09:32
*** arnaud___ has quit IRC09:33
*** obondarev has joined #openstack-nova09:33
johnthetubaguyclaudiub: I need to give this some more thought, but that spec is really interesting09:34
johnthetubaguyclaudiub: have you considered adding resize into the live-migrate path?09:34
johnthetubaguyclaudiub: its almost certainly a bad idea, but it helps see both sides of the design choices09:35
claudiubjohnthetubaguy: I'm not sure I get what you mean. You mean performing live-resize while doing a live-migrate?09:36
claudiubjohnthetubaguy: I don't know if that's possible, although, I have thought about it a bit, at least from the hyper-v perspective. hyper-v prepares a new vm on the destination host, so i'm wondering if it's not possible to have different memory/vcpu.. But again, in-place resize would still be the safest bet.09:38
johnthetubaguyclaudiub: I think thinking one after the other, you live-migrate to somewhere with space, then do the live-resize, worst case, wait till the next reboot to get the rest of the non-hotpluggable resources09:39
*** claudiub_ has joined #openstack-nova09:39
johnthetubaguyclaudiub_: maybe you missed, the last message of mine: I think thinking one after the other, you live-migrate to somewhere with space, then do the live-resize, worst case, wait till the next reboot to get the rest of the non-hotpluggable resources09:40
*** claudiub has quit IRC09:42
claudiub_johnthetubaguy: well, I'm thinking that, if you do not have enough resources on the current host, live-resize should fail. Then, yes, a user can request a live-migration to another free-er host and perform the live-resize there. as for waiting for a reboot for the non-hotpluggable resources.. that just sounds like a regular resize to me. :)09:45
*** aix has joined #openstack-nova09:45
*** tralon_ has joined #openstack-nova09:45
johnthetubaguyclaudiub_: thats really bad, live-migrate is admin focused, live-resize is user focused09:45
*** Hosam has joined #openstack-nova09:46
johnthetubaguyclaudiub_: now I don't mind not doing live-migrate in the initial live-resize thats part of the resize code path, but I think we need to leave the door open to doing live-migrate to find space for the resize (both regular and live resize, actually)09:46
*** zhangtralon has quit IRC09:46
johnthetubaguyclaudiub_: we so need a good summit session on this, I should try prep some "grand" vision devref for folks to review before hand09:47
claudiub_johnthetubaguy: I see what you mean now. Yeah, that's a good idea.09:47
johnthetubaguyclaudiub_: the problem is, this is not going to be too high priority, given the current vision of Nova, but there is some great tidy ups this could help bring us, which could really bump the priority of all this work09:48
*** jamielennox is now known as jamielennox|away09:49
claudiub_johnthetubaguy: I see.09:49
claudiub_johnthetubaguy: So, I'll take a look at how to also include the live-migration part in the spec, although, I think it would be simpler for this spec to be implemented in 2 parts: 1st part: live-resize on the same host; 2nd part: live-migration+live-resize09:51
kashyapmatrohon: Are you going to re-submit this with Doc-Impact? -- https://review.openstack.org/#/c/171098/1//COMMIT_MSG09:51
openstackgerritNha Pham proposed openstack/nova: Encrypted iSCSI volume has name too long  https://review.openstack.org/17465609:52
claudiub_johnthetubaguy: since this will require quite a bit of work, since it requires work on all the drivers.09:52
johnthetubaguyclaudiub_: totally agreed with the two phases, I was thinking about adding resize into live-migrate to make those two phases quicker, but its possible the wrong long term direction09:52
*** romainh has left #openstack-nova09:52
kashyapmatrohon: While you're respinning, you might also want to update the commit message with some of the rationale I noted here as a response to mriedem - https://review.openstack.org/#/c/171098/1/nova/virt/libvirt/driver.py09:53
johnthetubaguyclaudiub_: if its multiple phases, we might want to write out a devref for the overall long term plan for resize, migrate and live-migrate, and live-resize09:53
bauwserjohnthetubaguy: there is a running concern about if those migrations should enforce some scheduling rules, even if no dest host is provided09:54
bauwserjohnthetubaguy: I remember either a bug or a bp, lemme find it09:54
bauwserjohnthetubaguy: but that's really bummering operators09:54
claudiub_johnthetubaguy: So, i guess a devref is different from a spec?09:54
kashyapRandomly chiming in, agreed with johnthetubaguy's very useful suggestion -- about a cohesive devref patch that outlines resize, migrate, live-migrate plans09:55
bauwserclaudiub_: devref is documentation :)09:55
johnthetubaguyclaudiub_: its this kind of thing: http://docs.openstack.org/developer/nova/devref/cells.html09:55
johnthetubaguyclaudiub_: lives in the nova tree09:55
claudiub_johnthetubaguy: ah, I see.09:55
bauwserjohnthetubaguy: so yeah, about resizes and migrates, the problem is that some policies have to be checked, like alloc ratios09:56
bauwserjohnthetubaguy: like https://bugs.launchpad.net/nova/+bug/121494309:57
openstackLaunchpad bug 1214943 in OpenStack Compute (nova) "Live migration should use the same memory over subscription logic as instance boot" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza)09:57
claudiub_not sure, but doing a nova resize, isn't the scheduler choosing a viable host for the resize?09:57
johnthetubaguybauwser: agreed, thats part of the big issues, and actually you don't always want to check them09:57
johnthetubaguybauwser: we need to get that all down into the docs09:57
bauwserjohnthetubaguy: well, there are 2 things which are conflicting09:57
bauwserjohnthetubaguy: 1/ we all agreed on leaving the operator decide to migrate the host and bypass the scheduler if a dest host is provided, because that's like an emergency situation and the operator has a better knowledge than the scheduler09:58
bauwserjohnthetubaguy: 2/ we still need to make sure that a migration wouldn't conflict some policy like anti-affinity09:58
johnthetubaguybauwser: my current take is, admins that choose a host can break the rules, users requesting things have to follow the rules, we need to be better at making sure they know when they are conflicting09:59
bauwserclaudiub_: if a dest-host is specified in the CLI (and so in the API body), it totally prevents the scheduler to be called09:59
bauwserjohnthetubaguy: that's my take too09:59
claudiub_bauwser: yeah, i was referring if no dest-host is provided. :)10:00
bauwserjohnthetubaguy: we need somehow to find a tradeoff between a total liberty and something which wouldn't break user deployments10:00
johnthetubaguybauwser: we are missing a piece right, need a way an admin can check if their choice violates any rules10:00
bauwserjohnthetubaguy: yeah, a possible check_my_choice() scheduler API method eh ? :)10:00
johnthetubaguybauwser: we side step that by live-migrating host to host, generally, rather than mixing VMs from different hosts10:00
johnthetubaguybauwser: yup10:00
johnthetubaguyclaudiub_: this is the stuff I think we need to capture in the dev ref, like a future vision for all these related operations10:01
bauwserjohnthetubaguy: sounds doable, that's only in the scheduler manager that we pick one host to select it based on the whole hosts that passed the filters10:01
johnthetubaguyclaudiub_: I am happy helping write some of that10:01
bauwserjohnthetubaguy: that's kinda quick-win IMHO10:01
johnthetubaguybauwser: +110:02
bauwserjohnthetubaguy: okay, I like the idea to trace it10:02
johnthetubaguybauwser: claudiub_: lets be sure to get all these ideas written up into a dev ref for live-migrate, migrate and resize and live-resize10:02
claudiub_johnthetubaguy: cool. I can write it, no problem, but I would like your help in touching up the details. :)10:02
bauwserjohnthetubaguy: we also have a scheduler stuff topic for the summit, that's related10:02
johnthetubaguyclaudiub_: happy to10:02
bauwserjohnthetubaguy: like a declaration of intent ?10:03
johnthetubaguybauwser: exactly, like the cells and upgrade ones we added recently10:03
bauwserjohnthetubaguy: a manifesto or instead, a list of limitations that need to be sorted out10:03
bauwser?10:03
johnthetubaguybauwser: I really want to see what for the current scheduler work too10:03
johnthetubaguybauwser: its both really10:03
johnthetubaguybauwser: something we look back on in 12 months to know why we chose the direction we did10:04
bauwserjohnthetubaguy: well, I was thinking about a manifesto, but it was rather for the big hill stuff I'm going to ride on https://blueprints.launchpad.net/nova/+spec/shared-state-scheduler10:04
bauwserjohnthetubaguy: you mean splitting ? :)10:05
johnthetubaguybauwser: I don't like giving things names, they end up meaning too much, lets just try write down what we are thinking, basically10:05
johnthetubaguynot really splitting, more the interface, and the resource manager call back ideas, etc10:05
johnthetubaguyand the scaling ideas10:05
johnthetubaguylike the current transition ideas10:06
johnthetubaguy(in the end we get a history)10:06
bauwseroh :)10:06
johnthetubaguyif we keep adding to it10:06
bauwserwell, we discussed a lot during midcycles but sure we can draw that out10:06
openstackgerritPrzemyslaw Czesnowicz proposed openstack/nova: Fix handling of pci_requests in consume_from_instance.  https://review.openstack.org/17339410:06
openstackgerritPrzemyslaw Czesnowicz proposed openstack/nova: Use list of requests in InstancePCIRequests.obj_from_db.  https://review.openstack.org/17444810:06
johnthetubaguyyeah, I mean it can be really short, ideally10:06
openstackgerritAndrzej Kuriata proposed openstack/nova-specs: Scheduling VMs to servers taking into account the health of the platform.  https://review.openstack.org/17476410:07
johnthetubaguyand then we iterate as we know more10:07
bauwsermmm10:07
bauwsertake the cells effort, we need to amend some things IMHO10:07
bauwserso I appreciate the iterate approach10:07
johnthetubaguyright, that fine, we can do that10:07
bauwseriterative10:07
johnthetubaguyprobably will do post summit as we change course10:07
bauwseryeah, I would expect that devref to be merged post-summit10:08
johnthetubaguyI would like to keep an element of history, like here are the steps we took, but beyond that, we can change it10:08
*** ozamiatin has quit IRC10:08
johnthetubaguybauwser: before summit is better, as it shows we don't need that summit session10:08
johnthetubaguybauwser: but yes, after summit will be best for some stuff10:08
johnthetubaguyalthough I like the dev ref submission as a talking point in the summit session also… hmm10:08
bauwserjohnthetubaguy: before summit for proposing, after summit for merging IMHO10:08
johnthetubaguybauwser: right, but if we agree already, just merge it10:09
bauwserjohnthetubaguy: oh hell yeah, of course :)10:09
johnthetubaguybauwser: its how I hope to deal with folks who can't make the summit too10:09
bauwseretherpads are a good way to get thoughts from the summit (I did that a lot when I was having schedule conflicts)10:10
bauwserbut that requires clean etherpads with typy typy people :)10:10
johnthetubaguyyeah10:10
johnthetubaguywe started the prior reading stuff, which helped focus things10:11
bauwseragreed10:11
johnthetubaguybut I don't want heavy process for no reason!10:11
bauwser:)10:11
*** BobBall_AWOL is now known as BobBall10:12
bauwserjohnthetubaguy: so, to summarize and put actions on names, do you want me to draft something in devref about the sched stuff ?10:12
BobBalljohnthetubaguy: Apparently you can re-add Citrix CI to the voting list? :) {'Collected': 4, 'Finished': 119, 'Obsolete': 103, 'Running': 5}10:12
BobBallSorry - wrong paste..10:12
BobBalljohnthetubaguy: 'Failed': 27, 'Aborted: Failed to copy logs': 1, 'Passed': 186,10:13
BobBallNearly 90% pass rate10:13
johnthetubaguyBobBall: can I, hmm, thats news to me10:14
johnthetubaguyBobBall: I most have got that as part of releasing python-novaclient or something10:14
openstackgerritAndrzej Kuriata proposed openstack/nova-specs: Scheduling VMs to servers taking into account the health of the platform.  https://review.openstack.org/17476410:15
johnthetubaguyBobBall: did you just exclude the test?10:15
BobBalloh... Maybe not then :) mreirdem suggested I pinged him you or mikal10:15
BobBallyes10:15
*** lpetrut has joined #openstack-nova10:15
johnthetubaguyBobBall: I can tell mikal to do it… the infra folks might know otherwise10:16
BobBallOk - np10:16
johnthetubaguyBobBall: appreciate you fixing it though, at least its going to reporting well now, if not voting10:16
johnthetubaguyBobBall: are you testing the kilo proposed branch by chance? that might be useful10:17
BobBallNo, pretty sure we're not10:17
openstackgerritLianhao Lu proposed openstack/nova: Convert bandwidth_usage related timestamp to UTC native datetime  https://review.openstack.org/17321310:20
*** kmartin has quit IRC10:20
*** b3nt_pin has joined #openstack-nova10:20
*** b3nt_pin is now known as Guest9308110:20
BobBallbtw - pass rate is actually higher than that... Just realised that still includes some failures from before that test was excluded (when a 'recheck' happens the timestamp of the test is udpated when the test result is marked as obsolete)10:20
*** RuiChen has quit IRC10:23
openstackgerritAndrzej Kuriata proposed openstack/nova-specs: Scheduling VMs to servers taking into account the health of the platform.  https://review.openstack.org/17476410:24
openstackgerritzhangtralon proposed openstack/nova: remove the CONF.allow_migrate_to_same_host  https://review.openstack.org/11860410:25
*** abhijeetm has joined #openstack-nova10:28
openstackgerritTomi Juvonen proposed openstack/nova-specs: New nova API call to mark nova-compute down  https://review.openstack.org/16983610:30
openstackgerritzhangtralon proposed openstack/nova: add support for muti_statuses_query from migration  https://review.openstack.org/16963110:30
*** pixelb has joined #openstack-nova10:34
openstackgerritzhangtralon proposed openstack/nova: resize-revert can break anti-affinity policy  https://review.openstack.org/16011010:36
openstackgerritTomi Juvonen proposed openstack/nova-specs: New nova API call to mark nova-compute down  https://review.openstack.org/16983610:43
openstackgerritTomi Juvonen proposed openstack/nova-specs: New nova API call to mark nova-compute down  https://review.openstack.org/16983610:45
*** ritesh has quit IRC10:45
johnthetubaguybauwser: sorry, got distracted, I would love you to start drafting some scheduler devref stuff, happy to help review and contribute to that10:46
bauwserjohnthetubaguy: okay, will do then10:46
johnthetubaguybauwser: keeping this stuff documented is an important part of keeping us open to new folks, I feel10:46
bauwserjohnthetubaguy: I have some talks to do on Monday evening plus on the Summit, so I keep track of all the efforts10:46
bauwserjohnthetubaguy: totally agreed - see my email about Gantt naming for example10:47
bauwserit was confusing and sending bad signal10:47
*** zhenguo has quit IRC10:50
*** flwang1 has quit IRC10:54
johnthetubaguybauwser: yeah, thats cool10:55
*** dulek_ has joined #openstack-nova10:59
sdaguendipanov: https://bugs.launchpad.net/nova/+bug/1445021/comments/3 is a simple reproduce for that bug11:00
openstackLaunchpad bug 1445021 in OpenStack Compute (nova) "nova-compute does not start after upgrade from juno->kilo if there are boot from volume servers running" [High,Confirmed]11:00
sdagueand, it's 100% reproducable11:01
ndipanovsdague, right - let me comment on it11:01
ndipanovbut hte more I think about it11:01
ndipanovthe more I want to just completely remove get_disk_over_committed_size_total11:02
*** dulek has quit IRC11:02
ndipanovfor a bunch of backends it makes no sense11:02
ndipanovlike rbd11:02
*** dims__ has joined #openstack-nova11:02
johnthetubaguyndipanov: sdague: FWIW RC2 is delayed until monday due to unrelated testing issues11:02
sdaguejohnthetubaguy: ok, that's fine11:03
*** Marga_ has joined #openstack-nova11:03
johnthetubaguyyeah, gives us some more time11:03
ndipanovjohnthetubaguy, so by monday we need patches cherry picked to ?? brnch?11:03
ndipanovbranch*11:03
*** Marga_ has quit IRC11:03
sdaguestable/kilo branch11:03
johnthetubaguyndipanov: ideally, yes, but one step at once11:03
*** Marga_ has joined #openstack-nova11:04
ndipanovsdague, stable/kilo ?! interesting11:04
ndipanovnot proposed/kilo??11:05
*** aix has quit IRC11:05
ndipanovjohnthetubaguy, you mean tag bugs first? or?11:05
johnthetubaguyndipanov: yeah, I am not sure about that now11:05
johnthetubaguyndipanov: tag them and get them committed to master11:05
ndipanovok11:05
johnthetubaguynot sure about the branch now11:05
ndipanovand then we see11:06
ndipanovcool11:06
ndipanovmakes sense11:06
ndipanovpczesno, ^^11:06
ndipanovso it would be great to have those tests fixed ASAP11:07
ndipanovshout if you need help11:07
* ndipanov &11:07
*** annegentle has joined #openstack-nova11:07
*** tralon_ has quit IRC11:10
pczesnondipanov: ok, i'm working on that11:11
*** otter768 has joined #openstack-nova11:11
*** otter768 has quit IRC11:16
*** beagles has quit IRC11:16
*** abhijeetm has quit IRC11:19
*** abhijeetm has joined #openstack-nova11:20
*** ishant has quit IRC11:22
*** arnaud___ has joined #openstack-nova11:22
sdaguendipanov: so are you still working on a patch for the nova-compute crash?11:25
ndipanovsdague, yes11:25
sdagueok, great11:25
ndipanovsdague, will post an idea today11:26
ndipanovand then we can discuss from there11:26
ndipanovhad a quick fix done yesterday - but decided it's not actually aggressive enough11:26
*** mwagner_lap has quit IRC11:26
sdaguewell, do we know if it addresses the fail condition?11:27
sdaguebecause, honestly, lets just handle that for now, and a more invasive thing can come later11:28
*** arnaud___ has quit IRC11:29
*** ferest has joined #openstack-nova11:35
*** Administrator_ has quit IRC11:36
*** Kevin_Zheng has quit IRC11:36
ndipanovsdague, you think so?11:39
ndipanovlet me post the fix then...11:40
sdaguendipanov: we're really late in a release11:40
sdaguewe know a circumstance where nova-compute crashes11:40
sdagueeven though it's non fatal to the guests11:40
ndipanovfair enough11:40
sdaguewe should fix that with the smallest fix possible11:40
*** romainh has joined #openstack-nova11:41
*** ferest has quit IRC11:44
*** dulek_ has quit IRC11:46
*** dprince has joined #openstack-nova11:47
*** beagles has joined #openstack-nova11:48
*** beagles is now known as Guest1340411:49
*** dulek has joined #openstack-nova11:49
*** cbits has joined #openstack-nova11:52
*** cbits has left #openstack-nova11:55
openstackgerritNikola Dipanov proposed openstack/nova: libvirt: make _get_instance_disk_info conservative  https://review.openstack.org/17483611:57
ndipanovsdague, ^^11:57
ndipanovsdague, if you have a devstack running that can easily test it out - that would be awesome11:59
sdagueI don't right now, as I'm dealing with an oslo message bug11:59
ndipanovsdague, link12:00
ndipanovjust curious12:00
sdagueI haven't file it yet12:00
ndipanovah12:00
sdague2015-04-17 08:00:20.004 ERROR oslo_messaging._drivers.impl_rabbit [-] AMQP server on 127.0.0.1:5672 is unreachable: Too many heartbeats missed. Trying again in 1 seconds.12:00
sdague2015-04-17 08:00:20.873 INFO oslo_messaging._drivers.impl_rabbit [-] A recoverable connection/channel error occurred, trying to reconnect: Too many heartbeats missed12:00
sdaguebut that's it, over and over and over again12:00
ndipanovah yeah12:00
ndipanovheartbeat implementation is ... hacky at best12:00
ndipanovI complained about it on the original review12:00
ndipanovbut getting it right would require quit a bit of  refactoring12:01
sdagueok, so right now, it's effectively blocking all the rpc from working12:01
ndipanov hmmm12:01
ndipanovthere must be a timeout you can increase as a first quick fix?12:03
*** aix has joined #openstack-nova12:03
matrohonkashyap : hi12:04
matrohondanpb : hi12:05
*** rfolco has joined #openstack-nova12:06
kashyapmatrohon: Did you see the scrollback message for you?12:06
*** aysyd has joined #openstack-nova12:06
matrohonkashyap : yes, I'll resubmit and enhance the commit msg with your explanations12:06
matrohonkashyap : however, I would love to better understand danpb POV first12:07
kashyapmatrohon: Okay. You can ask him on the review, he's usually very responsive there as he's not in this channel.12:08
matrohonkashyap : IIUC, danpb wants me to remove TUNNELED flag for blog migration, and also for shared migration. Do you understand it this way too?12:08
kashyapmatrohon: Yeah, but on the other hand,12:09
matrohonkashyap : I'll do12:09
kashyapmatrohon: "it is nice to maximise security where possible for non-block migration by keeping TUNNELLED. I'm probably leaning slightly more towards the consistency view point though."12:09
kashyapmatrohon: So, for consistency's sake you can remove it for non-block migration as well. If there are objections, people will raise on the review.12:10
matrohonkashyap : yes, which means to me that he wants both flags to be removed12:10
matrohonkashyap : +112:10
kashyapYes.12:10
matrohonkashyap : May I put a co-authored flag, to mention your deep dive work on this bug?12:11
*** annegentle has quit IRC12:12
kashyapmatrohon: If you prefer, you can. I don't mind either way.12:12
matrohonkashyap : ok12:12
*** lucasagomes is now known as lucas-hungry12:13
kashyapmatrohon: Thanks for asking.12:14
*** tojuvone has quit IRC12:17
*** mwagner_lap has joined #openstack-nova12:19
*** dulek has quit IRC12:19
*** apuimedo has quit IRC12:21
*** dulek has joined #openstack-nova12:22
*** arnaud___ has joined #openstack-nova12:26
*** pask81 has joined #openstack-nova12:26
*** dave-mccowan has joined #openstack-nova12:26
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: Update hypervisor support matrix with kvm on system z  https://review.openstack.org/17239112:26
sdaguendipanov: surely it's not hard for you to build a local devstack to test that patch, right?12:30
*** arnaud___ has quit IRC12:30
ndipanovsdague, ah yeah sure - but you need to be convinced :)12:33
ndipanovthat12:33
ndipanov's what I meant12:33
*** doude_ has quit IRC12:35
*** popw has quit IRC12:35
*** doude has joined #openstack-nova12:37
*** jaypipes has joined #openstack-nova12:38
*** deepthi has quit IRC12:42
*** vladikr has joined #openstack-nova12:43
openstackgerritsahid proposed openstack/nova: libvirt: introduce new method to guest tablet device  https://review.openstack.org/17485312:48
openstackgerritsahid proposed openstack/nova: libvirt: enhance method to return use_usb_tablet from image prop  https://review.openstack.org/17485412:48
*** kudryashova has joined #openstack-nova12:50
sahidhello johnthetubaguy i have a blueprint and i was expected to make it approved without a spec. when you have a moment can you tell me if that makes sens for you ?12:51
sahidhttps://blueprints.launchpad.net/nova/+spec/virt-configure-usb-tablet-from-image12:51
*** baoli has joined #openstack-nova12:52
bauwsersahid: sounds like trivial, no ?12:53
tdurakovdansmith: ping12:53
*** cbader has joined #openstack-nova12:53
johnthetubaguysahid: so basically its about the option of extracting the hw_use_usb_tablet setting from the image?12:54
sahidjohnthetubaguy: exactly, i have refered a bp to give the opportunity to trace it12:54
sahidbauwser: yes that is why i ask12:54
bauwserwell, it's more about an implementation point that a real design modification, so I would hope to see no specs needed12:56
bauwsers/that/than (TGIF)12:56
johnthetubaguysahid: I will approve it, mostly as its best reviewed in code12:57
johnthetubaguysahid: we really need to sit down and formalise our use of image properties, but thats a separate concern12:58
*** baoli_ has joined #openstack-nova12:58
sahidjohnthetubaguy: yes, when i hve implemetned the code i thout about that too12:58
sahidi thought12:59
sahidjohnthetubaguy: anyway thanks12:59
bauwserjohnthetubaguy: do you know if we fixed the Image metadata properties in the objects btw. ?12:59
bauwserjohnthetubaguy: I faced a problem that they were not specifically versioned in Kilo, but I heard that it was WIP12:59
bauwserI can check, for sure13:00
johnthetubaguybauwser: I am unsure if we got that done, I don't remember seeing that blueprint get completed13:00
openstackgerritzhangtralon proposed openstack/nova: add support for muti_statuses_query from migration  https://review.openstack.org/16963113:00
* bauwser checkinh13:00
*** cbader has quit IRC13:00
bauwserhell no :(13:01
*** baoli has quit IRC13:01
johnthetubaguysahid: given the lack of a spec, please be sure to add the DocImpact flag in at commit you add the property13:01
johnthetubaguybauwser: I added a summit idea suggestion around that, but thats more an action for me to write up the approach around flavors and images, etc13:02
*** shaohe_feng has joined #openstack-nova13:02
bauwserjohnthetubaguy: cool, because that becomes necessary13:02
bauwserjohnthetubaguy: http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/request-spec-object.html is now merged and could be somehow needing it13:02
bauwserI mean s/merged/approved - dammit Friday !13:02
*** xyang1 has joined #openstack-nova13:04
sahidjohnthetubaguy: did not think about this, i will do, thanks13:04
johnthetubaguybauwser: ah, interesting, yeah, thats true, although technically its probably something the scheduler filters and weights could register interest in specific "request" types, or something, but yes13:04
johnthetubaguysahid: cool, thank you!13:04
bauwserjohnthetubaguy: yeah that's something we can trick, but it would be cleaner if our interfaces were clear13:05
*** gszasz has quit IRC13:06
johnthetubaguybauwser: agreed, but I guess I always expect user request + image pros + flavor -> request spec13:07
*** c0m0 has quit IRC13:07
bauwserjohnthetubaguy: congrats http://civs.cs.cornell.edu/cgi-bin/results.pl?id=E_4a879ff581b99e7a13:07
*** abhishekk has quit IRC13:08
ndipanovjohnthetubaguy, wow by 2 votes13:09
ndipanovcongrrats13:09
*** lucas-hungry is now known as lucasagomes13:09
openstackgerritsahid proposed openstack/nova: libvirt: enhance method to return use_usb_tablet from image prop  https://review.openstack.org/17485413:10
sahidjohnthetubaguy: o/13:10
johnthetubaguyoh, thats crazy13:10
johnthetubaguywow13:10
sahida lot of responsability now !13:11
ndipanovto both of those people :D13:11
sahidndipanov: +113:11
*** otter768 has joined #openstack-nova13:12
*** artom has joined #openstack-nova13:14
*** smccully has joined #openstack-nova13:14
kashyapjohnthetubaguy: Congratulations (or condolences? :-)). Very well deserved!13:15
bauwsereh eh :)13:15
gillllllliardkashyap :)13:15
*** shaohe_feng1 has joined #openstack-nova13:15
johnthetubaguyyeah, good news is, turns out every vote counts, a victory for making people vote (next time at least).13:16
*** shaohe_feng has quit IRC13:16
openstackgerritAlex Xu proposed openstack/nova: Add rpc client side version control for conductor task API  https://review.openstack.org/17415213:17
openstackgerritAlex Xu proposed openstack/nova: Update rpc version aliases for kilo  https://review.openstack.org/17414813:17
alex_xujohnthetubaguy: wow, congratulations :)13:17
*** otter768 has quit IRC13:17
bauwserjohnthetubaguy: can we begin spamming you ? :)13:19
johnthetubaguythanks all, I hope I will be a good servant to whole Nova community, I will certainly try my best!13:19
johnthetubaguybauwser: lol, sure, no promise I will read the spam though...13:20
bauwserjohnthetubaguy: ask Bill Gates, he has a dedicated team for that13:20
*** edleafe is now known as figleaf13:20
sdaguejohnthetubaguy: congrats13:20
*** hdd has quit IRC13:21
matrohonkashyap : re13:23
kashyapmatrohon: Yep?13:23
*** rushiagr is now known as rushiagr_away13:23
*** Nikolay_St has quit IRC13:24
matrohonkashyap : regarding your comment on the review, live migration succeed with shared storage, since it only needs one channel for memory, wheras it needs two for live block migration13:25
kashyapmatrohon: What about it?  Just correcting terms: s/channel/data stream :-)13:25
*** mriedem has joined #openstack-nova13:26
matrohonkashyap : fine, but the logic is correct?13:26
*** abhishekk has joined #openstack-nova13:27
mriedemwubba wubba wubba13:29
kashyapmatrohon: Hmm, that wording is not quite clear, let me paste what I have, you can reuse if it's clear to you:13:29
*** Murali has quit IRC13:29
*** markus_z has quit IRC13:29
matrohonkashyap :13:29
matrohonkashyap : I will reuse the paragraph you submited in your review13:30
kashyapmatrohon: Yes, you can add it as is too.13:30
kashyapmatrohon: More details never hurt :-)13:30
*** cfriesen has joined #openstack-nova13:32
*** jecarey has joined #openstack-nova13:32
kashyapmatrohon: I didn't submit any review per se, just added my analysis as a comment - I'll assume you're referring to that.13:33
matrohonkashyap : yep13:33
*** dulek has quit IRC13:33
*** busterswt has joined #openstack-nova13:33
dansmithsdague: know what I just realized?13:34
*** rushil has joined #openstack-nova13:35
sdaguedansmith: I *literally* have no idea13:35
*** Marga_ has quit IRC13:35
mriedemyou don't have your nick going?13:35
*** dansmith is now known as superdan13:35
*** Nikolay_St has joined #openstack-nova13:36
mriedemi hope there was more to the story13:36
superdansdague: we're going to have to upgrade the db to the migration *before* the check, then do the flavor migration, then finish13:36
sdaguesuperdan: so... I shouldn't have moved the order?13:36
superdansdague: no, it's more complicated than that13:37
superdansdague: we're going to have to do something like:13:37
superdannova-manage db sync --to-version=26013:37
superdannova-manage flavor-migrate13:37
superdannova-manage db-sync13:37
sdagueok, except, isn't this going to be the 1st migration in kilo?13:37
sdaguesorry, in liberty13:37
superdanoh13:38
superdanright, this was in kilo13:38
superdanduh13:38
superdanso we can do the flavor-migrate first13:38
sdagueyeh, so I think you just need to modify the first patch to not move the upgrade_service call13:38
superdanyou moved the order to run the db migrations first, right?13:38
superdanright13:38
sdagueyeh, which was probably wrong13:39
superdanfirst, though, I should get coffee started13:39
sdaguebut I didn't think it through13:39
*** busterswt has quit IRC13:39
sdagueyes, coffee is good13:39
*** busterswt has joined #openstack-nova13:39
*** Guest13404 is now known as beagles13:41
*** dulek has joined #openstack-nova13:42
*** vokhrimenko has joined #openstack-nova13:43
*** Nikolay_St has quit IRC13:43
openstackgerritMathieu Rohon proposed openstack/nova: Remove tunnel flag for live migration  https://review.openstack.org/17109813:44
*** neelashah has joined #openstack-nova13:46
*** neelashah1 has joined #openstack-nova13:47
*** mhorban has joined #openstack-nova13:49
*** eharney has joined #openstack-nova13:50
*** neelashah has quit IRC13:51
*** dnalezyt has joined #openstack-nova13:52
*** foexle has quit IRC13:52
*** mpaolino has quit IRC13:53
ndipanovsdague, so with my patch it will not fail anymore13:55
ndipanovit will hapily start13:55
sdaguendipanov: cool13:55
ndipanoveven though your instance is toast with the tgtd down13:55
ndipanovsdague, let me fix that pep8 fail13:55
sdaguendipanov: well if you start tgt again, it's fine13:55
ndipanovsdague, right13:56
sdagueso, honestly, an intermitent network fail to the iscsi host would cause the same issue13:56
ndipanovsdague, yeah it's a bad bug in the libvirt code13:56
ndipanovactually if you ask me - that whole thing needs to go away13:56
*** neelashah1 has quit IRC13:57
ndipanovit makes very little sense for rbd images13:57
sdagueheh, sure, but that can be a liberty discussion13:57
ndipanovright13:57
ndipanovI will propose it actually soon13:57
sdagueI finally have a working devstack again, once you push the patch, I'll local test13:57
ndipanovsince afact - it's not even used for scheduling13:57
ndipanovthe patch is up13:57
ndipanovthe quick and easy one13:58
ndipanovthe real fix would be to not do the disk space accounting when it makes no sense (which is most of the time)13:58
sdagueok, will check it in a few13:58
*** nkrinner has quit IRC13:58
ndipanovit fails pep8 but I will fix that now13:58
jaypipesbauwser: pong13:58
*** jaypipes is now known as leakypipes13:59
openstackgerritShaoHe Feng proposed openstack/nova: Move unlock_override policy enforcement into V2.1 REST API layer  https://review.openstack.org/16216813:59
sahidleakypipes: hello, easy shoot to warm up yourself https://review.openstack.org/#/c/171502/14:00
sahid:)14:00
leakypipessahid: :)14:00
bauwserleakypipes: argh, call conf beginning now for the next hour...14:01
leakypipessahid: done14:01
leakypipesbauwser: k, ping me after14:01
sahidleakypipes: thnk you14:01
openstackgerritNikola Dipanov proposed openstack/nova: libvirt: make _get_instance_disk_info conservative  https://review.openstack.org/17483614:02
bauwserleakypipes: ok, will try :)14:02
*** neelashah has joined #openstack-nova14:02
*** busterswt has quit IRC14:03
*** bnemec has joined #openstack-nova14:03
*** mriedem has quit IRC14:03
leakypipesbauwser: if not, feel free to send me a quick email14:03
*** igordcard_ has joined #openstack-nova14:03
bauwserleakypipes: well, you get my views in https://etherpad.openstack.org/p/liberty-nova-summit-ideas L60 and belwo14:04
*** thangp has joined #openstack-nova14:04
bauwserleakypipes: wanted to sync up with you about all of them14:04
bauwserleakypipes: to make sure we were going into the same direction14:05
*** rushil has quit IRC14:05
* bauwser should change his color for etherpad, just horrible blue14:05
ndipanovsdague, https://review.openstack.org/17483614:05
leakypipesbauwser: OK, after a couple code reviews, I will read and comment14:06
bauwserleakypipes: thanks - and if I have time, I ping you. Valid ?14:06
*** mriedem has joined #openstack-nova14:06
leakypipesbauwser: absolutement.14:06
*** BadCub_Hiding is now known as BadCub14:06
bauwserleakypipes: absolument14:06
leakypipesbauwser: doh, yep, desole.14:06
leakypipes:P14:07
bauwsereh, for once I can just fix someone's sentence :)14:07
leakypipes:)14:07
bauwserlike I said before, it's written in marble14:07
*** rushil has joined #openstack-nova14:07
bauwser:D14:07
leakypipesdims__: you see sdague's note on the ML about oslo.messaging issues (heartbeat) with pyamqp <1.4.0?14:08
*** rushiagr_away is now known as rushiagr14:08
dims__leakypipes: nope looking14:09
openstackgerritShaoHe Feng proposed openstack/nova: Move unlock_override policy enforcement into V2.1 REST API layer  https://review.openstack.org/16216814:09
leakypipessdague: FWIW, we here at Mirantis have been struggling with rabbitmq/heartbeat/oslo.messaging stuffs for the last couple months trying to determine a root cause for the things we've been seeing.14:10
sdagueleakypipes: ok, I'm not sure this gets too root cause, but it at least provides some more enlightenment14:12
*** signed8bit has joined #openstack-nova14:12
*** sdake has joined #openstack-nova14:12
leakypipessdague: you are an enlightening character indeed.14:14
*** arnaud___ has joined #openstack-nova14:15
*** _gryf is now known as _gryf|afk14:15
*** dikonoor has quit IRC14:19
*** arnaud___ has quit IRC14:20
*** signed8bit has quit IRC14:23
openstackgerritPrzemyslaw Czesnowicz proposed openstack/nova: Fix handling of pci_requests in consume_from_instance.  https://review.openstack.org/17339414:25
*** alexpilotti has quit IRC14:27
*** pask81 has quit IRC14:27
*** foexle has joined #openstack-nova14:28
*** neelashah has quit IRC14:28
ndipanovpczesno, thanks for fixing the tests!14:28
johnthetubaguyleakypipes: we have been having issues with that as well, although don't really have much more to add sadly (not yet anyways)14:29
leakypipesjohnthetubaguy: you deploy in active/active rabbitmq cluster as well?14:29
*** mwagner_lap has quit IRC14:29
leakypipespczesno: I added a request in the latest patch. please see...14:30
pczesnondipanov: ok, i'll do that14:31
johnthetubaguyleakypipes: no, not yet at least, were seeing some issues with just a single active, but still digging14:31
*** abhishekk has quit IRC14:31
leakypipesjohnthetubaguy: gotcha.14:31
ndipanovyeah that request makes sense14:32
leakypipesjohnthetubaguy: honestly, because I just don't know the code well enough, I feel like there's just too much magic in there (not just oslo.messaging, but also the way our pacemaker/OCF scripts do failover stuffs)14:32
ndipanovI'd keep the name tho14:32
leakypipesndipanov: yes, your updated functiona name was good.14:33
*** dims__ has quit IRC14:33
leakypipesndipanov: I'd actually thought about suggesting obj_from_request_spec_instance_props(), but that was too long ;)14:33
ndipanovheh14:33
*** TobiasE has left #openstack-nova14:34
*** zz_jgrimm is now known as jgrimm14:34
leakypipespczesno: BTW, do you have a short nickname that I can use? Typing either Przemyslaw or Przemyslaw Czesnowicz in a querty en-us keyboard layout is, well, quite challenging :)14:34
*** tonytan4ever has joined #openstack-nova14:35
johnthetubaguyleakypipes: similar here sadly, I went digging quick, got confused, and realised how much time I would need to invest to understand it properly14:35
openstackgerritRajesh Tailor proposed openstack/nova: Fix kwargs['migration'] KeyError in @errors_out_migration decorator  https://review.openstack.org/17428814:35
*** annegentle has joined #openstack-nova14:35
*** otter768 has joined #openstack-nova14:35
leakypipesjohnthetubaguy: agreed. we really need a good *functional* test suite that validates behaviours seen in various bugs...14:35
*** otter768 has quit IRC14:36
*** rajesht has quit IRC14:36
*** VW_ has joined #openstack-nova14:36
*** wwwBUKOLAYcom has joined #openstack-nova14:38
*** pask81 has joined #openstack-nova14:41
*** marun has joined #openstack-nova14:41
*** mtanino has joined #openstack-nova14:41
openstackgerritPrzemyslaw Czesnowicz proposed openstack/nova: Fix handling of pci_requests in consume_from_instance.  https://review.openstack.org/17339414:42
johnthetubaguyleakypipes: +114:45
sdaguendipanov: ok, confirmed - https://review.openstack.org/#/c/174836/2 fixes the reproduce here14:46
ndipanovsdague, awesome14:46
sdague+2 from me14:46
*** sdake_ has joined #openstack-nova14:47
leakypipesndipanov: pczesno's patch is ready to roll, IMO.14:48
ndipanovy14:48
ndipanovjust reading the test to be sure14:49
*** sdake has quit IRC14:50
*** annegentle has quit IRC14:51
*** annegentle has joined #openstack-nova14:51
*** Nikolay_St has joined #openstack-nova14:55
figleafjohnthetubaguy: just saw the results - congrats!14:56
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation  https://review.openstack.org/13825514:57
*** sgordon has quit IRC14:58
*** nelsnelson has joined #openstack-nova14:58
johnthetubaguyfigleaf: thank you :)14:58
*** markvoelker has joined #openstack-nova15:00
*** markvoelker has quit IRC15:00
leakypipesjohnthetubaguy: might be the closest PTL election ever... :)15:00
bauwserleakypipes: nah15:00
*** markvoelker has joined #openstack-nova15:00
superdanneutron was one vote I think15:00
superdanlast time15:00
leakypipesbauwser: was only 2 votes difference.15:00
bauwserleakypipes: I remember the Neutron one for Icehouse15:00
leakypipesahh..15:00
*** vilobhmm1 has joined #openstack-nova15:01
bauwserkyle won by 1 vote15:01
*** yamahata has joined #openstack-nova15:01
johnthetubaguyleakypipes: its odd, glance and nova both win by two votes, all other seats uncontested15:01
bauwsereh, beated (again) by superdan15:01
johnthetubaguybauwser: wow, I didn't know about that one15:01
leakypipescfriesen: you around by any chance?15:02
bauwserI mean, that's really nice to see elections with tight results, that just means that the representivity is good15:03
bauwserwe had like 33% voters15:03
*** Marga_ has joined #openstack-nova15:03
leakypipesvilobhmm1: hi! can you update https://review.openstack.org/#/c/172502/ to get rid of those pep8 errors?15:04
bauwserleakypipes: johnthetubaguy: for the records http://civs.cs.cornell.edu/cgi-bin/results.pl?id=E_230fb7953010b21915:04
*** dims__ has joined #openstack-nova15:04
leakypipesbauwser: heh, nice digging :)15:04
bauwserI was voting :)15:05
bauwserleakypipes: call ended, we can chat15:06
*** READ10 has quit IRC15:07
*** zzzeek has joined #openstack-nova15:08
leakypipesbauwser: gimme a few?15:08
bauwserleakypipes: np15:08
bauwserleakypipes: wifey just wants visiting Ikea but that can wait :)15:08
*** Maike_ has quit IRC15:09
leakypipesbauwser: do you know who is the light red on that etherpad?15:10
bauwserleakypipes: our former bp-czar :)15:10
leakypipes? you mean johnthetubaguy ?15:11
bauwserjohnthetubaguy not to name him15:11
leakypipesor jogo>?15:11
bauwserleakypipes: eh, I guess only johnthetubaguy was the blueprint czar15:11
johnthetubaguyit might be me, which etherpad?15:11
leakypipeshttps://etherpad.openstack.org/p/liberty-nova-summit-ideas15:11
bauwserjohnthetubaguy: that's you15:11
bauwser:)15:11
bauwserjohnthetubaguy: I guess leakypipes wants to know who opiniated for waiting to split the sched :)15:12
pczesnoleakypipes: i heard that complaint before, pczesno is fine with me15:13
bauwserleakypipes: honestly, about that, I think we can just discuss that at Summit time15:13
bauwserjohnthetubaguy: your point is valid as I said, I just want to discuss during Summit how we can make sure that the scheduler will have enough bandwidth for getting things done15:13
*** vilobhmm1 has quit IRC15:13
leakypipespczesno: :) ok.15:13
bauwserjohnthetubaguy: because the split is not just a tech split, but also a development scale out15:14
bauwserjohnthetubaguy: but I leave you take your time in particular now that you're PTL ;)15:14
johnthetubaguybauwser: I am not against the split, but I don't think we should focus on that bit15:14
leakypipesjohnthetubaguy: agreed.15:14
bauwserjohnthetubaguy: I understood, my point is not about that15:15
bauwserjohnthetubaguy: as I said, I think we need to focus on the interfaces, and even the internal tech devt15:15
zzzeekjogo: do you know if nova is going to stop requiring SQlite migrations like other projects or do we still need to support those ?15:15
bauwserdebt15:15
ndipanovzzzeek, sdague might now too15:16
bauwserjohnthetubaguy: I just wanted to make things clear at Summit time to not send bad signals to teams :)15:16
*** jistr is now known as jistr|mtg15:16
bauwserjohnthetubaguy: my point being that I seriously don't care if the scheduler is within the Nova repo or not, just that it needs to scale out fast to address other projects needs15:16
zzzeekSo, need to know:  1. sqlite migrations, Y/N, 2. downgrades, Y/N   3. migration tests, e.g. snakewalk + assert X is true/false, Y/N, and if Y what do we think of having the tests be extra code in the migration files themselves15:17
johnthetubaguybauwser: sure lets just delete that line if its causing friction15:17
bauwserjohnthetubaguy: no no, we can just mention that we need to see what's targetable for Liberty15:18
superdanzzzeek: for now, we have to have sqlite migrations, AFAIK15:18
bauwseragain your comment is valid15:18
sdaguezzzeek: 1. no idea15:18
zzzeeksuperdan: because of your test suite15:18
sdague2. downgrades No15:18
leakypipeszzzeek: My opinions: 1. N 2. N 3. Walk the upgrade-only path with real data.15:18
*** neelashah has joined #openstack-nova15:18
superdanleakypipes: we have to make the unit tests work with mysql if we're going to do that15:18
sdaguewhich means 3. no, we can remove snakewalk15:19
bauwserjohnthetubaguy: and having opinions is better than having no opinion so I prefer to leave your comment15:19
jogoAFAIK keystone dropped downgrades15:19
leakypipessuperdan: isn't that what T-H is?15:19
superdanleakypipes: no?15:19
sdaguewe still need the data testing in there, but that shouldn't be hurt by removing snakewalk15:19
johnthetubaguybauwser: i see the speed as a general problem everyone is facing, but anyways, talk on this more later15:19
leakypipessuperdan: I thought T-H was for functionally testing the DB schema migrations against real DBs15:19
jogozzzeek: I622f89fe63327d44f9b229d3bd9e76e15acbaa7a15:19
zzzeekleakypipes: OK but for migration tests, are we concerned that A. the migrations run without breaking, or B. that we assert the index/column/whatwver was cerated/dropped, like the current tests in test_migrations do ?    Beacuse I will work on having those tests be autogenerated just like the migrations themselves15:19
bauwserjohnthetubaguy: fair point15:20
sdaguezzzeek: we should be concerned about A15:20
superdanleakypipes: it does that, but that's now what I mean. I mean we have to either remove unit tests for migrations, or make the unit suite work against a real DB15:20
sdagueall the B checks are silly imho15:20
sdaguethe whole test framework was about A when it was added15:20
leakypipessuperdan: ah, sorry, I misunderstood. yes, totally correct.15:20
*** a5t3r1x has joined #openstack-nova15:20
zzzeeksdague: A beacuse, your test suite runs on SQLite, is that it?15:20
zzzeeksdague: oh15:21
a5t3r1xhello!15:21
zzzeeksdague: A migrations run w/o breaking :)15:21
zzzeeksdague: OK15:21
sdaguezzzeek: no, A because it turns out that some migrations pass on an empty mysql db that fail with data in them15:21
sdaguebecause mysql won't enforce constraints the same way if there is no data15:21
zzzeeksdague: yes definitely.   so right now there is some fixture that generates data in Mysql right15:21
sdaguewhich was *actual* issue15:21
zzzeeksdague: yup15:21
sdaguethat's what the pre_X stuff is supposed to do15:22
*** armax has joined #openstack-nova15:22
sdagueinject data that would trigger that kind of issue before a data touching migration15:22
a5t3r1xquick question, Error launching a defined domain with XML: <domain type='kvm'> can be related to a network issue?15:22
*** dims__ has quit IRC15:22
superdana5t3r1x: see topic15:23
zzzeeksdague: OK so…is there an approach for that, also I guess this was not in the scope of the online schema migrations spec right15:23
sdaguezzzeek: that I don't know15:23
zzzeeksdague: OK.15:23
superdanI don't think so15:24
a5t3r1xsuperdan: oh, i haven't read. sorry!15:24
sdaguezzzeek: so https://github.com/openstack/nova/blob/master/nova/tests/unit/db/test_migrations.py#L497-L512 is an instance of us doing A15:24
sdaguewe had a few more before various roll ups, because we did things like migrating int ids -> uuids15:25
zzzeeksdague: so, I’m going to work on a straightforward spec for alembic migrations, and ill propose using branching to split out major release streams e.g. juno/kilo/liberty and then additionally think about having within each individual “expand” and “contract” phases to enable some degree of “online” capability15:25
*** tonytan4ever has quit IRC15:25
zzzeekwe can do SQlite migrations though it is not as nice, and for testing I think i might leave it open ended for the moment15:25
sdagueI think all the schema assert tests in there should not be in there15:25
*** ZZelle has quit IRC15:25
*** ZZelle has joined #openstack-nova15:26
sdaguezzzeek: sounds good15:26
zzzeeksdague: nova’s current migrations are really easy to port to alembic directly, you don’t have very much at all in there, only 216 is big and that code can be dropped into an alembic file as-is15:26
superdanFWIW, I don't really want to move from sqla-migrate to alembic whilst also taking on the online schema stuff15:26
zzzeeksuperdan: Really?  OK !  then that is easier15:27
sdagueoh, right, wasn't that the plan. We just move to online migration, and can drop sqla eventually15:27
superdanright15:27
sdaguesorry sqla-migrate15:27
*** vilobhmm1 has joined #openstack-nova15:28
*** tonytan4ever has joined #openstack-nova15:28
*** dprince has quit IRC15:28
zzzeekonline schema as proposed has the issue that it never allows for “un-automatable” migrations and I think there should be a way for those to exist, and I also think a DB should be versioned in all cases.  I have a plan for how vendor-specific modifications to their DBs can also be versioned15:28
zzzeekid like to propose something more controlled than, “whatever the DB looks like we’ll just try to make it look like the current one”15:29
superdansdague: by the way, I think that the from-juno hack is breaking me because we're running flavor-migrate against juno code where it doesn't exist :/15:29
*** vilobhmm1 has quit IRC15:30
sdaguesuperdan: ah, yep15:31
*** arnaud___ has joined #openstack-nova15:31
sdagueso... we'll cut the branches next week, we can test it all then15:31
superdansdague: so I guess I have to wait.. when will that branch split happen?15:31
superdanokay15:31
superdansdague: I'll just sit here and pout until then15:31
sdagueyeh, just trying to figure out the last things to land in devstack before branch15:31
superdanI kinda think it'd be good to make this the first migration in lemming15:32
sdagueit has to branch first, then we can branch grenade, then we fix d-g15:32
superdannot that it really matters, but it would be cleaner15:32
sdagueyeh, well, just block people for a few days :)15:32
superdanwell, I'm only one man :)15:32
sdaguewith a ban hammer15:32
superdanme and my hammer sleep at night15:32
superdananyway, probably not a huge concern :)15:33
sdagueif someone lands a db migration in < 24 hours, I'll ban hammer them :)15:33
*** mpavone has quit IRC15:33
sdaguenormally we don't cut these branches until release week, so you're getting them earlier this time15:34
superdanI'm not complaining about you or grenade, I'm just commenting that something could slip in first15:35
superdanwe have a few proposed I think15:35
*** arnaud___ has quit IRC15:35
superdanmaybe I should generate the gerrit fu to show them to me15:35
superdanI definitely want to wait on this until I get a clean grenade run15:35
*** matrohon has quit IRC15:36
superdanjohnthetubaguy: you okay with an ML post saying "don't land any migrations in lemming until we land the flavor cleanup migration?15:37
jogohitting a fun libvirt bug http://logs.openstack.org/13/173213/4/check/check-tempest-dsvm-multinode-full/9ac7736/logs/10.208.198.2-subnode/screen-n-cpu.txt.gz?level=TRACE#_2015-04-17_10_57_46_35715:37
jogosahid kashyap other libvirt folks: ^15:38
cburgessSo to be clear, its not a libvirt bug. The issue here is that libvirt and nova both call ebtables and race each other.15:38
cburgessAs mentioned in the qa channel I see 3 possible fixes...15:39
johnthetubaguysuperdan: +115:39
superdanokay15:39
cburgess1) Require a new enough version of libvirt and ebtables that allows both to use the lock file to prevent races15:39
*** thedodd has joined #openstack-nova15:39
cburgess2) Have libvirt handle all ebtables call via nwfilter15:39
cburgess3) Use a sufficently old enough version of ebtables that just ignores this problem (Note: its a joke).15:40
cburgessPersonally I vote for #215:40
*** kmartin has joined #openstack-nova15:43
jogocburgess: how hard would #2 be?15:43
cburgessjogo: Not sure.. trying to look at that now.15:43
cburgessjogo: I don't even recall what these special ebtables rules are for.15:43
*** jistr|mtg is now known as jistr15:44
clarkbcburgess: I believe they are used to isolate traffic between nodes from different tenants15:44
*** Longgeek_ has quit IRC15:44
clarkbcburgess: since everyone is on the same l3 nova net uses ebtables to keep my VMs from talking to your VMs on their private IPs15:44
*** pask81 has quit IRC15:46
*** Nikolay_St has quit IRC15:48
*** ganso_ has joined #openstack-nova15:49
kashyapjogo: Hi, back. Reading the scrollback now.15:49
*** mtecer has joined #openstack-nova15:50
jogokashyap: thanks, looks the conversation stayed in -qa15:51
*** tsekiyama has joined #openstack-nova15:51
jogosorry for the confusion15:51
* kashyap moves there15:51
*** burt has joined #openstack-nova15:52
*** browne has joined #openstack-nova15:55
*** mtecer has quit IRC15:58
*** dsanders has joined #openstack-nova15:58
lucasagomesjohnthetubaguy, ping re https://review.openstack.org/#/c/173379/ lemme if I have answered ur questions there please. Thanks15:58
*** dulek has quit IRC15:58
*** thangp has quit IRC15:59
*** r-daneel has joined #openstack-nova16:00
*** kbyrne has quit IRC16:00
*** hemna has joined #openstack-nova16:01
*** armax has quit IRC16:01
*** dsanders has quit IRC16:01
*** garyk has quit IRC16:01
johnthetubaguylucasagomes: thanks for the update, I better understand what you are trying to do now, thanks16:02
*** dprince has joined #openstack-nova16:02
lucasagomesjohnthetubaguy, cool! thank you for the review btw16:03
johnthetubaguynp16:03
*** arnaud___ has joined #openstack-nova16:04
*** pask81 has joined #openstack-nova16:04
*** annegentle has quit IRC16:05
openstackgerritEric Brown proposed openstack/nova: Replace ssh exec calls with paramiko lib  https://review.openstack.org/15793116:06
openstackgerritPrzemyslaw Czesnowicz proposed openstack/nova: Fix handling of pci_requests in consume_from_instance.  https://review.openstack.org/17339416:06
*** vilobhmm1 has joined #openstack-nova16:07
pczesnondipanov: ^^ forgot to update unit test16:07
*** yamahata has quit IRC16:08
*** neelashah has quit IRC16:09
*** pask81 has quit IRC16:09
*** pcaruana has quit IRC16:10
*** unicell has quit IRC16:13
*** vilobhmm11 has joined #openstack-nova16:13
*** vilobhmm12 has joined #openstack-nova16:13
*** vilobhmm12 has quit IRC16:13
*** vilobhmm11 has joined #openstack-nova16:14
*** jlanoux has quit IRC16:14
vilobhmm11leakypipes : sure will do…just got in16:15
leakypipesvilobhmm1: see my comment on there as well...16:15
vilobhmm11leakypipes or jaypipes :)16:15
*** tonytan4ever has quit IRC16:15
vilobhmm11sure thanks for reviewing it16:15
*** vilobhmm1 has quit IRC16:15
*** claudiub_ has quit IRC16:16
leakypipesvilobhmm11: casual nick Fridays ;)16:16
johnthetubaguyndipanov: just looking at https://review.openstack.org/#/c/174836 spotted a tiny nit, but I guess its not a big deal, given the impact of the bug16:18
*** salv-orl_ has quit IRC16:19
*** mriedem1 has joined #openstack-nova16:19
vilobhmm11leakypipes : i have one more question in order to raise exception to the client who is initiating these servicegroup api how should the tooz->nova error transaltion be done https://gist.github.com/vilobhmm/2255592994b14f3b7cfd ? but at nova/services.py layer using something like "exc.HTTPBadRequest(explanation=msg)" doesn't make sense at the web server might not be initialized at that phase…..so since these API will be16:20
*** subscope has joined #openstack-nova16:20
*** mriedem has quit IRC16:21
leakypipesvilobhmm11: well, also because not everything that uses nova.service is actually a web server :)16:21
*** kmartin_ has joined #openstack-nova16:21
vilobhmm11true16:21
*** kmartin has quit IRC16:21
vilobhmm11so let me know what you think in that regards16:21
leakypipesvilobhmm11: the servicegroup API should raise nova.exception.ServiceGroupXXX errors, never HTTP errors or driver-specific errors.16:22
vilobhmm11so that i can add more error handling to the patch alongwith the unit test that i am working on16:22
*** claudiub has joined #openstack-nova16:24
vilobhmm11nova.exception.ServiceGroupXXX  should be raised from nova/servicegroup/drivers/tz.py but after catching them at nova/service.py do you expect to re-saise the same exception to the client ?16:24
*** salv-orlando has joined #openstack-nova16:24
*** dims__ has joined #openstack-nova16:24
vilobhmm11leakypipes : just to be sure ^^^16:25
superdanvilobhmm11: remember, there is no client there16:25
vilobhmm11superdan : why you say so ?16:25
leakypipesvilobhmm11: right, what superdan said :)16:25
superdanvilobhmm11: we talked about this yesterday, right?16:25
superdanvilobhmm11: start() is called when starting the service from the command line or an init service or whatever16:26
superdanit's not called in response to an HTTP request from anywhere16:26
vilobhmm11superdan or dansmith :) looks like i should also change my name…sure got it thanks16:26
*** patrickeast has joined #openstack-nova16:27
*** yamahata has joined #openstack-nova16:31
openstackgerritMitsuhiro Tanino proposed openstack/nova: Prevent unintentional detaching volume from instance  https://review.openstack.org/16669516:33
*** Nic has joined #openstack-nova16:33
*** jcoufal has quit IRC16:34
*** arnaud___ has quit IRC16:34
*** sdake has joined #openstack-nova16:35
*** otter768 has joined #openstack-nova16:36
*** derekh has quit IRC16:37
*** mdbooth has quit IRC16:37
*** sdake_ has quit IRC16:39
*** abhijeetm has quit IRC16:39
*** otter768 has quit IRC16:41
*** unicell has joined #openstack-nova16:41
*** unicell has quit IRC16:41
*** unicell has joined #openstack-nova16:42
*** Marga_ has quit IRC16:43
*** BobBall is now known as BobBall_AWOL16:43
*** mdbooth has joined #openstack-nova16:44
superdanndipanov: https://review.openstack.org/#/c/173394/616:44
superdanndipanov: going to re-approve that?16:44
superdanndipanov: I'm afraid I'll complain about something if I look at it in depth16:44
*** hightall_ has joined #openstack-nova16:45
hightall_hi I want to know how to change token valid time for novnc?16:46
*** armax has joined #openstack-nova16:46
*** jistr has quit IRC16:48
* leakypipes celebrates less than 300 unread emails in inbox16:48
figleafsuperdan: if?16:49
figleaf:)16:49
superdanfigleaf: yes, hence not looking at it in depth :)16:49
superdandoes the file: filter in our gerrit search not work?16:50
*** dims__ is now known as dimsum__16:50
openstackgerritPetrut Lucian proposed openstack/nova: Hyper-V: Nova serial console access support  https://review.openstack.org/14500416:50
bauwsersuperdan: file only works if you're adding it to the watchlist AFAIK16:51
bauwsersuperdan: but now that Gerrit 2.10 is there, I don't know if they fixed thaty16:52
superdanhmm16:52
*** sgordon has joined #openstack-nova16:52
superdanthat sucks16:52
superdanI thought we could filter like that already16:53
bauwsersuperdan: my watchlist uses the file filter16:53
*** pixelb has quit IRC16:53
bauwsersuperdan: it works fine, but as I said, I'm only using it there16:53
bauwsersuperdan: https://review.openstack.org/Documentation/user-search.html#_search_operators16:54
bauwsersuperdan: make sure your regex is good16:54
superdanI don't want it in the watchlist16:54
superdanI want it in a simple query16:54
bauwseran example file:^nova/scheduler/.* OR file:^nova/compute/resource_tracker.*16:54
bauwser(now, I'm no longer use it for finding reviews, but... :D)16:55
*** Nic has quit IRC16:56
bauwseroh, we're still with Gerrit 2.816:56
bauwsersuperdan: so yes, you need to use it using the watched projects16:56
*** wwwBUKOLAYcom has quit IRC16:57
bauwsersuperdan: oh no !16:57
bauwsersuperdan: https://review.openstack.org/#/q/file:%255Enova/scheduler/.*+OR+file:%255Enova/compute/resource_tracker.*,n,z16:57
bauwsersuperdan: sounds like it's fixed16:57
superdanhmm, wtf16:57
*** unicell has quit IRC16:57
superdandoesn't work for mew16:57
*** unicell has joined #openstack-nova16:57
*** hightall_ has quit IRC16:57
bauwsercheck the regex16:57
*** annegentle has joined #openstack-nova16:58
bauwserit's needed to use the ^ for it16:58
*** armax has quit IRC16:58
bauwserand use the greedy *16:58
*** atuvenie has quit IRC16:59
bauwserok, going out, bye folks16:59
*** bauwser is now known as bauzas17:00
*** tonytan4ever has joined #openstack-nova17:02
openstackgerritJay Pipes proposed openstack/nova: fixed tests in test_instance to pass with random PYTHONHASHSEED  https://review.openstack.org/16113517:02
openstackgerritJay Pipes proposed openstack/nova: refactored tests in test_objects to pass with random PYTHONHASHSEED  https://review.openstack.org/16131317:02
openstackgerritJay Pipes proposed openstack/nova: fixed tests in test_iptables_network to work with random PYTHONHASHSEED  https://review.openstack.org/16335217:02
openstackgerritJay Pipes proposed openstack/nova: refactored tests to pass in test_inject to pass with random PYTHONHASHSEED  https://review.openstack.org/16335417:02
openstackgerritJay Pipes proposed openstack/nova: fix for vmware test_driver_api to pass with random PYTHONHASHSEED  https://review.openstack.org/16335717:02
openstackgerritJay Pipes proposed openstack/nova: fixed tests in test_matchers to pass with random PYTHONHASHSEED  https://review.openstack.org/16335817:02
*** armax has joined #openstack-nova17:03
openstackgerritJay Pipes proposed openstack/nova: refactored test in vmware test_read_write_util to pass with random PYTHONHASHSEED  https://review.openstack.org/16335917:03
*** markvoelker has quit IRC17:03
openstackgerritJay Pipes proposed openstack/nova: Refactored tests in test_neutron_driver to pass with random PYTHONHASHSEED  https://review.openstack.org/16336117:03
openstackgerritJay Pipes proposed openstack/nova: Fixed tests in test_glance to pass with random PYTHONHASHSEED  https://review.openstack.org/16336417:03
openstackgerritJay Pipes proposed openstack/nova: Fixed test in test_tracker to work with random PYTHONHASHSEED  https://review.openstack.org/17435617:03
openstackgerritJay Pipes proposed openstack/nova: Removed 'PYTHONHASHSEED=0' from tox.ini  https://review.openstack.org/16387717:03
*** romainh has left #openstack-nova17:04
*** lucasagomes is now known as lucas-beer17:04
*** dsanders has joined #openstack-nova17:04
*** kmartin_ has quit IRC17:05
*** salv-orlando has quit IRC17:06
*** kmartin_ has joined #openstack-nova17:09
leakypipessuperdan, other cores: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1348818,n,z has a bunch of easy wins if you're looking for quick reviews...17:13
*** saltsa has left #openstack-nova17:13
*** russellb has quit IRC17:14
sdakedansmsith boy this ip address assignment is a rabbit hole for sure17:14
*** harlowja_away is now known as harlowja17:17
*** russellb has joined #openstack-nova17:17
*** sahid has quit IRC17:20
*** Nic has joined #openstack-nova17:20
*** neelashah has joined #openstack-nova17:23
sdakedansmith my thinking at this time is heal_instance_info_cache is not sending the updated information to nova17:24
*** VW_ has quit IRC17:24
sdakeit is adding it to the info cache, but not sending17:24
*** russellb has quit IRC17:25
sdakedansmith thoughts on that?17:25
sdakehttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L559317:26
sdakehttps://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L76117:27
sdakevs17:27
sdakehttps://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L74817:27
superdansdake: not sending it where?17:27
sdakeit is not calling             base_api.update_instance_cache_with_nw_info(self, context,17:28
sdakelook at 761 vs 74817:28
sdake5993 is calling 75417:28
sdakeso nova-compute heals the instance cache, shoudln't it store that info in the database?17:29
*** russellb has joined #openstack-nova17:29
*** salv-orlando has joined #openstack-nova17:29
sdakewhen launching 1 vm this isn't a problem - when launchign 20 it is because neutron slows down port DOWN to UP state17:30
sdakerather DOWN to ACTIVE state17:30
superdanin juno I think that was done by a decorator on that method17:31
superdanit just changed in master17:31
*** claudiub has quit IRC17:31
*** beagles is now known as beagles_brb17:34
sdakesuperdan decorator on which function?17:34
*** arnaud___ has joined #openstack-nova17:35
sdakei see no decorator on _heal_instance_info_cache or _get_instance_nw_info in stable/juno17:35
mriedem1sdake: there were decorators for those in the neutron api in juno17:36
superdansdake: spread all over17:36
mriedem1https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L80217:36
mriedem1https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L75717:36
mriedem1the lock there uses the same lock name as in @base_api.refresh_cache17:36
mriedem1btw, i'm completely walking into this conversation blind, just got back from lunch :)17:37
mriedem1that code was the same in juno17:37
sdakemrieden1 the allocate function is working, its the heal that isnt'17:37
*** mriedem1 is now known as mriedem17:38
sdakerather _heal_instance_info_cache has no decorator17:39
superdanI hate the decorator, by the way17:39
mriedemsuperdan: you shouldn't have told her to use cheetah print everywhere then17:39
mriedemthat's your fault17:39
superdanand _heal wouldn't have that decorator anyway17:40
superdanmriedem: buh-dum-dum17:40
mriedemi'm here all week17:40
*** arnaud___ has quit IRC17:40
sdakeafter the 60 second timer expires, heal runs, how does it send the data back to whatever stores it in the db?17:40
mriedemsdake: that's in the base api17:40
mriedemsec17:41
mriedemhttps://github.com/openstack/nova/blob/stable/juno/nova/network/base_api.py#L4617:41
mriedemrefresh_cache calls update_instance_cache_with_nw_info which updates the instance.info_cache in the db17:41
sdakelet me read code sec17:42
superdanjust to be clear,17:42
superdanthis code changes a lot because it's an effing disaster17:43
* sdake concurs with superdan ;)17:43
superdanso just be aware that something broken or working in master may not apply to your juno setup17:43
sdakeI hven't tried master, I guess I could, but I'd like a surgical patch for juno :)17:43
lpetrutmriedem: Hi17:44
* mriedem feels hyper-v lifecycle event listener discussion coming :)17:44
lpetrutmriedem: Since Liberty is now open, could you please remove the -1 from this patch? https://review.openstack.org/#/c/170433/17:44
*** jichen has quit IRC17:44
*** tonytan4ever has quit IRC17:44
lpetrutheh :D17:44
mriedemdone and done17:44
*** jichen has joined #openstack-nova17:45
lpetrutmriedem: great, thanks!17:45
mriedemlpetrut: btw, not sure if you've paid attention to my recent rantings about the lifecycle event listener stuff with libvirt,17:45
*** emagana has joined #openstack-nova17:45
mriedemlpetrut: but in your impl be very careful about how that handles reboot17:45
lpetrutmriedem: Yes, I've seen that there are issues with that on Libvirt. Took care of that in the Hyper-V implementation17:46
mriedemlpetrut: so you have a similiar delay mechanism?17:46
mriedemfor stopped?17:46
lpetrutno need for that, in case of a reboot, I'll get an intermediary state after which the instance is active. In the Hyper-V implementation, intermediary states are ignored17:47
mriedemok, cool17:47
*** achanda has joined #openstack-nova17:48
lpetrutwe use KVM in our Hyper-V CI and Cinder CI and had that issue as well. Cloudbase-init was rebooting the instances, so those got stuck to shutdown state. We managed this in our CI scripts by waiting for an expected shut down and rebooting the instances afterwards :)17:49
lpetrutI thought about submiting a bug fix for this using some delay mechanism as you already mentioned17:49
sdakemriedem1  how does https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5593 end up calling https://github.com/openstack/nova/blob/stable/juno/nova/network/base_api.py#L3317:49
sdakeit is not decorated?17:49
lpetrutbtw, is that being worked on?17:49
mriedemlpetrut: there have been a few things, sec i'll grab links17:50
*** annegentle has quit IRC17:50
mriedemlpetrut: so here is a squashed set of fixes for xen domains in libvirt that we need in juno: https://review.openstack.org/#/c/163378/17:51
*** Hosam has quit IRC17:51
mriedemlpetrut: this expands that to all libvirt domains https://review.openstack.org/#/c/172775/17:51
mriedemwhich i'll backport to kilo and juno17:51
mriedemlpetrut: this is on master now for liberty but i'm proposing to kilo https://review.openstack.org/#/c/174477/17:51
*** sgordon has quit IRC17:51
*** sgordon has joined #openstack-nova17:51
mriedem^ is the case when the compute service is shutting down, like host reboot17:52
mriedemthere was  comment in the master review that we should probably do the same on init_host - delay the lifecycle event listening in init_host() until after we've cleaned up the instances17:52
mriedemin _init_instance17:52
mriedemlpetrut: then i have this hammer to just disable lifecycle events from the hypervisor, but it's got some complaints against that https://review.openstack.org/#/c/159275/17:53
*** Maike has joined #openstack-nova17:53
*** annegentle has joined #openstack-nova17:54
lpetrutheh, well, thanks a lot for the heads up.17:54
superdandoes anyone know what this message from T-H means?17:55
superdanFAILURE - Did not find the end of a migration after a start17:55
superdanWARNING - Migration 290->291 failed to complete in 15m 54s17:55
*** Marga_ has joined #openstack-nova17:55
mriedemtimeout i guess?17:55
superdandoesn't seem so from the logs17:56
mriedemjhesketh: ^?17:56
*** isd has joined #openstack-nova17:56
mriedemsuperdan: looks like it expects it to complete in under 16 minutes17:56
*** isd has quit IRC17:56
mriedemunless it did and the tooling isn't detecting it?17:56
superdanright, but this migration does nothing17:56
superdanoh17:56
superdanI bet this is because their sample databases actually have violations in them17:56
mriedemheh17:57
superdanhmm, that's going to be interesting17:57
mriedemi found those with the null instances.uuid scan too17:57
superdanyeah17:57
superdanwell, the thing is,17:57
superdanthey're going to have to migrate the sample datasets in order to fix it17:57
mriedembasically jhesketh said he would clean those up once my migration got close to approval17:57
mriedemyeah, he did for mine17:57
superdanthis is going to be big tho17:57
superdannot just snipping out a few records17:57
superdananyway, yesterday I wasn't thinking about it17:57
superdanso I'll try to catch him at some point17:58
*** ijw has joined #openstack-nova18:00
*** armax has quit IRC18:05
*** aix has quit IRC18:06
*** tonytan4ever has joined #openstack-nova18:08
*** hdd has joined #openstack-nova18:10
*** isd has joined #openstack-nova18:10
*** VW_ has joined #openstack-nova18:10
*** ZZelle_ has joined #openstack-nova18:10
ZZelle_ns18:11
*** lpetrut has left #openstack-nova18:12
*** Sukhdev has joined #openstack-nova18:19
superdanmriedem: achievement unlocked: https://review.openstack.org/#/admin/groups/540,members18:19
*** claudiub has joined #openstack-nova18:19
openstackgerritJohn Garbutt proposed openstack/nova-specs: Quota Management in Nested Projects  https://review.openstack.org/16060518:19
mriedemmuwahahaha18:19
*** armax has joined #openstack-nova18:20
*** annegentle has quit IRC18:21
claudiubmriedem: hello. May I disturb you by asking a review  for a very small commit? Hyper-V CI passed etc etc. https://review.openstack.org/#/c/169267/18:22
mtreinishnow I know who to bug for backport +2s18:22
*** annegentle has joined #openstack-nova18:25
*** leakypipes has quit IRC18:25
mriedemi'll be your huckleberry18:26
mriedemclaudiub: sure, sec18:26
*** rushiagr is now known as rushiagr_away18:26
superdanmriedem: movie quote fail18:26
*** tonytan4ever has quit IRC18:26
superdanmriedem: I believe it's properly "I'm your huckleberry"18:27
*** annashen has joined #openstack-nova18:27
mriedemi'm not quoting tombstone18:27
mriedemit's just a saying in general i believe18:27
mriedembut i'm bringing hell with me!18:28
mriedemis that what you want?18:28
superdanmriedem: sure, cover it up. flat truth is.. you screwed up.18:28
*** johnthetubaguy is now known as zz_johnthetubagu18:28
*** andreykurilin__ has joined #openstack-nova18:29
*** mtecer has joined #openstack-nova18:30
mriedemclaudiub: vs is sure to have this Notes attribute right?18:30
claudiubmriedem: yeah18:31
mriedemcool, +218:32
mriedemeasy fix18:32
claudiubmriedem: thanks! :D18:32
openstackgerritNikola Dipanov proposed openstack/nova: libvirt: make _get_instance_disk_info conservative  https://review.openstack.org/17483618:36
*** otter768 has joined #openstack-nova18:37
dmzhey y'all, anyone have a good reference on sharing vlan tagged networks between projects?18:38
*** otter768 has quit IRC18:42
*** tonytan4ever has joined #openstack-nova18:43
*** lucas-beer has quit IRC18:44
*** lpetrut has joined #openstack-nova18:44
*** achanda has quit IRC18:46
*** annashen has quit IRC18:47
ndipanovsdague, fixed some indent issues here - re A?18:49
openstackgerritChris Friesen proposed openstack/nova: libvirt: check qemu version for NUMA & hugepage support  https://review.openstack.org/17078018:49
ndipanovhttps://review.openstack.org/#/c/174836/18:49
*** beagles_brb is now known as beagles18:49
figleafndipanov: it is valid indentation, but not correct :)18:54
*** neelashah has quit IRC18:54
*** achanda has joined #openstack-nova18:56
ndipanovlol figleaf - nice nick :)18:57
figleafndipanov: heh - one of the few advantages of having a name that means something18:57
*** salv-orlando has quit IRC18:57
sdaguendipanov: +218:58
sdaguemriedem / superdan - either of you want to double check - https://review.openstack.org/#/c/174836 before we send it in?18:58
ndipanovsdague, se se18:59
*** foexle has quit IRC18:59
mriedem....18:59
mriedemthere is another related bug there18:59
mriedemsec18:59
mriedembug 137167719:00
openstackbug 1371677 in OpenStack Compute (nova) "Race in resource tracker causes 500 response on deleting during verify_resize state " [Critical,Fix released] https://launchpad.net/bugs/1371677 - Assigned to Dan Smith (danms)19:00
ndipanovmriedem, yeah probably19:00
sdaguemriedem: oh, it's the same issue?19:00
superdanassigned to me?19:01
mriedemthat was https://review.openstack.org/#/c/123774/19:01
mriedemwhich screwed the gate for awhile in juno19:01
sdagueso we can blame superdan for the bug still existing19:01
mriedemsuperdan: moved the lock19:01
ndipanovsdague, like I said that method is a neverending source of bugs19:01
*** eglynn has quit IRC19:01
mriedemthe bigger issue was what the libvirt driver was doing with volumes in that loop19:01
ndipanovyeah we moved the lock but left libvirt happily call blockdev on massive volumes19:01
mriedemi think danpb and ndipanov pointed that out in the bug back when19:01
mriedemyup19:01
sdague🐞 🐞 🐞 🐞 🐞 🐞 🐞 🐞19:02
sdaguebugs for everyone19:02
mriedemit's the oprah of bugs19:03
superdankeeps giving?19:03
*** marun has quit IRC19:03
mriedemno19:03
mriedemyou get a bug, you get a bug, everyone gets a bug!19:04
mriedemcheck under your seat19:04
superdanright19:04
superdans'what I meant19:04
sdagueooooo, I don't really want bugs under my seat19:04
superdankinda like you meant when you fubar'd that quote19:04
mriedemsuperdan: sure, cover it up. flat truth is.. you screwed up.19:04
superdanhaha19:04
mriedemi couldn't find/copy/paste fast enough19:05
*** ijw has quit IRC19:09
openstackgerritSean Dague proposed openstack/nova: remove downgrade support from our database migrations  https://review.openstack.org/17501019:10
sdaguefriday afternoon hacking, you'll like that one superdan19:10
superdanoh my19:10
superdanmikal will be thrilled19:11
sdagueoh, crap, need one change, I leaked a thing into it19:11
mtreinishheh, I like how the placeholder migrations had a downgrade too19:12
openstackgerritSean Dague proposed openstack/nova: remove downgrade support from our database migrations  https://review.openstack.org/17501019:12
mtreinishI guess they would have to if someone called downgrade19:12
sdagueok, that should be right19:12
*** matrohon has joined #openstack-nova19:13
*** Maike has quit IRC19:13
*** hemna has quit IRC19:15
*** isd has quit IRC19:15
sdagueok, I think that's a week for me, have a good weekend folks19:17
*** annegentle has quit IRC19:19
*** ijw has joined #openstack-nova19:20
*** bnemec is now known as beekneemech19:22
openstackgerritJay Hennessey proposed openstack/nova-specs: Move qualified-uuids spec to liberty  https://review.openstack.org/13262319:23
*** arnaud___ has joined #openstack-nova19:24
*** hdd has quit IRC19:24
*** hdd has joined #openstack-nova19:26
*** arnaud___ has quit IRC19:29
*** welldannit has joined #openstack-nova19:29
mriedempci ci is super unhappy with that change19:30
mriedemsheesh19:30
*** vladikr has quit IRC19:30
*** yamahata has quit IRC19:31
*** pixelb has joined #openstack-nova19:32
*** isd has joined #openstack-nova19:35
*** ijw has quit IRC19:35
*** Marga_ has quit IRC19:38
*** hemna has joined #openstack-nova19:40
*** subscope has quit IRC19:40
*** ozialien has joined #openstack-nova19:41
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: make _get_instance_disk_info conservative  https://review.openstack.org/17483619:42
mriedemndipanov: https://review.openstack.org/#/c/174836/ is +W19:43
*** neelashah has joined #openstack-nova19:46
*** annashen has joined #openstack-nova19:48
*** salv-orlando has joined #openstack-nova19:49
*** achanda has quit IRC19:50
*** annashen has quit IRC19:53
*** patrickeast has quit IRC19:53
*** nelsnelson has quit IRC19:59
*** annashen has joined #openstack-nova20:00
*** jichen has quit IRC20:02
openstackgerritAndy Hill proposed openstack/nova-specs: Default New Compute Node Status to Disabled  https://review.openstack.org/17503720:03
*** Marga_ has joined #openstack-nova20:09
*** artom has quit IRC20:11
*** ndipanov has quit IRC20:13
*** Marga_ has quit IRC20:13
*** jogo has quit IRC20:14
*** Nikolay_St has joined #openstack-nova20:14
*** dprince has quit IRC20:16
*** dsanders has quit IRC20:17
*** Sukhdev has quit IRC20:19
*** openstackgerrit has quit IRC20:22
bauzasis it me or Gerrit is super slow now ?20:22
*** openstackgerrit has joined #openstack-nova20:23
*** Nikolay_St has quit IRC20:23
*** Nikolay_St has joined #openstack-nova20:24
*** marun has joined #openstack-nova20:25
bauzassuperdan: any idea why https://review.openstack.org/#/c/174480/ has Grenade failing ?20:25
superdanbauzas: yes20:25
superdanbauzas: because grenade isn't migrating the flavors of the instances in the database before it runs the migration20:26
superdanbauzas: basically, exactly the case that this is supposed to block20:26
superdanbauzas: see the depends-on patch :)20:26
bauzassuperdan: ooooooh20:26
superdanand my comment earlier there20:26
bauzassuperdan: so I think the partial-ncpu is -1 for the same reaosn ?20:26
superdanyes20:26
bauzasoki doki20:26
*** achanda has joined #openstack-nova20:27
superdanit's actually perfect that it's failing :)20:27
superdanproves that it works :)20:27
bauzassuperdan: eh eh20:27
* bauzas looking at your commen,t20:27
superdangrenade will support kilo next week and then we'll be able to merge20:27
bauzasoh man, I just missed reading the commit msg *facepalm*20:28
*** Sukhdev has joined #openstack-nova20:28
superdanbauzas: also: http://lists.openstack.org/pipermail/openstack-dev/2015-April/061814.html20:28
*** annashen has quit IRC20:28
*** andreykurilin___ has joined #openstack-nova20:28
*** andreykurilin__ has quit IRC20:29
bauzassuperdan: okay, I guess you're patching grenade/devstack then ?20:29
*** ijw has joined #openstack-nova20:29
bauzassuperdan: forget it, I'm dumb tonight20:29
superdanbauzas: well, you see the depends-on change there, which is part of it, and sdague is doing the branching for kilo bit soon I think20:29
* bauzas blushes20:30
superdanbauzas: go to sleep :)20:30
mriedemsuperdan: when you get a minute, want to hit this again? https://review.openstack.org/#/c/174524/20:30
superdanmriedem: fwiw, I was suggesting we just convert them all to kwarg dispatch20:31
superdanmriedem: but, +W20:31
superdanuntil the next thing :)20:31
*** Nikolay_St has quit IRC20:31
bauzassuperdan: nah, I just have 30 slides to write before Monday evening...20:31
mriedemyeah....so...we could technically push a patch to kill this in liberty now right?20:31
superdanouch20:32
*** Nikolay_St has joined #openstack-nova20:32
bauzassuperdan: so reviewing just makes me happy20:32
*** annashen has joined #openstack-nova20:32
superdanmriedem: kill what?20:32
mriedemthe proxy20:32
superdanmriedem: well, I was going to leave it until we're done with the RCs20:32
mriedemTODO(danms): This goes away immediately in Lemming20:32
mriedemyeah20:32
superdanjust because otherwise they're no longer backports20:32
mriedemyeah, was going to say that20:32
superdanand less obvious20:32
superdanyeah20:32
mriedemif we miss something how do we backport20:32
mriedemi guess we'd just have to fix stable20:32
superdanthen we just do it, but it's easier to justify while they're the same20:33
superdanyeah20:33
mriedemgod, someone should just convert these all to kwargs :P20:33
superdanheh20:33
mriedemzzzeek: for https://review.openstack.org/#/c/154243/ did you have some numbers in a bug report somewhere?20:36
zzzeekmriedem: yes hold on20:36
zzzeekmriedem: https://bugzilla.redhat.com/show_bug.cgi?id=108598920:37
openstackzzzeek: Error: Error getting bugzilla.redhat.com bug #1085989: NotPermitted20:37
zzzeekmriedem: ack its private20:37
zzzeekmriedem: erg….20:37
mriedemzzzeek: yeah....20:37
mriedemLP bug in nova?20:37
mriedemwith the public bits20:37
zzzeekmriedem: we can make one20:37
mriedemalso, do you want/need the deleted column in the index like in reservations_deleted_expire_idx ?20:37
*** otter768 has joined #openstack-nova20:38
zzzeekmriedem: we looked into that and it wasn’t really needed20:38
zzzeekwtf is this bug private for20:38
mriedemsecrets!20:39
mriedemssshhhh20:39
mriedemzzzeek: ok, ack on deleted column20:40
*** thedodd has quit IRC20:42
*** jichen has joined #openstack-nova20:43
*** otter768 has quit IRC20:43
*** annashen_ has joined #openstack-nova20:44
*** annashen has quit IRC20:44
mriedemzzzeek: -1 for a bug in LP, then i'll fast approve that one20:45
*** emagana has quit IRC20:45
zzzeekmriedem: huh20:46
*** dsanders has joined #openstack-nova20:46
*** emagana has joined #openstack-nova20:47
openstackgerritHans Lindgren proposed openstack/nova: Cleanup conductor unused methods  https://review.openstack.org/16885620:50
*** raildo has quit IRC20:51
*** VW_ has quit IRC20:53
*** sdake_ has joined #openstack-nova20:56
*** baoli_ has quit IRC20:56
*** tonytan4ever has quit IRC20:56
*** Nikolay_St has quit IRC20:57
*** vilobhmm11 has quit IRC20:58
*** rook has quit IRC20:59
mriedemzzzeek: btw, i'm out of here in about 10 minutes20:59
*** sdake has quit IRC20:59
zzzeekmriedem: no hurry on this one, dansmith said not to merge any migrations anyway right?21:00
*** vilobhmm1 has joined #openstack-nova21:00
mriedemzzzeek: we can in master, it's liberty21:00
superdanmriedem: no man21:00
zzzeekmriedem: OK.  im finishing up some SQLA stuff so ill just try to get the LP in can you get it on monday21:00
mriedemzzzeek: ping me on monday yeah21:00
mriedemsuperdan: no?21:00
superdanmriedem: http://lists.openstack.org/pipermail/openstack-dev/2015-April/061814.html21:00
*** vilobhmm1 has quit IRC21:00
mriedemsuperdan: you probably want to -2 this then https://review.openstack.org/#/c/154243/21:01
mriedemb/c it's got a +@21:01
mriedem+221:01
*** armax has quit IRC21:01
superdanyep21:01
*** vilobhmm1 has joined #openstack-nova21:01
superdanwas trying to get gerrit to show me any proposed migrations, but I failed21:01
*** figleaf is now known as edleafe21:02
mriedemwell, gerrit is down it looks like21:03
mriedemotherwise https://review.openstack.org/Documentation/user-search.html#_search_operators21:03
superdanworks for me21:03
superdanyeah, I got it21:03
superdanit's finicky21:03
*** hdd has quit IRC21:04
mriedemhmm21:04
mriedemhttps://review.openstack.org/#/q/status:open+file:.*migrate_repo/versions.*,n,z21:04
mriedemwhat did you use?21:04
-openstackstatus- NOTICE: Gerrit will be unavailable between 22:00 and 23:59 UTC for project renames and a database update.21:04
mriedemah nvm21:04
superdanhttps://review.openstack.org/#/q/project:openstack/nova+file:%22%255E.*migrate_repo.*%2524%22+status:open,n,z21:05
superdanworks21:05
mriedemhttps://review.openstack.org/#/q/status:open+file:^.*migrate_repo/versions.*+project:openstack/nova,n,z21:05
mriedemalright, well i'm done - babysitter shows up in an hour, date night!21:06
superdanooh boy!21:06
superdanevery night is date night in my house21:06
superdanthe cat watches himself21:06
mriedemi should have something here, but21:07
superdanheh21:07
*** mriedem has quit IRC21:07
*** Marga_ has joined #openstack-nova21:08
*** tonytan4ever has joined #openstack-nova21:09
*** arnaud___ has joined #openstack-nova21:13
*** ozialien has quit IRC21:16
*** arnaud___ has quit IRC21:17
*** BadCub is now known as Party_Bear21:17
*** openstackgerrit has quit IRC21:23
*** openstackgerrit has joined #openstack-nova21:23
*** dsanders has quit IRC21:23
*** dsanders has joined #openstack-nova21:25
*** sdake has joined #openstack-nova21:25
*** sdake_ has quit IRC21:29
*** erlon has quit IRC21:31
*** emagana has quit IRC21:33
*** annegentle has joined #openstack-nova21:34
*** vilobhmm1 has quit IRC21:35
*** annashen_ has quit IRC21:36
*** vilobhmm1 has joined #openstack-nova21:37
*** annashen has joined #openstack-nova21:37
*** ozialien has joined #openstack-nova21:37
*** emagana has joined #openstack-nova21:41
*** ArxCruz has joined #openstack-nova21:41
*** emagana has quit IRC21:41
*** emagana has joined #openstack-nova21:42
*** dsanders has quit IRC21:42
*** dsanders has joined #openstack-nova21:42
*** armax has joined #openstack-nova21:47
*** jamielennox|away is now known as jamielennox21:48
*** markvoelker has joined #openstack-nova21:49
*** markvoelker_ has joined #openstack-nova21:50
*** vilobhmm1 has quit IRC21:51
*** otter768 has joined #openstack-nova21:54
*** markvoelker has quit IRC21:54
*** mtecer has quit IRC21:55
*** hdd has joined #openstack-nova21:56
*** markvoelker_ has quit IRC21:58
*** matrohon has quit IRC22:00
openstackgerritMichael Bayer proposed openstack/nova: Add an index to virtual_interfaces.uuid  https://review.openstack.org/15424322:02
openstackgerritMitsuhiro Tanino proposed openstack/nova: libvirt: Handle mismatch of volume information between BDM and domain XML  https://review.openstack.org/17506722:04
*** jecarey has quit IRC22:04
-openstackstatus- NOTICE: Gerrit is unavailable until 23:59 UTC for project renames and a database update.22:04
*** annegentle has quit IRC22:06
-openstackstatus- NOTICE: Gerrit is unavailable until 23:59 UTC for project renames and a database update.22:08
*** ChanServ changes topic to "Gerrit is unavailable until 23:59 UTC for project renames and a database update."22:08
*** vilobhmm1 has joined #openstack-nova22:08
*** andreykurilin___ has quit IRC22:13
*** hdd has quit IRC22:15
*** annashen has quit IRC22:15
*** jichen has quit IRC22:16
*** annashen has joined #openstack-nova22:16
*** jgrimm is now known as zz_jgrimm22:20
*** sdake_ has joined #openstack-nova22:24
*** otter768 has quit IRC22:24
*** sdake has quit IRC22:27
*** vilobhmm1 has quit IRC22:27
*** vilobhmm1 has joined #openstack-nova22:28
*** Sukhdev has quit IRC22:28
*** otter768 has joined #openstack-nova22:36
*** annegentle has joined #openstack-nova22:38
*** tjones1 has joined #openstack-nova22:40
*** dboik_ has quit IRC22:41
*** dboik has joined #openstack-nova22:41
*** annashen has quit IRC22:41
*** ArxCruz has quit IRC22:42
*** isd has quit IRC22:43
*** dave-mccowan has quit IRC22:44
*** rushil has quit IRC22:46
*** sdake has joined #openstack-nova22:46
*** igordcard_ has quit IRC22:47
*** dnalezyt has quit IRC22:47
*** IanGovett1 has quit IRC22:48
*** annegentle has quit IRC22:49
*** sdake_ has quit IRC22:50
*** Sukhdev has joined #openstack-nova22:51
*** achanda has quit IRC22:54
*** annashen has joined #openstack-nova22:54
*** Hosam has joined #openstack-nova22:55
*** aysyd has quit IRC22:56
*** achanda has joined #openstack-nova22:56
*** arnaud___ has joined #openstack-nova23:02
*** dboik has quit IRC23:02
*** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. -=- master is open for Lemming development"23:05
-openstackstatus- NOTICE: Gerrit is available again.23:05
*** arnaud___ has quit IRC23:06
*** patrickeast has joined #openstack-nova23:10
*** doude has quit IRC23:11
*** achanda has quit IRC23:11
cfriesenany cores on still?  https://review.openstack.org/#/c/162746/9 just needs final approval23:11
*** lpetrut has quit IRC23:14
*** sdake_ has joined #openstack-nova23:14
*** annashen has quit IRC23:15
*** achanda has joined #openstack-nova23:15
*** ganso_ has quit IRC23:17
*** sdake has quit IRC23:17
*** tjones1 has quit IRC23:19
*** zzzeek has quit IRC23:23
*** dboik has joined #openstack-nova23:29
*** jamielennox is now known as jamielennox|away23:30
*** pixelb has quit IRC23:30
*** sdake has joined #openstack-nova23:32
*** zzzeek has joined #openstack-nova23:33
*** nikhil_k is now known as cpallares_23:36
*** sdake_ has quit IRC23:36
*** flaper87 has quit IRC23:38
*** cpallares_ is now known as nikhil_k23:40
*** patrickeast has quit IRC23:50
*** Marga_ has quit IRC23:50
*** hemna has quit IRC23:52
*** neelashah has quit IRC23:52
*** annegentle has joined #openstack-nova23:54
*** zzzeek has quit IRC23:55
*** dboik has quit IRC23:57
« Thursday, 2015-04-16 Index Saturday, 2015-04-18 »

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