| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/cinder stable/rocky: Fix _per_gb_min usage with _per_gb https://review.openstack.org/626047 | 00:04 |
|---|---|---|
| *** jmlowe has quit IRC | 00:07 | |
| *** itlinux has joined #openstack-cinder | 00:08 | |
| openstackgerrit | Merged openstack/os-brick stable/rocky: Tests: Fix PrivRootwrapTestCase failure https://review.openstack.org/625635 | 00:09 |
| *** mriedem has quit IRC | 00:27 | |
| *** itlinux has quit IRC | 00:28 | |
| *** itlinux has joined #openstack-cinder | 00:28 | |
| *** _alastor_ has joined #openstack-cinder | 00:28 | |
| *** itlinux has quit IRC | 00:29 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/cinder master: Allow to use _max qos option together with per_gb https://review.openstack.org/625058 | 00:29 |
| *** noonedeadpunk[h] has joined #openstack-cinder | 00:31 | |
| openstackgerrit | Merged openstack/cinder master: Retry on DBDeadLock affecting worker updates https://review.openstack.org/611856 | 00:40 |
| *** sapd1 has joined #openstack-cinder | 00:45 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/cinder master: Allow to use _max qos option together with per_gb https://review.openstack.org/625058 | 00:51 |
| noonedeadpunk[h] | smcginnis: hope, that this one ^ is the last one. At least tests are going to pass again. I don't like the variable name, but otherwise have to resolve long line problem, which will look afwul afterwards. | 01:03 |
| openstackgerrit | Merged openstack/cinder master: Add policy test for TENANT_ATTRIBUTE_POLICY https://review.openstack.org/616447 | 01:11 |
| *** itlinux has joined #openstack-cinder | 01:12 | |
| openstackgerrit | Dmitriy Rabotjagov (noonedeadpunk) proposed openstack/cinder master: Allow to use _max qos option together with per_gb https://review.openstack.org/625058 | 01:13 |
| openstackgerrit | Merged openstack/cinder master: Add user_id in backup list and show API https://review.openstack.org/615129 | 01:14 |
| openstackgerrit | Merged openstack/cinder stable/pike: VMAX Pike docs - no support for PowerMax OS https://review.openstack.org/614560 | 01:14 |
| *** noonedeadpunk[h] has quit IRC | 01:18 | |
| *** sapd1 has quit IRC | 01:20 | |
| openstackgerrit | Merged openstack/cinder master: Remove deprecated query_volume_filters config option https://review.openstack.org/620632 | 01:21 |
| *** hedvig_01 has quit IRC | 01:25 | |
| *** _alastor_ has quit IRC | 01:34 | |
| *** Liang__ has joined #openstack-cinder | 01:42 | |
| *** Liang__ is now known as LiangFang | 01:43 | |
| openstackgerrit | Merged openstack/cinder master: api-ref: add missing volumes_links in list-volume interfaces https://review.openstack.org/616094 | 01:59 |
| openstackgerrit | Merged openstack/cinder master: PowerMax driver doc - clarifying SE and Unisphere support https://review.openstack.org/623467 | 01:59 |
| openstackgerrit | Merged openstack/cinder master: Doc: Remove unnecessary note https://review.openstack.org/623435 | 01:59 |
| *** sapd1 has joined #openstack-cinder | 02:04 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 02:08 | |
| *** sapd1 has quit IRC | 02:09 | |
| *** mhen has quit IRC | 02:22 | |
| *** mhen has joined #openstack-cinder | 02:23 | |
| *** Dinesh_Bhor has quit IRC | 02:23 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 02:39 | |
| *** itlinux has quit IRC | 02:40 | |
| openstackgerrit | Ryan Liang proposed openstack/cinder stable/ocata: [Unity] Add support of removing empty host https://review.openstack.org/626069 | 02:47 |
| openstackgerrit | Yong Huang proposed openstack/cinder stable/pike: Unity: Enable ssl verification https://review.openstack.org/626070 | 02:53 |
| *** psachin has joined #openstack-cinder | 02:57 | |
| *** brinzhang has joined #openstack-cinder | 03:04 | |
| *** dave-mccowan has joined #openstack-cinder | 03:06 | |
| *** Bhujay has joined #openstack-cinder | 03:20 | |
| *** dave-mccowan has quit IRC | 03:24 | |
| *** Bhujay has quit IRC | 03:38 | |
| openstackgerrit | Merged openstack/cinder master: Rename BackupNFSSwiftBasedTestCase https://review.openstack.org/611137 | 03:40 |
| *** erlon has quit IRC | 03:44 | |
| *** gnufied has quit IRC | 03:44 | |
| *** Dinesh_Bhor has quit IRC | 03:46 | |
| *** dklyle has quit IRC | 04:02 | |
| *** david-lyle has joined #openstack-cinder | 04:02 | |
| *** sapd1 has joined #openstack-cinder | 04:07 | |
| *** udesale has joined #openstack-cinder | 04:14 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/cinder master: Use renamed template 'integrated-gate-py3' https://review.openstack.org/626083 | 04:28 |
| *** Bhujay has joined #openstack-cinder | 04:33 | |
| *** arne_wiebalck has quit IRC | 05:04 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 05:18 | |
| *** udesale has quit IRC | 05:45 | |
| *** brinzhang has quit IRC | 05:45 | |
| *** udesale has joined #openstack-cinder | 05:49 | |
| openstackgerrit | Yong Huang proposed openstack/cinder master: [Unity] Storage-assisted migration support https://review.openstack.org/612621 | 06:05 |
| *** dims has quit IRC | 06:27 | |
| *** dims has joined #openstack-cinder | 06:28 | |
| *** dims has quit IRC | 06:33 | |
| *** dims has joined #openstack-cinder | 06:34 | |
| *** mgagne has quit IRC | 06:35 | |
| *** mgagne has joined #openstack-cinder | 06:39 | |
| *** Dinesh_Bhor has quit IRC | 06:56 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 07:10 | |
| openstackgerrit | Jonghan Park proposed openstack/cinder master: Support HP MSA 2050 Storage https://review.openstack.org/626104 | 07:18 |
| *** dpawlik has joined #openstack-cinder | 07:21 | |
| *** dpawlik has quit IRC | 07:25 | |
| *** imacdonn has quit IRC | 07:29 | |
| *** imacdonn has joined #openstack-cinder | 07:29 | |
| *** dpawlik has joined #openstack-cinder | 07:36 | |
| *** dpawlik has quit IRC | 07:36 | |
| *** dpawlik_ has joined #openstack-cinder | 07:36 | |
| *** arne_wiebalck has joined #openstack-cinder | 07:46 | |
| *** sapd1 has quit IRC | 07:49 | |
| openstackgerrit | Jonghan Park proposed openstack/cinder master: Support HP MSA 2050 Storage https://review.openstack.org/626104 | 07:52 |
| *** Dinesh_Bhor has quit IRC | 08:03 | |
| *** pcaruana has joined #openstack-cinder | 08:03 | |
| *** arunkant has joined #openstack-cinder | 08:03 | |
| *** sapd1 has joined #openstack-cinder | 08:06 | |
| *** markvoelker has joined #openstack-cinder | 08:10 | |
| *** gkadam has joined #openstack-cinder | 08:11 | |
| *** Bhujay has quit IRC | 08:19 | |
| *** pcaruana has quit IRC | 08:24 | |
| *** helenafm has joined #openstack-cinder | 08:26 | |
| *** pcaruana has joined #openstack-cinder | 08:33 | |
| *** bhagyashris has joined #openstack-cinder | 08:34 | |
| *** pcaruana has quit IRC | 08:41 | |
| *** e0ne has joined #openstack-cinder | 08:47 | |
| *** rcernin has quit IRC | 08:51 | |
| *** wuchongyao has joined #openstack-cinder | 08:56 | |
| *** v12aml has quit IRC | 09:01 | |
| *** Bhujay has joined #openstack-cinder | 09:03 | |
| *** v12aml has joined #openstack-cinder | 09:04 | |
| wuchongyao | #openstack-cinder I'm setting up an CI System and post public key to openstack infra mail list but havn't get response. | 09:04 |
| *** Bhujay has quit IRC | 09:04 | |
| *** Bhujay has joined #openstack-cinder | 09:05 | |
| *** Bhujay has quit IRC | 09:06 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 09:06 | |
| *** Bhujay has joined #openstack-cinder | 09:06 | |
| *** Bhujay has quit IRC | 09:07 | |
| *** Bhujay has joined #openstack-cinder | 09:08 | |
| *** Bhujay has quit IRC | 09:09 | |
| *** Bhujay has joined #openstack-cinder | 09:09 | |
| *** Bhujay has quit IRC | 09:10 | |
| *** Bhujay has joined #openstack-cinder | 09:11 | |
| *** LiangFang has quit IRC | 09:18 | |
| *** zhangxiaofang has joined #openstack-cinder | 09:20 | |
| wuchongyao | I'm setting up an CI System and post public key to x bswartz | 09:21 |
| wuchongyao | | openstack infra mail list but havn't get response. | 09:21 |
| zhangxiaofang | H | 09:24 |
| *** Bhujay has quit IRC | 09:24 | |
| *** luizbag has joined #openstack-cinder | 09:26 | |
| *** wuchongyao has quit IRC | 09:27 | |
| *** e0ne has quit IRC | 09:28 | |
| *** luizbag has quit IRC | 09:31 | |
| *** sapd1 has quit IRC | 09:34 | |
| *** sapd1 has joined #openstack-cinder | 09:35 | |
| *** wuchongyao has joined #openstack-cinder | 09:36 | |
| *** zhangxiaofang has quit IRC | 09:37 | |
| *** luizbag has joined #openstack-cinder | 09:37 | |
| openstackgerrit | caixiaoyu proposed openstack/cinder master: Add test case: retype volume when driver not initialized https://review.openstack.org/618987 | 09:44 |
| *** Bhujay has joined #openstack-cinder | 09:48 | |
| *** e0ne has joined #openstack-cinder | 09:50 | |
| *** Bhujay has quit IRC | 09:51 | |
| *** ganso has joined #openstack-cinder | 09:51 | |
| lennyb | e0ne: frickler regarding MariaDB issue. http://13.74.249.42/30/611530/8/check-cinder/Cinder-ISER-LIO/50ce052/console.html.gz I've upgraded Fedora25 to Fedora27 and the issue was solved. This solution is good enough for me. Thanks | 09:59 |
| *** bhagyashris has quit IRC | 10:00 | |
| *** erlon has joined #openstack-cinder | 10:06 | |
| *** Dinesh_Bhor has quit IRC | 10:09 | |
| *** wuchongyao has quit IRC | 10:10 | |
| *** Bhujay has joined #openstack-cinder | 10:11 | |
| *** Bhujay has quit IRC | 10:12 | |
| *** Bhujay has joined #openstack-cinder | 10:12 | |
| *** Bhujay has quit IRC | 10:13 | |
| *** Bhujay has joined #openstack-cinder | 10:14 | |
| *** erlon_ has joined #openstack-cinder | 10:17 | |
| *** erlon has quit IRC | 10:20 | |
| *** Bhujay has quit IRC | 10:24 | |
| *** psachin has quit IRC | 10:29 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Ignore migration 127 error on older MariaDB https://review.openstack.org/626146 | 10:55 |
| *** dpawlik_ has quit IRC | 11:09 | |
| *** dpawlik has joined #openstack-cinder | 11:09 | |
| *** e0ne has quit IRC | 11:09 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX Driver - Failover Unisphere Support https://review.openstack.org/570401 | 11:10 |
| *** sapd1 has quit IRC | 11:10 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Ignore migration 127 error on MariaDB https://review.openstack.org/626146 | 11:11 |
| *** dpawlik has quit IRC | 11:13 | |
| *** udesale has quit IRC | 11:13 | |
| *** dpawlik has joined #openstack-cinder | 11:13 | |
| *** dpawlik has quit IRC | 11:14 | |
| *** dpawlik has joined #openstack-cinder | 11:14 | |
| *** dpawlik has quit IRC | 11:16 | |
| *** dpawlik has joined #openstack-cinder | 11:17 | |
| *** dpawlik has quit IRC | 11:17 | |
| *** sapd1 has joined #openstack-cinder | 11:17 | |
| *** dpawlik has joined #openstack-cinder | 11:17 | |
| *** Bhujay has joined #openstack-cinder | 11:21 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax driver - rebranding VMAX to PowerMax https://review.openstack.org/620659 | 11:22 |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: [WIP] Add _context to online snapshots in remotefs driver https://review.openstack.org/570885 | 11:27 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax driver - changing from 8.4 to 9.0 REST endpoints https://review.openstack.org/625297 | 11:29 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax driver - code cleanup and minor improvements https://review.openstack.org/625581 | 11:37 |
| openstackgerrit | Helen Walsh proposed openstack/cinder stable/rocky: PowerMax driver doc - clarifying SE and Unisphere support https://review.openstack.org/626161 | 11:52 |
| *** erlon_ has quit IRC | 11:59 | |
| *** e0ne has joined #openstack-cinder | 12:01 | |
| *** dpawlik has quit IRC | 12:06 | |
| *** erlon_ has joined #openstack-cinder | 12:15 | |
| *** tpsilva has joined #openstack-cinder | 12:26 | |
| frickler | lennyb: well, sadly that doesn't solve the issue for centos-7, but thanks for the update | 12:32 |
| lennyb | frickler, what version of centos? | 12:34 |
| frickler | lennyb: probably any version of centos 7, but in gate we are testing the latest, which iirc should be 7.6. see http://zuul.openstack.org/builds?job_name=devstack-platform-centos-7 | 12:37 |
| *** abishop has joined #openstack-cinder | 12:48 | |
| openstackgerrit | Merged openstack/cinder master: VMAX Driver - Place volume in SG as part of unmanage volume https://review.openstack.org/603283 | 12:50 |
| *** dpawlik has joined #openstack-cinder | 12:50 | |
| *** rosmaita has joined #openstack-cinder | 13:03 | |
| *** sapd1 has quit IRC | 13:03 | |
| *** mriedem has joined #openstack-cinder | 13:06 | |
| *** dave-mccowan has joined #openstack-cinder | 13:08 | |
| *** markvoelker has quit IRC | 13:24 | |
| *** helenafm has quit IRC | 13:34 | |
| *** e0ne has quit IRC | 13:43 | |
| *** helenafm has joined #openstack-cinder | 13:44 | |
| *** mmethot has quit IRC | 13:45 | |
| *** mmethot has joined #openstack-cinder | 13:47 | |
| *** e0ne has joined #openstack-cinder | 14:02 | |
| *** Bhujay has quit IRC | 14:02 | |
| *** Bhujay has joined #openstack-cinder | 14:02 | |
| *** Bhujay has quit IRC | 14:03 | |
| *** Bhujay has joined #openstack-cinder | 14:04 | |
| *** Bhujay has quit IRC | 14:05 | |
| openstackgerrit | Sofia Enriquez proposed openstack/cinder stable/queens: Ensure image utils don't block greenthreads https://review.openstack.org/625994 | 14:05 |
| *** Bhujay has joined #openstack-cinder | 14:05 | |
| *** Bhujay has quit IRC | 14:06 | |
| *** Bhujay has joined #openstack-cinder | 14:07 | |
| smcginnis | geguileo: Thanks for the mariadb patch. All the comments I've seen is the error is on newer centos distros, but your patch says it affects older mariadb versions. | 14:09 |
| smcginnis | geguileo: Do you know if that is really the case and things like CentOS 7 are using this older version? | 14:09 |
| geguileo | smcginnis: when I say older I mean 5.6 or earlier | 14:09 |
| geguileo | smcginnis: for centos version I believe the issue is caused by the 4K page sizes | 14:09 |
| smcginnis | So two different causes, same issue? | 14:10 |
| geguileo | yup | 14:10 |
| smcginnis | I thought we set the page size when we initialized the DB... | 14:10 |
| geguileo | I must admit I don't know if we do or don't... | 14:10 |
| smcginnis | Hmm, no, looks like we just set the charset. | 14:11 |
| geguileo | the default seems to be 16K, but maybe for some reason in those installations it's 4K | 14:12 |
| *** Bhujay has quit IRC | 14:12 | |
| e0ne | jungleboyj, smcginnis: hi. I've got few questions accorsing to https://bugs.launchpad.net/cinder/+bug/1802070 | 14:13 |
| openstack | Launchpad bug 1802070 in openstack-ansible "Add resource_filter field in cinder.conf for non-admin user can retrieve volume & snapshot volume" [Undecided,In progress] - Assigned to KimMinsik (for-beatitudo) | 14:13 |
| geguileo | smcginnis: mmmmm, maybe I'm missreading something... looking again.. | 14:14 |
| *** eharney has joined #openstack-cinder | 14:15 | |
| smcginnis | e0ne: I hadn't seen that one yet. | 14:17 |
| e0ne | smcginnis: we discussed it few meetings ago | 14:17 |
| geguileo | smcginnis: OK, I didn't missread it, what I said was right | 14:18 |
| smcginnis | geguileo: Great, thanks! | 14:18 |
| e0ne | is it OK to skip filters in there is no any 'resource_query_filters_file' file or we need to have some defaults in the code? | 14:19 |
| smcginnis | I would think we probably need some defaults. | 14:20 |
| e0ne | smcginnis: +1. I just wanted to be sure on it | 14:21 |
| noonedeadpunk | smcginnis e0ne can you please re-check this patch? https://review.openstack.org/#/c/625058/13 | 14:30 |
| openstackgerrit | Eric Harney proposed openstack/cinder stable/rocky: DNM: Test Ceph job https://review.openstack.org/626178 | 14:32 |
| *** udesale has joined #openstack-cinder | 14:50 | |
| *** idlemind has joined #openstack-cinder | 14:55 | |
| *** erlon_ has quit IRC | 15:01 | |
| *** erlon has joined #openstack-cinder | 15:01 | |
| *** dpawlik has quit IRC | 15:04 | |
| lixiaoy1 | eharney, Hi Eric, as discussed in last week Cinder meeting, could you help to review https://review.openstack.org/#/c/624853 ? | 15:04 |
| *** dpawlik has joined #openstack-cinder | 15:05 | |
| lixiaoy1 | eharney, it is related to driver reinitialization | 15:05 |
| eharney | lixiaoy1: yes, will take a look | 15:06 |
| *** dpawlik has quit IRC | 15:06 | |
| *** dpawlik has joined #openstack-cinder | 15:06 | |
| *** gnufied has joined #openstack-cinder | 15:07 | |
| lixiaoy1 | eharney, thank you | 15:07 |
| *** dpawlik has quit IRC | 15:11 | |
| *** dpawlik has joined #openstack-cinder | 15:23 | |
| *** dpawlik has quit IRC | 15:27 | |
| openstackgerrit | Tiago Pasqualini da Silva proposed openstack/cinder master: Remove support for NetApp E-Series systems https://review.openstack.org/626219 | 15:34 |
| *** erlon_ has joined #openstack-cinder | 15:36 | |
| *** luizbag has quit IRC | 15:39 | |
| *** erlon has quit IRC | 15:39 | |
| smcginnis | tpsilva: Was e-series deprecated in Rocky? I can't recall. | 15:40 |
| tpsilva | smcginnis: yep | 15:41 |
| tpsilva | lemme find the commit | 15:41 |
| tpsilva | smcginnis: https://github.com/openstack/cinder/commit/747373f4bdfe4814000a88e4443b56545d5d55bd#diff-3b21a3b5b1b6cf0593783320da295abb | 15:41 |
| smcginnis | Perfect, thanks tpsilva! | 15:42 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX Driver - Failover Unisphere Support https://review.openstack.org/570401 | 15:42 |
| smcginnis | 10,000+ less lines of code in Cinder. #winning | 15:42 |
| tpsilva | hahaha | 15:42 |
| *** luizbag has joined #openstack-cinder | 15:43 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax driver - rebranding VMAX to PowerMax https://review.openstack.org/620659 | 15:44 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax driver - changing from 8.4 to 9.0 REST endpoints https://review.openstack.org/625297 | 15:47 |
| *** jmlowe has joined #openstack-cinder | 15:47 | |
| *** luizbag_ has joined #openstack-cinder | 15:49 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax driver - code cleanup and minor improvements https://review.openstack.org/625581 | 15:50 |
| *** Liang__ has joined #openstack-cinder | 15:52 | |
| *** luizbag has quit IRC | 15:52 | |
| *** Bhujay has joined #openstack-cinder | 15:57 | |
| *** Liang__ is now known as LiangFang | 15:58 | |
| *** Bhujay has quit IRC | 15:58 | |
| *** Bhujay has joined #openstack-cinder | 15:58 | |
| *** Bhujay has quit IRC | 15:59 | |
| *** Bhujay has joined #openstack-cinder | 16:00 | |
| *** Bhujay has quit IRC | 16:01 | |
| *** Bhujay has joined #openstack-cinder | 16:01 | |
| *** Bhujay has quit IRC | 16:02 | |
| *** Bhujay has joined #openstack-cinder | 16:03 | |
| *** Bhujay has quit IRC | 16:04 | |
| *** Bhujay has joined #openstack-cinder | 16:04 | |
| *** Bhujay has quit IRC | 16:05 | |
| *** Bhujay has joined #openstack-cinder | 16:06 | |
| *** Bhujay has quit IRC | 16:07 | |
| *** Bhujay has joined #openstack-cinder | 16:07 | |
| *** Bhujay has quit IRC | 16:08 | |
| *** Bhujay has joined #openstack-cinder | 16:09 | |
| *** _alastor_ has joined #openstack-cinder | 16:10 | |
| *** Bhujay has quit IRC | 16:10 | |
| *** Bhujay has joined #openstack-cinder | 16:10 | |
| *** Bhujay has quit IRC | 16:11 | |
| *** Bhujay has joined #openstack-cinder | 16:12 | |
| *** Bhujay has quit IRC | 16:13 | |
| *** Bhujay has joined #openstack-cinder | 16:13 | |
| *** Bhujay has quit IRC | 16:14 | |
| *** Bhujay has joined #openstack-cinder | 16:15 | |
| *** itlinux has joined #openstack-cinder | 16:15 | |
| *** Bhujay has quit IRC | 16:16 | |
| *** Bhujay has joined #openstack-cinder | 16:16 | |
| *** Bhujay has quit IRC | 16:17 | |
| *** Bhujay has joined #openstack-cinder | 16:18 | |
| *** Bhujay has quit IRC | 16:19 | |
| *** Bhujay has joined #openstack-cinder | 16:19 | |
| *** Bhujay has quit IRC | 16:20 | |
| *** Bhujay has joined #openstack-cinder | 16:21 | |
| *** Bhujay has quit IRC | 16:22 | |
| *** Bhujay has joined #openstack-cinder | 16:22 | |
| *** Bhujay has quit IRC | 16:23 | |
| *** Bhujay has joined #openstack-cinder | 16:24 | |
| *** luizbag_ has quit IRC | 16:24 | |
| *** eharney has quit IRC | 16:24 | |
| *** Bhujay has quit IRC | 16:25 | |
| *** luizbag has joined #openstack-cinder | 16:25 | |
| *** Bhujay has joined #openstack-cinder | 16:25 | |
| *** dpawlik has joined #openstack-cinder | 16:26 | |
| *** Bhujay has quit IRC | 16:26 | |
| *** Bhujay has joined #openstack-cinder | 16:27 | |
| *** Bhujay has quit IRC | 16:28 | |
| *** Bhujay has joined #openstack-cinder | 16:28 | |
| *** Bhujay has quit IRC | 16:29 | |
| *** luizbag has quit IRC | 16:30 | |
| *** Bhujay has joined #openstack-cinder | 16:30 | |
| *** dpawlik has quit IRC | 16:30 | |
| *** Bhujay has quit IRC | 16:31 | |
| *** Bhujay has joined #openstack-cinder | 16:31 | |
| *** Bhujay has quit IRC | 16:32 | |
| *** Bhujay has joined #openstack-cinder | 16:33 | |
| *** Bhujay has quit IRC | 16:34 | |
| *** Bhujay has joined #openstack-cinder | 16:34 | |
| *** udesale has quit IRC | 16:35 | |
| *** Bhujay has quit IRC | 16:35 | |
| *** Bhujay has joined #openstack-cinder | 16:36 | |
| *** Bhujay has quit IRC | 16:37 | |
| *** Bhujay has joined #openstack-cinder | 16:37 | |
| *** Bhujay has quit IRC | 16:38 | |
| *** Bhujay has joined #openstack-cinder | 16:39 | |
| *** Bhujay has quit IRC | 16:40 | |
| *** Bhujay has joined #openstack-cinder | 16:40 | |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: [WIP] Add _context to online snapshots in remotefs driver https://review.openstack.org/570885 | 16:41 |
| *** jistr has quit IRC | 16:41 | |
| *** Bhujay has quit IRC | 16:41 | |
| *** jistr has joined #openstack-cinder | 16:42 | |
| *** Bhujay has joined #openstack-cinder | 16:42 | |
| *** dpawlik has joined #openstack-cinder | 16:42 | |
| *** Bhujay has quit IRC | 16:43 | |
| *** Bhujay has joined #openstack-cinder | 16:43 | |
| *** Bhujay has quit IRC | 16:44 | |
| *** Bhujay has joined #openstack-cinder | 16:45 | |
| *** luizbag has joined #openstack-cinder | 16:45 | |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: [WIP] Add _context to online snapshots in remotefs driver https://review.openstack.org/570885 | 16:46 |
| *** Bhujay has quit IRC | 16:46 | |
| *** jistr_ has joined #openstack-cinder | 16:46 | |
| *** dpawlik has quit IRC | 16:46 | |
| *** Bhujay has joined #openstack-cinder | 16:46 | |
| *** jistr has quit IRC | 16:47 | |
| *** Bhujay has quit IRC | 16:47 | |
| *** Bhujay has joined #openstack-cinder | 16:48 | |
| *** helenafm has quit IRC | 16:49 | |
| *** Bhujay has quit IRC | 16:49 | |
| *** Bhujay has joined #openstack-cinder | 16:49 | |
| *** Bhujay has quit IRC | 16:50 | |
| *** Bhujay has joined #openstack-cinder | 16:51 | |
| *** jistr_ has quit IRC | 16:51 | |
| *** jistr has joined #openstack-cinder | 16:52 | |
| *** Bhujay has quit IRC | 16:52 | |
| *** Bhujay has joined #openstack-cinder | 16:52 | |
| *** Bhujay has quit IRC | 16:53 | |
| *** Bhujay has joined #openstack-cinder | 16:54 | |
| *** Bhujay has quit IRC | 16:55 | |
| *** Bhujay has joined #openstack-cinder | 16:55 | |
| *** Bhujay has quit IRC | 16:56 | |
| *** Bhujay has joined #openstack-cinder | 16:57 | |
| *** dpawlik has joined #openstack-cinder | 16:58 | |
| *** Bhujay has quit IRC | 16:58 | |
| *** Bhujay has joined #openstack-cinder | 16:58 | |
| *** Bhujay has quit IRC | 16:59 | |
| *** Bhujay has joined #openstack-cinder | 17:00 | |
| lbragstad | o/ | 17:00 |
| rosmaita | hi | 17:00 |
| rosmaita | here's my concern | 17:01 |
| *** Bhujay has quit IRC | 17:01 | |
| rosmaita | the code is built around two kinds of users: admins and regular | 17:01 |
| *** Bhujay has joined #openstack-cinder | 17:01 | |
| rosmaita | the policies affect who can be put into each category, but not what they can do once there | 17:02 |
| rosmaita | what i mean is, stuff like cross-project access is restricted in code | 17:02 |
| rosmaita | you cannot control it in policy | 17:02 |
| lbragstad | do you have an example of what you mean by cross-project access? | 17:02 |
| rosmaita | yes, project A looking at project B's resources | 17:02 |
| *** Bhujay has quit IRC | 17:02 | |
| *** dpawlik has quit IRC | 17:03 | |
| rosmaita | an admin can do that | 17:03 |
| lbragstad | oh - sure | 17:03 |
| *** Bhujay has joined #openstack-cinder | 17:03 | |
| lbragstad | so - if i'm understanding correctly, your concern is that particular usecase isn't allowed through configuration? | 17:04 |
| *** Bhujay has quit IRC | 17:04 | |
| rosmaita | i don't think we want it allowed in configuration | 17:04 |
| lbragstad | oh - ok | 17:04 |
| lbragstad | yes, then i would agree | 17:04 |
| *** Bhujay has joined #openstack-cinder | 17:04 | |
| rosmaita | my concern is that the policy model and the user model aren't entirely consistent | 17:04 |
| *** e0ne has quit IRC | 17:05 | |
| *** tbarron has joined #openstack-cinder | 17:05 | |
| lbragstad | sorry for the million questions, i just want to make sure i'm on the same page - what do you mean by user model? | 17:05 |
| rosmaita | basically, admin vs. regular | 17:05 |
| *** Bhujay has quit IRC | 17:05 | |
| lbragstad | got it | 17:06 |
| *** Bhujay has joined #openstack-cinder | 17:06 | |
| *** gnufied has quit IRC | 17:06 | |
| rosmaita | here's a question for you | 17:06 |
| lbragstad | and what you mean by "admin" is anyone who happens to have the `admin` role on $something, where $something is a project | 17:06 |
| rosmaita | not so much the admin role, as much as whoever the project recognizes as acting in the admin context | 17:07 |
| *** Bhujay has quit IRC | 17:07 | |
| *** Bhujay has joined #openstack-cinder | 17:07 | |
| lbragstad | sure, ok.. good ahead with your question | 17:07 |
| tbarron | i seem to have lost my auto-join setting for #openstack-cinder so apologies if I'm re-hashing old ground but | 17:07 |
| tbarron | i'm not sure that the fix in https://review.openstack.org/#/c/626146/ works | 17:08 |
| tbarron | i made a null change review that depends on it | 17:08 |
| tbarron | https://review.openstack.org/#/c/626173/ | 17:08 |
| *** Bhujay has quit IRC | 17:08 | |
| tbarron | the centos jobs there still fail setting up devstack | 17:09 |
| *** Bhujay has joined #openstack-cinder | 17:09 | |
| tbarron | with e.g. http://logs.openstack.org/73/626173/1/check/manila-tempest-minimal-dsvm-cephfs-native-centos-7/fe76d59/logs/devstacklog.txt.gz#_2018-12-19_14_22_28_285 | 17:09 |
| *** Bhujay has quit IRC | 17:10 | |
| *** Bhujay has joined #openstack-cinder | 17:10 | |
| rosmaita | tbarron: i don't think that has come up this morning | 17:11 |
| tbarron | rosmaita: kk, thanks | 17:11 |
| tbarron | not an emergency btw, our centos jobs on master are all non-voting | 17:11 |
| *** Bhujay has quit IRC | 17:11 | |
| smcginnis | geguileo: Still around? Wondering if that is due to a different than expected exception? ^ | 17:11 |
| smcginnis | I would think DBAPIError should work - "DBAPIError exception wrapped from (pymysql.err.InternalError)" | 17:12 |
| tbarron | for other projects like kolla it may be more important though | 17:12 |
| *** Bhujay has joined #openstack-cinder | 17:12 | |
| smcginnis | Definitely need to sort that out. | 17:12 |
| lbragstad | rosmaita did you have another question? | 17:13 |
| smcginnis | Didn't want to use TEXT field like they initially proposed, but maybe that's what we need to do. | 17:13 |
| *** Bhujay has quit IRC | 17:13 | |
| rosmaita | lbragstad: sorry, my system just froze up for a few minutes | 17:13 |
| lbragstad | oh - no worries, i just didn't want to cut you off :) | 17:13 |
| *** Bhujay has joined #openstack-cinder | 17:13 | |
| rosmaita | lbragstad: what is the advantage of adding these tests rather than just checking the default policy string? | 17:14 |
| *** Bhujay has quit IRC | 17:14 | |
| *** Bhujay has joined #openstack-cinder | 17:15 | |
| lbragstad | by default check string you mean this? https://github.com/openstack/oslo.policy/blob/master/oslo_policy/opts.py#L42 | 17:15 |
| rosmaita | no, sorry, i mean the defaults we define in code | 17:16 |
| *** Bhujay has quit IRC | 17:16 | |
| rosmaita | what i mean is | 17:16 |
| rosmaita | there are already tests to make sure that polices work | 17:16 |
| lbragstad | well - good question | 17:16 |
| *** Bhujay has joined #openstack-cinder | 17:16 | |
| lbragstad | the problem is that the test policies are overridden | 17:17 |
| lbragstad | these are the policies for backups https://github.com/openstack/cinder/blob/master/cinder/policies/backups.py | 17:17 |
| *** Bhujay has quit IRC | 17:17 | |
| lbragstad | which is what someone would have out of the box when they install cinder, right? | 17:17 |
| rosmaita | right | 17:18 |
| *** Bhujay has joined #openstack-cinder | 17:18 | |
| lbragstad | but those are not what it tested https://github.com/openstack/cinder/blob/master/cinder/tests/unit/policy.json#L55-L58 | 17:18 |
| lbragstad | s/it/is/ | 17:18 |
| rosmaita | right, but the tests show that whatever policy you provide, the code respects | 17:18 |
| rosmaita | so why not just do a static check of the policy-in-code string | 17:19 |
| *** e0ne has joined #openstack-cinder | 17:19 | |
| *** Bhujay has quit IRC | 17:19 | |
| lbragstad | which tests are you referring to? | 17:19 |
| *** Bhujay has joined #openstack-cinder | 17:19 | |
| rosmaita | the cinder policy code tests + the oslo.policy tests | 17:19 |
| lbragstad | the ones currently in review? or the existing ones? | 17:20 |
| rosmaita | sorry, the existing ones | 17:20 |
| rosmaita | what i'm saying is | 17:20 |
| rosmaita | why not replace the ones under review with checks of the string in the code to make sure the policy has not changed | 17:20 |
| *** Bhujay has quit IRC | 17:20 | |
| rosmaita | like for instance, the policies whose default is currently "" | 17:21 |
| lbragstad | you mean put the defaults in https://github.com/openstack/cinder/blob/master/cinder/tests/unit/policy.json#L55-L58 | 17:21 |
| rosmaita | we can't really check that | 17:21 |
| *** Bhujay has joined #openstack-cinder | 17:21 | |
| *** Bhujay has quit IRC | 17:22 | |
| lbragstad | yeah - "" is essentially an unauthorized API | 17:22 |
| rosmaita | lbragstad: yes, basically the equivalent of generating the policy.yaml and doing a diff and making sure there is no change | 17:22 |
| geguileo | smcginnis: I'm on a meeting, I'll get back to you | 17:22 |
| *** Bhujay has joined #openstack-cinder | 17:22 | |
| lbragstad | you could do that, but since the policies are in code by default, you don't really need a policy file defined for testing | 17:22 |
| rosmaita | i think we are talking past each other here | 17:23 |
| *** Bhujay has quit IRC | 17:23 | |
| *** Bhujay has joined #openstack-cinder | 17:24 | |
| *** e0ne has quit IRC | 17:24 | |
| rosmaita | my question is: if the goal of the tests is to make sure that the default policies defined by cinder in code don't change, why not just look directly at the defined policy? why go through all the request-making stuff the proposed tests are doing? | 17:24 |
| rosmaita | (i told you i was having trouble articulating my concern!) | 17:25 |
| rosmaita | i think i am missing something | 17:25 |
| *** Bhujay has quit IRC | 17:25 | |
| lbragstad | i can hop on audio if it's easier to discuss | 17:25 |
| *** Bhujay has joined #openstack-cinder | 17:25 | |
| *** Bhujay has quit IRC | 17:26 | |
| *** Bhujay has joined #openstack-cinder | 17:27 | |
| rosmaita | sure | 17:27 |
| rosmaita | lbragstad: https://bluejeans.com/brosmait | 17:28 |
| *** Bhujay has quit IRC | 17:28 | |
| *** Bhujay has joined #openstack-cinder | 17:28 | |
| lbragstad | sweet - joined | 17:29 |
| *** Bhujay has quit IRC | 17:29 | |
| *** Bhujay has joined #openstack-cinder | 17:30 | |
| *** Bhujay has quit IRC | 17:31 | |
| *** Bhujay has joined #openstack-cinder | 17:31 | |
| *** Bhujay has quit IRC | 17:32 | |
| *** Bhujay has joined #openstack-cinder | 17:33 | |
| *** Bhujay has quit IRC | 17:34 | |
| *** Bhujay has joined #openstack-cinder | 17:34 | |
| *** Bhujay has quit IRC | 17:35 | |
| *** Bhujay has joined #openstack-cinder | 17:36 | |
| *** Bhujay has quit IRC | 17:37 | |
| *** Bhujay has joined #openstack-cinder | 17:37 | |
| *** Bhujay has quit IRC | 17:38 | |
| openstackgerrit | Merged openstack/python-cinderclient master: Re-enable shell UUID completion cache https://review.openstack.org/610619 | 17:39 |
| *** Bhujay has joined #openstack-cinder | 17:39 | |
| *** Bhujay has quit IRC | 17:40 | |
| *** Bhujay has joined #openstack-cinder | 17:40 | |
| *** Bhujay has quit IRC | 17:41 | |
| *** Bhujay has joined #openstack-cinder | 17:42 | |
| *** Bhujay has quit IRC | 17:43 | |
| *** Bhujay has joined #openstack-cinder | 17:43 | |
| *** Bhujay has quit IRC | 17:44 | |
| *** Bhujay has joined #openstack-cinder | 17:45 | |
| *** Bhujay has quit IRC | 17:46 | |
| *** gkadam has quit IRC | 17:46 | |
| *** Bhujay has joined #openstack-cinder | 17:46 | |
| *** Bhujay has quit IRC | 17:47 | |
| *** dpawlik has joined #openstack-cinder | 17:48 | |
| *** Bhujay has joined #openstack-cinder | 17:48 | |
| *** Bhujay has quit IRC | 17:49 | |
| *** Bhujay has joined #openstack-cinder | 17:49 | |
| *** Bhujay has quit IRC | 17:50 | |
| *** Bhujay has joined #openstack-cinder | 17:51 | |
| *** Bhujay has quit IRC | 17:52 | |
| *** Bhujay has joined #openstack-cinder | 17:52 | |
| *** dpawlik has quit IRC | 17:53 | |
| *** Bhujay has quit IRC | 17:53 | |
| *** Bhujay has joined #openstack-cinder | 17:54 | |
| *** Bhujay has quit IRC | 17:55 | |
| *** Bhujay has joined #openstack-cinder | 17:55 | |
| *** Bhujay has quit IRC | 17:56 | |
| *** Bhujay has joined #openstack-cinder | 17:57 | |
| *** Bhujay has quit IRC | 17:58 | |
| *** Bhujay has joined #openstack-cinder | 17:58 | |
| *** Bhujay has quit IRC | 17:59 | |
| *** Bhujay has joined #openstack-cinder | 18:00 | |
| *** dpawlik has joined #openstack-cinder | 18:04 | |
| openstackgerrit | Merged openstack/os-brick master: Windows SMBFS: fix using share subdirs https://review.openstack.org/620253 | 18:04 |
| *** Bhujay has quit IRC | 18:05 | |
| *** dpawlik has quit IRC | 18:09 | |
| geguileo | smcginnis: tbarron it is weird... | 18:16 |
| geguileo | I can only think that it is raising the InternalError instead of the DBAPIError | 18:17 |
| geguileo | and the wrap on DBAPIError is done at a higher level | 18:17 |
| smcginnis | Yeah, that's all I could think of. | 18:17 |
| smcginnis | Or if somehow the depends-on was not working in this case. | 18:17 |
| hemna | sup peeps | 18:18 |
| geguileo | smcginnis: Since the patch hasn't merged, I'll change it to just catch Exception, like we do in https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/migrate_repo/versions/085_cinder_init.py#L782 | 18:18 |
| smcginnis | geguileo: Works for me. I suppose we really don't care about specific exceptions. If anything is thrown then oh well. | 18:20 |
| smcginnis | Howdy hemna | 18:20 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Ignore migration 127 error on MariaDB https://review.openstack.org/626146 | 18:20 |
| geguileo | smcginnis: tbarron submitted new patch and going to hit recheck on the dependent patch | 18:21 |
| openstackgerrit | Merged openstack/cinder master: Allow to use _max qos option together with per_gb https://review.openstack.org/625058 | 18:22 |
| *** gnufied has joined #openstack-cinder | 18:24 | |
| lbragstad | rosmaita here is an example of what we're doing in keystone that add the same test coverage yikun is working on for cinder https://review.openstack.org/#/c/623334/2 | 18:24 |
| rosmaita | lbragstad: thanks! | 18:28 |
| tbarron | geguileo: thanks for rechecking and trying the broad exception, if it still fails then prob. Depends-on: isn't right or is broken | 18:31 |
| geguileo | tbarron: I checked that, and it said that it was merging the right patch | 18:31 |
| geguileo | in the devstack logs | 18:31 |
| tbarron | geguileo: kk | 18:32 |
| *** e0ne has joined #openstack-cinder | 18:33 | |
| *** e0ne has quit IRC | 18:40 | |
| openstackgerrit | Merged openstack/cinder master: Handle image download and conversion errors https://review.openstack.org/618976 | 18:42 |
| *** gnufied has quit IRC | 18:42 | |
| *** gnufied has joined #openstack-cinder | 18:43 | |
| openstackgerrit | Merged openstack/cinder master: Move tgt targets to privsep https://review.openstack.org/613137 | 18:49 |
| smcginnis | _alastor_: "Sure, you can scan my badge for free stuff." https://www.flickr.com/photos/143247548@N03/46307863422/in/photostream/ | 19:10 |
| smcginnis | :) | 19:10 |
| _alastor_ | smcginnis: Ha! I think that was the first booth I hit at the con | 19:12 |
| _alastor_ | smcginnis: My gmail filter for all the con spam has been working hard this morning | 19:13 |
| smcginnis | _alastor_: Some of those folks don't give up easily. I steer way clear of the DataDog booth at any of those. | 19:14 |
| *** dpawlik has joined #openstack-cinder | 19:25 | |
| *** luizbag has quit IRC | 19:27 | |
| *** dpawlik has quit IRC | 19:30 | |
| *** gouthamr_ is now known as gouthamr | 19:38 | |
| *** e0ne has joined #openstack-cinder | 19:40 | |
| *** dpawlik has joined #openstack-cinder | 19:41 | |
| *** dpawlik has quit IRC | 19:46 | |
| *** erlon_ has quit IRC | 19:55 | |
| *** david-lyle has quit IRC | 20:04 | |
| *** abishop has quit IRC | 20:10 | |
| *** markvoelker has joined #openstack-cinder | 20:14 | |
| *** dklyle has joined #openstack-cinder | 20:15 | |
| *** markvoelker has quit IRC | 20:19 | |
| *** e0ne has quit IRC | 20:23 | |
| geguileo | tbarron: smcginnis it looks like this one actually fixed it :-) | 20:24 |
| geguileo | https://review.openstack.org/#/c/626173/ | 20:24 |
| tbarron | geguileo: yup! | 20:24 |
| geguileo | tbarron: oh, you had already noticed XD | 20:25 |
| * geguileo checked first tbarron's patch | 20:25 | |
| tbarron | geguileo: i got an email and looked | 20:25 |
| geguileo | :-) | 20:25 |
| smcginnis | Oh good. Must have been wrapped higher up or something. | 20:25 |
| tbarron | geguileo: can we get that corner case downstream? | 20:25 |
| geguileo | tbarron: I don't know... my brain is liquid right now... | 20:26 |
| tbarron | geguileo: well if we do we'll figure it out in 9 months | 20:27 |
| geguileo | rofl | 20:27 |
| geguileo | tbarron: without thinking too much, I doubt that many people are doing 255 chars on anything... | 20:28 |
| tbarron | geguileo: right, but I dunno much about unicode length expansion -- seems not likely though | 20:28 |
| geguileo | tbarron: no, no, 255 unicode chars | 20:29 |
| tbarron | geguileo: ah | 20:29 |
| *** jmlowe has quit IRC | 20:31 | |
| geguileo | jungleboyj: NetApp and erlon are happy with this patch you already looked at (don't know if you reviewed it) https://review.openstack.org/#/c/621575 | 20:32 |
| *** alkhodos has quit IRC | 21:00 | |
| *** alkhodos has joined #openstack-cinder | 21:01 | |
| *** alkhodos has quit IRC | 21:06 | |
| *** ganso has quit IRC | 21:19 | |
| *** hedvig_01 has joined #openstack-cinder | 21:22 | |
| *** dpawlik has joined #openstack-cinder | 21:42 | |
| *** dpawlik has quit IRC | 21:46 | |
| *** dpawlik has joined #openstack-cinder | 21:58 | |
| *** dpawlik has quit IRC | 22:02 | |
| smcginnis | geguileo: Still around? Wondering if I should take care of the pep8 error in the migration patch. | 22:03 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Ignore migration 127 error on MariaDB https://review.openstack.org/626146 | 22:15 |
| *** markvoelker has joined #openstack-cinder | 22:15 | |
| openstackgerrit | Angela Smith proposed openstack/cinder master: REST connector for Brocade zone driver https://review.openstack.org/617100 | 22:17 |
| *** rcernin has joined #openstack-cinder | 22:19 | |
| *** rcernin has quit IRC | 22:36 | |
| *** rcernin has joined #openstack-cinder | 22:37 | |
| *** erlon_ has joined #openstack-cinder | 22:43 | |
| *** tpsilva has quit IRC | 22:53 | |
| *** rcernin has quit IRC | 22:57 | |
| *** rcernin has joined #openstack-cinder | 22:58 | |
| *** rcernin has quit IRC | 22:59 | |
| *** alkhodos has joined #openstack-cinder | 23:05 | |
| *** rcernin has joined #openstack-cinder | 23:07 | |
| *** jmlowe has joined #openstack-cinder | 23:16 | |
| *** mchlumsky has quit IRC | 23:18 | |
| *** LiangFang has quit IRC | 23:24 | |
| *** rcernin has quit IRC | 23:38 | |
| *** rcernin has joined #openstack-cinder | 23:41 | |
| *** erlon_ has quit IRC | 23:46 | |
| hedvig_01 | @smcginnis: thank you for the comments. I'm working on it at the moment. We have all 254 tests passing successfully now. We made some changes in our backend to get this working. Thank you for the support. I shall respond to the comments inline | 23:58 |
| *** dpawlik has joined #openstack-cinder | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!