| *** jistr has quit IRC | 00:03 | |
| *** jistr has joined #openstack-cinder | 00:07 | |
| *** dave-mcc_ has joined #openstack-cinder | 00:08 | |
| *** yangyape_ has quit IRC | 00:09 | |
| *** dave-mccowan has quit IRC | 00:10 | |
| *** dave-mcc_ has quit IRC | 00:13 | |
| *** lixiaoy1 has joined #openstack-cinder | 00:16 | |
| *** salv-orl_ has quit IRC | 00:27 | |
| *** dave-mccowan has joined #openstack-cinder | 00:40 | |
| *** wanghao has joined #openstack-cinder | 00:41 | |
| *** s-shiono has joined #openstack-cinder | 00:41 | |
| *** stvnoyes has quit IRC | 00:51 | |
| *** dave-mccowan has quit IRC | 00:51 | |
| *** stvnoyes has joined #openstack-cinder | 00:51 | |
| *** Apoorva has joined #openstack-cinder | 00:57 | |
| *** gouthamr has joined #openstack-cinder | 01:15 | |
| *** yangyapeng has joined #openstack-cinder | 01:17 | |
| *** Apoorva has quit IRC | 01:17 | |
| *** knomura has joined #openstack-cinder | 01:18 | |
| *** knomura has quit IRC | 01:23 | |
| *** salv-orlando has joined #openstack-cinder | 01:28 | |
| *** s-shiono has quit IRC | 01:30 | |
| *** huanxuan has joined #openstack-cinder | 01:33 | |
| *** salv-orlando has quit IRC | 01:35 | |
| *** gouthamr has quit IRC | 01:41 | |
| openstackgerrit | Zhao Liqiang proposed openstack/cinder: RBD: Fix RBD replication on volumes from image source. https://review.openstack.org/418288 | 01:45 |
|---|---|---|
| *** gcb has joined #openstack-cinder | 01:54 | |
| *** mars has joined #openstack-cinder | 01:59 | |
| *** mars has quit IRC | 01:59 | |
| *** mrda is now known as mrda-lca | 02:00 | |
| *** mrda-lca is now known as mrda | 02:00 | |
| *** nkrinner_afk has quit IRC | 02:12 | |
| *** nkrinner_afk has joined #openstack-cinder | 02:12 | |
| *** stevemar has quit IRC | 02:21 | |
| *** stevemar has joined #openstack-cinder | 02:21 | |
| *** links has joined #openstack-cinder | 02:25 | |
| *** salv-orlando has joined #openstack-cinder | 02:31 | |
| *** salv-orlando has quit IRC | 02:36 | |
| *** links has quit IRC | 02:38 | |
| *** links has joined #openstack-cinder | 02:46 | |
| *** dave-mccowan has joined #openstack-cinder | 03:02 | |
| openstackgerrit | Zhao Liqiang proposed openstack/cinder: RBD: two types of 'volume_type' should be handled respectively. https://review.openstack.org/418278 | 03:02 |
| *** dave-mccowan has quit IRC | 03:06 | |
| *** tlian has quit IRC | 03:13 | |
| *** mriedem has quit IRC | 03:15 | |
| *** bardia has joined #openstack-cinder | 03:17 | |
| openstackgerrit | Zhao Liqiang proposed openstack/cinder: RBD: two types of 'volume_type' should be handled respectively. https://review.openstack.org/418278 | 03:18 |
| *** bardia has quit IRC | 03:21 | |
| *** pgadiya has joined #openstack-cinder | 03:22 | |
| *** sticker has joined #openstack-cinder | 03:24 | |
| *** dave-mccowan has joined #openstack-cinder | 03:31 | |
| *** pbandark has joined #openstack-cinder | 03:32 | |
| *** salv-orlando has joined #openstack-cinder | 03:32 | |
| *** salv-orlando has quit IRC | 03:36 | |
| openstackgerrit | Ryan Liang proposed openstack/cinder: Unity Driver: Backup volume via snapshot https://review.openstack.org/410587 | 03:37 |
| *** dave-mccowan has quit IRC | 03:45 | |
| *** sudipto_ has joined #openstack-cinder | 03:50 | |
| *** sudipto has joined #openstack-cinder | 03:50 | |
| *** psachin has joined #openstack-cinder | 04:01 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/os-brick: H803 hacking have been deprecated https://review.openstack.org/420490 | 04:02 |
| *** bswartz has quit IRC | 04:03 | |
| *** mdnadeem has joined #openstack-cinder | 04:05 | |
| *** gouthamr has joined #openstack-cinder | 04:23 | |
| *** bswartz has joined #openstack-cinder | 04:31 | |
| *** bswartz has quit IRC | 04:32 | |
| *** salv-orlando has joined #openstack-cinder | 04:33 | |
| *** bswartz has joined #openstack-cinder | 04:36 | |
| *** salv-orlando has quit IRC | 04:38 | |
| *** bswartz has quit IRC | 04:44 | |
| *** bswartz has joined #openstack-cinder | 04:54 | |
| *** Lee1092 has joined #openstack-cinder | 05:01 | |
| *** sudipto_ has quit IRC | 05:01 | |
| *** sudipto has quit IRC | 05:01 | |
| *** pdeore has joined #openstack-cinder | 05:08 | |
| *** knomura has joined #openstack-cinder | 05:24 | |
| *** knomura has quit IRC | 05:28 | |
| *** Apoorva has joined #openstack-cinder | 05:29 | |
| *** salv-orlando has joined #openstack-cinder | 05:34 | |
| *** sudipto has joined #openstack-cinder | 05:36 | |
| *** sudipto_ has joined #openstack-cinder | 05:36 | |
| *** Apoorva has quit IRC | 05:37 | |
| *** salv-orlando has quit IRC | 05:38 | |
| *** lixiaoy1 has quit IRC | 05:40 | |
| *** lixiaoy1 has joined #openstack-cinder | 05:43 | |
| *** wanghao has quit IRC | 05:58 | |
| *** wanghao has joined #openstack-cinder | 05:58 | |
| *** Jack_Iv has joined #openstack-cinder | 06:00 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 06:05 | |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/cinder: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/420507 | 06:08 |
| *** lkuchlan has joined #openstack-cinder | 06:11 | |
| *** Jack_Iv has quit IRC | 06:12 | |
| *** bkopilov_ has joined #openstack-cinder | 06:14 | |
| gcb | cinder core reviewers, please help review https://review.openstack.org/#/c/413446/, which is a easy straight fix deprecation warnings from oslo.message. | 06:19 |
| *** lixiaoy1 has quit IRC | 06:26 | |
| openstackgerrit | Satish Venkatasubramanian proposed openstack/cinder: Add snapshots/summary API support https://review.openstack.org/407123 | 06:31 |
| *** salv-orlando has joined #openstack-cinder | 06:35 | |
| *** salv-orlando has quit IRC | 06:39 | |
| *** lpetrut has joined #openstack-cinder | 06:42 | |
| *** mnaser has quit IRC | 06:53 | |
| *** afazekas has quit IRC | 06:53 | |
| *** afazekas has joined #openstack-cinder | 06:57 | |
| *** Jack_I has joined #openstack-cinder | 07:00 | |
| *** e0ne has joined #openstack-cinder | 07:00 | |
| *** afazekas has quit IRC | 07:02 | |
| *** mars has joined #openstack-cinder | 07:08 | |
| *** mnaser has joined #openstack-cinder | 07:10 | |
| *** lkuchlan is now known as lkuchlan_afk | 07:11 | |
| *** afazekas has joined #openstack-cinder | 07:13 | |
| *** e0ne has quit IRC | 07:14 | |
| *** tesseract has joined #openstack-cinder | 07:21 | |
| *** hogepodge_ has joined #openstack-cinder | 07:24 | |
| *** nkrinner_afk is now known as nkrinner | 07:24 | |
| *** Poornima has joined #openstack-cinder | 07:27 | |
| *** mdnadeem has quit IRC | 07:27 | |
| *** arne_r has joined #openstack-cinder | 07:28 | |
| *** gouthamr has quit IRC | 07:32 | |
| *** e0ne has joined #openstack-cinder | 07:33 | |
| *** pcaruana has joined #openstack-cinder | 07:34 | |
| *** salv-orlando has joined #openstack-cinder | 07:36 | |
| *** hogepodge_ has quit IRC | 07:37 | |
| *** salv-orlando has quit IRC | 07:40 | |
| *** mdnadeem has joined #openstack-cinder | 07:42 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/cinder: Replace yaml.load() with yaml.safe_load() https://review.openstack.org/420541 | 07:44 |
| *** gcb has quit IRC | 07:44 | |
| *** gcb has joined #openstack-cinder | 07:46 | |
| *** mjura has joined #openstack-cinder | 08:12 | |
| *** e0ne has quit IRC | 08:14 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Add an option to impleve the performance of attach/detach volume https://review.openstack.org/420559 | 08:15 |
| *** Poornima has quit IRC | 08:19 | |
| *** obre has joined #openstack-cinder | 08:30 | |
| *** trananhkma has joined #openstack-cinder | 08:30 | |
| *** adisky_ has joined #openstack-cinder | 08:35 | |
| *** salv-orlando has joined #openstack-cinder | 08:37 | |
| *** salv-orlando has quit IRC | 08:41 | |
| *** lkuchlan_afk has quit IRC | 08:43 | |
| *** lkuchlan_afk has joined #openstack-cinder | 08:45 | |
| *** wanghao_ has joined #openstack-cinder | 08:47 | |
| *** wanghao has quit IRC | 08:47 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: Storwize: do the correct update after failover https://review.openstack.org/292570 | 08:53 |
| *** lkuchlan_afk is now known as lkuchlan_wfh | 08:58 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-cinder | 09:00 | |
| *** pgadiya has quit IRC | 09:01 | |
| *** mars has quit IRC | 09:05 | |
| *** alonmarx has joined #openstack-cinder | 09:05 | |
| *** mars has joined #openstack-cinder | 09:05 | |
| *** thurloat has quit IRC | 09:06 | |
| *** e0ne has joined #openstack-cinder | 09:08 | |
| *** thurloat has joined #openstack-cinder | 09:08 | |
| *** houming has joined #openstack-cinder | 09:09 | |
| *** pgadiya has joined #openstack-cinder | 09:10 | |
| *** jordanP has joined #openstack-cinder | 09:15 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX driver - QoS key fix https://review.openstack.org/420072 | 09:22 |
| *** wanghao_ has quit IRC | 09:24 | |
| *** e0ne has quit IRC | 09:25 | |
| *** e0ne has joined #openstack-cinder | 09:26 | |
| *** lkuchlan_wfh has quit IRC | 09:31 | |
| *** rha has joined #openstack-cinder | 09:33 | |
| *** rha has quit IRC | 09:33 | |
| *** rha has joined #openstack-cinder | 09:33 | |
| *** lkuchlan_wfh has joined #openstack-cinder | 09:35 | |
| *** salv-orlando has joined #openstack-cinder | 09:37 | |
| *** salv-orlando has quit IRC | 09:42 | |
| *** mvk has quit IRC | 09:47 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/os-brick: Fix sphinx type error https://review.openstack.org/410877 | 09:54 |
| *** e0ne has quit IRC | 09:55 | |
| *** e0ne has joined #openstack-cinder | 09:57 | |
| *** yumiriam has joined #openstack-cinder | 10:00 | |
| *** eantyshev has quit IRC | 10:01 | |
| *** yangyapeng has quit IRC | 10:06 | |
| *** nikeshm has joined #openstack-cinder | 10:10 | |
| nikeshm | hi did any one face this issue in CI : NotImplementedError: Manage existing snapshot not implemented | 10:11 |
| *** mdovgal has joined #openstack-cinder | 10:18 | |
| dulek | nikeshm: Let me find some info on that… | 10:19 |
| dulek | nikeshm: http://lists.openstack.org/pipermail/openstack-dev/2017-January/110184.html | 10:19 |
| *** mvk has joined #openstack-cinder | 10:20 | |
| dulek | nikeshm: Look for the end of the meeting: http://eavesdrop.openstack.org/meetings/cinder/2017/cinder.2017-01-11-16.00.log.html | 10:20 |
| dulek | 16:53:17 - that's when the topic starts. | 10:21 |
| dulek | nikeshm: And: https://review.openstack.org/#/c/418922/ | 10:21 |
| nikeshm | http://54.209.116.144/78/418278/3/check/kaminario-dsvm-tempest-full-iscsi/ae81809/logs/tempest.txt.gz | 10:21 |
| dulek | nikeshm: And here: https://review.openstack.org/#/c/418928 | 10:21 |
| nikeshm | volume-feature-enabled.manage_snapshot = True in my CI | 10:21 |
| nikeshm | by default, i hope it should be false | 10:22 |
| dulek | nikeshm: Exactly. :) | 10:22 |
| nikeshm | https://github.com/openstack/tempest/blob/3125625e0f7904f138e1ed0c5a9e5321c68f686c/tempest/config.py#L807 | 10:22 |
| nikeshm | but since i did not mention any thing in CI job, it should make it True | 10:23 |
| nikeshm | it should not make it True | 10:24 |
| dulek | e0ne: Hi. Care to look at https://review.openstack.org/#/c/363010 again? CI had passed successfully. | 10:25 |
| e0ne | dulek: hi. sure. I'll do it | 10:26 |
| *** ntt has joined #openstack-cinder | 10:33 | |
| ntt | Hi, it is possible to use cinder replication with 2 separated lvm backend devices? | 10:33 |
| *** salv-orlando has joined #openstack-cinder | 10:38 | |
| *** salv-orlando has quit IRC | 10:43 | |
| *** jordanP has quit IRC | 11:01 | |
| *** lpetrut has quit IRC | 11:03 | |
| mdovgal | hemna, hi! | 11:05 |
| nikeshm | dulek: thanks | 11:10 |
| pjm6 | Hi there | 11:10 |
| pjm6 | anyone here tried shared storage for nova with pNFS (NFS v4.1)? | 11:10 |
| *** JoseMello has joined #openstack-cinder | 11:13 | |
| *** erlon_pto has joined #openstack-cinder | 11:15 | |
| *** sudipto has quit IRC | 11:17 | |
| *** sudipto_ has quit IRC | 11:17 | |
| *** yangyapeng has joined #openstack-cinder | 11:17 | |
| *** dmesg has joined #openstack-cinder | 11:18 | |
| *** Namrata has joined #openstack-cinder | 11:20 | |
| *** Namrata has quit IRC | 11:21 | |
| *** Jack_Iv has joined #openstack-cinder | 11:22 | |
| *** lpetrut has joined #openstack-cinder | 11:26 | |
| *** lpetrut has quit IRC | 11:30 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Specify the ports for compute nodes and copy functions in VSP drivers https://review.openstack.org/414076 | 11:36 |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Specify the ports for compute nodes and copy functions in VSP drivers https://review.openstack.org/414076 | 11:39 |
| *** salv-orlando has joined #openstack-cinder | 11:39 | |
| openstackgerrit | Arnon Yaari proposed openstack/cinder: Use infinisdk for communication with InfiniBox storage array https://review.openstack.org/418451 | 11:43 |
| *** ccesario has joined #openstack-cinder | 11:43 | |
| *** salv-orlando has quit IRC | 11:43 | |
| *** alyson_ has joined #openstack-cinder | 11:44 | |
| openstackgerrit | Arnon Yaari proposed openstack/cinder: Use infinisdk for communication with InfiniBox storage array https://review.openstack.org/418451 | 11:48 |
| *** pdeore has quit IRC | 11:50 | |
| *** lseki has joined #openstack-cinder | 11:51 | |
| *** arne_r has quit IRC | 11:53 | |
| *** lkuchlan_wfh has quit IRC | 11:54 | |
| *** lkuchlan has joined #openstack-cinder | 11:55 | |
| *** lkuchlan has quit IRC | 11:56 | |
| *** lkuchlan has joined #openstack-cinder | 11:57 | |
| *** bkopilov_ has quit IRC | 11:57 | |
| *** bkopilov_ has joined #openstack-cinder | 11:58 | |
| *** ccesario has quit IRC | 11:58 | |
| *** lpetrut has joined #openstack-cinder | 11:59 | |
| *** pgadiya has quit IRC | 12:02 | |
| *** ccesario has joined #openstack-cinder | 12:02 | |
| *** Jack_Iv has quit IRC | 12:03 | |
| *** jbernard has quit IRC | 12:05 | |
| *** jbernard has joined #openstack-cinder | 12:11 | |
| *** eantyshev has joined #openstack-cinder | 12:11 | |
| *** ociuhandu has quit IRC | 12:15 | |
| *** thiagolib has joined #openstack-cinder | 12:18 | |
| *** ociuhandu has joined #openstack-cinder | 12:19 | |
| *** adrianofr has joined #openstack-cinder | 12:19 | |
| *** sdague has joined #openstack-cinder | 12:22 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Add an option to impleve the performance of attach/detach volume https://review.openstack.org/420559 | 12:22 |
| *** sudipto has joined #openstack-cinder | 12:31 | |
| *** sudipto_ has joined #openstack-cinder | 12:31 | |
| *** nicolasbock has joined #openstack-cinder | 12:34 | |
| *** sudipto_ has quit IRC | 12:37 | |
| *** sudipto has quit IRC | 12:37 | |
| *** khushbu_ has joined #openstack-cinder | 12:37 | |
| *** sudipto has joined #openstack-cinder | 12:40 | |
| *** salv-orlando has joined #openstack-cinder | 12:40 | |
| *** sudipto_ has joined #openstack-cinder | 12:41 | |
| *** salv-orlando has quit IRC | 12:44 | |
| *** salv-orlando has joined #openstack-cinder | 12:46 | |
| *** houming has quit IRC | 12:49 | |
| *** lkuchlan is now known as lkuchlan_wfh | 12:53 | |
| nikeshm | how to make volume-feature-enabled.manage_snapshot it false in CI job | 12:57 |
| *** mars has quit IRC | 12:58 | |
| *** huanxuan has quit IRC | 13:00 | |
| *** gcb has quit IRC | 13:01 | |
| *** Lee1092 has quit IRC | 13:06 | |
| ildikov | dulek: hi | 13:08 |
| dulek | ildikov: Hi, what's up? | 13:09 |
| ildikov | dulek: I just wanted to ask whether yo had a chance to take a look at the new attach/detach API patch? | 13:10 |
| ildikov | *you | 13:11 |
| dulek | ildikov: I was reviewing it recently, but meanwhile geguileo rebased his A/A HA patches, so I've jumped there. I'm planning to get back to attachment APIs today. | 13:11 |
| ildikov | dulek: I remember last week you referred to it as you're just about to and I'm curious what's your take on it :) | 13:11 |
| ildikov | dulek: sounds great | 13:12 |
| ildikov | dulek: thanks much! | 13:12 |
| *** khushbu_ has quit IRC | 13:16 | |
| *** zhonghua has quit IRC | 13:17 | |
| *** zhonghua has joined #openstack-cinder | 13:19 | |
| *** liverpooler has joined #openstack-cinder | 13:28 | |
| *** erlon_pto is now known as erlon | 13:33 | |
| *** psachin has quit IRC | 13:34 | |
| openstackgerrit | Merged openstack/cinder: tgt: Remove existence check before delete https://review.openstack.org/416274 | 13:40 |
| openstackgerrit | Merged openstack/cinder: Fix response bootable parameter type in api document https://review.openstack.org/404581 | 13:42 |
| *** xyang1 has joined #openstack-cinder | 13:44 | |
| *** salv-orlando has quit IRC | 13:46 | |
| *** mdnadeem has quit IRC | 13:46 | |
| *** zengyingzhe has joined #openstack-cinder | 13:52 | |
| *** tlian has joined #openstack-cinder | 13:55 | |
| *** jwcroppe_ has quit IRC | 14:00 | |
| zengyingzhe | Hi all, does anyone know why the CI reports this error "InvalidRequestError: Entity '<class 'cinder.db.sqlalchemy.models.Volume'>' has no property 'admin_metadata'"? | 14:01 |
| *** crose has joined #openstack-cinder | 14:06 | |
| dulek | zengyingzhe: Can you point to where it happens? It seems that you just need to use volume_admin_metadata in that place. | 14:06 |
| dulek | Or make sure an o.vo instead of a SQLA model will be passed there. | 14:06 |
| *** jwcroppe has joined #openstack-cinder | 14:13 | |
| *** paw has quit IRC | 14:13 | |
| *** dave-mccowan has joined #openstack-cinder | 14:14 | |
| zengyingzhe | dulek, thanks for your reply. It happens while creating volume, volume.save() | 14:18 |
| *** mkoderer has joined #openstack-cinder | 14:20 | |
| dulek | zengyingzhe: Hm, I think it should be an o.vo passed everywhere in the create_volume flow, so that's most curious. Are you able to provide a link to CI results with this bug? | 14:21 |
| openstackgerrit | Merged openstack/cinder: Allow triggering cleanup from API https://review.openstack.org/363010 | 14:22 |
| zengyingzhe | dulek, our CI run locally, I can't provide a link. but I've copy the log out, could you please take a look at it? | 14:22 |
| dulek | zengyingzhe: Sure, thanks! | 14:23 |
| dulek | zengyingzhe: You'll need to host it somewhere or send to my email box (michal.dulko@intel.com) - my corporate proxy blocks this port. | 14:26 |
| zengyingzhe | dulek, OK, please wait a sec. Thanks. | 14:27 |
| *** links has quit IRC | 14:30 | |
| mdovgal | geguileo, Gorka, hi. Do you have a minute? I want to ask you about your comment to https://review.openstack.org/#/c/415194/ | 14:34 |
| *** mriedem has joined #openstack-cinder | 14:38 | |
| dulek | zengyingzhe: Is the driver you're using upstream? | 14:39 |
| zengyingzhe | dulek, sorry, I don't quite understand 'upstream' you said. | 14:41 |
| dulek | zengyingzhe: I mean - is it merged to openstack/cinder master? | 14:41 |
| dulek | zengyingzhe: I wanted to take a quick look on the code. | 14:41 |
| zengyingzhe | dulek, yes, it's huawei driver. | 14:42 |
| dulek | zengyingzhe: Okay, looking! | 14:42 |
| zengyingzhe | Is this error related to driver? The exception trace stack doesn't look like involving with driver. | 14:44 |
| dulek | zengyingzhe: I believe it's because driver returns something inappropriate in model_update. I need a moment to think this through. | 14:44 |
| *** ccesario has quit IRC | 14:45 | |
| openstackgerrit | Lucian Petrut proposed openstack/cinder: Deprecate the Linux SMBFS driver https://review.openstack.org/420745 | 14:46 |
| *** salv-orlando has joined #openstack-cinder | 14:47 | |
| *** pbandark has quit IRC | 14:47 | |
| zengyingzhe | dulek, you mean something like this model_update['admin_metadata'] = admin_metadata? | 14:48 |
| dulek | zengyingzhe: When had it started to happen? I think I understand why there's a failure, but I can't figure out what could cause it to manifest recently. | 14:49 |
| dulek | zengyingzhe: Actually that line seems correct, it's how we're saving the admin_metadata that concerns me. | 14:49 |
| *** gcb has joined #openstack-cinder | 14:50 | |
| *** salv-orlando has quit IRC | 14:51 | |
| *** pbandark has joined #openstack-cinder | 14:51 | |
| zengyingzhe | dulek, it happened since last month, I updated the tempest and cinder code. | 14:51 |
| *** jungleboyj has joined #openstack-cinder | 14:54 | |
| *** pbandark has quit IRC | 14:54 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Save model_update as admin in create_volume https://review.openstack.org/420749 | 14:55 |
| dulek | zengyingzhe: Can you try if it happens with above patch applied? | 14:55 |
| dulek | I mean this one: https://review.openstack.org/420749 | 14:55 |
| openstackgerrit | wangxiyuan proposed openstack/cinder: Improvement to get group detail(Part 2) https://review.openstack.org/409694 | 14:55 |
| openstackgerrit | wangxiyuan proposed openstack/cinder: Add api version history for 3.23 https://review.openstack.org/419228 | 14:55 |
| *** paw has joined #openstack-cinder | 14:57 | |
| zengyingzhe | dulek, this patch seems to make the volume.save() always run as admin? | 14:58 |
| *** e0ne has quit IRC | 15:00 | |
| dulek | zengyingzhe: Yup, but it's only internal context, so it doesn't matter too much. | 15:01 |
| dulek | zengyingzhe: https://github.com/openstack/cinder/blob/17b374a9a030dc076615e5154c53b78788f7829d/cinder/objects/volume.py#L364-L364 | 15:01 |
| dulek | zengyingzhe: That's the line that requires it, I suppose. | 15:01 |
| *** mtanino has joined #openstack-cinder | 15:01 | |
| zengyingzhe | dulek, I see. Thank you very much. So if turns out this's indeed the reason causing this error, what should I do? Does it mean I must change huawei driver code to not return admin_metadata in model_update? | 15:03 |
| dulek | zengyingzhe: Either we merge my fix or you can change the driver code to return volume_admin_metadata in model_update instead. | 15:07 |
| openstackgerrit | Merged openstack/python-cinderclient: modify the wrong comment of the Client class https://review.openstack.org/417812 | 15:07 |
| dulek | But I prefer to merge the generic fix. Can you fill a bug for that and I'll get my patch into a merge'able state. | 15:07 |
| smcginnis | dulek: Ping me if/when you submit that. | 15:08 |
| dulek | smcginnis: Sure, no problem. | 15:08 |
| jungleboyj | smcginnis: Welcome back? Weren't killed my your karate cohorts? | 15:09 |
| smcginnis | jungleboyj: Just a little sore. ;) | 15:09 |
| jungleboyj | :-) | 15:10 |
| dulek | Hey, by the way - since achievement of assert:supports-rolling-upgrades, we've maxed maturity metric in project navigator: https://www.openstack.org/software/releases/newton/components/cinder | 15:10 |
| dulek | :) | 15:10 |
| smcginnis | dulek: Woo hoo! :) | 15:10 |
| smcginnis | Level up. | 15:10 |
| jungleboyj | smcginnis: Sore is good. :-) | 15:13 |
| smcginnis | jungleboyj: Yes - it's a good sore. | 15:13 |
| *** e0ne has joined #openstack-cinder | 15:14 | |
| *** diablo_rojo_phon has joined #openstack-cinder | 15:15 | |
| openstackgerrit | Michael Dovgal proposed openstack/python-cinderclient: Fix adding non-ascii attrs to Resource objects error https://review.openstack.org/415194 | 15:17 |
| zengyingzhe | dulek, thanks a lot for your help. I'll rerun the CI tomorrow, to see if it works. Thanks. | 15:18 |
| dulek | zengyingzhe: Okay, ping me once you'll get the results. :) | 15:19 |
| smcginnis | Cores, I would like to do a release of os-brick today ahead of the non-client library freeze this Thursday to make sure there aren't any surprises. | 15:21 |
| smcginnis | I see a couple patches out there that aren't critical, but are also just need +A. | 15:21 |
| smcginnis | https://review.openstack.org/#/q/status:open+project:openstack/os-brick | 15:21 |
| smcginnis | I'll wait until this afternoon in case anyone wants to take a look at those. | 15:21 |
| smcginnis | Specifically, https://review.openstack.org/413684 and https://review.openstack.org/416672 | 15:22 |
| *** Poornima has joined #openstack-cinder | 15:30 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Make Image Volume Cache cluster aware https://review.openstack.org/386186 | 15:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add replication info to generate_driver_list https://review.openstack.org/403854 | 15:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Prevent claiming and updating races on worker https://review.openstack.org/412534 | 15:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add active/active info to generate_driver_list https://review.openstack.org/404180 | 15:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [PoC][Don't review] Testing delay https://review.openstack.org/353069 | 15:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix replication freeze mechanism https://review.openstack.org/402923 | 15:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Move service and cluster creation in test to utils https://review.openstack.org/402922 | 15:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Make Replication support Active-Active https://review.openstack.org/381835 | 15:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Make notify_service_capabilities cluster aware https://review.openstack.org/413200 | 15:31 |
| *** dustins has joined #openstack-cinder | 15:33 | |
| *** nkrinner is now known as nkrinner_afk | 15:35 | |
| *** diablo_rojo has joined #openstack-cinder | 15:37 | |
| geguileo | mdovgal: sorry, I was having lunch | 15:42 |
| geguileo | mdovgal: I see that you already figured out what I meant :-) | 15:42 |
| mdovgal | geguileo, yeap) i just want to discuss your comment in a way like "to do or not to do", but after i agreed, that your advise is better) | 15:44 |
| geguileo | mdovgal: I reviewed it and there are only a couple of nits, so I +2 it | 15:45 |
| smcginnis | geguileo: Hey, not sure if you saw it last week, but we were discussing a focused sprint this Wednesday and Thursday to work on getting A/A through. | 15:47 |
| *** gouthamr has joined #openstack-cinder | 15:47 | |
| smcginnis | geguileo: Are you going to be available if we do that? | 15:47 |
| *** salv-orlando has joined #openstack-cinder | 15:48 | |
| *** thiagolib has quit IRC | 15:48 | |
| geguileo | smcginnis: that would be awesome!!! | 15:48 |
| geguileo | smcginnis: I *will* be available!! | 15:49 |
| smcginnis | geguileo: It really would be great if we can get those out of the way. | 15:49 |
| smcginnis | geguileo: Great! | 15:49 |
| smcginnis | geguileo: Hopefully we can save you from a few weeks more of rebasing. ;) | 15:49 |
| geguileo | lol | 15:49 |
| *** dave-mccowan has quit IRC | 15:50 | |
| *** mvk has quit IRC | 15:50 | |
| *** ducttape_ has joined #openstack-cinder | 15:52 | |
| *** lpetrut has quit IRC | 15:52 | |
| *** salv-orlando has quit IRC | 15:52 | |
| openstackgerrit | Michael Dovgal proposed openstack/python-cinderclient: Fix adding non-ascii attrs to Resource objects error https://review.openstack.org/415194 | 15:55 |
| mdovgal | geguileo, hm...thank you for a comment about github. i didn't think about it | 15:55 |
| *** ducttape_ has quit IRC | 15:55 | |
| geguileo | np | 15:55 |
| *** thiagolib has joined #openstack-cinder | 15:56 | |
| *** crose has quit IRC | 15:56 | |
| *** eharney has joined #openstack-cinder | 15:58 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Save model_update as admin in create_volume https://review.openstack.org/420749 | 15:59 |
| mdovgal | geguileo, i've updated commit message according to your comments. Can you take a look at it again? | 15:59 |
| mdovgal | thank you) | 15:59 |
| geguileo | mdovgal: thanks! | 16:00 |
| dulek | smcginnis: Here's the fix for admin_metadata ready. I was able to create a regression unit test, so I'm pretty sure this solves the problem for Huawei driver. | 16:01 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Save model_update as admin in create_volume https://review.openstack.org/420749 | 16:02 |
| *** eharney has quit IRC | 16:05 | |
| *** dave-mccowan has joined #openstack-cinder | 16:05 | |
| smcginnis | dulek: Awesome, thanks! | 16:05 |
| *** dave-mcc_ has joined #openstack-cinder | 16:08 | |
| *** dave-mccowan has quit IRC | 16:10 | |
| *** crose has joined #openstack-cinder | 16:11 | |
| *** eharney has joined #openstack-cinder | 16:18 | |
| *** alonmarx has quit IRC | 16:20 | |
| *** kfarr has joined #openstack-cinder | 16:23 | |
| *** crose has quit IRC | 16:23 | |
| *** Apoorva has joined #openstack-cinder | 16:24 | |
| *** alonmarx has joined #openstack-cinder | 16:25 | |
| *** salv-orlando has joined #openstack-cinder | 16:30 | |
| *** ducttape_ has joined #openstack-cinder | 16:34 | |
| *** lkuchlan_wfh has quit IRC | 16:37 | |
| *** dustins_ has joined #openstack-cinder | 16:40 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/420320 | 16:41 |
| *** gcb has quit IRC | 16:42 | |
| *** Apoorva has quit IRC | 16:44 | |
| *** e0ne has quit IRC | 16:45 | |
| *** lkuchlan_wfh has joined #openstack-cinder | 16:46 | |
| *** dustins_ has quit IRC | 16:47 | |
| *** kfarr has quit IRC | 16:49 | |
| *** jose-phillips has joined #openstack-cinder | 16:52 | |
| *** jose-phi_ has joined #openstack-cinder | 16:54 | |
| *** jose-phillips has quit IRC | 16:54 | |
| hemna | sup peeps | 16:57 |
| *** ducttape_ has quit IRC | 16:58 | |
| smcginnis | hemna: Hey, I'd like to cut a new os-brick today to make sure there aren't any surprises after the freeze Thurs. If you get a chance, can you take a look at open os-brick patches? | 17:00 |
| hemna | sure I'll go through them again today | 17:01 |
| smcginnis | hemna: Might just be a couple we want to get through. I didn't see anything critical thoguh. | 17:01 |
| smcginnis | hemna: Thanks! | 17:01 |
| hemna | I've been trying to stay on them the last week or so | 17:01 |
| hemna | when is our code freeze? 27th? | 17:01 |
| hemna | for cinder | 17:01 |
| *** knomura has joined #openstack-cinder | 17:02 | |
| smcginnis | hemna: This Thurs for non-client (os-brick), next Thurs for clients, next next Thurs is RC1 and hard string freeze for everything. | 17:02 |
| hemna | ok so we need our driver patch to land by next thurs | 17:03 |
| hemna | https://review.openstack.org/#/c/404958/ | 17:04 |
| hemna | it's sitting there | 17:04 |
| smcginnis | hemna: Will try to take a look soon. | 17:04 |
| hemna | thanks | 17:05 |
| *** dave-mcc_ has quit IRC | 17:06 | |
| *** Apoorva has joined #openstack-cinder | 17:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/420320 | 17:17 |
| erlon | does anyone know how to put devstack screen to log the beautiful old colors? | 17:19 |
| Swanson | erlon, LOG_COLOR=True? | 17:24 |
| *** jose-phi_ has quit IRC | 17:24 | |
| *** mdnadeem has joined #openstack-cinder | 17:25 | |
| erlon | Swanson: this is enabled, the thing is that from a few weeks the colors are different (and very confusing) from what they were before. All debug messages use to be green, but now are bold pink! | 17:26 |
| Swanson | erlon, oh! Nice. I don't know how to change that. | 17:27 |
| *** sudipto has quit IRC | 17:27 | |
| *** sudipto_ has quit IRC | 17:27 | |
| erlon | Swanson: I wonder how other people aren't also mad on that | 17:28 |
| Swanson | erlon, I have it set to false. Makes looking at the logs in vi much nicer. Makes screen less so. | 17:28 |
| erlon | Swanson: hmm, the TRACE ERROR messages also are not bold red, which also makes think much harder | 17:28 |
| ildikov | erlon: I'm getting there to be mad if that helps :) | 17:29 |
| erlon | Swanson: all with 1 color? | 17:29 |
| erlon | ildikov: :) ow great! I'm not alone | 17:29 |
| *** tesseract has quit IRC | 17:29 | |
| *** jose-phillips has joined #openstack-cinder | 17:29 | |
| Swanson | erlon, yeah, I use my editor to highlight what I want to see. | 17:29 |
| *** Poornima has quit IRC | 17:30 | |
| Swanson | erlon, it isn't as nice. | 17:30 |
| ildikov | erlon: I've spent some time with the logs the past couple of days especially and it hurts my eyes to find anything there | 17:30 |
| Swanson | I think I need to stack with color on just to see this. | 17:30 |
| smcginnis | (Swanson still has a black and white TV) | 17:31 |
| erlon | Swanson: that helps, I actually use all graphical, shame on me, ok, :), but still hard | 17:31 |
| erlon | smcginnis: haha | 17:31 |
| erlon | ildikov: yeah, I relied so much in the color | 17:32 |
| ildikov | erlon: definitely easier, saves some nerves when one get that right | 17:33 |
| erlon | ildikov: Swanson: I *believe* that can be related with an external library, which I don't know. The first time I noticed, I had a devstack, had to update cinder, but also some pip packages. Then, the screen of cinder got the way it is today, while the others was normal | 17:34 |
| ildikov | erlon: screen kills my terminal almost every time I try to restart a service nowadays... | 17:35 |
| *** jose-phillips has quit IRC | 17:35 | |
| erlon | ildikov: hmm, I don't use screen also, I have my own version of screen/rejoin screen using KDE konsole | 17:36 |
| *** jose-phillips has joined #openstack-cinder | 17:37 | |
| ildikov | erlon: hmm, good point, maybe I'll look into alternatives as well if it remains like this | 17:37 |
| erlon | ildikov: so I dont have to type Ctrl+A+command every time, my logs does noe have limit, I have a easier search etc | 17:37 |
| erlon | ildikov: but the colors are not set by screen, even in my terminal the colors are weird | 17:38 |
| ildikov | erlon: I guess this could save me significant time if I managed to sort it out at some point | 17:39 |
| ildikov | erlon: sounds better than what I do currently | 17:39 |
| erlon | ildikov: Ill create a project and add it to github when I can, people in my team are liking it too | 17:40 |
| *** jose-phi_ has joined #openstack-cinder | 17:40 | |
| ildikov | erlon: that sounds pretty great, I would be happy to give it a try if you have it up :) | 17:40 |
| erlon | ildikov: the devstack team have actually removed the rejoin-stack, I don't know why | 17:40 |
| erlon | ildikov: nice | 17:40 |
| ildikov | erlon: I've never used rejoin for it's initial purpose, I think it didn't even work well | 17:41 |
| ildikov | erlon: but I don't know the details either | 17:41 |
| erlon | ildikov: so you start devstack and never turns it off right? | 17:42 |
| ildikov | erlon: or just do unstack/stack | 17:42 |
| *** knomura has quit IRC | 17:43 | |
| *** jose-phillips has quit IRC | 17:43 | |
| ildikov | I have a simple env and I don't re-clone too many things so it does not take that much time | 17:43 |
| *** pots2 has joined #openstack-cinder | 17:48 | |
| *** pots2 has quit IRC | 17:52 | |
| *** e0ne has joined #openstack-cinder | 17:58 | |
| *** ducttape_ has joined #openstack-cinder | 17:59 | |
| *** chris_morrell has joined #openstack-cinder | 18:00 | |
| *** chris_morrell has quit IRC | 18:02 | |
| *** clenimar has joined #openstack-cinder | 18:02 | |
| *** chris_morrell has joined #openstack-cinder | 18:02 | |
| *** mjura has quit IRC | 18:03 | |
| *** ducttape_ has quit IRC | 18:03 | |
| *** chris_morrell has quit IRC | 18:04 | |
| hemna | smcginnis, can you look at this https://blueprints.launchpad.net/cinder/+spec/ibmsvciogrpselection | 18:04 |
| hemna | it's been up for a while | 18:04 |
| hemna | they have a patch up for it | 18:05 |
| *** swamireddy has quit IRC | 18:05 | |
| *** mvk has joined #openstack-cinder | 18:06 | |
| *** jdurgin1 has quit IRC | 18:09 | |
| *** alonmarx has quit IRC | 18:10 | |
| *** crose has joined #openstack-cinder | 18:12 | |
| smcginnis | hemna: Done | 18:15 |
| hemna | smcginnis, thanks | 18:16 |
| *** mdnadeem has quit IRC | 18:16 | |
| *** swamireddy has joined #openstack-cinder | 18:17 | |
| *** e0ne has quit IRC | 18:19 | |
| *** aleksey9 has joined #openstack-cinder | 18:21 | |
| *** aleksey9 is now known as alekseyruban | 18:21 | |
| nikeshm | hi what to change in CI jobs to avoid manage snapshots failure | 18:29 |
| nikeshm | i tried http://paste.openstack.org/show/595089/ | 18:31 |
| nikeshm | but looks like not working | 18:31 |
| smcginnis | nikeshm: The latest tempest should disable those by default. | 18:31 |
| nikeshm | i saw one reverting patch from you | 18:32 |
| nikeshm | is it merged | 18:33 |
| nikeshm | hm, but devstack is making it True | 18:33 |
| smcginnis | No, but there was another one that added the config option to disable it by default. | 18:33 |
| *** e0ne has joined #openstack-cinder | 18:33 | |
| smcginnis | This was merged: https://review.openstack.org/#/c/418928/ | 18:34 |
| nikeshm | hm still it is failing for me | 18:34 |
| smcginnis | nikeshm: Try pulling the latest. | 18:35 |
| nikeshm | due to devstack | 18:35 |
| *** salv-orlando has quit IRC | 18:35 | |
| *** salv-orlando has joined #openstack-cinder | 18:36 | |
| nikeshm | it is always running on a fresh VM | 18:36 |
| nikeshm | so it should get updated one | 18:36 |
| nikeshm | i guess | 18:36 |
| nikeshm | smcginnis: what about this https://review.openstack.org/#/c/419073/ | 18:38 |
| *** e0ne has quit IRC | 18:38 | |
| smcginnis | Hmm, hadn't seen that one. | 18:39 |
| Apoorva | smcginnis I am also facing the same issue on our CI. Looks like patch you mentioned was merged on Jan 11, but this failure is from today http://openstack-ci.tintri.com/tintri/refs-changes-94-409694-17/testr_results.html | 18:40 |
| nikeshm | looks like devstack is making it true | 18:40 |
| nikeshm | so we have to disable it in CI jobs or in devstack | 18:41 |
| Apoorva | kaisers smcginnis nikeshm if there is any config option to disable manage snapshot tests in tempest please let me know | 18:42 |
| smcginnis | nikeshm, Apoorva: https://review.openstack.org/#/c/420906/ | 18:43 |
| smcginnis | Apoorva: Yes, the other option is to set that flag to False. | 18:43 |
| *** dave-mccowan has joined #openstack-cinder | 18:44 | |
| Apoorva | smcginnis would that be <volume-feature-enabled manage_snapshot False> in localrc for devstack? | 18:44 |
| *** lpetrut has joined #openstack-cinder | 18:44 | |
| smcginnis | Apoorva: I guess so? localrc has been deprecated for a long time now, so I only use local.conf, but that looks correct. | 18:45 |
| Apoorva | smcginnis: oops, I meant local.conf. thanks | 18:45 |
| smcginnis | :) | 18:45 |
| *** e0ne has joined #openstack-cinder | 18:47 | |
| Apoorva | smcginnis also could you please let me know how would I requalify Tintri driver as supported? https://www.mail-archive.com/openstack-dev@lists.openstack.org/msg98781.html | 18:47 |
| smcginnis | Apoorva: We just need to see the CI running again. Well, running and passing. | 18:47 |
| smcginnis | Apoorva: As soon as there are a few successful runs I can put up a patch to revert the unsupported flag. | 18:48 |
| smcginnis | Apoorva: Or you can and I can help approve it. | 18:48 |
| nikeshm | smcginnis: thanks for revert | 18:49 |
| Apoorva | smcginnis thanks! | 18:49 |
| nikeshm | i tried http://paste.openstack.org/show/595089/ | 18:49 |
| nikeshm | in jenkins jobs but did not work | 18:49 |
| nikeshm | may be have to change some thing else before that revert is merged | 18:50 |
| nikeshm | or have to wait for revert to be merged :) | 18:50 |
| *** tasker has joined #openstack-cinder | 18:51 | |
| tasker | the gerrit review page says "needs workflow" at the top. what does that mean? is there a reference for what's expected? | 18:52 |
| *** bardia has joined #openstack-cinder | 18:55 | |
| *** xinli has joined #openstack-cinder | 18:56 | |
| patrickeast | smcginnis: haha, i put up https://review.openstack.org/#/c/420169 for that dumb tempest test thing too | 18:57 |
| patrickeast | smcginnis: can we just revert the tempest test itself yet? | 18:57 |
| eharney | tasker: just means it hasn't been approved by core reviewers yet | 18:58 |
| smcginnis | patrickeast: That's kind of my preference. The only small thing is it is at least a little value that it tests that code path for LVM. But that's probably better as a functional test perhaps. | 18:58 |
| patrickeast | smcginnis: yea : / | 18:58 |
| patrickeast | smcginnis: i have one thats specific for our drivers that my ci runs (when its ever actually online), which is probably more the direction we need for that kind of test | 18:59 |
| smcginnis | patrickeast: I'll leave mine up for now. Maybe if they are flooded with patches for this they will realize it's not a good idea. | 18:59 |
| smcginnis | patrickeast: Right, I think this is going to vary by driver, so you really need driver specific tests. | 19:00 |
| patrickeast | smcginnis: i hope so... the feedback i've gotten on my comments, irc chats, and that patch has been underwhelming | 19:00 |
| patrickeast | my main concern is that folks don't seem to understand how specific to the backend that test is | 19:01 |
| hemna | heh, this is the problem with not having every driver implement every cinder api feature :( | 19:01 |
| smcginnis | patrickeast: Right. I keep trying to point that out every time, but it doesn't seem to be landing. | 19:01 |
| smcginnis | hemna: No, a lot do implement it. | 19:01 |
| smcginnis | hemna: But the input is different depending on the backend. | 19:02 |
| hemna | oh is this the manage existing snapshot thing? | 19:02 |
| patrickeast | yah | 19:02 |
| smcginnis | hemna: Yerp | 19:02 |
| hemna | bleh | 19:03 |
| patrickeast | hemna: +1 | 19:03 |
| *** Jack_Iv has joined #openstack-cinder | 19:03 | |
| *** Jack_Iv has quit IRC | 19:04 | |
| *** Jack_Iv has joined #openstack-cinder | 19:04 | |
| *** alonmarx has joined #openstack-cinder | 19:07 | |
| *** Jack_Iv has quit IRC | 19:09 | |
| hemna | Is today shower day? | 19:12 |
| jungleboyj | erlon and ildikov They removed rejoin stack because it often didn't work right. I was said when they did that, but I have gotten used to just using screen to start things up. | 19:12 |
| smcginnis | hemna: lol | 19:13 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: DB: Optimize update methods https://review.openstack.org/332944 | 19:14 |
| *** ducttape_ has joined #openstack-cinder | 19:15 | |
| jungleboyj | hemna: I thought you said that was Wednesday. | 19:16 |
| jungleboyj | hemna: I am going to need one though. | 19:16 |
| Swanson | They stole my idea! http://www.businessinsider.com/young-blood-plasma-transfusions-anti-aging-remedy-2017-1 | 19:17 |
| *** yumiriam has quit IRC | 19:18 | |
| *** ducttape_ has quit IRC | 19:19 | |
| *** ducttape_ has joined #openstack-cinder | 19:25 | |
| *** ducttape_ has quit IRC | 19:30 | |
| diablo_rojo_phon | Happy belated birthday jungleboyj and _alastor_ :) | 19:33 |
| jungleboyj | diablo_rojo_phon: Thank you. | 19:39 |
| smcginnis | jungleboyj: Yesterday, right? | 19:39 |
| jungleboyj | smcginnis: Yes. :-) | 19:39 |
| *** paw has quit IRC | 19:40 | |
| smcginnis | jungleboyj: Happy birthday old man. :) | 19:40 |
| markstur | jungleboyj: Happy B-Day! Folks should all take the day off for that | 19:40 |
| jungleboyj | smcginnis: :-p You are just jealous. | 19:40 |
| jungleboyj | markstur: Well, some people do. | 19:40 |
| markstur | but I haven't even declared it JSB Day yet. | 19:41 |
| markstur | How can it be catching on already? | 19:41 |
| smcginnis | jungleboyj: I'm accepted getting old. Can't wait until I can sit on my front step all day and yell at the kids going by. :D | 19:41 |
| *** diablo_rojo has quit IRC | 19:41 | |
| smcginnis | s/I'm/I've/ | 19:41 |
| markstur | +1 to "get off my lawn" | 19:41 |
| smcginnis | markstur: ;) | 19:42 |
| jungleboyj | markstur: MLK beat you to it. | 19:42 |
| jungleboyj | smcginnis: That is a great mental image. | 19:42 |
| openstackgerrit | Rakesh Jain proposed openstack/cinder: Autoselection of IBM SVC/Storwize IO group Currently in IBM storwize_SVC driver, user can specify only one IO group per backend definition. With this change, now user can specify a list of IO groups, and at the time of creating the volume, the driver will https://review.openstack.org/415816 | 19:43 |
| openstackgerrit | Merged openstack/cinder: Dell SC: Failback timeout extended https://review.openstack.org/417972 | 19:44 |
| smcginnis | Hmm, a little more than 54 characters on that one. :D | 19:44 |
| jungleboyj | Oops. | 19:45 |
| *** lkuchlan_wfh has quit IRC | 19:45 | |
| openstackgerrit | Rohan Arora proposed openstack/cinder: Nosec Paramiko exec_command() warning from Bandit https://review.openstack.org/420915 | 19:46 |
| *** kfarr has joined #openstack-cinder | 19:52 | |
| jungleboyj | _alastor_: Happy birthday. When is your birthday? | 19:54 |
| *** knomura has joined #openstack-cinder | 19:54 | |
| *** jerrygb has joined #openstack-cinder | 19:57 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Replace deprecated pylint disable-msg with disable https://review.openstack.org/420918 | 19:57 |
| *** knomura has quit IRC | 19:59 | |
| openstackgerrit | Merged openstack/cinder: Updated from global requirements https://review.openstack.org/420320 | 20:07 |
| *** diablo_rojo has joined #openstack-cinder | 20:09 | |
| openstackgerrit | Merged openstack/cinder: Save model_update as admin in create_volume https://review.openstack.org/420749 | 20:09 |
| *** lkuchlan_wfh has joined #openstack-cinder | 20:18 | |
| *** adisky_ has quit IRC | 20:19 | |
| *** lseki has quit IRC | 20:20 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add replication info to generate_driver_list https://review.openstack.org/403854 | 20:21 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Prevent claiming and updating races on worker https://review.openstack.org/412534 | 20:21 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add active/active info to generate_driver_list https://review.openstack.org/404180 | 20:21 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [PoC][Don't review] Testing delay https://review.openstack.org/353069 | 20:21 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix replication freeze mechanism https://review.openstack.org/402923 | 20:21 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Make notify_service_capabilities cluster aware https://review.openstack.org/413200 | 20:21 |
| *** lkuchlan_wfh has quit IRC | 20:23 | |
| *** salv-orlando has quit IRC | 20:23 | |
| openstackgerrit | Yucong Feng proposed openstack/cinder: Resolve Brocade HTTPS connection error https://review.openstack.org/416371 | 20:26 |
| *** lpetrut has quit IRC | 20:27 | |
| *** kfarr_ has joined #openstack-cinder | 20:30 | |
| *** jose-phi_ has quit IRC | 20:33 | |
| *** dave-mccowan has quit IRC | 20:34 | |
| tasker | what is the "_()" I see surrounding some strings? for example: msg = _("Metadata restore failed due to incompatible version") | 20:34 |
| e0ne | tasker: it's for translations | 20:34 |
| tasker | ahh | 20:34 |
| tasker | thank you. | 20:34 |
| e0ne | tasker: I'm sorry, I'm not able to send you a link to localizations guidelines atm | 20:34 |
| tasker | no worries. | 20:35 |
| tasker | now that I know what it is, I can find it later when I need it. | 20:35 |
| *** kfarr__ has joined #openstack-cinder | 20:35 | |
| *** kfarr__ has quit IRC | 20:35 | |
| *** jose-phillips has joined #openstack-cinder | 20:36 | |
| *** kfarr__ has joined #openstack-cinder | 20:38 | |
| *** kfarr__ has quit IRC | 20:39 | |
| *** kfarr_ has quit IRC | 20:40 | |
| *** kfarr has quit IRC | 20:41 | |
| *** kfarr has joined #openstack-cinder | 20:41 | |
| *** kfarr has quit IRC | 20:41 | |
| *** ducttape_ has joined #openstack-cinder | 20:45 | |
| *** JackWaterworth[m has joined #openstack-cinder | 20:46 | |
| *** Jack_Iv has joined #openstack-cinder | 20:46 | |
| *** charlesr has joined #openstack-cinder | 20:49 | |
| *** stvnoyes has quit IRC | 20:49 | |
| *** ducttape_ has quit IRC | 20:49 | |
| openstackgerrit | Rakesh Jain proposed openstack/cinder: Autoselection of IBM SVC/Storwize IO group Currently in IBM storwize_SVC driver, user can specify only one IO group per backend definition. With this change, now user can specify a list of IO groups, and at the time of creating the volume, the driver will https://review.openstack.org/415816 | 20:54 |
| *** jamielennox|away is now known as jamielennox | 20:59 | |
| *** tasker has quit IRC | 20:59 | |
| *** Jack_Iv has quit IRC | 21:06 | |
| *** alyson_ has quit IRC | 21:07 | |
| *** salv-orlando has joined #openstack-cinder | 21:09 | |
| *** jose-phillips has quit IRC | 21:12 | |
| *** Jack_I has quit IRC | 21:12 | |
| *** jose-phillips has joined #openstack-cinder | 21:14 | |
| *** JoseMello has quit IRC | 21:15 | |
| *** crose has quit IRC | 21:15 | |
| *** charlesr has quit IRC | 21:20 | |
| *** paw has joined #openstack-cinder | 21:25 | |
| *** dustins has quit IRC | 21:31 | |
| *** xinli has quit IRC | 21:31 | |
| *** dave-mccowan has joined #openstack-cinder | 21:32 | |
| *** tasker has joined #openstack-cinder | 21:32 | |
| *** ducttape_ has joined #openstack-cinder | 21:35 | |
| openstackgerrit | Merged openstack/cinder: Skip CI link in list for infra supported drivers https://review.openstack.org/417491 | 21:37 |
| *** ducttape_ has quit IRC | 21:43 | |
| *** ducttape_ has joined #openstack-cinder | 21:44 | |
| *** ducttape_ has quit IRC | 21:44 | |
| tasker | how much should I worry about the 'FAILED' test results on gerrit. I'm trying to go through them and figure out how best to make Jenkins happy. | 21:44 |
| *** diablo_rojo has quit IRC | 21:44 | |
| eharney | tasker: there will be a lot of them -- you really just need to worry about the ones that come from Jenkins | 21:46 |
| *** salv-orlando has quit IRC | 21:47 | |
| tasker | yea. I've been looking through the logs for the Jenkins suite and I'm not seeing anything that screams "this was your fault". one, "gate-cinder-docs-ubuntu-xenial" failed due to: "InvocationError: '/home/jenkins/workspace/gate-cinder-docs-ubuntu-xenial/.tox/venv/bin/python setup.py build_sphinx' | 21:49 |
| tasker | which, i have no idea what that is. | 21:49 |
| eharney | tasker: which patchset is this? | 21:50 |
| tasker | https://review.openstack.org/#/c/413178/ | 21:50 |
| eharney | on py34, ostestr fails with "Illegal instruction"... yikes | 21:52 |
| smcginnis | Hmm, "Illegal instruction" in the logs for one of them. | 21:52 |
| smcginnis | tasker: Might want to just leave the comment "recheck" on that patch. | 21:53 |
| eharney | that could be coming out of librbd etc | 21:53 |
| smcginnis | eharney: Oh, so it could be legitimate? | 21:53 |
| eharney | smcginnis: hard to say really, but it's possible | 21:53 |
| smcginnis | I was thinking something just got messed up in the gate. | 21:53 |
| smcginnis | It actually doesn't look like it even got to the point of using librbd though. | 21:54 |
| *** dave-mccowan has quit IRC | 21:54 | |
| smcginnis | tasker: Try just leaving a comment of "recheck" on that patch to retrigger Jenkins. Something seems fishy there. | 21:54 |
| tasker | ok. | 21:54 |
| eharney | the ceph job passed, so maybe it's not related | 21:55 |
| tasker | sorry, still learning how to use gerrit; I've used 'a' (or reply button), typed "recheck" and submitted. I can see the comment at the bottom. did I do that right? | 21:58 |
| *** alonmarx has quit IRC | 21:58 | |
| eharney | yes | 21:58 |
| tasker | cool. | 21:59 |
| tasker | any idea how long it will take the tests to start again? | 21:59 |
| *** diablo_rojo_phon has quit IRC | 22:00 | |
| eharney | you can get an idea here: http://status.openstack.org/zuul/ | 22:00 |
| eharney | they're already running | 22:01 |
| tasker | ahh, cool. | 22:01 |
| tasker | oh, that's neat. | 22:01 |
| tasker | the good little angle on my shoulder tells me I should only use "recheck" when asked to. or not at all. | 22:06 |
| smcginnis | tasker: ;) | 22:06 |
| tasker | for future reference, where did you find "illegal instruction"? | 22:08 |
| *** eharney has quit IRC | 22:08 | |
| smcginnis | tasker: That was in the results from the py34 test run. If you click on the failed test that brings you to all of the output. | 22:09 |
| smcginnis | tasker: In the console.log it was near the bottom. | 22:09 |
| tasker | ahh. not being familar with Jenkins results, I probably skimmed right past it. | 22:10 |
| smcginnis | tasker: There's so much displayed on that page, it's easy to miss. | 22:11 |
| *** salv-orlando has joined #openstack-cinder | 22:12 | |
| tasker | the status page shows that the docs and py34 tests have passed. | 22:14 |
| smcginnis | tasker: Great. So it must have just been some hiccup in the gate. Hopefully this time is better. | 22:15 |
| tasker | crossing my fingers for the grenade-cinder test. | 22:15 |
| tasker | thanks for the help! | 22:15 |
| smcginnis | tasker: Good not to blindly use recheck, but if you see really odd results it's sometimes better to just recheck. | 22:15 |
| tasker | I'll make sure I read the logs fully and ask here before proceeding. . ) | 22:15 |
| *** diablo_rojo has joined #openstack-cinder | 22:18 | |
| openstackgerrit | Rohan Arora proposed openstack/cinder: Nosec subprocess with shell equal True from Bandit https://review.openstack.org/420979 | 22:21 |
| *** salv-orl_ has joined #openstack-cinder | 22:24 | |
| *** alekseyruban has quit IRC | 22:25 | |
| *** salv-orlando has quit IRC | 22:26 | |
| *** jerrygb has quit IRC | 22:34 | |
| *** jdurgin has quit IRC | 22:39 | |
| *** ducttape_ has joined #openstack-cinder | 22:45 | |
| *** ducttape_ has quit IRC | 22:49 | |
| *** mriedem has quit IRC | 22:51 | |
| *** knomura has joined #openstack-cinder | 22:55 | |
| *** jdurgin has joined #openstack-cinder | 22:57 | |
| openstackgerrit | Rohan Arora proposed openstack/cinder: Nosec Paramiko exec_command() warning from Bandit https://review.openstack.org/420915 | 22:59 |
| *** knomura has quit IRC | 22:59 | |
| openstackgerrit | Rohan Arora proposed openstack/cinder: Nosec subprocess with shell equal True from Bandit https://review.openstack.org/420979 | 23:02 |
| *** diablo_rojo has quit IRC | 23:02 | |
| smcginnis | patrickeast: Excellent analogy! | 23:04 |
| *** medberry is now known as med_ | 23:05 | |
| patrickeast | smcginnis: haha thanks! | 23:08 |
| * patrickeast might start drawing diagrams if it still doesn't work | 23:08 | |
| smcginnis | patrickeast: Let's hope it doesn't come to that. ;) | 23:11 |
| tasker | changing my topic, I'm being hampered by https://bugs.launchpad.net/cinder/+bug/1578036 , but I can't find any trail that says that this has been resolved, or that it isn't an issue in Newton. (my ops folk are currently working to deploy newton to a rack so i can test.) does anyone here have any knowledge that Ceph incremental backups are working in Newton? | 23:14 |
| openstack | Launchpad bug 1578036 in Cinder "ceph incremental backup fails in mitaka" [Medium,Confirmed] - Assigned to Shubham (shubham0d) | 23:14 |
| *** jerrygb has joined #openstack-cinder | 23:22 | |
| *** mriedem has joined #openstack-cinder | 23:23 | |
| *** jose-phillips has quit IRC | 23:23 | |
| *** jwcroppe has quit IRC | 23:26 | |
| *** jerrygb has quit IRC | 23:26 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Prevent driver load failures from objects https://review.openstack.org/420993 | 23:27 |
| *** jose-phillips has joined #openstack-cinder | 23:30 | |
| *** jerrygb has joined #openstack-cinder | 23:34 | |
| *** xyang1 has quit IRC | 23:34 | |
| *** ducttape_ has joined #openstack-cinder | 23:35 | |
| Swanson | This https://review.openstack.org/#/c/420169/ still isn't merged? How is this possible? | 23:40 |
| *** ircuser-1 has joined #openstack-cinder | 23:43 | |
| *** knikolla has quit IRC | 23:47 | |
| *** knikolla has joined #openstack-cinder | 23:47 | |
| *** knikolla has quit IRC | 23:47 | |
| *** knikolla has joined #openstack-cinder | 23:48 | |
| *** tasker has quit IRC | 23:49 | |
| *** salv-orl_ has quit IRC | 23:50 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!