*** netapp has quit IRC | 00:01 | |
*** netapp_ has joined #openstack-cinder | 00:02 | |
*** netapp_ is now known as netapp | 00:03 | |
*** gouthamr has quit IRC | 00:21 | |
*** dikonoor has joined #openstack-cinder | 00:25 | |
*** markvoelker has quit IRC | 00:31 | |
*** wanghao has joined #openstack-cinder | 00:37 | |
*** Shunli has joined #openstack-cinder | 00:38 | |
*** dikonoor has quit IRC | 00:46 | |
*** jdwidari has quit IRC | 00:48 | |
*** zhengyin has joined #openstack-cinder | 00:54 | |
*** edisonxiang has quit IRC | 01:19 | |
*** gcb has joined #openstack-cinder | 01:19 | |
*** xiangxinyong has joined #openstack-cinder | 01:28 | |
*** zhurong has joined #openstack-cinder | 01:34 | |
*** Apoorva has joined #openstack-cinder | 01:37 | |
*** zhaochao has joined #openstack-cinder | 01:39 | |
*** stakeda has joined #openstack-cinder | 01:40 | |
*** edmondsw has joined #openstack-cinder | 01:41 | |
openstackgerrit | wangxiyuan proposed openstack/cinder master: Add project_id admin filter to limits API https://review.openstack.org/443437 | 01:43 |
---|---|---|
*** zhurong has quit IRC | 01:44 | |
*** edmondsw has quit IRC | 01:45 | |
openstackgerrit | melissaml proposed openstack/cinder-specs master: Fix doc generation for Python3 https://review.openstack.org/470794 | 01:53 |
*** Apoorva has quit IRC | 01:59 | |
*** crushil has joined #openstack-cinder | 02:15 | |
*** markvoelker has joined #openstack-cinder | 02:28 | |
*** zzzeek has joined #openstack-cinder | 02:30 | |
*** zhurong has joined #openstack-cinder | 02:32 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/cinder master: Replace assertTrue(isinstance()) with assertIsInstance() https://review.openstack.org/470811 | 02:32 |
*** markvoelker has quit IRC | 03:02 | |
*** dhinesh has joined #openstack-cinder | 03:02 | |
*** zhengyin has quit IRC | 03:05 | |
*** zhengyin has joined #openstack-cinder | 03:06 | |
openstackgerrit | wangxiyuan proposed openstack/cinder master: Support sort backup with name https://review.openstack.org/417272 | 03:13 |
openstackgerrit | wangxiyuan proposed openstack/cinder master: Support to query volume filter by changes since/before https://review.openstack.org/352688 | 03:19 |
*** gouthamr has joined #openstack-cinder | 03:19 | |
openstackgerrit | wangxiyuan proposed openstack/cinder master: Support sort backup with name https://review.openstack.org/417272 | 03:25 |
*** gkadam has joined #openstack-cinder | 03:32 | |
*** lhx__ has joined #openstack-cinder | 03:35 | |
*** psachin has joined #openstack-cinder | 03:42 | |
*** gkadam has quit IRC | 03:47 | |
openstackgerrit | Peter Wang proposed openstack/cinder master: Add VNX replication group support https://review.openstack.org/459205 | 03:51 |
*** gkadam has joined #openstack-cinder | 03:51 | |
*** markvoelker has joined #openstack-cinder | 03:58 | |
*** Dinesh_Bhor has joined #openstack-cinder | 04:03 | |
*** gouthamr has quit IRC | 04:12 | |
*** dikonoor has joined #openstack-cinder | 04:13 | |
*** bhagyashris has joined #openstack-cinder | 04:25 | |
*** markvoelker has quit IRC | 04:32 | |
*** dhinesh has quit IRC | 04:42 | |
*** dikonoor has quit IRC | 04:47 | |
*** Poornima_K has joined #openstack-cinder | 04:55 | |
*** zhengyin has quit IRC | 05:05 | |
*** zhengyin has joined #openstack-cinder | 05:05 | |
*** faizy has joined #openstack-cinder | 05:07 | |
*** lkuchlan has joined #openstack-cinder | 05:07 | |
*** salv-orlando has joined #openstack-cinder | 05:08 | |
*** Fdaisuke has quit IRC | 05:11 | |
*** edmondsw has joined #openstack-cinder | 05:17 | |
*** dikonoor has joined #openstack-cinder | 05:19 | |
*** edmondsw has quit IRC | 05:21 | |
*** dikonoor has quit IRC | 05:38 | |
*** pgadiya has joined #openstack-cinder | 05:49 | |
*** pgadiya has quit IRC | 05:49 | |
*** kwathore has joined #openstack-cinder | 05:49 | |
*** asselin has joined #openstack-cinder | 05:57 | |
*** dikonoor has joined #openstack-cinder | 05:59 | |
*** salv-orlando has quit IRC | 06:05 | |
*** lkuchlan has quit IRC | 06:06 | |
*** chhavi has joined #openstack-cinder | 06:08 | |
chhavi | https://github.com/openstack/cinder/blob/master/cinder/tests/unit/volume/drivers/ibm/test_storwize_svc.py#L2863 | 06:25 |
chhavi | anyone knows why this error injection is there in this, it seems to be wrong | 06:27 |
*** markvoelker has joined #openstack-cinder | 06:28 | |
*** rcernin has joined #openstack-cinder | 06:29 | |
openstackgerrit | Chhavi Agarwal proposed openstack/cinder master: Storwize Support iSCSI host when FC volume is there https://review.openstack.org/461850 | 06:34 |
openstackgerrit | wangxiyuan proposed openstack/cinder master: Support sort backup with name https://review.openstack.org/417272 | 06:35 |
openstackgerrit | TommyLike proposed openstack/cinder master: Cinder volume revert to snapshot https://review.openstack.org/391736 | 06:37 |
*** chhavi has quit IRC | 06:38 | |
*** lkuchlan has joined #openstack-cinder | 06:48 | |
*** dikonoo has joined #openstack-cinder | 06:49 | |
*** s-shiono_ has joined #openstack-cinder | 06:49 | |
*** dikonoor has quit IRC | 06:50 | |
*** bhagyashris has quit IRC | 06:50 | |
*** gkadam has quit IRC | 06:50 | |
*** zzzeek has quit IRC | 06:50 | |
*** crushil has quit IRC | 06:50 | |
*** stakeda has quit IRC | 06:50 | |
*** xiangxinyong has quit IRC | 06:50 | |
*** s-shiono has quit IRC | 06:50 | |
*** hemna has quit IRC | 06:50 | |
*** hoonetorg has quit IRC | 06:50 | |
*** amit213 has quit IRC | 06:50 | |
*** patrickeast has quit IRC | 06:50 | |
*** kong has quit IRC | 06:50 | |
*** TheJulia has quit IRC | 06:50 | |
*** ameade has quit IRC | 06:50 | |
*** guyr-infinidat has quit IRC | 06:50 | |
*** diablo_rojo_phon has quit IRC | 06:50 | |
*** pkoraca has quit IRC | 06:50 | |
*** bhagyashri_s has joined #openstack-cinder | 06:50 | |
*** stakeda has joined #openstack-cinder | 06:50 | |
*** gkadam has joined #openstack-cinder | 06:50 | |
*** crushil has joined #openstack-cinder | 06:50 | |
*** kong has joined #openstack-cinder | 06:51 | |
*** TheJulia has joined #openstack-cinder | 06:51 | |
*** hoonetorg has joined #openstack-cinder | 06:51 | |
*** patrickeast has joined #openstack-cinder | 06:51 | |
*** amit213 has joined #openstack-cinder | 06:51 | |
*** ameade has joined #openstack-cinder | 06:51 | |
*** diablo_rojo_phon has joined #openstack-cinder | 06:51 | |
*** hemna has joined #openstack-cinder | 06:53 | |
*** zzzeek has joined #openstack-cinder | 06:54 | |
*** markvoelker has quit IRC | 07:02 | |
*** Yangyapeng has joined #openstack-cinder | 07:03 | |
*** salv-orlando has joined #openstack-cinder | 07:04 | |
*** pcaruana has joined #openstack-cinder | 07:04 | |
*** edmondsw has joined #openstack-cinder | 07:05 | |
*** edmondsw has quit IRC | 07:09 | |
*** salv-orl_ has joined #openstack-cinder | 07:28 | |
*** danpawlik has joined #openstack-cinder | 07:30 | |
*** salv-orlando has quit IRC | 07:31 | |
*** zhurong has quit IRC | 07:31 | |
*** zhurong has joined #openstack-cinder | 07:40 | |
*** bkopilov has quit IRC | 07:53 | |
openstackgerrit | wangxiyuan proposed openstack/cinder master: [wip] Mapping parallel and ploop https://review.openstack.org/470869 | 07:55 |
*** markvoelker has joined #openstack-cinder | 07:59 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-cinder | 08:00 | |
*** bkopilov has joined #openstack-cinder | 08:02 | |
*** salv-orl_ has quit IRC | 08:07 | |
*** arnewiebalck_ has joined #openstack-cinder | 08:08 | |
*** salv-orlando has joined #openstack-cinder | 08:10 | |
*** zhurong has quit IRC | 08:20 | |
*** chhavi has joined #openstack-cinder | 08:22 | |
*** dikonoo has quit IRC | 08:29 | |
*** dikonoo has joined #openstack-cinder | 08:30 | |
*** markvoelker has quit IRC | 08:32 | |
*** chhavi_ has joined #openstack-cinder | 08:32 | |
*** chhavi has quit IRC | 08:33 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Base functionality, replacing SMI-S with REST https://review.openstack.org/455428 | 08:40 |
openstackgerrit | wangxiyuan proposed openstack/python-cinderclient master: Update visibility help message https://review.openstack.org/470881 | 08:47 |
*** Yangyapeng has quit IRC | 08:47 | |
*** pkoraca has joined #openstack-cinder | 08:48 | |
*** guyr-infinidat has joined #openstack-cinder | 08:48 | |
*** zhurong has joined #openstack-cinder | 08:50 | |
*** salv-orlando has quit IRC | 08:50 | |
*** salv-orlando has joined #openstack-cinder | 08:51 | |
*** nicolasbock has joined #openstack-cinder | 08:52 | |
*** edmondsw has joined #openstack-cinder | 08:53 | |
*** winston-d_ has joined #openstack-cinder | 08:55 | |
openstackgerrit | TommyLike proposed openstack/cinder master: Using wrong connector parameter in V3 attach https://review.openstack.org/467781 | 08:56 |
*** edmondsw has quit IRC | 08:57 | |
openstackgerrit | Dou Rui Yuan proposed openstack/cinder master: Correct cinder dev doc typo https://review.openstack.org/470885 | 09:04 |
*** wanghao_ has joined #openstack-cinder | 09:06 | |
*** wanghao has quit IRC | 09:06 | |
*** faizy_ has joined #openstack-cinder | 09:08 | |
*** faizy has quit IRC | 09:09 | |
openstackgerrit | TommyLike proposed openstack/cinder master: Cinder volume revert to snapshot https://review.openstack.org/391736 | 09:10 |
*** salv-orlando has quit IRC | 09:13 | |
*** salv-orlando has joined #openstack-cinder | 09:14 | |
*** Shashi has joined #openstack-cinder | 09:14 | |
*** wanghao_ has quit IRC | 09:21 | |
openstackgerrit | Kushal Wathore proposed openstack/cinder master: HPE Lefthand: Fixed cloning operation https://review.openstack.org/462624 | 09:23 |
*** arnewiebalck_ has quit IRC | 09:24 | |
openstackgerrit | Kushal Wathore proposed openstack/cinder master: 3PAR: Fixed image cache enabled capability. https://review.openstack.org/461744 | 09:24 |
*** markvoelker has joined #openstack-cinder | 09:29 | |
*** Shunli has quit IRC | 09:33 | |
*** dalgaaf has quit IRC | 09:35 | |
*** dalgaaf has joined #openstack-cinder | 09:37 | |
*** faizy_ has quit IRC | 09:57 | |
*** Poornima_K has quit IRC | 09:59 | |
*** zhurong has quit IRC | 09:59 | |
*** markvoelker has quit IRC | 10:02 | |
*** s-shiono_ has quit IRC | 10:06 | |
*** nicolasbock has quit IRC | 10:07 | |
*** Poornima_K has joined #openstack-cinder | 10:09 | |
*** pjm6 has quit IRC | 10:12 | |
*** zhurong has joined #openstack-cinder | 10:13 | |
*** pjm6 has joined #openstack-cinder | 10:19 | |
*** dikonoo has quit IRC | 10:24 | |
openstackgerrit | Gaurang Tapase proposed openstack/cinder master: Add CG capability to generic groups in GPFS driver https://review.openstack.org/469825 | 10:25 |
*** stakeda has quit IRC | 10:32 | |
*** odyssey4me has joined #openstack-cinder | 10:36 | |
odyssey4me | morning all - I'm busy doing an Ansible playbook to orchestrate a rolling upgrade according to https://docs.openstack.org/developer/cinder/upgrade.html | 10:37 |
odyssey4me | I'm at the stage now where I want to verify whether the environment is ready for the final restart as per https://docs.openstack.org/developer/cinder/upgrade.html#after-maintenance-window | 10:37 |
odyssey4me | when doing 'cinder-manage service list' I see two version outputs for each service - object version and rpc version | 10:39 |
odyssey4me | RPC version for a Pike deployment appears to be different on purpose for different agent types | 10:39 |
odyssey4me | what is the best way for me to determine whether an agent is using a downgraded RPC version and therefore should be restarted? | 10:40 |
*** edmondsw has joined #openstack-cinder | 10:41 | |
*** edmondsw has quit IRC | 10:45 | |
*** nicolasbock has joined #openstack-cinder | 10:59 | |
*** markvoelker has joined #openstack-cinder | 11:00 | |
*** sdague has joined #openstack-cinder | 11:05 | |
*** gkadam is now known as gkadam-afk | 11:11 | |
*** dikonoor has joined #openstack-cinder | 11:13 | |
*** pewp has quit IRC | 11:19 | |
*** pewp has joined #openstack-cinder | 11:20 | |
*** pkshiras__ has joined #openstack-cinder | 11:23 | |
*** Poornima_K has quit IRC | 11:24 | |
*** mtanino has joined #openstack-cinder | 11:28 | |
*** pkshiras__ has quit IRC | 11:33 | |
*** liverpooler has quit IRC | 11:33 | |
*** markvoelker has quit IRC | 11:33 | |
*** chhavi_ has quit IRC | 11:34 | |
*** salv-orlando has quit IRC | 11:35 | |
*** salv-orlando has joined #openstack-cinder | 11:35 | |
*** zhengyin has quit IRC | 11:39 | |
*** salv-orlando has quit IRC | 11:39 | |
*** dikonoor has quit IRC | 11:42 | |
*** ericyoung has joined #openstack-cinder | 11:43 | |
openstackgerrit | Isaac Beckman proposed openstack/cinder master: cleanup: remove DEVSTACK_CINDER_SECURE_DELETE https://review.openstack.org/465531 | 11:44 |
openstackgerrit | wangxiyuan proposed openstack/cinder master: Mapping parallel and ploop https://review.openstack.org/470869 | 11:46 |
openstackgerrit | Eric Young proposed openstack/cinder master: ScaleIO Driver: get manageable volumes https://review.openstack.org/469957 | 11:49 |
*** abishop_ is now known as abishop | 11:51 | |
*** dikonoor has joined #openstack-cinder | 12:01 | |
*** links has joined #openstack-cinder | 12:04 | |
*** zhurong has quit IRC | 12:04 | |
*** dillaman has quit IRC | 12:05 | |
*** lkuchlan has quit IRC | 12:06 | |
*** Shashi has quit IRC | 12:07 | |
*** mtanino has quit IRC | 12:11 | |
*** gkadam-afk is now known as gkadam | 12:13 | |
*** markvoelker has joined #openstack-cinder | 12:16 | |
*** dillaman has joined #openstack-cinder | 12:19 | |
*** winston-d_ has quit IRC | 12:23 | |
*** lkuchlan has joined #openstack-cinder | 12:26 | |
*** catintheroof has joined #openstack-cinder | 12:35 | |
*** MVenesio has joined #openstack-cinder | 12:38 | |
*** salv-orlando has joined #openstack-cinder | 12:40 | |
*** dave-mccowan has joined #openstack-cinder | 12:48 | |
*** dustins has joined #openstack-cinder | 12:51 | |
*** edmondsw has joined #openstack-cinder | 12:55 | |
*** cknight has joined #openstack-cinder | 12:56 | |
*** chhavi_ has joined #openstack-cinder | 12:57 | |
openstackgerrit | Tzur Eliyahu proposed openstack/cinder master: [WIP] Ibm storage: Added replication group support https://review.openstack.org/468738 | 12:59 |
openstackgerrit | Tzur Eliyahu proposed openstack/cinder master: Add IBM-Storage replication group support https://review.openstack.org/463314 | 12:59 |
*** mriedem has joined #openstack-cinder | 13:02 | |
smcginnis | dulek: You around? There was a rolling upgrade question earlier that I'm not sure about. | 13:02 |
openstackgerrit | Kushal Wathore proposed openstack/cinder master: HPE Lefthand: Fixed cloning operation https://review.openstack.org/462624 | 13:02 |
*** jwcroppe has quit IRC | 13:05 | |
dulek | smcginnis, odyssey4me: I don't think there's such possibility. What you see in `cinder-manage service list` is the version that is *supported*, not the version the service is pinned to. | 13:06 |
*** lhx__ has quit IRC | 13:06 | |
odyssey4me | hmm :/ | 13:06 |
dulek | odyssey4me: I think the RPC/object version that the service is pinned to appears in logs. And that's all. | 13:06 |
dulek | odyssey4me: What prevents you from sending SIGHUP to any service? | 13:07 |
odyssey4me | dulek if I'm running and re-running I'd prefer to be more deterministic, but also not forcibly SIGHUP every service every time | 13:07 |
dulek | odyssey4me: Also note that cinder-api isn't appearing in `cinder-manage service list`, so you would need to restart them anyway. | 13:07 |
odyssey4me | yep | 13:07 |
odyssey4me | this is why I was wondering whether there is some way of knowing | 13:08 |
odyssey4me | perhaps a db entry somewhere, or some sort of CLI tooling | 13:08 |
smcginnis | dulek: Have you tested SIGHUP lately? I've been trying to verify some of the new oslo.config changes. Supposed to be able to SIGHUP and have the service restart, but when I tried c-vol it just shut down. | 13:08 |
odyssey4me | it'd be super neat to be able to do a read-only db check and get return codes for outstanding db sync/online migrations | 13:08 |
openstackgerrit | Merged openstack/cinder-specs master: Fix doc generation for Python3 https://review.openstack.org/470794 | 13:09 |
odyssey4me | the keystone guys implemented that at our request and it helps make upgrades far more preditable | 13:09 |
smcginnis | odyssey4me: Seems like that would be useful, and not a lot of work. | 13:10 |
smcginnis | odyssey4me: I can look, but you don't happen to have the keystone patch that added it handy? | 13:10 |
dulek | odyssey4me: You could check the DB version I think. The table is called migrate_version. | 13:11 |
odyssey4me | smcginnis I'm looking for it. :) | 13:11 |
dulek | odyssey4me: And for data migrations there was a patch onceā¦ | 13:11 |
smcginnis | odyssey4me: I would guess it would mostly be a copy/paste. | 13:11 |
dulek | odyssey4me: https://review.openstack.org/#/c/438754/ | 13:12 |
odyssey4me | dulek the only trouble with that is that it's still not read-only | 13:12 |
odyssey4me | as an operator, I want to check whether there are migrations outstanding - then plan for them | 13:13 |
dulek | odyssey4me: I guess allowing to specify "--max_count 0" would work? ;) | 13:13 |
odyssey4me | or as an automated tooling developer, I want to be able to check for outstanding migrations, then determinstically execute follow on actions based on the informaiton provided | 13:13 |
dulek | smcginnis: I haven't tested it lately, but I remember it required sending SIGHUP to parent process. Otherwise it just killed the child process. | 13:14 |
odyssey4me | it might, although executing something meant for write access to do a read only action doesn't sound very safe ;) | 13:14 |
dulek | odyssey4me: Yeah, I guess we should add online_data_migrations_<something> command meant to do that. | 13:15 |
*** lucasxu has joined #openstack-cinder | 13:17 | |
tommylikehu | Hey smcginnis | 13:17 |
tommylikehu | around? | 13:17 |
smcginnis | tommylikehu: Yep | 13:17 |
tommylikehu | yes, I have a question about your comment about revert feature | 13:18 |
odyssey4me | smcginnis so this was the original ask from the keystone crew - I can put together something similar for cinder if you'd like? https://bugs.launchpad.net/keystone/+bug/1642212 | 13:18 |
openstack | Launchpad bug 1642212 in OpenStack Identity (keystone) "RFE: keystone-manage db_sync --check" [Wishlist,Fix released] - Assigned to Richard (csravelar) | 13:18 |
tommylikehu | I added those codes based on these two links | 13:18 |
tommylikehu | 1. https://linux.die.net/man/8/lvconvert the --merge option part | 13:18 |
odyssey4me | the initial implementation was https://review.openstack.org/#/c/416383/ | 13:19 |
smcginnis | odyssey4me: Doesn't look too painful. | 13:19 |
tommylikehu | 2. https://github.com/openstack/manila/blob/master/manila/share/drivers/lvm.py#L383 | 13:19 |
smcginnis | tommylikehu: OK great, I just wanted to confirm you had tested that. I know we have other things that need a deactivation, and I hadn't seen it there. | 13:19 |
tommylikehu | smcginnis: Does I misunderstand something? | 13:20 |
smcginnis | tommylikehu: I just wasn't sure because I think the man page for that option says something about it being done later if it wasn't in the right state. | 13:20 |
smcginnis | tommylikehu: No, you're fine. I was just asking. | 13:20 |
*** crushil has quit IRC | 13:20 | |
tommylikehu | smcginnis: ok | 13:20 |
*** pewp has quit IRC | 13:23 | |
*** salv-orlando has quit IRC | 13:23 | |
*** pewp has joined #openstack-cinder | 13:26 | |
*** mtanino has joined #openstack-cinder | 13:27 | |
openstackgerrit | Tzur Eliyahu proposed openstack/cinder master: [WIP] Ibm storage: Added replication group support https://review.openstack.org/468738 | 13:29 |
*** jwcroppe has joined #openstack-cinder | 13:32 | |
*** gouthamr has joined #openstack-cinder | 13:33 | |
*** eharney has joined #openstack-cinder | 13:34 | |
*** xyang has joined #openstack-cinder | 13:35 | |
openstackgerrit | Eric Young proposed openstack/cinder master: ScaleIO Driver: get manageable volumes https://review.openstack.org/469957 | 13:36 |
*** crushil has joined #openstack-cinder | 13:41 | |
*** zhurong has joined #openstack-cinder | 13:46 | |
*** felipemonteiro has joined #openstack-cinder | 13:50 | |
*** felipemonteiro_ has joined #openstack-cinder | 13:53 | |
*** felipemonteiro has quit IRC | 13:56 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Add support for OS-Brick force disconnect https://review.openstack.org/459454 | 13:57 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Multipath tempest scenarios https://review.openstack.org/469445 | 13:57 |
openstackgerrit | Merged openstack/cinder master: Remove service filter for service list https://review.openstack.org/467474 | 14:02 |
openstackgerrit | Merged openstack/cinder master: Add description about tox in dev_env.rst https://review.openstack.org/470143 | 14:03 |
openstackgerrit | Merged openstack/cinder master: Support sort backup with name https://review.openstack.org/417272 | 14:04 |
openstackgerrit | Merged openstack/cinder master: cleanup: remove DEVSTACK_CINDER_SECURE_DELETE https://review.openstack.org/465531 | 14:04 |
*** pewp has quit IRC | 14:07 | |
*** pewp has joined #openstack-cinder | 14:08 | |
stvnoyes | jgriffith: can you take a look at this bug when you have a chance? thx - https://bugs.launchpad.net/cinder/+bug/1695628 | 14:08 |
openstack | Launchpad bug 1695628 in Cinder "v3 attachment_delete does not delete record from db" [Undecided,New] | 14:08 |
openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Base functionality, replacing SMI-S with REST https://review.openstack.org/455428 | 14:13 |
*** pewp has quit IRC | 14:15 | |
*** jonesn has joined #openstack-cinder | 14:16 | |
*** pewp has joined #openstack-cinder | 14:16 | |
*** lkuchlan has quit IRC | 14:17 | |
openstackgerrit | Merged openstack/cinder master: Replace assertTrue(isinstance()) with assertIsInstance() https://review.openstack.org/470811 | 14:20 |
*** pewp has quit IRC | 14:21 | |
*** hongbin has joined #openstack-cinder | 14:21 | |
jgriffith | stvnoyes The record should still be there but updated status to delete. I'll take a look this morning | 14:24 |
*** pewp has joined #openstack-cinder | 14:24 | |
jgriffith | grrr... "updated delete status" | 14:26 |
jgriffith | Looks like tommylikehu already had the same response :). I'll just add my 2 cents as well | 14:26 |
*** gkadam has quit IRC | 14:30 | |
*** psachin has quit IRC | 14:31 | |
openstackgerrit | wzyuliyang proposed openstack/cinder master: Add pigz support for cinder-backup chunked driver compression https://review.openstack.org/450291 | 14:32 |
*** psachin has joined #openstack-cinder | 14:33 | |
eharney | i would really appreciate review on this patch for create from encrypted image: https://review.openstack.org/#/c/453343/ | 14:34 |
stvnoyes | jgriffith: ok thanks. I'll check that delete field when I deploy devstack again (right now working on unit tests) | 14:36 |
jgriffith | eharney looking at it now | 14:36 |
eharney | thanks | 14:37 |
jgriffith | eharney Looks good, I had two questions if you have a minute | 14:44 |
eharney | jgriffith: sure | 14:44 |
jgriffith | eharney in the past I'd run into problems trying to use image_meta size... IIRC it's not a mandatory thing. Is that no longer the case? | 14:45 |
eharney | jgriffith: our code already assumes that it's there for this image->volume path | 14:46 |
eharney | in _check_image_metadata | 14:46 |
*** psachin has quit IRC | 14:47 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Base functionality, replacing SMI-S with REST https://review.openstack.org/455428 | 14:47 |
jgriffith | eharney Ok, I guess I didn't see that since the unit test fakes didn't set it until your patch | 14:47 |
jgriffith | I might be mistaken on that so that's cool | 14:47 |
jgriffith | The only other thing I wanted to ask; do you think there's any value in adding any additional logging/info in manager.create_volume:_copy_image_to_volume aroudn the try/except block? | 14:48 |
jgriffith | I suppose the driver calls will log enough info that it shouldn't be necessary.... | 14:48 |
jgriffith | or not | 14:49 |
eharney | hmm, any ideas for things that would be useful to add there? | 14:50 |
jgriffith | Well, maybe outside of your patch | 14:50 |
jgriffith | but for example in driver.py the __copy_image_data_to_volume.... | 14:50 |
jgriffith | there are a number of things that can go wrong and no real logging there. Just wondered if some log.error would help when something fails to pin point why. | 14:51 |
jgriffith | but I'm still going down the rabbit holes to see what happens in each of those so it may already be there | 14:51 |
jgriffith | Regardless it shouldn't be a part of your fix anyway I guess | 14:51 |
eharney | i'll definitely keep an eye out for things to improve in that regard | 14:52 |
jgriffith | Other than that +2 from me | 14:53 |
*** liverpooler has joined #openstack-cinder | 15:02 | |
*** felipemonteiro has joined #openstack-cinder | 15:03 | |
*** crushil has quit IRC | 15:06 | |
*** felipemonteiro_ has quit IRC | 15:07 | |
*** esker has joined #openstack-cinder | 15:08 | |
*** felipemonteiro has quit IRC | 15:12 | |
*** dikonoor has quit IRC | 15:12 | |
chhavi_ | hemna: please review this https://review.openstack.org/#/c/461850/ | 15:13 |
*** liverpooler has quit IRC | 15:14 | |
*** amit345 has joined #openstack-cinder | 15:14 | |
*** liverpooler has joined #openstack-cinder | 15:14 | |
amit345 | BuildAbortException: Build of instance b5a88242-c5b1-466a-be23-76f096314724 aborted: Block Device Mapping is Invalid. | 15:15 |
amit345 | /var/log/nova/nova-compute.log:2017-06-05 18:00:26.340 2839 ERROR nova.compute.manager [instance: b5a88242-c5b1-466a-be23-76f096314724] | 15:15 |
amit345 | Hi, i've installed openstack environment on rhel 7.3 and i'm trying to launch an instance. I'm failing on | 15:15 |
amit345 | BuildAbortException: Build of instance b5a88242-c5b1-466a-be23-76f096314724 aborted: Block Device Mapping is Invalid. | 15:15 |
amit345 | /var/log/nova/nova-compute.log:2017-06-05 18:00:26.340 2839 ERROR nova.compute.manager [instance: b5a88242-c5b1-466a-be23-76f096314724] | 15:15 |
*** zhurong has quit IRC | 15:15 | |
hemna | amit345, you might want to ask in the nova channel | 15:15 |
amit345 | ok, thanks | 15:16 |
smcginnis | And repeat it a couple more times. :D | 15:16 |
*** aselius has joined #openstack-cinder | 15:16 | |
hemna | smcginnis, :) | 15:16 |
*** amit345 has left #openstack-cinder | 15:16 | |
*** amit345 has joined #openstack-cinder | 15:16 | |
*** amit345 has left #openstack-cinder | 15:16 | |
hemna | yah, make sure you flood the channel in nova. they enjoy that a lot | 15:17 |
jgriffith | hemna smcginnis shame on both of you | 15:17 |
hemna | :P | 15:17 |
*** amit345 has joined #openstack-cinder | 15:17 | |
smcginnis | jgriffith: I meant it lightly! | 15:17 |
jgriffith | :) | 15:17 |
*** amit345 has left #openstack-cinder | 15:17 | |
*** amit345 has joined #openstack-cinder | 15:17 | |
jgriffith | amit213 FYI, folks on dev channels typically don't like getting support questions in their channels | 15:18 |
openstackgerrit | xing-yang proposed openstack/cinder master: Tiramisu: replication group support https://review.openstack.org/352228 | 15:18 |
smcginnis | Can't take Monday too seriously. ;) | 15:18 |
jgriffith | amit213 #openstack is typically a good starting place | 15:18 |
jgriffith | amit213 and the first thing someone like myself would ask is a pastebin of the entire logfile from nova-compute | 15:18 |
*** amit345 has left #openstack-cinder | 15:18 | |
*** mvk has quit IRC | 15:19 | |
*** felipemonteiro has joined #openstack-cinder | 15:20 | |
openstackgerrit | Eric Harney proposed openstack/cinder master: API/DB: Fix hash randomization test issues https://review.openstack.org/468142 | 15:23 |
*** esker has quit IRC | 15:26 | |
*** esker has joined #openstack-cinder | 15:28 | |
*** MVenesio has quit IRC | 15:31 | |
*** lucasxu has quit IRC | 15:35 | |
openstackgerrit | wangxiyuan proposed openstack/cinder master: Add project_id admin filter to limits API https://review.openstack.org/443437 | 15:35 |
*** esker has quit IRC | 15:39 | |
*** esker has joined #openstack-cinder | 15:40 | |
*** esker has quit IRC | 15:42 | |
*** esker has joined #openstack-cinder | 15:43 | |
openstackgerrit | xing-yang proposed openstack/python-cinderclient master: Tiramisu: replication group support https://review.openstack.org/352229 | 15:45 |
*** crushil has joined #openstack-cinder | 15:45 | |
*** rcernin has quit IRC | 15:52 | |
*** Apoorva has joined #openstack-cinder | 15:53 | |
*** kwathore_ has joined #openstack-cinder | 15:58 | |
*** chhavi_ has quit IRC | 15:58 | |
*** kwathore has quit IRC | 15:59 | |
*** mtanino has quit IRC | 16:06 | |
*** mtanino has joined #openstack-cinder | 16:06 | |
*** salv-orlando has joined #openstack-cinder | 16:07 | |
hemna | comcast is craptastic today | 16:18 |
*** diablo_rojo has joined #openstack-cinder | 16:22 | |
*** pcaruana has quit IRC | 16:23 | |
*** mriedem1 has joined #openstack-cinder | 16:26 | |
*** mriedem has quit IRC | 16:26 | |
*** mriedem1 is now known as mriedem | 16:28 | |
*** lucasxu has joined #openstack-cinder | 16:28 | |
*** links has quit IRC | 16:35 | |
*** chhavi has joined #openstack-cinder | 16:35 | |
*** eharney has quit IRC | 16:39 | |
chhavi | hemna: need to discuss on review comments https://review.openstack.org/#/c/461850/ | 16:40 |
chhavi | I updated the response, we can discuss further on irc | 16:41 |
*** chenying has quit IRC | 16:41 | |
*** chenying has joined #openstack-cinder | 16:42 | |
*** harlowja has joined #openstack-cinder | 16:45 | |
*** xyang has quit IRC | 16:47 | |
*** esker has quit IRC | 16:49 | |
*** Shashi has joined #openstack-cinder | 16:50 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Base functionality, replacing SMI-S with REST https://review.openstack.org/455428 | 16:56 |
*** xyang has joined #openstack-cinder | 17:00 | |
*** liverpooler has quit IRC | 17:01 | |
*** liverpooler has joined #openstack-cinder | 17:06 | |
*** lucasxu has quit IRC | 17:13 | |
*** lucasxu has joined #openstack-cinder | 17:14 | |
chhavi | Hi all | 17:15 |
chhavi | I need the views of other core reviewers for https://review.openstack.org/#/c/461850/ | 17:15 |
chhavi | do we really need to change the approach and use protocol specific, if we are already taking care of connector which is been sent by brick. | 17:16 |
*** arunkant has quit IRC | 17:18 | |
*** gkadam has joined #openstack-cinder | 17:19 | |
chhavi | smcginnis: | 17:20 |
openstackgerrit | Gaurang Tapase proposed openstack/cinder master: Add CG capability to generic groups in GPFS driver https://review.openstack.org/469825 | 17:23 |
*** e0ne has joined #openstack-cinder | 17:27 | |
*** gkadam has quit IRC | 17:31 | |
*** MVenesio has joined #openstack-cinder | 17:35 | |
*** dhinesh has joined #openstack-cinder | 17:39 | |
dhinesh | i made changes to cinder/opts.py and tox was suggesting me to run "tox -e genopts" | 17:45 |
dhinesh | after i ran the command from cinder folder it throws an error "ContextualVersionConflict: (castellan 0.5.0 (/home/cinder/.tox/pep8/lib/python2.7/site-packages), Requirement.parse('castellan>=0.7.0'), set(['cinder']))" | 17:45 |
dhinesh | how to upgrade castellan to 0.7.0? | 17:46 |
dhinesh | tried "pip install castellan" | 17:47 |
dhinesh | still the same error | 17:47 |
smcginnis | dhinesh: Don't edit the opts.py file directly, that will be generated by the genopts target. | 17:47 |
smcginnis | dhinesh: You may have something messed up in your tox venv for that. | 17:47 |
smcginnis | dhinesh: You can "rm -fr .tox" to start fresh. | 17:47 |
ildikov | dhinesh: in general you can use "pip install --upgrade <whatever>" when you want the latest | 17:47 |
ildikov | dhinesh: but with tox as smcginnis says delete the folder it uses to get it start fresh | 17:48 |
dhinesh | smcginnis: got review comments to change the opts.py file, was the reason for editing it | 17:50 |
dhinesh | mcginnis: ildikov: ok i will try to start fresh | 17:50 |
smcginnis | dhinesh: Right, but you don't need to edit it yourself. You just run that command and it gets generated. | 17:50 |
openstackgerrit | Chhavi Agarwal proposed openstack/cinder master: Storwize Support iSCSI host when FC volume is there https://review.openstack.org/461850 | 17:51 |
*** lucasxu has quit IRC | 17:51 | |
*** lucasxu has joined #openstack-cinder | 17:52 | |
openstackgerrit | Chhavi Agarwal proposed openstack/cinder master: Storwize Support iSCSI host when FC volume is there https://review.openstack.org/461850 | 17:54 |
chhavi | hemna: I had updated the review as u suggested, u can check now | 17:56 |
*** chhavi has quit IRC | 18:01 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Add warning header to opts.py file https://review.openstack.org/471074 | 18:02 |
*** salv-orlando has quit IRC | 18:02 | |
*** salv-orlando has joined #openstack-cinder | 18:03 | |
*** dillaman has quit IRC | 18:05 | |
*** Shashi has quit IRC | 18:05 | |
*** salv-orlando has quit IRC | 18:08 | |
*** lucasxu has quit IRC | 18:08 | |
*** lucasxu has joined #openstack-cinder | 18:09 | |
openstackgerrit | Merged openstack/cinder master: API/DB: Fix hash randomization test issues https://review.openstack.org/468142 | 18:13 |
*** mvk has joined #openstack-cinder | 18:17 | |
*** bjornar__ has joined #openstack-cinder | 18:19 | |
*** dillaman has joined #openstack-cinder | 18:20 | |
dhinesh | mcginnis: ildikov: it works now thanks guys. | 18:34 |
openstackgerrit | Dhinesh Balasubramaniam proposed openstack/cinder master: Hedvig Cinder driver implementation https://review.openstack.org/433341 | 18:35 |
ildikov | dhinesh: cool! np :) | 18:36 |
*** eharney has joined #openstack-cinder | 18:37 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/python-brick-cinderclient-ext master: Add new v3 attach/detach workflow https://review.openstack.org/469594 | 18:47 |
*** lucasxu has quit IRC | 19:15 | |
*** zhaochao has quit IRC | 19:19 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/python-brick-cinderclient-ext master: Add new v3 attach/detach workflow https://review.openstack.org/469594 | 19:20 |
*** savvas has quit IRC | 19:21 | |
*** winston-d has quit IRC | 19:24 | |
*** winston-d has joined #openstack-cinder | 19:25 | |
*** lucasxu has joined #openstack-cinder | 19:34 | |
*** eharney has quit IRC | 19:39 | |
*** rcernin has joined #openstack-cinder | 19:49 | |
*** e0ne has quit IRC | 19:49 | |
*** eharney has joined #openstack-cinder | 19:54 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Base functionality, replacing SMI-S with REST https://review.openstack.org/455428 | 19:57 |
*** salv-orlando has joined #openstack-cinder | 20:03 | |
*** MVenesio has quit IRC | 20:05 | |
*** liverpooler has quit IRC | 20:07 | |
*** kfarr has joined #openstack-cinder | 20:10 | |
openstackgerrit | Merged openstack/python-brick-cinderclient-ext master: Add new v3 attach/detach workflow https://review.openstack.org/469594 | 20:17 |
* ildikov loves seeing the v3 attach progress everywhere :) ^^ | 20:18 | |
*** lucasxu has quit IRC | 20:22 | |
*** eharney has quit IRC | 20:23 | |
*** nicolasbock has quit IRC | 20:24 | |
*** edmondsw has quit IRC | 20:26 | |
*** kwathore has joined #openstack-cinder | 20:27 | |
*** kwathore_ has quit IRC | 20:28 | |
*** clenimar has joined #openstack-cinder | 20:39 | |
*** markstur has joined #openstack-cinder | 20:46 | |
*** lucasxu has joined #openstack-cinder | 20:47 | |
*** rcernin has quit IRC | 20:47 | |
*** dave-mccowan has quit IRC | 20:54 | |
*** kfarr has quit IRC | 20:54 | |
*** liuqing has quit IRC | 20:55 | |
*** liuqing has joined #openstack-cinder | 20:56 | |
*** jonesn has quit IRC | 20:56 | |
*** lucasxu has quit IRC | 21:07 | |
*** crushil has quit IRC | 21:14 | |
*** dustins has quit IRC | 21:22 | |
*** xyang has quit IRC | 21:28 | |
*** edmondsw has joined #openstack-cinder | 21:28 | |
*** crushil has joined #openstack-cinder | 21:31 | |
*** edmondsw has quit IRC | 21:32 | |
*** crushil has quit IRC | 21:40 | |
*** gouthamr has quit IRC | 21:43 | |
*** catintheroof has quit IRC | 21:53 | |
*** diablo_rojo has quit IRC | 21:55 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext master: Updated from global requirements https://review.openstack.org/468090 | 21:57 |
openstackgerrit | Tom Swanson proposed openstack/cinder master: Dell EMC SC: Updated Created by Message https://review.openstack.org/471144 | 22:04 |
*** felipemonteiro has quit IRC | 22:12 | |
*** jamielennox|away is now known as jamielennox | 22:13 | |
*** mtanino has quit IRC | 22:14 | |
*** cknight has quit IRC | 22:14 | |
*** kwathore has quit IRC | 22:16 | |
*** kwathore has joined #openstack-cinder | 22:16 | |
*** salv-orlando has quit IRC | 22:19 | |
*** gouthamr has joined #openstack-cinder | 22:22 | |
*** mriedem has quit IRC | 22:43 | |
*** gouthamr has quit IRC | 22:46 | |
*** gouthamr has joined #openstack-cinder | 22:49 | |
*** jwcroppe has quit IRC | 22:53 | |
*** bjornar__ has quit IRC | 22:57 | |
*** mriedem has joined #openstack-cinder | 23:11 | |
*** catintheroof has joined #openstack-cinder | 23:13 | |
*** jwcroppe has joined #openstack-cinder | 23:24 | |
*** sdague has quit IRC | 23:26 | |
*** jwcroppe has quit IRC | 23:28 | |
*** jamielennox is now known as jamielennox|away | 23:30 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Mark Virtuozzo driver as unsupported https://review.openstack.org/463030 | 23:41 |
openstackgerrit | Merged openstack/cinder master: Using wrong connector parameter in V3 attach https://review.openstack.org/467781 | 23:41 |
*** gouthamr has quit IRC | 23:43 | |
*** jamielennox|away is now known as jamielennox | 23:46 | |
*** gouthamr has joined #openstack-cinder | 23:52 | |
*** hongbin has quit IRC | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!