| *** catintheroof has joined #openstack-cinder | 00:03 | |
| *** jmlowe has quit IRC | 00:13 | |
| *** itlinux has joined #openstack-cinder | 00:22 | |
| *** jmlowe has joined #openstack-cinder | 00:23 | |
| *** s-shiono has joined #openstack-cinder | 00:31 | |
| *** kaisers has joined #openstack-cinder | 00:32 | |
| *** DFFlanders has quit IRC | 00:33 | |
| *** DFFlanders has joined #openstack-cinder | 00:36 | |
| *** Nel1x has joined #openstack-cinder | 00:40 | |
| *** baojg_ has quit IRC | 00:41 | |
| *** baojg has joined #openstack-cinder | 00:42 | |
| *** _pewp_ has quit IRC | 00:43 | |
| *** randomha1k has joined #openstack-cinder | 00:44 | |
| *** catintheroof has quit IRC | 00:44 | |
| *** _pewp_ has joined #openstack-cinder | 00:45 | |
| *** baojg has quit IRC | 00:46 | |
| *** randomha1k has quit IRC | 00:48 | |
| *** iceyao has joined #openstack-cinder | 00:58 | |
| openstackgerrit | Armando Migliaccio proposed openstack/cinder master: Switch to using bool for filtering non-deleted volume attributes https://review.openstack.org/489706 | 01:02 |
|---|---|---|
| openstackgerrit | TommyLike proposed openstack/cinder master: Correct 'os-show_image_metadata' API document https://review.openstack.org/489542 | 01:03 |
| *** iceyao has quit IRC | 01:03 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Update resource_filter to keep backward compatibility https://review.openstack.org/489503 | 01:10 |
| *** iceyao has joined #openstack-cinder | 01:11 | |
| *** kaisers_ has joined #openstack-cinder | 01:11 | |
| *** kaisers has quit IRC | 01:14 | |
| *** Apoorva has joined #openstack-cinder | 01:16 | |
| *** Apoorva_ has quit IRC | 01:20 | |
| *** Apoorva has quit IRC | 01:21 | |
| *** jmlowe has quit IRC | 01:21 | |
| *** Shunli has joined #openstack-cinder | 01:24 | |
| *** itlinux has quit IRC | 01:24 | |
| *** jmlowe has joined #openstack-cinder | 01:24 | |
| *** https_GK1wmSU has joined #openstack-cinder | 01:28 | |
| *** mriedem has quit IRC | 01:30 | |
| *** https_GK1wmSU has left #openstack-cinder | 01:31 | |
| *** sam_nowitzki has joined #openstack-cinder | 01:31 | |
| *** gouthamr has joined #openstack-cinder | 01:41 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder master: [DOC] VMware: Update config table https://review.openstack.org/489824 | 01:45 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder master: [DOC] VMware: Update config table https://review.openstack.org/489824 | 01:47 |
| openstackgerrit | Merged openstack/cinder master: Volume type with encryption comparison https://review.openstack.org/477888 | 01:49 |
| *** markvoelker has joined #openstack-cinder | 01:54 | |
| *** kaisers_ has quit IRC | 01:59 | |
| *** lhx__ has joined #openstack-cinder | 02:00 | |
| *** itlinux has joined #openstack-cinder | 02:07 | |
| *** bkopilov_ has quit IRC | 02:11 | |
| *** KeithMnemonic has quit IRC | 02:14 | |
| *** crushil has quit IRC | 02:18 | |
| *** crushil has joined #openstack-cinder | 02:18 | |
| *** markvoelker has quit IRC | 02:23 | |
| *** diablo_rojo has quit IRC | 02:36 | |
| *** saphi has joined #openstack-cinder | 02:39 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder master: [DOC] VMware: Add doc for vmware:adapter_type https://review.openstack.org/489832 | 02:40 |
| *** dave-mcc_ has joined #openstack-cinder | 02:41 | |
| *** dave-mccowan has quit IRC | 02:42 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder master: [DOC] VMware: Add doc for vmware:adapter_type https://review.openstack.org/489832 | 02:42 |
| *** kaisers has joined #openstack-cinder | 02:55 | |
| *** nicolasbock has joined #openstack-cinder | 02:57 | |
| *** markvoelker has joined #openstack-cinder | 02:59 | |
| *** itlinux has quit IRC | 03:00 | |
| *** kaisers has quit IRC | 03:05 | |
| *** diablo_rojo has joined #openstack-cinder | 03:08 | |
| *** dave-mccowan has joined #openstack-cinder | 03:12 | |
| *** dave-mcc_ has quit IRC | 03:14 | |
| *** dave-mccowan has quit IRC | 03:33 | |
| openstackgerrit | Felipe Monteiro proposed openstack/cinder master: [api-ref] Add api-ref for os-quota-class-sets APIs https://review.openstack.org/477740 | 03:34 |
| *** yangyapeng has joined #openstack-cinder | 03:36 | |
| *** johnhaan has joined #openstack-cinder | 03:39 | |
| *** gkadam has joined #openstack-cinder | 03:41 | |
| *** vivsoni_ has quit IRC | 03:43 | |
| *** bkopilov_ has joined #openstack-cinder | 03:44 | |
| *** crushil has quit IRC | 03:45 | |
| *** crushil has joined #openstack-cinder | 03:46 | |
| *** vivsoni has joined #openstack-cinder | 03:46 | |
| *** stakeda has joined #openstack-cinder | 03:47 | |
| *** links has joined #openstack-cinder | 03:48 | |
| *** kaisers has joined #openstack-cinder | 04:02 | |
| *** itlinux has joined #openstack-cinder | 04:02 | |
| *** savvas_ has joined #openstack-cinder | 04:05 | |
| *** gouthamr has quit IRC | 04:06 | |
| *** gouthamr has joined #openstack-cinder | 04:06 | |
| *** savvas has quit IRC | 04:07 | |
| *** iceyao has quit IRC | 04:14 | |
| *** chaynikasaikia has joined #openstack-cinder | 04:14 | |
| *** csaikia has quit IRC | 04:17 | |
| *** kaisers has quit IRC | 04:19 | |
| *** adisky__ has joined #openstack-cinder | 04:24 | |
| *** Nel1x has quit IRC | 04:33 | |
| *** iceyao has joined #openstack-cinder | 04:37 | |
| *** iceyao has quit IRC | 04:42 | |
| *** randomha1k has joined #openstack-cinder | 04:44 | |
| *** randomha1k has quit IRC | 04:48 | |
| *** iceyao has joined #openstack-cinder | 04:48 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Refresh az cache if target az is not found in cache https://review.openstack.org/467505 | 04:52 |
| *** lkuchlan has joined #openstack-cinder | 05:01 | |
| *** diablo_rojo has quit IRC | 05:05 | |
| *** kaisers has joined #openstack-cinder | 05:08 | |
| *** arnewiebalck_ has joined #openstack-cinder | 05:11 | |
| *** Poornima has joined #openstack-cinder | 05:12 | |
| *** kaisers has quit IRC | 05:13 | |
| *** kaisers has joined #openstack-cinder | 05:13 | |
| *** yangyapeng has quit IRC | 05:17 | |
| *** yangyapeng has joined #openstack-cinder | 05:19 | |
| *** psachin has joined #openstack-cinder | 05:20 | |
| *** iceyao has quit IRC | 05:22 | |
| *** iceyao has joined #openstack-cinder | 05:24 | |
| *** vivsoni_ has joined #openstack-cinder | 05:45 | |
| *** vivsoni has quit IRC | 05:45 | |
| *** chaynikasaikia has quit IRC | 05:46 | |
| *** kwathore has joined #openstack-cinder | 05:48 | |
| *** pgadiya has joined #openstack-cinder | 05:48 | |
| kwathore | Team, please review below patchses : | 05:49 |
| kwathore | https://review.openstack.org/#/c/462624/ | 05:49 |
| kwathore | https://review.openstack.org/#/c/461744/ | 05:50 |
| *** gouthamr has quit IRC | 05:53 | |
| *** e0ne has joined #openstack-cinder | 06:01 | |
| *** hoonetorg has quit IRC | 06:02 | |
| *** e0ne has quit IRC | 06:03 | |
| *** arnewiebalck_ has quit IRC | 06:13 | |
| *** hoonetorg has joined #openstack-cinder | 06:15 | |
| *** venu_ has quit IRC | 06:17 | |
| *** sam_nowitzki has quit IRC | 06:34 | |
| *** pcaruana has joined #openstack-cinder | 06:34 | |
| *** sam_nowitzki has joined #openstack-cinder | 06:34 | |
| *** johnhaan has quit IRC | 06:39 | |
| *** ChubYann has quit IRC | 06:42 | |
| *** yangyape_ has joined #openstack-cinder | 06:43 | |
| *** edmondsw has joined #openstack-cinder | 06:43 | |
| *** yangyape_ has quit IRC | 06:46 | |
| *** yangyape_ has joined #openstack-cinder | 06:46 | |
| *** yangyapeng has quit IRC | 06:46 | |
| *** rcernin has joined #openstack-cinder | 06:46 | |
| *** edmondsw has quit IRC | 06:48 | |
| *** yangyapeng has joined #openstack-cinder | 06:50 | |
| *** yangyape_ has quit IRC | 06:50 | |
| *** markvoelker has quit IRC | 06:51 | |
| *** sam_nowitzki has quit IRC | 06:52 | |
| *** yangyapeng has quit IRC | 06:54 | |
| arnewiebalck | smcginnis: thanks! | 07:00 |
| *** aarefiev_afk is now known as aarefiev | 07:07 | |
| *** Arianrhod has joined #openstack-cinder | 07:17 | |
| *** https_GK1wmSU has joined #openstack-cinder | 07:20 | |
| *** https_GK1wmSU has left #openstack-cinder | 07:22 | |
| openstackgerrit | Lingyong Xu proposed openstack/python-brick-cinderclient-ext master: Update the documentation link for doc migration https://review.openstack.org/489901 | 07:35 |
| *** vivsoni_ has quit IRC | 07:37 | |
| *** vivsoni_ has joined #openstack-cinder | 07:37 | |
| *** lpetrut has joined #openstack-cinder | 07:37 | |
| *** kwathore_ has joined #openstack-cinder | 07:37 | |
| *** Tzur has joined #openstack-cinder | 07:38 | |
| *** kwathore has quit IRC | 07:40 | |
| *** randomha1k has joined #openstack-cinder | 07:44 | |
| *** arnewiebalck_ has joined #openstack-cinder | 07:44 | |
| *** baojg has joined #openstack-cinder | 07:48 | |
| *** randomha1k has quit IRC | 07:48 | |
| *** gcb_ has quit IRC | 07:48 | |
| *** tesseract has joined #openstack-cinder | 07:48 | |
| *** yangyapeng has joined #openstack-cinder | 07:50 | |
| *** baojg has quit IRC | 07:52 | |
| *** gkadam is now known as gkadam-afk | 07:54 | |
| Tzur | Hi, can any core member review this? it's a 2 line review and we only need another +2 to finish https://review.openstack.org/#/c/486413/ | 07:54 |
| Tzur | Thanks | 07:54 |
| *** josecastroleon has quit IRC | 08:00 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Refresh az cache if target az is not found in cache https://review.openstack.org/467505 | 08:00 |
| *** yangyapeng has quit IRC | 08:05 | |
| *** yangyapeng has joined #openstack-cinder | 08:06 | |
| *** saphi has quit IRC | 08:17 | |
| openstackgerrit | zhangdaolong proposed openstack/cinder master: Add desc for os-services API https://review.openstack.org/489920 | 08:19 |
| *** https_GK1wmSU has joined #openstack-cinder | 08:20 | |
| *** arnewiebalck_ has quit IRC | 08:20 | |
| *** https_GK1wmSU has left #openstack-cinder | 08:22 | |
| *** josecastroleon has joined #openstack-cinder | 08:23 | |
| *** pgadiya has quit IRC | 08:25 | |
| *** s-shiono has quit IRC | 08:27 | |
| Anticimex | 1. trying to understand how to get the time periods of cinder-volume-usage-audit correct (in particular relative times crossing DST boundaries). 2. realizes it is TZ-unaware. 3. tries to figure out what is correct. 4. possibly get it right. 5. realizes cinder-db is completely tz-unaware. 6. concludes only way to operate cinder (and presumably rest of openstack) correctly is to use UTC on serve | 08:30 |
| Anticimex | rs | 08:31 |
| *** edmondsw has joined #openstack-cinder | 08:31 | |
| *** openstackgerrit has quit IRC | 08:33 | |
| *** edmondsw has quit IRC | 08:36 | |
| *** pgadiya has joined #openstack-cinder | 08:41 | |
| *** yangyapeng has quit IRC | 08:43 | |
| *** yangyapeng has joined #openstack-cinder | 08:43 | |
| *** openstackgerrit has joined #openstack-cinder | 08:48 | |
| openstackgerrit | zheng yin proposed openstack/python-cinderclient master: add snapshot_id param note https://review.openstack.org/489931 | 08:48 |
| *** baojg has joined #openstack-cinder | 08:49 | |
| *** baojg has quit IRC | 08:53 | |
| lhx__ | hi guys, is there a doc teach how to use replication features? | 09:00 |
| lhx__ | how to set the extar specs for a replcated type? | 09:03 |
| openstackgerrit | Kushal Wathore proposed openstack/cinder master: HPE 3PAR: ISCSI/FC drivers – code refactoring https://review.openstack.org/446475 | 09:07 |
| *** yangyapeng has quit IRC | 09:08 | |
| *** yangyapeng has joined #openstack-cinder | 09:14 | |
| *** yangyapeng has quit IRC | 09:16 | |
| *** yangyapeng has joined #openstack-cinder | 09:17 | |
| tommylikehu | Hey lhx__ , I havn't tried this feature myself, just googled the related blog and videos, hope this could help :) | 09:21 |
| tommylikehu | https://www.sebastien-han.fr/blog/2017/06/19/OpenStack-Cinder-configure-replication-api-with-ceph/ | 09:21 |
| tommylikehu | http://www.trystack.cn/Articles/cinder-replication.html | 09:21 |
| tommylikehu | https://www.openstack.org/videos/tokio-2015/block-storage-replication-with-cinder | 09:21 |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Support az filter for snapshot https://review.openstack.org/489945 | 09:28 |
| *** gcb has joined #openstack-cinder | 09:28 | |
| *** Wenjun has joined #openstack-cinder | 09:28 | |
| Wenjun | Team, please review below patchses : | 09:29 |
| Wenjun | https://review.openstack.org/#/c/477857/ | 09:30 |
| *** sdague has joined #openstack-cinder | 09:32 | |
| *** Shunli has quit IRC | 09:35 | |
| openstackgerrit | zheng yin proposed openstack/cinder master: backup/ceph: remove redundant pool https://review.openstack.org/489949 | 09:36 |
| lhx__ | tommylikehu, thanks. btw, do you know how to get the 'backend_id'? | 09:37 |
| kwathore_ | @lhx__ in HPE, we always gives 3par system name (seondary array system name) | 09:45 |
| *** Wenjun has quit IRC | 09:45 | |
| tommylikehu | lhx__: please take a look: https://github.com/openstack/cinder/blob/master/doc/source/contributor/replication.rst | 09:46 |
| lhx__ | kwathore_, is that just a host name? | 09:48 |
| *** baojg has joined #openstack-cinder | 09:50 | |
| lhx__ | tommylikehu, cool, I will have a look :) | 09:50 |
| *** dalgaaf has quit IRC | 09:50 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Refresh az cache if target az is not found in cache https://review.openstack.org/467505 | 09:50 |
| *** e0ne has joined #openstack-cinder | 09:53 | |
| *** baojg has quit IRC | 09:54 | |
| *** abhitechie has joined #openstack-cinder | 09:54 | |
| *** dalgaaf has joined #openstack-cinder | 09:55 | |
| ildikov | e0ne: hi | 09:57 |
| e0ne | ildikov: hi | 09:58 |
| ildikov | e0ne: we got into some public_url problem with the uWSGI changes in devstack for Cinder | 09:58 |
| e0ne | ildikov: do you have any logs? | 09:58 |
| e0ne | ildikov: what exactly error did you face? | 09:59 |
| ildikov | e0ne: smcginnis had a patch yesterday to switch back to DEFAULT: https://review.openstack.org/#/c/489715/ | 09:59 |
| tommylikehu | hey e0ne , do you mind to take a look at this patch? https://review.openstack.org/#/c/467505/ :) | 10:00 |
| ildikov | e0ne: it changes the logs, but the version check still blows up here: http://logs.openstack.org/85/330285/107/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/a68338b/logs/screen-n-api.txt.gz?level=ERROR#_Aug_01_14_20_52_764340 | 10:00 |
| ildikov | e0ne: in this exact call: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/client.py#L88 | 10:01 |
| ildikov | e0ne: it hits the Horizon login page instead of Cinder, at least it did before Sean's patch | 10:02 |
| ildikov | e0ne: the log is from the new attach flow patch in Nova, I added Sean's patch as a depends-on to see whether it fixes the problem, but it didn't | 10:02 |
| e0ne | ildikov: I understood the issue | 10:16 |
| *** iceyao has quit IRC | 10:17 | |
| e0ne | ildikov: the problem is we've the same endpoint for horizon and cinder:( | 10:18 |
| *** abhitechie has quit IRC | 10:18 | |
| *** edmondsw has joined #openstack-cinder | 10:19 | |
| *** Arianrhod has quit IRC | 10:20 | |
| *** edmondsw has quit IRC | 10:24 | |
| e0ne | ildikov: let me setup clean devstack to try reproduce it | 10:26 |
| *** savvas has joined #openstack-cinder | 10:28 | |
| *** savvas_ has quit IRC | 10:30 | |
| *** iceyao has joined #openstack-cinder | 10:32 | |
| ildikov | e0ne: yeah, something like that :( | 10:35 |
| ildikov | e0ne: thanks for looking into it! | 10:35 |
| *** iceyao has quit IRC | 10:36 | |
| openstackgerrit | zhangdaolong proposed openstack/cinder master: Add desc for os-services API https://review.openstack.org/489920 | 10:38 |
| *** zhurong has quit IRC | 10:43 | |
| *** totadkiran has joined #openstack-cinder | 10:46 | |
| *** Poornima has quit IRC | 10:48 | |
| *** bkopilov_ has quit IRC | 10:50 | |
| *** baojg has joined #openstack-cinder | 10:50 | |
| *** markvoelker has joined #openstack-cinder | 10:52 | |
| *** iceyao has joined #openstack-cinder | 10:54 | |
| *** Poornima has joined #openstack-cinder | 10:54 | |
| *** pgadiya has quit IRC | 10:56 | |
| *** baojg has quit IRC | 10:56 | |
| *** markvoelker has quit IRC | 10:57 | |
| *** Shunli has joined #openstack-cinder | 11:00 | |
| *** Tzur has quit IRC | 11:01 | |
| e0ne | ildikov: I've got few ideas. be back in ~40mins | 11:01 |
| e0ne | ildikov: the problem is here https://github.com/openstack/python-cinderclient/blob/63ac82a55489d55246da939b5ae60b8a65fb9ec7/cinderclient/client.py#L87 | 11:01 |
| e0ne | cinderclient doesn't work without a port number in a url | 11:01 |
| ildikov | e0ne: oh, ok | 11:02 |
| e0ne | we need some hack like we have in tempest :( https://review.openstack.org/#/c/473514/7/tempest/cmd/verify_tempest_config.py | 11:02 |
| ildikov | e0ne: we were touching the port idea yesterday, but not in the url explicitly | 11:03 |
| e0ne | I'm afraid that we'll need one more release of cinderclient :( | 11:03 |
| * e0ne heads out to have a lanch | 11:04 | |
| ildikov | e0ne: yeah, I wanted to ask how other projects are handling this | 11:04 |
| ildikov | e0ne: I need to grab lunch too :) | 11:04 |
| ildikov | e0ne: have a good one and chat later | 11:05 |
| e0ne | ildikov: thanks | 11:05 |
| *** pgadiya has joined #openstack-cinder | 11:08 | |
| *** kaisers1 has quit IRC | 11:15 | |
| *** lhx__ has quit IRC | 11:16 | |
| *** kaisers1 has joined #openstack-cinder | 11:21 | |
| *** Shunli has quit IRC | 11:22 | |
| openstackgerrit | xiaoqin proposed openstack/cinder master: Add Storwize replication group support https://review.openstack.org/469394 | 11:22 |
| *** edmondsw has joined #openstack-cinder | 11:29 | |
| *** edmondsw has quit IRC | 11:31 | |
| *** nicolasbock_ has joined #openstack-cinder | 11:41 | |
| *** nicolasbock__ has joined #openstack-cinder | 11:42 | |
| *** nicolasbock has quit IRC | 11:44 | |
| *** nicolasbock has joined #openstack-cinder | 11:45 | |
| *** lhx_ has joined #openstack-cinder | 11:46 | |
| *** nicolasbock_ has quit IRC | 11:46 | |
| *** nicolasbock__ has quit IRC | 11:47 | |
| *** abhitechie has joined #openstack-cinder | 11:48 | |
| *** kaisers1 has quit IRC | 11:48 | |
| *** Poornima_K has joined #openstack-cinder | 11:50 | |
| *** Poornima has quit IRC | 11:51 | |
| *** baojg has joined #openstack-cinder | 11:53 | |
| *** markvoelker has joined #openstack-cinder | 11:53 | |
| *** Tzur has joined #openstack-cinder | 11:53 | |
| Tzur | Hi, can any core member review this? it's a 2 line review and we only need another +2 to finish https://review.openstack.org/#/c/486413/ Thanks | 11:53 |
| *** zhaochao has quit IRC | 11:53 | |
| *** baojg has quit IRC | 11:57 | |
| *** Poornima_K has quit IRC | 12:00 | |
| *** Poornima has joined #openstack-cinder | 12:00 | |
| *** kaisers1 has joined #openstack-cinder | 12:02 | |
| *** stakeda has quit IRC | 12:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/488856 | 12:15 |
| *** totadkiran has quit IRC | 12:17 | |
| *** Tzur has quit IRC | 12:19 | |
| *** markvoelker has quit IRC | 12:22 | |
| *** markvoelker has joined #openstack-cinder | 12:22 | |
| *** Poornima has quit IRC | 12:23 | |
| *** dustins has joined #openstack-cinder | 12:25 | |
| *** edmondsw has joined #openstack-cinder | 12:33 | |
| *** iceyao has quit IRC | 12:35 | |
| *** liverpooler has joined #openstack-cinder | 12:36 | |
| *** sdague_ has joined #openstack-cinder | 12:43 | |
| *** gkadam-afk has quit IRC | 12:48 | |
| *** baojg has joined #openstack-cinder | 12:54 | |
| *** gouthamr has joined #openstack-cinder | 12:55 | |
| *** crushil has quit IRC | 12:58 | |
| *** baojg has quit IRC | 12:59 | |
| *** crushil has joined #openstack-cinder | 12:59 | |
| *** iceyao has joined #openstack-cinder | 13:03 | |
| *** xyang1 has joined #openstack-cinder | 13:04 | |
| *** iceyao has quit IRC | 13:07 | |
| *** firebat has joined #openstack-cinder | 13:10 | |
| *** lucasxu has joined #openstack-cinder | 13:12 | |
| *** bkopilov_ has joined #openstack-cinder | 13:13 | |
| *** rawanh has joined #openstack-cinder | 13:14 | |
| *** Tzur has joined #openstack-cinder | 13:27 | |
| *** pgadiya has quit IRC | 13:29 | |
| *** felipemonteiro has joined #openstack-cinder | 13:31 | |
| *** jistr is now known as jistr|mtg | 13:32 | |
| *** felipemonteiro has quit IRC | 13:37 | |
| *** abhitechie has quit IRC | 13:38 | |
| *** yangyapeng has quit IRC | 13:38 | |
| *** mriedem has joined #openstack-cinder | 13:41 | |
| *** sdague has quit IRC | 13:43 | |
| *** cknight has joined #openstack-cinder | 13:44 | |
| *** randomha1k has joined #openstack-cinder | 13:45 | |
| *** aarefiev is now known as aarefiev_afk | 13:51 | |
| *** jistr|mtg is now known as jistr | 13:52 | |
| -openstackstatus- NOTICE: We have disable infracloud-vanilla due to the compute host running mirror.regionone.infracloud-vanilla.o.o being offline. Please recheck your failed jobs to schedule them to another cloud. | 13:56 | |
| *** sdague has joined #openstack-cinder | 13:56 | |
| *** zhurong has joined #openstack-cinder | 13:57 | |
| openstackgerrit | Tzur Eliyahu proposed openstack/cinder master: Ibm_storage - fix delete group with replication https://review.openstack.org/487018 | 14:00 |
| *** nicolasbock_ has joined #openstack-cinder | 14:04 | |
| *** nicolasbock__ has joined #openstack-cinder | 14:07 | |
| *** nicolasbock has quit IRC | 14:07 | |
| *** nicolasbock_ has quit IRC | 14:10 | |
| *** crushil has quit IRC | 14:11 | |
| *** crushil has joined #openstack-cinder | 14:12 | |
| *** mriedem1 has joined #openstack-cinder | 14:14 | |
| *** mriedem has quit IRC | 14:16 | |
| *** mriedem1 is now known as mriedem | 14:20 | |
| *** nicolasbock_ has joined #openstack-cinder | 14:21 | |
| *** eharney has joined #openstack-cinder | 14:22 | |
| *** iceyao has joined #openstack-cinder | 14:22 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient master: WIP. Fix get_highest_client_server_version with Cinder API + uWSGI https://review.openstack.org/490040 | 14:24 |
| e0ne | ildikov: could you please test nova with patch above? ^^ | 14:24 |
| *** baojg has joined #openstack-cinder | 14:24 | |
| *** belmoreira has joined #openstack-cinder | 14:25 | |
| *** nicolasbock__ has quit IRC | 14:25 | |
| openstackgerrit | Merged openstack/cinder master: ibm_storage - fix disable/delete replication. https://review.openstack.org/486413 | 14:29 |
| ildikov | e0ne: on it | 14:33 |
| e0ne | ildikov: thanks | 14:33 |
| *** Poornima has joined #openstack-cinder | 14:35 | |
| *** Poornima has quit IRC | 14:35 | |
| *** baojg has quit IRC | 14:36 | |
| *** baojg has joined #openstack-cinder | 14:36 | |
| *** baojg has quit IRC | 14:36 | |
| smcginnis | e0ne, ildikov: Thanks for digging into that. If my devstack patch is not necessary let me know and I can abandon it. | 14:36 |
| *** baojg has joined #openstack-cinder | 14:37 | |
| *** Poornima has joined #openstack-cinder | 14:37 | |
| *** baojg has quit IRC | 14:37 | |
| *** baojg has joined #openstack-cinder | 14:37 | |
| *** baojg has quit IRC | 14:38 | |
| *** zhurong has quit IRC | 14:38 | |
| e0ne | smcginnis: np. if my patch helps, I'll finish it in a less hacky way and it will require a new release :( | 14:38 |
| e0ne | I mean cinderclient release for Pike | 14:39 |
| smcginnis | e0ne: We'll have to ask for an exception to release a new client too. | 14:39 |
| smcginnis | e0ne: But if it prevents one of the Pike series goals, we probably are OK there. | 14:40 |
| e0ne | smcginnis: sure | 14:40 |
| e0ne | smcginnis: I'm trying to find better solution now | 14:40 |
| smcginnis | e0ne: Awesome, thanks Ivan! | 14:40 |
| e0ne | smcginnis, ildikov: do we have bug filed for this issu? | 14:41 |
| e0ne | s/issu/issue | 14:41 |
| ildikov | e0ne: I need to setup my Devstack again, but will let you know how things went | 14:42 |
| ildikov | e0ne: and I believe we don't | 14:42 |
| e0ne | ildikov: ok, thanks | 14:42 |
| ildikov | at least I haven't filed one yet | 14:42 |
| e0ne | I'm not sure that I'll be available tonight, but I'll do my best to fix it asap | 14:43 |
| e0ne | ildikov: I've tested my patch with this code: http://paste.openstack.org/show/617283/ | 14:45 |
| *** yangyapeng has joined #openstack-cinder | 14:45 | |
| smcginnis | Yeah, we should probably have a bug on that. | 14:45 |
| ildikov | e0ne: if that worked it should be all good, I just thought to see it in a full env | 14:46 |
| ildikov | e0ne: I can also play with the depends-on tag, a patch just cannot have enough :) | 14:46 |
| e0ne | ildikov: sure, we need to test it with devstack too | 14:46 |
| e0ne | smcginnis, ildikov: btw, does it block any gate? | 14:48 |
| ildikov | e0ne: it blocks a patch on review to switch to the new Cinder attach/detach API in Nova | 14:49 |
| e0ne | ildikov: so the good news are: we don't block any other cinder or nova activities | 14:49 |
| *** yangyapeng has quit IRC | 14:49 | |
| smcginnis | ildikov: Nova isn't going to take that in Pike, right? So we can wait until Queens to release a fixed client? | 14:50 |
| ildikov | e0ne: so it doesn't block everyone, but like three people and one of them is getting miserable :) | 14:50 |
| ildikov | smcginnis: we would like to get it in before the PTG | 14:50 |
| ildikov | smcginnis: so we will need a release before that too if the client has to be modified finally | 14:51 |
| smcginnis | ildikov: Definitely. Just thinking if we have a client fix, even if we can't release it you could have a depends-on so it doesn't hold up development during the freeze. | 14:51 |
| ildikov | smcginnis: sure, I added your and e0ne's patch to that list already in that patch | 14:51 |
| *** lkuchlan has quit IRC | 14:52 | |
| *** yangyapeng has joined #openstack-cinder | 14:53 | |
| *** armax_ has joined #openstack-cinder | 14:53 | |
| *** armax has quit IRC | 14:53 | |
| *** armax_ is now known as armax | 14:53 | |
| smcginnis | ildikov: Cool. Sorry, I know there's just been a series of bad luck with timing and things with this. | 14:53 |
| ildikov | smcginnis: yeah, quite a bit :( | 14:54 |
| smcginnis | ildikov: I think you deserve a medal. | 14:54 |
| ildikov | smcginnis: the problem is that we cannot deal with the blockdev error as the code blows up in the cinderclient now when you use Devstack way before that error :( | 14:55 |
| smcginnis | :/ | 14:55 |
| ildikov | smcginnis: nah, it's all good, it's not the only tough use case, I'm just practicing patience :) | 14:55 |
| smcginnis | ildikov: You're getting more practice than you need. :] | 14:56 |
| ildikov | smcginnis: well, maybe I need this much, who knows... :) | 14:57 |
| *** mtanino has joined #openstack-cinder | 14:58 | |
| *** felipemonteiro has joined #openstack-cinder | 15:00 | |
| ildikov | smcginnis: I will report a bug for this if you haven't already | 15:00 |
| smcginnis | ildikov: No, I haven't gotten to it yet. If you could, that would be great. | 15:01 |
| smcginnis | ildikov: I'm sure you can explain what is going on much better than I could anyway. ;) | 15:01 |
| ildikov | smcginnis: nice one ;) | 15:01 |
| smcginnis | :) | 15:01 |
| *** links has quit IRC | 15:01 | |
| ildikov | smcginnis: should this be primarily reported for Devstack or cinderclient? | 15:03 |
| *** felipemonteiro_ has joined #openstack-cinder | 15:04 | |
| smcginnis | ildikov: Hmm, probably cinderclient. | 15:04 |
| smcginnis | ildikov: Though I think my devstack patch is probably still needed. | 15:04 |
| *** felipemonteiro has quit IRC | 15:04 | |
| ildikov | smcginnis: ok, I started there, I can always add a tag or smth | 15:04 |
| smcginnis | +1 | 15:04 |
| *** felipemonteiro__ has joined #openstack-cinder | 15:04 | |
| *** jistr is now known as jistr|mtg | 15:06 | |
| *** felipemonteiro__ has quit IRC | 15:06 | |
| *** felipemonteiro__ has joined #openstack-cinder | 15:07 | |
| *** felipemonteiro_ has quit IRC | 15:08 | |
| *** iceyao has quit IRC | 15:13 | |
| *** Tzur has quit IRC | 15:14 | |
| *** e0ne has quit IRC | 15:16 | |
| *** liverpooler has quit IRC | 15:16 | |
| *** liverpooler has joined #openstack-cinder | 15:19 | |
| *** armax has quit IRC | 15:19 | |
| ildikov | smcginnis: https://bugs.launchpad.net/devstack/+bug/1708188 | 15:19 |
| openstack | Launchpad bug 1708188 in python-cinderclient "get_highest_client_server_version is failing with the new uWSGI changes in Devstack" [Undecided,New] | 15:19 |
| *** nicolasbock__ has joined #openstack-cinder | 15:21 | |
| *** psachin has quit IRC | 15:22 | |
| smcginnis | ildikov: Awesone, thanks. | 15:23 |
| smcginnis | And this is why I was hoping all of that would have been approved earlier so we could have had time to work through any side effects. | 15:24 |
| ildikov | smcginnis: np | 15:24 |
| *** nicolasbock_ has quit IRC | 15:24 | |
| ildikov | smcginnis: yeah, timing is pretty unfortunate | 15:24 |
| *** iceyao has joined #openstack-cinder | 15:25 | |
| ildikov | smcginnis: or well, maybe it's just RC1 phase as is :) | 15:25 |
| smcginnis | Always issues. :) | 15:28 |
| ildikov | smcginnis: having e0ne's and your change applied the code works again | 15:28 |
| openstackgerrit | Kushal Wathore proposed openstack/cinder master: HPE 3PAR: ISCSI/FC drivers – code refactoring https://review.openstack.org/446475 | 15:28 |
| *** armax has joined #openstack-cinder | 15:28 | |
| smcginnis | ildikov: Cool! At least your not blocked now. | 15:29 |
| Swanson | smcginnis, "you're" | 15:29 |
| * smcginnis slaps himself | 15:29 | |
| ildikov | smcginnis: I only attached one volume to an instance in my env, but at least it works fine now | 15:29 |
| ildikov | lol :) | 15:29 |
| smcginnis | ildikov: It got by the failure at least then. | 15:29 |
| *** iceyao has quit IRC | 15:30 | |
| ildikov | smcginnis: it's running in the check queue now too so we will see how things go | 15:30 |
| *** tomh has joined #openstack-cinder | 15:33 | |
| *** armax has quit IRC | 15:33 | |
| *** liverpooler has quit IRC | 15:34 | |
| *** liverpooler has joined #openstack-cinder | 15:36 | |
| *** baojg has joined #openstack-cinder | 15:38 | |
| *** baojg has quit IRC | 15:39 | |
| *** baojg has joined #openstack-cinder | 15:39 | |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - performance improvements in retype https://review.openstack.org/483933 | 15:40 |
| eharney | smcginnis: hopefully this is unrelated to current issues, but heads up anyway: https://review.openstack.org/#/c/490060/ | 15:43 |
| *** sdague_ has quit IRC | 15:43 | |
| smcginnis | eharney: Darn spaces! | 15:44 |
| eharney | smcginnis: i really, really, really dislike writing things in bash :) | 15:44 |
| smcginnis | eharney: There's just so many little gotchas like that. | 15:45 |
| *** itlinux has quit IRC | 15:45 | |
| *** e0ne has joined #openstack-cinder | 15:46 | |
| *** mtanino_ has joined #openstack-cinder | 15:46 | |
| *** wxy- has joined #openstack-cinder | 15:50 | |
| *** mlavalle has joined #openstack-cinder | 15:50 | |
| openstackgerrit | Merged openstack/cinder master: Updated from global requirements https://review.openstack.org/488856 | 15:51 |
| *** mlavalle has left #openstack-cinder | 15:51 | |
| *** belmoreira has quit IRC | 15:55 | |
| *** Apoorva has joined #openstack-cinder | 15:56 | |
| *** links has joined #openstack-cinder | 15:59 | |
| *** randomha1k has quit IRC | 15:59 | |
| *** catintheroof has joined #openstack-cinder | 15:59 | |
| *** josecastroleon has quit IRC | 16:00 | |
| *** catintheroof has quit IRC | 16:00 | |
| *** catintheroof has joined #openstack-cinder | 16:00 | |
| *** randomha1k has joined #openstack-cinder | 16:01 | |
| *** sdague has quit IRC | 16:03 | |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - None connector object in a terminate_connection in Pike https://review.openstack.org/482130 | 16:03 |
| *** arnewiebalck has quit IRC | 16:04 | |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Error attaching failed over volumes. https://review.openstack.org/488410 | 16:05 |
| *** Poornima has quit IRC | 16:05 | |
| e0ne | ildikov: we've got the same error with my patch:( | 16:07 |
| *** lpetrut has quit IRC | 16:07 | |
| ildikov | e0ne: I just checked the gate logs :( | 16:07 |
| ildikov | e0ne: manually attach worked in my env now | 16:08 |
| e0ne | ildikov: good to know | 16:08 |
| ildikov | e0ne: and I only applied your patch and smcginnis's, didn't do anything else | 16:08 |
| *** rcernin has quit IRC | 16:08 | |
| ildikov | e0ne: so I have no idea now what's going on | 16:08 |
| e0ne | ildikov: how to check that gate uses my patch? | 16:09 |
| *** kaisers has quit IRC | 16:10 | |
| *** kaisers has joined #openstack-cinder | 16:10 | |
| ildikov | e0ne: no idea, I added the Depends-On tag so in theory it should work... | 16:11 |
| e0ne | ildikov: I can't find anything related in the logs:( | 16:11 |
| e0ne | I ask this question to infra team | 16:13 |
| ildikov | ok | 16:13 |
| ildikov | I also run tempest locally now | 16:13 |
| ildikov | will let you know what the results were once it's done | 16:13 |
| ildikov | might take a bit more time though, I was optimistic and started a full run... | 16:13 |
| e0ne | ildikov: thanks | 16:14 |
| *** pcaruana has quit IRC | 16:14 | |
| *** kaisers has quit IRC | 16:14 | |
| *** armax has joined #openstack-cinder | 16:18 | |
| *** baojg has quit IRC | 16:19 | |
| *** baojg has joined #openstack-cinder | 16:19 | |
| *** baojg has quit IRC | 16:20 | |
| *** baojg has joined #openstack-cinder | 16:20 | |
| *** baojg has quit IRC | 16:20 | |
| *** baojg has joined #openstack-cinder | 16:21 | |
| *** baojg has quit IRC | 16:21 | |
| *** rawanh has quit IRC | 16:21 | |
| *** jistr|mtg is now known as jistr | 16:22 | |
| *** itlinux has joined #openstack-cinder | 16:28 | |
| armax | smcginnis: another fix to make cinder be happy with postgres as backend: https://review.openstack.org/#/c/489706/ | 16:30 |
| armax | smcginnis: hi! :) | 16:30 |
| *** hoonetorg has quit IRC | 16:30 | |
| smcginnis | armax: Thanks! Looks like the pg job is happy now. I'll take a look at that shortly. | 16:31 |
| armax | smcginnis: aye, it looks like there’s a migration/model mismatch that went uncaught | 16:31 |
| smcginnis | armax: Oh dang. Nice catch then. | 16:32 |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - performance improvements in retype https://review.openstack.org/483933 | 16:33 |
| armax | smcginnis: totally low hanging fruit, but there’s a way to catch these errors in a functional test | 16:34 |
| armax | http://git.openstack.org/cgit/openstack/neutron/tree/doc/source/contributor/testing/template_model_sync_test.rst | 16:34 |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - None connector object in a terminate_connection in Pike https://review.openstack.org/482130 | 16:34 |
| armax | that’s how we do it for neutron | 16:34 |
| armax | but most of the heavy lifting is in oslo.db | 16:34 |
| smcginnis | armax: Nice, with documentation even. | 16:35 |
| smcginnis | armax: Weren't some of those oslo.db tests deprecated though? | 16:36 |
| smcginnis | armax: I saw some of that being done in Glance but haven't dug into it. | 16:36 |
| smcginnis | I just saw lots of deprecation warnings. | 16:36 |
| *** gnufied has joined #openstack-cinder | 16:36 | |
| armax | smcginnis: um, that’s something I am going have to look into :) | 16:36 |
| smcginnis | armax: If you're not seeing warnings then maybe the glance tests just need to be updated to a new base class or something. | 16:37 |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Error attaching failed over volumes. https://review.openstack.org/488410 | 16:37 |
| tommylikehu | hah, anyone could take a look at these three simple patches? https://review.openstack.org/#/c/489503/, https://review.openstack.org/#/c/489542/, https://review.openstack.org/#/c/489160/ | 16:37 |
| smcginnis | Swimingly: You around? lhx_ had some questions about the drdb driver. | 16:37 |
| Swimingly | smcginnis: present | 16:37 |
| armax | the only deprecation message I see is: Use of postgresql in devstack is deprecated, and will be removed during the Pike cycle :/ | 16:38 |
| armax | http://logs.openstack.org/14/488914/6/check/gate-neutron-dsvm-functional-ubuntu-xenial/39e61a8/console.html#_2017-08-02_13_07_55_605334 | 16:38 |
| lhx_ | smcginnis, thanks refer to me :) | 16:38 |
| smcginnis | Swimingly, meet lhx_ - lhx_, meet Swimingly :) | 16:39 |
| Swimingly | lhx_: nice to meet you | 16:39 |
| smcginnis | armax: Hah! Well, glad that didn't happen yet. | 16:39 |
| armax | that goes back to https://review.openstack.org/#/c/430885/ | 16:39 |
| lhx_ | Swimingly, hi, nice to talk with you | 16:40 |
| *** sdague has joined #openstack-cinder | 16:40 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient master: Do not review. test nova dependencies https://review.openstack.org/490079 | 16:40 |
| *** randomha1k has quit IRC | 16:40 | |
| armax | smcginnis: not sure if based on https://review.openstack.org/#/c/427880/14/resolutions/20170613-postgresql-status.rst that would need to be reverted? | 16:41 |
| lhx_ | I considered if I can use drbd driver in product env | 16:41 |
| e0ne | ildikov: fyi https://review.openstack.org/490079 | 16:41 |
| lhx_ | Swimingly, | 16:41 |
| armax | sdague: is postgres support really going to be removed in Pike? | 16:41 |
| armax | from devstack? | 16:41 |
| ildikov | e0ne: coolio | 16:42 |
| lhx_ | Swimingly, you know drbd and lvm can implement the feature of replication | 16:42 |
| smcginnis | lhx_: You mean production, right? Product meaning a product that is sold, production meaning running real world (business critical) workloads. | 16:42 |
| ildikov | e0ne: thanks | 16:42 |
| e0ne | ildikov: you're welcome | 16:42 |
| Swimingly | lhx_: there is someone that I know is using it in a _large_ production cluster, but they're on the other side of the planet, so I don't work with them directly | 16:42 |
| Swimingly | lhx_: all DRBD does it replicate :D | 16:42 |
| *** hoonetorg has joined #openstack-cinder | 16:43 | |
| lhx_ | smcginnis, Oh, sorry, definitely production :D | 16:43 |
| smcginnis | lhx_: Cool, got it then. | 16:43 |
| smcginnis | armax: There was originally a push to make that resolution be that PG support was going away. | 16:43 |
| armax | smcginnis: right, but since then the position changed slightly, didn’t it? | 16:43 |
| smcginnis | armax: It morphed into just stating the current state that it isn't really the primary DB the majority of folks are targetting. | 16:44 |
| smcginnis | armax: So the devstack deprecation warning was maybe just premature. | 16:44 |
| lhx_ | smcginnis, haha, forgive my poor English | 16:44 |
| smcginnis | lhx_: No, it's good. That's just one of those subtle differences that changes the meaning a little. | 16:44 |
| armax | smcginnis: yup, that’s my thinking | 16:44 |
| armax | I’ll kick a revert and see what happens | 16:46 |
| smcginnis | armax: A sure way to get some feedback. ;) | 16:46 |
| lhx_ | Swimingly, but it has no more benefits in production cluster on my impression | 16:48 |
| lhx_ | smcginnis, it's always kind of you ;) | 16:49 |
| *** abhitechie has joined #openstack-cinder | 16:49 | |
| *** firebat has quit IRC | 16:50 | |
| gouthamr | smcginnis: ping, question regarding deprecated drivers.. | 16:50 |
| Swimingly | lhx_: if you just need traditional HA replication then your solution should be fine | 16:51 |
| e0ne | ildikov: need to go away:(. I'll be available tomorrow UTC+3 | 16:53 |
| e0ne | maybe, I'll be online later tonight | 16:53 |
| ildikov | e0ne: thanks for the help! | 16:54 |
| ildikov | e0ne: and have a good evening :) | 16:54 |
| e0ne | ildikov: np. thanks | 16:54 |
| lhx_ | Swimingly, yeah, it's the point, I don't know the comparison of DRBD + lvm and other replicated driver | 16:54 |
| lhx_ | could you tell something in your point of view? | 16:55 |
| Swimingly | lhx_: here's the user's guide section on our driver: http://docs.linbit.com/doc/users-guide-90/ch-openstack/ | 16:55 |
| *** yangyapeng has quit IRC | 16:56 | |
| *** yangyapeng has joined #openstack-cinder | 16:56 | |
| lhx_ | Swimingly, I saw it these days, but not get the high point of view | 16:57 |
| *** e0ne has quit IRC | 16:57 | |
| Swimingly | lhx_: I imagine that scaling out would be simpler than using the LVM solution, e.g. adding new storage nodes. I don't know _anything_ about the lvm cinder driver so I can't really compare them | 16:57 |
| *** randomha1k has joined #openstack-cinder | 16:57 | |
| *** e0ne has joined #openstack-cinder | 16:58 | |
| *** crushil has quit IRC | 16:58 | |
| *** e0ne has quit IRC | 16:58 | |
| *** abhitechie has quit IRC | 16:59 | |
| *** rodri has joined #openstack-cinder | 16:59 | |
| *** abhitechie has joined #openstack-cinder | 16:59 | |
| *** yangyapeng has quit IRC | 17:00 | |
| lhx_ | Swimingly, btw, have you use this?https://github.com/openstack/drbd-devstack | 17:01 |
| lhx_ | or just deploy DRBD with devstack? | 17:01 |
| Swimingly | lhx_: I'm not the original author of the driver | 17:02 |
| Swimingly | lhx_: and I've been pulled away recently to work on other projects so I haven't made major changes just yet | 17:02 |
| lhx_ | Swimingly, That's ok, :) | 17:03 |
| lhx_ | thanks | 17:04 |
| smcginnis | gouthamr: What's the question? | 17:05 |
| *** rodri has left #openstack-cinder | 17:05 | |
| gouthamr | smcginnis: Hey Sean! so we wanted to rollback a deprecation announced at the beginning of Pike.. | 17:05 |
| gouthamr | smcginnis: I was going to send out an email again, but wanted to check with you before reverting https://review.openstack.org/#/c/456990 | 17:06 |
| smcginnis | gouthamr: Hah, changed your mind, huh? :) | 17:07 |
| smcginnis | gouthamr: Should be fine to go ahead. CI is running, right? | 17:07 |
| *** tesseract has quit IRC | 17:07 | |
| gouthamr | smcginnis: :) yessir.. it's still up.. will this be subject to any string freeze? | 17:07 |
| gouthamr | (we're past it i know) | 17:07 |
| smcginnis | gouthamr: Since this is actually just removing a translated string by doing a revert, that should be fine. | 17:08 |
| smcginnis | Worse case scenario is they translate a string in the catalog that no longer exists in the code. | 17:08 |
| openstackgerrit | Felipe Monteiro proposed openstack/cinder master: [api-ref] Add api-ref for os-quota-class-sets APIs https://review.openstack.org/477740 | 17:08 |
| gouthamr | smcginnis: oh.. thank you! this is good news. I'll revert/send the email! | 17:09 |
| smcginnis | gouthamr: Sounds good. I'll check it out later. | 17:09 |
| *** crushil has joined #openstack-cinder | 17:09 | |
| *** dikonoor has joined #openstack-cinder | 17:11 | |
| *** diablo_rojo has joined #openstack-cinder | 17:12 | |
| *** harlowja has joined #openstack-cinder | 17:13 | |
| gouthamr | smcginnis: sure thing.. | 17:13 |
| openstackgerrit | Gerald McBrearty proposed openstack/cinder master: Make VolumeTypeExtraSpecsController policy more granular https://review.openstack.org/486740 | 17:16 |
| *** randomha1k has quit IRC | 17:18 | |
| *** ntpttr_laptop__ has joined #openstack-cinder | 17:20 | |
| *** ntpttr_laptop__ has quit IRC | 17:20 | |
| *** ntpttr_laptop has joined #openstack-cinder | 17:20 | |
| *** baojg has joined #openstack-cinder | 17:22 | |
| *** baojg has quit IRC | 17:27 | |
| *** arnewiebalck_ has joined #openstack-cinder | 17:29 | |
| *** ChubYann has joined #openstack-cinder | 17:30 | |
| openstackgerrit | Merged openstack/python-brick-cinderclient-ext master: Update the documentation link for doc migration https://review.openstack.org/489901 | 17:32 |
| ildikov | scottda: the bug appears in a DevStack environment as far as I'm aware | 17:38 |
| *** gnufied has quit IRC | 17:39 | |
| ildikov | scottda: not sure whether the client should be any more advanced with regards to URL handling in that call as of yet | 17:39 |
| scottda | ildikov: Well, the method is designed to just take the URL that you give it, and then get the version info. So the caller must figure out the correct URL. | 17:40 |
| scottda | ildikov: So, this occurs when nova instantiates the cinderclient? | 17:40 |
| ildikov | scottda: this is from an earlier gate log: iniset /etc/cinder/cinder.conf DEFAULT public_endpoint https://10.16.218.31 | 17:41 |
| ildikov | scottda: the endpoint_url is wrong | 17:41 |
| ildikov | so whenever get_server_version is hit it blows up | 17:42 |
| scottda | OK. Just trying to figure out where the problem is. | 17:43 |
| ildikov | which is called from get_highest_client_server_version that we use in Nova to figure out whether 3.27 is available for the new attach flow or not | 17:43 |
| scottda | But the caller needs to handle that. | 17:43 |
| ildikov | I wish it's a DevStack issue and we don't need a new client release TBH | 17:44 |
| ildikov | How do you mean the caller needs to handle that? | 17:44 |
| scottda | I don't see any issue with the client. IF you pass in garbage, things won't work. The caller should try...except to deal with it. | 17:45 |
| ildikov | then I guess it's a pure DevStack issue | 17:49 |
| ildikov | switching back to the default public_endpoint there didn't solve the issue yesterday | 17:49 |
| *** randomha1k has joined #openstack-cinder | 17:49 | |
| scottda | So, it could also be in the way Nova is getting the endpoint. | 17:49 |
| *** abhitechie has quit IRC | 17:50 | |
| scottda | At least there could be better handling of the failure case. | 17:50 |
| *** abhitechie has joined #openstack-cinder | 17:50 | |
| ildikov | well, the point is not blowing up or not | 17:50 |
| ildikov | but not being able to use the new attach flow | 17:51 |
| ildikov | we can catch the exception and re-raise or smth like, but that doesn't solve the base issue | 17:51 |
| *** abhitechie has quit IRC | 17:52 | |
| ildikov | I'm not aware of changes in how Nova's getting the endpoint | 17:52 |
| *** abhitechie has joined #openstack-cinder | 17:52 | |
| ildikov | the above pasted iniset is surely wrong | 17:53 |
| ildikov | and the failures started when the uWSGI patch in DevStack got merged that I posted in the bug report | 17:53 |
| scottda | Yes, but whatever Nova gets from the service catalog, Nova must strip all the path off, and just have the Cinder endpoint | 17:53 |
| ildikov | I think in this case it gets this: https://10.16.218.31 | 17:54 |
| scottda | right, it needs the cinder port | 17:55 |
| scottda | (and cinder endpoint) | 17:55 |
| scottda | You cannot fix that in the client. | 17:56 |
| smcginnis | With the uWSGI changes I think there's no port, just https://10.16.218.31/volume/ | 17:56 |
| *** yangyapeng has joined #openstack-cinder | 17:56 | |
| scottda | Sorry, not familiar with uWSGI changes... | 17:57 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder-specs master: Provisioning Improvements https://review.openstack.org/490116 | 17:57 |
| smcginnis | I started with the changes since it was a goal for this release, but I'm not really that familiar with how it's supposed to work either. :) | 17:57 |
| ildikov | smcginnis: do you know who we could ask about it? | 17:58 |
| ildikov | smcginnis: it seems like we're a bit in the dark here with this part | 17:58 |
| smcginnis | ildikov: I'm not sure who the expert is on how that is all supposed to work, but maybe sdague? | 17:58 |
| scottda | So, are you creating subnets with a new IP for each service when you have multiple services on the same machine? | 17:58 |
| ildikov | scottda: non of us seems to be :( | 17:58 |
| smcginnis | scottda: No, I think it depends on the path. | 17:59 |
| scottda | In the past, you'd use the same IP (possibly) and different ports, to differentiate, say Cinder and Glance running on the same machine. | 17:59 |
| geguileo | xyang1: winston-d smcginnis After the other day's meeting regarding over-provisioning I've had a look at it and created this spec to try to get the conversation rolling and hopefully be done with it once and for all: https://review.openstack.org/490116 | 17:59 |
| smcginnis | scottda: So http://host/volume/, http:/host/compute/, etc. | 17:59 |
| scottda | And does it need authentication to got to that <ip>/volume path? | 18:00 |
| jgriffith | so basically we have this uWSGI stuff, and this micro-versioning stuff; and nobody really knows what the hell to do with any of it | 18:00 |
| jgriffith | seems about right | 18:00 |
| scottda | Well, I know what to do with microversioning :) | 18:00 |
| smcginnis | jgriffith: Yep | 18:00 |
| jgriffith | haha... yeah scottda I know what to do with it alright :) | 18:00 |
| scottda | haha | 18:00 |
| smcginnis | :) | 18:00 |
| xyang1 | geguileo: ok, I'll take a look | 18:01 |
| geguileo | xyang1: thanks | 18:01 |
| jgriffith | seriously though... we're kind of in a bad state; there's not much agreement or understanding on all the levers here. How do we fix that? | 18:01 |
| scottda | Seems like mucking with the paths can break a lot of things. At least scripts that count on the paths | 18:01 |
| smcginnis | Default is to still run the way we have been running. | 18:01 |
| *** yangyapeng has quit IRC | 18:01 | |
| jgriffith | scottda perhaps, but unless someone proposes an alternative fix I'm not sure what else we should do? | 18:02 |
| smcginnis | My take was this enabled the groundwork to start being able to use one root endpoint for all services. | 18:02 |
| smcginnis | So I'm sure more work will likely need to be done. | 18:02 |
| scottda | Well, it certainly breaks the ability to hit the cinder root endpoint, and get version info. | 18:02 |
| eharney | one root http service, they still have separate endpoints | 18:02 |
| jgriffith | smcginnis +1 that was my belief too... and being able to drop in things like nginx | 18:04 |
| smcginnis | Here's the goal doc FWIW: https://governance.openstack.org/tc/goals/pike/deploy-api-in-wsgi.html | 18:04 |
| jgriffith | regardless | 18:04 |
| scottda | eharney: And what's returned for a GET to the root endpoint? Basic API practice would say version info for all the services beneath that endpoint. | 18:04 |
| smcginnis | But I'm noticing a huge lack in additional context and overall plan reading through it now. | 18:04 |
| *** arnewiebalck_ has quit IRC | 18:04 | |
| jgriffith | smcginnis haah... that's funny | 18:04 |
| eharney | scottda: i think the root is not an endpoint? | 18:05 |
| jgriffith | wait.. did I start to say that out loud | 18:05 |
| *** rawanh has joined #openstack-cinder | 18:05 | |
| smcginnis | :) | 18:05 |
| scottda | eharney: OK, well maybe just cinder api needs to change so that 'http://<ip>/volume' returns the version info that is expected. | 18:05 |
| smcginnis | scottda: If I understand right, that will get our version info. | 18:06 |
| eharney | scottda: i don't understand why that isn't happening already | 18:06 |
| scottda | eharney: smcginnis Maybe it is . I just started looking at this. Then then it would seem that the caller needs to know what that cinder "root" endpoint looks like, and use it. | 18:07 |
| smcginnis | scottda: It does get registered as xxx/volume/. So maybe nova is stripping off more than it should? | 18:08 |
| ildikov | that doesn't sound backward compatible to me as is | 18:08 |
| scottda | https://github.com/openstack/nova/blob/master/nova/volume/cinder.py#L121 | 18:09 |
| ildikov | so with this patch the whole thing seems to work properly: https://review.openstack.org/#/c/490040/1/cinderclient/client.py | 18:09 |
| scottda | Doesn't look like Nova is doing any parsing. | 18:09 |
| smcginnis | It should be backwards compatible. It shouldn't matter if the catalog returns http://host:12345/ or http://host/volume | 18:09 |
| eharney | what's the compatibility concern? | 18:09 |
| ildikov | which means the client gets the '/volume' end | 18:09 |
| ildikov | scottda: it does not | 18:09 |
| ildikov | eharney: if we say that the caller should handle things differently from now one | 18:09 |
| ildikov | s/one/on/ | 18:09 |
| *** lhx_ has quit IRC | 18:10 | |
| *** gnufied has joined #openstack-cinder | 18:11 | |
| smcginnis | The caller should get the endpoint from the catalog and use that to construct the client. | 18:11 |
| ildikov | without additional magic | 18:12 |
| scottda | Yeah, nova just gets the url here: https://github.com/openstack/nova/blob/master/nova/context.py#L73 | 18:13 |
| scottda | So, it depends on what's in the service catalog, and the values we have in nova.conf.cinder | 18:15 |
| *** rawanh has quit IRC | 18:17 | |
| scottda | What's in 'openstack catalog |grep volume'? My devstack is a couple weeks old, spinning up a new one now... | 18:22 |
| *** dikonoor has quit IRC | 18:23 | |
| *** baojg has joined #openstack-cinder | 18:23 | |
| ildikov | scottda: running a tempest suite now, will check after | 18:23 |
| *** stvnoyes has quit IRC | 18:24 | |
| *** baojg has quit IRC | 18:28 | |
| openstackgerrit | Merged openstack/cinder master: Switch to using bool for filtering non-deleted volume attributes https://review.openstack.org/489706 | 18:28 |
| ildikov | https://www.irccloud.com/pastebin/UKJeCow3/ | 18:29 |
| ildikov | scottda: that's the catalog list ^^ | 18:29 |
| ildikov | scottda: but I have smcginnis's one line change applied in my env, so no idea what it would be like without that | 18:30 |
| smcginnis | Those looks fine to me. Without my change, it was just overridding that to the public_endpoint value. | 18:31 |
| *** links has quit IRC | 18:31 | |
| *** adisky__ has quit IRC | 18:32 | |
| *** ntpttr_laptop has quit IRC | 18:33 | |
| *** abhitechie has quit IRC | 18:35 | |
| *** abhitechie has joined #openstack-cinder | 18:35 | |
| ildikov | smcginnis: without e0ne's change there was still an error in the logs at the same place | 18:35 |
| smcginnis | ildikov: That's the odd part. If it's taking the URL from the catalog, I don't know what else would need to change. | 18:36 |
| ildikov | smcginnis: I have no idea :( | 18:37 |
| smcginnis | ildikov: And you don't know what it was blowing up on? | 18:37 |
| ildikov | smcginnis: I saw that only on the gate | 18:37 |
| scottda | Well, it look like you cannot just take the URL returned from keystone catalog, you need only http://<ip>/volume | 18:39 |
| *** abhitechie has quit IRC | 18:39 | |
| scottda | you don't want the v3/908ad6c238664ca2895e7f23c707482e part | 18:39 |
| *** abhitechie has joined #openstack-cinder | 18:39 | |
| *** abhitechie has quit IRC | 18:41 | |
| *** abhitechie has joined #openstack-cinder | 18:41 | |
| ildikov | smcginnis: can't find that now, maybe I mixed it up with something :S | 18:42 |
| ildikov | smcginnis: here it is: http://logs.openstack.org/85/330285/109/check/gate-tempest-dsvm-py35-ubuntu-xenial/a6c266f/logs/screen-n-api.txt.gz?level=ERROR | 18:44 |
| smcginnis | ildikov: Does that show somewhere what's being passed in? | 18:45 |
| ildikov | smcginnis: so another job from the same run reports the other error I copied in the bug report | 18:45 |
| ildikov | smcginnis: I don't think so | 18:45 |
| smcginnis | Found it: https://147.135.132.61/volume/v3/2f8af088a4db4870b36851d55b6331d8/volumes/8ee1b580-c7a3-4641-947b-4fc2a8cc559c | 18:46 |
| ildikov | that looks weird | 18:46 |
| ildikov | I saw that when I was testing it manually yesterday | 18:47 |
| scottda | Well, I think Ivan's fix is on the right path, modulo eharney 's concerns... | 18:53 |
| *** gnufied has quit IRC | 18:56 | |
| *** abhinavtechie has joined #openstack-cinder | 18:56 | |
| *** abhitechie has quit IRC | 18:59 | |
| *** kaisers has joined #openstack-cinder | 19:09 | |
| *** nicolasbock__ has quit IRC | 19:10 | |
| *** abhinavtechie has quit IRC | 19:14 | |
| *** edmondsw has quit IRC | 19:15 | |
| *** edmondsw has joined #openstack-cinder | 19:17 | |
| *** edmondsw has quit IRC | 19:21 | |
| *** edmondsw has joined #openstack-cinder | 19:23 | |
| *** edmondsw has quit IRC | 19:27 | |
| *** e0ne has joined #openstack-cinder | 19:27 | |
| *** edmondsw has joined #openstack-cinder | 19:29 | |
| *** edmondsw has quit IRC | 19:33 | |
| *** e0ne has quit IRC | 19:34 | |
| *** edmondsw has joined #openstack-cinder | 19:35 | |
| *** edmondsw has quit IRC | 19:39 | |
| *** edmondsw has joined #openstack-cinder | 19:41 | |
| *** edmondsw has quit IRC | 19:45 | |
| *** edmondsw has joined #openstack-cinder | 19:46 | |
| *** edmondsw has quit IRC | 19:47 | |
| *** edmondsw has joined #openstack-cinder | 19:47 | |
| *** edmondsw has quit IRC | 19:50 | |
| *** randomha1k has quit IRC | 19:51 | |
| *** randomha1k has joined #openstack-cinder | 19:54 | |
| *** ntpttr_laptop has joined #openstack-cinder | 19:54 | |
| *** ntpttr_laptop has quit IRC | 19:54 | |
| *** yangyapeng has joined #openstack-cinder | 19:57 | |
| *** e0ne has joined #openstack-cinder | 20:01 | |
| *** edmondsw has joined #openstack-cinder | 20:01 | |
| *** yangyapeng has quit IRC | 20:02 | |
| *** hoonetorg has quit IRC | 20:03 | |
| *** edmondsw has quit IRC | 20:05 | |
| *** edmondsw has joined #openstack-cinder | 20:07 | |
| openstackgerrit | Rawan Herzallah proposed openstack/os-brick master: Adding NVMEoF for initiator CLI https://review.openstack.org/482642 | 20:09 |
| *** edmondsw has quit IRC | 20:09 | |
| *** edmondsw has joined #openstack-cinder | 20:10 | |
| openstackgerrit | Rawan Herzallah proposed openstack/cinder master: Adding NVMEoF target for LVM driver https://review.openstack.org/482562 | 20:10 |
| hemna | geguileo, still around? | 20:12 |
| *** e0ne has quit IRC | 20:15 | |
| *** liverpooler has quit IRC | 20:16 | |
| hemna | smcginnis, ping | 20:20 |
| *** randomha1k has quit IRC | 20:20 | |
| *** hoonetorg has joined #openstack-cinder | 20:20 | |
| *** Julien-zte has quit IRC | 20:21 | |
| *** baojg has joined #openstack-cinder | 20:25 | |
| *** Julien-zte has joined #openstack-cinder | 20:25 | |
| *** e0ne has joined #openstack-cinder | 20:27 | |
| *** baojg has quit IRC | 20:29 | |
| *** dikonoor has joined #openstack-cinder | 20:31 | |
| *** https_GK1wmSU has joined #openstack-cinder | 20:31 | |
| smcginnis | hemna: Back now. What's up? | 20:33 |
| hemna | do we have any drivers that support HA yet? | 20:33 |
| hemna | ceph? | 20:33 |
| patrickeast | nope :( | 20:33 |
| hemna | none? | 20:33 |
| hemna | pewp! | 20:33 |
| smcginnis | hemna: None. | 20:33 |
| hemna | ok. damn | 20:34 |
| patrickeast | we never did the exercise of deciding how/when to let them flip the flags on | 20:34 |
| smcginnis | Waiting on the certification test criteria. | 20:34 |
| *** https_GK1wmSU has left #openstack-cinder | 20:34 | |
| hemna | wasn't there a spec that we had to insert into get_volume_stats to enable it? | 20:34 |
| patrickeast | its just a flag on the driver, and if c-vol starts up with one in a cluster it pewps out | 20:35 |
| smcginnis | hemna: Yeah, I believe so. But we disabled all drivers until they can certify they are OK running that way. | 20:35 |
| smcginnis | So certainly possible to just flip the flag and run, but we need actual criteria to know if it's working as it should. | 20:35 |
| patrickeast | easy to just flip to a "True" in the source though ;) makes a good barrier for entry... if a deployer knows enough to find the flag, change it in the source code, *and* deploy cinder that way... then they can probably handle cleaning up the flaming wreckage if it blows up | 20:36 |
| smcginnis | ;) | 20:36 |
| hemna | and HA is only for c-vol right? not c-sched, nor c-api or c-bak ? | 20:36 |
| patrickeast | nah, its all of them | 20:36 |
| patrickeast | c-vol is the only one that can't be part of a cluster right now though | 20:37 |
| patrickeast | everything else is (afaik) good to go | 20:37 |
| smcginnis | Interesting point about c-bak though. I wonder if we need to do the same for backup drivers. | 20:37 |
| patrickeast | people use that one? :D | 20:38 |
| smcginnis | Apparently one or two. :) | 20:38 |
| smcginnis | Although I've taken informal polls at ops midcycles and no one at those were using it. | 20:38 |
| hemna | I'm asking internally... | 20:39 |
| *** kaisers has quit IRC | 20:40 | |
| hemna | ah ok so c-bak can HA ? | 20:43 |
| hemna | did geguileo ever write up a test scenario for this? | 20:43 |
| scottda | c-api was always HA, since it reads from the messageQ. | 20:43 |
| hemna | I presume the docs on setting this up are now in cinder? | 20:43 |
| scottda | Just kill 'em off and let the Q sort them out.. | 20:43 |
| openstackgerrit | Merged openstack/cinder master: Correct 'os-show_image_metadata' API document https://review.openstack.org/489542 | 20:45 |
| smcginnis | hemna: Hmm, I'm not finding it in the config reference. | 20:46 |
| *** kaisers has joined #openstack-cinder | 20:47 | |
| smcginnis | I thought we had that, but maybe that was also part of the task of documenting the certification testing/ | 20:47 |
| *** cknight has quit IRC | 20:49 | |
| *** kaisers has quit IRC | 20:52 | |
| *** e0ne has quit IRC | 20:52 | |
| *** savvas has quit IRC | 20:53 | |
| *** lucasxu has quit IRC | 21:00 | |
| *** dustins has quit IRC | 21:01 | |
| hemna | that would be nice to get some update from geguileo on that. | 21:01 |
| hemna | I'm tasked with the HA support and testing | 21:01 |
| hemna | so, I might try and get it working for ceph and maybe 3par, since I think I have access to those soon | 21:02 |
| *** savvas has joined #openstack-cinder | 21:04 | |
| *** imacdonn has quit IRC | 21:08 | |
| *** imacdonn has joined #openstack-cinder | 21:08 | |
| *** eharney has quit IRC | 21:10 | |
| *** kaisers has joined #openstack-cinder | 21:10 | |
| smcginnis | hemna: You can probably use his blog post about testing it out and just use those backends for it. | 21:10 |
| openstackgerrit | Merged openstack/cinder master: HPE Lefthand: Fixed cloning operation https://review.openstack.org/462624 | 21:13 |
| openstackgerrit | Merged openstack/cinder master: Cloning a volume from a snapshot in thin lv https://review.openstack.org/488264 | 21:13 |
| openstackgerrit | Merged openstack/cinder master: [DOC] VMware: Update config table https://review.openstack.org/489824 | 21:13 |
| openstackgerrit | Merged openstack/cinder master: [DOC] VMware: Add doc for vmware:adapter_type https://review.openstack.org/489832 | 21:13 |
| *** savvas has quit IRC | 21:15 | |
| *** savvas has joined #openstack-cinder | 21:15 | |
| *** savvas has quit IRC | 21:20 | |
| *** lpetrut has joined #openstack-cinder | 21:21 | |
| *** savvas has joined #openstack-cinder | 21:22 | |
| *** baojg has joined #openstack-cinder | 21:26 | |
| *** savvas has quit IRC | 21:27 | |
| *** baojg has quit IRC | 21:30 | |
| *** edmondsw has quit IRC | 21:39 | |
| *** edmondsw has joined #openstack-cinder | 21:40 | |
| *** savvas has joined #openstack-cinder | 21:40 | |
| *** dikonoor has quit IRC | 21:41 | |
| *** edmondsw has quit IRC | 21:46 | |
| openstackgerrit | Merged openstack/cinder master: Add API document for quota usage. https://review.openstack.org/489160 | 21:48 |
| *** savvas has quit IRC | 21:53 | |
| *** yangyapeng has joined #openstack-cinder | 21:58 | |
| *** xyang1 has quit IRC | 21:59 | |
| *** armax has quit IRC | 22:01 | |
| *** yangyapeng has quit IRC | 22:03 | |
| *** felipemonteiro__ has quit IRC | 22:10 | |
| *** lpetrut has quit IRC | 22:14 | |
| *** https_GK1wmSU has joined #openstack-cinder | 22:15 | |
| *** gouthamr has quit IRC | 22:16 | |
| *** https_GK1wmSU has left #openstack-cinder | 22:17 | |
| *** thingee has quit IRC | 22:19 | |
| *** thingee has joined #openstack-cinder | 22:32 | |
| *** catintheroof has quit IRC | 22:44 | |
| *** sdague has quit IRC | 22:50 | |
| hemna | is s the encryption_key_id ever set in the volume table? | 22:58 |
| hemna | I just created an encrypted volume and attached it | 22:58 |
| hemna | the enc_key_id is all 0 | 22:58 |
| *** yangyapeng has joined #openstack-cinder | 23:02 | |
| *** savvas has joined #openstack-cinder | 23:04 | |
| *** yangyapeng has quit IRC | 23:07 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder master: VMware: Bump driver version https://review.openstack.org/490245 | 23:08 |
| *** savvas has quit IRC | 23:09 | |
| *** itlinux has quit IRC | 23:20 | |
| *** wxy- has quit IRC | 23:31 | |
| *** gouthamr has joined #openstack-cinder | 23:32 | |
| *** mtanino has quit IRC | 23:39 | |
| *** mtanino_ is now known as mtanino | 23:39 | |
| hemna | hrmm | 23:49 |
| hemna | looks like cinder create encrypted volume with barbican is broken | 23:49 |
| hemna | http://paste.openstack.org/show/617351/ | 23:52 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!