| *** kaufer has quit IRC | 00:01 | |
| *** annashen has joined #openstack-cinder | 00:03 | |
| *** ebalduf has joined #openstack-cinder | 00:04 | |
| *** ho has joined #openstack-cinder | 00:04 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Namespace updates for _i18n and imageutils mods https://review.openstack.org/157956 | 00:05 |
|---|---|---|
| openstackgerrit | Patrick East proposed openstack/cinder: Add CHAP support to PureISCSIDriver https://review.openstack.org/137175 | 00:06 |
| openstackgerrit | Patrick East proposed openstack/cinder: Add DB table for driver specific data https://review.openstack.org/151837 | 00:06 |
| *** hemna is now known as hemnafk | 00:06 | |
| *** ebalduf has quit IRC | 00:08 | |
| *** akerr is now known as akerr_away | 00:13 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Remove unused options: iscsi_num_targets, iser_num_targets https://review.openstack.org/161036 | 00:14 |
| mtanino | anish: ^^ | 00:14 |
| *** akerr_away is now known as akerr | 00:16 | |
| *** patrickeast_ has joined #openstack-cinder | 00:19 | |
| *** Yogi1 has joined #openstack-cinder | 00:19 | |
| *** patrickeast__ has joined #openstack-cinder | 00:20 | |
| *** patri728 has joined #openstack-cinder | 00:20 | |
| *** patri586 has joined #openstack-cinder | 00:20 | |
| anish | mtanino: yep, looking at it now | 00:25 |
| mtanino | anish: thanks. I think that method is deadcode in your target. | 00:25 |
| *** markvoelker has quit IRC | 00:26 | |
| anish | mtanino: kinda. our limit is 1024 | 00:27 |
| *** lpabon has joined #openstack-cinder | 00:28 | |
| *** lpabon has quit IRC | 00:28 | |
| *** akerr has quit IRC | 00:28 | |
| mtanino | anish: _ensure_iscsi_targets in your target is never called. so I just removed it in this patch along with removing two config options. | 00:32 |
| anish | yes. I implemented it based on tgt code, which may not have been the correct approach | 00:36 |
| *** annashen has quit IRC | 00:38 | |
| anish | mtanino: I added the iser guys as well for some feedback | 00:42 |
| mtanino | anish: thank you. | 00:44 |
| *** tsekiyam_ has joined #openstack-cinder | 00:46 | |
| *** mtanino has quit IRC | 00:50 | |
| *** tsekiyama has quit IRC | 00:50 | |
| *** tsekiyam_ has quit IRC | 00:51 | |
| *** topshare has joined #openstack-cinder | 00:53 | |
| *** Yogi1 has quit IRC | 00:57 | |
| *** topshare_ has joined #openstack-cinder | 00:57 | |
| *** topshare has quit IRC | 01:00 | |
| *** tbarron has joined #openstack-cinder | 01:00 | |
| *** Mandell has quit IRC | 01:04 | |
| *** Mandell has joined #openstack-cinder | 01:05 | |
| *** ebalduf has joined #openstack-cinder | 01:05 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Switch to oslo_versionedobjects https://review.openstack.org/160209 | 01:06 |
| anish | jungleboyj: replied re https://review.openstack.org/#/c/161036/ | 01:07 |
| *** annegentle has quit IRC | 01:08 | |
| *** markvoelker has joined #openstack-cinder | 01:09 | |
| *** tbarron has quit IRC | 01:09 | |
| *** ebalduf has quit IRC | 01:09 | |
| *** Mandell has quit IRC | 01:09 | |
| *** tbarron has joined #openstack-cinder | 01:10 | |
| *** jaypipes has quit IRC | 01:11 | |
| jungleboyj | anish: Ok, can you update the commit message? | 01:12 |
| *** annegentle has joined #openstack-cinder | 01:12 | |
| *** markvoelker has quit IRC | 01:14 | |
| *** kmartin has quit IRC | 01:15 | |
| anish | jungleboyj: I can but I'll wait for mtanino to do it since it's his patchset | 01:15 |
| *** dannywilson has quit IRC | 01:16 | |
| jungleboyj | Ok. Sounds good. Thanks. | 01:16 |
| *** Mandell has joined #openstack-cinder | 01:16 | |
| *** davechen has joined #openstack-cinder | 01:17 | |
| *** Mandell has quit IRC | 01:17 | |
| *** Mandell has joined #openstack-cinder | 01:19 | |
| *** annegentle has quit IRC | 01:19 | |
| *** IanGovett1 has joined #openstack-cinder | 01:20 | |
| *** IanGovett has quit IRC | 01:21 | |
| *** Apoorva has quit IRC | 01:24 | |
| *** Apoorva has joined #openstack-cinder | 01:24 | |
| *** scottda_ has joined #openstack-cinder | 01:29 | |
| *** _cjones_ has quit IRC | 01:29 | |
| *** _cjones_ has joined #openstack-cinder | 01:32 | |
| *** scottda_ has quit IRC | 01:32 | |
| *** scottda_ has joined #openstack-cinder | 01:36 | |
| *** Mandell has quit IRC | 01:37 | |
| *** gugl3 has quit IRC | 01:38 | |
| *** cebruns has joined #openstack-cinder | 01:38 | |
| *** cdelatte has joined #openstack-cinder | 01:40 | |
| *** rcallawa has quit IRC | 01:41 | |
| *** scottda_ has quit IRC | 01:43 | |
| *** pradip_vedams has quit IRC | 01:47 | |
| *** pradip_vedams has joined #openstack-cinder | 01:47 | |
| *** vilobhmm has quit IRC | 01:48 | |
| openstackgerrit | harsh mishra proposed openstack/cinder: Fix for inconsistent cinder-services state change https://review.openstack.org/160104 | 01:49 |
| *** _cjones_ has quit IRC | 01:49 | |
| *** longboard has joined #openstack-cinder | 01:52 | |
| *** IanGovett1 has quit IRC | 01:55 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Remove unused options: iscsi_num_targets, iser_num_targets https://review.openstack.org/161036 | 01:55 |
| *** Mandell has joined #openstack-cinder | 01:58 | |
| *** jamielennox is now known as jamielennox|away | 01:59 | |
| *** leeantho has quit IRC | 01:59 | |
| *** Tross has quit IRC | 02:07 | |
| *** markvoelker has joined #openstack-cinder | 02:10 | |
| *** markvoelker has quit IRC | 02:15 | |
| *** patrickeast has quit IRC | 02:19 | |
| *** patrickeast__ has quit IRC | 02:20 | |
| *** patri728 has quit IRC | 02:20 | |
| *** patri586 has quit IRC | 02:20 | |
| *** patrickeast_ has quit IRC | 02:20 | |
| *** annegentle has joined #openstack-cinder | 02:20 | |
| *** annegentle has quit IRC | 02:25 | |
| *** markvoelker has joined #openstack-cinder | 02:27 | |
| *** Mandell has quit IRC | 02:29 | |
| *** markvoelker has quit IRC | 02:33 | |
| *** jamielennox|away is now known as jamielennox | 02:37 | |
| *** kaisers has joined #openstack-cinder | 02:40 | |
| *** kaisers1 has quit IRC | 02:42 | |
| *** longboard has quit IRC | 02:43 | |
| *** Longgeek has joined #openstack-cinder | 02:47 | |
| openstackgerrit | wanghao proposed openstack/cinder: Get volume from db again after driver upload image https://review.openstack.org/160781 | 02:50 |
| *** jamielennox is now known as jamielennox|away | 02:52 | |
| *** anish has quit IRC | 02:53 | |
| *** jamielennox|away is now known as jamielennox | 02:55 | |
| openstackgerrit | Jeegn Chen proposed openstack/cinder: More error handling on VNX migration failure. https://review.openstack.org/159704 | 02:56 |
| *** cdelatte has quit IRC | 02:56 | |
| *** jamielennox is now known as jamielennox|away | 02:56 | |
| *** jamielennox|away is now known as jamielennox | 02:57 | |
| *** annegentle has joined #openstack-cinder | 03:05 | |
| *** scottda_ has joined #openstack-cinder | 03:06 | |
| *** annegentle has quit IRC | 03:07 | |
| *** jcru has joined #openstack-cinder | 03:07 | |
| *** scottda_ has quit IRC | 03:09 | |
| *** BharatK has joined #openstack-cinder | 03:09 | |
| openstackgerrit | Dave Chen proposed openstack/python-cinderclient: Add commands for modifying image metadata https://review.openstack.org/147077 | 03:10 |
| *** Apoorva has quit IRC | 03:12 | |
| openstackgerrit | Dave Chen proposed openstack/python-cinderclient: Add commands for modifying image metadata https://review.openstack.org/147077 | 03:17 |
| *** scottda_ has joined #openstack-cinder | 03:17 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Switch get_all_snapshots to use objects https://review.openstack.org/161077 | 03:19 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Switch get_all_snapshots to use objects https://review.openstack.org/161077 | 03:20 |
| *** harlowja_ is now known as harlowja_away | 03:21 | |
| *** ebalduf has joined #openstack-cinder | 03:27 | |
| *** markvoelker has joined #openstack-cinder | 03:28 | |
| *** patrickeast has joined #openstack-cinder | 03:31 | |
| *** patrickeast has quit IRC | 03:33 | |
| *** markvoelker has quit IRC | 03:33 | |
| *** scottda_ has quit IRC | 03:38 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Switch get_all_snapshots to use objects https://review.openstack.org/161077 | 03:43 |
| openstackgerrit | Sasikanth Eda proposed openstack/cinder: Change default value of gpfs_images_share_mode to None https://review.openstack.org/161083 | 03:45 |
| *** changbl has joined #openstack-cinder | 03:48 | |
| *** tbarron has quit IRC | 03:53 | |
| *** vilobhmm has joined #openstack-cinder | 03:57 | |
| openstackgerrit | xing-yang proposed openstack/python-cinderclient: Create Consistency Group from CG Snapshot CLI https://review.openstack.org/148790 | 03:59 |
| *** haomaiwang has joined #openstack-cinder | 04:04 | |
| *** ho__ has joined #openstack-cinder | 04:06 | |
| *** ho has quit IRC | 04:06 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Make objects behave more like our old dictionaries https://review.openstack.org/160987 | 04:07 |
| *** rushil has joined #openstack-cinder | 04:12 | |
| *** dannywilson has joined #openstack-cinder | 04:13 | |
| *** pradipta has joined #openstack-cinder | 04:14 | |
| *** dannywilson has quit IRC | 04:17 | |
| *** rushil has quit IRC | 04:20 | |
| *** adam_g has quit IRC | 04:20 | |
| *** _cjones_ has joined #openstack-cinder | 04:20 | |
| *** _cjones_ has quit IRC | 04:22 | |
| *** kaufer has joined #openstack-cinder | 04:24 | |
| *** guitarzan has quit IRC | 04:24 | |
| *** kaufer has quit IRC | 04:26 | |
| *** kaufer has joined #openstack-cinder | 04:26 | |
| *** adam_g has joined #openstack-cinder | 04:28 | |
| *** markvoelker has joined #openstack-cinder | 04:29 | |
| *** mgagne has quit IRC | 04:31 | |
| *** ho has joined #openstack-cinder | 04:34 | |
| *** markvoelker has quit IRC | 04:34 | |
| *** ho__ has quit IRC | 04:35 | |
| openstackgerrit | harsh mishra proposed openstack/cinder: Fix for inconsistent cinder-services state change https://review.openstack.org/160104 | 04:35 |
| *** vilobhmm has quit IRC | 04:44 | |
| *** mgagne has joined #openstack-cinder | 04:45 | |
| *** mgagne is now known as Guest75711 | 04:45 | |
| *** anish has joined #openstack-cinder | 04:45 | |
| *** openstackgerrit has quit IRC | 04:46 | |
| *** _cjones_ has joined #openstack-cinder | 04:50 | |
| *** anuragpalsule has joined #openstack-cinder | 04:50 | |
| *** openstackgerrit has joined #openstack-cinder | 04:53 | |
| *** _cjones_ has quit IRC | 04:54 | |
| *** ebalduf has quit IRC | 04:57 | |
| *** bkopilov has joined #openstack-cinder | 05:05 | |
| *** vilobhmm has joined #openstack-cinder | 05:11 | |
| *** vilobhmm1 has joined #openstack-cinder | 05:16 | |
| openstackgerrit | Huang Zhiteng proposed openstack/cinder: Generic Volume Migration Support - Proof-of-Concept (DO NO MERGE) https://review.openstack.org/161094 | 05:19 |
| *** vilobhmm has quit IRC | 05:19 | |
| *** xyang has joined #openstack-cinder | 05:20 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add multiattach capabilities injection https://review.openstack.org/159650 | 05:23 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support https://review.openstack.org/85847 | 05:23 |
| *** Longgeek has quit IRC | 05:28 | |
| *** BharatK has quit IRC | 05:28 | |
| *** markvoelker has joined #openstack-cinder | 05:31 | |
| winston-d | jbernard: ping | 05:32 |
| *** jamielennox is now known as jamielennox|away | 05:32 | |
| winston-d | jbernard: FYI, I leave some comment in your generic volume migration support change, and put up one example here: https://review.openstack.org/#/c/161094/1. FYI. | 05:33 |
| winston-d | jbernard: sorry it took me so long to finally get a time to do this. hopefully it helps a bit. | 05:34 |
| *** openstack has joined #openstack-cinder | 05:36 | |
| *** esker has joined #openstack-cinder | 05:40 | |
| *** esker has quit IRC | 05:41 | |
| *** BharatK has joined #openstack-cinder | 05:45 | |
| *** jcru has quit IRC | 05:47 | |
| openstackgerrit | yatin proposed openstack/cinder: Added more options while uploading volume to glance https://review.openstack.org/152475 | 06:06 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/152401 | 06:07 |
| *** vilobhmm1 has quit IRC | 06:07 | |
| *** Longgeek has joined #openstack-cinder | 06:08 | |
| nikesh_vedams | hi | 06:09 |
| nikesh_vedams | want to know some information on manage/unmanage feature | 06:09 |
| nikesh_vedams | is there anyone online | 06:09 |
| nikesh_vedams | who knows manage/unmanage | 06:09 |
| *** eharney has quit IRC | 06:10 | |
| *** eharney has joined #openstack-cinder | 06:10 | |
| anish | you could proly take a look at https://review.openstack.org/#/c/138093/ https://review.openstack.org/#/c/155127/ | 06:11 |
| *** lpetrut has joined #openstack-cinder | 06:18 | |
| *** yamada-h has quit IRC | 06:21 | |
| *** yamada-h has joined #openstack-cinder | 06:23 | |
| *** topshare_ has quit IRC | 06:28 | |
| *** markvoelker has joined #openstack-cinder | 06:32 | |
| *** topshare has joined #openstack-cinder | 06:33 | |
| *** topshare_ has joined #openstack-cinder | 06:35 | |
| *** topshare has quit IRC | 06:38 | |
| *** markvoelker has quit IRC | 06:38 | |
| *** nkrinner has joined #openstack-cinder | 06:41 | |
| *** xyang has quit IRC | 06:48 | |
| *** kaufer has quit IRC | 06:50 | |
| *** nlevinki has joined #openstack-cinder | 06:50 | |
| *** deepakcs has joined #openstack-cinder | 07:05 | |
| openstackgerrit | Dave Chen proposed openstack/cinder: Role based properties protection https://review.openstack.org/160090 | 07:09 |
| openstackgerrit | Dave Chen proposed openstack/cinder: Policies based properties protection https://review.openstack.org/160145 | 07:09 |
| *** krishna_vedams has joined #openstack-cinder | 07:15 | |
| *** thrawn01 has quit IRC | 07:17 | |
| *** aniket has joined #openstack-cinder | 07:19 | |
| *** eikke has quit IRC | 07:19 | |
| *** dulek has joined #openstack-cinder | 07:21 | |
| *** Longgeek has quit IRC | 07:21 | |
| openstackgerrit | rajiv proposed openstack/python-cinderclient: cinderclient accepts arguments after metadata without -- separator https://review.openstack.org/156499 | 07:29 |
| *** david-lyle_afk has joined #openstack-cinder | 07:33 | |
| *** Longgeek has joined #openstack-cinder | 07:34 | |
| *** markvoelker has joined #openstack-cinder | 07:34 | |
| *** TobiasE has joined #openstack-cinder | 07:35 | |
| *** topshare_ has quit IRC | 07:39 | |
| *** markvoelker has quit IRC | 07:39 | |
| *** topshare has joined #openstack-cinder | 07:43 | |
| *** sgotliv has quit IRC | 07:43 | |
| *** Ilja has joined #openstack-cinder | 07:46 | |
| DuncanT | nikesh_vedams: What info do you want? and are you keeping good notes of what you're learning to improve our docs? | 07:47 |
| *** afazekas has quit IRC | 07:48 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Adapt volume manager to new driver structure https://review.openstack.org/138999 | 07:49 |
| *** openstackgerrit has quit IRC | 07:49 | |
| *** openstackgerrit has joined #openstack-cinder | 07:50 | |
| *** anshul has joined #openstack-cinder | 07:50 | |
| *** anshul has quit IRC | 07:50 | |
| *** anshul has joined #openstack-cinder | 07:51 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: A patch for test. Disregard it. https://review.openstack.org/160682 | 08:02 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: A patch for test. Disregard it. https://review.openstack.org/160682 | 08:04 |
| *** ronis has joined #openstack-cinder | 08:18 | |
| *** chlong has quit IRC | 08:20 | |
| *** e0ne has joined #openstack-cinder | 08:20 | |
| *** topshare has quit IRC | 08:29 | |
| -openstackstatus- NOTICE: Zuul check queue stuck due to reboot maintenance window at one of our cloud providers - no need to recheck changes at the moment, they won't move forward. | 08:29 | |
| *** ChanServ changes topic to "Zuul check queue stuck due to reboot maintenance window at one of our cloud providers - no need to recheck changes at the moment, they won't move forward." | 08:29 | |
| *** topshare has joined #openstack-cinder | 08:30 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: A patch for test. Disregard it. https://review.openstack.org/160682 | 08:33 |
| *** markvoelker has joined #openstack-cinder | 08:35 | |
| *** Miouge has joined #openstack-cinder | 08:38 | |
| *** pradipta has quit IRC | 08:39 | |
| *** markvoelker has quit IRC | 08:39 | |
| krishna_vedams | Hi | 08:40 |
| krishna_vedams | Recenlty we did a commit for blueprint "Add iSCSI SCST Target support to cinder" and merged | 08:41 |
| *** _afezekas|pub has joined #openstack-cinder | 08:42 | |
| *** e0ne has quit IRC | 08:42 | |
| *** alecv has joined #openstack-cinder | 08:42 | |
| *** karimb has joined #openstack-cinder | 08:42 | |
| krishna_vedams | After code merge happen, is there any thing remaining to do ? | 08:42 |
| krishna_vedams | can anybody please suggest me? | 08:43 |
| *** lpetrut has quit IRC | 08:43 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: A patch for test. Disregard it. https://review.openstack.org/160682 | 08:45 |
| *** jordanP has joined #openstack-cinder | 08:48 | |
| *** tshefi has quit IRC | 08:50 | |
| *** jistr has joined #openstack-cinder | 08:50 | |
| nikesh_vedams | DuncanT: yes i have notes | 08:52 |
| nikesh_vedams | DuncanT: my doubt about manage/unmanage is clear from this | 08:53 |
| nikesh_vedams | https://review.openstack.org/#/c/129652/ | 08:53 |
| DuncanT | nikesh_vedams: Improvements to the cinder wiki pages for developers are very welcome. These are hard to start for those of us who've been working on the code for years, since we don't know what info newcomers want. | 08:53 |
| DuncanT | nikesh_vedams: Looking at the patch now | 08:53 |
| nikesh_vedams | ok | 08:55 |
| DuncanT | nikesh_vedams: Ok, that patch adds a horizon interface to the manage/unmanage functionality that has been in cinder for some time. What are you unsure about? Unmanage causes cinder to forget about a volume, while leaving it on the backend storage. Manage imports a reference to an existing volume into cinder, where it can then be used with the full range of | 08:56 |
| DuncanT | cinder commands - clone, snap migrate, etc | 08:56 |
| *** kaisers1 has joined #openstack-cinder | 08:58 | |
| *** ho has quit IRC | 08:58 | |
| *** kaisers has quit IRC | 08:58 | |
| nikesh_vedams | DuncanT: after seeing this patch message its clear now,earlier before asking question i havenot seen this patch | 08:59 |
| DuncanT | nikesh_vedams: Ah, I see. Thanks. If anything useful can be added to the wiki that would have made figuring out each of the driver functions easier, please start the page and fill in the details you have. Myself and others will be glad to add more detail | 09:00 |
| nikesh_vedams | DuncanT: yes i will start,what is the procedure to change wiki | 09:00 |
| DuncanT | nikesh_vedams: Either find a page that starts to cover what you want and edit it to add more, or start a new page and link to it off the main cinder pages | 09:01 |
| DuncanT | nikesh_vedams: Anybody can edit, it keeps the history so we can tidy up mistakes | 09:01 |
| nikesh_vedams | ok | 09:01 |
| nikesh_vedams | thanks | 09:02 |
| *** nibalizer has quit IRC | 09:02 | |
| DuncanT | Wiki is down at the moment due to a maintance window at rackspace | 09:05 |
| DuncanT | Should be back shorty | 09:05 |
| DuncanT | *shortly | 09:05 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Test. https://review.openstack.org/161141 | 09:06 |
| *** openstack has joined #openstack-cinder | 15:28 | |
| -sendak.freenode.net- [freenode-info] channel trolls and no channel staff around to help? please check with freenode support: http://freenode.net/faq.shtml#gettinghelp | 15:28 | |
| *** scottda has joined #openstack-cinder | 15:33 | |
| *** scottda has quit IRC | 15:34 | |
| *** nlevinki has quit IRC | 15:36 | |
| *** kazum has joined #openstack-cinder | 15:39 | |
| *** kazum has quit IRC | 15:40 | |
| *** scottda has joined #openstack-cinder | 15:40 | |
| *** kazum has joined #openstack-cinder | 15:41 | |
| *** teruaki has joined #openstack-cinder | 15:41 | |
| *** rhedlind has quit IRC | 15:42 | |
| *** hemnafk is now known as hemna | 15:44 | |
| *** pradipm has quit IRC | 15:44 | |
| *** frankm has joined #openstack-cinder | 15:46 | |
| *** dulek has joined #openstack-cinder | 15:49 | |
| *** timcl has quit IRC | 15:49 | |
| *** dustins_ has quit IRC | 15:50 | |
| *** tbarron has quit IRC | 15:52 | |
| *** tbarron has joined #openstack-cinder | 15:52 | |
| *** esker has quit IRC | 15:55 | |
| *** tbarron has quit IRC | 15:55 | |
| *** tbarron has joined #openstack-cinder | 15:56 | |
| *** deepakcs has quit IRC | 15:56 | |
| *** bswartz has joined #openstack-cinder | 15:57 | |
| *** dustins has joined #openstack-cinder | 15:57 | |
| *** kmartin has joined #openstack-cinder | 15:58 | |
| *** tbarron has quit IRC | 15:58 | |
| *** tbarron has joined #openstack-cinder | 15:58 | |
| *** eikke has joined #openstack-cinder | 16:02 | |
| *** Bsony has quit IRC | 16:03 | |
| *** dustins has quit IRC | 16:04 | |
| *** tsekiyama has joined #openstack-cinder | 16:05 | |
| *** jungleboyj has joined #openstack-cinder | 16:12 | |
| j_king | anyone mind if I take https://bugs.launchpad.net/cinder/+bug/1298135 ? | 16:15 |
| openstack | Launchpad bug 1298135 in Cinder "Cinder should handle token expiration for long ops" [Medium,Confirmed] | 16:15 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Make lio iSCSI changes persistent to avoid lost https://review.openstack.org/160428 | 16:17 |
| *** timcl has joined #openstack-cinder | 16:18 | |
| e0ne | j_king: i think, you could just to assign this issue you yourself | 16:18 |
| *** annegentle has quit IRC | 16:21 | |
| *** annegentle has joined #openstack-cinder | 16:22 | |
| *** asselin__ has joined #openstack-cinder | 16:25 | |
| *** jaypipes has joined #openstack-cinder | 16:26 | |
| *** Ilja has quit IRC | 16:26 | |
| *** asselin_ has quit IRC | 16:29 | |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: SMBFS: Add minimum qemu-img version requirement https://review.openstack.org/161299 | 16:31 |
| *** dulek has quit IRC | 16:32 | |
| *** Ilja has joined #openstack-cinder | 16:37 | |
| *** Redcavalier has joined #openstack-cinder | 16:38 | |
| Redcavalier | Hi, I was suggested to come here, regarding this bug : https://bugs.launchpad.net/cinder/+bug/1348811 | 16:40 |
| openstack | Launchpad bug 1348811 in Cinder "RBD + Nova migration fails because imports an existing disk" [High,Confirmed] | 16:40 |
| Redcavalier | Basically, this is very critical for us to have fixed before we send our setup into production. I was wondering if there was any internal patch, anything that was not yet sent for code review? | 16:41 |
| *** jwcroppe has quit IRC | 16:41 | |
| *** jwcroppe has joined #openstack-cinder | 16:41 | |
| *** rwsu-afk is now known as rwsu | 16:42 | |
| Redcavalier | I know that jbernard said he may have a look, so I'm wondering if there is any update on that. | 16:42 |
| jbernard | Redcavalier: give me a few mins, cinder meeting is right now | 16:43 |
| *** julim has quit IRC | 16:44 | |
| *** julim has joined #openstack-cinder | 16:45 | |
| mtreinish | does someone want to +A: https://review.openstack.org/#/c/158020/ it already has 3 +2s? Or am I missing something? | 16:45 |
| *** jwcroppe has quit IRC | 16:46 | |
| *** _cjones_ has joined #openstack-cinder | 16:47 | |
| *** sgotliv has quit IRC | 16:51 | |
| *** Ilja has quit IRC | 16:51 | |
| *** Ilja has joined #openstack-cinder | 16:52 | |
| openstackgerrit | Rajini Ram proposed openstack/cinder: Convert all eqlx tests from mox to mock https://review.openstack.org/161310 | 16:53 |
| *** Yogi11 has quit IRC | 16:54 | |
| *** anshul has quit IRC | 16:54 | |
| *** asselin__ has quit IRC | 16:56 | |
| *** ronis__ has quit IRC | 16:56 | |
| *** Ilja1 has joined #openstack-cinder | 16:56 | |
| *** teruaki has left #openstack-cinder | 16:57 | |
| *** sgotliv has joined #openstack-cinder | 16:58 | |
| *** Ilja has quit IRC | 16:59 | |
| *** patrickeast has quit IRC | 16:59 | |
| jbernard | might it be better to raise volume migration at summit? | 17:00 |
| jbernard | Redcavalier: ok, hit me | 17:00 |
| jbernard | Redcavalier: so you're saying that just resizing an rbd-backed instance causes error | 17:01 |
| jungleboyj | jbernard: Yes, we will need to. | 17:01 |
| jungleboyj | winston-d: What did you need to talk about? | 17:01 |
| jbernard | jungleboyj: ok, ill submit a proposal | 17:01 |
| jbernard | jungleboyj: thanks | 17:01 |
| winston-d | This bug here: https://bugs.launchpad.net/cinder/+bug/1406703 | 17:01 |
| openstack | Launchpad bug 1406703 in Cinder "Deleting VM with an attached volume during copy-volume-to-image causes the volume remains in-use state" [Undecided,In progress] - Assigned to Abhijeet Malawade (abhijeet-malawade) | 17:01 |
| jungleboyj | jbernard: Migration in general needs to be discussed. | 17:01 |
| Redcavalier | jbernard, yup, because openstack tries to migrate it to another node. | 17:01 |
| jbernard | yeah, it's becoming clear that opinions are yet well formed | 17:02 |
| hemna | winston-d, I wonder if that bug goes away if my multiattach patch lands | 17:02 |
| hemna | winston-d, as we track individual attachments now | 17:02 |
| winston-d | Does anyone feel the same as I do about Cinder shouldn't allow uploading a 'in-use' vol to image? | 17:02 |
| jbernard | Redcavalier: when you say migrate, you're referring to the instance, right? | 17:02 |
| hemna | winston-d, +1 | 17:02 |
| DuncanT | winston-d: Yes, yes I do | 17:02 |
| jbernard | winston-d: +1 | 17:02 |
| jbernard | winston-d: unless there's some usecase im unable to visualize | 17:03 |
| hemna | winston-d, I'm not sure what the user expects in that case. data will be out of sync in the image | 17:03 |
| *** kazum has quit IRC | 17:03 | |
| Redcavalier | yea, the instance moving from compute 1 to compute 2. It looks like openstack tries to move its block device as if it was epehmeral storage on the computes disks. | 17:03 |
| winston-d | hemna: yeah, maybe, but for those backends doesn't support multiattach, doing such will corrupt data. | 17:03 |
| DuncanT | winston-d: That's a clear bug, same with backup - a slow copy of a live block device will almost always break stuff | 17:03 |
| hemna | winston-d, yah | 17:03 |
| eharney | winston-d: the driver could create a temporary snapshot and upload it from that... doesn't seem unreasonable to do at some point | 17:03 |
| *** dustins has joined #openstack-cinder | 17:03 | |
| *** dustins has quit IRC | 17:04 | |
| *** NarenNarendra has joined #openstack-cinder | 17:04 | |
| jbernard | Redcavalier: ok, ive not tested that particular case | 17:04 |
| DuncanT | eharney: Backup of snapshot is being looked at as a first class explicit operation in L. We could use the same code to do upload-of-snap I think | 17:04 |
| NarenNarendra | Hello Cinder team... | 17:04 |
| jbernard | Redcavalier: and the gate doesn't either, so that could have easily slipped through | 17:04 |
| winston-d | wait, so we are *OK* to allow upload-to-image for a 'in-use' vol, just want to make sure vol stats got to correct state like the bug mentioned? | 17:04 |
| winston-d | Or, we shouldn't do that util we have sth like upload-of-snap? | 17:05 |
| *** bswartz has quit IRC | 17:05 | |
| DuncanT | winston-d: We shouldn't allow upload of in-use, in my oppinion | 17:05 |
| *** tbarron has quit IRC | 17:06 | |
| winston-d | DuncanT: +2 | 17:06 |
| *** tbarron has joined #openstack-cinder | 17:06 | |
| DuncanT | winston-d: Implement upload-snap in L, explicit is better than implicit and the implicit snap is really easy to mess up in the driver | 17:06 |
| winston-d | DuncanT: agree with that too. | 17:06 |
| jbernard | Redcavalier: in general, shared storage is a relatively new concept to nova, so i have no doubt there are still some bugs ther | 17:06 |
| Redcavalier | jbernard, ok. For us it'S really important to be able to migrate/resize VMs before we move into production, so we're a bit stressed to have it working as soon as possible. | 17:07 |
| jbernard | Redcavalier: what you describe is technically a nova bug | 17:07 |
| winston-d | DuncanT, hemna, eharney, please chime in in bup report if you feel like it: https://bugs.launchpad.net/cinder/+bug/1406703 | 17:07 |
| openstack | Launchpad bug 1406703 in Cinder "Deleting VM with an attached volume during copy-volume-to-image causes the volume remains in-use state" [Undecided,In progress] - Assigned to Abhijeet Malawade (abhijeet-malawade) | 17:07 |
| hemna | winston-d, ok | 17:07 |
| winston-d | we can open another bug if needed. | 17:07 |
| winston-d | hemna: thx | 17:07 |
| aarefiev | jungleboyj: hi | 17:07 |
| jbernard | Redcavalier: but i can still try to reproduce and possilbly provide a fix if time allows | 17:07 |
| *** lpetrut has quit IRC | 17:08 | |
| NarenNarendra | Hi Cinder team! Pardon my perhaps novice IRC etiquette here... I am working with the OpenStack Product WorkGroup. We had a Kilo midcycle meetup back in January. One of the immediate tasks that we have picked is to speak with each of the project teams/PTLs on data gathering. Questions to address are: | 17:08 |
| NarenNarendra | 1. What are you delivering for Kilo? | 17:08 |
| NarenNarendra | 2. What do you plan on delivering for Liberty cycle? (we understand confidence is lower with time). | 17:08 |
| NarenNarendra | 3. What do you plan on delivering for M cycle? (we understand confidence is lower with time). | 17:08 |
| NarenNarendra | 4. How can the product WG help? | 17:08 |
| NarenNarendra | Would one of you be kind to address these? | 17:08 |
| davechen | thingee: hi, | 17:09 |
| davechen | thingee: are you around? | 17:09 |
| jungleboyj | aarefiev: What is up? | 17:10 |
| aarefiev | jungleboyj: do we have plans to switch to oslo.middleware in k | 17:10 |
| jungleboyj | The library instead of the olso-incubator? | 17:10 |
| *** lpetrut has joined #openstack-cinder | 17:11 | |
| aarefiev | yep | 17:11 |
| winston-d | jbernard: hey, about the example here: https://review.openstack.org/#/c/161094/1 what's your concern? | 17:11 |
| hemna | aarefiev, I hope not at this point. | 17:11 |
| aarefiev | jungleboyj: e0ne asked me to make patch for grenade to update paste api | 17:11 |
| jbernard | winston-d: honestly i like it | 17:11 |
| jungleboyj | aarefiev: We already have the patch out there for that? | 17:11 |
| winston-d | jbernard: if you still time to look at it, i'm fine, we can discuss later | 17:12 |
| jbernard | winston-d: i just want to make sure there's some agreement, so im not losing time | 17:12 |
| jungleboyj | hemna: That one is a smaller impact that the scheduler. | 17:12 |
| winston-d | jbernard: i'll try to get avishay to look at it, and hopefully jgriffith | 17:12 |
| hemna | jungleboyj, url ? | 17:12 |
| jungleboyj | Trying to find it. | 17:12 |
| aarefiev | jungleboyj: not yet | 17:12 |
| *** e0ne is now known as e0ne_ | 17:12 | |
| *** e0ne_ is now known as e0ne | 17:12 | |
| jbernard | winston-d: that would be great, thanks for that | 17:13 |
| DuncanT | NarenNarendra: One think I think we could do with guidance/help on is what features of ours are actually being used in the wild, rather than what vendors tell us. A recent post on the operators list suggests most operators don't understand a huge chunk of our features | 17:13 |
| hemna | jungleboyj, honestly, I'm a bit worried with lots of oslo_ migrations this late in the game. | 17:13 |
| jungleboyj | hemna: https://review.openstack.org/#/c/134759/ | 17:13 |
| hemna | just due to past experience | 17:13 |
| DuncanT | NarenNarendra: That can go under question (4) | 17:13 |
| *** jistr has quit IRC | 17:13 | |
| winston-d | DuncanT: good point | 17:13 |
| DuncanT | NarenNarendra: I suspect the answer to (3) is going to be pretty much unknown, given past experience | 17:13 |
| jungleboyj | hemna: Agreed, but I believe we have kept middleware up to date. This is just using the library version instead. | 17:14 |
| hemna | jungleboyj, it's not passing jenkins at the moment. | 17:14 |
| hemna | :( | 17:14 |
| hemna | no warm and fuzzies | 17:14 |
| jungleboyj | hemna: Yes, that is what aarefiev is trying to fix. | 17:14 |
| -openstackstatus- NOTICE: Issue solved, gate slowly digesting accumulated changes | 17:14 | |
| winston-d | jbernard: i'm pretty confident avishay is(will be) on board, I need to hear feedback from jgriffith | 17:14 |
| DuncanT | NarenNarendra: Thingee should be able to give you an answer to (1) and (2), if you haven't heard something soon I'll try to throw together a summary of what I know of | 17:14 |
| *** alecv has quit IRC | 17:14 | |
| jungleboyj | hemna: It is a grenade problem with package versions. | 17:15 |
| jbernard | winston-d: kk, ill get to work | 17:15 |
| hemna | winston-d, are you ok with xyang's feedback on the multiattach injection ? | 17:15 |
| NarenNarendra | DuncanT: thanks a bunch! | 17:15 |
| *** davechen has quit IRC | 17:15 | |
| jbernard | winston-d: should i try to hit the k3 cutoff, or is that unrealisitic? | 17:15 |
| hemna | I'd really like to get some other folks testing the multiattach patch to get it to land this week if possible. | 17:15 |
| hemna | https://review.openstack.org/#/c/85847/ | 17:15 |
| NarenNarendra | DuncanT: pls try to answer 1 and 2 also | 17:15 |
| jungleboyj | hemna: I agree the scheduler change is dangerous, but this one I am less concerned about. | 17:16 |
| NarenNarendra | DuncanT: I have tried to reach out Thingee for the last couple of weeks over multiple channels... it looks like he may to too busy rounding up things for Kilo | 17:16 |
| hemna | winston-d, https://review.openstack.org/#/c/159650/ | 17:16 |
| hemna | that one | 17:16 |
| DuncanT | NarenNarendra: Actually, it might be best to start an etherpad and let others chime in too. https://etherpad.openstack.org/p/cinder-cycle-features | 17:17 |
| winston-d | jbernard: i'll try my best to help. honestly i feel guilty about not providing enough help on this for past few months, it should be landed already if i had more time to look closer to it. | 17:17 |
| jungleboyj | aarefiev: So, if you can get the patch passing Jenkins that would be greatly apprecaited. | 17:17 |
| winston-d | hemna: not really, i would like to talk to her if possible. | 17:18 |
| hemna | winston-d, ok | 17:18 |
| winston-d | xyang: around? | 17:18 |
| DuncanT | NarenNarendra: I'll try to fill in what I can over the next few hours (I need to drop offline for an hour or so momentarily), and I'll encourage others to chime in too | 17:18 |
| hemna | she seemed it was necessary | 17:18 |
| hemna | or desired | 17:18 |
| NarenNarendra | DuncanT: that will also work great! can you pls help me circulate that with the Cinder team? I can watch the etherpad for updates. Perhaps, give a deadline of one week from now for all updates? | 17:18 |
| *** nellysmitt has quit IRC | 17:18 | |
| NarenNarendra | DuncanT: thanks so much! I appreciate it a lot | 17:19 |
| aarefiev | jungleboyj: I think there will the problem, patches will block each other | 17:19 |
| winston-d | hemna: the injection is only needed when a volume is created with 'multiattach' flag in vol metadata, while not in type extra specs, right. | 17:19 |
| DuncanT | NarenNarendra: That seems reasonable. A week and a day means we can bring it up at the next weekly meeting for any final details, plus we'll be after code feature freeze so the answers will be more reliable | 17:19 |
| hemna | the multiattach flag is on the volume itself | 17:19 |
| *** jwcroppe has joined #openstack-cinder | 17:20 | |
| aarefiev | jungleboyj: my fix and switch to oslo lib | 17:20 |
| winston-d | hemna: but that logic we are talking about is for creating a consistency group | 17:20 |
| NarenNarendra | DuncanT: thats fine.. until next Friday, 03/13 is also fine | 17:20 |
| NarenNarendra | DuncanT: you can bring this up in the next project mtg on next wed too | 17:20 |
| winston-d | hemna: yes, but the flag was persistent into cinder as volume metadata, right? | 17:20 |
| DuncanT | NarenNarendra: Excellent. Keep an eye on that etherpad | 17:20 |
| NarenNarendra | DuncanT: awesome awesome! thanks so much! | 17:20 |
| jungleboyj | aarefiev: Can you add it to https://review.openstack.org/#/c/134759/ ? | 17:20 |
| hemna | technically it's not in volume metadata, but part of the volume itself. | 17:21 |
| jungleboyj | So that they don't block each other. | 17:21 |
| hemna | winston-d, https://review.openstack.org/#/c/85847/41/cinder/db/sqlalchemy/migrate_repo/versions/038_add_volume_attachment.py | 17:21 |
| winston-d | hemna: oh? where do we save it to? | 17:21 |
| hemna | like 40 | 17:22 |
| aarefiev | jungleboyj: how do that? there are in diff projects | 17:22 |
| hemna | line 40 | 17:22 |
| hemna | it's a column in the volume table | 17:22 |
| DuncanT | Right, I'm off for a while, back later. Leave a message after the tone and all that... | 17:22 |
| hemna | beep | 17:22 |
| winston-d | hemna: ok, so a new column in vol table. that's fine. | 17:23 |
| hemna | yuh | 17:23 |
| *** e0ne is now known as e0ne_ | 17:23 | |
| winston-d | hemna: but we don't have a new column for ConsistencyGroup table, right? | 17:23 |
| hemna | so the _get_weighted_candidates_group is for scheduling/filtering on CG create ? | 17:23 |
| jungleboyj | aarefiev: Didn't understand that. | 17:23 |
| hemna | winston-d, correct | 17:23 |
| winston-d | hemna: and ConsistencyGroup doesn't have metadata table. | 17:24 |
| jungleboyj | aarefiev: There must be a way to resolve this as others have dealt with this problem in Grenade before. | 17:24 |
| aarefiev | jungleboyj: me too) | 17:24 |
| winston-d | hemna: so my point is, unless there is a way to tell Cinder, please create a CG that has a multiattach flag, we don't need that logic in scheduler. | 17:24 |
| aarefiev | jungleboyj: I don't know such way | 17:24 |
| *** jwcroppe_ has joined #openstack-cinder | 17:25 | |
| *** jwcroppe has quit IRC | 17:25 | |
| jungleboyj | aarefiev: Have you tried asking on the openstack-infra channel? | 17:25 |
| hemna | ok I can remove it | 17:25 |
| hemna | xyang, ? | 17:25 |
| aarefiev | jungleboyj: no | 17:25 |
| jungleboyj | aarefiev: I would try asking there. I thought e0ne knew how to resolve this based on our last discussion. | 17:26 |
| winston-d | hemna: the only way to do that, now, is to create a CG with a type, that has 'multiattach' flag in extra specs. and that has already be taken care of by scheduler itself, no extra logic is needed. | 17:26 |
| hemna | ok yah that makes sense | 17:26 |
| hemna | I'll put up another patch removing it. | 17:26 |
| hemna | thanks | 17:26 |
| aarefiev | jungleboyj: thanks | 17:27 |
| *** sgotliv has quit IRC | 17:27 | |
| winston-d | hemna: sure | 17:27 |
| *** sgotliv has joined #openstack-cinder | 17:27 | |
| winston-d | xyang: hopefully you can see the discussion above and will be able to ack when you are available | 17:27 |
| *** jwcroppe_ has quit IRC | 17:28 | |
| jungleboyj | aarefiev: Thank you for looking into it. | 17:28 |
| *** aarefiev has quit IRC | 17:28 | |
| *** jwcroppe has joined #openstack-cinder | 17:28 | |
| nikesh_vedams | hi | 17:28 |
| *** aarefiev has joined #openstack-cinder | 17:28 | |
| *** aarefiev has quit IRC | 17:29 | |
| nikesh_vedams | what is unit of size returned by manage_existing_get_size | 17:29 |
| *** Apoorva has joined #openstack-cinder | 17:29 | |
| *** aarefiev has joined #openstack-cinder | 17:29 | |
| xyang | winston-d: hi | 17:32 |
| *** aarefiev has quit IRC | 17:32 | |
| *** aarefiev has joined #openstack-cinder | 17:32 | |
| xyang | winston-d: did you see my comments in the patch? I think that is also needed for CG because scheduler needs to choose one backend that can serve all volume types | 17:32 |
| winston-d | xyang: yeah I saw that. | 17:33 |
| winston-d | xyang: if multiattach is part of type extra spec, we don't need such logic in scheduler. | 17:33 |
| xyang | winston-d: you are saying it will take care of itself? | 17:34 |
| xyang | winston-d: basically we want to be able to find a backend that support multiattach for CG as well | 17:34 |
| winston-d | xyang: your extra logic for consistency group is there only because one can request creating a CG with a type that doesn't have 'cg=supported' in extra spec | 17:35 |
| xyang | winston-d: ok, that is true. so why do we need extra logic for create volume with multiattach? | 17:36 |
| winston-d | xyang: now, the create CG request itself can be considered as the 'cg=supported' flag, so filter scheduler needs to inject a flag to extra spec if type definiton doesn't have it. | 17:36 |
| winston-d | xyang: for creating a CG with multiattach flag, how should the user specify multiattach flag for this CG, if not using type extra spec? | 17:38 |
| *** narenhn has joined #openstack-cinder | 17:38 | |
| *** ndipanov has quit IRC | 17:38 | |
| xyang | winston-d: yes, should be same as create volume. so if you want the volume created to support multiattach, you need to add that to extra spec | 17:38 |
| *** leeantho has joined #openstack-cinder | 17:38 | |
| xyang | winston-d: that's why I'm confused why that is not needed in that code path | 17:39 |
| winston-d | xyang: if that's possible, then we need the same logic as cg=supported injected here. otherwise, scheduler takes care of type extra spec | 17:39 |
| *** NarenNarendra has quit IRC | 17:40 | |
| winston-d | xyang: no, in hemna's change, you can create a volume with multiattach flag *WITHOUT* usng a type that has multiattach=true in extra spec | 17:40 |
| xyang | winston-d: so it seems to me if that logic is needed for create volume, then it should be needed for create CG | 17:40 |
| xyang | winston-d: ok, maybe I'm confused with the need of that patch | 17:41 |
| *** e0ne has joined #openstack-cinder | 17:41 | |
| *** e0ne has quit IRC | 17:41 | |
| winston-d | xyang: question is, can i create a CG with multiattach flag *WITHOUT* multiattach=true in type extra spec? | 17:41 |
| winston-d | xyang: yes, or no? | 17:42 |
| xyang | winston-d: I think you do have to pass that info in extra spec. | 17:43 |
| xyang | winston-d: we don't have a flag to pass to CG directly | 17:44 |
| winston-d | xyang: then scheduler takes care of that already. | 17:44 |
| *** afazekas has quit IRC | 17:44 | |
| winston-d | xyang: do you remember why you add that extra logic for CG in filter scheduler? | 17:45 |
| winston-d | same idea | 17:45 |
| xyang | winston-d: yes, that is for the CG create command to succeed without that in extra specs | 17:45 |
| winston-d | xyang: exactly, CG create command itself present a need to inject the flag, without every type having to be updated. | 17:46 |
| *** patrickeast has joined #openstack-cinder | 17:47 | |
| winston-d | xyang: so you agree that hemna doesn't need that logic in get_weighted_cg_candidates() for multiattach? | 17:47 |
| hemna | I have the patch ready to submit :P | 17:48 |
| hemna | just give me the word | 17:48 |
| xyang | winston-d: so for CG to pick a backend that supports multiattach, it requires multiattach to be in extra specs, I mean user has to define it | 17:48 |
| winston-d | xyang: yes | 17:49 |
| *** akerr is now known as akerr_away | 17:51 | |
| nikesh_vedams | xyang : hi | 17:51 |
| xyang | nikesh_vedams: hi | 17:52 |
| hemna | xyang, so are we good? | 17:53 |
| hemna | can I submit my updated patchset ? | 17:53 |
| *** anshul has joined #openstack-cinder | 17:53 | |
| nikesh_vedams | xyang: we are working on CI,but one merged code is breaking our scst merged patch we have reported bug in this patch https://review.openstack.org/#/c/161143/ | 17:53 |
| *** akerr_away is now known as akerr | 17:55 | |
| *** afazekas has joined #openstack-cinder | 17:57 | |
| *** patrickeast_ has joined #openstack-cinder | 17:58 | |
| *** patrickeast__ has joined #openstack-cinder | 17:58 | |
| *** patri251 has joined #openstack-cinder | 17:58 | |
| *** patri498 has joined #openstack-cinder | 17:58 | |
| hemna | xyang, ? | 17:59 |
| *** pschaef has quit IRC | 18:00 | |
| *** erlon has joined #openstack-cinder | 18:00 | |
| xyang | hemna: hi, I think it is fine. looks like a doc item now | 18:01 |
| hemna | ok thanks, I'll push the review up | 18:01 |
| *** patri461 has joined #openstack-cinder | 18:01 | |
| xyang | hemna: so when you create a volume, you don't need that in extra specs, but when you create CG, you need it | 18:01 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add multiattach capabilities injection https://review.openstack.org/159650 | 18:01 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support https://review.openstack.org/85847 | 18:01 |
| *** patri582 has joined #openstack-cinder | 18:01 | |
| nikesh_vedams | jgriffith: there? | 18:02 |
| *** annashen has joined #openstack-cinder | 18:03 | |
| openstackgerrit | Rajini Ram proposed openstack/cinder: Convert all eqlx tests from mox to mock https://review.openstack.org/161310 | 18:03 |
| *** karimb has quit IRC | 18:04 | |
| *** winston-d is now known as winston-d_zZZ | 18:04 | |
| nikesh_vedams | xyang: thanks,is anyone else online for one more +2 :) | 18:05 |
| *** Yogi11 has joined #openstack-cinder | 18:10 | |
| *** Mandell has joined #openstack-cinder | 18:14 | |
| *** rushil has joined #openstack-cinder | 18:15 | |
| *** leeantho has quit IRC | 18:17 | |
| *** narenhn has quit IRC | 18:19 | |
| *** NarenNarendra has joined #openstack-cinder | 18:19 | |
| *** bswartz has joined #openstack-cinder | 18:19 | |
| *** NarenNarendra has quit IRC | 18:19 | |
| *** harlowja_away is now known as harlowja_ | 18:19 | |
| *** jordanP has quit IRC | 18:20 | |
| *** e0ne has joined #openstack-cinder | 18:22 | |
| *** vilobhmm has joined #openstack-cinder | 18:24 | |
| *** Yogi11 has quit IRC | 18:25 | |
| *** vilobhmm1 has joined #openstack-cinder | 18:25 | |
| *** leeantho has joined #openstack-cinder | 18:25 | |
| *** Yogi1 has joined #openstack-cinder | 18:26 | |
| *** leeantho has quit IRC | 18:26 | |
| *** diemt has joined #openstack-cinder | 18:26 | |
| nikesh_vedams | if i am correct,size of cinder volume is shown in integer GBs,so in manage_existing_get_size if size of volume is say 4.7 GB,what should we return | 18:26 |
| *** leeantho has joined #openstack-cinder | 18:26 | |
| nikesh_vedams | 4 or 5 | 18:27 |
| *** vilobhmm has quit IRC | 18:29 | |
| *** afazekas has quit IRC | 18:31 | |
| *** ronis__ has joined #openstack-cinder | 18:32 | |
| mgagne | jgriffith: does cinder handle xExceededLimit exception from SolidFire? =) | 18:33 |
| jbernard | nikesh_vedams: 5 would be my guess | 18:33 |
| mgagne | jgriffith: my point is that there is no logs in cinder-volume about the lack of capacity. But I'm not sure if there should be any. | 18:33 |
| *** thingee has joined #openstack-cinder | 18:36 | |
| thingee | leeantho: going to check out the docs today for https://review.openstack.org/152325 | 18:40 |
| thingee | after that will you remove the -1 workflow? | 18:40 |
| leeantho | thingee, yes once I get some feedback on the content I can remove the -1 workflow tag | 18:41 |
| nikesh_vedams | thingee: morning | 18:43 |
| jungleboyj | thingee: When you have a chance, can you remove your -2 from here given that the Nova side went in: https://review.openstack.org/#/c/149256/ Thanks. | 18:44 |
| thingee | jungleboyj: done | 18:47 |
| jungleboyj | thingee: Thank you! | 18:47 |
| *** sgotliv has quit IRC | 18:48 | |
| jungleboyj | hemna: Can you also take a look ^^ | 18:48 |
| hemna | jungleboyj, ok I gave my feedback | 18:49 |
| hemna | the system check ifs kinda suck | 18:50 |
| hemna | I think it'd be better to have a way of automatically building a platform specific targeted connector | 18:50 |
| jungleboyj | hemna: Thanks for the feedback. | 18:50 |
| hemna | so we don't start pooping if systemX, elseif systemY, etc etc | 18:50 |
| jungleboyj | hemna: Yeah, but that isn't something that is going to happen for Kilo. | 18:51 |
| hemna | nope | 18:51 |
| hemna | on the upside, I think everyone uses the factory to build the correct Connector | 18:51 |
| hemna | we could modify the factory to build the right one for the right platform | 18:52 |
| hemna | but at a minimum the os-brick patch needs to exist as well. | 18:52 |
| jungleboyj | hemna: Do you want it included in that same patch>? | 18:52 |
| *** fischerw has joined #openstack-cinder | 18:53 | |
| hemna | jungleboyj, and if you see any other reviews that touch cinder/brick/initiator, the same changes need to exist in os-brick as well. | 18:53 |
| hemna | jungleboyj, I don't think you can, because os-brick is a separate repository | 18:53 |
| jungleboyj | hemna: I realized that as I was typing that. | 18:53 |
| hemna | :) | 18:53 |
| jungleboyj | hemna: So here: https://github.com/openstack/os-brick | 18:53 |
| hemna | yup | 18:54 |
| *** alecv has joined #openstack-cinder | 18:54 | |
| jungleboyj | Ok. Cool. | 18:54 |
| *** xyang has quit IRC | 18:55 | |
| jungleboyj | You approve this one if we get the other one pushed up as well. | 18:55 |
| *** xyang has joined #openstack-cinder | 18:55 | |
| *** dustins_ has joined #openstack-cinder | 18:57 | |
| *** dustins_ is now known as dustins | 18:58 | |
| hemna | I don't think it would be too hard to make a platform specific Connector object | 18:59 |
| hemna | and just test for system.machine() in the factory | 18:59 |
| hemna | I think that's a bit cleaner | 18:59 |
| nikesh_vedams | looks like ietadm also missing in https://review.openstack.org/#/c/161143/2/ | 19:00 |
| hemna | in fact I think the changes would be easy to do | 19:00 |
| hemna | tiny refactoring in the FibreChannelConnector to add a hook for a child class to extend | 19:01 |
| *** alexpilotti_ has joined #openstack-cinder | 19:01 | |
| hemna | in connect_volume and disconnect_volume() | 19:01 |
| hemna | and a test in the factory to build the correct platform specific Connector | 19:01 |
| hemna | done | 19:01 |
| jungleboyj | hemna: Ok, if you think that is cleaner ... do you want to propose that in the review then? | 19:02 |
| *** timcl1 has joined #openstack-cinder | 19:02 | |
| hemna | yah | 19:02 |
| hemna | I was about to add comments about it in the review | 19:02 |
| nikesh_vedams | what should i use in help of iscsi_helper for ietadm for https://review.openstack.org/#/c/161143/2/ | 19:02 |
| hemna | give me a few | 19:02 |
| jungleboyj | hemna: Great, I will push them and try to get that turned around. | 19:03 |
| *** alexpilotti has quit IRC | 19:03 | |
| *** alexpilotti_ is now known as alexpilotti | 19:03 | |
| *** bswartz has quit IRC | 19:05 | |
| *** e0ne is now known as e0ne_ | 19:05 | |
| *** e0ne_ has quit IRC | 19:05 | |
| *** timcl has quit IRC | 19:05 | |
| *** rushil has quit IRC | 19:06 | |
| hemna | ok hopefully that's clear. | 19:07 |
| *** dannywilson has joined #openstack-cinder | 19:07 | |
| *** bswartz has joined #openstack-cinder | 19:09 | |
| *** dannywilson has quit IRC | 19:09 | |
| *** dannywilson has joined #openstack-cinder | 19:10 | |
| *** dannywilson has quit IRC | 19:10 | |
| *** dannywilson has joined #openstack-cinder | 19:11 | |
| jungleboyj | hemna: Let me look. | 19:12 |
| *** lpabon has joined #openstack-cinder | 19:15 | |
| *** ChanServ changes topic to "The OpenStack Block Storage Project Cinder | The New Kids On the Block | https://wiki.openstack.org/wiki/Cinder" | 19:15 | |
| *** e0ne has joined #openstack-cinder | 19:16 | |
| *** e0ne is now known as e0ne_ | 19:16 | |
| mtanino | nikesh_vedams: Please add this "ietadm for iSCSI Enterprise Target" | 19:16 |
| *** Ilja1 has quit IRC | 19:18 | |
| *** Mandell has quit IRC | 19:19 | |
| jungleboyj | e0ne_: Did aarefiev get hold of you? | 19:19 |
| *** e0ne_ is now known as e0ne | 19:20 | |
| e0ne | jungleboyj: not sure that i anderstood correctly your message | 19:21 |
| openstackgerrit | Billy Olsen proposed openstack/cinder: Refactoring attach volume api to use taskflow https://review.openstack.org/148922 | 19:21 |
| * e0ne needs to improve english | 19:21 | |
| jungleboyj | e0ne: aarefiev Was asking me about issues getting the patch for paste.api into Grenade to fix the oslo.messaging issue. | 19:22 |
| e0ne | jungleboyj: yep. i read chat history | 19:22 |
| jungleboyj | e0ne: He was concerned about how the two can be checked in when they are in different repositories but he thought they depended upon each oother. | 19:22 |
| jungleboyj | e0ne: Ok. | 19:22 |
| *** nkrinner has joined #openstack-cinder | 19:23 | |
| e0ne | jungleboyj: good question. unfortunately, i don't know the answer yet | 19:23 |
| *** Bsony has joined #openstack-cinder | 19:24 | |
| jungleboyj | e0ne: Ok, I didn't either . Referred him to the the infra channel. | 19:24 |
| *** nkrinner has quit IRC | 19:24 | |
| *** nkrinner has joined #openstack-cinder | 19:24 | |
| *** BharatK has quit IRC | 19:24 | |
| *** aix has quit IRC | 19:24 | |
| e0ne | jungleboyj: agree. we defenatly need to ask infra team | 19:24 |
| jungleboyj | e0ne: Ok, good, I sent him the right direction. | 19:26 |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: Fix fetch_to_volume_format if vhd is requested https://review.openstack.org/161363 | 19:26 |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: Move vhd/x resize logic to image utils https://review.openstack.org/161364 | 19:26 |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: Fix qemu-img info vhd format issue https://review.openstack.org/161365 | 19:26 |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: Add image subformat option to fetch_to_volume_format function https://review.openstack.org/161366 | 19:26 |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: Add support for caching images fetched from Glance https://review.openstack.org/161367 | 19:26 |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: SMBFS: Add image caching support https://review.openstack.org/161368 | 19:26 |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: Windows iSCSI: Add image caching support https://review.openstack.org/161369 | 19:26 |
| *** angela-s has joined #openstack-cinder | 19:28 | |
| *** Mandell has joined #openstack-cinder | 19:28 | |
| jgriffith | mgagne: it does not | 19:31 |
| jgriffith | mgagne: that *should* get proopogated if hit, but I'd have to look and verify | 19:32 |
| mgagne | jgriffith: I got a No valid host found error though | 19:32 |
| jgriffith | mgagne: yeah, that's what I'd expect | 19:32 |
| jgriffith | mgagne: I'd expect the sched to say "no" before it got to us | 19:33 |
| jgriffith | mgagne: but that's likely not right either, until something is done about effective etc | 19:33 |
| *** lpetrut has quit IRC | 19:33 | |
| mgagne | jgriffith: oh, is SF reporting volume capacity left? | 19:33 |
| jgriffith | mgagne: indeed | 19:33 |
| e0ne | jungleboyj: i've got some converation in #openstack-infra | 19:33 |
| jgriffith | mgagne: free_capacity | 19:34 |
| jungleboyj | e0ne: Ok. Need me to join? | 19:34 |
| e0ne | jungleboyj: no | 19:34 |
| jungleboyj | e0ne: Ok. | 19:34 |
| e0ne | jungleboyj: it's very easy | 19:34 |
| mgagne | jgriffith: this could explain why cinder-volume didn't log it as it's a "normal" behavior: no space left, go somewhere else. | 19:34 |
| jungleboyj | e0ne: Sweet. | 19:34 |
| jgriffith | mgagne: exactly... "piss off" :) | 19:34 |
| jgriffith | mgagne: Dave's not here | 19:34 |
| e0ne | jungleboyj, aarefiev: please, follow this guideline for your fix to grenade http://docs.openstack.org/infra/manual/developers.html#cross-project-dependencies | 19:35 |
| *** ebalduf has quit IRC | 19:35 | |
| e0ne | jungleboyj: one more noob question from me | 19:37 |
| jungleboyj | e0ne: Thanks for finding that. | 19:37 |
| jungleboyj | e0ne: Noob? You aren't a noob. ;-) | 19:37 |
| *** sgotliv has joined #openstack-cinder | 19:37 | |
| e0ne | jungleboyj: what is the right way to add a note for https://wiki.openstack.org/wiki/ReleaseNotes/Kilo? just edit a wiki page or we ve got some repo gor it? | 19:37 |
| jungleboyj | e0ne: That is a good question. I should know that but don't remember the answer. jgriffith Should know. | 19:39 |
| e0ne | :) | 19:39 |
| e0ne | need to go sleep | 19:39 |
| thingee | jgriffith: I've asked you a few times about this now and feel like you're ignoring me. Can you please either remove your -2 or respond to this https://review.openstack.org/#/c/135002/ | 19:39 |
| e0ne | i failed and scheduled car service for 9am for tomorrow :( | 19:39 |
| mgagne | e0ne: on a side note, that's a problem if it's auto-generated from a repo and there is no note about that fact | 19:39 |
| jungleboyj | e0ne: :-( Hope it isn't major service. | 19:40 |
| e0ne | jungleboyj: i just want to install fog lights. but it's tooo early for me | 19:40 |
| jungleboyj | e0ne: Oh, good, fun service. | 19:41 |
| jungleboyj | thingee: Do you know how to add to the release notes? | 19:42 |
| thingee | jungleboyj: it's usually a wiki page. You just edit it | 19:42 |
| e0ne | bb tomorrow. have a good day! | 19:42 |
| jungleboyj | e0ne: Have a good night. | 19:42 |
| * jungleboyj face palms | 19:43 | |
| hemna | doh | 19:43 |
| jungleboyj | Yeah, we can edit that. Thanks thingee | 19:43 |
| jungleboyj | Next I ask hemna how to configure the storwize driver. | 19:43 |
| nikesh_vedams | mtanino: thanks | 19:44 |
| openstackgerrit | nikeshmahalka proposed openstack/cinder: Choices for iscsi_helper does not contain scstadmin. https://review.openstack.org/161143 | 19:44 |
| mtanino | nikesh_vedams: sure. | 19:45 |
| hemna | jungleboyj, volume_driver=cinder.volume.drivers.san.hp.hp_3par_fc.HP3PARFCDriver | 19:45 |
| hemna | :P | 19:45 |
| jungleboyj | hemna: Sweet, thanks. I will pass that along to our customer .... hey wait a minute ... | 19:46 |
| hemna | :P | 19:46 |
| hemna | why does my driver not work ?! | 19:46 |
| jungleboyj | :-) | 19:46 |
| nikesh_vedams | mtanino: is it good now | 19:46 |
| e0ne | :) | 19:47 |
| *** r-daneel has joined #openstack-cinder | 19:47 | |
| nikesh_vedams | jgriffith : morning | 19:48 |
| *** Bsony has quit IRC | 19:48 | |
| nikesh_vedams | thingee : morning | 19:48 |
| thingee | nikesh_vedams: tip, just ask for what you need instead of pinging me. | 19:49 |
| thingee | nikesh_vedams: I don't respond to pings | 19:49 |
| thingee | usually | 19:49 |
| *** Ilja has joined #openstack-cinder | 19:49 | |
| *** anshul has quit IRC | 19:51 | |
| *** crose has joined #openstack-cinder | 19:52 | |
| hemna | icmp blocked. | 19:52 |
| nikesh_vedams | thingee:sure,just to inform that we are working on CI for scst and https://review.openstack.org/#/c/158580/5/cinder/volume/driver.py was breaking our scst code so uploaded fix for this https://review.openstack.org/#/c/161143/3 | 19:53 |
| *** e0ne is now known as e0ne_ | 19:55 | |
| *** EmilienM is now known as EmilienM|afk | 19:56 | |
| *** harlowja_ has quit IRC | 19:57 | |
| *** ronis__ has quit IRC | 19:57 | |
| thingee | nikesh_vedams: excellent! | 19:57 |
| *** devlaps has joined #openstack-cinder | 19:58 | |
| *** harlowja has joined #openstack-cinder | 20:01 | |
| *** e0ne_ has quit IRC | 20:01 | |
| *** jaypipes has quit IRC | 20:01 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: REST API removed option fails snapshot creation https://review.openstack.org/161382 | 20:02 |
| *** lpetrut has joined #openstack-cinder | 20:02 | |
| *** jaypipes has joined #openstack-cinder | 20:02 | |
| Swanson | I spaced making the branch in the gerrit commit bit. How should I fix this? | 20:02 |
| *** lpetrut has left #openstack-cinder | 20:03 | |
| *** e0ne has joined #openstack-cinder | 20:05 | |
| *** anshul has joined #openstack-cinder | 20:07 | |
| *** Mandell has quit IRC | 20:08 | |
| *** _cjones_ has quit IRC | 20:08 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC API change fails snapshot creation in Dell driver https://review.openstack.org/161382 | 20:10 |
| jungleboyj | Swanson: He he. I used to do taht all the time. | 20:11 |
| *** alecv has quit IRC | 20:11 | |
| Swanson | I scrolled the workflow page a bit too fast this time. | 20:12 |
| *** timcl1 has quit IRC | 20:13 | |
| *** dustins has quit IRC | 20:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/157642 | 20:14 |
| mtanino | hemna: https://review.openstack.org/#/c/161036/ thank you for your comment. I will add deprecated comments and add warning message when these configs will be used. | 20:15 |
| *** tbarron has quit IRC | 20:16 | |
| *** timcl has joined #openstack-cinder | 20:16 | |
| *** annashen has quit IRC | 20:17 | |
| *** timcl has quit IRC | 20:19 | |
| *** tbarron has joined #openstack-cinder | 20:22 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC API change fails snapshot creation https://review.openstack.org/161382 | 20:23 |
| *** timcl has joined #openstack-cinder | 20:23 | |
| Swanson | *sigh* | 20:23 |
| *** _cjones_ has joined #openstack-cinder | 20:25 | |
| *** gmurphy has joined #openstack-cinder | 20:26 | |
| *** tbarron has quit IRC | 20:26 | |
| openstackgerrit | YAMADA Hideki proposed openstack/cinder: Add missing copy_volume_to_image method to Sheepdog driver https://review.openstack.org/161400 | 20:27 |
| *** rushil has joined #openstack-cinder | 20:28 | |
| *** logan2 has joined #openstack-cinder | 20:29 | |
| *** kmartin has quit IRC | 20:29 | |
| *** chlong has joined #openstack-cinder | 20:30 | |
| *** tbarron has joined #openstack-cinder | 20:33 | |
| jungleboyj | Swanson: Did you figure it out. Sorry, manager walked in. | 20:34 |
| *** ebalduf has joined #openstack-cinder | 20:36 | |
| *** timcl1 has joined #openstack-cinder | 20:38 | |
| *** Mandell has joined #openstack-cinder | 20:38 | |
| *** madskier has joined #openstack-cinder | 20:38 | |
| *** timcl has quit IRC | 20:38 | |
| Swanson | jungleboyj: yeah, I think so. | 20:38 |
| Swanson | Zuul seems broken. 450 in check. And why did it run python26 against my driver and mine alone!?!? | 20:39 |
| jungleboyj | Swanson: Ok, sorry, I didn't respond. Bascially you just need to do a 'git checkout -b <branchname>' in master before doing anything else. | 20:39 |
| jungleboyj | If your review is still on master, it doesn't really matter, just need to change it locally so you don't lose it. | 20:39 |
| smcginnis | Swanson: Zuul got backed up last night. Still trying to catch up. | 20:40 |
| Swanson | That's what I did. Seems fine now. | 20:40 |
| angela-s | thingee: May I ask when BPs which were rejected for K can be resubmitted to L? Or is this documented on wiki and I'm just not seeing it? Is this a topic that is discussed at the summit? Thanks. | 20:40 |
| jungleboyj | Swanson: Good. | 20:40 |
| *** ebalduf has quit IRC | 20:41 | |
| Swanson | jungleboyj: Thanks! | 20:42 |
| jungleboyj | Swanson: Welcome. I just started editing in the wrong branch and couldn't figure out why my other changes weren't there. *Sigh* | 20:42 |
| jungleboyj | And they still aren't *Sigh* | 20:43 |
| *** akerr has quit IRC | 20:43 | |
| *** Yogi1 has quit IRC | 20:44 | |
| *** esker has joined #openstack-cinder | 20:44 | |
| thingee | angela-s: it is not documented anywhere. | 20:47 |
| thingee | angela-s: very likely you can look at gerrit reviews for cinder that are open and marked with a topic branch bp/* | 20:47 |
| thingee | that have a -2 at this point | 20:47 |
| angela-s | thingee: yes, we have 2 BPs in this state. trying to determine the date at which we can move forward with those for L. I was assuming that we have to wait until L is master. is that the correct assumption? | 20:51 |
| *** crose has quit IRC | 20:53 | |
| *** crose has joined #openstack-cinder | 20:53 | |
| thingee | angela-s: yes eventually we'll propose a patch that will switch things to L. | 20:54 |
| openstackgerrit | nikeshmahalka proposed openstack/cinder: Two choices for iscsi_helper is missing https://review.openstack.org/161143 | 20:54 |
| *** annashen has joined #openstack-cinder | 20:54 | |
| thingee | We will until march 19th + whatever time needed for RCs | 20:54 |
| *** Bsony has joined #openstack-cinder | 20:55 | |
| *** kmartin has joined #openstack-cinder | 20:55 | |
| angela-s | thingee: Ok, thanks for the confirmation. i've not been to a summit before, but will attend Vancouver. Are BP proposals discussed at the summit? | 20:55 |
| thingee | angela-s: if there is discussion really needed, sure | 20:56 |
| thingee | there are multiple kinds of sessions that happen though | 20:56 |
| thingee | angela-s: are you on the openstack dev ML? | 20:56 |
| thingee | angela-s: these are the different kinds of sessions that will exist http://lists.openstack.org/pipermail/openstack-dev/2015-January/054122.html | 20:57 |
| thingee | angela-s: and here's where I talked about proposals http://lists.openstack.org/pipermail/openstack-dev/2015-February/057383.html | 20:58 |
| thingee | for cinder | 20:58 |
| angela-s | thingee: ok. besides the BPs, we have a couple of topics for discussion, i will post those to the etherpad for the April 27 meeting. thanks for the info! | 20:58 |
| thingee | angela-s: np | 20:59 |
| *** dustins has joined #openstack-cinder | 20:59 | |
| *** Bsony has quit IRC | 20:59 | |
| *** jungleboyj has quit IRC | 20:59 | |
| *** kmartin has quit IRC | 21:00 | |
| *** angela-s_ has joined #openstack-cinder | 21:01 | |
| *** angela-s has quit IRC | 21:03 | |
| *** nkrinner has quit IRC | 21:05 | |
| *** Longgeek has quit IRC | 21:10 | |
| *** e0ne has quit IRC | 21:14 | |
| *** cdelatte has quit IRC | 21:14 | |
| *** alexpilotti has quit IRC | 21:17 | |
| *** Anticimex has quit IRC | 21:21 | |
| *** sandywalsh has quit IRC | 21:23 | |
| *** sandywalsh has joined #openstack-cinder | 21:24 | |
| *** kmartin has joined #openstack-cinder | 21:26 | |
| *** Anticimex has joined #openstack-cinder | 21:27 | |
| *** rushil has quit IRC | 21:28 | |
| *** patrickeast has quit IRC | 21:29 | |
| hemna | mtanino, ok cool, so you get my point about deprecating the config options? | 21:30 |
| *** Redcavalier has quit IRC | 21:31 | |
| mtanino | hemna: yes. I understand it. I will add same code of lio_initiator_iqns config option in my patch. | 21:31 |
| *** rushil has joined #openstack-cinder | 21:31 | |
| *** patri582 has quit IRC | 21:31 | |
| mtanino | hemna: This option shows warning message when it is uded. | 21:32 |
| mtanino | used. | 21:32 |
| hemna | mtanino, ok cool. | 21:32 |
| *** patrickeast__ has quit IRC | 21:32 | |
| *** patrickeast_ has quit IRC | 21:32 | |
| hemna | mtanino, we can remove it in L. we just need to at least deprecate options for a release afaik. | 21:32 |
| *** patri251 has quit IRC | 21:32 | |
| *** patri498 has quit IRC | 21:32 | |
| mtanino | hemna: I got it. | 21:32 |
| hemna | mtanino, thanks man. | 21:33 |
| *** patri461 has quit IRC | 21:33 | |
| *** patrickeast has joined #openstack-cinder | 21:34 | |
| *** patrickeast_ has joined #openstack-cinder | 21:35 | |
| *** patrickeast__ has joined #openstack-cinder | 21:35 | |
| *** patri255 has joined #openstack-cinder | 21:35 | |
| *** patri910 has joined #openstack-cinder | 21:35 | |
| *** patri169 has joined #openstack-cinder | 21:35 | |
| *** patri733 has joined #openstack-cinder | 21:35 | |
| Swanson | jungleboyj: thanks for the review! | 21:35 |
| hemna | http://qph.is.quoracdn.net/main-qimg-46eef57c86ac0c1522b0f8f4a57e03e7 | 21:39 |
| *** bswartz has quit IRC | 21:41 | |
| *** EmilienM|afk is now known as EmilienM | 21:43 | |
| *** bswartz has joined #openstack-cinder | 21:46 | |
| *** madskier has quit IRC | 21:46 | |
| *** dustins_ has joined #openstack-cinder | 21:47 | |
| *** dustins has quit IRC | 21:48 | |
| *** dustins_ has quit IRC | 21:49 | |
| *** rushil has quit IRC | 21:49 | |
| *** dhellmann has quit IRC | 21:52 | |
| *** rushil has joined #openstack-cinder | 21:55 | |
| *** dhellmann has joined #openstack-cinder | 21:55 | |
| *** rushil has quit IRC | 21:56 | |
| *** dhellmann has quit IRC | 21:56 | |
| *** dhellmann has joined #openstack-cinder | 21:57 | |
| *** rushil has joined #openstack-cinder | 21:57 | |
| *** bswartz has quit IRC | 21:58 | |
| *** harlowja has quit IRC | 21:58 | |
| *** rushil has quit IRC | 22:02 | |
| *** patrickeast has quit IRC | 22:06 | |
| *** esker has quit IRC | 22:09 | |
| *** patrickeast_ has quit IRC | 22:09 | |
| *** patrickeast__ has quit IRC | 22:09 | |
| *** patri169 has quit IRC | 22:10 | |
| *** patri910 has quit IRC | 22:10 | |
| *** patri255 has quit IRC | 22:10 | |
| *** annegentle has quit IRC | 22:10 | |
| *** patri733 has quit IRC | 22:10 | |
| *** patrickeast has joined #openstack-cinder | 22:11 | |
| *** annegentle has joined #openstack-cinder | 22:11 | |
| *** patrickeast has quit IRC | 22:12 | |
| *** patrickeast has joined #openstack-cinder | 22:12 | |
| *** chlong has quit IRC | 22:12 | |
| *** patrickeast_ has joined #openstack-cinder | 22:13 | |
| *** patrickeast__ has joined #openstack-cinder | 22:13 | |
| *** patri582 has joined #openstack-cinder | 22:13 | |
| *** patri455 has joined #openstack-cinder | 22:13 | |
| *** patri696 has joined #openstack-cinder | 22:13 | |
| *** angela-s_ has quit IRC | 22:13 | |
| *** jamielennox|away is now known as jamielennox | 22:14 | |
| *** anshul has quit IRC | 22:20 | |
| *** e0ne has joined #openstack-cinder | 22:22 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Make unused iscsi_num_targets, iser_num_targets configs as deprecated https://review.openstack.org/161036 | 22:24 |
| *** thangp has quit IRC | 22:25 | |
| *** tbarron has quit IRC | 22:27 | |
| *** tbarron has joined #openstack-cinder | 22:27 | |
| *** harlowja has joined #openstack-cinder | 22:28 | |
| *** Mandell has quit IRC | 22:29 | |
| *** e0ne has quit IRC | 22:30 | |
| *** mriedem is now known as mriedem-away | 22:30 | |
| *** patrickeast___ has joined #openstack-cinder | 22:31 | |
| *** tbarron has quit IRC | 22:32 | |
| *** diemt has quit IRC | 22:33 | |
| Swanson | hemna: Thanks! | 22:33 |
| hemna | yw | 22:34 |
| anish | mtanino: I thought you were only deprecating iser_num_targets ? | 22:39 |
| mtanino | anish: No. both iscsi_num_targets and iser_num_targets | 22:40 |
| mtanino | anish: Do you still need iscsi_num_targets? | 22:40 |
| *** erlon has quit IRC | 22:41 | |
| anish | mtanino: nope, my understanding from hemna's comments was only iser needed to be deprecated | 22:41 |
| hemna | anish, any config options that existed in a previous release. | 22:43 |
| hemna | if you want to remove a config option, and it existed in a prior release, it needs to first be deprecated for a release, before removal. | 22:44 |
| mtanino | hemna: I updated the patch. If you have chance, please visit again. https://review.openstack.org/#/c/161036/3 | 22:45 |
| anish | hemna: gotcha, thanks | 22:46 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Adding manage/unmanage support for LeftHand driver https://review.openstack.org/155127 | 22:46 |
| *** ebalduf has joined #openstack-cinder | 22:47 | |
| *** Anticimex has quit IRC | 22:49 | |
| *** ebalduf has quit IRC | 22:51 | |
| *** Anticimex has joined #openstack-cinder | 22:51 | |
| *** asselin_ has joined #openstack-cinder | 22:52 | |
| *** asselin_ has quit IRC | 22:57 | |
| *** Mandell has joined #openstack-cinder | 22:57 | |
| *** crose has quit IRC | 22:57 | |
| *** eharney has quit IRC | 23:00 | |
| *** NarenNarendra has joined #openstack-cinder | 23:00 | |
| *** annegentle has quit IRC | 23:04 | |
| *** alonmarx has quit IRC | 23:04 | |
| *** alonmarx has joined #openstack-cinder | 23:05 | |
| hemna | mtanino, can you do a driver cert run w/ the IET target configured for lvm ? | 23:06 |
| *** alexpilotti has joined #openstack-cinder | 23:06 | |
| hemna | I don't think we've thought about the CI implications of the target drivers | 23:06 |
| hemna | they really should be CI'd separately, since they fundamentally change the capability of a driver | 23:07 |
| mtanino | hemna: OK. I can post cert test. | 23:07 |
| hemna | mtanino, I think that'd be great. thank you! | 23:07 |
| *** jaypipes has quit IRC | 23:09 | |
| mtanino | hemna: A comment from jgriffith, we don't need CI for IET because this driver is reference driver. | 23:11 |
| hemna | well lvm is a reference driver | 23:11 |
| hemna | but it can be changed based upon which target is enabled | 23:11 |
| hemna | and that changes the capability of lvm | 23:11 |
| mtanino | hemna: but your concern is understandable. | 23:12 |
| hemna | since lvm is a reference driver, if anything it should be tested more rigorously than anything | 23:12 |
| hemna | including all of the possible target configurations that lvm can have. | 23:12 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: HP 3par driver filter and evaluator function https://review.openstack.org/151353 | 23:12 |
| hemna | mtanino, and iet is not the default target for lvm | 23:12 |
| hemna | so it actually never gets tested anywhere. | 23:12 |
| hemna | :( | 23:12 |
| mtanino | hemna: correct... | 23:13 |
| hemna | hence the request for the driver cert results at a minimum | 23:13 |
| mtanino | hemna: personally, I tried cert test and all tests were passed. | 23:14 |
| mtanino | so I will try again and post the result | 23:14 |
| hemna | perfect, post the results. :) | 23:14 |
| mtanino | using latest patch set | 23:14 |
| anish | mtanino: maybe you want to open the bug for cert results now and update commit msg | 23:15 |
| anish | (jenkins is slowwww today) | 23:15 |
| mtanino | anish: sure | 23:15 |
| mtanino | and make the bug invalid | 23:15 |
| anish | that way, jenins will be done by the time you actually update with results :) | 23:15 |
| anish | never understood why a change restricted to the commit msg forces a recheck (if I am remembering this correctly), seems unnecessary | 23:16 |
| *** kaufer has quit IRC | 23:16 | |
| *** ganso_ has quit IRC | 23:17 | |
| hemna | mtanino, ok I added this as an agenda item for next week's meeting. | 23:17 |
| hemna | we need to talk about it as a cinder team. | 23:17 |
| hemna | what to do about CI for the different target objects. | 23:17 |
| mtanino | hemna: sure. thank you. | 23:18 |
| *** jungleboyj has joined #openstack-cinder | 23:18 | |
| anish | hemna: add my name to the ping list please (or point me to where I can do this) | 23:20 |
| hemna | anish, https://wiki.openstack.org/wiki/CinderMeetings#Next_meeting | 23:20 |
| hemna | just login and edit :) | 23:20 |
| *** jcru has joined #openstack-cinder | 23:22 | |
| anish | copying stuff over from nova | 23:22 |
| anish | grr, why is one of them wiki/CinderMeetings and the other wiki/Meetings/Nova | 23:22 |
| hemna | um, one is for Nova and the other is for Cinder ? | 23:23 |
| anish | no | 23:25 |
| anish | I mean | 23:25 |
| anish | why CinderMeetings instead of Meetings/Cinder | 23:25 |
| hemna | no idea | 23:25 |
| *** annegentle has joined #openstack-cinder | 23:25 | |
| hemna | because we want to be difficult ? | 23:25 |
| anish | :) | 23:25 |
| Swanson | Nova's not the boss of us! | 23:26 |
| *** alexpilotti has quit IRC | 23:26 | |
| hemna | :) | 23:26 |
| *** jcru has quit IRC | 23:26 | |
| *** annegentle has quit IRC | 23:32 | |
| *** annegentle has joined #openstack-cinder | 23:33 | |
| *** IanGovett has quit IRC | 23:39 | |
| *** bswartz has joined #openstack-cinder | 23:39 | |
| openstackgerrit | Julie Gravel proposed openstack/cinder: Validate volume type extra specs https://review.openstack.org/161480 | 23:42 |
| *** chlong has joined #openstack-cinder | 23:42 | |
| *** annegentle has quit IRC | 23:43 | |
| *** annegentle has joined #openstack-cinder | 23:43 | |
| anish | can someone take a look at https://review.openstack.org/#/c/158926/ and https://review.openstack.org/#/c/154713/ ? | 23:44 |
| anish | they're very small changes I'm hoping to get in before the deadline | 23:44 |
| *** chlong has quit IRC | 23:46 | |
| *** jungleboyj has quit IRC | 23:46 | |
| *** annashen has quit IRC | 23:46 | |
| *** chlong has joined #openstack-cinder | 23:46 | |
| hemna | anish, careful what you ask for :P | 23:48 |
| *** annegentle has quit IRC | 23:48 | |
| *** ebalduf has joined #openstack-cinder | 23:48 | |
| anish | hemna: you too :) there's a giant chain of irc messages on why six should not be used | 23:50 |
| hemna | yah that's what I thought as well | 23:50 |
| anish | the end conclusion was if you're using , instead of %, six is not required irrespective of log level | 23:52 |
| *** ebalduf has quit IRC | 23:53 | |
| hemna | yah I think that wrapping happens inside the call then | 23:54 |
| hemna | you aren't building a string, so don't wrap it. | 23:54 |
| anish | % without six can issues (I forget what but xyang/smcginnis/jgriffith know the deets) but was discussed here as well https://review.openstack.org/#/c/158829/1/cinder/volume/targets/iet.py | 23:54 |
| hemna | yah because % is trying to construct a string | 23:54 |
| hemna | and if you don't wrap it, the exception object doesn't get decoded correctly into a string. | 23:55 |
| hemna | sometimes | 23:55 |
| *** EmilienM is now known as EmilienM|afk | 23:56 | |
| *** annegentle has joined #openstack-cinder | 23:56 | |
| openstackgerrit | Anish Bhatt proposed openstack/cinder: Simplify cxt test_create_export() unit test https://review.openstack.org/158926 | 23:57 |
| *** annegentle has quit IRC | 23:57 | |
| anish | hemna: added an explanation ^ | 23:57 |
| *** annegentle has joined #openstack-cinder | 23:57 | |
| hemna | anish, ok thank you. | 23:58 |
| *** annegentle has quit IRC | 23:58 | |
| hemna | just remember the commit messages are all you have sometimes when looking through git commits locally. | 23:58 |
| hemna | and something that explains the patch is all that you have to go on at times. | 23:58 |
| anish | gotcha, the new one commit message is much more self explanatory | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!