| *** felipemonteiro has joined #openstack-cinder | 00:18 | |
| *** felipemonteiro_ has joined #openstack-cinder | 00:19 | |
| *** itlinux has quit IRC | 00:22 | |
| *** felipemonteiro has quit IRC | 00:23 | |
| *** wanghao has joined #openstack-cinder | 00:26 | |
| *** wanghao has quit IRC | 00:26 | |
| *** felipemonteiro_ has quit IRC | 00:26 | |
| *** wanghao has joined #openstack-cinder | 00:27 | |
| *** wanghao has quit IRC | 00:27 | |
| openstackgerrit | Merged openstack/cinder master: TrivialFix: Delete word ‘I’ in admin/blockstorage-backup-disks.rst https://review.openstack.org/525846 | 00:29 |
|---|---|---|
| openstackgerrit | Merged openstack/cinder master: added clarification in docs for usage of "volume_clear*" options https://review.openstack.org/523923 | 00:29 |
| openstackgerrit | Merged openstack/cinder master: Fix several instances of chap secret being output to logs. https://review.openstack.org/519734 | 00:29 |
| *** armax has joined #openstack-cinder | 00:33 | |
| *** ntpttr_laptop has joined #openstack-cinder | 00:33 | |
| *** prateek_ has quit IRC | 00:37 | |
| *** prateek_ has joined #openstack-cinder | 00:38 | |
| *** wanghao has joined #openstack-cinder | 00:39 | |
| *** gouthamr has joined #openstack-cinder | 00:42 | |
| *** harlowja has quit IRC | 00:45 | |
| openstackgerrit | Wanjing Xu proposed openstack/cinder master: [DNM] Testing Cisco ZM CI https://review.openstack.org/527824 | 00:51 |
| *** AlexeyAbashkin has joined #openstack-cinder | 00:53 | |
| *** AlexeyAbashkin has quit IRC | 00:57 | |
| openstackgerrit | Merged openstack/cinder master: XtremIO: optional clean IG with no mappings https://review.openstack.org/515392 | 00:59 |
| openstackgerrit | Merged openstack/cinder master: SMBFS: manageable volumes https://review.openstack.org/521158 | 00:59 |
| openstackgerrit | Merged openstack/cinder master: QNAP: Add support for QES 2.0.0 https://review.openstack.org/526994 | 00:59 |
| openstackgerrit | Merged openstack/cinder master: Fix discrepancy in api-ref for volume_types APIs https://review.openstack.org/526344 | 00:59 |
| openstackgerrit | Merged openstack/cinder master: Don't raise 'NotAuthorized' inside when soft authorization https://review.openstack.org/526327 | 00:59 |
| openstackgerrit | Merged openstack/cinder master: Restore_backup: set error when volume status not matched https://review.openstack.org/520026 | 00:59 |
| openstackgerrit | Wanjing Xu proposed openstack/cinder master: [DNM] Testing Cisco Third Party CI Cisco-CI-Experimental-Branch: adding_experimental_zm_cinder https://review.openstack.org/527826 | 01:03 |
| *** mriedem has quit IRC | 01:06 | |
| tommylikehu | hey smcginnis I guess pooja_jadhav 's idea is to remove the additional parameters in the request body, no in the response:) | 01:06 |
| tommylikehu | https://review.openstack.org/#/c/527637 | 01:07 |
| tommylikehu | also pooja_jadhav there are more attributes I think we should remove them, not only the status | 01:08 |
| openstackgerrit | Wanjing Xu proposed openstack/cinder master: [DNM] Testing Cisco Third Party CI https://review.openstack.org/527826 | 01:09 |
| *** mriedem has joined #openstack-cinder | 01:10 | |
| tommylikehu | smcginnis: basically we don't verify the additional parameters in the API controller before. But now we introduced the JSON schema, there could be more changes in the cinderclient. | 01:10 |
| tommylikehu | ping mriedem | 01:11 |
| mriedem | tommylikehu: hi | 01:11 |
| tommylikehu | hi mriedem could you take a look at this bug? https://bugs.launchpad.net/cinder/+bug/1737724 | 01:11 |
| openstack | Launchpad bug 1737724 in Cinder "FC attachment issue" [Undecided,New] | 01:11 |
| *** yangyapeng has joined #openstack-cinder | 01:12 | |
| mriedem | i'm kind of in the middle of fixing some other bugs associated with the new attach flow in nova | 01:13 |
| mriedem | is there something specific i need to look at? | 01:13 |
| tommylikehu | I am wondering how you correctly handle and save this connection info at nova side:) | 01:13 |
| *** psachin has joined #openstack-cinder | 01:13 | |
| *** s-shiono has joined #openstack-cinder | 01:14 | |
| mriedem | i don't see errors in the c-api, c-vol or n-cpu logs here http://54.209.116.144/93/526893/2/check/kaminario-dsvm-tempest-full-FC/82abd60/logs/ | 01:14 |
| tommylikehu | mriedem, cause cinder save the connection info in the form of simple key/value strings and it could not meet the requirement of arrays | 01:14 |
| mriedem | nova is not sending connection_info dicts to cinder | 01:15 |
| mriedem | those come *from* cinder | 01:15 |
| tommylikehu | mriedem: I thought you would save them | 01:15 |
| tommylikehu | here is the root cause: INSERT INTO attachment_specs (created_at, updated_at, deleted_at, deleted, `key`, value, attachment_id) | 01:15 |
| tommylikehu | VALUES ("2017-12-11 01:49:36", "2017-12-11 01:49:36", "", 0, "wwpns", ["21000024ff34c92d", "21000024ff34c92c"], "e3aab3b7-539c-45c9-855b-ec7aeb908f21")'] [parameters: {'attachment_id': 'e3aab3b7-539c-45c9-855b-ec7aeb908f21', 'deleted': 0, 'created_at': datetime.datetime(2017, 12, 12, 9, 2, 8, 449122), 'updated_at': None, 'value': [u'21000024ff34c92d', u'21000024ff34c92c'], 'key': u'wwpns', 'deleted_at': | 01:15 |
| tommylikehu | None}] | 01:15 |
| mriedem | that's coming from the host connector | 01:16 |
| mriedem | nova asks os-brick for the host connector | 01:16 |
| mriedem | and passes that to cinder | 01:16 |
| mriedem | in the PUT /attachments call | 01:16 |
| mriedem | cinder stores the connector dict values in the attachment_specs table | 01:16 |
| mriedem | as key/value pairs | 01:16 |
| mriedem | is there a new release of os-brick maybe that's causing issues? | 01:16 |
| tommylikehu | mriedem: cinder will try to save the connection info in the new attach/detach APIs. we don't in the older style | 01:17 |
| *** Apoorva has quit IRC | 01:18 | |
| tommylikehu | mriedem: I thought you would save them in the nova side in the older attach/detach APIs. so I am wrong:) | 01:18 |
| tommylikehu | s/in/at | 01:18 |
| mriedem | nova saves the connection_info dict, returned by cinder, in the nova.block_device_mappings table as a serialized json blob | 01:20 |
| mriedem | but it's assumed that the connection_info is a dict | 01:20 |
| mriedem | and the host connector from os-brick is also a dict | 01:20 |
| mriedem | what the values are within that dict, idk | 01:20 |
| mriedem | does cinder's attachment_specs table assume all the key/values are strings? | 01:21 |
| tommylikehu | mriedem: ok, now we save them in the key/value pairs such as metadata:) | 01:21 |
| mriedem | yup http://git.openstack.org/cgit/openstack/cinder/tree/cinder/db/sqlalchemy/migrate_repo/versions/091_add_attachment_specs.py#n25 | 01:21 |
| mriedem | it expects strings | 01:21 |
| mriedem | idk why | 01:21 |
| mriedem | the attachment_specs table probably doesn't even need to exist, it could have just been serialized json stored in the attachments table | 01:22 |
| mriedem | it would have been faster that way anyway | 01:22 |
| mriedem | so wwpns is a list and attachment_specs can't handle that | 01:22 |
| mriedem | for fibrechannel | 01:22 |
| mriedem | anyway, this is a cinder bug | 01:22 |
| tommylikehu | mriedem: i have no idea, that's the problem, guess we don't have any trouble when trying iscsi | 01:22 |
| mriedem | well i can see right here: "wwpns", ["21000024ff34c92d", "21000024ff34c92c"], | 01:23 |
| mriedem | that's not going to work on a String column | 01:23 |
| mriedem | unless you serialize it | 01:23 |
| *** liverpooler has joined #openstack-cinder | 01:23 | |
| mriedem | so, the fix on the cinder side is to probably always serialize the values before putting them into the attachment_specs table, and deserialize when you pull them out | 01:23 |
| tommylikehu | yeah, if we serialize the connector, there is no need to separate them and store in the key/value | 01:23 |
| mriedem | agree | 01:24 |
| mriedem | but you've got this attachment_specs table right now | 01:24 |
| tommylikehu | lol | 01:24 |
| mriedem | there is no versioned object for attachment specs either | 01:24 |
| mriedem | the connector is not returned back out of the API i don't think... | 01:25 |
| mriedem | from the attachment_specs table | 01:25 |
| mriedem | so idk what you'd want to do - either stop using attachment_specs altogether, and add a 'connector' column to the attachments table that is serialized json version of the connector dict you get from nova | 01:25 |
| mriedem | that's what i'd do | 01:25 |
| tommylikehu | mriedem: yeah, jgriffith will have the call | 01:26 |
| mriedem | you could do an online data migration to migrate the existing data out of the attachment_specs table and into the new attachments.connector column | 01:26 |
| mriedem | via the volume attachment object | 01:26 |
| mriedem | migrate the data on read | 01:26 |
| mriedem | use the new column on write | 01:26 |
| tommylikehu | mriedem: will it looks like a little silly if we change the attachment_spec's text column from string to text :) | 01:28 |
| *** tsuzuki has joined #openstack-cinder | 01:28 | |
| tommylikehu | 'value' column | 01:28 |
| mriedem | tommylikehu: comment added https://bugs.launchpad.net/cinder/+bug/1737724 | 01:30 |
| openstack | Launchpad bug 1737724 in Cinder "FC attachment issue" [High,Triaged] | 01:30 |
| mriedem | tommylikehu: string to text is not what i'm suggesting | 01:30 |
| mriedem | i would stop using the attachment_specs table altogether | 01:31 |
| mriedem | it's unnecessary | 01:31 |
| mriedem | just store it in a text column in the attachments table as serialized json | 01:31 |
| mriedem | nothing is ever going to index this data | 01:31 |
| mriedem | i.e. you're not going to page volume attachments and filter on the connector blob :) | 01:31 |
| mriedem | unless you're crazy | 01:32 |
| mriedem | definitely a stop ship bug though for cinder | 01:32 |
| tommylikehu | mriedem: yeah, it exists for several days, I noticed this bug, but everyone is waiting for jgriffith's feedback | 01:33 |
| mriedem | if i weren't dealing with nova bugs of my own i could help write the fix for this | 01:35 |
| *** namnh has joined #openstack-cinder | 01:37 | |
| tommylikehu | mriedem: I will try to fix this if there is nobody pick up the bug today | 01:37 |
| *** ntpttr_laptop has quit IRC | 01:40 | |
| *** dalgaaf has quit IRC | 01:45 | |
| *** dalgaaf has joined #openstack-cinder | 01:47 | |
| *** lkwan has quit IRC | 01:47 | |
| *** lkwan has joined #openstack-cinder | 01:47 | |
| *** lkwan has quit IRC | 01:53 | |
| *** lkwan has joined #openstack-cinder | 01:53 | |
| openstackgerrit | TommyLike proposed openstack/python-cinderclient master: Backup create is not available from 3.0 to 3.42 https://review.openstack.org/527568 | 01:56 |
| openstackgerrit | Merged openstack/cinder master: Cleanup XtremIO IG cleanup note https://review.openstack.org/527795 | 01:59 |
| *** kukacz has quit IRC | 02:00 | |
| *** kukacz has joined #openstack-cinder | 02:01 | |
| *** Apoorva has joined #openstack-cinder | 02:03 | |
| *** Apoorva has quit IRC | 02:09 | |
| *** itlinux has joined #openstack-cinder | 02:10 | |
| *** threestrands_ has joined #openstack-cinder | 02:22 | |
| *** threestrands_ has quit IRC | 02:22 | |
| *** threestrands_ has joined #openstack-cinder | 02:22 | |
| *** threestrands has quit IRC | 02:22 | |
| *** threestrands_ has quit IRC | 02:22 | |
| *** threestrands_ has joined #openstack-cinder | 02:23 | |
| *** threestrands_ has joined #openstack-cinder | 02:23 | |
| *** hoangcx has quit IRC | 02:29 | |
| *** moshele has joined #openstack-cinder | 02:29 | |
| *** salv-orl_ has joined #openstack-cinder | 02:29 | |
| *** hoangcx has joined #openstack-cinder | 02:29 | |
| openstackgerrit | Felipe Monteiro proposed openstack/cinder master: Fix volume image metadata endpoints raising None https://review.openstack.org/527838 | 02:31 |
| *** threestrands_ has quit IRC | 02:32 | |
| *** liverpooler has quit IRC | 02:32 | |
| *** psachin has quit IRC | 02:32 | |
| *** abhishek has quit IRC | 02:32 | |
| *** openstackstatus has quit IRC | 02:32 | |
| *** salv-orlando has quit IRC | 02:32 | |
| *** bkopilov has quit IRC | 02:32 | |
| *** zhonghua has quit IRC | 02:32 | |
| *** jose-phillips has quit IRC | 02:32 | |
| *** zhaochao has quit IRC | 02:32 | |
| *** stakeda has quit IRC | 02:32 | |
| *** pooja_jadhav has quit IRC | 02:32 | |
| *** y_storshoo has quit IRC | 02:32 | |
| *** amrith has quit IRC | 02:32 | |
| *** ChanServ has quit IRC | 02:32 | |
| *** rmk has quit IRC | 02:32 | |
| *** ChanServ has joined #openstack-cinder | 02:34 | |
| *** barjavel.freenode.net sets mode: +o ChanServ | 02:34 | |
| *** lkwan is now known as 18WAAA398 | 02:35 | |
| *** lkwan has joined #openstack-cinder | 02:36 | |
| *** threestrands_ has joined #openstack-cinder | 02:36 | |
| *** liverpooler has joined #openstack-cinder | 02:36 | |
| *** psachin has joined #openstack-cinder | 02:36 | |
| *** abhishek has joined #openstack-cinder | 02:36 | |
| *** openstackstatus has joined #openstack-cinder | 02:36 | |
| *** bkopilov has joined #openstack-cinder | 02:36 | |
| *** zhonghua has joined #openstack-cinder | 02:36 | |
| *** jose-phillips has joined #openstack-cinder | 02:36 | |
| *** zhaochao has joined #openstack-cinder | 02:36 | |
| *** stakeda has joined #openstack-cinder | 02:36 | |
| *** pooja_jadhav has joined #openstack-cinder | 02:36 | |
| *** y_storshoo has joined #openstack-cinder | 02:36 | |
| *** amrith has joined #openstack-cinder | 02:36 | |
| *** barjavel.freenode.net sets mode: +v openstackstatus | 02:36 | |
| *** rmk has joined #openstack-cinder | 02:36 | |
| *** zhurong has joined #openstack-cinder | 02:41 | |
| *** wanghao_ has joined #openstack-cinder | 02:46 | |
| *** wanghao has quit IRC | 02:46 | |
| *** liverpooler has quit IRC | 02:48 | |
| *** wanghao has joined #openstack-cinder | 02:51 | |
| *** wanghao_ has quit IRC | 02:55 | |
| *** gouthamr has quit IRC | 03:17 | |
| openstackgerrit | TommyLike proposed openstack/python-cinderclient master: Backup create is not available from 3.0 to 3.42 https://review.openstack.org/527568 | 03:18 |
| *** bkopilov has quit IRC | 03:20 | |
| *** armax has quit IRC | 03:21 | |
| *** markvoelker_ has quit IRC | 03:24 | |
| mriedem | tommylikehu: has cinder dropped using shadow tables? | 03:27 |
| mriedem | or did cinder ever have shadow tables? | 03:27 |
| *** Apoorva has joined #openstack-cinder | 03:31 | |
| tommylikehu | mriedem: you want to fix this bug by using this? I have no idea on this history | 03:32 |
| mriedem | no, i'm just adding the migration script | 03:32 |
| mriedem | nova has shadow tables for archive deleted rows | 03:32 |
| mriedem | because apparently we don't like to ever delete anything | 03:33 |
| tommylikehu | mriedem: ok | 03:33 |
| *** zhurong has quit IRC | 03:34 | |
| *** lhx_ has joined #openstack-cinder | 03:35 | |
| *** lbragstad has quit IRC | 03:44 | |
| *** moshele has quit IRC | 03:55 | |
| *** armax has joined #openstack-cinder | 03:55 | |
| *** moshele has joined #openstack-cinder | 03:57 | |
| *** zhonghua has quit IRC | 04:00 | |
| *** zhonghua has joined #openstack-cinder | 04:01 | |
| *** tsuzuki has quit IRC | 04:02 | |
| *** owalsh_ has joined #openstack-cinder | 04:04 | |
| *** owalsh has quit IRC | 04:07 | |
| *** armax has quit IRC | 04:12 | |
| *** armax has joined #openstack-cinder | 04:13 | |
| *** armax has quit IRC | 04:13 | |
| *** armax has joined #openstack-cinder | 04:14 | |
| *** armax has quit IRC | 04:14 | |
| *** armax has joined #openstack-cinder | 04:14 | |
| *** moshele has quit IRC | 04:14 | |
| *** armax has quit IRC | 04:15 | |
| *** lhx__ has joined #openstack-cinder | 04:27 | |
| *** lhx_ has quit IRC | 04:27 | |
| openstackgerrit | Brin Zhang proposed openstack/cinder master: add enough notification for QoS https://review.openstack.org/522482 | 04:29 |
| jgriffith | mriedem: tommylikehu feel free to submit a patch | 04:35 |
| mriedem | be here in 30 seconds | 04:36 |
| jgriffith | tommylikehu: it would've been even better if you responded to my requests and tested your devices inparticular FC sometime over the past year | 04:36 |
| mriedem | just got the test to pass | 04:36 |
| jgriffith | mriedem: awesome | 04:37 |
| mriedem | well, pep8 so you're gonna have to wait | 04:39 |
| mriedem | 20 seconds | 04:39 |
| *** hoangcx has quit IRC | 04:40 | |
| *** namnh has quit IRC | 04:40 | |
| *** hoangcx has joined #openstack-cinder | 04:40 | |
| *** namnh has joined #openstack-cinder | 04:40 | |
| jgriffith | mriedem: that's a small price to pay... and a fairly fast machine | 04:41 |
| jgriffith | must be running fast8? | 04:41 |
| mriedem | yup | 04:41 |
| mriedem | had to recreate the venv though | 04:41 |
| openstackgerrit | Matt Riedemann proposed openstack/cinder master: Store host connector in volume_attachment.connector column https://review.openstack.org/527852 | 04:41 |
| mriedem | i don't do cinder code much | 04:41 |
| mriedem | tommylikehu: ^ | 04:41 |
| jgriffith | mriedem: migrate the attachment-specs to your fancy new column? | 04:45 |
| *** itlinux has quit IRC | 04:45 | |
| mriedem | jgriffith: that's what _from_db_object does | 04:45 |
| mriedem | at runtime | 04:45 |
| mriedem | rather than during cinder-manage db sync | 04:46 |
| *** pgadiya has joined #openstack-cinder | 04:46 | |
| jgriffith | mriedem: well, don't you know all the fancy tricks!! | 04:46 |
| jgriffith | :) | 04:46 |
| mriedem | will add a cli hook later | 04:46 |
| mriedem | tommy sprung this on me just a while ago and now it's past my bed time | 04:46 |
| mriedem | maybe sprung is the wrong word | 04:47 |
| mriedem | "gave me the opportunity to try and help the cinder community" | 04:47 |
| jgriffith | mriedem: it's greatly appreciated by me and others I'm sure | 04:47 |
| jgriffith | including nikesh WRT his Kamanario driver | 04:47 |
| *** bkopilov has joined #openstack-cinder | 04:48 | |
| mriedem | least i can do for the new flow stuff in nova | 04:48 |
| mriedem | alright ttyl | 04:49 |
| *** mriedem has quit IRC | 04:49 | |
| jgriffith | mriedem: good night, and thanks again! | 04:49 |
| *** lhx__ has quit IRC | 04:51 | |
| pooja_jadhav | tommylikehu:Hi | 04:51 |
| *** abhi89 has joined #openstack-cinder | 04:58 | |
| *** abhishek has quit IRC | 05:00 | |
| *** lhx__ has joined #openstack-cinder | 05:18 | |
| *** adisky__ has joined #openstack-cinder | 05:18 | |
| *** abhishek has joined #openstack-cinder | 05:23 | |
| *** markvoelker has joined #openstack-cinder | 05:25 | |
| *** abhi89 has quit IRC | 05:25 | |
| *** links has joined #openstack-cinder | 05:35 | |
| *** chhavi has joined #openstack-cinder | 05:38 | |
| *** sapd has quit IRC | 05:38 | |
| *** sapd_ has joined #openstack-cinder | 05:38 | |
| *** lkuchlan has joined #openstack-cinder | 05:38 | |
| *** eandersson has quit IRC | 05:47 | |
| *** sticker has quit IRC | 05:49 | |
| openstackgerrit | Xiaojun Liao proposed openstack/cinder master: Fix v3 api-ref for showing API details url error https://review.openstack.org/522492 | 05:49 |
| *** markvoelker has quit IRC | 05:58 | |
| *** lkuchlan has quit IRC | 05:58 | |
| *** lkuchlan has joined #openstack-cinder | 05:59 | |
| openstackgerrit | Govardhan Chintha proposed openstack/cinder master: 3PAR: Update CHAP on host record when volume is migrated to new compute host https://review.openstack.org/527417 | 05:59 |
| tommylikehu | mriedem, jgriffith will try to fix this :) | 06:11 |
| tommylikehu | pooja_jadhav: hi | 06:11 |
| tommylikehu | mriedem , oh!!! https://review.openstack.org/#/c/527852/ | 06:12 |
| *** links has quit IRC | 06:13 | |
| *** links has joined #openstack-cinder | 06:21 | |
| *** links has quit IRC | 06:21 | |
| *** lkuchlan has quit IRC | 06:23 | |
| *** lpetrut has joined #openstack-cinder | 06:24 | |
| *** Apoorva has quit IRC | 06:31 | |
| *** hoangcx has quit IRC | 06:41 | |
| *** namnh has quit IRC | 06:41 | |
| *** namnh has joined #openstack-cinder | 06:41 | |
| *** hoangcx has joined #openstack-cinder | 06:41 | |
| *** openstackgerrit has quit IRC | 06:47 | |
| *** threestrands_ has quit IRC | 06:53 | |
| *** markvoelker has joined #openstack-cinder | 06:55 | |
| *** Srinivas_ has joined #openstack-cinder | 06:58 | |
| *** sapd__ has joined #openstack-cinder | 07:07 | |
| *** sapd_ has quit IRC | 07:07 | |
| Srinivas_ | ildikov: Hi | 07:07 |
| *** josecastroleon has joined #openstack-cinder | 07:07 | |
| *** e0ne has joined #openstack-cinder | 07:09 | |
| *** hoangcx has quit IRC | 07:14 | |
| *** namnh has quit IRC | 07:14 | |
| *** openstackgerrit has joined #openstack-cinder | 07:14 | |
| openstackgerrit | yixuan zhang proposed openstack/cinder master: Storwize: add hyperswap volume support https://review.openstack.org/492414 | 07:14 |
| *** namnh has joined #openstack-cinder | 07:14 | |
| *** hoangcx has joined #openstack-cinder | 07:15 | |
| *** lkuchlan has joined #openstack-cinder | 07:20 | |
| *** lpetrut has quit IRC | 07:28 | |
| *** markvoelker has quit IRC | 07:30 | |
| *** felipemonteiro_ has joined #openstack-cinder | 07:31 | |
| *** lkuchlan has quit IRC | 07:32 | |
| *** armaan has quit IRC | 07:36 | |
| *** armaan has joined #openstack-cinder | 07:36 | |
| *** bkopilov has quit IRC | 07:37 | |
| *** moshele has joined #openstack-cinder | 07:38 | |
| openstackgerrit | Govardhan Chintha proposed openstack/cinder master: 3PAR: Update CHAP on host record when volume is migrated to new compute host https://review.openstack.org/527417 | 07:40 |
| openstackgerrit | Xiaojun Liao proposed openstack/cinder master: Fix v3 api-ref for showing API details url error https://review.openstack.org/522492 | 07:42 |
| openstackgerrit | Pooja Jadhav proposed openstack/python-cinderclient master: Removed unnecessary parameters from group and group_snapshots create APIs https://review.openstack.org/527637 | 07:43 |
| openstackgerrit | Shunei Shiono proposed openstack/cinder master: NEC driver: implement manage/unmanage functions. https://review.openstack.org/526268 | 07:48 |
| *** lkuchlan has joined #openstack-cinder | 07:49 | |
| *** gkadam has joined #openstack-cinder | 07:50 | |
| *** bkopilov has joined #openstack-cinder | 07:53 | |
| *** gkadam has quit IRC | 07:55 | |
| openstackgerrit | Xiaojun Liao proposed openstack/cinder master: Fix v3 api-ref for showing API details url error https://review.openstack.org/522492 | 08:01 |
| *** felipemonteiro__ has joined #openstack-cinder | 08:03 | |
| *** daidv has joined #openstack-cinder | 08:04 | |
| *** felipemonteiro_ has quit IRC | 08:06 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 08:09 | |
| *** alexchadin has joined #openstack-cinder | 08:13 | |
| *** rcernin has quit IRC | 08:15 | |
| *** s-shiono has quit IRC | 08:17 | |
| daidv | Hi all, can anybody let me know about that volume multi attach status of Cinder? thanks | 08:21 |
| *** markvoelker has joined #openstack-cinder | 08:26 | |
| *** tesseract has joined #openstack-cinder | 08:28 | |
| tommylikehu | hey daidv FYI: https://review.openstack.org/#/c/523608/ | 08:29 |
| daidv | tommylikehu, thanks, that mean we have no way for multi attach for now, right? or is there any special backend have been supported multi attach? | 08:36 |
| *** hoonetorg has quit IRC | 08:37 | |
| *** Srinivas_ has quit IRC | 08:39 | |
| *** zzzeek has quit IRC | 08:43 | |
| *** zzzeek has joined #openstack-cinder | 08:45 | |
| *** imran_ansari has joined #openstack-cinder | 08:46 | |
| *** links has joined #openstack-cinder | 08:52 | |
| *** pckizer_ has joined #openstack-cinder | 08:52 | |
| *** e0ne has quit IRC | 08:53 | |
| *** pckizer has quit IRC | 08:54 | |
| *** obre has quit IRC | 08:54 | |
| *** hoonetorg has joined #openstack-cinder | 08:55 | |
| *** obre has joined #openstack-cinder | 08:56 | |
| *** links has quit IRC | 08:57 | |
| *** markvoelker has quit IRC | 09:00 | |
| *** ociuhandu has joined #openstack-cinder | 09:08 | |
| *** amoralej|off is now known as amoralej | 09:09 | |
| *** alexchadin has quit IRC | 09:09 | |
| *** alexchadin has joined #openstack-cinder | 09:09 | |
| *** links has joined #openstack-cinder | 09:10 | |
| *** hamdyk has joined #openstack-cinder | 09:12 | |
| *** kwathore__ has joined #openstack-cinder | 09:12 | |
| openstackgerrit | Jeremy Zhang proposed openstack/cinder master: [WIP] RBD: get manageable volumes https://review.openstack.org/527903 | 09:12 |
| *** chhavi has quit IRC | 09:15 | |
| *** felipemonteiro__ has quit IRC | 09:16 | |
| kwathore__ | @Team, Please take out some time for reviwing patch:https://review.openstack.org/#/c/524185/ | 09:16 |
| *** bkopilov has quit IRC | 09:17 | |
| *** ociuhandu has quit IRC | 09:18 | |
| *** pgadiya has quit IRC | 09:22 | |
| *** ociuhandu has joined #openstack-cinder | 09:22 | |
| *** ociuhandu has quit IRC | 09:22 | |
| *** rcernin has joined #openstack-cinder | 09:23 | |
| *** pgadiya has joined #openstack-cinder | 09:24 | |
| *** owalsh_ is now known as owalsh | 09:28 | |
| *** yangyapeng has quit IRC | 09:29 | |
| *** yangyapeng has joined #openstack-cinder | 09:30 | |
| *** wanghao has quit IRC | 09:31 | |
| *** wanghao has joined #openstack-cinder | 09:32 | |
| openstackgerrit | Pooja Jadhav proposed openstack/cinder master: V3 jsonschema validation: Group Snapshots https://review.openstack.org/524528 | 09:32 |
| *** ganso has joined #openstack-cinder | 09:47 | |
| *** yangyapeng has quit IRC | 09:49 | |
| *** yangyapeng has joined #openstack-cinder | 09:49 | |
| *** ganso has quit IRC | 09:52 | |
| *** ganso has joined #openstack-cinder | 09:54 | |
| *** markvoelker has joined #openstack-cinder | 09:57 | |
| *** lpetrut has joined #openstack-cinder | 09:59 | |
| *** sdague has joined #openstack-cinder | 09:59 | |
| *** yangyapeng has quit IRC | 10:00 | |
| *** e0ne has joined #openstack-cinder | 10:07 | |
| *** bkopilov has joined #openstack-cinder | 10:09 | |
| *** namnh has quit IRC | 10:09 | |
| *** josecastroleon has quit IRC | 10:18 | |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - revert a volume to last snapshot https://review.openstack.org/520659 | 10:27 |
| *** yangyapeng has joined #openstack-cinder | 10:28 | |
| *** markvoelker has quit IRC | 10:29 | |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Fix error handling and checks for generic volume groups https://review.openstack.org/521080 | 10:30 |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Errors extending replicated volumes https://review.openstack.org/524215 | 10:31 |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: Adds Overlay Volumes Created from Snapshots to Quobyte https://review.openstack.org/507050 | 10:32 |
| *** alexchadin has quit IRC | 10:32 | |
| *** alexchadin has joined #openstack-cinder | 10:33 | |
| *** sapd__ has quit IRC | 10:37 | |
| *** yangyapeng has quit IRC | 10:37 | |
| e0ne | tommylikehu: hi. please, take a look on https://review.openstack.org/#/c/524185/ | 10:38 |
| e0ne | tommylikehu: I'm ready to -2 on it because of time.sleep in unit tests | 10:38 |
| *** yangyapeng has joined #openstack-cinder | 10:40 | |
| *** links has quit IRC | 10:43 | |
| *** sapd__ has joined #openstack-cinder | 10:51 | |
| *** alexchadin has quit IRC | 10:55 | |
| *** alexchadin has joined #openstack-cinder | 10:55 | |
| *** links has joined #openstack-cinder | 10:57 | |
| *** zzzeek has quit IRC | 11:05 | |
| *** _pewp_ has quit IRC | 11:08 | |
| *** hamdyk has quit IRC | 11:09 | |
| *** _pewp_ has joined #openstack-cinder | 11:09 | |
| *** AlexeyAbashkin has quit IRC | 11:10 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 11:11 | |
| tommylikehu | e0ne: thanks for your reminder! | 11:11 |
| *** armaan has quit IRC | 11:11 | |
| e0ne | tommylikehu: np. just wanted to be on the same page with you | 11:11 |
| tommylikehu | e0ne: could you take a look at this one:) https://review.openstack.org/#/c/527568/ | 11:12 |
| tommylikehu | e0ne: :) | 11:12 |
| *** armaan has joined #openstack-cinder | 11:12 | |
| *** zzzeek has joined #openstack-cinder | 11:12 | |
| e0ne | tommylikehu: will do after the meeting | 11:12 |
| *** armaan has quit IRC | 11:17 | |
| *** armaan has joined #openstack-cinder | 11:18 | |
| *** alexchadin has quit IRC | 11:18 | |
| *** wanghao_ has joined #openstack-cinder | 11:21 | |
| *** wanghao has quit IRC | 11:24 | |
| *** markvoelker has joined #openstack-cinder | 11:26 | |
| openstackgerrit | Neha Alhat proposed openstack/cinder master: V2/V3 jsonschema validation: snapshots https://review.openstack.org/527937 | 11:27 |
| *** chhavi has joined #openstack-cinder | 11:31 | |
| *** linkmark has joined #openstack-cinder | 11:40 | |
| *** alexchadin has joined #openstack-cinder | 11:49 | |
| *** markvoelker has quit IRC | 12:00 | |
| openstackgerrit | Dongdong Pei proposed openstack/cinder master: Fix starting cinder-api & cinder-scheduler failed. https://review.openstack.org/527944 | 12:01 |
| *** amoralej is now known as amoralej|lunch | 12:02 | |
| *** sapd__ has quit IRC | 12:11 | |
| *** sapd__ has joined #openstack-cinder | 12:12 | |
| *** chhavi__ has joined #openstack-cinder | 12:14 | |
| *** chhavi has quit IRC | 12:15 | |
| *** dave-mccowan has joined #openstack-cinder | 12:21 | |
| *** dave-mcc_ has joined #openstack-cinder | 12:24 | |
| *** chhavi__ has quit IRC | 12:24 | |
| *** dave-mccowan has quit IRC | 12:25 | |
| *** chhavi__ has joined #openstack-cinder | 12:26 | |
| *** alexchadin has quit IRC | 12:27 | |
| openstackgerrit | Yingzhe Zeng proposed openstack/cinder master: Huawei driver refactor https://review.openstack.org/514126 | 12:27 |
| *** alexchadin has joined #openstack-cinder | 12:28 | |
| *** chhavi__ has quit IRC | 12:37 | |
| openstackgerrit | Yingzhe Zeng proposed openstack/cinder master: Huawei driver refactor https://review.openstack.org/514126 | 12:41 |
| *** psachin has quit IRC | 12:43 | |
| *** catintheroof has joined #openstack-cinder | 12:49 | |
| *** lhx__ has quit IRC | 12:51 | |
| *** markvoelker has joined #openstack-cinder | 12:57 | |
| *** mvenesio has joined #openstack-cinder | 13:01 | |
| *** bkopilov has quit IRC | 13:07 | |
| *** kaisers has quit IRC | 13:09 | |
| *** armaan has quit IRC | 13:13 | |
| *** lhx_ has joined #openstack-cinder | 13:15 | |
| *** kaisers has joined #openstack-cinder | 13:15 | |
| *** wanghao_ has quit IRC | 13:16 | |
| *** wanghao has joined #openstack-cinder | 13:16 | |
| *** wanghao has quit IRC | 13:17 | |
| *** wanghao has joined #openstack-cinder | 13:17 | |
| *** wanghao has quit IRC | 13:17 | |
| *** wanghao has joined #openstack-cinder | 13:18 | |
| *** wanghao has quit IRC | 13:18 | |
| *** wanghao has joined #openstack-cinder | 13:18 | |
| *** wanghao has quit IRC | 13:19 | |
| *** wanghao has joined #openstack-cinder | 13:19 | |
| *** wanghao has quit IRC | 13:20 | |
| *** wanghao has joined #openstack-cinder | 13:20 | |
| *** wanghao has quit IRC | 13:20 | |
| *** wanghao has joined #openstack-cinder | 13:21 | |
| *** wanghao has quit IRC | 13:21 | |
| *** wanghao has joined #openstack-cinder | 13:22 | |
| *** wanghao has quit IRC | 13:22 | |
| *** wanghao has joined #openstack-cinder | 13:23 | |
| *** liverpooler has joined #openstack-cinder | 13:23 | |
| openstackgerrit | Aseel Awwad proposed openstack/cinder master: Check if already managed before manage snapshot https://review.openstack.org/527957 | 13:26 |
| *** wanghao has quit IRC | 13:27 | |
| *** markvoelker has quit IRC | 13:28 | |
| *** markvoelker has joined #openstack-cinder | 13:28 | |
| *** imran_ansari has quit IRC | 13:31 | |
| *** mchlumsky has joined #openstack-cinder | 13:33 | |
| *** liverpooler has quit IRC | 13:35 | |
| *** liverpooler has joined #openstack-cinder | 13:35 | |
| *** chhavi has joined #openstack-cinder | 13:36 | |
| *** edmondsw has joined #openstack-cinder | 13:37 | |
| *** abhishekk has joined #openstack-cinder | 13:37 | |
| *** mchlumsky has quit IRC | 13:39 | |
| *** mchlumsky has joined #openstack-cinder | 13:40 | |
| *** amoralej|lunch is now known as amoralej | 13:46 | |
| *** mriedem has joined #openstack-cinder | 13:51 | |
| dulek | Hi guys! Anyone knows the status of Ironic-Cinder integration? | 13:56 |
| *** dustins has joined #openstack-cinder | 13:57 | |
| *** rcernin has quit IRC | 13:59 | |
| e0ne | dulek: hi. AFAIK, ironic team has some progress on it | 14:01 |
| e0ne | dulek: they don't need anything special from cinder | 14:01 |
| dulek | e0ne: https://blueprints.launchpad.net/nova/+spec/ironic-boot-from-volume - I see this as implemented, so it might work. :) | 14:02 |
| e0ne | :) | 14:02 |
| *** pgadiya has quit IRC | 14:04 | |
| *** dosaboy has quit IRC | 14:04 | |
| *** lkuchlan has quit IRC | 14:05 | |
| *** armaan has joined #openstack-cinder | 14:05 | |
| *** dosaboy has joined #openstack-cinder | 14:10 | |
| mriedem | tommylikehu: good point on https://review.openstack.org/#/c/527852/ - i thought about that when i couldn't sleep last night :) | 14:11 |
| mriedem | i'll audit where we're getting attachment specs and see if we need to use the volume attachment object now to get the connector | 14:11 |
| openstackgerrit | Govardhan Chintha proposed openstack/cinder master: 3PAR: Update CHAP on host record when volume is migrated to new compute host https://review.openstack.org/527417 | 14:16 |
| *** erlon has joined #openstack-cinder | 14:17 | |
| *** gouthamr has joined #openstack-cinder | 14:24 | |
| tommylikehu | hey mriedem , I audited all the methods related the attachment spec , that seems the only place we need to consider this time (I could be wrong), btw how can you fix this in such a short time :) | 14:26 |
| openstackgerrit | Merged openstack/cinder master: V3 jsonschema validation: Volume metadata https://review.openstack.org/526606 | 14:26 |
| mriedem | tommylikehu: lots of amphetamine | 14:37 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/cinder master: NetApp ONTAP: Copy offload bugfix https://review.openstack.org/527762 | 14:37 |
| * mriedem makes a little drug abuse joke | 14:38 | |
| *** yangyapeng has quit IRC | 14:38 | |
| tommylikehu | mriedem: lol | 14:39 |
| *** sapd__ has quit IRC | 14:41 | |
| *** alexchadin has quit IRC | 14:46 | |
| *** armaan has quit IRC | 14:49 | |
| *** lbragstad has joined #openstack-cinder | 14:50 | |
| openstackgerrit | Lee Yarwood proposed openstack/cinder master: WIP rbd: Allow for the creation of encrypted volumes https://review.openstack.org/527979 | 14:50 |
| *** xyang1 has joined #openstack-cinder | 14:50 | |
| *** armaan has joined #openstack-cinder | 14:50 | |
| *** abhishekk has quit IRC | 14:53 | |
| *** markstur has joined #openstack-cinder | 14:54 | |
| *** yangyapeng has joined #openstack-cinder | 14:56 | |
| *** jmlowe_ has joined #openstack-cinder | 14:59 | |
| *** jmlowe has quit IRC | 15:00 | |
| *** yangyapeng has quit IRC | 15:02 | |
| *** jmlowe has joined #openstack-cinder | 15:04 | |
| *** jmlowe_ has quit IRC | 15:06 | |
| *** felipemonteiro__ has joined #openstack-cinder | 15:13 | |
| *** yangyapeng has joined #openstack-cinder | 15:14 | |
| *** r-daneel has joined #openstack-cinder | 15:16 | |
| *** felipemonteiro__ has quit IRC | 15:17 | |
| *** yangyapeng has quit IRC | 15:18 | |
| *** itlinux has joined #openstack-cinder | 15:19 | |
| openstackgerrit | Jeremy Zhang proposed openstack/cinder master: Fix for getting manageable volume in cinder/volume/manager.py https://review.openstack.org/527988 | 15:20 |
| *** itlinux has quit IRC | 15:22 | |
| *** felipemonteiro__ has joined #openstack-cinder | 15:25 | |
| *** felipemonteiro_ has joined #openstack-cinder | 15:26 | |
| *** links has quit IRC | 15:26 | |
| *** Sanatics has joined #openstack-cinder | 15:29 | |
| *** felipemonteiro__ has quit IRC | 15:30 | |
| *** moshele has quit IRC | 15:30 | |
| *** yangyapeng has joined #openstack-cinder | 15:36 | |
| *** yangyapeng has quit IRC | 15:40 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: qemu-img info --force-share for NFS driver https://review.openstack.org/518782 | 15:42 |
| *** imran_ansari has joined #openstack-cinder | 15:44 | |
| *** armax has joined #openstack-cinder | 15:47 | |
| *** yangyapeng has joined #openstack-cinder | 15:56 | |
| *** Sanatics has quit IRC | 15:56 | |
| *** vivsoni__ has joined #openstack-cinder | 15:57 | |
| *** markstur has quit IRC | 15:59 | |
| *** catintheroof has quit IRC | 16:00 | |
| *** catintheroof has joined #openstack-cinder | 16:00 | |
| *** yangyapeng has quit IRC | 16:01 | |
| *** armaan has quit IRC | 16:04 | |
| *** armaan has joined #openstack-cinder | 16:05 | |
| *** markstur has joined #openstack-cinder | 16:05 | |
| *** catintheroof has quit IRC | 16:05 | |
| *** jdillaman has joined #openstack-cinder | 16:07 | |
| *** lhx__ has joined #openstack-cinder | 16:09 | |
| *** abhishek has quit IRC | 16:09 | |
| *** lhx_ has quit IRC | 16:12 | |
| *** yangyapeng has joined #openstack-cinder | 16:17 | |
| *** yangyapeng has quit IRC | 16:21 | |
| *** itlinux has joined #openstack-cinder | 16:21 | |
| *** itlinux has quit IRC | 16:28 | |
| *** armaan has quit IRC | 16:33 | |
| *** armaan has joined #openstack-cinder | 16:33 | |
| *** hoonetorg has quit IRC | 16:35 | |
| *** Apoorva has joined #openstack-cinder | 16:36 | |
| *** yangyapeng has joined #openstack-cinder | 16:37 | |
| *** e0ne has quit IRC | 16:38 | |
| *** moshele has joined #openstack-cinder | 16:39 | |
| *** yangyapeng has quit IRC | 16:41 | |
| *** hoonetorg has joined #openstack-cinder | 16:48 | |
| *** yangyapeng has joined #openstack-cinder | 16:58 | |
| *** ntpttr_laptop has joined #openstack-cinder | 16:59 | |
| *** xyang1 has quit IRC | 17:01 | |
| *** yangyapeng has quit IRC | 17:03 | |
| *** edmondsw has quit IRC | 17:06 | |
| *** edmondsw has joined #openstack-cinder | 17:06 | |
| *** imran_ansari has quit IRC | 17:07 | |
| *** chhavi has quit IRC | 17:07 | |
| *** edmondsw_ has joined #openstack-cinder | 17:10 | |
| *** edmondsw has quit IRC | 17:11 | |
| *** lpetrut has quit IRC | 17:14 | |
| *** edmondsw_ has quit IRC | 17:14 | |
| openstackgerrit | Merged openstack/cinder master: V3 jsonschema validation: Group type specs https://review.openstack.org/520285 | 17:17 |
| *** yangyapeng has joined #openstack-cinder | 17:18 | |
| *** imran_ansari has joined #openstack-cinder | 17:19 | |
| *** markstur_ has joined #openstack-cinder | 17:19 | |
| *** markstur has quit IRC | 17:22 | |
| *** edmondsw has joined #openstack-cinder | 17:22 | |
| *** AlexeyAbashkin has quit IRC | 17:23 | |
| openstackgerrit | Merged openstack/python-cinderclient master: Removed unnecessary parameters from group and group_snapshots create APIs https://review.openstack.org/527637 | 17:23 |
| *** yangyapeng has quit IRC | 17:24 | |
| *** edmondsw has quit IRC | 17:25 | |
| *** crose has joined #openstack-cinder | 17:27 | |
| *** lhx__ has quit IRC | 17:32 | |
| openstackgerrit | Imran Ansari proposed openstack/cinder master: 3PAR - Fix temp snapshot that remains post online copy https://review.openstack.org/524185 | 17:34 |
| *** Apoorva has quit IRC | 17:36 | |
| *** mdbooth has quit IRC | 17:37 | |
| *** edmondsw has joined #openstack-cinder | 17:40 | |
| *** mdbooth has joined #openstack-cinder | 17:40 | |
| *** felipemonteiro_ has quit IRC | 17:43 | |
| *** logan- has quit IRC | 17:44 | |
| *** crushil has joined #openstack-cinder | 17:44 | |
| *** logan- has joined #openstack-cinder | 17:45 | |
| *** adrianofr has joined #openstack-cinder | 17:48 | |
| openstackgerrit | Merged openstack/cinder master: Migrate fixed_key encryption keys to Barbican https://review.openstack.org/524720 | 17:49 |
| *** moshele has quit IRC | 17:50 | |
| *** jmlowe has quit IRC | 17:51 | |
| *** edmondsw has quit IRC | 17:53 | |
| *** tesseract has quit IRC | 17:53 | |
| *** edmondsw has joined #openstack-cinder | 17:54 | |
| *** xyang1 has joined #openstack-cinder | 17:55 | |
| *** catintheroof has joined #openstack-cinder | 17:57 | |
| *** xyang1 has quit IRC | 17:59 | |
| *** Apoorva has joined #openstack-cinder | 18:03 | |
| *** Apoorva has quit IRC | 18:03 | |
| *** Apoorva has joined #openstack-cinder | 18:03 | |
| *** edmondsw has quit IRC | 18:14 | |
| *** rmcall has joined #openstack-cinder | 18:16 | |
| *** edmondsw has joined #openstack-cinder | 18:21 | |
| openstackgerrit | Matt Riedemann proposed openstack/cinder master: WIP: Don't call driver.terminate_connection if there is no connector https://review.openstack.org/528028 | 18:23 |
| mriedem | vivsoni__: ^ needs tests | 18:23 |
| *** gouthamr has quit IRC | 18:24 | |
| *** edmondsw has quit IRC | 18:25 | |
| *** itlinux has joined #openstack-cinder | 18:26 | |
| *** edmondsw has joined #openstack-cinder | 18:27 | |
| *** harlowja has joined #openstack-cinder | 18:29 | |
| *** edmondsw has quit IRC | 18:32 | |
| *** stvnoyes has joined #openstack-cinder | 18:32 | |
| *** edmondsw has joined #openstack-cinder | 18:33 | |
| *** imacdonn_ has quit IRC | 18:35 | |
| *** imacdonn_ has joined #openstack-cinder | 18:35 | |
| *** crushil has quit IRC | 18:36 | |
| *** gouthamr has joined #openstack-cinder | 18:37 | |
| *** crushil has joined #openstack-cinder | 18:37 | |
| *** edmondsw has quit IRC | 18:37 | |
| *** edmondsw has joined #openstack-cinder | 18:39 | |
| *** edmondsw has quit IRC | 18:43 | |
| *** edmondsw has joined #openstack-cinder | 18:45 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 18:47 | |
| *** amoralej is now known as amoralej|off | 18:48 | |
| *** ntpttr_laptop has quit IRC | 18:51 | |
| *** AlexeyAbashkin has quit IRC | 18:52 | |
| *** ntpttr_laptop has joined #openstack-cinder | 18:52 | |
| *** armaan has quit IRC | 18:56 | |
| *** felipemonteiro_ has joined #openstack-cinder | 18:56 | |
| *** armaan has joined #openstack-cinder | 18:56 | |
| *** itlinux has quit IRC | 18:57 | |
| *** felipemonteiro__ has joined #openstack-cinder | 18:58 | |
| *** mvk has quit IRC | 19:00 | |
| *** mvk has joined #openstack-cinder | 19:01 | |
| *** felipemonteiro_ has quit IRC | 19:02 | |
| openstackgerrit | Matt Riedemann proposed openstack/cinder master: Don't call driver.terminate_connection if there is no connector https://review.openstack.org/528028 | 19:03 |
| mriedem | erlon: you're looking for ^ | 19:05 |
| mriedem | for that terminate_connection thing | 19:05 |
| mriedem | http://logs.openstack.netapp.com/logs/52/527852/1/upstream-check/cinder-cDOT-iSCSI/11e9870/logs/screen-c-vol.txt.gz?level=ERROR | 19:05 |
| mriedem | different bug | 19:05 |
| *** dustins has quit IRC | 19:06 | |
| *** catintheroof has quit IRC | 19:06 | |
| *** xyang has joined #openstack-cinder | 19:10 | |
| *** tomh__ has quit IRC | 19:11 | |
| *** adisky__ has quit IRC | 19:11 | |
| *** jmlowe has joined #openstack-cinder | 19:12 | |
| *** edmondsw has quit IRC | 19:12 | |
| *** edmondsw has joined #openstack-cinder | 19:13 | |
| *** vivsoni__ has quit IRC | 19:13 | |
| *** openstack has joined #openstack-cinder | 20:34 | |
| *** ChanServ sets mode: +o openstack | 20:34 | |
| *** yangyapeng has quit IRC | 20:34 | |
| *** crose has joined #openstack-cinder | 20:35 | |
| *** edmondsw has quit IRC | 20:36 | |
| *** lpetrut has joined #openstack-cinder | 20:38 | |
| *** e0ne has quit IRC | 20:43 | |
| *** e0ne has joined #openstack-cinder | 20:44 | |
| *** e0ne has quit IRC | 20:45 | |
| *** jmlowe has joined #openstack-cinder | 20:45 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 20:46 | |
| *** gouthamr has quit IRC | 20:47 | |
| *** liverpooler has quit IRC | 20:47 | |
| *** yangyapeng has joined #openstack-cinder | 20:50 | |
| *** AlexeyAbashkin has quit IRC | 20:51 | |
| *** rmcall has quit IRC | 20:54 | |
| *** yangyapeng has quit IRC | 20:55 | |
| openstackgerrit | Aseel Awwad proposed openstack/cinder master: Check if already managed before manage snapshot https://review.openstack.org/527957 | 20:55 |
| *** crushil has quit IRC | 21:05 | |
| *** mchlumsky has quit IRC | 21:07 | |
| *** SamYaple has quit IRC | 21:07 | |
| *** SamYaple has joined #openstack-cinder | 21:08 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Correct documented service upgrade order https://review.openstack.org/528084 | 21:08 |
| *** openstack has joined #openstack-cinder | 21:14 | |
| *** ChanServ sets mode: +o openstack | 21:14 | |
| *** yangyapeng has quit IRC | 21:15 | |
| *** SamYaple has quit IRC | 21:22 | |
| *** SamYaple has joined #openstack-cinder | 21:23 | |
| *** crose has quit IRC | 21:27 | |
| mnaser | jgriffith: smcginnis ill check the permissions and see if thats the reason | 21:29 |
| smcginnis | mnaser: If that is it, we should probably get some better documentation added for that backend driver. | 21:30 |
| *** yangyapeng has joined #openstack-cinder | 21:30 | |
| *** threestrands has joined #openstack-cinder | 21:31 | |
| *** threestrands has quit IRC | 21:31 | |
| *** threestrands has joined #openstack-cinder | 21:31 | |
| *** yangyapeng has quit IRC | 21:35 | |
| mnaser | smcginnis: yeah, bummer is that the ceph user for cinder has access to the pool | 21:38 |
| mnaser | so that means we're back to the drawing board | 21:38 |
| smcginnis | mnaser: Shoot. Maybe jbernard or eharney will show up with some tips. | 21:39 |
| mnaser | its doing a create almost every 2 seconds | 21:40 |
| mnaser | i just wonder if it's blocking soemhow somewhere | 21:40 |
| smcginnis | mnaser: Still not ideal, but you could at least make sure image_volume_cache_enabled is true. | 21:41 |
| mnaser | smcginnis: if my brain parser of the code is operating, i dont think it uses image volume cache if it can clone it | 21:41 |
| smcginnis | mnaser: It would still download the image unnecessarily, but at least it would just be once. | 21:41 |
| smcginnis | mnaser: Hmm, maybe. It's been awhile for me since I've looked at that part. | 21:42 |
| *** gouthamr has joined #openstack-cinder | 21:43 | |
| *** erlon has joined #openstack-cinder | 21:46 | |
| *** gouthamr_ has joined #openstack-cinder | 21:46 | |
| *** sticker has joined #openstack-cinder | 21:46 | |
| *** gouthamr has quit IRC | 21:48 | |
| *** armaan has quit IRC | 21:48 | |
| *** yangyapeng has joined #openstack-cinder | 21:52 | |
| *** yangyapeng has quit IRC | 21:57 | |
| *** rcernin has joined #openstack-cinder | 21:58 | |
| *** crushil has joined #openstack-cinder | 21:59 | |
| *** crushil_ has joined #openstack-cinder | 22:00 | |
| *** xyang has quit IRC | 22:02 | |
| *** crushil has quit IRC | 22:04 | |
| *** gouthamr_ is now known as gouthamr | 22:04 | |
| *** diablo_rojo has quit IRC | 22:04 | |
| jgriffith | the whole thing about Ceph is it shouldn't download anything, just does the internal copy stuff and done | 22:05 |
| *** mvenesio has quit IRC | 22:05 | |
| mnaser | woo | 22:07 |
| mnaser | think we found a cinder bug | 22:07 |
| *** diablo_rojo has joined #openstack-cinder | 22:07 | |
| mnaser | Deadlock detected when running 'quota_reserve': Retrying... | 22:07 |
| mriedem | https://github.com/openstack/nova/blob/stable/pike/nova/db/sqlalchemy/api.py#L3821 | 22:11 |
| mriedem | yeah you'll probably want a retry decorator there | 22:11 |
| mnaser | mriedem: its in cinder (and its retrying forever i guess) | 22:17 |
| *** Apoorva has quit IRC | 22:17 | |
| smcginnis | Hmm, we had a patch for something similar lately. Let me see if I can track that down. | 22:17 |
| mnaser | https://github.com/openstack/cinder/blob/stable/pike/cinder/db/sqlalchemy/api.py#L1144 | 22:17 |
| *** ChanServ has quit IRC | 22:17 | |
| mriedem | mnaser: that's because it's using a super super super old version of that decorator from when cinder was nova-volume | 22:17 |
| mriedem | :) | 22:17 |
| mriedem | there is an oslo.db decorator for this | 22:17 |
| mriedem | which limits the max number of retries | 22:17 |
| mnaser | i figured just as much | 22:17 |
| mnaser | i think they're just looping forever now | 22:18 |
| mriedem | @oslo_db_api.wrap_db_retry(max_retries=5, retry_on_deadlock=True) | 22:18 |
| mnaser | max_retries=5 is probably healthier | 22:18 |
| mriedem | yeah there is no limit here https://github.com/openstack/cinder/blob/stable/pike/cinder/db/sqlalchemy/api.py#L244 | 22:18 |
| mriedem | this seems like a task for a part time contributor | 22:18 |
| mriedem | is that me?! | 22:18 |
| mriedem | could it be possible for me to have 3 active cinder patches at one time | 22:19 |
| mnaser | heh | 22:19 |
| *** yangyapeng has joined #openstack-cinder | 22:19 | |
| mriedem | mnaser: open a bug | 22:19 |
| mriedem | please | 22:19 |
| mnaser | mriedem: sure one sec | 22:19 |
| smcginnis | mriedem: I can grab it if you want. But feel free to continue. :) | 22:20 |
| smcginnis | mriedem: This is the newer retry decorator? oslo_db_api.wrap_db_retry(max_retries=5, retry_on_deadlock=True) | 22:21 |
| mnaser | https://bugs.launchpad.net/cinder/+bug/1738292 | 22:21 |
| openstack | Launchpad bug 1738292 in Cinder "Deadlocked queries retry forever" [Undecided,New] | 22:21 |
| smcginnis | mriedem: By your silence I am assuming you are busy making that change, but if not, let me know. | 22:24 |
| *** yangyapeng has quit IRC | 22:24 | |
| *** ChanServ has joined #openstack-cinder | 22:24 | |
| *** barjavel.freenode.net sets mode: +o ChanServ | 22:24 | |
| *** gouthamr has quit IRC | 22:28 | |
| *** ChanServ has quit IRC | 22:28 | |
| *** moshele has quit IRC | 22:30 | |
| *** ChanServ has joined #openstack-cinder | 22:31 | |
| *** barjavel.freenode.net sets mode: +o ChanServ | 22:31 | |
| *** e0ne has joined #openstack-cinder | 22:33 | |
| *** yangyapeng has joined #openstack-cinder | 22:37 | |
| mriedem | smcginnis: go for it | 22:38 |
| mriedem | and that's that the new (old now) decorator | 22:38 |
| *** yangyapeng has quit IRC | 22:42 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Switch to oslo_db retry decorator https://review.openstack.org/528107 | 22:42 |
| smcginnis | mnaser: I will propose that for backport after it lands. ^ | 22:44 |
| mnaser | thank you so much | 22:44 |
| smcginnis | mnaser: You and mriedem made it pretty easy. ;) | 22:44 |
| *** xyang has joined #openstack-cinder | 22:47 | |
| *** felipemonteiro_ has joined #openstack-cinder | 22:50 | |
| *** xyang has quit IRC | 22:53 | |
| *** felipemonteiro__ has quit IRC | 22:53 | |
| *** dustins has quit IRC | 22:55 | |
| *** felipemonteiro_ has quit IRC | 22:56 | |
| *** yangyapeng has joined #openstack-cinder | 22:59 | |
| *** e0ne has quit IRC | 22:59 | |
| *** crushil_ has quit IRC | 23:01 | |
| *** yangyapeng has quit IRC | 23:04 | |
| *** threestrands_ has joined #openstack-cinder | 23:10 | |
| *** threestrands has quit IRC | 23:13 | |
| *** felipemonteiro_ has joined #openstack-cinder | 23:14 | |
| *** masuberu has quit IRC | 23:15 | |
| *** yangyapeng has joined #openstack-cinder | 23:19 | |
| *** lpetrut has quit IRC | 23:21 | |
| *** ntpttr_laptop has quit IRC | 23:22 | |
| *** yangyapeng has quit IRC | 23:24 | |
| *** moshele has joined #openstack-cinder | 23:27 | |
| *** stakeda has quit IRC | 23:28 | |
| *** harlowja has quit IRC | 23:32 | |
| *** felipemonteiro_ has quit IRC | 23:35 | |
| *** dave-mcc_ has quit IRC | 23:37 | |
| *** Apoorva has joined #openstack-cinder | 23:38 | |
| *** harlowja has joined #openstack-cinder | 23:39 | |
| *** yangyapeng has joined #openstack-cinder | 23:40 | |
| *** harlowja has quit IRC | 23:40 | |
| *** harlowja has joined #openstack-cinder | 23:42 | |
| *** yangyapeng has quit IRC | 23:44 | |
| *** harlowja has quit IRC | 23:50 | |
| *** lennyb has quit IRC | 23:51 | |
| *** ntpttr_laptop has joined #openstack-cinder | 23:54 | |
| *** erlon has quit IRC | 23:55 | |
| *** stakeda has joined #openstack-cinder | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!