| *** jamielennox|away is now known as jamielennox | 00:02 | |
| *** wanghao has joined #openstack-cinder | 00:09 | |
| *** wanghao has quit IRC | 00:10 | |
| *** wanghao has joined #openstack-cinder | 00:10 | |
| *** markvoelker has quit IRC | 00:11 | |
| *** winston-d_ has joined #openstack-cinder | 00:20 | |
| *** mriedem has joined #openstack-cinder | 00:22 | |
| *** mriedem has quit IRC | 00:22 | |
| *** liverpooler has joined #openstack-cinder | 00:26 | |
| *** liverpooler has quit IRC | 00:31 | |
| *** Shunli has joined #openstack-cinder | 00:33 | |
| *** wanghao has quit IRC | 00:48 | |
| *** wanghao has joined #openstack-cinder | 00:49 | |
| *** wanghao has quit IRC | 00:49 | |
| *** wanghao has joined #openstack-cinder | 00:50 | |
| *** wanghao has quit IRC | 00:50 | |
| *** wanghao has joined #openstack-cinder | 00:50 | |
| *** wanghao has quit IRC | 00:51 | |
| *** wanghao has joined #openstack-cinder | 00:51 | |
| *** wanghao has quit IRC | 00:52 | |
| *** zhangguoqing has joined #openstack-cinder | 00:52 | |
| *** wanghao has joined #openstack-cinder | 00:52 | |
| *** wanghao has quit IRC | 00:52 | |
| *** wanghao has joined #openstack-cinder | 00:53 | |
| *** wanghao has quit IRC | 00:53 | |
| *** wanghao has joined #openstack-cinder | 00:54 | |
| *** wanghao has quit IRC | 00:54 | |
| *** links has joined #openstack-cinder | 00:54 | |
| *** kaisers has joined #openstack-cinder | 01:06 | |
| *** kaisers_ has quit IRC | 01:08 | |
| *** zengyingzhe has quit IRC | 01:17 | |
| *** cknight has joined #openstack-cinder | 01:19 | |
| *** zhonghua has quit IRC | 01:22 | |
| *** chenying_ has quit IRC | 01:22 | |
| *** zhaochao has joined #openstack-cinder | 01:35 | |
| *** cknight has quit IRC | 01:39 | |
| openstackgerrit | Merged openstack/cinder master: Rolling Upgrades: Fix VolumeAttachment https://review.openstack.org/466254 | 01:42 |
|---|---|---|
| openstackgerrit | yuyafei proposed openstack/cinder master: Remove "readonly=True" when creating volume from image cache https://review.openstack.org/466563 | 01:51 |
| openstackgerrit | Merged openstack/cinder master: Add 'resource_filters' API to support generalized filtering https://review.openstack.org/461668 | 01:54 |
| *** wanghao has joined #openstack-cinder | 01:55 | |
| openstackgerrit | yuyafei proposed openstack/cinder master: Remove "readonly=True" when creating volume from image cache https://review.openstack.org/466563 | 01:56 |
| *** wanghao has quit IRC | 02:00 | |
| *** jamielennox is now known as jamielennox|away | 02:00 | |
| *** esker has quit IRC | 02:00 | |
| *** sofie_ has joined #openstack-cinder | 02:01 | |
| *** chenying has joined #openstack-cinder | 02:02 | |
| *** zhonghua has joined #openstack-cinder | 02:02 | |
| *** gcb has joined #openstack-cinder | 02:04 | |
| *** johnhaan has joined #openstack-cinder | 02:10 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Return metadata in volume summary https://review.openstack.org/459143 | 02:39 |
| *** stakeda has joined #openstack-cinder | 02:54 | |
| *** salv-orlando has joined #openstack-cinder | 02:54 | |
| *** zhangguoqing has quit IRC | 02:54 | |
| *** zhangguoqing has joined #openstack-cinder | 03:00 | |
| *** zhangguoqing has quit IRC | 03:02 | |
| *** esker has joined #openstack-cinder | 03:08 | |
| *** markstur has joined #openstack-cinder | 03:09 | |
| *** markstur has quit IRC | 03:10 | |
| *** zhangguoqing has joined #openstack-cinder | 03:12 | |
| *** zhangguo_ has joined #openstack-cinder | 03:16 | |
| *** zhangguoqing has quit IRC | 03:17 | |
| *** zhangguo_ has quit IRC | 03:18 | |
| *** zhangguoqing has joined #openstack-cinder | 03:18 | |
| *** psachin has joined #openstack-cinder | 03:23 | |
| *** gkadam has joined #openstack-cinder | 03:28 | |
| *** zhangguoqing has quit IRC | 03:35 | |
| openstackgerrit | Merged openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/455988 | 03:35 |
| *** zhangguoqing has joined #openstack-cinder | 03:35 | |
| *** zhangguoqing has quit IRC | 03:40 | |
| openstackgerrit | Merged openstack/os-brick master: Return correct device path from Veritas connector https://review.openstack.org/466437 | 03:44 |
| *** lamt has joined #openstack-cinder | 03:46 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Add tenant_id admin filter to limits API https://review.openstack.org/443437 | 03:47 |
| *** esker has quit IRC | 03:47 | |
| *** Poornima has joined #openstack-cinder | 03:51 | |
| *** Cibo_ has joined #openstack-cinder | 03:53 | |
| *** jamielennox|away is now known as jamielennox | 03:53 | |
| openstackgerrit | Bin Zhou proposed openstack/cinder master: Insert the description of the change. https://review.openstack.org/466583 | 03:54 |
| *** salv-orlando has quit IRC | 03:58 | |
| openstackgerrit | Merged openstack/cinder master: Updated from global requirements https://review.openstack.org/466460 | 04:06 |
| *** arnewiebalck__ has joined #openstack-cinder | 04:10 | |
| openstackgerrit | Merged openstack/cinder master: INFINIDAT: add locking around initialize/terminate connection https://review.openstack.org/422277 | 04:19 |
| *** adisky_ has joined #openstack-cinder | 04:21 | |
| *** johnhaan has quit IRC | 04:28 | |
| *** kongwei has joined #openstack-cinder | 04:30 | |
| *** kongwei has quit IRC | 04:33 | |
| *** markstur has joined #openstack-cinder | 04:39 | |
| *** markstur has quit IRC | 04:44 | |
| *** wanghao has joined #openstack-cinder | 04:57 | |
| *** wanghao has quit IRC | 04:59 | |
| *** isaacb has joined #openstack-cinder | 04:59 | |
| *** markstur has joined #openstack-cinder | 04:59 | |
| *** wanghao has joined #openstack-cinder | 04:59 | |
| *** pgadiya has joined #openstack-cinder | 05:03 | |
| *** markstur has quit IRC | 05:04 | |
| *** Poornima has quit IRC | 05:05 | |
| *** pgadiya has quit IRC | 05:06 | |
| *** Poornima has joined #openstack-cinder | 05:10 | |
| *** wanghao_ has joined #openstack-cinder | 05:10 | |
| *** wanghao has quit IRC | 05:10 | |
| *** shausy has joined #openstack-cinder | 05:12 | |
| *** shausy has quit IRC | 05:12 | |
| *** wanghao_ has quit IRC | 05:13 | |
| *** wanghao has joined #openstack-cinder | 05:14 | |
| *** wanghao has quit IRC | 05:14 | |
| *** wanghao has joined #openstack-cinder | 05:16 | |
| *** lamt has quit IRC | 05:16 | |
| *** arnewiebalck__ has quit IRC | 05:16 | |
| *** salv-orlando has joined #openstack-cinder | 05:17 | |
| *** wanghao has quit IRC | 05:17 | |
| *** wanghao has joined #openstack-cinder | 05:18 | |
| *** wanghao_ has joined #openstack-cinder | 05:19 | |
| *** wanghao has quit IRC | 05:19 | |
| *** wanghao has joined #openstack-cinder | 05:24 | |
| *** wanghao_ has quit IRC | 05:24 | |
| *** wanghao_ has joined #openstack-cinder | 05:26 | |
| *** wanghao has quit IRC | 05:26 | |
| *** lkuchlan has joined #openstack-cinder | 05:32 | |
| *** wanghao has joined #openstack-cinder | 05:33 | |
| *** wanghao_ has quit IRC | 05:33 | |
| *** wanghao_ has joined #openstack-cinder | 05:38 | |
| *** isaacb has quit IRC | 05:41 | |
| *** wanghao has quit IRC | 05:42 | |
| *** reduxio has quit IRC | 05:47 | |
| *** reduxio has joined #openstack-cinder | 05:47 | |
| *** ChubYann has quit IRC | 05:49 | |
| *** sofie_ has quit IRC | 05:55 | |
| *** wanghao_ has quit IRC | 05:57 | |
| *** reduxio has quit IRC | 06:02 | |
| *** reduxio has joined #openstack-cinder | 06:03 | |
| openstackgerrit | liuyamin proposed openstack/cinder master: Add the missing parameters in the api-ref of ext-backups.inc https://review.openstack.org/450537 | 06:03 |
| *** wanghao has joined #openstack-cinder | 06:04 | |
| *** salv-orlando has quit IRC | 06:04 | |
| *** lamt has joined #openstack-cinder | 06:06 | |
| *** reduxio has quit IRC | 06:07 | |
| *** belmoreira has joined #openstack-cinder | 06:07 | |
| *** salv-orlando has joined #openstack-cinder | 06:07 | |
| *** Poornima has quit IRC | 06:08 | |
| *** bhagyashris has quit IRC | 06:18 | |
| *** bhagyashris_ has quit IRC | 06:18 | |
| *** rcernin has joined #openstack-cinder | 06:19 | |
| *** reduxio has joined #openstack-cinder | 06:20 | |
| *** lkuchlan has quit IRC | 06:22 | |
| *** lhx__ has joined #openstack-cinder | 06:25 | |
| *** isaacb has joined #openstack-cinder | 06:33 | |
| *** isaacb has quit IRC | 06:33 | |
| *** markstur has joined #openstack-cinder | 06:35 | |
| *** markstur has quit IRC | 06:40 | |
| *** lkuchlan has joined #openstack-cinder | 06:51 | |
| openstackgerrit | zheng yin proposed openstack/cinder master: fix bug-1403948 https://review.openstack.org/466607 | 06:53 |
| *** chenying has quit IRC | 06:53 | |
| *** chenying has joined #openstack-cinder | 06:54 | |
| *** wxy has joined #openstack-cinder | 06:55 | |
| *** salv-orlando has quit IRC | 07:01 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Support 'LIKE' operator to filter resource https://review.openstack.org/445440 | 07:02 |
| *** salv-orlando has joined #openstack-cinder | 07:03 | |
| *** markstur has joined #openstack-cinder | 07:05 | |
| reduxio | Hello, I've tried upgrading my devstack and CI to latest version, and running into trouble. When nodepool tries to create a node it fails and in horizon I can see an error instance saying 'No valid host was found. There are not enough hosts available.'. Could not find the reason in the logs. the n-sch log shows 'Starting with 0 host(s)' but i'm not sure why. clarkb offered that i need to perform cell discovery, but I don | 07:05 |
| reduxio | ' | 07:05 |
| reduxio | reduxio | 07:05 |
| reduxio | t think this is the issue as I have a single host (devstack) and it's connected to a cell. Also, manually firing up a VM through horizon works.. | 07:05 |
| *** salv-orlando has quit IRC | 07:08 | |
| *** markstur has quit IRC | 07:09 | |
| *** pcaruana has joined #openstack-cinder | 07:11 | |
| *** zsli_ has joined #openstack-cinder | 07:12 | |
| *** pcaruana has quit IRC | 07:12 | |
| *** pcaruana has joined #openstack-cinder | 07:13 | |
| *** rcernin has quit IRC | 07:14 | |
| *** rcernin has joined #openstack-cinder | 07:14 | |
| *** wanghao has quit IRC | 07:15 | |
| *** Shunli has quit IRC | 07:15 | |
| *** asselin_ has quit IRC | 07:16 | |
| *** lhx__ has quit IRC | 07:20 | |
| *** mlakat has joined #openstack-cinder | 07:23 | |
| *** wanghao has joined #openstack-cinder | 07:26 | |
| *** e0ne has joined #openstack-cinder | 07:28 | |
| openstackgerrit | Digvijay Ukirde proposed openstack/cinder master: Fix bugs of GPFSNFS & GPFSRemote Driver initialization failure https://review.openstack.org/465961 | 07:30 |
| *** markstur has joined #openstack-cinder | 07:34 | |
| *** lamt has quit IRC | 07:35 | |
| *** david-lyle has quit IRC | 07:36 | |
| *** markstur has quit IRC | 07:39 | |
| *** hoonetorg has quit IRC | 07:39 | |
| *** david-lyle has joined #openstack-cinder | 07:42 | |
| *** david-lyle has quit IRC | 07:46 | |
| *** lhx__ has joined #openstack-cinder | 07:49 | |
| *** _dpawlik is now known as danpawlik | 07:50 | |
| *** gouthamr has joined #openstack-cinder | 07:51 | |
| *** hoonetorg has joined #openstack-cinder | 07:54 | |
| *** Cibo_ has quit IRC | 07:55 | |
| openstackgerrit | Merged openstack/python-brick-cinderclient-ext master: Updated from global requirements https://review.openstack.org/464461 | 07:57 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:01 | |
| *** reduxio has quit IRC | 08:03 | |
| *** david-lyle has joined #openstack-cinder | 08:03 | |
| *** lhx__ has quit IRC | 08:06 | |
| *** lhx__ has joined #openstack-cinder | 08:07 | |
| *** wanghao has quit IRC | 08:07 | |
| *** reduxio has joined #openstack-cinder | 08:12 | |
| *** e0ne has quit IRC | 08:12 | |
| *** e0ne has joined #openstack-cinder | 08:14 | |
| *** gouthamr has quit IRC | 08:19 | |
| *** markstur has joined #openstack-cinder | 08:19 | |
| *** markstur has quit IRC | 08:23 | |
| *** lpetrut has joined #openstack-cinder | 08:31 | |
| *** wanghao has joined #openstack-cinder | 08:35 | |
| *** Cibo_ has joined #openstack-cinder | 08:36 | |
| *** markstur has joined #openstack-cinder | 08:38 | |
| *** markstur has quit IRC | 08:43 | |
| *** zsli__ has joined #openstack-cinder | 08:46 | |
| openstackgerrit | Pavel Gluschak (scsnow) proposed openstack/cinder master: vstorage: added vz:volume_format vendor property https://review.openstack.org/465091 | 08:48 |
| *** zsli_ has quit IRC | 08:49 | |
| openstackgerrit | Cheng Yong Zhao proposed openstack/cinder master: Add gmcv support in SVC driver https://review.openstack.org/461191 | 08:49 |
| openstackgerrit | Chhavi Agarwal proposed openstack/cinder master: Support iSCSI host when Fc volume is there https://review.openstack.org/461850 | 08:49 |
| *** chhavi has joined #openstack-cinder | 08:55 | |
| *** wanghao has quit IRC | 08:59 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: OVO: Add reminder to add backporting code https://review.openstack.org/466641 | 09:08 |
| *** e0ne has quit IRC | 09:08 | |
| *** e0ne has joined #openstack-cinder | 09:09 | |
| openstackgerrit | Merged openstack/cinder master: Fix async mirroring on XIV limited range backends https://review.openstack.org/438199 | 09:18 |
| openstackgerrit | Merged openstack/cinder master: backup: handle device path represented by dir https://review.openstack.org/464012 | 09:21 |
| *** wanghao has joined #openstack-cinder | 09:23 | |
| *** dalgaaf has quit IRC | 09:25 | |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: SMBFS: remove redundant check https://review.openstack.org/464004 | 09:27 |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: SMBFS: switch to distributed locks https://review.openstack.org/464000 | 09:27 |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: SMBFS: report each share as a pool https://review.openstack.org/424235 | 09:27 |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: SMBFS: drop JSON file storing allocation data https://review.openstack.org/463989 | 09:27 |
| *** dalgaaf has joined #openstack-cinder | 09:27 | |
| *** bhagyashris has joined #openstack-cinder | 09:33 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Update replication v2.1 devref https://review.openstack.org/400807 | 09:35 |
| *** mvk has quit IRC | 09:36 | |
| *** wanghao has quit IRC | 09:37 | |
| *** zsli__ has quit IRC | 09:38 | |
| bhagyashris | wxy: Hi, | 09:40 |
| *** Cibo_ has quit IRC | 09:48 | |
| *** lkuchlan has quit IRC | 09:51 | |
| *** reduxio has quit IRC | 09:54 | |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder master: vzstorage: use remotefs.VZStorageRemoteFSClient https://review.openstack.org/462563 | 10:05 |
| *** stakeda has quit IRC | 10:06 | |
| *** zhangguoqing has joined #openstack-cinder | 10:06 | |
| *** mvk has joined #openstack-cinder | 10:06 | |
| *** reduxio has joined #openstack-cinder | 10:08 | |
| *** zhangguoqing has quit IRC | 10:10 | |
| *** e0ne has quit IRC | 10:13 | |
| *** reduxio has quit IRC | 10:28 | |
| *** reduxio has joined #openstack-cinder | 10:29 | |
| *** markstur has joined #openstack-cinder | 10:29 | |
| *** sdague has joined #openstack-cinder | 10:31 | |
| *** reduxio has quit IRC | 10:33 | |
| *** markstur has quit IRC | 10:34 | |
| wxy | bhagyashris: hi | 10:34 |
| bhagyashris | wxy: I want to discuss regarding bug: https://bugs.launchpad.net/cinder/+bug/1681374 . Actually I am trying to reproduce the bug as mentioned but I don't have that much of space to create volume on my machine so is there any alternative option to reproduce the bug. On my machine I have only 80 Gb disk free so I am able to create 80 volumes and i guess that will not cause any problem. | 10:40 |
| openstack | Launchpad bug 1681374 in Cinder "cinder-volume can't be inited with large number of volume" [Medium,Confirmed] - Assigned to Dinesh Bhor (dinesh-bhor) | 10:40 |
| *** lhx__ has quit IRC | 10:47 | |
| bhagyashris | wxy: Is it necessary that the volume should be "in-use" state or "available" state? | 10:47 |
| *** lhx__ has joined #openstack-cinder | 10:47 | |
| bhagyashris | wxy: and how many max volume should I required to create? Is it same number as mentioned in the bug? | 10:48 |
| *** reduxio has joined #openstack-cinder | 10:48 | |
| *** Shunli has joined #openstack-cinder | 10:48 | |
| *** Shunli has quit IRC | 10:49 | |
| wxy | bhagyashris: It's hard to say. We got this problem in our public prodution env. There are many volumes. I can ask our downstream guys, then update the bug later. Can you wait a few hours? | 10:53 |
| bhagyashris | wxy: yeah sure. | 10:54 |
| wxy | bhagyashris: :) | 10:54 |
| bhagyashris | wxy: One thing I want to ask, Is it possible to reproduce this issue with minimum available resources? I mean to say Is there any idea How I will be reproduce this issue with least resources? (Actually I have 90 GB of DiskSpace in my environment) | 10:59 |
| *** markstur has joined #openstack-cinder | 10:59 | |
| *** markstur has quit IRC | 11:03 | |
| openstackgerrit | Merged openstack/cinder master: ibm-storage: update version number and history https://review.openstack.org/462496 | 11:05 |
| *** reduxio has quit IRC | 11:08 | |
| *** reduxio has joined #openstack-cinder | 11:09 | |
| tommylikehu | hey smcginnis , winston-d , jungleboyj , could you take a look at this patch? https://review.openstack.org/#/c/445440/ | 11:10 |
| *** thiagolib_ has joined #openstack-cinder | 11:14 | |
| *** reduxio has quit IRC | 11:14 | |
| *** itzdilip has joined #openstack-cinder | 11:17 | |
| wxy | bhagyashris: Confirmed again, there are 100 thousand volumes. I have mentioned it in the bug already. | 11:17 |
| wxy | bhagyashris: For reproduce, you can insert 100 thousand volume db records. | 11:18 |
| wxy | bhagyashris: and then restart cinder-volume | 11:18 |
| *** gcb has quit IRC | 11:19 | |
| *** chhavi has quit IRC | 11:20 | |
| *** lennyb_ has quit IRC | 11:24 | |
| *** lennyb has quit IRC | 11:24 | |
| *** lennyb has joined #openstack-cinder | 11:25 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder-specs master: Support get volume metadata summary https://review.openstack.org/466691 | 11:26 |
| *** lennyb has quit IRC | 11:27 | |
| *** dilipsunkum has joined #openstack-cinder | 11:29 | |
| *** itzdilip has quit IRC | 11:30 | |
| *** dave-mccowan has joined #openstack-cinder | 11:30 | |
| *** dilipsunkum has quit IRC | 11:31 | |
| danpawlik | Morning | 11:34 |
| danpawlik | jbernard: hi. Have you test my PS about ceph keyring https://review.openstack.org/#/c/456672/ ? | 11:35 |
| *** lkuchlan has joined #openstack-cinder | 11:36 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder master: Modify the length of project_id for Messages https://review.openstack.org/466700 | 11:38 |
| openstackgerrit | Pranali Deore proposed openstack/cinder master: Modify the length of project_id for Messages https://review.openstack.org/466700 | 11:39 |
| bhagyashris | wxy: ok. | 11:41 |
| bhagyashris | wxy: Means Direct inserting the volume record in the db will reproduce this issue? There is no compulsion that I should create the volume through cinder create api right? | 11:43 |
| wxy | bhagyashris: yeah. | 11:43 |
| bhagyashris | wxy: ok Thank you for info. I will try out. | 11:43 |
| *** abishop has joined #openstack-cinder | 11:45 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder-specs master: Support metadata for backup resource https://review.openstack.org/456088 | 11:52 |
| *** zhangguoqing has joined #openstack-cinder | 12:04 | |
| *** mvk has quit IRC | 12:07 | |
| *** zhangguoqing has quit IRC | 12:09 | |
| *** reduxio has joined #openstack-cinder | 12:12 | |
| *** zhangguo_ has joined #openstack-cinder | 12:15 | |
| *** edmondsw has joined #openstack-cinder | 12:15 | |
| *** edmondsw has quit IRC | 12:16 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Convert marker format for listing manageable resources https://review.openstack.org/466147 | 12:21 |
| *** markstur has joined #openstack-cinder | 12:25 | |
| *** markstur has quit IRC | 12:29 | |
| *** chlong has joined #openstack-cinder | 12:30 | |
| bhagyashris | wxy: Do you have script to insert the test data or to make volumes entries in the volumes table. and Is it necessary to make the entries of volume record to the associated tables? (Like after creating the volume the volume record is creates the entries in the volume_type table) | 12:32 |
| *** mvk has joined #openstack-cinder | 12:32 | |
| *** jwcroppe has quit IRC | 12:33 | |
| *** jwcroppe has joined #openstack-cinder | 12:33 | |
| bhagyashris | wxy: Actually I have written the python program to make the entries of volume in the volume table but it's not creating the volume entries in the associated tables as mentioned above so I am not sure that after creating the entries in the volumes table only will reproduce the issue. | 12:34 |
| *** lhx__ has quit IRC | 12:36 | |
| *** dikonoor has joined #openstack-cinder | 12:36 | |
| *** jwcroppe has quit IRC | 12:37 | |
| *** chhavi has joined #openstack-cinder | 12:39 | |
| reduxio | Hello, I've upgraded my CI environment and devstack that support it, and now I fail to create new nodes. I think that provisioning fails since the local nova free disk space is insufficient, although the allocation is supposed to come from external san storage. Is there a new flag I need to use? | 12:41 |
| *** catintheroof has joined #openstack-cinder | 12:41 | |
| wxy | bhagyashris: sorrt i don't. Can you leave your commet in the Launchpad. i can take a test and reply you tomorrow. I'm in China TZ. Sorry that i don't have enough time today. | 12:44 |
| bhagyashris | wxy: ok. | 12:47 |
| bhagyashris | wxy: np. | 12:47 |
| bhagyashris | wxy: thanx for guidance. | 12:47 |
| *** chlong has quit IRC | 12:55 | |
| *** markstur has joined #openstack-cinder | 12:55 | |
| *** gkadam has quit IRC | 12:57 | |
| *** jwcroppe has joined #openstack-cinder | 12:58 | |
| *** markstur has quit IRC | 12:59 | |
| *** salv-orlando has joined #openstack-cinder | 13:01 | |
| *** lamt has joined #openstack-cinder | 13:04 | |
| *** salv-orlando has quit IRC | 13:07 | |
| *** salv-orlando has joined #openstack-cinder | 13:07 | |
| *** reduxio has quit IRC | 13:08 | |
| *** Cibo_ has joined #openstack-cinder | 13:10 | |
| *** lhx__ has joined #openstack-cinder | 13:12 | |
| *** mriedem has joined #openstack-cinder | 13:14 | |
| *** wN has joined #openstack-cinder | 13:17 | |
| *** wN has quit IRC | 13:17 | |
| *** wN has joined #openstack-cinder | 13:17 | |
| *** liverpooler has joined #openstack-cinder | 13:19 | |
| *** salv-orlando has quit IRC | 13:21 | |
| *** thiagolib_ has quit IRC | 13:22 | |
| *** markvoelker has joined #openstack-cinder | 13:22 | |
| *** markstur has joined #openstack-cinder | 13:25 | |
| *** xyang1 has joined #openstack-cinder | 13:27 | |
| *** gcb has joined #openstack-cinder | 13:28 | |
| *** markstur has quit IRC | 13:29 | |
| openstackgerrit | Drew Fisher proposed openstack/cinder master: ZFSSA: Allow clones to differ in size from parent https://review.openstack.org/466397 | 13:31 |
| *** reduxio has joined #openstack-cinder | 13:33 | |
| reduxio | Hello, I've upgraded my CI environment and devstack that support it, and now I fail to create new nodes. I think that provisioning fails since the local nova free disk space is insufficient, although the allocation is supposed to come from external san storage. I have a feeling it is somehow related to the new Nova cell feature. Any ideas? | 13:34 |
| *** markvoelker has quit IRC | 13:35 | |
| *** markvoelker has joined #openstack-cinder | 13:35 | |
| smcginnis | reduxio: So you've just spun up a new instance with the latest devstack and it doesn't work anymore? | 13:36 |
| *** MVenesio has joined #openstack-cinder | 13:38 | |
| reduxio | smcginnis: If i spin up a VM with small amount of storage (that the Nova host can handle) - the allocation succeeds, but it effectively allocates storage on the san. However for high storage requirement it fails before testing the san | 13:43 |
| reduxio | smcginnis: I've pulled from master both the devstack env and the CI env | 13:44 |
| smcginnis | reduxio: You're booting from volume? Sorry, not sure I understand the situation. It's possible recent changes went into devstack that changed some behavior, but I haven't seen failures myself. | 13:44 |
| *** markstur has joined #openstack-cinder | 13:44 | |
| *** Cibo_ has quit IRC | 13:45 | |
| reduxio | I'm not sure I have the terminology right. I'm booting from an image, into a 'flavor'. The same image on flavor m1.medium works, but fails on m1.large. | 13:46 |
| reduxio | smcginnis: the default cinder backend is our storage... So it's supposed to allocate a volume on our storage to mount the image on. | 13:47 |
| smcginnis | reduxio: Volumes created and attached to the instance will come from your storage, but the instance backing storage still needs to be available on the nova host. | 13:48 |
| reduxio | smcginnis: the instance backing storage? how much is that? | 13:48 |
| reduxio | smcginnis: shouldn't be different between flavors if i'm using the same image should it? | 13:49 |
| *** felipemonteiro has joined #openstack-cinder | 13:49 | |
| *** markstur has quit IRC | 13:49 | |
| smcginnis | reduxio: It's part of the flavor definition. | 13:49 |
| smcginnis | reduxio: m1.large is something like 40G IIRC. So it will look for a nova host that has 40G of free space. | 13:50 |
| reduxio | smcginnis: I see. I will try to free up some space then and check if this resolves the issue. Thanks! | 13:50 |
| *** felipemonteiro_ has joined #openstack-cinder | 13:51 | |
| smcginnis | reduxio: You can also define your own flavors if you want more CPU and memory but less backing storage. | 13:51 |
| *** markstur has joined #openstack-cinder | 13:52 | |
| reduxio | smcginnis: CI uses this flavor though so I prefer to make this work and have CI run as close to the origin as possible. | 13:52 |
| smcginnis | reduxio: Probably safer that way. | 13:53 |
| *** catinthe_ has joined #openstack-cinder | 13:53 | |
| *** catinth__ has joined #openstack-cinder | 13:53 | |
| *** catintheroof has quit IRC | 13:54 | |
| *** felipemonteiro has quit IRC | 13:54 | |
| *** jonesn has joined #openstack-cinder | 13:56 | |
| *** links has quit IRC | 13:57 | |
| *** catinthe_ has quit IRC | 13:57 | |
| *** crushil has joined #openstack-cinder | 13:58 | |
| *** mmedvede has quit IRC | 13:58 | |
| hemna | guwak | 13:58 |
| *** mmedvede has joined #openstack-cinder | 14:01 | |
| *** salv-orlando has joined #openstack-cinder | 14:03 | |
| *** salv-orlando has quit IRC | 14:08 | |
| *** zhaochao has quit IRC | 14:08 | |
| *** adisky_ has quit IRC | 14:11 | |
| jgriffith | reduxio you can use smaller images in terms of disk space without much problem | 14:11 |
| jgriffith | reduxio if you use thin LVM you can use as small as like 10 Gig root disk IIRC | 14:12 |
| jgriffith | reduxio the other option is attach a secondary volume to use for the Cinder LVM PV | 14:12 |
| jgriffith | The main thing is you’ll want the higher ram and VCPU if you want the test to run in a reasonable amount of time | 14:13 |
| *** eharney has joined #openstack-cinder | 14:14 | |
| *** reduxio has quit IRC | 14:16 | |
| *** reduxio has joined #openstack-cinder | 14:16 | |
| *** e0ne has joined #openstack-cinder | 14:16 | |
| *** esker has joined #openstack-cinder | 14:18 | |
| *** zhangguo_ has quit IRC | 14:20 | |
| *** reduxio has quit IRC | 14:20 | |
| *** esker has quit IRC | 14:23 | |
| jgriffith | smcginnis ildikov I’m open to suggestions on this: https://review.openstack.org/#/c/457021/ | 14:23 |
| jgriffith | smcginnis ildikov unfortunately I couldn’t come up with a good way to just make it do what either of you suggested without reworking the whole format of the docs for client | 14:24 |
| jgriffith | Which might be fine, just a bit out of scope for what I wanted :) | 14:24 |
| smcginnis | jgriffith: Eh, I think it's probably fine where it is. | 14:24 |
| smcginnis | jgriffith: Yeah, definitely out of scope. We can try to figure out a way to address that later. | 14:24 |
| jgriffith | smcginnis yeah, at some point it might be worth creating a full index page and TOC for client, but I dunno | 14:25 |
| *** dikonoor has quit IRC | 14:25 | |
| hemna | jgriffith, smcginnis hey, when you guys have a minute I need to discuss an issue I've found w/ the new attach workflow.... | 14:25 |
| smcginnis | jgriffith: There's also a lot of stuff we have in "devref" that isn't necessarily developer reference. | 14:26 |
| smcginnis | jgriffith: I've shown a few things to operators and gotten the feedback that it would be good to have some of that linked in a better place. | 14:26 |
| smcginnis | hemna: What's the issue? | 14:26 |
| hemna | https://bugs.launchpad.net/cinder/+bug/1692153 | 14:27 |
| openstack | Launchpad bug 1692153 in Cinder "v3 attachment connection_info formatting is different than v2" [High,New] | 14:27 |
| hemna | I believe it's a problem with the oslo objects | 14:27 |
| smcginnis | hemna: Oh right, saw that Friday. | 14:27 |
| hemna | it's modifying the values in the dictionaries and quoting them, turning everything into strings. | 14:27 |
| openstackgerrit | John Griffith proposed openstack/python-cinderclient master: Add doc for noauth usage https://review.openstack.org/457021 | 14:28 |
| hemna | which then causes anything that uses those values as ints or booleans to puke. | 14:28 |
| hemna | re: brick | 14:28 |
| hemna | target_lun: 1 => target_lun: "1" | 14:28 |
| hemna | sadness ensues | 14:28 |
| hemna | encrypted: False => encrypted: "False" | 14:29 |
| hemna | pewp | 14:29 |
| *** edmondsw has joined #openstack-cinder | 14:29 | |
| smcginnis | hemna: Where does that formatting happen? | 14:29 |
| smcginnis | jgriffith: Have you seen that? ^^ | 14:29 |
| *** markvoelker_ has joined #openstack-cinder | 14:29 | |
| jgriffith | smcginnis no actually, I was just reading through it | 14:30 |
| hemna | smcginnis, https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1965 | 14:30 |
| hemna | I think it happens on save | 14:30 |
| hemna | I traced the volume manager's return of attachment_update, and the connection_info looks fine there. | 14:30 |
| *** cknight has joined #openstack-cinder | 14:30 | |
| hemna | the API returns strings for the entire dict | 14:31 |
| smcginnis | hemna: Weird. What are we doing different between v2 and v3? Doesn't seem like oslo would just decide to change the way it's formatted. | 14:31 |
| hemna | v2 doesn't use oslo versioned objects | 14:31 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/objects/volume_attachment.py#L52 | 14:31 |
| jgriffith | Sorry, I’m not really following here | 14:32 |
| hemna | I think that is some sort of type enforcement | 14:32 |
| smcginnis | Really? I thought it did? | 14:32 |
| *** markvoelker has quit IRC | 14:33 | |
| hemna | jgriffith, the connection_info dictionary returned in the newer v3 attach/detach workflow has all of the values converted to strings. | 14:33 |
| hemna | u'target_lun': u'1', | 14:33 |
| hemna | instead of | 14:33 |
| hemna | u'target_lun': 1, | 14:33 |
| hemna | same with booleans | 14:34 |
| jgriffith | hemna Yeah, I get what you’re saying, I’m not following where it’s goofed though and what caused it. I’m also trying to figure out how it’s been working when I run it :) | 14:34 |
| hemna | u'encrypted': u'False', | 14:34 |
| hemna | instead of | 14:34 |
| hemna | u'encrypted': False, | 14:34 |
| jgriffith | hemna i.e. local-attach | 14:34 |
| hemna | yah | 14:34 |
| *** david-lyle has quit IRC | 14:34 | |
| hemna | I'm testing local-attach | 14:34 |
| hemna | with my hacked up brick cinderclient extension | 14:34 |
| hemna | and I'm seeing it | 14:34 |
| jgriffith | So change brick to deal with strings :) | 14:34 |
| jgriffith | Brick doesn’t have the versioning madness yet :) | 14:35 |
| *** david-lyle has joined #openstack-cinder | 14:35 | |
| hemna | not yet | 14:35 |
| *** dustins has joined #openstack-cinder | 14:35 | |
| hemna | heh | 14:35 |
| smcginnis | Seems really kind of add that the results aren't the same. | 14:35 |
| jgriffith | Otherwise we’ll have to have new obj versions and new micro versions for the API and still work with all of them :) | 14:35 |
| hemna | I've had requests from folks to change brick to use oslo vo's for the connector dict | 14:35 |
| smcginnis | Especially as we officially state v2 is exactly the same and compatible as v3.0. | 14:35 |
| jgriffith | Yeah, I’m still not completely following why they’re different | 14:35 |
| jgriffith | hemna sigh | 14:35 |
| hemna | jgriffith, https://github.com/openstack/cinder/blob/master/cinder/objects/volume_attachment.py#L52 | 14:35 |
| hemna | I think that declaration does it | 14:36 |
| jgriffith | Let me pull a fresh setup so I can follow along at home :) | 14:36 |
| hemna | and when we set the connection_info and then call save on the object, it converts everything. | 14:36 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1965-L1966 | 14:36 |
| hemna | there | 14:36 |
| hemna | jgriffith, want me to push a gist of my extension so you can hack/see it? | 14:36 |
| *** dustins has quit IRC | 14:37 | |
| smcginnis | dulek: If you're around, any idea on the stuff above? ^^ | 14:37 |
| e0ne | jgriffith: hi. could you please take a look on my comment for https://bugs.launchpad.net/python-cinderclient/+bug/1685390? | 14:38 |
| openstack | Launchpad bug 1685390 in python-cinderclient "need to fix noauth option for Client calls from python" [Undecided,Incomplete] | 14:38 |
| *** psachin has quit IRC | 14:38 | |
| hemna | jgriffith, https://gist.github.com/be954564d9dd0cb9671ce3a10147a727 | 14:39 |
| *** dustins has joined #openstack-cinder | 14:41 | |
| *** diablo_rojo has joined #openstack-cinder | 14:41 | |
| *** crushil has quit IRC | 14:42 | |
| hemna | in case anyone is using journactl and missing their colored log output. | 14:44 |
| hemna | apt-get install ccze | 14:44 |
| hemna | journalctl -f --unit devstack@c-vol.service | ccze -A | 14:44 |
| *** mtanino has joined #openstack-cinder | 14:46 | |
| smcginnis | hemna: I thought I just recently saw in the ML or another channel that there was just an option to pass to journalctl to get the color markers. | 14:47 |
| jgriffith | https://docs.openstack.org/developer/devstack/systemd.html | 14:47 |
| jgriffith | smcginnis hemna there was a ML post on that and it’s been added to the docs I believe | 14:47 |
| hemna | ah ok coolio | 14:48 |
| hemna | :) | 14:48 |
| hemna | that works too | 14:48 |
| hemna | thanks | 14:48 |
| jgriffith | hemna if that doesn’t work you should update the docs | 14:48 |
| jgriffith | Or if your method is mo-betta add it as an alternative | 14:49 |
| hemna | guess that -a option was buried in the text.....you had to read..... | 14:50 |
| hemna | reading is hard. | 14:50 |
| danpawlik | jbernard: hi. Have you test my PS about ceph keyring https://review.openstack.org/#/c/456672/ ? | 14:50 |
| openstackgerrit | wangxiyuan proposed openstack/cinder-specs master: Support metadata for backup resource https://review.openstack.org/456088 | 14:52 |
| *** david-lyle has quit IRC | 14:52 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Python3: Don't use Exception.message https://review.openstack.org/463230 | 14:52 |
| *** salv-orlando has joined #openstack-cinder | 14:52 | |
| jbernard | danpawlik: hi, when i do i will post my findings, i havent got to it yet | 14:52 |
| *** dillaman has quit IRC | 14:52 | |
| danpawlik | jbernard: ack | 14:53 |
| openstackgerrit | wangxiyuan proposed openstack/cinder-specs master: Support metadata for backup resource https://review.openstack.org/456088 | 14:53 |
| jgriffith | Sigh… Loci stuff is broken once again | 14:53 |
| jgriffith | Honestly I don’t know why each project doesn’t just own their own dockerfile | 14:53 |
| jgriffith | It’s kinda stooopid | 14:54 |
| hemna | doh | 14:54 |
| hemna | :( | 14:54 |
| *** diablo_rojo has quit IRC | 14:54 | |
| smcginnis | jgriffith: I still think it would be good to put some of that in the repo. | 14:54 |
| *** diablo_rojo has joined #openstack-cinder | 14:54 | |
| smcginnis | jgriffith: Maybe under toools or something? | 14:54 |
| jgriffith | And how much do you need to polish that turd anyway | 14:54 |
| smcginnis | hah | 14:54 |
| jgriffith | smcginnis yeah, I have a branch with a contrib dir | 14:55 |
| *** crushil has joined #openstack-cinder | 14:55 | |
| jgriffith | https://github.com/j-griffith/cinder/tree/add_blockbox | 14:55 |
| openstackgerrit | wangxiyuan proposed openstack/cinder-specs master: Support get volume metadata summary https://review.openstack.org/466691 | 14:55 |
| *** lucasxu has joined #openstack-cinder | 14:56 | |
| smcginnis | jgriffith: Feel free to propose it. | 14:56 |
| jgriffith | I want to get it cleaned up and then was going to raise it in this weeks meeting | 14:56 |
| jgriffith | Very well then :) | 14:56 |
| smcginnis | jgriffith: Good plan. | 14:56 |
| *** salv-orlando has quit IRC | 14:57 | |
| *** mvk_ has joined #openstack-cinder | 14:57 | |
| *** asselin has joined #openstack-cinder | 14:59 | |
| hemna | jgriffith, +1 | 15:01 |
| * hemna is a big fan of docker | 15:02 | |
| ildikov | smcginnis: jgriffith: I think not having docs exposed to users that would eb useful for them is just cruel | 15:03 |
| ildikov | smcginnis: jgriffith: my 2 cents | 15:03 |
| smcginnis | ildikov: It's like a hide and seek game though. We don't want to make it too easy for them. :P | 15:03 |
| * ildikov is in a Monday mood obviously, will get better by tomorrow the latest... | 15:03 | |
| e0ne | :) | 15:04 |
| hemna | ildikov, I'm overcoming Monday with lots of caffeine | 15:04 |
| ildikov | smcginnis: I reserve the right to disagree at this point | 15:04 |
| hemna | C8H10N4O2 | 15:05 |
| ildikov | smcginnis: if it bothers me to fix it in my non existing free time I might even fix it one day :) | 15:05 |
| ildikov | hemna: caffeine just makes me hyperactive about my mood :) | 15:05 |
| hemna | so, in other words, get out of your way? :P | 15:05 |
| *** dillaman has joined #openstack-cinder | 15:05 | |
| ildikov | hemna: so I went with chocolate this time | 15:05 |
| smcginnis | ildikov: Sounds good to me. Really I would like all of our docs much easier to find and much more comprehensive. | 15:06 |
| *** abhitechie has joined #openstack-cinder | 15:06 | |
| ildikov | smcginnis: shouldn't be that difficult really, I will try to check later the structure of the current one | 15:06 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Replication, replacing SMI-S with REST https://review.openstack.org/466027 | 15:06 |
| ildikov | jgriffith: any news about your plans with swap/migrate/whatever volume? :) | 15:07 |
| *** Julien-zte has joined #openstack-cinder | 15:10 | |
| *** lkuchlan has quit IRC | 15:11 | |
| openstackgerrit | Jeremy Zhang proposed openstack/cinder master: Fix typo errors https://review.openstack.org/454171 | 15:12 |
| *** jonesn has quit IRC | 15:13 | |
| openstackgerrit | Arnon Yaari proposed openstack/os-brick master: Change code to be more Pythonic https://review.openstack.org/466766 | 15:17 |
| openstackgerrit | Arnon Yaari proposed openstack/os-brick master: _rescan_iscsi should scan all relevant hosts connected to device not just one per IQN https://review.openstack.org/466767 | 15:18 |
| openstackgerrit | Merged openstack/cinder master: OVO: Add reminder to add backporting code https://review.openstack.org/466641 | 15:18 |
| jgriffith | ildikov yes! | 15:19 |
| jgriffith | ildikov the unfortunate thing is that it depends upon having attach implemented first | 15:19 |
| jgriffith | Which I *almost* have done | 15:19 |
| jgriffith | ildikov BTW, I don’t disagree with you about the Docs thing | 15:20 |
| jgriffith | ildikov I just don’t want to make restructuring our entire client docs a dependency to documenting how stand-alone works | 15:20 |
| jgriffith | ildikov basically I spent too much time trying to get it to work so thought it would be good to write it up *somewhere* | 15:21 |
| e0ne | why do we still don't have time limit for unit tests? | 15:24 |
| jgriffith | e0ne that’s an excellent question :) | 15:24 |
| e0ne | 30 second per unit test!!! | 15:25 |
| e0ne | o_0 | 15:25 |
| jgriffith | e0ne Yikes! Who’s hitting that now? | 15:25 |
| jgriffith | That’s crazy | 15:25 |
| jgriffith | It’s always a driver :) | 15:25 |
| e0ne | jgriffith: I'm going to fix it right now | 15:25 |
| e0ne | cinder.tests.unit.brick.test_brick_lvm.BrickLvmTestCase.test_lv_deactivate_timeout 30.039 | 15:25 |
| e0ne | cinder.tests.unit.brick.test_brick_lvm.BrickLvmTestCaseIgnoreFDWarnings.test_lv_deactivate_timeout 30.039 | 15:25 |
| jgriffith | Oh! LOL.. not what I expected | 15:26 |
| e0ne | 5s per drivers test now looks great | 15:26 |
| ildikov | jgriffith: I think just to link the file to the index.rst file could solve the docs part | 15:27 |
| eharney | e0ne: i think those need sleep mocks | 15:27 |
| tommylikehu | eharney: +1 | 15:27 |
| ildikov | jgriffith: I don't like overcomplicating things either so it might not how it looks like, but I don't want to overcomplicate your life either :) | 15:28 |
| e0ne | eharney: we'd and issue with sleep mocks in the past | 15:28 |
| ildikov | jgriffith: I just wanted to have some parts of the swap changes up somewhere if you have it | 15:28 |
| eharney | e0ne: ? | 15:29 |
| ildikov | jgriffith: to point people there so they know what to expect from the new flow version | 15:29 |
| ildikov | jgriffith: regardless of attach | 15:29 |
| tommylikehu | eharney: could you take a look at this patch again? https://review.openstack.org/#/c/446356/ | 15:30 |
| eharney | e0ne: we have sleep mocks in many tests, they seem to work fine | 15:30 |
| eharney | tommylikehu: yes | 15:30 |
| e0ne | eharney: https://bugs.launchpad.net/cinder/+bug/1578986 | 15:31 |
| openstack | Launchpad bug 1578986 in Cinder "Unit tests hang forever" [Medium,Fix released] - Assigned to Gorka Eguileor (gorka) | 15:31 |
| ildikov | jgriffith: BTW, are you working on attach too? | 15:31 |
| e0ne | and one of the commits: https://github.com/openstack/cinder/commit/a6eca292fc93acaf5aa4dc5302154b96076195d9 | 15:31 |
| jgriffith | ildikov maybe I’m missing something, the problem with that (link in index.rst) is that there’s no TOC entry or links to begin with. So it’s a new introduction | 15:31 |
| jgriffith | ildikov how about this... | 15:32 |
| jgriffith | ildikov I’ll propose a separate patch that creates a TOC and links all the docs in the src directory? | 15:32 |
| eharney | e0ne: yes, it's always possible to do things incorrectly, that doesn't mean they don't work as a whole | 15:32 |
| e0ne | eharney: I agree with you | 15:32 |
| ildikov | jgriffith: I can do that | 15:33 |
| ildikov | jgriffith: I don't want to distract you from other things with this | 15:33 |
| *** markvoelker_ has quit IRC | 15:35 | |
| *** lpetrut has quit IRC | 15:35 | |
| *** nicolasbock has joined #openstack-cinder | 15:36 | |
| *** salv-orlando has joined #openstack-cinder | 15:36 | |
| *** nicolasbock has quit IRC | 15:36 | |
| *** nicolasbock has joined #openstack-cinder | 15:37 | |
| jgriffith | No big deal | 15:38 |
| jgriffith | ildikov so on the swap thing... | 15:38 |
| jgriffith | ildikov I’m working on implementing attachment_create and attachment_update in Nova so we can just remove the call to Cinder altogether | 15:38 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: Add open-iscsi manual scan support https://review.openstack.org/455394 | 15:38 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: Refactor iSCSI disconnect https://review.openstack.org/455392 | 15:38 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: Refactor iSCSI connect https://review.openstack.org/455393 | 15:38 |
| ildikov | jgriffith: you mean the full attach flow based on the new API? | 15:39 |
| *** chlong has joined #openstack-cinder | 15:39 | |
| *** Cibo_ has joined #openstack-cinder | 15:40 | |
| *** lostRhino has joined #openstack-cinder | 15:41 | |
| lostRhino | random question does anyone have the new cinder mascot image available for viewing? There was a preview at the conference - was wondering if someone had a link to it? | 15:42 |
| *** itzdilip has joined #openstack-cinder | 15:43 | |
| *** Cibo_ has quit IRC | 15:44 | |
| jgriffith | ildikov unfortunately yes | 15:44 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Base functionality, replacing SMI-S with REST https://review.openstack.org/455428 | 15:45 |
| jgriffith | ildikov it doesn’t need the implementation for nova API’s attach etc, but it needs all the supporting methods and plumbing | 15:45 |
| jgriffith | If that makes sense? | 15:45 |
| *** belmoreira has quit IRC | 15:45 | |
| jgriffith | I can share what I’ve got going on here a bit later | 15:48 |
| ildikov | jgriffith: I basically asked to see what I can help out with, I wanted to get attach work for testing, but honestly couldn't get there as of yet | 15:49 |
| ildikov | jgriffith: but the plan hasn't changed so besides seeing where we are I also asked to see what I can do | 15:50 |
| jgriffith | ildikov ok, well I could certainly use help, let me finish up my current fire-drill and then I’ll get the code I have posted and see where you want to help out | 15:51 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: Add time.sleep mock for test_lv_deactivate_timeout https://review.openstack.org/466794 | 15:51 |
| ildikov | jgriffith: sounds good, tnx | 15:52 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Retype, replacing SMI-S with REST https://review.openstack.org/456757 | 15:52 |
| *** itzdilip has quit IRC | 15:57 | |
| *** pcaruana has quit IRC | 15:59 | |
| *** e0ne has quit IRC | 16:00 | |
| *** david-lyle has joined #openstack-cinder | 16:03 | |
| *** mvk_ has quit IRC | 16:03 | |
| *** itzdilip has joined #openstack-cinder | 16:04 | |
| *** lucasxu has quit IRC | 16:04 | |
| *** gouthamr has joined #openstack-cinder | 16:05 | |
| *** itzdilip has quit IRC | 16:05 | |
| *** mvk has quit IRC | 16:07 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - QoS, replacing SMI-S with REST https://review.openstack.org/456759 | 16:08 |
| *** gouthamr has quit IRC | 16:09 | |
| *** mikal has quit IRC | 16:14 | |
| *** rcernin has quit IRC | 16:14 | |
| *** mikal has joined #openstack-cinder | 16:16 | |
| *** lucasxu has joined #openstack-cinder | 16:18 | |
| openstackgerrit | Merged openstack/cinder master: 3PAR: Added volume to vvset in online copy https://review.openstack.org/463474 | 16:19 |
| *** Apoorva has joined #openstack-cinder | 16:19 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Compression, replacing SMI-S with REST https://review.openstack.org/460191 | 16:21 |
| *** ntpttr__ is now known as ntpttr | 16:21 | |
| openstackgerrit | Merged openstack/cinder master: Remove backup-restore methods in driver code https://review.openstack.org/432948 | 16:22 |
| openstackgerrit | Merged openstack/cinder master: Fix incorrect volume_id request description in api-ref https://review.openstack.org/465421 | 16:22 |
| *** kaisers has quit IRC | 16:22 | |
| *** kaisers has joined #openstack-cinder | 16:23 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Bump pylint to 1.4.3 https://review.openstack.org/442857 | 16:28 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Replication, replacing SMI-S with REST https://review.openstack.org/466027 | 16:29 |
| *** rcernin has joined #openstack-cinder | 16:31 | |
| *** gouthamr has joined #openstack-cinder | 16:34 | |
| Swimingly | smcginnis: https://review.openstack.org/#/c/464003/ is ready for a person to take a gander at, it's a resubmission of a change you've already looked at, so it shouldn't be too bad. | 16:35 |
| Swimingly | Hopefully it's not rude to ping you! I'm still a stranger in this strange land :D | 16:36 |
| smcginnis | Swimingly: No worries, I don't mind. I've got it open in a tab now and hopefully I'll get to it soon. | 16:37 |
| Swimingly | Awesome! Thank you! | 16:38 |
| *** abhitechie has quit IRC | 16:38 | |
| *** abhitechie has joined #openstack-cinder | 16:39 | |
| hemna | jgriffith, http://paste.openstack.org/show/610253/ | 16:39 |
| hemna | can't get the blockbox to make | 16:39 |
| *** gouthamr has quit IRC | 16:40 | |
| *** salv-orlando has quit IRC | 16:40 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Mock qemu calls in sheepdog tests https://review.openstack.org/465626 | 16:42 |
| *** abhitechie has quit IRC | 16:42 | |
| *** abhitechie has joined #openstack-cinder | 16:43 | |
| openstackgerrit | Merged openstack/cinder master: Remove unused volume_types field when creating group. https://review.openstack.org/465868 | 16:44 |
| *** lostRhino has left #openstack-cinder | 16:45 | |
| *** Julien-zte has quit IRC | 16:45 | |
| *** reduxio has joined #openstack-cinder | 16:46 | |
| *** reduxio has quit IRC | 16:50 | |
| *** lisali has joined #openstack-cinder | 16:55 | |
| *** lisali has quit IRC | 16:55 | |
| *** chlong has quit IRC | 16:59 | |
| *** Cibo_ has joined #openstack-cinder | 17:00 | |
| *** abishop has quit IRC | 17:00 | |
| *** salv-orlando has joined #openstack-cinder | 17:01 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove BRCD primary_switch_wwn config option https://review.openstack.org/463204 | 17:02 |
| openstackgerrit | Merged openstack/python-cinderclient master: Add doc for noauth usage https://review.openstack.org/457021 | 17:02 |
| *** dhinesh has joined #openstack-cinder | 17:04 | |
| *** mvk has joined #openstack-cinder | 17:05 | |
| *** salv-orlando has quit IRC | 17:05 | |
| jgriffith | hemna yeah, that’s a new error coming from the LOCI build that I mentioned earlier | 17:07 |
| jgriffith | hemna chatting with SamYaple in openstack-loci about it | 17:07 |
| hemna | ah ok, I was hoping it wasn't something you hadn't seen | 17:09 |
| *** viks has joined #openstack-cinder | 17:10 | |
| jgriffith | hemna for now should be able to pull from docker hub for the base image: https://hub.docker.com/r/openstackloci/cinder/ | 17:12 |
| jgriffith | Although those be older images | 17:13 |
| *** lucasxu has quit IRC | 17:14 | |
| *** chlong has joined #openstack-cinder | 17:15 | |
| *** mvk_ has joined #openstack-cinder | 17:15 | |
| *** chlong has quit IRC | 17:15 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Mark ETERNUS drivers as unsupported https://review.openstack.org/462997 | 17:15 |
| *** jmccrory_away is now known as jmccrory | 17:18 | |
| viks | hi, need some help on https://review.openstack.org/#/c/443951/18/nova/virt/libvirt/volume/vrtshyperscale.py@58 . We have already checked-in the fix for OS-brick at https://review.openstack.org/#/c/466437/ which is merged now. Do we need a new OS-brick version? How is it updated? Once we know these we can update the global-requirements... | 17:18 |
| *** ChubYann has joined #openstack-cinder | 17:18 | |
| smcginnis | viks: Let me check. I can probably request a new os-brick release now. | 17:19 |
| viks | Thanks smcginnis... | 17:20 |
| openstackgerrit | Merged openstack/cinder master: Fix bugs of GPFSNFS & GPFSRemote Driver initialization failure https://review.openstack.org/465961 | 17:20 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Mark Infortrend drivers as unsupported https://review.openstack.org/463007 | 17:20 |
| openstackgerrit | Merged openstack/cinder master: Add the missing parameters in the api-ref of ext-backups.inc https://review.openstack.org/450537 | 17:21 |
| openstackgerrit | Merged openstack/cinder master: Convert marker format for listing manageable resources https://review.openstack.org/466147 | 17:21 |
| *** abhitechie has quit IRC | 17:23 | |
| smcginnis | viks: https://review.openstack.org/466825 | 17:24 |
| *** lhx__ has quit IRC | 17:24 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Mark Nimble driver as unsupported https://review.openstack.org/463009 | 17:26 |
| openstackgerrit | Merged openstack/cinder master: Don't check thin provisioning when manage volumes https://review.openstack.org/457119 | 17:31 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Mark Reduxio driver as unsupported https://review.openstack.org/463026 | 17:32 |
| *** markstur has quit IRC | 17:36 | |
| *** e0ne has joined #openstack-cinder | 17:37 | |
| openstackgerrit | Chhavi Agarwal proposed openstack/cinder master: Storwize Support iSCSI host when FC volume is there https://review.openstack.org/461850 | 17:38 |
| chhavi | hemna: provided clarification for the review comments, please have a look. https://review.openstack.org/461850 | 17:39 |
| viks | Thanks again smcginnis... | 17:40 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Mark Synology driver as unsupported https://review.openstack.org/463027 | 17:40 |
| *** asselin has quit IRC | 17:42 | |
| *** reduxio has joined #openstack-cinder | 17:43 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Mark Tegile driver as unsupported https://review.openstack.org/463029 | 17:43 |
| *** Cibo_ has quit IRC | 17:44 | |
| viks | Just a question on similar lines(os-brick version upgrade), do we need a version upgrade everytime we check-in a fix...or is it based on release? current model suggests that it is not based on release...is the understanding correct or am i missing something here? | 17:45 |
| smcginnis | viks: Not sure what you mean. When a new release is done, the version is updated. | 17:45 |
| smcginnis | viks: So to use something that is merged, you either need to use LIBS_FROM_GIT=os-brick to pull in the current master, or you need to wait for a new release. | 17:46 |
| *** reduxio has quit IRC | 17:47 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Mark X-IO driver as unsupported https://review.openstack.org/463032 | 17:48 |
| viks | ok | 17:48 |
| *** mvk_ has quit IRC | 17:54 | |
| *** chlong has joined #openstack-cinder | 17:55 | |
| *** e0ne has quit IRC | 18:06 | |
| *** asselin has joined #openstack-cinder | 18:06 | |
| *** abishop has joined #openstack-cinder | 18:09 | |
| *** r-daneel has joined #openstack-cinder | 18:09 | |
| *** crushil has quit IRC | 18:10 | |
| *** lucasxu has joined #openstack-cinder | 18:12 | |
| *** kaisers has quit IRC | 18:14 | |
| *** kaisers has joined #openstack-cinder | 18:14 | |
| *** kaisers has quit IRC | 18:17 | |
| *** kaisers has joined #openstack-cinder | 18:17 | |
| *** e0ne has joined #openstack-cinder | 18:18 | |
| openstackgerrit | Merged openstack/cinder master: Fix quota reserve for manage snapshot https://review.openstack.org/458746 | 18:21 |
| openstackgerrit | Merged openstack/cinder master: Fix typo errors https://review.openstack.org/454171 | 18:21 |
| *** itlinux has joined #openstack-cinder | 18:21 | |
| openstackgerrit | Merged openstack/cinder master: Add time.sleep mock for test_lv_deactivate_timeout https://review.openstack.org/466794 | 18:21 |
| jgriffith | hemna let me know if you hit any other snags with the BlockBox stuff | 18:23 |
| hemna | well..... | 18:23 |
| jgriffith | hehe | 18:23 |
| jgriffith | I debated whether I should ask or not LOL | 18:24 |
| hemna | haha | 18:24 |
| jgriffith | Just kidding | 18:24 |
| hemna | I'm trying to run the example command you gave in the readme for a devenv | 18:24 |
| hemna | and it's not happy | 18:24 |
| *** gouthamr has joined #openstack-cinder | 18:24 | |
| jgriffith | hemna Ahh.. which repo did you clone? | 18:25 |
| jgriffith | That cinder repo or the actual BlockBox repo? | 18:25 |
| hemna | https://hastebin.com/jiposuwuca.sql | 18:25 |
| hemna | I cloned the blockbox repo | 18:25 |
| hemna | and tried to follow the destructions there | 18:25 |
| jgriffith | That won’t work :( | 18:25 |
| hemna | after I hacked the makefile to incorporate the fix from SamYaple | 18:26 |
| jgriffith | Oh… wait, no you got past that point | 18:26 |
| jgriffith | Lemme see | 18:26 |
| hemna | the build completed | 18:26 |
| jgriffith | hemna yeah, ok… so the oslo-config-generator thing didn’t work | 18:26 |
| jgriffith | ? | 18:26 |
| openstackgerrit | xing-yang proposed openstack/cinder master: Tiramisu: replication group support https://review.openstack.org/352228 | 18:27 |
| hemna | yah it just dumped out that warning and bailed | 18:27 |
| jgriffith | Did you make sure the tag was set right in the docker_files/Dockerfile.cinder-devenv file? | 18:28 |
| jgriffith | Although that’s what I thought the problem was at first, but I think you got past that | 18:28 |
| *** catinth__ has quit IRC | 18:28 | |
| hemna | I didn't look in those at all | 18:29 |
| jgriffith | I’m trying it now… will see what happens on my side | 18:29 |
| hemna | ok | 18:29 |
| jgriffith | hemna if you didn’t change the tag of the base image it shouldn’t matter | 18:29 |
| jgriffith | I’m building it now, we’ll see what fails | 18:29 |
| *** catintheroof has joined #openstack-cinder | 18:30 | |
| jgriffith | Our test requirements install is CRAZY | 18:30 |
| jgriffith | Lots and lots o'bits | 18:30 |
| hemna | yah it's a TON of stuffs | 18:30 |
| *** harlowja has joined #openstack-cinder | 18:37 | |
| *** markstur has joined #openstack-cinder | 18:38 | |
| jgriffith | hemna look in your cinder/etc directory | 18:39 |
| jgriffith | That warning pops up for me, but the config file is generated | 18:39 |
| jgriffith | hemna I’m assuming the same is true on your side, just not used to not seeing the output message and it’s kinda fast compared to the usual way | 18:40 |
| *** dave-mccowan has quit IRC | 18:41 | |
| *** lucasxu has quit IRC | 18:41 | |
| *** itlinux has quit IRC | 18:41 | |
| *** markstur has quit IRC | 18:42 | |
| *** itlinux has joined #openstack-cinder | 18:42 | |
| *** lucasxu has joined #openstack-cinder | 18:43 | |
| *** harlowja has quit IRC | 18:43 | |
| *** crushil has joined #openstack-cinder | 18:43 | |
| *** Cibo_ has joined #openstack-cinder | 18:45 | |
| jgriffith | hemna anyway… let me know | 18:46 |
| jgriffith | I need to get back to the nova attach stuff or else ildikov might beat me up!! | 18:46 |
| jgriffith | Or flat out murder me | 18:46 |
| *** itlinux has quit IRC | 18:48 | |
| ildikov | jgriffith: I have my methods for sure ;) | 18:54 |
| *** gouthamr has quit IRC | 18:54 | |
| *** e0ne has quit IRC | 18:55 | |
| *** lucasxu has quit IRC | 18:58 | |
| jgriffith | Haha | 18:58 |
| jgriffith | Every week I say “this is the week” then poof | 18:59 |
| *** lucasxu has joined #openstack-cinder | 19:03 | |
| *** markstur has joined #openstack-cinder | 19:05 | |
| openstackgerrit | Merged openstack/cinder master: qemu_img_info: Don't autodetect source format https://review.openstack.org/453341 | 19:09 |
| openstackgerrit | Merged openstack/cinder master: Glance: attach volume encryption key id to image https://review.openstack.org/453342 | 19:09 |
| *** nicolasbock has quit IRC | 19:13 | |
| *** salv-orlando has joined #openstack-cinder | 19:18 | |
| openstackgerrit | Merged openstack/cinder master: Fix the filter when list volume with group https://review.openstack.org/457855 | 19:24 |
| *** salv-orlando has quit IRC | 19:28 | |
| *** reduxio has joined #openstack-cinder | 19:40 | |
| *** reduxio has quit IRC | 19:44 | |
| *** chhavi has quit IRC | 19:48 | |
| *** salv-orlando has joined #openstack-cinder | 19:56 | |
| openstackgerrit | Gaurang Tapase proposed openstack/cinder master: GPFS: Raise VolumeBackendAPI exception for unsupported operations https://review.openstack.org/466889 | 20:03 |
| *** nicolasbock has joined #openstack-cinder | 20:06 | |
| *** chlong has quit IRC | 20:07 | |
| *** arunkant_ has quit IRC | 20:13 | |
| *** liverpooler has quit IRC | 20:21 | |
| *** chlong has joined #openstack-cinder | 20:31 | |
| *** dillaman has quit IRC | 20:36 | |
| *** mriedem is now known as mriedem_afk | 20:36 | |
| hemna | smcginnis, ping | 20:38 |
| smcginnis | hemna: Hey | 20:38 |
| hemna | question for you | 20:38 |
| hemna | I was looking at this guy | 20:38 |
| hemna | https://review.openstack.org/#/c/466889/1/cinder/volume/drivers/ibm/gpfs.py | 20:38 |
| hemna | and I was thinking, does it make sense do add this https://hastebin.com/iqijaqecok to that review? | 20:39 |
| hemna | is that the direction we should go ? | 20:39 |
| hemna | and in addition to the LOG.error() calls, also call the message_api with those new defined_messages | 20:39 |
| *** salv-orlando has quit IRC | 20:39 | |
| smcginnis | hemna: Ah, I see what you're saying. | 20:40 |
| smcginnis | Yeah, that could be useful. | 20:40 |
| *** salv-orlando has joined #openstack-cinder | 20:40 | |
| hemna | we haven't really propogated the message api stuffs much yet. | 20:40 |
| hemna | that review made me think of it | 20:40 |
| smcginnis | hemna: Yeah... | 20:40 |
| smcginnis | heI think we're going to have to all educate ourselves and start changing things to make that useful. | 20:40 |
| *** chlong has quit IRC | 20:41 | |
| hemna | yah I think so | 20:41 |
| hemna | is that a good review to start? | 20:41 |
| hemna | do we care about the defined messages and their values? | 20:41 |
| hemna | I can see the VOLUME_1234567 getting large | 20:41 |
| hemna | do we care about organizing them? numbering them? | 20:42 |
| smcginnis | hemna: I guess so. I feel a tiny bit bad that it's extra work form what they were doinng, but we need to start enforcing adding useful info there. | 20:42 |
| hemna | or does it even matter | 20:42 |
| hemna | VOLUME_SCREWED is just as useful as VOLUME_999432 | 20:42 |
| smcginnis | hemna: Not really sure. You thinking some kind of numbering system based on category? | 20:42 |
| smcginnis | Hah, true. | 20:42 |
| smcginnis | And easier to read. | 20:42 |
| hemna | how and who uses those values outside of cinder? | 20:43 |
| hemna | I had thought of numbering the ones I hacked in there something different | 20:43 |
| smcginnis | hemna: Right now, I don't think anyone. | 20:43 |
| hemna | VOLUME_000100 | 20:43 |
| hemna | no idea though | 20:44 |
| hemna | I can see us adding lots of UNSUPPORTED_DRIVER_FEATURE_XXXX ones | 20:44 |
| smcginnis | hemna: tommylikehu was the last one to work on the messaging stuff. I'm not sure how far he's gone with it though. | 20:44 |
| smcginnis | hemna: Might be interesting to get his perspective too. | 20:44 |
| *** salv-orlando has quit IRC | 20:44 | |
| hemna | he's in Asia ? | 20:45 |
| hemna | probably sleeping is my guess | 20:45 |
| smcginnis | hemna: Also thinking we should probably talk to horizon at some point to make sure they are aware of this and how they see using it. | 20:45 |
| smcginnis | hemna: Yeah. Though he's surprised me a bunch of times. :) | 20:45 |
| hemna | ok, so these "messages" just get stuffed into a DB table now. | 20:45 |
| * hemna wonders if there is an API call to fetch them | 20:46 | |
| smcginnis | hemna: Yep. Not sure if there are any consumers of those messages yet. | 20:46 |
| smcginnis | hemna: Yes, there is. | 20:46 |
| hemna | and if this is a generic thing...... | 20:46 |
| smcginnis | https://developer.openstack.org/api-ref/block-storage/v3/index.html#messages | 20:46 |
| hemna | might be worth coordinating with other projects | 20:47 |
| hemna | so we don't have 3x10^50 implementations of messages | 20:47 |
| hemna | dunno | 20:47 |
| smcginnis | hemna: It was a cross-project effort I thought. Or at least manila was doingn it too. | 20:47 |
| smcginnis | hemna: ameade and sheel were both driving it. | 20:47 |
| smcginnis | At least within cinder. | 20:47 |
| eharney | smcginnis: some horizon folks are already on it | 20:48 |
| hemna | https://blueprints.launchpad.net/manila/+spec/user-messages | 20:48 |
| hemna | that thingy ? | 20:48 |
| eharney | https://review.openstack.org/#/c/446644/ | 20:48 |
| hemna | https://review.openstack.org/#/q/topic:bp/user-messages,n,z | 20:48 |
| hemna | looks like a bunch of outstanding reviews | 20:48 |
| hemna | you would think there should be an easy way to add common plugins like this into each project | 20:49 |
| smcginnis | eharney: Cool | 20:49 |
| hemna | if we all use the same table in the db and schema for those messages | 20:49 |
| hemna | at least from the API perspective | 20:50 |
| smcginnis | hemna: Yeah, I really hope anyone doing this is talking to each other and being consistent. | 20:50 |
| hemna | https://review.openstack.org/#/c/313549/ | 20:50 |
| smcginnis | hemna: Almost should be some kind of oslo thing to plug into projects. | 20:50 |
| hemna | so, manilla doesn't have it yet | 20:50 |
| hemna | smcginnis, yup | 20:50 |
| hemna | ok so I guess we should hold off on doing too much with it yet | 20:53 |
| jgriffith | hemna did you check if that config.sample was there? | 20:53 |
| *** dillaman has joined #openstack-cinder | 20:53 | |
| eharney | hold off on doing what? it's already an API that we've shipped | 20:53 |
| hemna | jgriffith, I got sidetracked | 20:53 |
| jgriffith | hemna I know the feeling :) | 20:54 |
| smcginnis | hemna: I'm mixed. If we make it actually useful, then more consumers will use it. | 20:54 |
| * ameade lurks | 20:54 | |
| smcginnis | Shh, he's here. | 20:54 |
| hemna | eharney, well, just propagating the usage of it | 20:54 |
| *** gouthamr has joined #openstack-cinder | 20:54 | |
| hemna | and adding more defined messages | 20:54 |
| hemna | eharney,https://hastebin.com/iqijaqecok | 20:55 |
| smcginnis | hemna: Maybe suggest your gist on the review, but don't downvote on it? | 20:55 |
| smcginnis | hemna: Then maybe they can at least follow up with a patch afterwards. | 20:55 |
| hemna | eharney, that was what I hacked after seeing this: https://review.openstack.org/#/c/466889/1/cinder/volume/drivers/ibm/gpfs.py | 20:55 |
| eharney | i'm not sure why we would hold back on much, it's already functional enough to be quite useful to users, and we can always tweak it over time as needed | 20:55 |
| smcginnis | +1 | 20:56 |
| hemna | I guess that's one of the reasons I raised the issue here | 20:56 |
| hemna | I wanted to get on the same page as everyone else | 20:56 |
| hemna | and I had other questions wrt the defined message strings/value etc | 20:56 |
| hemna | scrollback | 20:57 |
| eharney | yeah, i haven't taken the time yet to understand the recent proposals about changing some of that | 20:57 |
| *** felipemonteiro_ has quit IRC | 20:59 | |
| hemna | smcginnis, ok I posted my review on that | 20:59 |
| eharney | i'm not sure if we have a good way to raise those messages from within a driver yet | 21:01 |
| hemna | the message_api.create() just poops a new entry into the db table. | 21:02 |
| *** dillaman has quit IRC | 21:03 | |
| hemna | also looks like it wasn't designed in mind to take in optional strings as extra info in each message | 21:03 |
| hemna | maybe that's on purpose. | 21:03 |
| hemna | like FAILED_TO_PEWP = "We failed to pewp something because %s" | 21:04 |
| hemna | I suppose that makes each id less useful | 21:04 |
| smcginnis | :D | 21:04 |
| hemna | I hate over designing things, but just thought we should decide on this stuffs before it's used a million times over in code. | 21:05 |
| *** dave-mccowan has joined #openstack-cinder | 21:05 | |
| *** gluytium has quit IRC | 21:10 | |
| *** kaisers has quit IRC | 21:10 | |
| *** gouthamr has quit IRC | 21:13 | |
| *** gluytium has joined #openstack-cinder | 21:14 | |
| *** dillaman has joined #openstack-cinder | 21:15 | |
| *** gouthamr has joined #openstack-cinder | 21:16 | |
| *** catintheroof has quit IRC | 21:22 | |
| *** MVenesio has quit IRC | 21:23 | |
| *** kaisers1 has quit IRC | 21:25 | |
| *** MVenesio has joined #openstack-cinder | 21:28 | |
| *** catintheroof has joined #openstack-cinder | 21:29 | |
| *** eharney has quit IRC | 21:40 | |
| *** kaisers has joined #openstack-cinder | 21:40 | |
| *** gouthamr has quit IRC | 21:41 | |
| *** reduxio has joined #openstack-cinder | 21:42 | |
| *** jwcroppe has quit IRC | 21:42 | |
| *** jwcroppe has joined #openstack-cinder | 21:43 | |
| *** reduxio has quit IRC | 21:47 | |
| *** jwcroppe has quit IRC | 21:47 | |
| *** cknight has quit IRC | 21:53 | |
| *** lucasxu has quit IRC | 21:58 | |
| *** MVenesio has quit IRC | 21:59 | |
| *** mtanino has quit IRC | 22:03 | |
| *** Julien-zte has joined #openstack-cinder | 22:10 | |
| *** xyang1 has quit IRC | 22:13 | |
| *** itlinux has joined #openstack-cinder | 22:14 | |
| *** Cibo_ has quit IRC | 22:27 | |
| *** crushil has quit IRC | 22:27 | |
| tommylikehu | hey hemna , around? | 22:27 |
| *** jwcroppe has joined #openstack-cinder | 22:28 | |
| *** dustins has quit IRC | 22:32 | |
| hemna | tommylikehu, hey | 22:33 |
| tommylikehu | hemna morning, I have another version of user message there: https://review.openstack.org/#/c/451761/ | 22:34 |
| tommylikehu | hemna: in short we add the relationship from exception to message and also bring message attribute 'action' there. | 22:35 |
| *** Apoorva_ has joined #openstack-cinder | 22:36 | |
| tommylikehu | so if I get your point correctly, I think it's easy to add the user message of you case by raise some specific exceptions there and then catch & translate the exceptions | 22:36 |
| *** Apoorva has quit IRC | 22:40 | |
| hemna | ok checking | 22:40 |
| *** itlinux has quit IRC | 22:42 | |
| hemna | tommylikehu, I think you should try and include some usage examples of that | 22:43 |
| tommylikehu | hemna: yes we already have, please take a look at this one line#1044 and #1061 | 22:45 |
| tommylikehu | https://review.openstack.org/#/c/448384/18/cinder/volume/manager.py | 22:45 |
| tommylikehu | hemna: the original part | 22:46 |
| hemna | can't we extract the project_id from the context we are already passing in ? | 22:47 |
| *** Apoorva_ has quit IRC | 22:47 | |
| *** r-daneel has quit IRC | 22:47 | |
| tommylikehu | yes, we could | 22:47 |
| *** Apoorva has joined #openstack-cinder | 22:47 | |
| hemna | the number of parameters being passed in to that call is......a pain. | 22:48 |
| tommylikehu | hemna, that's another improvment | 22:49 |
| tommylikehu | :) | 22:49 |
| tommylikehu | We still have chance because my patch is not merged:) | 22:50 |
| tommylikehu | Hey jungleboyj , I am a little afraid maybe I didn't get your point correctly | 22:51 |
| tommylikehu | The difference there is if the input is {'metadata':{'iops': None} , then the original one will print 'metadata key is invalid', but comment one will print 'metadata key: iops is invalid'. why it could mess up the Oslo? | 22:55 |
| *** itlinux has joined #openstack-cinder | 23:01 | |
| *** sdague has quit IRC | 23:03 | |
| *** itlinux has quit IRC | 23:05 | |
| *** lamt has quit IRC | 23:06 | |
| *** kaisers_ has joined #openstack-cinder | 23:10 | |
| *** dhinesh has quit IRC | 23:10 | |
| *** kaisers_ has quit IRC | 23:14 | |
| *** jwcroppe has quit IRC | 23:18 | |
| *** arnewiebalck__ has joined #openstack-cinder | 23:35 | |
| *** Julien-zte has quit IRC | 23:37 | |
| *** arnewiebalck_ has quit IRC | 23:38 | |
| *** felipemonteiro has joined #openstack-cinder | 23:39 | |
| *** catintheroof has quit IRC | 23:40 | |
| *** nicolasbock has quit IRC | 23:40 | |
| *** salv-orlando has joined #openstack-cinder | 23:41 | |
| *** jwcroppe has joined #openstack-cinder | 23:43 | |
| *** reduxio has joined #openstack-cinder | 23:44 | |
| *** stakeda has joined #openstack-cinder | 23:44 | |
| *** reduxio has quit IRC | 23:48 | |
| *** catintheroof has joined #openstack-cinder | 23:48 | |
| openstackgerrit | Merged openstack/cinder master: Fix swift auth_url/url check https://review.openstack.org/464106 | 23:54 |
| openstackgerrit | Merged openstack/cinder master: Deprecate osapi_volume_base_url duplicate option https://review.openstack.org/462979 | 23:54 |
| *** Apoorva_ has joined #openstack-cinder | 23:54 | |
| openstackgerrit | Merged openstack/cinder master: NFS Backup: Fix overwritting backups https://review.openstack.org/465869 | 23:54 |
| *** diablo_rojo has quit IRC | 23:57 | |
| *** Apoorva has quit IRC | 23:57 | |
| *** Apoorva_ has quit IRC | 23:58 | |
| *** catintheroof has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!