| jgriffith | patrickeast: yeah, I'll see if I can't load it up and play around with it | 00:00 |
|---|---|---|
| *** annegentle has joined #openstack-cinder | 00:00 | |
| patrickeast | jgriffith: it should work, ive been playing around with it on a local devstack… you just need to create a new project and user and set all of the config options | 00:01 |
| patrickeast | jgriffith: keep the projects volume quota low tho since it will fill them all up right now, lol | 00:01 |
| *** dannywilson has quit IRC | 00:01 | |
| *** jwcroppe_ has quit IRC | 00:03 | |
| *** annegentle has quit IRC | 00:05 | |
| jgriffith | patrickeast: :) | 00:07 |
| *** hemna is now known as hemnafk | 00:11 | |
| *** Rockyg has joined #openstack-cinder | 00:12 | |
| *** IlyaG has quit IRC | 00:15 | |
| openstackgerrit | xing-yang proposed openstack/cinder: EMC VMAX Create CG from CG Snapshot https://review.openstack.org/195280 | 00:16 |
| *** patrickeast has quit IRC | 00:18 | |
| *** markvoelker has joined #openstack-cinder | 00:24 | |
| *** markvoelker has quit IRC | 00:30 | |
| *** heyun has joined #openstack-cinder | 00:32 | |
| *** Longgeek has joined #openstack-cinder | 00:38 | |
| *** leeantho has quit IRC | 00:41 | |
| *** IanGovett has quit IRC | 00:42 | |
| *** Longgeek has quit IRC | 00:44 | |
| *** daneyon has joined #openstack-cinder | 00:50 | |
| *** Rockyg has quit IRC | 00:52 | |
| *** VManiac16 has quit IRC | 00:52 | |
| *** annegentle has joined #openstack-cinder | 01:01 | |
| *** daneyon_ has joined #openstack-cinder | 01:02 | |
| *** jwcroppe has joined #openstack-cinder | 01:02 | |
| *** daneyon has quit IRC | 01:05 | |
| *** tobe has joined #openstack-cinder | 01:05 | |
| *** jwcroppe_ has joined #openstack-cinder | 01:05 | |
| *** jwcroppe has quit IRC | 01:06 | |
| *** annegentle has quit IRC | 01:06 | |
| *** david-ly_ has joined #openstack-cinder | 01:06 | |
| *** david-lyle has quit IRC | 01:09 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Separate FlashSystem FC and iSCSI common code https://review.openstack.org/195382 | 01:12 |
| *** julim has joined #openstack-cinder | 01:13 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Separate FlashSystem FC and iSCSI common code https://review.openstack.org/195382 | 01:14 |
| *** IlyaG has joined #openstack-cinder | 01:15 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:15 | |
| *** IlyaG has quit IRC | 01:19 | |
| *** lixiaoy11 has joined #openstack-cinder | 01:21 | |
| *** lixiaoy1 has quit IRC | 01:23 | |
| *** anshul has joined #openstack-cinder | 01:31 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:31 | |
| *** lixiaoy11 has quit IRC | 01:33 | |
| *** jasondotstar has joined #openstack-cinder | 01:33 | |
| *** et10man has joined #openstack-cinder | 01:39 | |
| *** boris-42 has quit IRC | 01:42 | |
| *** Yogi1 has joined #openstack-cinder | 01:44 | |
| *** amoturi has quit IRC | 01:45 | |
| *** _cjones_ has quit IRC | 01:58 | |
| *** jasondotstar has quit IRC | 01:59 | |
| *** dosaboy has quit IRC | 02:04 | |
| *** wolsen has quit IRC | 02:05 | |
| *** wolsen has joined #openstack-cinder | 02:05 | |
| *** dosaboy has joined #openstack-cinder | 02:06 | |
| *** Apoorva has quit IRC | 02:06 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Separate FlashSystem FC and iSCSI common code https://review.openstack.org/195382 | 02:08 |
| *** daneyon_ has quit IRC | 02:09 | |
| *** jasondotstar has joined #openstack-cinder | 02:10 | |
| openstackgerrit | xing-yang proposed openstack/os-brick: Add connector driver for the ScaleIO cinder driver https://review.openstack.org/185835 | 02:13 |
| *** markvoelker has joined #openstack-cinder | 02:14 | |
| *** dims has joined #openstack-cinder | 02:14 | |
| *** dims_ has quit IRC | 02:15 | |
| openstackgerrit | wanghao proposed openstack/cinder: Support volume backup of qcow2 format volume https://review.openstack.org/182276 | 02:16 |
| *** Yogi1 has quit IRC | 02:17 | |
| *** daneyon has joined #openstack-cinder | 02:17 | |
| *** markvoelker has quit IRC | 02:18 | |
| *** daneyon has quit IRC | 02:18 | |
| *** dims has quit IRC | 02:20 | |
| *** lan has quit IRC | 02:22 | |
| *** dims has joined #openstack-cinder | 02:22 | |
| *** annegentle has joined #openstack-cinder | 02:24 | |
| *** lan has joined #openstack-cinder | 02:25 | |
| *** IlyaG has joined #openstack-cinder | 02:25 | |
| *** annashen has joined #openstack-cinder | 02:32 | |
| *** jwcroppe_ has quit IRC | 02:33 | |
| *** alexpilotti has quit IRC | 02:36 | |
| *** r-daneel has quit IRC | 02:39 | |
| *** annegentle has quit IRC | 02:39 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: sheepdog: Add class for dog command executor https://review.openstack.org/194532 | 02:40 |
| *** Longgeek has joined #openstack-cinder | 02:40 | |
| *** jasondotstar has quit IRC | 02:42 | |
| *** Yogi1 has joined #openstack-cinder | 02:46 | |
| *** lpabon has quit IRC | 02:47 | |
| *** Longgeek has quit IRC | 02:52 | |
| *** Yogi1 has quit IRC | 02:53 | |
| *** esker has joined #openstack-cinder | 02:54 | |
| *** esker has quit IRC | 02:54 | |
| *** esker has joined #openstack-cinder | 02:55 | |
| *** dims has quit IRC | 03:01 | |
| *** mriedem has quit IRC | 03:02 | |
| *** vincent_hou has joined #openstack-cinder | 03:05 | |
| vincent_hou | xyang1: Ping. | 03:06 |
| xyang1 | vincent_hou: Hi | 03:06 |
| *** IlyaG has quit IRC | 03:06 | |
| vincent_hou | xyang1: I have answered your question in patch https://review.openstack.org/#/c/180873/ | 03:06 |
| vincent_hou | It is a long story and hope you can understand. :-) | 03:07 |
| xyang1 | vincent_hou: Ok, I'll take a look | 03:07 |
| *** xyang1 has quit IRC | 03:09 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Separate FlashSystem FC and iSCSI common code https://review.openstack.org/195382 | 03:10 |
| *** xyang1 has joined #openstack-cinder | 03:14 | |
| openstackgerrit | wanghao proposed openstack/cinder-specs: Incremental backup improvements for L https://review.openstack.org/187926 | 03:15 |
| *** boris-42 has joined #openstack-cinder | 03:22 | |
| xyang1 | vincent_hou: I'll check with Sean tomorrow to make sure. There are a couple of others that implemented this too. | 03:22 |
| xyang1 | vincent_hou: What is the downside of keeping this call for attached? That will preserve the existing behavior for attached in this patch | 03:24 |
| *** harlowja_at_home has joined #openstack-cinder | 03:31 | |
| Swanson | Update_migrated_volume has changed? | 03:31 |
| Swanson | xyang1: is update_migrated_volume no longer being called in the patch above? | 03:36 |
| xyang1 | Swanson: It is no longer called for attached volume | 03:37 |
| xyang1 | Swanson: It is still called for available volume | 03:37 |
| xyang1 | Swanson: Will that be a problem for your driver | 03:37 |
| Swanson | xyang1: If the volume is migrated to a new backend it will be copied to a new volume with a new guid. If the new guid is used going forward we have no issues. If not we will lose track of the volume. | 03:38 |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 03:39 |
| Swanson | xyang1: We use the id as our volume name. That is the only link to the volume. If that isn't the same as what the db thinks it is the volume is as good as gone to us. | 03:40 |
| *** annegentle has joined #openstack-cinder | 03:40 | |
| xyang1 | vincent_hou: Still here? | 03:40 |
| Swanson | xyang1: I can talk to Sean tomorrow morning as I think he has paid more attention to this patch than I have. (I stopped looking at it after implementing the update_migrated_volume method fixed our problem.) | 03:41 |
| xyang1 | Swanson: Sure | 03:42 |
| xyang1 | Swanson: I added a question for sean in the patch as well | 03:42 |
| *** vincent_hou_ has joined #openstack-cinder | 03:43 | |
| *** Longgeek has joined #openstack-cinder | 03:44 | |
| *** vincent_hou has quit IRC | 03:44 | |
| *** vincent_hou_ is now known as vincent_hou | 03:44 | |
| *** annegentle has quit IRC | 03:45 | |
| *** laughterwym has quit IRC | 03:45 | |
| *** laughterwym_ has joined #openstack-cinder | 03:45 | |
| *** Longgeek has quit IRC | 03:46 | |
| *** Longgeek has joined #openstack-cinder | 03:47 | |
| *** jwcroppe has joined #openstack-cinder | 03:47 | |
| *** david-ly_ is now known as david-lyle | 03:51 | |
| *** julim has quit IRC | 03:52 | |
| *** lpetrut has joined #openstack-cinder | 03:52 | |
| *** annashen has quit IRC | 03:57 | |
| *** markvoelker has joined #openstack-cinder | 04:02 | |
| *** vilobhmm has joined #openstack-cinder | 04:04 | |
| *** markvoelker has quit IRC | 04:07 | |
| *** shyama has joined #openstack-cinder | 04:07 | |
| *** annashen has joined #openstack-cinder | 04:11 | |
| *** rushiagr_away is now known as rushiagr | 04:13 | |
| *** annashen has quit IRC | 04:14 | |
| *** xyang1 has quit IRC | 04:15 | |
| vincent_hou | smcginnis: Hi Sean. Xing has a question for you in the comment for https://review.openstack.org/#/c/180873/61/cinder/volume/manager.py. | 04:15 |
| *** annashen has joined #openstack-cinder | 04:19 | |
| *** lpetrut has quit IRC | 04:34 | |
| *** vilobhmm has quit IRC | 04:36 | |
| *** harlowja_at_home has quit IRC | 04:36 | |
| *** tobe has quit IRC | 04:40 | |
| *** annegentle has joined #openstack-cinder | 04:41 | |
| vincent_hou | Swanson: Still there? | 04:43 |
| *** annegentle has quit IRC | 04:46 | |
| *** mdbooth has quit IRC | 04:54 | |
| *** tobe has joined #openstack-cinder | 05:19 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: sheepdog: Add class for dog command executor https://review.openstack.org/194532 | 05:28 |
| *** annashen has quit IRC | 05:29 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Add dependency check in RBD delete_snapshot https://review.openstack.org/193003 | 05:32 |
| openstackgerrit | TaoBai proposed openstack/cinder: Storwize driver report capability for multiattach https://review.openstack.org/195471 | 05:42 |
| *** annegentle has joined #openstack-cinder | 05:42 | |
| *** rushiagr is now known as rushiagr_away | 05:43 | |
| *** mtanino has joined #openstack-cinder | 05:43 | |
| mtanino | vincent_hou: hi, | 05:44 |
| vincent_hou | mtanino: Hi | 05:45 |
| mtanino | vincent_hou: Did you see my comment on my patch? https://review.openstack.org/#/c/194223/1 | 05:45 |
| vincent_hou | Ah. | 05:45 |
| vincent_hou | Yes. | 05:46 |
| vincent_hou | mtanino: I think currently we can fix it this way. | 05:46 |
| mtanino | vincent_hou: hmm | 05:47 |
| mtanino | someting happen in the future, you mean? | 05:47 |
| *** annegentle has quit IRC | 05:47 | |
| vincent_hou | I was think why in terminate_connect , the export is not removed. | 05:47 |
| mtanino | because terminate_connection does not call remove export | 05:48 |
| vincent_hou | why doesn't | 05:48 |
| mtanino | just terminate connection is a feature of terminate_connection | 05:48 |
| vincent_hou | What if we add it? Because during init_connect, we create the export. Why cannot we remove it in terminate-connect? | 05:49 |
| mtanino | If the terminte_connection has remove_export feature, nova instance(with volume) live migration will be failed. | 05:49 |
| vincent_hou | i am not sure if there is any reason for it. | 05:50 |
| mtanino | Nova's live migration code calls terminate_connection for the purpose of only terminating connection.(without removing export) | 05:51 |
| vincent_hou | Wow | 05:51 |
| *** markvoelker has joined #openstack-cinder | 05:51 | |
| vincent_hou | That is sth I did not realize. | 05:51 |
| *** kaisers has quit IRC | 05:51 | |
| *** laughterwym_ has quit IRC | 05:51 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for Liberty version https://review.openstack.org/186327 | 05:52 |
| mtanino | If you will add remove_export in the terminate_connection and then migrate nova instance with volume, | 05:52 |
| *** boris-42 has quit IRC | 05:52 | |
| mtanino | I think you will fail it. | 05:52 |
| vincent_hou | OK | 05:52 |
| vincent_hou | Thanks for your explanation. | 05:53 |
| *** kaisers has joined #openstack-cinder | 05:53 | |
| mtanino | therefore, I called attach volume and detach volume. This is same logic of Nova's live migration | 05:53 |
| mtanino | sure. | 05:53 |
| vincent_hou | OK I support it. | 05:54 |
| vincent_hou | I will check the patch once more. | 05:54 |
| *** links has joined #openstack-cinder | 05:55 | |
| vincent_hou | BTW, did you receive the latest mail from me? | 05:55 |
| *** markvoelker has quit IRC | 05:56 | |
| vincent_hou | about my idea for the back-end name consistency for attached volumes? | 05:56 |
| mtanino | Ah, I recieved it but couldn't read enough. | 05:58 |
| mtanino | vincent_hou: will be reply tomorrow or next week. it's OK? | 05:59 |
| vincent_hou | take your time | 05:59 |
| vincent_hou | NP | 05:59 |
| mtanino | :) | 05:59 |
| mtanino | ok, go to sleep, zzz | 05:59 |
| mtanino | bye | 05:59 |
| *** mtanino is now known as mtanino_zzz | 06:00 | |
| *** vincent_hou_ has joined #openstack-cinder | 06:01 | |
| *** tobe has quit IRC | 06:05 | |
| *** vincent_hou has quit IRC | 06:05 | |
| *** vincent_hou_ is now known as vincent_hou | 06:05 | |
| *** mtanino_zzz has quit IRC | 06:14 | |
| *** rushiagr_away is now known as rushiagr | 06:15 | |
| vincent_hou | bye | 06:16 |
| *** boris-42 has joined #openstack-cinder | 06:26 | |
| *** annashen has joined #openstack-cinder | 06:30 | |
| *** lpetrut has joined #openstack-cinder | 06:36 | |
| *** tobe has joined #openstack-cinder | 06:40 | |
| *** belmoreira has joined #openstack-cinder | 06:43 | |
| *** annegentle has joined #openstack-cinder | 06:43 | |
| *** coolsvap|away is now known as coolsvap | 06:44 | |
| *** annegentle has quit IRC | 06:48 | |
| *** coolsvap is now known as coolsvap|away | 06:56 | |
| *** lpetrut has quit IRC | 07:00 | |
| *** ronis has joined #openstack-cinder | 07:03 | |
| *** yamada-h has joined #openstack-cinder | 07:05 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Add dependency check in RBD delete_snapshot https://review.openstack.org/193003 | 07:06 |
| *** lpetrut has joined #openstack-cinder | 07:10 | |
| *** vincent_hou_ has joined #openstack-cinder | 07:11 | |
| openstackgerrit | Andrey Pavlov proposed openstack/cinder: Avoid race condition at snapshot deletion stage https://review.openstack.org/193596 | 07:12 |
| *** vincent_hou has quit IRC | 07:13 | |
| *** vincent_hou_ is now known as vincent_hou | 07:14 | |
| *** lpetrut has quit IRC | 07:14 | |
| *** yamada-h has quit IRC | 07:16 | |
| *** annashen has quit IRC | 07:23 | |
| *** bkopilov has quit IRC | 07:23 | |
| *** rushiagr is now known as rushiagr_away | 07:36 | |
| *** shyama has quit IRC | 07:44 | |
| *** markus_z has joined #openstack-cinder | 07:55 | |
| *** chlong has quit IRC | 07:55 | |
| *** bluex-pl has joined #openstack-cinder | 07:58 | |
| aarefiev | vincent_hou: hi, could you revisit it https://review.openstack.org/#/c/178588 , and comment please ? | 08:05 |
| vincent_hou | aarefiev: V1 won't exist in Cinder. Do you think it is meaningful to have them in client? | 08:13 |
| aarefiev | vincent_hou: what about existing deployments with v1 | 08:13 |
| *** jwcroppe has quit IRC | 08:14 | |
| aarefiev | vincent_hou: I think we should support it too | 08:14 |
| *** mdbooth has joined #openstack-cinder | 08:15 | |
| vincent_hou | I need other folks' comments. If there are more cores fine with it. I am OK. | 08:16 |
| *** jwcroppe has joined #openstack-cinder | 08:18 | |
| *** jwcroppe has quit IRC | 08:19 | |
| *** jwcroppe has joined #openstack-cinder | 08:19 | |
| *** jwcroppe has quit IRC | 08:21 | |
| *** jwcroppe has joined #openstack-cinder | 08:22 | |
| *** jwcroppe has quit IRC | 08:22 | |
| *** jwcroppe has joined #openstack-cinder | 08:24 | |
| *** jwcroppe has quit IRC | 08:24 | |
| *** Lee1092 has quit IRC | 08:25 | |
| vincent_hou | aarefiev: https://review.openstack.org/#/c/92768/ you can check this patch. There are similar comments. | 08:27 |
| *** jwcroppe has joined #openstack-cinder | 08:28 | |
| aarefiev | vincent_hou: ok, but I'm not agree with you, we a have lot of envs with v1, and I'd like to see bugs fixed for them too | 08:28 |
| vincent_hou | aarefiev: I am gonna ask other folks as well. Hope we will a solution to it. | 08:30 |
| *** jwcroppe has quit IRC | 08:30 | |
| aarefiev | vincent_hou: also when we delete v1 in api, we should support v1 in client, because we can't upgrade all envs to v2 in time | 08:30 |
| *** jwcroppe has joined #openstack-cinder | 08:30 | |
| aarefiev | vincent_hou: ok, thanks, please comment patches after decision | 08:35 |
| vincent_hou | Sure. | 08:36 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 08:41 |
| *** shyama has joined #openstack-cinder | 08:49 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 08:50 |
| *** jwcroppe has quit IRC | 08:56 | |
| *** jwcroppe has joined #openstack-cinder | 08:58 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 08:58 |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Add the --force flag to cinder migrate https://review.openstack.org/189547 | 09:00 |
| *** tobe has quit IRC | 09:02 | |
| *** abhishekk has joined #openstack-cinder | 09:03 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/195932 | 09:04 |
| *** breitz has quit IRC | 09:05 | |
| *** breitz has joined #openstack-cinder | 09:06 | |
| *** vincent_hou has quit IRC | 09:08 | |
| *** husanu0 has joined #openstack-cinder | 09:09 | |
| *** jordanP has joined #openstack-cinder | 09:10 | |
| *** husanu0 has quit IRC | 09:18 | |
| *** boris-42 has quit IRC | 09:22 | |
| *** ndipanov has joined #openstack-cinder | 09:22 | |
| *** husanu4 has joined #openstack-cinder | 09:23 | |
| *** markus_z has quit IRC | 09:23 | |
| *** markus_z has joined #openstack-cinder | 09:28 | |
| *** husanu4 has quit IRC | 09:28 | |
| *** markvoelker has joined #openstack-cinder | 09:29 | |
| openstackgerrit | Forest Romain proposed openstack/cinder: Correct overquota error message https://review.openstack.org/193018 | 09:33 |
| *** markvoelker has quit IRC | 09:33 | |
| *** annegentle has joined #openstack-cinder | 09:45 | |
| *** annegentle has quit IRC | 09:50 | |
| *** lixiaoy1 has quit IRC | 09:52 | |
| *** dims has joined #openstack-cinder | 09:59 | |
| openstackgerrit | Cedric Zhuang proposed openstack/cinder: Use symbol for error code in VNX cinder driver https://review.openstack.org/189208 | 10:00 |
| *** bluex-pl has quit IRC | 10:02 | |
| *** ankit_ag has joined #openstack-cinder | 10:10 | |
| *** e0ne has joined #openstack-cinder | 10:15 | |
| *** dims has quit IRC | 10:18 | |
| *** agarciam has joined #openstack-cinder | 10:21 | |
| openstackgerrit | Andrey Pavlov proposed openstack/cinder: Avoid race condition at snapshot deletion stage https://review.openstack.org/193596 | 10:23 |
| *** annashen has joined #openstack-cinder | 10:24 | |
| abhishekk | ndipanov: hi, you around? | 10:25 |
| ndipanov | abhishekk, hey | 10:27 |
| abhishekk | ndipanov: need 5 minuts, is it good time? | 10:27 |
| *** annashen has quit IRC | 10:28 | |
| ndipanov | start typing but I'll go get a coffee real quick :) | 10:29 |
| abhishekk | ndipanov: https://bugs.launchpad.net/nova/+bug/1416132 | 10:29 |
| openstack | Launchpad bug 1416132 in OpenStack Compute (nova) "_get_instance_disk_info fails to read files from NFS due to permissions" [High,In progress] - Assigned to Eric Harney (eharney) | 10:29 |
| ndipanov | yes | 10:29 |
| abhishekk | I have applied your patch, but the permission issue is still there | 10:29 |
| ndipanov | hmm so it's not fixing it then :( | 10:30 |
| abhishekk | see comment #4 | 10:30 |
| ndipanov | can you give me more details? | 10:30 |
| ndipanov | instance xml for example | 10:30 |
| abhishekk | ok | 10:30 |
| ndipanov | and block_device_mapping table | 10:30 |
| ndipanov | WHERE instance_uuid = your uuuid | 10:31 |
| abhishekk | ok just give me a minute, or you can proceed to coffee break, I will paste in here | 10:35 |
| *** links has quit IRC | 10:36 | |
| *** et10man has quit IRC | 10:37 | |
| abhishekk | ndipanov: http://paste.openstack.org/show/321326 | 10:43 |
| *** heyun has quit IRC | 10:50 | |
| ndipanov | abhishekk, thanks! that is very helpful! | 10:53 |
| abhishekk | ndipanov: thank you too :) | 10:53 |
| *** IanGovett has joined #openstack-cinder | 10:56 | |
| *** dims has joined #openstack-cinder | 10:59 | |
| *** e0ne is now known as e0ne_ | 11:08 | |
| *** Lee1092 has joined #openstack-cinder | 11:10 | |
| *** kbyrne has quit IRC | 11:16 | |
| *** links has joined #openstack-cinder | 11:17 | |
| *** markvoelker has joined #openstack-cinder | 11:17 | |
| *** sgotliv has quit IRC | 11:18 | |
| *** e0ne_ has quit IRC | 11:18 | |
| *** sgotliv has joined #openstack-cinder | 11:19 | |
| *** links has quit IRC | 11:19 | |
| *** kbyrne has joined #openstack-cinder | 11:20 | |
| *** kbyrne has quit IRC | 11:21 | |
| *** kbyrne has joined #openstack-cinder | 11:21 | |
| *** markvoelker has quit IRC | 11:22 | |
| *** lpetrut has joined #openstack-cinder | 11:25 | |
| *** delattec has quit IRC | 11:31 | |
| *** cdelatte has quit IRC | 11:31 | |
| *** bluex-pl has joined #openstack-cinder | 11:36 | |
| *** e0ne has joined #openstack-cinder | 11:39 | |
| openstackgerrit | Andrey Pavlov proposed openstack/cinder: Avoid race condition at snapshot deletion stage https://review.openstack.org/193596 | 11:40 |
| *** aix has quit IRC | 11:41 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Validate string, integer limit for input parameter https://review.openstack.org/194968 | 11:46 |
| *** annegentle has joined #openstack-cinder | 11:47 | |
| *** annegentle has quit IRC | 11:52 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: WIP: Implement service healthcheck tool https://review.openstack.org/195665 | 11:58 |
| *** timcl has joined #openstack-cinder | 11:59 | |
| *** markvoelker has joined #openstack-cinder | 12:00 | |
| *** Ctina_ has joined #openstack-cinder | 12:04 | |
| *** aix has joined #openstack-cinder | 12:08 | |
| *** dims is now known as dimsum__ | 12:09 | |
| *** laughterwym has joined #openstack-cinder | 12:09 | |
| *** akerr has joined #openstack-cinder | 12:11 | |
| *** dims_ has joined #openstack-cinder | 12:13 | |
| *** dimsum__ has quit IRC | 12:16 | |
| *** Longgeek_ has joined #openstack-cinder | 12:17 | |
| *** rmstar has joined #openstack-cinder | 12:20 | |
| *** Longgeek has quit IRC | 12:21 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 12:21 |
| *** bswartz has quit IRC | 12:26 | |
| *** aix has quit IRC | 12:28 | |
| *** kjnelson has joined #openstack-cinder | 12:29 | |
| *** edmondsw has joined #openstack-cinder | 12:32 | |
| *** jaypipes has joined #openstack-cinder | 12:41 | |
| *** Longgeek has joined #openstack-cinder | 12:41 | |
| *** tellesnobrega has joined #openstack-cinder | 12:45 | |
| *** Longgeek_ has quit IRC | 12:45 | |
| *** julim has joined #openstack-cinder | 12:47 | |
| *** smoriya has quit IRC | 12:48 | |
| *** tellesnobrega has quit IRC | 13:04 | |
| openstackgerrit | Andrey Pavlov proposed openstack/cinder: Avoid race condition at snapshot deletion stage https://review.openstack.org/193596 | 13:04 |
| *** dimsum__ has joined #openstack-cinder | 13:07 | |
| *** gouthamr has joined #openstack-cinder | 13:07 | |
| *** tellesnobrega has joined #openstack-cinder | 13:08 | |
| *** dims_ has quit IRC | 13:10 | |
| *** kaisers has quit IRC | 13:11 | |
| *** dtynan has quit IRC | 13:12 | |
| *** dtynan has joined #openstack-cinder | 13:12 | |
| *** kaisers has joined #openstack-cinder | 13:13 | |
| *** ankit_ag has quit IRC | 13:13 | |
| *** Yogi1 has joined #openstack-cinder | 13:18 | |
| *** shyama has quit IRC | 13:18 | |
| *** merooney has joined #openstack-cinder | 13:18 | |
| *** bswartz has joined #openstack-cinder | 13:19 | |
| *** mriedem has joined #openstack-cinder | 13:25 | |
| *** cdelatte has joined #openstack-cinder | 13:25 | |
| *** delattec has joined #openstack-cinder | 13:25 | |
| delattec | Does anyone by chance know of a way to disable cinder snapshots? we want to temporarily disable just our users abilities to create snapshots while we are going through some changes. Ideally through the policy.json file, but open to suggestions | 13:29 |
| *** dustins has joined #openstack-cinder | 13:31 | |
| smcginnis | delattec: You should be able to change their quota to cap them. | 13:31 |
| smcginnis | delattec: https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux_OpenStack_Platform/4/html/Administration_User_Guide/cli_set_quotas.html#cli_set_block_storage_quotas | 13:32 |
| delattec | #smcginnis we were looking at that at well. the issue with quotas being we have modified project quotas per request for our users. so we have some default, some not default. | 13:33 |
| smcginnis | delattec: OK, I see. Maybe someone else has a suggestion then. | 13:34 |
| delattec | smcginnis: thanks for the suggestion. would have been easy if our customers were all default. could have used puppet-cinder to make all those changes as well | 13:35 |
| *** merooney has quit IRC | 13:37 | |
| *** thangp has joined #openstack-cinder | 13:40 | |
| *** primechuck has joined #openstack-cinder | 13:41 | |
| *** e0ne is now known as e0ne_ | 13:46 | |
| *** boris-42 has joined #openstack-cinder | 13:47 | |
| *** annegentle has joined #openstack-cinder | 13:48 | |
| *** dansmith is now known as superdan | 13:49 | |
| *** annegentle has quit IRC | 13:53 | |
| *** abhishekk has quit IRC | 13:54 | |
| thangp | dulek: ping | 13:54 |
| thangp | dulek: did you see this bug - https://bugs.launchpad.net/cinder/+bug/1469120 | 13:55 |
| openstack | Launchpad bug 1469120 in Cinder "Cannot save dates in Versioned Objects" [Undecided,New] - Assigned to Gorka Eguileor (gorka) | 13:55 |
| *** annegentle has joined #openstack-cinder | 13:55 | |
| dulek | thangp: Looking at it... | 13:55 |
| thangp | dulek: gorka might have a good point | 13:56 |
| *** e0ne_ is now known as e0ne | 13:56 | |
| thangp | dulek: we might have to make every DateTime fields tzinfo_aware=False | 13:56 |
| dulek | geguileo: Hey, Gorka, you should have pinged me on that. ;) | 13:56 |
| dulek | thangp: Why haven't we looked at other projects? | 13:57 |
| thangp | dulek: i started to look at nova | 13:57 |
| geguileo | dulek: I mentioned it in the backup patch yesterday | 13:58 |
| geguileo | dulek: I have a patch for that | 13:58 |
| dulek | geguileo: Oh, I haven't noticed. | 13:58 |
| dulek | Fun fact - Heat has all objects timezone-aware. | 13:58 |
| dulek | I don't get why they haven't run into problems. | 13:59 |
| thangp | dulek: they are using fields.DateTimeField from oslo_versionedobjects | 13:59 |
| thangp | and using the default | 13:59 |
| thangp | https://github.com/openstack/nova/blob/master/nova/objects/fields.py#L48 | 13:59 |
| *** eharney has joined #openstack-cinder | 14:00 | |
| geguileo | We don't have problems either in delete time because we change it using the db methods | 14:00 |
| dulek | thangp: So what's wrong with Cinder if it works for Nova and Heat? | 14:00 |
| *** salv-orl_ has quit IRC | 14:01 | |
| dulek | geguileo: Ah, I get it... But each of these projects has services table. That's the place where you bump up date manually. | 14:01 |
| thangp | dulek: I havent run into it, but I remember running into your patch or mine where we do update the *_at datetime field | 14:01 |
| geguileo | dulek: If I'm not mistaken they don't use the objects to change datetimes | 14:01 |
| thangp | dulek: I dont remember where, but did the save work? | 14:01 |
| dulek | https://review.openstack.org/#/c/160417/ | 14:03 |
| dulek | Probably this one. | 14:03 |
| *** julim has quit IRC | 14:03 | |
| *** jecarey has joined #openstack-cinder | 14:03 | |
| geguileo | I don't think we should disable the tzinfo | 14:04 |
| geguileo | We agreed not to do that in the meeting | 14:04 |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Validate string, integer limit for input parameter https://review.openstack.org/194968 | 14:04 |
| dulek | Oh, I see why services worked - updated_at is handled inside the db.sqlalchemy | 14:05 |
| dulek | But wait, if we don't have timezone info in the DB, then why should we in the code? | 14:06 |
| *** Yogi1 has quit IRC | 14:06 | |
| dulek | That makes no sense to have tz_aware=True. We don't persist such information. | 14:06 |
| thangp | the services object modifies the CinderPersistentObject to have tzinfo_aware=False | 14:07 |
| geguileo | Since it's oslo's default I wouldn't change it | 14:07 |
| *** annegentle has quit IRC | 14:07 | |
| dulek | thangp: Yeah, I haven't had time to update the patch after the meeting. I've meant services in Heat for example. | 14:08 |
| *** DTadrzak has joined #openstack-cinder | 14:08 | |
| dulek | geguileo: Fact that this is default in Oslo doesn't mean that it's a good idea. | 14:08 |
| geguileo | XD | 14:08 |
| geguileo | Well, the other day in the meeting I was alone when suggested to remove it | 14:09 |
| geguileo | So I have fixed this another way | 14:09 |
| geguileo | XD | 14:09 |
| thangp | https://github.com/openstack/heat/blob/master/heat/objects/service.py#L37 | 14:09 |
| dulek | geguileo: If I remember correctly tz awareness was added by my colleague from the office, he isn't some nova-core-god-never-making-mistakes. :D | 14:09 |
| thangp | heat service is using the oslo_versionedobjects fields | 14:09 |
| thangp | with the default | 14:09 |
| geguileo | Nova is doing it as well | 14:10 |
| dulek | thangp: Yes, but they bump up the updated_at using just update method. They don't change it directly. | 14:10 |
| *** krtaylor has quit IRC | 14:10 | |
| *** Yogi1 has joined #openstack-cinder | 14:10 | |
| dulek | thangp: The thing is - if you update it directly - then you're unable to save the object. | 14:10 |
| dulek | I see three options: | 14:11 |
| dulek | 1. Change model (not an option really). | 14:11 |
| dulek | 2. Change fields in objects. | 14:11 |
| dulek | 3. Make conversion (a little nasty...) | 14:11 |
| dulek | Anything else? | 14:11 |
| *** annegentle has joined #openstack-cinder | 14:11 | |
| *** annegentle has quit IRC | 14:11 | |
| thangp | dulek: which update method? | 14:11 |
| thangp | could you point to the code? | 14:12 |
| geguileo | Give me 1 minute and I'll update my solution and we can discuss it | 14:12 |
| *** annegentle has joined #openstack-cinder | 14:13 | |
| *** annegentle has quit IRC | 14:13 | |
| *** amoturi has joined #openstack-cinder | 14:14 | |
| *** e0ne is now known as e0ne_ | 14:14 | |
| dulek | thangp: I'm unable to see where's heartbeat bumping in the code in Nova or Heat. | 14:14 |
| *** merooney has joined #openstack-cinder | 14:15 | |
| thangp | dulek: ok | 14:15 |
| thangp | I think we should keep using the default and modify the update() in cinder's base to handle the save correctly | 14:16 |
| thangp | and the save() | 14:16 |
| dulek | That would be option 3. | 14:16 |
| *** e0ne_ is now known as e0ne | 14:16 | |
| thangp | but it would apply to all objects | 14:16 |
| thangp | if we change it in CinderObject | 14:17 |
| dulek | Yes, that's what I've expected. | 14:17 |
| geguileo | That's something similar to what I've done | 14:17 |
| thangp | but I still dont know why nova works | 14:17 |
| dulek | thangp: :D | 14:17 |
| geguileo | It's not that "nasty" ;-) | 14:17 |
| dulek | So when we're loading non-tz-aware DB row into tz-aware field in object - it assumes it's UTC? | 14:17 |
| thangp | dulek: yup | 14:18 |
| dulek | thangp: https://github.com/openstack/nova/search?utf8=%E2%9C%93&q=scheduled_at+%3D+&type=Code | 14:18 |
| thangp | thangp: and it should based on our meeting yesterday | 14:18 |
| dulek | thangp: I cannot find any occurrence in Nova where they modify the date in an object. | 14:18 |
| thangp | hmm... :-/ | 14:19 |
| geguileo | dulek: That's what I said before, they don't modify dates in the objects | 14:19 |
| geguileo | They access the DB directly, like we do | 14:19 |
| thangp | option 4: get oslo to change their default | 14:21 |
| thangp | would need to talk with superdan | 14:21 |
| dulek | Huh, what's happening in Nova that makes danms become superdan? | 14:22 |
| thangp | dulek: it's a friday | 14:23 |
| thingee | dulek: casual nick friday | 14:23 |
| thangp | dulek: they change to alias | 14:23 |
| dulek | I love the idea. :D | 14:23 |
| geguileo | XD XD | 14:23 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup metadata import missing fields https://review.openstack.org/183222 | 14:24 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix saving tz aware datetimes in Versioned Objects https://review.openstack.org/196071 | 14:24 |
| geguileo | dulek: thangp ^ | 14:26 |
| dulek | Thanks geguileo, let's consult that with superdan and get over it. :) | 14:26 |
| geguileo | dulek: Ok | 14:26 |
| *** r-daneel has joined #openstack-cinder | 14:26 | |
| thangp | geguileo: looks cleaner then changing everything | 14:27 |
| geguileo | thangp: And it supports timezones in the objects | 14:27 |
| thangp | agree, we should consult with superdan | 14:27 |
| geguileo | Which I think it's a good thing | 14:27 |
| geguileo | I have to leave now, I have a wedding to go to, but I'd appretiate if you could keep me updated with you conversation with superdan | 14:28 |
| thangp | geguileo: np, thanks! | 14:29 |
| geguileo | By the way, I also changed the approach to the backup migration to use objects | 14:29 |
| *** lcurtis has joined #openstack-cinder | 14:29 | |
| geguileo | And I think it looks a lot nicer now | 14:29 |
| dulek | geguileo: Thanks, have a nice evening! | 14:30 |
| geguileo | Your suggestions actually helped me improve readability (I think), so thanks | 14:30 |
| *** merooney has quit IRC | 14:30 | |
| *** alexpilotti has joined #openstack-cinder | 14:30 | |
| geguileo | Thanks, you too, and a nice weekend as well | 14:30 |
| thangp | dulek: lets jump to the nova channel to find superdan | 14:31 |
| dulek | thangp: okay! | 14:31 |
| thangp | dulek: he's on a call | 14:32 |
| thangp | dulek: he'll response soon | 14:32 |
| dulek | :) | 14:32 |
| *** sgotliv has quit IRC | 14:34 | |
| *** alexpilotti_ has joined #openstack-cinder | 14:35 | |
| *** alexpilotti has quit IRC | 14:36 | |
| *** alexpilotti_ is now known as alexpilotti | 14:36 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Don't allow >4-byte Unicode chars in volume names https://review.openstack.org/195695 | 14:36 |
| *** merooney has joined #openstack-cinder | 14:38 | |
| ameade | hemnafk: https://review.openstack.org/#/c/157439/13 wrt splitting, i have them as one to show an example, the api.py stuff is how you could do the API level tracing | 14:39 |
| *** rmesta has joined #openstack-cinder | 14:39 | |
| *** lpabon has joined #openstack-cinder | 14:41 | |
| *** xyang1 has joined #openstack-cinder | 14:43 | |
| *** nikeshm has quit IRC | 14:45 | |
| *** jordanP has quit IRC | 14:45 | |
| *** krtaylor has joined #openstack-cinder | 14:46 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Don't allow >3-byte Unicode chars in volume names https://review.openstack.org/195695 | 14:50 |
| *** vilobhmm has joined #openstack-cinder | 14:50 | |
| *** MentalRay has joined #openstack-cinder | 14:52 | |
| *** anshul has quit IRC | 14:53 | |
| *** jaypipes is now known as leakypipes | 14:56 | |
| *** belmoreira has quit IRC | 14:57 | |
| *** kjnelson has quit IRC | 14:58 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 14:59 |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 14:59 |
| Swanson | vincent_hou: ping | 15:02 |
| *** kjnelson has joined #openstack-cinder | 15:04 | |
| *** salv-orlando has joined #openstack-cinder | 15:05 | |
| superdan | thangp: okay, I see some of the backlog here, but probably easier if you summaries | 15:05 |
| superdan | er, summarize | 15:05 |
| *** bluex-pl has quit IRC | 15:05 | |
| thangp | superdan: ok...the problem is outline here - https://bugs.launchpad.net/cinder/+bug/1469120 | 15:06 |
| openstack | Launchpad bug 1469120 in Cinder "Cannot save dates in Versioned Objects" [Undecided,In progress] - Assigned to Gorka Eguileor (gorka) | 15:06 |
| thangp | superdan: basically... | 15:06 |
| *** timcl has quit IRC | 15:06 | |
| *** primechuck has quit IRC | 15:06 | |
| thangp | superdan: the db does not save the timezone | 15:06 |
| thangp | superdan: the objects do | 15:06 |
| superdan | ah, yeah, so we just convert them all to unaware UTC timestamps in db_api | 15:06 |
| thangp | superdan: do should we make the fields.DateTimeField use tzinfo_aware=False? | 15:07 |
| *** ronis has quit IRC | 15:07 | |
| thangp | because by default it is True | 15:07 |
| superdan | you can do that if you want | 15:07 |
| superdan | thangp: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L323 | 15:08 |
| thangp | geguileo: had another suggestion - https://review.openstack.org/#/c/196071/1 | 15:08 |
| superdan | thangp: I probably wouldn't do it that way, but that's certainly easy | 15:08 |
| thangp | superdan: or could we make the oslo_versionedobjects use tzinfo_aware=False by default? | 15:09 |
| superdan | thangp: I think I'd probably make a obj_cinder_get_changes() or something just to make it clearly a cinderism | 15:09 |
| superdan | thangp: yeah, but that changes the RPC API, if you've already started passing these objects over the wire... | 15:09 |
| superdan | thangp: TBH, I think the TZ-aware-ness is most important over RPC | 15:09 |
| thangp | superdan: hmm...ok | 15:09 |
| superdan | we specifically decided to default to TZ-aware just to be very clear about things, | 15:10 |
| superdan | because we had like two timestamps that were in local time or something silly like that | 15:10 |
| superdan | and it was always confusing | 15:10 |
| *** rakesh_mishra has joined #openstack-cinder | 15:10 | |
| thangp | ok, so changing oslo is not an option, so we will make some changes in cinder obj to handle it | 15:11 |
| thangp | thx! was quick | 15:11 |
| geguileo | superdan: Thanks | 15:11 |
| superdan | geguileo: np | 15:11 |
| superdan | for what it's worth, this decision to be explicit about TZs has caused no end of pain for everyone | 15:12 |
| geguileo | And that you all for having the conversation here, I got to read it before leaving :) | 15:12 |
| thangp | :) | 15:12 |
| superdan | I still think it's the right thing to do in a model, but it's still painful :) | 15:12 |
| superdan | geguileo: no problem, I was just on a call earlier and had to pay attention | 15:12 |
| *** dimsum__ has quit IRC | 15:13 | |
| geguileo | Usually a good idea paying attention to phone conversations XD | 15:13 |
| superdan | heh | 15:13 |
| *** garthb has joined #openstack-cinder | 15:14 | |
| *** salv-orlando has quit IRC | 15:14 | |
| *** garthb_ has joined #openstack-cinder | 15:14 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Use right oslo.service entry points https://review.openstack.org/196099 | 15:15 |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Use right oslo.service entry points https://review.openstack.org/196099 | 15:17 |
| *** DericHorn-HP has joined #openstack-cinder | 15:23 | |
| *** e0ne is now known as e0ne_ | 15:25 | |
| *** e0ne_ is now known as e0ne | 15:25 | |
| *** zul has quit IRC | 15:28 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Use right oslo.service entry points https://review.openstack.org/196099 | 15:29 |
| *** garthb_ has quit IRC | 15:30 | |
| *** garthb has quit IRC | 15:30 | |
| *** merooney has quit IRC | 15:31 | |
| *** zul has joined #openstack-cinder | 15:33 | |
| *** tellesnobrega has quit IRC | 15:34 | |
| *** DericHorn-HP has quit IRC | 15:36 | |
| *** tellesnobrega has joined #openstack-cinder | 15:38 | |
| *** yuriy_n17 has quit IRC | 15:41 | |
| *** hemna has joined #openstack-cinder | 15:43 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Implement service healthcheck tool https://review.openstack.org/195665 | 15:45 |
| mriedem | jgriffith: so i guess i'll need to bring this up in a cinder meeting next week huh https://review.openstack.org/#/c/193673/ | 15:53 |
| mriedem | given the 3rd party ci explosion | 15:53 |
| mriedem | i guess i could start with the mailing list as a heads up | 15:54 |
| mriedem | xyang1: ^ | 15:54 |
| *** mriedem is now known as mriedem_lunch | 15:54 | |
| xyang1 | mriedem: Hi | 15:54 |
| xyang1 | mriedem_lunch: Yes, we should discuss at team meeting | 15:55 |
| xyang1 | mriedem_lunch: Since this affects lots of CI's | 15:55 |
| *** amoturi has quit IRC | 15:55 | |
| xyang1 | mriedem_lunch: start with Mailing list is good idea | 15:56 |
| smcginnis | +1 | 15:56 |
| *** Yogi1 has quit IRC | 15:56 | |
| jgriffith | mriedem_lunch: heads up on ML seems appropriate to me | 15:57 |
| *** vilobhmm has quit IRC | 15:58 | |
| *** markstur has quit IRC | 15:58 | |
| xyang1 | jgriffith: Did you do anything special in your driver to support this, other than setting the key? | 15:59 |
| hemna | doesn't the current patch set the key in the manager? | 15:59 |
| hemna | regardless of the driver | 15:59 |
| xyang1 | hemna: Yes | 15:59 |
| tbarron | ping eharney | 15:59 |
| jgriffith | hemna: yes, which is how it should've been in the first place | 16:00 |
| xyang1 | hemna: But ci still fails for some | 16:00 |
| *** markstur has joined #openstack-cinder | 16:00 | |
| hemna | jgriffith, +1 | 16:00 |
| jgriffith | xyang1: no, I didn't need to do anything. But I do need to look closer at it again | 16:00 |
| jgriffith | xyang1: in theory drivers shouldn't "matter" for this | 16:00 |
| xyang1 | hemna: So I don't know what else needs to be fixed in the driver | 16:00 |
| tbarron | eharney: w.r.t. https://review.openstack.org/195695, here's what I'm trying to say: http://paste.openstack.org/show/321391/ | 16:00 |
| jgriffith | xyang1: it's all done on hypervisor layer etc | 16:00 |
| xyang1 | jgriffith: so that is why I don't understand why it failed netapp ci on this patch | 16:01 |
| jgriffith | xyang1: but "stuffs on fire yo" | 16:01 |
| xyang1 | jgriffith: It should work | 16:01 |
| eharney | tbarron: humm.... what version of python is that? | 16:01 |
| tbarron | mriedem_lunch: the patch looks good and I am certainly not proposing that we not do it, we just wanted to know if we can skip | 16:02 |
| tbarron | and what the plan was for removing the skip eventually | 16:02 |
| xyang1 | jgriffith: If it is taken care of by hypervisor | 16:02 |
| tbarron | eharney: 2.7.6 | 16:02 |
| xyang1 | jgriffith: And this patch sets the key for everyone | 16:02 |
| jgriffith | tbarron: might be good to explain why it breaks? | 16:02 |
| jgriffith | tbarron: the only error I see in your cDOT driver is: dpkg-query | 16:03 |
| tbarron | jgriffith: :-) that's expected | 16:03 |
| *** harlowja_at_home has joined #openstack-cinder | 16:03 | |
| eharney | tbarron: that even works on my 2.6.6 install... | 16:04 |
| jgriffith | tbarron: my point is that you fail, but there's no useful info in the logs | 16:04 |
| jgriffith | tbarron: ie no way to know "why" :) | 16:04 |
| tbarron | jgriffith: good point, that's why I haven't been able to figure it out myself yet | 16:04 |
| jgriffith | tbarron: same with the other systems FWIW | 16:04 |
| jgriffith | tbarron: I *could* spend my morning debuggin everybody elses stuff, but I'd rather not :) | 16:05 |
| tbarron | jgriffith: feel free to do so, but we didn't ask you to :-) | 16:05 |
| jgriffith | tbarron: LOL :) | 16:05 |
| *** ronis has joined #openstack-cinder | 16:05 | |
| jgriffith | tbarron: all I know is it breaks attach... can't see why though :( | 16:06 |
| tbarron | jgriffith: my plan was to start learning about encrypted volumes, which weren't even on my horizon (no pun) before | 16:06 |
| Swanson | jgriffith: you could be a geek super hero. D'Bugger! Coding by day, debugging by night! | 16:06 |
| jgriffith | Swanson: no thanks.. been there done that. Doesn't pay well :) | 16:06 |
| Swanson | jgriffith: or at all | 16:06 |
| tbarron | jgriffith: me either, looks like the LUKS commands don't like the blocks we present to it though | 16:06 |
| tbarron | jgriffith: I dunno why | 16:06 |
| hemna | xyang1, yah that's what I'm curious about, but then why is ceph failing then | 16:06 |
| *** leeantho has joined #openstack-cinder | 16:06 | |
| jgriffith | of course the f'ing spewing of errors in the nova logs doesn't help | 16:07 |
| tbarron | eharney: so that failure I see may be spurious, if it doesn't show up in CI, I won't worry about it | 16:07 |
| xyang1 | hemna: I Did you anything else other than setting the key in your driver? | 16:07 |
| jgriffith | oh... interesting | 16:07 |
| tbarron | eharney: maybe "wide" vs "narrow" python build? | 16:07 |
| eharney | tbarron: well, it's pretty weird. makes me wonder if something is wrong w/ your version of python on mac... | 16:07 |
| hemna | xyang1, I don't think we did anything in our driver for encryption | 16:08 |
| tbarron | eharney: mebbe so | 16:08 |
| jgriffith | tbarron: xyang1 ok, so here's a clue maybe | 16:08 |
| *** rushiagr_away is now known as rushiagr | 16:09 | |
| tbarron | eharney: that code ran just fine on ubuntu trusty with python 2.7.6 | 16:10 |
| tbarron | just checked now | 16:10 |
| tbarron | eharney: so I'll write that up to mac anomaly | 16:11 |
| *** DericHorn-HP has joined #openstack-cinder | 16:11 | |
| eharney | tbarron: can you try replacing \uffff with \U00001000 ? | 16:11 |
| tbarron | eharney: and it looks ok on ubuntu trusty with 3.4 | 16:12 |
| eharney | tbarron: wait... that's not right | 16:12 |
| eharney | tbarron: \U00010000 | 16:12 |
| eharney | tbarron: or maybe \U0000FFFF | 16:13 |
| tbarron | eharney: yeah, I already tried that and it had the same error, hence my "too wide" theory, but let me try those you just posted again to be sure | 16:13 |
| *** e0ne is now known as e0ne_ | 16:14 | |
| eharney | tbarron: i suspect it's https://bugs.python.org/issue12749 which includes a test case to see if that's the issue | 16:14 |
| *** e0ne_ has quit IRC | 16:15 | |
| *** dannywilson has joined #openstack-cinder | 16:16 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 16:16 |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: RPC and object versioning https://review.openstack.org/184404 | 16:17 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 16:18 |
| *** rmesta has quit IRC | 16:18 | |
| *** rmesta has joined #openstack-cinder | 16:18 | |
| tbarron | eharney: http://paste.openstack.org/show/321399/ | 16:19 |
| tbarron | eharney: looking at https://bugs.python.org/issue12749 now | 16:19 |
| *** Apoorva has joined #openstack-cinder | 16:23 | |
| tbarron | eharney: 'if re.search("[𝒜-𝒵]", "𝒞", re.UNICODE):' (from the python bug you cite) works on the mac 2.7.6 | 16:23 |
| eharney | tbarron: well... seems like i need to figure out something so that i don't have a horde of mac users showing up saying i broke everything :/ | 16:24 |
| *** harlowja_at_home has quit IRC | 16:24 | |
| *** haypo has left #openstack-cinder | 16:24 | |
| tbarron | eharney: they only reason I have a mac was that when they refreshed my old stinkpad running (cough) debian, I couldn't get a new one from our corp supplier with as much ram & ssd as this powerbook | 16:25 |
| tbarron | eharney: and now I've grown accustomed to running unit tests on it | 16:25 |
| tbarron | eharney: which to my pleasant surprise were running just fine | 16:26 |
| tbarron | eharney: but I deploy to a running devstack on an ubuntu trusty vm everytime I make a code change anyways | 16:26 |
| *** bill_az has joined #openstack-cinder | 16:26 | |
| tbarron | eharney: so it wouldn't be a big burden for me to just run UT there as well | 16:27 |
| *** tellesnobrega has quit IRC | 16:28 | |
| eharney | tbarron: unfortunately without understanding the cause, it's hard to know where else it will be broken too | 16:28 |
| *** merooney has joined #openstack-cinder | 16:29 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 16:33 |
| akerr | jgriffith: just saw the conversation about the patch. The errors appear in the nova log for some reason. They try to run a mount command during attach but the device they use is "already in use." No idea why though :( | 16:33 |
| *** tellesnobrega has joined #openstack-cinder | 16:35 | |
| jgriffith | akerr: indeed | 16:35 |
| jgriffith | akerr: only thing I'm wondering is if part of the Luks formatting is hanging up or puking on the device | 16:36 |
| jgriffith | akerr: also quick look trying to coorelate what backends are impacted... looks like all Netapp and HDS have similar signature | 16:36 |
| jgriffith | akerr: number of the other failures appear to be different, but didn't dig real deep into the logs | 16:37 |
| jgriffith | was thinking the other thing is Netapp and HDS have some "out of tree" initialize connection stuff... but that doesn't seem too relevant | 16:37 |
| jgriffith | akerr: anywho... it would be really cool to have a bug and know what someobody from Netapp is actively working and trying to resolve it | 16:38 |
| *** Yogi1 has joined #openstack-cinder | 16:38 | |
| akerr | jgriffith: I think tbarron is looking into it, but I'm not sure we filed a bug yet on LP | 16:38 |
| *** _cjones_ has joined #openstack-cinder | 16:39 | |
| *** dannywilson has quit IRC | 16:41 | |
| *** ronis has quit IRC | 16:41 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 16:42 |
| *** e0ne has joined #openstack-cinder | 16:43 | |
| *** e0ne has quit IRC | 16:45 | |
| *** dimsum__ has joined #openstack-cinder | 16:53 | |
| *** jungleboyj has joined #openstack-cinder | 16:57 | |
| *** laughterwym has quit IRC | 16:57 | |
| jgriffith | mriedem_lunch: akerr tbarron something that might be worth looking at | 16:58 |
| jgriffith | mriedem_lunch: akerr tbarron on the nova side we do a connect, encryptor.attach then the guest attach dev | 16:59 |
| jgriffith | mriedem_lunch: akerr tbarron given the notes on the detach side about order and "still in use" I'm wondering if that's something Netapp and others should be focusing on | 16:59 |
| *** markus_z has quit IRC | 16:59 | |
| jgriffith | ie you don't do multi-attach support or something along those lines | 16:59 |
| jgriffith | or if that sequence is just "not right" | 17:00 |
| *** setmason has joined #openstack-cinder | 17:01 | |
| *** Longgeek has quit IRC | 17:02 | |
| *** mriedem_lunch is now known as mriedem | 17:02 | |
| mriedem | jgriffith: xyang1: tbarron: eharney: the problem on the nova side is the provider code (the luks or cryptsetup modules in nova) expects the 'device_path' key to be set in the connection_info from the libvirt volume driver being used | 17:03 |
| *** tellesnobrega has quit IRC | 17:03 | |
| *** annashen has joined #openstack-cinder | 17:03 | |
| mriedem | that isn't set for some of these other volume drivers so they get a KeyError in the encryption provider in nova | 17:03 |
| mriedem | see the related nova change here https://review.openstack.org/#/c/193830/ | 17:03 |
| mriedem | to turn that KeyError into something not as dumb | 17:03 |
| *** patrickeast has joined #openstack-cinder | 17:04 | |
| *** eharney has quit IRC | 17:05 | |
| mriedem | i'll send something to the ML | 17:05 |
| xyang1 | mriedem: Device_path key is set for iscsi and fc drivers, so they should work then | 17:05 |
| xyang1 | hemna: around? | 17:06 |
| *** bswartz has quit IRC | 17:07 | |
| mriedem | xyang1: yes that's true | 17:07 |
| mriedem | xyang1: that's why the encrypted volumes test is fine in the integrated gate since it's using lvm | 17:07 |
| xyang1 | mriedem: Then the failure in NetApp CI is something else | 17:07 |
| mriedem | the failure in the ceph job was something else too | 17:08 |
| mriedem | that's why i rechecked it | 17:08 |
| xyang1 | mriedem: Their iscsi test failed | 17:08 |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Use right oslo.service entry points https://review.openstack.org/196099 | 17:08 |
| *** ronis has joined #openstack-cinder | 17:09 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix 'no actual-pathname' API error https://review.openstack.org/196148 | 17:11 |
| *** ameade is now known as ameade_vacation | 17:16 | |
| openstackgerrit | Dmitry Guryanov proposed openstack/cinder: smbfs: fix invalid check for smbfs_used_ratio correctness https://review.openstack.org/190326 | 17:20 |
| hemna | xyang1, yah | 17:20 |
| *** eharney has joined #openstack-cinder | 17:21 | |
| xyang1 | hemna: Did you see mriedem's comments about device_path | 17:21 |
| hemna | no | 17:21 |
| xyang1 | hemna: So FC and iSCSI drivers should work | 17:21 |
| xyang1 | hemna: It is about encryption | 17:22 |
| hemna | xyang1, which patch are we talking about ? | 17:22 |
| xyang1 | hemna: On nova side, it requires device_path | 17:22 |
| *** e0ne has joined #openstack-cinder | 17:22 | |
| xyang1 | https://review.openstack.org/#/c/193673/ | 17:23 |
| xyang1 | hemna: mriedem's patch | 17:23 |
| hemna | ok | 17:24 |
| hemna | so the RBD libvirt volume driver isn't setting device_path ? | 17:24 |
| xyang1 | hemna: sounds like it | 17:24 |
| mriedem | hemna: nope | 17:24 |
| *** dannywilson has joined #openstack-cinder | 17:24 | |
| mriedem | hemna: jgriffith: xyang1: tbarron: eharney: here is the long form http://lists.openstack.org/pipermail/openstack-dev/2015-June/068117.html | 17:25 |
| mriedem | brb | 17:25 |
| *** eharney has quit IRC | 17:25 | |
| *** EmilienM is now known as EmilienM|brb | 17:25 | |
| *** eharney has joined #openstack-cinder | 17:26 | |
| hemna | mriedem, yah the RBD driver in libvirt doesn't even do connect_volume | 17:27 |
| hemna | it's simply a call to get_config | 17:27 |
| hemna | hrmm | 17:27 |
| hemna | I'm not an RBD guy, so I have no idea how it works. | 17:27 |
| hemna | I thought every libvirt volume driver's job was to return the device_path | 17:28 |
| hemna | for connect_volume | 17:28 |
| mriedem | there isn't actually an rbd volume driver in nova, it's in the network volume driver | 17:28 |
| hemna | mriedem, yah | 17:28 |
| hemna | I was just looking through that stuff again | 17:28 |
| hemna | I've been hacking that file up quite a bit lately with my os-brick patch | 17:29 |
| mriedem | http://git.openstack.org/cgit/openstack/nova/tree/nova/virt/libvirt/volume.py#n242 | 17:29 |
| hemna | but left alone the net driver | 17:29 |
| mriedem | the connect happens in hte base class | 17:29 |
| mriedem | which is a pass :) | 17:29 |
| *** MentalRay has quit IRC | 17:29 | |
| *** bswartz has joined #openstack-cinder | 17:30 | |
| hemna | yup | 17:30 |
| hemna | no idea how RBD volumes get into the VM's then if there is not device path set | 17:30 |
| *** timcl has joined #openstack-cinder | 17:30 | |
| hemna | maybe it's a native virt supported protocol ? | 17:30 |
| *** annegentle has joined #openstack-cinder | 17:31 | |
| hemna | that libvirt itself knows how to connect to it. | 17:31 |
| hemna | hence no need for the device path | 17:31 |
| hemna | in that case, then libvirt itself would need encrypted volume support in order to do encrypted RBD volumes | 17:31 |
| *** MentalRay has joined #openstack-cinder | 17:31 | |
| mriedem | it's not just rbd, i also don't see it for quobyte and some others | 17:32 |
| hemna | https://libvirt.org/storage.html#StorageBackendRBD | 17:32 |
| hemna | yup | 17:32 |
| hemna | that's why the get_config for the net driver is setting those values and not a device_path | 17:32 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Cinder API WSGI application under Apache/Nginx https://review.openstack.org/192683 | 17:34 |
| *** shyama has joined #openstack-cinder | 17:34 | |
| hemna | mriedem, I think the get_config().source_path has it though | 17:34 |
| hemna | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume.py#L1536 | 17:34 |
| *** annegentle has quit IRC | 17:36 | |
| *** amoturi has joined #openstack-cinder | 17:39 | |
| mriedem | hemna: for quobyte you mean, yeah, just not set during connect_volume though | 17:39 |
| hemna | yah | 17:39 |
| xyang1 | mriedem: I have a different question for you. What should I do about the scaleio libvirt volume driver spec? I saw an email about nova spec freeze today. Not sure if I need to move it somewhere | 17:39 |
| e0ne | thingee: hi. i'm going to post update about removing api v2 to openstack-dev. | 17:39 |
| e0ne | thingee: do you have any wishes or objections? | 17:40 |
| mriedem | xyang1: i think libvirt volume drivers in nova are going to depend on os-brick | 17:40 |
| mriedem | xyang1: so we need to get hemna's os-brick integration patch reviewed and in | 17:40 |
| mriedem | i haven't reviewed it yet | 17:40 |
| xyang1 | mriedem: Ok, so I don't need to worry about that spec then | 17:41 |
| *** ndipanov has quit IRC | 17:42 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Improve 3PAR driver VLUN creation and deletion https://review.openstack.org/196164 | 17:44 |
| mriedem | xyang1: i don't think so. assuming your scaleio thing gets into os-brick and os-brick gets into nova, then the only changes needed in nova i think are the volume driver registration in the libvirt driver.py, right? | 17:44 |
| mriedem | xyang1: this part: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L271 | 17:45 |
| xyang1 | mriedem: It still needs a shim libvirt volume driver: https://review.openstack.org/#/c/194454/ | 17:46 |
| xyang1 | mriedem: My patch is dependent on hemna 's os-brick patch | 17:46 |
| mriedem | xyang1: where is your os-brick change? | 17:47 |
| mriedem | https://review.openstack.org/#/c/185835/ | 17:47 |
| xyang1 | Yes | 17:48 |
| mriedem | xyang1: so i'm confused, when https://review.openstack.org/#/c/185835/5/os_brick/initiator/connector.py gets into os-brick, does the shim libvirt volume driver still need the connect/disconnect_volume and get_config methods? | 17:48 |
| mriedem | well, for like iscsi in hemna's change https://review.openstack.org/#/c/175569/19/nova/virt/libvirt/volume.py | 17:49 |
| mriedem | the connect/disconnect calls just pass through to os-brick | 17:49 |
| hemna | yah, they should | 17:49 |
| hemna | and populate anything in connection_info after the brick call is made | 17:49 |
| mriedem | ok | 17:49 |
| mriedem | get_config() remains untouched though? | 17:50 |
| hemna | yah most likely. if everything is done in connect_volume, then get_config should have everything it needs at that point. | 17:50 |
| *** dannywilson has quit IRC | 17:50 | |
| *** dannywilson has joined #openstack-cinder | 17:51 | |
| mriedem | hemna: ok, i thought the libvirt volume driver would just call os-brick for get_config too | 17:51 |
| mriedem | but i honestly don't know how all of it works either | 17:51 |
| mriedem | since for example there is a get_config in os-brick for the scaleio connector https://review.openstack.org/#/c/185835/5/os_brick/initiator/connector.py | 17:52 |
| mriedem | oh well, they aren't really the same. and in nova we could probably abstract some of the get_config in a base class since most of it is copy/paste in those drivers | 17:54 |
| mriedem | set the source_type and source_path, we could just delegate that to the subclasses to return their type and path given connection_info, let the base class set the config values | 17:54 |
| *** e0ne is now known as e0ne_ | 17:55 | |
| *** rakesh_mishra has quit IRC | 17:57 | |
| hemna | mriedem, the config stuff is specific to nova | 17:57 |
| hemna | so brick connectors don't have a get_config. they just do the connect_volume and disconnect_volume work | 17:58 |
| hemna | and the libvirt drivers will build the config out of the connection_info returned from the connect_volume calls | 17:58 |
| *** mriedem has quit IRC | 17:59 | |
| hemna | I'm all for refactoring the nova libvirt volume drivers, but we have to do step 1 first :) | 17:59 |
| *** mriedem has joined #openstack-cinder | 18:01 | |
| *** Longgeek has joined #openstack-cinder | 18:02 | |
| *** gouthamr has quit IRC | 18:03 | |
| *** EmilienM|brb is now known as EmilienM | 18:04 | |
| *** akerr has quit IRC | 18:04 | |
| *** rushiagr is now known as rushiagr_away | 18:04 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix 'no actual-pathname' NetApp API error https://review.openstack.org/196148 | 18:07 |
| *** annashen has quit IRC | 18:08 | |
| *** Longgeek has quit IRC | 18:08 | |
| *** annashen has joined #openstack-cinder | 18:08 | |
| *** e0ne_ is now known as e0ne | 18:10 | |
| *** DericHorn-HP has quit IRC | 18:12 | |
| *** DericHorn-HP has joined #openstack-cinder | 18:13 | |
| *** rushiagr_away is now known as rushiagr | 18:17 | |
| *** DericHorn-HP has quit IRC | 18:17 | |
| *** sjcherry has joined #openstack-cinder | 18:17 | |
| *** DericHorn-HP has joined #openstack-cinder | 18:18 | |
| *** arif-ali has quit IRC | 18:31 | |
| *** annegentle has joined #openstack-cinder | 18:32 | |
| *** Yogi1 has quit IRC | 18:34 | |
| *** arif-ali has joined #openstack-cinder | 18:36 | |
| *** annegentle has quit IRC | 18:37 | |
| *** arif-ali has quit IRC | 18:37 | |
| *** arif-ali has joined #openstack-cinder | 18:38 | |
| *** dustins has quit IRC | 18:38 | |
| *** rushiagr is now known as rushiagr_away | 18:39 | |
| *** e0ne has quit IRC | 18:42 | |
| *** annegentle has joined #openstack-cinder | 18:51 | |
| *** e0ne has joined #openstack-cinder | 18:54 | |
| *** rushiagr_away is now known as rushiagr | 18:56 | |
| tbarron | mriedem: thanks for writing that up! I had to leave my keyboard for a while and am just now catching up with the backlog here. | 19:03 |
| *** Ctina__ has joined #openstack-cinder | 19:03 | |
| *** merooney has quit IRC | 19:03 | |
| *** scottda has quit IRC | 19:03 | |
| mriedem | tbarron: you didn't miss much | 19:03 |
| tbarron | I think it will well repay my efforts to learn what is going on here | 19:03 |
| tbarron | mriedem: well, I had to leave suddenly (realized I had forgotten all about lunch meeting) just as jgriffith was offering his suggestions | 19:04 |
| tbarron | then got back to the office and had to take care of stuff with folks there for a bit | 19:05 |
| xyang1 | tbarron: I don't know why your iscsi driver failed because it should have device_path | 19:05 |
| *** Ctina_ has quit IRC | 19:06 | |
| *** Ctina__ has quit IRC | 19:07 | |
| *** harlowja has quit IRC | 19:09 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Remove oslo logging from backup unit tests https://review.openstack.org/191233 | 19:11 |
| *** Lee1092 has quit IRC | 19:15 | |
| *** r-daneel has quit IRC | 19:17 | |
| *** primechuck has joined #openstack-cinder | 19:18 | |
| *** rushiagr is now known as rushiagr_away | 19:19 | |
| *** markvoelker has quit IRC | 19:20 | |
| *** annegentle has quit IRC | 19:20 | |
| *** harlowja has joined #openstack-cinder | 19:22 | |
| *** rmesta has left #openstack-cinder | 19:26 | |
| *** haypo has joined #openstack-cinder | 19:26 | |
| patrickeast | jgriffith: hey, back again with a question about virtual size, for an image like http://paste.openstack.org/show/321426/ is virtual size supposed to be in GB rounded up to an int? or a float number? | 19:26 |
| *** alexpilotti has quit IRC | 19:26 | |
| *** markvoelker has joined #openstack-cinder | 19:26 | |
| patrickeast | jgriffith: i’m assuming an in int based on what you’ve got here https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/solidfire.py#L554 but i wanted to double check | 19:27 |
| haypo | smcginnis: hi. you approved two of my python 3 patches on backup cinder, but they depend on https://review.openstack.org/#/c/194587/ would you like to review it? IMO it's even simpler than the two other patches | 19:27 |
| haypo | (well, smcginnis or another core reviewer ;-)) | 19:28 |
| jgriffith | patrickeast: I round everything up to int/GiB | 19:28 |
| jgriffith | patrickeast: it doesn't make much sense not to given the resolution of volumes in Cinder is GiB | 19:28 |
| patrickeast | jgriffith: ah yea, that makes sense | 19:28 |
| patrickeast | jgriffith: ok cool | 19:28 |
| patrickeast | jgriffith: ill do the same then | 19:28 |
| patrickeast | thanks! | 19:29 |
| jgriffith | patrickeast: cool | 19:29 |
| haypo | smcginnis: (anything, thanks for the review you already did ;-)) | 19:29 |
| tbarron | xyang1: right, that's a good question, maybe the best place to start investigating. | 19:30 |
| *** timcl has left #openstack-cinder | 19:32 | |
| *** markvoelker has quit IRC | 19:32 | |
| *** markvoelker has joined #openstack-cinder | 19:32 | |
| *** e0ne has quit IRC | 19:36 | |
| *** markvoelker_ has joined #openstack-cinder | 19:38 | |
| *** markvoelker has quit IRC | 19:40 | |
| *** shyama has quit IRC | 19:41 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Port NetApp DATAONTAP blocks drivers to ABC model https://review.openstack.org/196202 | 19:42 |
| *** markvoelker has joined #openstack-cinder | 19:44 | |
| *** markvoelker_ has quit IRC | 19:45 | |
| *** skylerberg has joined #openstack-cinder | 19:45 | |
| mtreinish | jgriffith: btw, just a heads up you might want to make sure you're running the latest subunit2sql release, the last oslo.db release broke things so I had to add an exclude to requirements | 19:45 |
| *** sjcherry has quit IRC | 19:45 | |
| mtreinish | err, well I guess it's only an issue if you ever reinstall subunit2sql or it's deps in your env | 19:46 |
| mtreinish | which I don't think you would from what I remember of the sos code | 19:46 |
| mtreinish | so nm | 19:46 |
| jgriffith | mtreinish: :) | 19:46 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Improve 3PAR driver VLUN creation and deletion https://review.openstack.org/196164 | 19:47 |
| jgriffith | mtreinish: so the good think is my Docker file just pulls latest for everything anyway if/when I rebuild | 19:47 |
| jgriffith | mtreinish: which is on my list of things to do next week as a matter of fact | 19:47 |
| jgriffith | mtreinish: I'm trying an experiment with isolating the services and scaling out the workers as seperate containers | 19:48 |
| *** daneyon has joined #openstack-cinder | 19:50 | |
| mtreinish | jgriffith: heh, ok cool | 19:51 |
| mtreinish | I just wanted to make sure it didn't break you, because it killed yesterday's nodepool image build | 19:51 |
| *** MentalRay has quit IRC | 19:52 | |
| jgriffith | mtreinish: thanks... and yeah, forget sometimes I'm the only one that cares about the nonsense I try and blabber about :) | 19:52 |
| *** markvoelker has quit IRC | 19:54 | |
| mtreinish | jgriffith: heh, nah trying to run the workers isolated in different containers is an interesting idea | 19:54 |
| mtreinish | just doesn't apply much to what I would ever be doing :) | 19:54 |
| openstackgerrit | Patrick East proposed openstack/cinder: WIP generic image cache https://review.openstack.org/195795 | 19:55 |
| patrickeast | jgriffith: ^ that version doesn’t do the silly recursion thing and actually uses the virtual size | 19:56 |
| jgriffith | patrickeast: sweeeet! | 19:57 |
| jgriffith | patrickeast: it's in my queue | 19:57 |
| jgriffith | patrickeast: I promise feedback on Monday so don't let me forget :) | 19:57 |
| patrickeast | jgriffith: thanks! | 19:58 |
| *** markvoelker has joined #openstack-cinder | 19:58 | |
| jgriffith | patrickeast: today's all about Puppet and keeping myself from jumping out the window because I dislike it soooo much | 19:58 |
| patrickeast | haha | 19:58 |
| patrickeast | jgriffith: i understand just enough about puppet to really like the concept, and enough to really hate trying to change/write modules | 19:59 |
| jgriffith | patrickeast: the writing modules is "ok" IMHO, but the shit it does constantly going back and modiyfing/resetting stuff just pisses me off :) | 19:59 |
| jgriffith | patrickeast: I prefer run-once and get on with life :) | 20:00 |
| jgriffith | even though I know that's not what most want | 20:00 |
| patrickeast | jgriffith: yea its nice sometimes like when i know something is wrong but I don’t know what it is, i can just re-apply the puppet stuff and not worry about it | 20:01 |
| patrickeast | but if there is one specific thing i want it to do its annoying to wait | 20:02 |
| jgriffith | :) | 20:02 |
| *** e0ne has joined #openstack-cinder | 20:03 | |
| *** zhenguo has quit IRC | 20:03 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Port NetApp E-Series iSCSI driver to ABC model https://review.openstack.org/196225 | 20:03 |
| *** Longgeek has joined #openstack-cinder | 20:04 | |
| mriedem | tbarron: jgriffith: xyang1: as promised https://wiki.openstack.org/wiki/CinderMeetings#Next_meeting | 20:06 |
| xyang1 | mriedem: Sounds good | 20:07 |
| tbarron | mriedem: excellent | 20:07 |
| *** Longgeek has quit IRC | 20:09 | |
| *** markvoelker_ has joined #openstack-cinder | 20:09 | |
| *** markvoelker has quit IRC | 20:11 | |
| *** markvoelker_ has quit IRC | 20:12 | |
| *** pots1 has quit IRC | 20:15 | |
| *** MentalRay has joined #openstack-cinder | 20:16 | |
| *** krtaylor has quit IRC | 20:21 | |
| *** e0ne has quit IRC | 20:24 | |
| *** hemna has quit IRC | 20:25 | |
| skylerberg | Hi tbarron: I wanted to check in on https://review.openstack.org/#/c/193673/. Right now it causes the Tintri-ci (which I am working on) to go into and inconsistent state. For now we are going to not respond to changes on this change to avoid false failures on subsequent changes. | 20:25 |
| skylerberg | Do you think these tests will be skipped if this patch goes in or will this require changes to our CI or driver? | 20:26 |
| *** amoturi has quit IRC | 20:27 | |
| *** ronis has quit IRC | 20:34 | |
| *** rmesta has joined #openstack-cinder | 20:37 | |
| *** annegentle has joined #openstack-cinder | 20:37 | |
| *** e0ne has joined #openstack-cinder | 20:39 | |
| *** e0ne has quit IRC | 20:44 | |
| *** daneyon_ has joined #openstack-cinder | 20:46 | |
| *** daneyon has quit IRC | 20:49 | |
| *** jecarey has quit IRC | 20:50 | |
| *** xyang1 has quit IRC | 20:50 | |
| *** markvoelker has joined #openstack-cinder | 20:51 | |
| *** sgotliv has joined #openstack-cinder | 20:52 | |
| *** skylerberg has quit IRC | 20:55 | |
| *** thangp has quit IRC | 20:55 | |
| *** jwcroppe_ has joined #openstack-cinder | 20:58 | |
| *** jwcroppe has quit IRC | 21:00 | |
| *** jwcroppe_ has quit IRC | 21:02 | |
| *** jwcroppe has joined #openstack-cinder | 21:02 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Sync the latest fileutils module from oslo-incubator https://review.openstack.org/196245 | 21:04 |
| *** krtaylor has joined #openstack-cinder | 21:05 | |
| *** jwcroppe_ has joined #openstack-cinder | 21:05 | |
| *** skylerberg has joined #openstack-cinder | 21:06 | |
| *** jwcroppe has quit IRC | 21:06 | |
| *** lpetrut has quit IRC | 21:09 | |
| *** kjnelson has left #openstack-cinder | 21:12 | |
| *** primechuck has quit IRC | 21:18 | |
| *** dannywilson has quit IRC | 21:19 | |
| *** markvoelker has quit IRC | 21:19 | |
| *** Longgeek has joined #openstack-cinder | 21:22 | |
| *** harlowja has quit IRC | 21:23 | |
| *** openstack has joined #openstack-cinder | 21:25 | |
| tbarron | skylerberg: I think we're all trying to figure this out and will sync up at the next weekly cinder meeting | 21:25 |
| jgriffith | mriedem: thanks for putting that on the agenda | 21:26 |
| tbarron | https://wiki.openstack.org/wiki/CinderMeetings#Next_meeting | 21:26 |
| *** Longgeek has quit IRC | 21:26 | |
| tbarron | I know I'm going to be trying to figure what's going on with our driver in the mean time, lot's to learn. | 21:26 |
| tbarron | mriedem's patch is doing the right thing, we just need to figure out collectively how to adapt | 21:27 |
| skylerberg | tbarron: Thanks. I will be keeping an eye on what gets decided. | 21:27 |
| tbarron | most of our drivers weren't (just my opinion) that aware of the possibility of encrypted volumes | 21:27 |
| tbarron | so this is a good thing | 21:28 |
| *** markvoelker has joined #openstack-cinder | 21:28 | |
| *** jungleboyj has quit IRC | 21:30 | |
| *** markvoelker has quit IRC | 21:30 | |
| *** IlyaG has joined #openstack-cinder | 21:31 | |
| *** earlephilhower has joined #openstack-cinder | 21:34 | |
| *** DericHorn-HP has quit IRC | 21:34 | |
| *** harlowja has joined #openstack-cinder | 21:35 | |
| *** hemna has joined #openstack-cinder | 21:38 | |
| *** annegentle has quit IRC | 21:39 | |
| *** annegentle has joined #openstack-cinder | 21:40 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/195105 | 21:48 |
| *** rwsu has quit IRC | 21:56 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/193996 | 22:00 |
| *** hemna_ has joined #openstack-cinder | 22:01 | |
| *** MentalRay has quit IRC | 22:02 | |
| *** hemna has quit IRC | 22:02 | |
| *** patrickeast has quit IRC | 22:03 | |
| *** patrickeast has joined #openstack-cinder | 22:04 | |
| *** haypo has quit IRC | 22:05 | |
| *** edmondsw has quit IRC | 22:08 | |
| *** edmondsw has joined #openstack-cinder | 22:08 | |
| *** edmondsw has quit IRC | 22:08 | |
| *** earlephilhower has quit IRC | 22:16 | |
| *** setmason has quit IRC | 22:17 | |
| *** DericHorn-HP has joined #openstack-cinder | 22:18 | |
| *** DericHorn-HP has quit IRC | 22:19 | |
| *** annashen has quit IRC | 22:22 | |
| *** annashen has joined #openstack-cinder | 22:22 | |
| *** IlyaG has quit IRC | 22:30 | |
| *** annashen has quit IRC | 22:39 | |
| *** DericHorn-HP has joined #openstack-cinder | 22:40 | |
| *** annegentle has quit IRC | 22:41 | |
| *** annashen has joined #openstack-cinder | 22:50 | |
| *** annegentle has joined #openstack-cinder | 23:01 | |
| *** lcurtis has quit IRC | 23:12 | |
| *** rmesta has quit IRC | 23:13 | |
| *** rmesta has joined #openstack-cinder | 23:13 | |
| *** rmesta has quit IRC | 23:14 | |
| *** annashen has quit IRC | 23:14 | |
| *** rmesta has joined #openstack-cinder | 23:14 | |
| *** rmesta has quit IRC | 23:14 | |
| *** ekarlso has quit IRC | 23:18 | |
| *** ekarlso has joined #openstack-cinder | 23:18 | |
| *** annegentle has quit IRC | 23:19 | |
| *** Longgeek has joined #openstack-cinder | 23:22 | |
| *** annashen has joined #openstack-cinder | 23:26 | |
| *** _cjones_ has quit IRC | 23:27 | |
| *** DericHorn-HP has quit IRC | 23:27 | |
| *** annashen has quit IRC | 23:31 | |
| *** skylerberg has quit IRC | 23:33 | |
| *** angela-s has quit IRC | 23:36 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: update os-brick to pass python3 tests https://review.openstack.org/196284 | 23:43 |
| *** Longgeek has quit IRC | 23:44 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!