| *** bswartz has joined #openstack-cinder | 00:00 | |
| *** nellysmitt has joined #openstack-cinder | 00:03 | |
| *** Longgeek has joined #openstack-cinder | 00:05 | |
| *** tellesnobrega_ has joined #openstack-cinder | 00:05 | |
| *** Apoorva_ has quit IRC | 00:05 | |
| *** Apoorva has joined #openstack-cinder | 00:06 | |
| *** nellysmitt has quit IRC | 00:08 | |
| *** krtaylor has joined #openstack-cinder | 00:10 | |
| *** annegent_ has quit IRC | 00:14 | |
| nikesh_vedams | jgriffith: jenkins +1 | 00:17 |
|---|---|---|
| jgriffith | nikesh_vedams: should hope so it won't actually "check" anything :) | 00:17 |
| *** scottda_ has joined #openstack-cinder | 00:17 | |
| jgriffith | nikesh_vedams: what "your" jenkins says is more interesting :) | 00:17 |
| nikesh_vedams | before it was giving -1 | 00:18 |
| jgriffith | nikesh_vedams: ahh... wait, you didn't change it | 00:18 |
| nikesh_vedams | but there was no issue in code according to errors | 00:19 |
| jgriffith | nikesh_vedams: haha... sorry, I'm talking about rootwrap still :) | 00:19 |
| hemna | I posted another question/comment standard capabilities spec | 00:22 |
| *** markvoelker has quit IRC | 00:22 | |
| nikesh_vedams | i am bit confused where should i use "/usr/loca/sbin/scsadmin",in scst.py or volumefilters | 00:22 |
| nikesh_vedams | :) | 00:22 |
| jgriffith | nikesh_vedams: don't worry about tit | 00:22 |
| jgriffith | it | 00:22 |
| nikesh_vedams | because tgt and lio are not using in tgt.py and lio.py | 00:23 |
| nikesh_vedams | so what should i do | 00:23 |
| *** smcginnis_ has joined #openstack-cinder | 00:25 | |
| jgriffith | nikesh_vedams: just leave it | 00:25 |
| jgriffith | it's fine | 00:25 |
| *** mdenny has quit IRC | 00:25 | |
| jgriffith | I need to look closer at it but don't have time right now | 00:25 |
| smcginnis_ | jgriffith, Just logged on so it looks like you may be discussing it, but you gave Nikesh a second +2 on https://review.openstack.org/#/c/145778/. | 00:26 |
| jgriffith | nikesh_vedams: you'll notice I gave it a +2 | 00:26 |
| *** mtanino has quit IRC | 00:26 | |
| smcginnis_ | jgriffith, Can you A+ it? | 00:26 |
| *** scottda_ has quit IRC | 00:26 | |
| jgriffith | smcginnis_: I can :) | 00:26 |
| * jgriffith feels so dirty and used | 00:26 | |
| smcginnis_ | jgriffith, Good, it will be nice to get that through, then sort out the rootwrap stuff. | 00:26 |
| jgriffith | nikesh_vedams: congrats! | 00:26 |
| smcginnis_ | Sorry if you were just discussing this when I stepped in. | 00:26 |
| jgriffith | smcginnis_: haha... no worries | 00:27 |
| jgriffith | smcginnis_: glad to see people paying attention :) | 00:27 |
| jgriffith | smcginnis_: an on that note, I've got a couple horses to get some rides on | 00:27 |
| jgriffith | catch you all later this evening | 00:27 |
| smcginnis_ | jgriffith, To the horse barn!! | 00:27 |
| smcginnis_ | cya | 00:27 |
| jgriffith | That doesn't have a ring to it! | 00:28 |
| jgriffith | catch ya later | 00:28 |
| *** vilobhmm has quit IRC | 00:28 | |
| *** vilobhmm has joined #openstack-cinder | 00:28 | |
| nikesh_vedams | so will it merge automatically | 00:28 |
| nikesh_vedams | jgriffith: thanks | 00:31 |
| nikesh_vedams | :) | 00:31 |
| *** rmesta has quit IRC | 00:31 | |
| *** Longgeek has quit IRC | 00:34 | |
| *** IanGovett has quit IRC | 00:35 | |
| smcginnis_ | nikesh_vedams, If you're waiting in anticipation, you can watch the gate queue here: http://status.openstack.org/zuul/ :) | 00:35 |
| *** smcginnis_ has quit IRC | 00:36 | |
| *** david-lyle is now known as david-lyle_afk | 00:37 | |
| nikesh_vedams | smcginnis:thanks | 00:38 |
| nikesh_vedams | jgriffith:thanks but is this https://review.openstack.org/#/c/148038/ is going to merge | 00:40 |
| nikesh_vedams | jgriffith: because we have to do small changes according to this if it is going to merge | 00:41 |
| *** jungleboyj has joined #openstack-cinder | 00:42 | |
| nikesh_vedams | jgriffith: because this code have some pure virtual functions which every target has to override | 00:43 |
| openstackgerrit | Danny Wilson proposed openstack/cinder: Enabling volume replication on PureISCSIDriver https://review.openstack.org/145090 | 00:43 |
| jgriffith | nikesh_vedams: yes, I told you that Monday | 00:58 |
| *** fischerw has quit IRC | 00:59 | |
| nikesh_vedams | jgriffith:so without merging of that code we cannot use right | 01:03 |
| nikesh_vedams | thats why we didnot change | 01:03 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: WIP: Remove Brick from cinder codebase https://review.openstack.org/155552 | 01:04 |
| openstackgerrit | Yusuke Hayashi proposed openstack/cinder: Fix wrong dictionary keyword in Fujitsu ETERNUS driver https://review.openstack.org/155554 | 01:07 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Adding manage/unmanage support for LeftHand driver https://review.openstack.org/155127 | 01:15 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Adding manage/unmanage support for LeftHand driver https://review.openstack.org/155127 | 01:17 |
| nikesh_vedams | jgriffith: hmm | 01:24 |
| nikesh_vedams | jgriffith:so if https://review.openstack.org/#/c/148038/ patch is merged after our patch than in that case we have to file bug i guess. | 01:27 |
| jgriffith | nikesh_vedams: just fix your code and submit a patch | 01:28 |
| jgriffith | nikesh_vedams: it's not that hard | 01:28 |
| jgriffith | or a big deal | 01:28 |
| jgriffith | I suggested the other day you either: | 01:28 |
| jgriffith | 1. add the other patch as a dependency and fix yours up | 01:28 |
| jgriffith | 2. just submit yours knowing that your going to have to come back and fix it | 01:28 |
| nikesh_vedams | so can i submit it today night | 01:28 |
| jgriffith | nikesh_vedams: this seems difficult for you, so if you'd like I"ll just revert my +2/A on your patch and you can wait | 01:29 |
| jgriffith | nikesh_vedams: sure, but you sort of need 148038 to merge before you submit | 01:30 |
| jgriffith | nikesh_vedams: but yes, you certainly can ... or just wait for everything to land then fix your code up tomorrow | 01:30 |
| jgriffith | nikesh_vedams: one of those patches is likely to cause unit test failures and need reworked anyway... we'll see | 01:30 |
| jgriffith | nikesh_vedams: do you understand what I'm getting at here? | 01:32 |
| jgriffith | nikesh_vedams: there will be a race in the gate, one will break the other | 01:33 |
| jgriffith | nikesh_vedams: I was trying to be nice and just approve yours | 01:33 |
| *** davechen has joined #openstack-cinder | 01:33 | |
| nikesh_vedams | ok | 01:34 |
| *** tsekiyam_ has joined #openstack-cinder | 01:34 | |
| jgriffith | nikesh_vedams: you should already have the changes needed ready to go | 01:34 |
| *** tsekiyama has quit IRC | 01:37 | |
| nikesh_vedams | jgriffith:sorry john for this mess | 01:37 |
| *** tsekiyam_ has quit IRC | 01:38 | |
| jgriffith | nikesh_vedams: nahh... it's not a mess at all | 01:38 |
| jgriffith | nikesh_vedams: that's kinda my point, it's not a big deal | 01:38 |
| jgriffith | nikesh_vedams: either your patch fails gate, or mtanino's will (or both) | 01:39 |
| jgriffith | or they don't (which means the unit tests need work) :) | 01:39 |
| jgriffith | and we fix it and submit it | 01:39 |
| jgriffith | nikesh_vedams: the key is this way it's a quick turn around, because it's just fixing the issue and readding the +2/A | 01:39 |
| jgriffith | not a big deal | 01:39 |
| jgriffith | nikesh_vedams: you worry too much! :) | 01:39 |
| nikesh_vedams | thanks | 01:40 |
| anish | jgriffith: aww sweet, mtanino's patch finally landed | 01:43 |
| anish | err, -ish | 01:43 |
| jgriffith | nikesh_vedams: you got lucky | 01:43 |
| anish | makes scst unhappy I see | 01:43 |
| jgriffith | your patch won | 01:43 |
| jgriffith | anish: well... yeah about that :( | 01:43 |
| anish | it's gonna be stuck on gate eh | 01:43 |
| jgriffith | anish: we need to refactor it to fix the code Nikesh just landed | 01:43 |
| jgriffith | anish: yep, it was going to be mtanino's or nikesh_vedams 's | 01:44 |
| jgriffith | it's mtaninos | 01:44 |
| anish | hehe | 01:44 |
| anish | less pain for me, not mine | 01:44 |
| * anish gets busy rebasing | 01:44 | |
| jgriffith | anish: ahh... I was just getting ready to pull it | 01:44 |
| jgriffith | anish: you mean rebase "your" patch? | 01:45 |
| anish | yes | 01:45 |
| jgriffith | anish: you'll be ok | 01:45 |
| anish | yeah, never made it in | 01:45 |
| jgriffith | anish: it's the scst code that merged | 01:45 |
| anish | yeah, I figured | 01:45 |
| jgriffith | anish: so mtanino's patch won't have the updates needed for scst to work | 01:45 |
| jgriffith | I'll see if I can get around to just fixing that | 01:45 |
| anish | I can take a look at that if no one else is, I'll be rebasing mine anwyays | 01:45 |
| jgriffith | but then I'll get it merged and you should be fine | 01:45 |
| jgriffith | anish: sure | 01:46 |
| jgriffith | we can both look :) | 01:46 |
| anish | sounds good. I feel like you're going to be faster than me :) | 01:46 |
| jgriffith | I don't know about that | 01:46 |
| anish | to be seen | 01:46 |
| *** r-daneel has quit IRC | 01:48 | |
| jgriffith | anish: well since I have to reload a venv you've got a good 15 minute head start on me :) | 01:49 |
| *** _cjones_ has quit IRC | 01:50 | |
| *** leeantho has quit IRC | 01:54 | |
| *** emagana has quit IRC | 01:59 | |
| *** vilobhmm has quit IRC | 01:59 | |
| *** emagana has joined #openstack-cinder | 01:59 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/155579 | 02:00 |
| *** jamielennox is now known as jamielennox|away | 02:01 | |
| *** jcru has quit IRC | 02:03 | |
| *** emagana has quit IRC | 02:03 | |
| jgriffith | nikesh_vedams: about have this done.... | 02:04 |
| jgriffith | nikesh_vedams: quick question... might make it faster | 02:04 |
| jgriffith | nikesh_vedams: I need to create "_get_target_chap_auth" | 02:04 |
| jgriffith | nikesh_vedams: looks like you just use config file settings only for this? | 02:04 |
| *** nellysmitt has joined #openstack-cinder | 02:04 | |
| *** thingee has quit IRC | 02:08 | |
| *** nellysmitt has quit IRC | 02:09 | |
| nikesh_vedams | so we are checking that self.chap_username and self.chap_password in cinder.conf | 02:11 |
| jgriffith | nikesh_vedams: yeah, or random generate | 02:11 |
| jgriffith | nikesh_vedams: but I just noticed something.... | 02:11 |
| nikesh_vedams | ya | 02:12 |
| jgriffith | nikesh_vedams: well... first you only do that on create, so it's not persisted anywhere (if it's not in config) | 02:12 |
| jgriffith | nikesh_vedams: that's not going to work very well if you restart the service and try reexport | 02:12 |
| jgriffith | nikesh_vedams: the other thing..... | 02:12 |
| jgriffith | nikesh_vedams: create_iscsi_target takes an optional "chap_auth" | 02:13 |
| jgriffith | nikesh_vedams: but it's not used anywhere | 02:13 |
| jgriffith | nikesh_vedams: so I'm kinda confused how you're doing chap? | 02:13 |
| jgriffith | nikesh_vedams: well.. I mean... | 02:13 |
| jgriffith | nikesh_vedams: ok, so you're just handling the chap pieces on create-export | 02:13 |
| jgriffith | guess that's ok | 02:13 |
| jgriffith | but still have this pesky persist problem | 02:14 |
| patrickeast | oo this sounds familiar | 02:14 |
| jgriffith | patrickeast: LOL | 02:14 |
| *** Yogi1 has joined #openstack-cinder | 02:15 | |
| jgriffith | nikesh_vedams: ahh... ok | 02:15 |
| jgriffith | nikesh_vedams: so I can at least fix it as good as it was | 02:15 |
| jgriffith | nikesh_vedams: but you still have an issue here I think | 02:15 |
| *** Yogi11 has joined #openstack-cinder | 02:16 | |
| *** mriedem is now known as mriedem_zz | 02:16 | |
| *** med_ has quit IRC | 02:18 | |
| jgriffith | patrickeast: the good thing for them is it's used with LVM so they can just write out to the persist file like we do with TGT | 02:19 |
| *** sreedhar_vedams has joined #openstack-cinder | 02:19 | |
| patrickeast | ahh, yea thats handy | 02:19 |
| jgriffith | patrickeast: but that's beyond the scope of kindness this evening | 02:19 |
| *** Longgeek has joined #openstack-cinder | 02:19 | |
| jgriffith | patrickeast: and I'm not implementing it :) | 02:19 |
| patrickeast | lol | 02:19 |
| *** Yogi1 has quit IRC | 02:19 | |
| *** I has joined #openstack-cinder | 02:19 | |
| sreedhar_vedams | Hi John, I work along with Nikesh_vedams. Thanks for merging the scst code. I will tru and answer some of the questions. | 02:20 |
| *** I is now known as Guest65633 | 02:20 | |
| *** Guest65633 is now known as hemnafk | 02:20 | |
| openstackgerrit | wuyuting proposed openstack/cinder: Admin extend tenant's volume but change self quota https://review.openstack.org/155607 | 02:23 |
| *** Mandell has quit IRC | 02:23 | |
| sreedhar_vedams | Regarding the CHAP settings for SCST targets, we are using config file for targets to define their own chap strings if they don't support dynamic chap strings. If user does not define the chap strings in cinder.conf file, then we use the vutils to create the strings. | 02:23 |
| openstackgerrit | John Griffith proposed openstack/cinder: Refactoring for export functions in Target object https://review.openstack.org/148038 | 02:25 |
| jgriffith | nikesh_vedams: sreedhar_vedams ^^ | 02:26 |
| jgriffith | There are three methods stubbed in there that you'll need to address in a patch to your code *after* that lands | 02:26 |
| *** med_ has joined #openstack-cinder | 02:27 | |
| sreedhar_vedams | jgriffith. Definitely, we will address it today. | 02:27 |
| *** med_ has quit IRC | 02:28 | |
| *** med_ has joined #openstack-cinder | 02:28 | |
| jgriffith | sreedhar_vedams: cool, thanks! | 02:28 |
| jgriffith | sreedhar_vedams: that should make everybody happy for at least an evening :) | 02:28 |
| jgriffith | anish: ^^ I kinda cheated. :) | 02:28 |
| *** patrickeast has quit IRC | 02:29 | |
| jgriffith | Well this will teach you to be an admin: http://goo.gl/sE7I4X | 02:30 |
| jgriffith | :) | 02:30 |
| *** dannywilson has quit IRC | 02:30 | |
| *** Longgeek has quit IRC | 02:30 | |
| sreedhar_vedams | jgriffith, we should add a new bug to resolve the SCST persist problem, right? | 02:31 |
| jgriffith | sreedhar_vedams: I would if it were me :) | 02:36 |
| *** tellesnobrega_ has quit IRC | 02:36 | |
| sreedhar_vedams | jgriffith, we will register a bug, and work on it. Thanks. | 02:36 |
| *** hemnafk has quit IRC | 02:42 | |
| *** Yogi11 has quit IRC | 02:56 | |
| *** tellesnobrega_ has joined #openstack-cinder | 02:57 | |
| *** mberlin1 has joined #openstack-cinder | 02:59 | |
| *** kaisers has joined #openstack-cinder | 02:59 | |
| *** kaisers1 has quit IRC | 02:59 | |
| *** mberlin has quit IRC | 03:00 | |
| *** markvoelker has joined #openstack-cinder | 03:02 | |
| *** markvoelker has quit IRC | 03:07 | |
| *** Apoorva_ has joined #openstack-cinder | 03:10 | |
| *** Apoorva has quit IRC | 03:13 | |
| *** Apoorva_ has quit IRC | 03:15 | |
| *** harlowja_ is now known as harlowja_away | 03:27 | |
| *** rushil has joined #openstack-cinder | 03:29 | |
| *** rushil has quit IRC | 03:29 | |
| *** tellesnobrega_ has quit IRC | 03:34 | |
| *** tellesnobrega_ has joined #openstack-cinder | 03:34 | |
| *** tellesnobrega_ has quit IRC | 03:34 | |
| *** bill_az has quit IRC | 03:45 | |
| *** Apoorva has joined #openstack-cinder | 03:53 | |
| *** Longgeek has joined #openstack-cinder | 04:01 | |
| *** ebalduf has joined #openstack-cinder | 04:03 | |
| *** markvoelker has joined #openstack-cinder | 04:04 | |
| *** liusheng has joined #openstack-cinder | 04:05 | |
| *** nellysmitt has joined #openstack-cinder | 04:05 | |
| openstackgerrit | wuyuting proposed openstack/cinder: Admin extend tenant's volume but change self quota https://review.openstack.org/155607 | 04:07 |
| *** markvoelker has quit IRC | 04:08 | |
| *** Apoorva has quit IRC | 04:10 | |
| *** nellysmitt has quit IRC | 04:10 | |
| *** fischerw has joined #openstack-cinder | 04:11 | |
| *** fischerw has quit IRC | 04:23 | |
| *** liusheng has quit IRC | 04:26 | |
| *** liusheng has joined #openstack-cinder | 04:26 | |
| *** asselin_ has joined #openstack-cinder | 04:27 | |
| *** Mandell has joined #openstack-cinder | 04:31 | |
| *** asselin_ has quit IRC | 04:36 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:48 | |
| *** vnagpal has joined #openstack-cinder | 04:50 | |
| *** _cjones_ has joined #openstack-cinder | 04:51 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Snapshot object https://review.openstack.org/133566 | 04:51 |
| *** _cjones_ has quit IRC | 04:51 | |
| *** _cjones_ has joined #openstack-cinder | 04:52 | |
| *** vnagpal has joined #openstack-cinder | 04:54 | |
| *** _cjones_ has quit IRC | 04:56 | |
| *** harlowja_at_home has quit IRC | 05:05 | |
| *** deepakcs has joined #openstack-cinder | 05:10 | |
| *** deepakcs has quit IRC | 05:18 | |
| openstackgerrit | John Griffith proposed openstack/cinder-specs: Cinder Volume replication V2 https://review.openstack.org/155644 | 05:19 |
| *** deepakcs has joined #openstack-cinder | 05:20 | |
| *** vincent_hou has joined #openstack-cinder | 05:22 | |
| *** ankit_ag has joined #openstack-cinder | 05:22 | |
| *** ebalduf has quit IRC | 05:28 | |
| *** ebalduf has joined #openstack-cinder | 05:28 | |
| *** ebalduf has quit IRC | 05:32 | |
| openstackgerrit | John Griffith proposed openstack/cinder-specs: Cinder Volume replication V2 https://review.openstack.org/155644 | 05:33 |
| *** Raj_ has joined #openstack-cinder | 05:49 | |
| *** sreedhar_vedams has quit IRC | 05:55 | |
| *** abehl has joined #openstack-cinder | 06:02 | |
| *** nellysmitt has joined #openstack-cinder | 06:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/152401 | 06:09 |
| *** nellysmitt has quit IRC | 06:11 | |
| *** coolsvap_ is now known as coolsvap | 06:18 | |
| *** lpetrut has joined #openstack-cinder | 06:26 | |
| *** nshaikh has joined #openstack-cinder | 06:28 | |
| openstackgerrit | wuyuting proposed openstack/cinder: Admin extend tenant's volume but change self quota https://review.openstack.org/155607 | 06:30 |
| *** Lee1092 has joined #openstack-cinder | 06:30 | |
| *** nkrinner has joined #openstack-cinder | 06:37 | |
| *** sgotliv__ has joined #openstack-cinder | 06:40 | |
| *** jamielennox|away is now known as jamielennox | 06:44 | |
| *** afazekas_ has joined #openstack-cinder | 06:47 | |
| *** test2222 has joined #openstack-cinder | 06:55 | |
| *** test2222 is now known as alex_xu | 06:55 | |
| *** humble_ has joined #openstack-cinder | 07:04 | |
| *** vincent_hou_ has joined #openstack-cinder | 07:09 | |
| *** vincent_hou has quit IRC | 07:11 | |
| *** vincent_hou_ is now known as vincent_hou | 07:11 | |
| *** ankit_ag has quit IRC | 07:13 | |
| *** ankit_ag has joined #openstack-cinder | 07:13 | |
| *** Miouge has joined #openstack-cinder | 07:36 | |
| openstackgerrit | wuyuting proposed openstack/cinder: Tenant cann't set readonly metadata as True https://review.openstack.org/155668 | 07:37 |
| openstackgerrit | wuyuting proposed openstack/cinder: Tenant cann't set readonly metadata as True https://review.openstack.org/155668 | 07:39 |
| *** abehl has quit IRC | 07:39 | |
| *** alex_xu has quit IRC | 07:43 | |
| *** nellysmitt has joined #openstack-cinder | 07:49 | |
| *** TobiasE has joined #openstack-cinder | 07:53 | |
| *** nellysmitt has quit IRC | 07:58 | |
| *** ronis_ has joined #openstack-cinder | 08:01 | |
| openstackgerrit | Anish Bhatt proposed openstack/cinder: Add support for chiscsi iscsi helper https://review.openstack.org/147758 | 08:02 |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Adapt volume manager to new driver structure https://review.openstack.org/138999 | 08:03 |
| openstackgerrit | Anish Bhatt proposed openstack/cinder: Improve error handling in Targets refactoring Only allow IOError to fall through when file not found is acceptable, return more precise exception in remove_target https://review.openstack.org/154713 | 08:06 |
| *** alecv has joined #openstack-cinder | 08:08 | |
| *** lpetrut has quit IRC | 08:08 | |
| *** markvoelker has joined #openstack-cinder | 08:09 | |
| openstackgerrit | Anish Bhatt proposed openstack/cinder: Improve error handling in Targets refactoring https://review.openstack.org/154713 | 08:09 |
| *** markvoelker has quit IRC | 08:14 | |
| *** ndipanov has joined #openstack-cinder | 08:14 | |
| *** Longgeek has quit IRC | 08:18 | |
| *** Longgeek has joined #openstack-cinder | 08:19 | |
| *** dulek has joined #openstack-cinder | 08:24 | |
| openstackgerrit | wuyuting proposed openstack/cinder: Create a type volume also update none type quota https://review.openstack.org/153525 | 08:25 |
| *** vnagpal has quit IRC | 08:26 | |
| *** vnagpal has joined #openstack-cinder | 08:27 | |
| *** Raj__ has joined #openstack-cinder | 08:29 | |
| *** Raj_ has quit IRC | 08:30 | |
| *** vincent_hou_ has joined #openstack-cinder | 08:31 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware:Use datastore selection logic in new module https://review.openstack.org/155313 | 08:31 |
| *** vincent_hou has quit IRC | 08:32 | |
| openstackgerrit | TaoBai proposed openstack/cinder: Storwize: Replication status still active when primary copy is offline https://review.openstack.org/154827 | 08:34 |
| *** Mandell has quit IRC | 08:35 | |
| *** vincent_hou_ has quit IRC | 08:36 | |
| openstackgerrit | Anish Bhatt proposed openstack/cinder: Improve error handling in Targets refactoring https://review.openstack.org/154713 | 08:38 |
| *** openstackgerrit has quit IRC | 08:42 | |
| *** openstackgerrit has joined #openstack-cinder | 08:43 | |
| *** pradeep_s has joined #openstack-cinder | 08:43 | |
| *** jpich has joined #openstack-cinder | 08:51 | |
| *** lpetrut has joined #openstack-cinder | 08:51 | |
| *** jordanP has joined #openstack-cinder | 08:56 | |
| *** MasterPiece has joined #openstack-cinder | 09:05 | |
| *** Raj__ has quit IRC | 09:09 | |
| *** markvoelker has joined #openstack-cinder | 09:10 | |
| *** MasterPiece has quit IRC | 09:11 | |
| *** humble_ has quit IRC | 09:13 | |
| *** markvoelker has quit IRC | 09:14 | |
| *** jistr has joined #openstack-cinder | 09:16 | |
| *** karimb has joined #openstack-cinder | 09:18 | |
| *** sgotliv__ has quit IRC | 09:21 | |
| *** e0ne has joined #openstack-cinder | 09:26 | |
| *** Longgeek has quit IRC | 09:29 | |
| *** yuriy_n17 has joined #openstack-cinder | 09:37 | |
| *** obutenko has joined #openstack-cinder | 09:45 | |
| *** ronis_ has quit IRC | 09:46 | |
| *** chlong has quit IRC | 09:49 | |
| *** davechen has quit IRC | 09:50 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Huawei driver remove LUN controller change https://review.openstack.org/152116 | 10:01 |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Huawei driver check before associating LUN to a LUN group https://review.openstack.org/148144 | 10:01 |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Huawei driver fix problems under multipath https://review.openstack.org/150767 | 10:01 |
| openstackgerrit | wuyuting proposed openstack/cinder: Create a type volume also update none type quota https://review.openstack.org/153525 | 10:02 |
| *** rajesht has joined #openstack-cinder | 10:03 | |
| *** MasterPiece has joined #openstack-cinder | 10:09 | |
| *** marcusvrn has joined #openstack-cinder | 10:09 | |
| *** markvoelker has joined #openstack-cinder | 10:11 | |
| openstackgerrit | wuyuting proposed openstack/cinder: Wrong logging when volume type quota exceed https://review.openstack.org/153784 | 10:11 |
| *** marcusvrn1 has quit IRC | 10:12 | |
| *** marcusvrn has quit IRC | 10:13 | |
| *** marcusvrn has joined #openstack-cinder | 10:13 | |
| *** alexpilotti has joined #openstack-cinder | 10:14 | |
| *** marcusvrn1 has joined #openstack-cinder | 10:15 | |
| *** markvoelker has quit IRC | 10:15 | |
| *** marcusvrn has quit IRC | 10:17 | |
| *** marcusvrn1 has quit IRC | 10:20 | |
| *** karimb has quit IRC | 10:21 | |
| *** e0ne is now known as e0ne_ | 10:23 | |
| *** e0ne_ is now known as e0ne | 10:32 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Relocate volume to compliant datastore https://review.openstack.org/155702 | 10:34 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Relocate volume to compliant datastore https://review.openstack.org/155702 | 10:42 |
| *** marcusvrn1 has joined #openstack-cinder | 10:45 | |
| *** Miouge_ has joined #openstack-cinder | 10:45 | |
| *** Miouge has quit IRC | 10:46 | |
| *** Miouge_ is now known as Miouge | 10:46 | |
| *** marcusvrn1 has quit IRC | 10:49 | |
| *** karimb has joined #openstack-cinder | 10:51 | |
| *** dulek_ has joined #openstack-cinder | 11:03 | |
| *** marcusvrn1 has joined #openstack-cinder | 11:05 | |
| *** dulek has quit IRC | 11:06 | |
| dulek_ | e0ne: hi, I wonder why your proposed solution to 1409012 was unstable. Can you elaborate on that? | 11:07 |
| *** dulek_ is now known as dulek | 11:07 | |
| *** markvoelker has joined #openstack-cinder | 11:12 | |
| *** markvoelker has quit IRC | 11:16 | |
| *** kmartin has quit IRC | 11:19 | |
| *** asselin has quit IRC | 11:21 | |
| *** aix has joined #openstack-cinder | 11:27 | |
| e0ne | dulek_: hi! i could try it once more to get back it in my mind | 11:29 |
| *** karimb has quit IRC | 11:30 | |
| *** karimb has joined #openstack-cinder | 11:33 | |
| *** Raj__ has joined #openstack-cinder | 11:34 | |
| *** marcusvrn1 has quit IRC | 11:36 | |
| dulek | e0ne: It is wrong idea to wait 60s on every NoValidHost found, but definitely we can try to wait 60 s on the start of every request. | 11:37 |
| *** aix has quit IRC | 11:38 | |
| *** e0ne is now known as e0ne_ | 11:38 | |
| dulek | e0ne: I mean waiting: 60 - (now() - start_of_service) | 11:38 |
| dulek | e0ne_: What I wonder is if this won't block the service - but it shouldn't as this is eventlet | 11:39 |
| dulek | e0ne_: I'll see if it is possible to implement it this way. | 11:39 |
| dulek | e0ne_: Can you share how tested it when you encountered instability? | 11:39 |
| *** TobiasE1 has joined #openstack-cinder | 11:40 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Import only modules: H302 https://review.openstack.org/145780 | 11:41 |
| *** ankit_ag has quit IRC | 11:41 | |
| *** ankit_ag has joined #openstack-cinder | 11:41 | |
| *** TobiasE has quit IRC | 11:42 | |
| *** marcusvrn has joined #openstack-cinder | 11:42 | |
| *** e0ne_ has quit IRC | 11:43 | |
| *** e0ne has joined #openstack-cinder | 11:52 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Change exception message in volume api https://review.openstack.org/143656 | 11:57 |
| *** marcusvrn has quit IRC | 12:02 | |
| *** dulek_ has joined #openstack-cinder | 12:02 | |
| *** aix has joined #openstack-cinder | 12:04 | |
| *** e0ne is now known as e0ne_ | 12:04 | |
| *** dulek has quit IRC | 12:04 | |
| *** marcusvrn has joined #openstack-cinder | 12:06 | |
| *** markvoelker has joined #openstack-cinder | 12:13 | |
| *** jaypipes has quit IRC | 12:13 | |
| *** e0ne_ has quit IRC | 12:14 | |
| *** deepakcs has quit IRC | 12:15 | |
| *** jaypipes has joined #openstack-cinder | 12:16 | |
| *** dencaval has joined #openstack-cinder | 12:16 | |
| *** markvoelker has quit IRC | 12:17 | |
| dencaval | Hey guys, will be possible to use V1 API at kilo? | 12:23 |
| *** kallebe has joined #openstack-cinder | 12:23 | |
| dulek_ | https://blueprints.launchpad.net/cinder/+spec/remove-v1-api | 12:26 |
| dulek_ | As it isn't targeted then probably yes | 12:26 |
| *** smoriya has quit IRC | 12:28 | |
| *** Mandell has joined #openstack-cinder | 12:36 | |
| dencaval | dulek_ got it | 12:36 |
| *** IanGovett has joined #openstack-cinder | 12:39 | |
| *** dulek_ has quit IRC | 12:40 | |
| *** Mandell has quit IRC | 12:40 | |
| *** rajesht has quit IRC | 12:43 | |
| *** ankit_ag has quit IRC | 12:46 | |
| *** TobiasE1 has quit IRC | 12:49 | |
| *** TobiasE has joined #openstack-cinder | 12:55 | |
| *** TobiasE has quit IRC | 12:59 | |
| *** ronis_ has joined #openstack-cinder | 13:06 | |
| *** markvoelker has joined #openstack-cinder | 13:10 | |
| *** jpich has quit IRC | 13:12 | |
| *** EmilienM|afk is now known as EmilienM | 13:19 | |
| *** TobiasE has joined #openstack-cinder | 13:22 | |
| *** TobiasE has left #openstack-cinder | 13:23 | |
| *** Tross has quit IRC | 13:23 | |
| *** Tross1 has joined #openstack-cinder | 13:23 | |
| *** akerr has joined #openstack-cinder | 13:34 | |
| *** bswartz has quit IRC | 13:35 | |
| *** Raj__ has quit IRC | 13:37 | |
| *** vnagpal has quit IRC | 13:45 | |
| *** bill_az has joined #openstack-cinder | 13:46 | |
| *** timcl has joined #openstack-cinder | 13:47 | |
| *** karimb has quit IRC | 13:48 | |
| *** karimb has joined #openstack-cinder | 13:48 | |
| *** delattec has joined #openstack-cinder | 13:52 | |
| *** kaufer has joined #openstack-cinder | 13:52 | |
| *** Miouge has quit IRC | 13:53 | |
| *** Yogi1 has joined #openstack-cinder | 13:54 | |
| *** cdelatte has quit IRC | 13:55 | |
| *** xyang1 has joined #openstack-cinder | 13:57 | |
| *** Yogi11 has joined #openstack-cinder | 13:59 | |
| *** Yogi1 has quit IRC | 14:03 | |
| *** Miouge has joined #openstack-cinder | 14:03 | |
| *** dulek has joined #openstack-cinder | 14:04 | |
| *** dulek has quit IRC | 14:09 | |
| *** eharney has quit IRC | 14:10 | |
| *** bswartz has joined #openstack-cinder | 14:13 | |
| *** dustins has joined #openstack-cinder | 14:16 | |
| *** karimb has quit IRC | 14:22 | |
| *** david-lyle_afk is now known as david-lyle | 14:22 | |
| *** kallebe has left #openstack-cinder | 14:23 | |
| *** e0ne has joined #openstack-cinder | 14:26 | |
| *** Yogi11 has quit IRC | 14:33 | |
| *** annegentle has quit IRC | 14:33 | |
| *** Miouge has quit IRC | 14:34 | |
| *** nshaikh has quit IRC | 14:35 | |
| kaufer | hemna: Do you have time for a question on period tasks related to a comment you made here? https://review.openstack.org/#/c/154673/2/cinder/volume/manager.py | 14:38 |
| *** Yogi1 has joined #openstack-cinder | 14:39 | |
| *** rushil has joined #openstack-cinder | 14:39 | |
| *** jungleboyj has quit IRC | 14:40 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:40 | |
| smcginnis | So L is officially Liberty. | 14:41 |
| *** Miouge has joined #openstack-cinder | 14:41 | |
| *** mriedem_zz has quit IRC | 14:44 | |
| *** mriedem has joined #openstack-cinder | 14:44 | |
| *** e0ne is now known as e0ne_ | 14:45 | |
| *** nkrinner has quit IRC | 14:50 | |
| *** e0ne_ is now known as e0ne | 14:55 | |
| *** jaypipes is now known as leakypipes | 14:56 | |
| *** enterprisedc has quit IRC | 14:56 | |
| *** xyang has joined #openstack-cinder | 14:57 | |
| *** akerr has quit IRC | 14:58 | |
| *** dansmith is now known as superdan | 14:58 | |
| *** rmesta has joined #openstack-cinder | 14:58 | |
| *** r-daneel has joined #openstack-cinder | 15:00 | |
| *** Mandell has joined #openstack-cinder | 15:00 | |
| *** liusheng has quit IRC | 15:01 | |
| *** liusheng has joined #openstack-cinder | 15:01 | |
| *** IanGovett1 has joined #openstack-cinder | 15:04 | |
| *** IanGovett has quit IRC | 15:06 | |
| *** eharney has joined #openstack-cinder | 15:08 | |
| *** primechuck has joined #openstack-cinder | 15:10 | |
| *** enterprisedc has joined #openstack-cinder | 15:12 | |
| *** akerr has joined #openstack-cinder | 15:12 | |
| *** MasterPiece has quit IRC | 15:14 | |
| *** MasterPiece has joined #openstack-cinder | 15:15 | |
| *** I has joined #openstack-cinder | 15:16 | |
| *** kaufer has quit IRC | 15:16 | |
| *** I is now known as Guest33715 | 15:16 | |
| *** Mandell has quit IRC | 15:20 | |
| cebruns | Thanks smcginnis - we can use the "Give me Liberty!" quote as deadlines approach. | 15:22 |
| smcginnis | cebruns: +1 :) | 15:23 |
| *** jungleboyj has joined #openstack-cinder | 15:26 | |
| *** thangp has joined #openstack-cinder | 15:26 | |
| *** annegentle has joined #openstack-cinder | 15:27 | |
| *** dulek has joined #openstack-cinder | 15:28 | |
| *** kaufer has joined #openstack-cinder | 15:29 | |
| *** Miouge has quit IRC | 15:32 | |
| *** Miouge has joined #openstack-cinder | 15:34 | |
| *** Guest33715 has quit IRC | 15:36 | |
| *** marcusvrn1 has joined #openstack-cinder | 15:36 | |
| *** asselin has joined #openstack-cinder | 15:37 | |
| *** tbarron has joined #openstack-cinder | 15:38 | |
| *** marcusvrn has quit IRC | 15:38 | |
| *** e0ne is now known as e0ne_ | 15:38 | |
| *** bill_ibm_ has joined #openstack-cinder | 15:40 | |
| *** bill_ibm__ has joined #openstack-cinder | 15:41 | |
| *** coolsvap is now known as coolsvap_ | 15:43 | |
| *** e0ne_ is now known as e0ne | 15:43 | |
| *** bill_ibm has quit IRC | 15:44 | |
| *** bill_ibm_ has quit IRC | 15:44 | |
| *** marcusvrn has joined #openstack-cinder | 15:47 | |
| *** MasterPiece has quit IRC | 15:48 | |
| *** marcusvrn1 has quit IRC | 15:49 | |
| *** Miouge has quit IRC | 15:54 | |
| *** Miouge has joined #openstack-cinder | 15:54 | |
| *** MasterPiece has joined #openstack-cinder | 15:57 | |
| *** Miouge has quit IRC | 15:58 | |
| *** marcusvrn1 has joined #openstack-cinder | 15:59 | |
| *** marcusvrn has quit IRC | 16:01 | |
| *** kmartin has joined #openstack-cinder | 16:02 | |
| *** Yogi1 has quit IRC | 16:05 | |
| *** marcusvrn1 has quit IRC | 16:06 | |
| *** marcusvrn has joined #openstack-cinder | 16:06 | |
| *** mriedem1 has joined #openstack-cinder | 16:06 | |
| *** ndipanov is now known as ndipanoff | 16:07 | |
| *** mriedem has quit IRC | 16:08 | |
| *** muraiki has joined #openstack-cinder | 16:09 | |
| muraiki | hi all. I'm trying to understand how ceilometer gets snapshot samples (such as snapshot.exists). am I correct in understanding that these are generated in cinder, as opposed to being something gathered through polling? | 16:10 |
| *** dulek has quit IRC | 16:10 | |
| *** dulek has joined #openstack-cinder | 16:11 | |
| *** dulek has quit IRC | 16:16 | |
| *** emagana has joined #openstack-cinder | 16:20 | |
| *** MasterPiece has quit IRC | 16:20 | |
| *** MasterPiece has joined #openstack-cinder | 16:24 | |
| *** delatte has joined #openstack-cinder | 16:25 | |
| *** delattec has quit IRC | 16:26 | |
| jordanP | muraiki, you are correct | 16:26 |
| jordanP | cinder sends "notifications" on a messages bus (ie rabbitmq) | 16:26 |
| muraiki | I saw that in older versions of cinder there was an audit script that generated these notifications, but that script doesn't exist in icehouse. I've tried searching through the cinder repo, but haven't been able to figure out how the notifications are now generated | 16:27 |
| muraiki | would you be able to point me in the right direction? | 16:27 |
| muraiki | oh, actually it looks like I have it backwards... the audit script is in master, but not icehouse or juno | 16:29 |
| muraiki | ultimately, my problem is that I'm stuck on icehouse and wanted to see if I could tie into those snapshot notifications somehow | 16:30 |
| muraiki | but perhaps that isn't possible in icehouse :( | 16:30 |
| *** jdurgin1 has joined #openstack-cinder | 16:31 | |
| *** Mandell has joined #openstack-cinder | 16:31 | |
| *** Yogi1 has joined #openstack-cinder | 16:33 | |
| nikesh_vedams | thanks all reviewers for their reviews on https://review.openstack.org/#/c/145778/ .we learned lot of things from your reviews :) | 16:36 |
| jgriffith | nikesh_vedams: you're very welcome | 16:38 |
| *** timcl has quit IRC | 16:38 | |
| jordanP | muraiki, notification should be available in icehouse | 16:39 |
| *** xyang has quit IRC | 16:39 | |
| jordanP | it should be around https://github.com/openstack/cinder/blob/stable/icehouse/cinder/volume/flows/manager/create_volume.py | 16:39 |
| *** xyang has joined #openstack-cinder | 16:39 | |
| jordanP | but i am not sure | 16:40 |
| muraiki | jordanP: thanks so much! I'll check things out | 16:40 |
| e0ne | jgriffith: thanks for merge https://review.openstack.org/#/c/148038/. I did not have time to review it after update | 16:44 |
| *** dulek has joined #openstack-cinder | 16:45 | |
| nikesh_vedams | we have a small team but we put our best efforts for giving quick responses :) on https://review.openstack.org/#/c/145778/ | 16:47 |
| *** _cjones_ has joined #openstack-cinder | 16:48 | |
| *** dulek_ has joined #openstack-cinder | 16:49 | |
| *** dulek_ has quit IRC | 16:52 | |
| *** dulek_ has joined #openstack-cinder | 16:52 | |
| *** afazekas_ has quit IRC | 16:53 | |
| *** dulek has quit IRC | 16:53 | |
| *** yuriy_n17 has quit IRC | 16:55 | |
| *** MasterPiece has quit IRC | 16:55 | |
| *** Mandell has quit IRC | 16:55 | |
| *** EmilienM is now known as EmilienM|afk | 16:56 | |
| *** dulek has joined #openstack-cinder | 16:56 | |
| *** e0ne is now known as e0ne_ | 16:58 | |
| *** jordanP has quit IRC | 16:59 | |
| jgriffith | e0ne_: sure, thanks for the review. He seemed to hit everything you pointed out | 16:59 |
| *** MasterPiece has joined #openstack-cinder | 17:00 | |
| *** dulek_ has quit IRC | 17:00 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Remove randomness from NFS mount tests https://review.openstack.org/155823 | 17:02 |
| *** hemna_ has joined #openstack-cinder | 17:03 | |
| *** e0ne_ is now known as e0ne | 17:03 | |
| e0ne | jgriffith: yes, i've take a look on it after merge | 17:04 |
| *** bswartz has quit IRC | 17:04 | |
| *** vilobhmm has joined #openstack-cinder | 17:07 | |
| *** alecv has quit IRC | 17:07 | |
| *** MasterPiece has quit IRC | 17:10 | |
| *** e0ne is now known as e0ne_ | 17:11 | |
| dulek | e0ne: Wanna join me and vilobhmm to discuss state management? | 17:12 |
| dulek | e0ne_, | 17:13 |
| *** timcl has joined #openstack-cinder | 17:13 | |
| *** Apoorva has joined #openstack-cinder | 17:13 | |
| *** alexpilotti has quit IRC | 17:13 | |
| *** alexpilotti has joined #openstack-cinder | 17:15 | |
| *** alexpilotti has quit IRC | 17:16 | |
| *** e0ne_ is now known as e0ne | 17:16 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support https://review.openstack.org/85847 | 17:18 |
| *** dulek_ has joined #openstack-cinder | 17:18 | |
| *** timcl has quit IRC | 17:19 | |
| *** abehl has joined #openstack-cinder | 17:20 | |
| *** bswartz has joined #openstack-cinder | 17:20 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/python-cinderclient: Add volume multi attach support https://review.openstack.org/85856 | 17:21 |
| *** timcl has joined #openstack-cinder | 17:22 | |
| *** dulek has quit IRC | 17:22 | |
| hemna_ | jgriffith, ping | 17:23 |
| jgriffith | hemna_: poing | 17:23 |
| hemna_ | so it looks like we are in a chicken/egg issue with multi attach | 17:23 |
| hemna_ | https://review.openstack.org/#/c/153033/ | 17:23 |
| hemna_ | jay pipes' patch is failing in check because of the needed cinderclient | 17:23 |
| hemna_ | which hasn't landed and been released yet | 17:23 |
| hemna_ | fwiw | 17:24 |
| jgriffith | hemna_: yeah | 17:24 |
| jgriffith | hemna_: need to get thingee to push a release | 17:24 |
| jgriffith | hemna_: I'm still wondering why it's not possible to code in back compatability there | 17:24 |
| hemna_ | so if we can get the code reviewed and released if everyone is happy with it, that'd help the nova side out. | 17:24 |
| hemna_ | I'm sure there is a way | 17:25 |
| *** ekarlso has quit IRC | 17:25 | |
| *** ekarlso has joined #openstack-cinder | 17:25 | |
| jgriffith | leakypipes: hola senor | 17:25 |
| hemna_ | they'd have to either check for the future version | 17:25 |
| hemna_ | or check to see if the method supports the new parameter or not | 17:25 |
| jgriffith | hemna_: yeah, I was thinking even just handle the exception and fall back but that's pretty hacky I suppose | 17:25 |
| hemna_ | yah, both of my options feel hacky as well | 17:26 |
| jgriffith | hemna_: so honestly you are probably going to need to run through this with Mike | 17:26 |
| jgriffith | hemna_: My suggestsion would in fact be bump the client... BUT | 17:26 |
| hemna_ | yah I was going to ping him as well, but he's not on currently | 17:26 |
| jgriffith | hemna_: that's not so easy | 17:26 |
| *** dulek has joined #openstack-cinder | 17:27 | |
| jgriffith | hemna_: in terms of updating the reqs everywhere | 17:27 |
| *** marcusvrn1 has joined #openstack-cinder | 17:27 | |
| hemna_ | yah | 17:27 |
| jgriffith | hemna_: and it feels "dirty" that we're changing an API call in a way that "breaks" things | 17:27 |
| hemna_ | the client needs to land | 17:27 |
| hemna_ | then the mirrors need to update | 17:27 |
| hemna_ | then the global reqs, nova reqs, etc | 17:28 |
| jgriffith | hemna_: well, that's not really what I was thinking | 17:28 |
| *** marcusvrn has quit IRC | 17:28 | |
| hemna_ | oh | 17:28 |
| jgriffith | that's not a terribly big deal | 17:28 |
| jgriffith | and I suppose since it's not a Cinder API version bump it's not terrible | 17:28 |
| jgriffith | but that's my concern... is for all purposes it probably should be an API version bump | 17:28 |
| jgriffith | and remember that took a year to propogate | 17:29 |
| hemna_ | ugh | 17:29 |
| *** annegentle has quit IRC | 17:30 | |
| *** dulek_ has quit IRC | 17:30 | |
| *** leeantho has joined #openstack-cinder | 17:31 | |
| *** dulek_ has joined #openstack-cinder | 17:32 | |
| *** dulek has quit IRC | 17:36 | |
| jgriffith | hemna_: ok... maybe not as big a deal as I thought | 17:37 |
| hemna_ | phew | 17:37 |
| jgriffith | hemna_: so make the third call in Cinderclient an optional with a default | 17:37 |
| jgriffith | hemna_: nothing breaks (might already be that way, should be) | 17:37 |
| hemna_ | yup it is | 17:38 |
| jgriffith | hemna_: in the nova/volume/cinder code I'm not sure why we can't version that... | 17:38 |
| jgriffith | hemna_: or query the version in order to form the correct call | 17:38 |
| *** dulek_ has quit IRC | 17:39 | |
| eharney | can anyone ask CloudFounders CI to stop posting "Merge Failed." messages on patchsets? That result can only mean "my CI is broken". | 17:39 |
| *** jistr has quit IRC | 17:41 | |
| hemna_ | jgriffith, thanks. I'll see what I can hack together in the nova side to test the version. I'll just assume anything greater than the current version will have the option. | 17:41 |
| jgriffith | hemna_: see my comments on the Nova patch | 17:47 |
| hemna_ | thanks! | 17:48 |
| *** harlowja_away is now known as harlowja_ | 17:48 | |
| jgriffith | eharney: I think Mike wanted to "own" all things CI | 17:48 |
| jgriffith | eharney: that being said, we could just ask infra to disable the account until it's fixed | 17:49 |
| eharney | jgriffith: yeah, I didn't want to go harass them myself. But it's been doing that for a while, and is pretty obnoxious. | 17:49 |
| *** marcusvrn1 has quit IRC | 17:49 | |
| jgriffith | eharney: agreed... honestly I "toggle CI" off and don't even bother looking at it anymore | 17:49 |
| jgriffith | eharney: between that and cool things I pointed out last night like CI's that run and pass in less than a minut of the patch hitting gerrit :) | 17:50 |
| jgriffith | I which I had that kind of compute power :) | 17:50 |
| eharney | :) | 17:50 |
| eharney | jgriffith: the answer is more SSDs, right? | 17:52 |
| jgriffith | eharney: LOL | 17:53 |
| jgriffith | eharney: and 1000 way processors I guess | 17:53 |
| jgriffith | If I could sell a system that deployed OpenStack on a VM, ran unit tests and dsvm full all in under a minute I suspect I could be a millionaire | 17:54 |
| jgriffith | I'm sure people would pay for it | 17:54 |
| jgriffith | That's wicked fast :) | 17:55 |
| *** MasterPiece has joined #openstack-cinder | 17:55 | |
| *** e0ne has quit IRC | 17:56 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Remove TestWSGIService.test_reset_pool_size_to_default https://review.openstack.org/155840 | 17:58 |
| openstackgerrit | Anish Bhatt proposed openstack/cinder: Add support for chiscsi iscsi helper https://review.openstack.org/147758 | 17:59 |
| *** mtanino has joined #openstack-cinder | 18:01 | |
| *** jcru has joined #openstack-cinder | 18:01 | |
| *** emagana has quit IRC | 18:06 | |
| *** tsekiyama has joined #openstack-cinder | 18:07 | |
| *** dannywilson has joined #openstack-cinder | 18:11 | |
| *** patrickeast has joined #openstack-cinder | 18:17 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Don't require binding to port 4444 https://review.openstack.org/155849 | 18:20 |
| *** Guest40096 has joined #openstack-cinder | 18:21 | |
| *** Guest40096 is now known as annegentle | 18:22 | |
| *** liusheng has quit IRC | 18:25 | |
| *** liusheng has joined #openstack-cinder | 18:25 | |
| *** kallebe has joined #openstack-cinder | 18:26 | |
| *** Mandell has joined #openstack-cinder | 18:30 | |
| kallebe | Hello. I have a question, but first I want to say that I am very new to Gerrit: why does gerrit sometimes create a new change request instead of just commiting a new patch set? | 18:31 |
| jgriffith | kallebe: not sure what you mean, can you ellaborate a bit? | 18:32 |
| *** EmilienM|afk is now known as EmilienM | 18:32 | |
| kallebe | jgriffith it is that I don't know how to continue work from a given patch set. | 18:32 |
| kallebe | is it that ammend option? | 18:33 |
| jgriffith | kallebe: ahh... yes | 18:33 |
| jgriffith | kallebe: git commit --amend -a | 18:33 |
| kallebe | oh, ok. thanks. Now it will work. | 18:33 |
| kmartin | kallebe, if you use the --amend option it will amend to the existing patch, without in will create a new change. see https://wiki.openstack.org/wiki/Gerrit_Workflow | 18:33 |
| jgriffith | kallebe: also, you can pull a patch down from gerrit really easy with "git review -d xxxx" | 18:33 |
| kallebe | kmartin jgriffith ok, thanks for your help.Now what do I do with the new change with that dependency? Can I delete it somehow? | 18:34 |
| *** aix has quit IRC | 18:34 | |
| *** bswartz has quit IRC | 18:42 | |
| *** bswartz has joined #openstack-cinder | 18:44 | |
| *** emagana has joined #openstack-cinder | 18:51 | |
| *** emagana has quit IRC | 18:51 | |
| *** emagana has joined #openstack-cinder | 18:52 | |
| *** MasterPiece has quit IRC | 18:54 | |
| kmartin | kallebe, not sure I understand your question, do you want to Abandon one of the change that was posted up to review.openstack.org by mistake? | 18:55 |
| kallebe | kmartin yes | 18:57 |
| kallebe | abandon I know how to do | 18:57 |
| kallebe | but I would like to delete so it does not occupy space in servers | 18:57 |
| *** barra204 has joined #openstack-cinder | 18:57 | |
| *** liusheng has quit IRC | 18:57 | |
| kmartin | abandon is the only option I know about | 18:58 |
| *** barra204 is now known as shamamunyi | 18:58 | |
| *** shamamunyi is now known as shakamunyi | 18:58 | |
| *** liusheng has joined #openstack-cinder | 18:58 | |
| *** MasterPiece has joined #openstack-cinder | 19:01 | |
| *** Apoorva_ has joined #openstack-cinder | 19:03 | |
| *** Apoorva_ has quit IRC | 19:04 | |
| *** Apoorva_ has joined #openstack-cinder | 19:04 | |
| *** Apoorva has quit IRC | 19:05 | |
| *** lpetrut has quit IRC | 19:06 | |
| *** IanGovett1 has quit IRC | 19:21 | |
| *** Yogi1 has quit IRC | 19:23 | |
| *** IanGovett has joined #openstack-cinder | 19:25 | |
| *** dannywilson has quit IRC | 19:28 | |
| *** dannywilson has joined #openstack-cinder | 19:29 | |
| *** ndipanoff has quit IRC | 19:32 | |
| *** emagana has quit IRC | 19:32 | |
| *** kallebe has left #openstack-cinder | 19:32 | |
| *** dencaval has quit IRC | 19:33 | |
| *** emagana has joined #openstack-cinder | 19:33 | |
| *** emagana has quit IRC | 19:33 | |
| *** lpetrut has joined #openstack-cinder | 19:33 | |
| *** emagana has joined #openstack-cinder | 19:33 | |
| *** emagana has quit IRC | 19:34 | |
| *** Yogi1 has joined #openstack-cinder | 19:35 | |
| *** saltsa has quit IRC | 19:37 | |
| *** emagana has joined #openstack-cinder | 19:38 | |
| *** Lee1092 has quit IRC | 19:40 | |
| *** saltsa has joined #openstack-cinder | 19:44 | |
| *** kfox1111 has joined #openstack-cinder | 19:53 | |
| kfox1111 | how do you update the default cinder quota's? | 19:53 |
| kfox1111 | I want to default to 0 a bunch of things so I can manually add the quota to specific projects. | 19:54 |
| *** thingee has joined #openstack-cinder | 19:58 | |
| kfox1111 | quota_volumes = 0 in /etc/cinder/cinder.conf didnt' seem to work. | 19:58 |
| *** akerr has quit IRC | 20:00 | |
| thingee | jgriffith: ping | 20:01 |
| guitarzan | kfox1111: set the default quota class | 20:01 |
| *** barra204 has joined #openstack-cinder | 20:02 | |
| *** shakamunyi has quit IRC | 20:02 | |
| kfox1111 | ah. the option that is: use_default_quota_class=true | 20:02 |
| kfox1111 | so does that mean i update the 'default' quota class via the cli instead? | 20:02 |
| guitarzan | kfox1111: I mean cinder quota-class-update default --volumes 0 | 20:03 |
| guitarzan | yeah | 20:03 |
| guitarzan | kfox1111: no guarantees those args are in the right order... | 20:03 |
| kfox1111 | cinder quota-class-update --volumes 0 default | 20:03 |
| kfox1111 | ERROR: 'NoneType' object has no attribute '_info' | 20:03 |
| kfox1111 | do I have to create one? | 20:04 |
| *** MasterPiece has quit IRC | 20:04 | |
| thingee | jgriffith: I'm struggling to understand your comment on https://review.openstack.org/#/c/150511/2 ... "YES the filter scheduler currently will look at any key and try and match it to a hosts capabilities, UNLESS that keys is scoped." | 20:04 |
| *** hemna_ has quit IRC | 20:04 | |
| thingee | I don't understand what scope means in this context | 20:04 |
| thingee | if the key/value is in a volume type, that to me means it's scoped to be filtered on the driver's reported capabilities. | 20:05 |
| guitarzan | kfox1111: hmm, I'm not sure | 20:07 |
| guitarzan | kfox1111: what log is throwing that error? | 20:07 |
| kfox1111 | the cli. | 20:08 |
| guitarzan | ah, a bug in cinderclient? no way :) | 20:08 |
| kfox1111 | it looks like it wants a quota class name, 'default' in this case. but is there really one named default? | 20:08 |
| guitarzan | oh hey, I get the same thing, fun | 20:08 |
| guitarzan | kfox1111: there should be one | 20:09 |
| kfox1111 | let me look at the logs. | 20:09 |
| guitarzan | do a cinder quota-class-show default | 20:09 |
| guitarzan | I think it's working | 20:09 |
| guitarzan | but the cli is busted | 20:09 |
| kfox1111 | ah... the cli is busted. it did work. | 20:10 |
| guitarzan | I did cinder quota-class-update --volumes 42 somenewclass | 20:10 |
| *** hemna_ has joined #openstack-cinder | 20:10 | |
| *** bill_az has quit IRC | 20:10 | |
| *** patrickeast has quit IRC | 20:11 | |
| kfox1111 | thanks for the help. :) | 20:11 |
| thingee | hemna, jgriffith: I've commented back on https://review.openstack.org/#/c/150511/ ... I need some help with understanding what "scoped" means. If I can get an example of how people would like to see things structured for the capabilities data structure in handling both well defined keys and vendor keys, that would help me a lot. | 20:11 |
| *** thingee has quit IRC | 20:15 | |
| openstackgerrit | Brianna Poulos proposed openstack/cinder: Add project_id to barbican keymgr wrapper https://review.openstack.org/155875 | 20:17 |
| *** diegows has joined #openstack-cinder | 20:18 | |
| *** MasterPiece has joined #openstack-cinder | 20:18 | |
| *** ndipanoff has joined #openstack-cinder | 20:19 | |
| *** barra204 has quit IRC | 20:19 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Clear migration_status from a destination volume if migration fails https://review.openstack.org/154739 | 20:21 |
| *** zzzeek has joined #openstack-cinder | 20:25 | |
| zzzeek | hi folks, can someone give me some insight on https://bugs.launchpad.net/cinder/+bug/1417018 and under what circumstances this occurs, how frequently, when it started, etc. ? | 20:26 |
| openstack | Launchpad bug 1417018 in Cinder "Cinder encounters dbapi error: NoSuchColumnError: "Could not locate column in row for column '%(140070887032400 anon)s.volumes_id'"" [Undecided,New] | 20:26 |
| eharney | zzzeek: that specific report happened on EMC's CI platform... xyang: happen to know anything about that? | 20:28 |
| zzzeek | eharney: that kind of issue is often something in SQLA internals, but they are typically very difficult to trip over, especially with the simple models openstack uses, id imagine this is related to the pagination code but there’s a lot there and seeing the SQL would help | 20:29 |
| eharney | zzzeek: it does look like it happened at initial cinder-volume startup, and likely when the volumes table is empty, if that helps any | 20:31 |
| zzzeek | eharney: so you’re saying, it was with an entirely deterministic startup sequence, yet it only happened once | 20:31 |
| eharney | zzzeek: well... i've heard of two specific occurrences (not mine, unfortunately) | 20:32 |
| eharney | zzzeek: i ended up looking at this bug because jbernard ran into it somehow | 20:32 |
| jbernard | eharney, zzzeek: ive been seeing this most of the day | 20:33 |
| zzzeek | it is also possible that MySQL itself is screwing up and confusing the result processing code. if this issue happens very randomly when the program is being run in an otherwise entirely predictable and often-repeated way, that’s a possibility | 20:33 |
| zzzeek | jbernard: OK when did it start | 20:33 |
| zzzeek | jbernard: and is it local to some environemnts ? | 20:33 |
| jbernard | zzzeek: since this morning | 20:34 |
| zzzeek | jbernard: if i just load up a devstack and run that command will i get the error ? | 20:34 |
| jbernard | zzzeek: i cant speak for other env's ive only had access to my local one today | 20:34 |
| jbernard | zzzeek: yep, that should do it | 20:34 |
| zzzeek | jbernard: so you can reprodcue it every time, right | 20:34 |
| *** bill_az has joined #openstack-cinder | 20:34 | |
| *** barra204 has joined #openstack-cinder | 20:34 | |
| jbernard | zzzeek: it seems to be non-deterministic | 20:34 |
| zzzeek | jbernard: OK, you can reprodiuce it often, locally ? | 20:34 |
| jbernard | zzzeek: sometimes a volume operation succeeds, sometimes i see it | 20:35 |
| jbernard | but never in the same way | 20:35 |
| jbernard | and never exactly repeatable | 20:35 |
| *** barra204 has quit IRC | 20:35 | |
| eharney | i wonder if update_volume_stats is racing against set_voldb_empty_at_startup_indicator's (volume_get_all), may explain why it's not everywhere. both of those happen right at c-vol startup | 20:35 |
| jbernard | ive been working around it (drinking heavily) to rework my migration code | 20:35 |
| *** patrickeast has joined #openstack-cinder | 20:36 | |
| zzzeek | jbernard: any chance you can get it to log the SQL for me ? | 20:36 |
| jbernard | zzzeek: i can enable anything you need | 20:37 |
| jbernard | zzzeek: what specifically will help? | 20:37 |
| zzzeek | jbernard: ideally this would be an “echo=‘debug’” kind of thing. i wonder if even the MySQL connection is being used in multiple threads | 20:37 |
| *** bnemec is now known as beekneemech | 20:37 | |
| zzzeek | jbernard: uh well SQLAclhemy has a log level “DEBUG” for the “sqlalchemy.engine” logger, can you turn that on ? | 20:37 |
| jbernard | zzzeek: sure, brb | 20:37 |
| zzzeek | let me see if i can run a cinder locally here, my acutally-running-openstack-apps fu is very low | 20:38 |
| jbernard | zzzeek: if you can get devstack to work, that *should* be sufficient | 20:38 |
| * eharney tries to think through whether the two places he noted above would be in the same thread... maybe, i think? | 20:38 | |
| zzzeek | eharney: does cinder use threads? | 20:38 |
| eharney | zzzeek: well, eventlet greenthreads | 20:39 |
| zzzeek | eharney: ah. not likely then | 20:39 |
| zzzeek | eharney: because you aren’t using the MySQLdb patching feature, i hope | 20:39 |
| eharney | zzzeek: don't know really, but doesn't ring a bell | 20:39 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Dell Storage Center: Add retries to API calls https://review.openstack.org/155479 | 20:40 |
| jbernard | zzzeek: where is the sqlalchemy.engine config? | 20:42 |
| jbernard | zzzeek: im not seeing anything related in /etc | 20:42 |
| zzzeek | jbernard: openstack has a DB conf that id imagine is in cinder.conf, but in this case, this is just logging config that can be set anywhere python logging is being configured | 20:43 |
| eharney | ah yeah, default_log_levels | 20:44 |
| eharney | add sqlalchemy=DEBUG to that list option, i think | 20:44 |
| jbernard | eharney: what does that list contain by default? | 20:46 |
| jbernard | eharney: is missing entirely from my config | 20:46 |
| *** scottda_ has joined #openstack-cinder | 20:46 | |
| eharney | jbernard: all this stuff: http://git.openstack.org/cgit/openstack/cinder/tree/cinder/openstack/common/log.py#n124 | 20:47 |
| jbernard | eharney: ahh, thanks | 20:47 |
| harlowja_ | vilobhmm was there meeting notes from your guys little meetup about the state stuff? | 20:47 |
| kaufer | jungleboyj: You handle updates from oslo? See my comment here: https://review.openstack.org/#/c/154673/2/cinder/volume/manager.py | 20:47 |
| vilobhmm | yes harlowja lemme update that I had this etherpad as the starting point https://etherpad.openstack.org/p/cinder-state-management lemme update the notes 5 min please | 20:49 |
| harlowja_ | vilobhmm u might want to send that out to the ML so that others can have feedback | 20:50 |
| harlowja_ | sound ok? | 20:50 |
| harlowja_ | more visilbity the better | 20:50 |
| vilobhmm | yup sounds good…actually that was one of the action item from the meeting | 20:50 |
| vilobhmm | i will | 20:50 |
| harlowja_ | k | 20:50 |
| harlowja_ | thx | 20:50 |
| *** rushil_ has joined #openstack-cinder | 20:50 | |
| *** Mandell has quit IRC | 20:53 | |
| *** rushil has quit IRC | 20:53 | |
| *** bswartz has quit IRC | 20:55 | |
| *** timcl has quit IRC | 20:56 | |
| jbernard | zzzeek: http://sprunge.us/EZhL | 20:57 |
| zzzeek | jbernard: that’s a MySQL bug | 20:57 |
| zzzeek | that’s not at all the same stack shown in the other bug report nor is it the same error | 20:57 |
| *** dustins has quit IRC | 20:57 | |
| jbernard | yeah, that is different from the others | 20:58 |
| zzzeek | jbernard: also, I dont see any SQL here. it seems like this has “sqlalchemy.pool” turned on. we want “sqlalchemy.engine” | 20:58 |
| *** scottda_ has quit IRC | 20:58 | |
| jbernard | normally i see NoColumnError | 20:58 |
| zzzeek | jbernard: this is a MySQL thing that happens particularly if result sets are getting conflated | 20:58 |
| muraiki | hi all. I'm trying to set up the cinder-volume-usage-audit script in icehouse in my crontab. the comments in the script say I should be able to select a time period such as hour or month, but it doesn't seem to actually accept those as a command line argument. I can specify datetimes for the start and end, but it'd be nice if I could just use "hour" and have it cron hourly | 20:59 |
| *** Mandell has joined #openstack-cinder | 20:59 | |
| *** timcl has joined #openstack-cinder | 21:00 | |
| jbernard | eharney: do you know how to specify engine instead of pool? | 21:00 |
| jbernard | eharney: also, postres you say? | 21:02 |
| vilobhmm | harlowja : updated link https://etherpad.openstack.org/p/cinder-state-management will send it out to the ML as well | 21:02 |
| eharney | jbernard: i'm wondering if oslo.db's connection_debug option would do this.. | 21:02 |
| eharney | maybe not though | 21:02 |
| jbernard | eharney: all hell has broken loose in my devstack now… :) | 21:03 |
| eharney | jbernard: try sqlalchemy.engine=WARN or sqlalchemy.engine.base.Engine=WARN | 21:04 |
| jbernard | eharney: in default_log_levels ? | 21:04 |
| eharney | jbernard: yeah | 21:04 |
| eharney | er | 21:05 |
| eharney | jbernard: DEBUG, not WARN | 21:05 |
| jungleboyj | kaufer: Yes. | 21:05 |
| jungleboyj | I may actually have a patch for that sitting on my laptop here. | 21:06 |
| jungleboyj | kaufer: Oh, that is newer than when I last did a sync. | 21:07 |
| jungleboyj | I can push up a patch to add that. | 21:08 |
| jungleboyj | If I don't get to it today I will do it this weekend. | 21:08 |
| kaufer | jungleboyj: excellent | 21:08 |
| kaufer | jungleboyj: Can you update my review with it whenever it is available? | 21:09 |
| vilobhmm | cinder cores ( thingee, jgriffith, hemna, jungleboy, xyang) : we (e0ne, dulek, vilobhmm) as in people working on cinder state management met today to discuss about how to combine the two appraoches and take the goodies out of it and propose a single approach that will help to resume workflows, validate and make it more fault tolerant…please have a look at https://etherpad.openstack.org/p/cinder-state-management fr more details | 21:09 |
| jbernard | eharney: right | 21:12 |
| jbernard | eharney: it works | 21:12 |
| jbernard | eharney: but now it's not failing | 21:12 |
| jbernard | eharney, zzzeek: ill keep trying with debug enabled | 21:13 |
| eharney | jbernard: maybe turn that off and turn on mysql server-side statement logging? would maybe keep it more likely to fail if it is a race | 21:13 |
| eharney | just guessing | 21:13 |
| zzzeek | jbernard: are you getting SQL statements in the log ? | 21:13 |
| jbernard | zzzeek: yes | 21:13 |
| zzzeek | jbernard / eharney : the second error makes it look a lot like either MySQLdb is screwing up or the connection is being mis-used in a concurrent way | 21:14 |
| zzzeek | the “this result has no rows” is typically a concurrency thing | 21:14 |
| *** rushil_ has quit IRC | 21:16 | |
| *** e0ne has joined #openstack-cinder | 21:17 | |
| *** abehl has quit IRC | 21:19 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Fix logging guideline violations in volume/api.py https://review.openstack.org/155894 | 21:19 |
| *** Yogi1 has quit IRC | 21:22 | |
| *** e0ne has quit IRC | 21:26 | |
| *** timcl has quit IRC | 21:28 | |
| openstackgerrit | Jon Bernard proposed openstack/cinder: Add support for generic volume migration https://review.openstack.org/135002 | 21:30 |
| *** scottda_ has joined #openstack-cinder | 21:34 | |
| *** hemna_ has quit IRC | 21:35 | |
| *** Apoorva_ has quit IRC | 21:37 | |
| *** e0ne has joined #openstack-cinder | 21:37 | |
| *** Apoorva has joined #openstack-cinder | 21:37 | |
| *** hemna_ has joined #openstack-cinder | 21:38 | |
| *** emagana has quit IRC | 21:39 | |
| *** emagana has joined #openstack-cinder | 21:39 | |
| *** david-lyle has quit IRC | 21:42 | |
| *** david-lyle has joined #openstack-cinder | 21:42 | |
| *** Apoorva has quit IRC | 21:42 | |
| *** rushil has joined #openstack-cinder | 21:43 | |
| *** Apoorva has joined #openstack-cinder | 21:43 | |
| *** _cjones_ has quit IRC | 21:43 | |
| *** rushil has quit IRC | 21:44 | |
| *** rushil has joined #openstack-cinder | 21:44 | |
| *** emagana has quit IRC | 21:44 | |
| *** leakypipes has quit IRC | 21:45 | |
| *** david-lyle has quit IRC | 21:47 | |
| *** bswartz has joined #openstack-cinder | 21:50 | |
| jgriffith | hmmm... gary-smith you around? | 21:50 |
| gary-smith | yes | 21:50 |
| jgriffith | gary-smith: hey there | 21:50 |
| gary-smith | jgriffith: hey | 21:50 |
| jgriffith | gary-smith: do you have a link to the original capabilities spec you guys worked on? | 21:50 |
| *** scottda_ has quit IRC | 21:50 | |
| gary-smith | jgriffith: one sec, lemme dig it out | 21:51 |
| hemna_ | https://review.openstack.org/#/c/131280/ | 21:51 |
| *** scottda_ has joined #openstack-cinder | 21:52 | |
| gary-smith | hemna_: that's for validating cinder specs | 21:52 |
| gary-smith | jgriffith: the original one for retrieving them is https://github.com/openstack/cinder-specs/blob/master/specs/kilo/get-vol-type-extra-specs.rst | 21:52 |
| hemna_ | oops yah sorry | 21:52 |
| *** _cjones_ has joined #openstack-cinder | 21:52 | |
| smcginnis | https://www.youtube.com/watch?v=EYlDbv7MqE8 | 21:53 |
| gary-smith | jgriffith: https://review.openstack.org/127646/ in gerrit, fwiw | 21:54 |
| *** e0ne has quit IRC | 21:54 | |
| jgriffith | gary-smith: cool... thanks | 21:56 |
| gary-smith | jgriffith: sure. enjoy! | 21:56 |
| jgriffith | gary-smith: hehe | 21:56 |
| jgriffith | so this topic is getting kinda crazy | 21:56 |
| gary-smith | jgriffith: +1 | 21:56 |
| jgriffith | gary-smith: part of the trouble is that it's being made wayyy harder than it needs to be I think; Also there seems to be some confusion on how the capabilities filter works | 21:57 |
| jgriffith | it already does most of what everybody wants, the only piece is coming up with a data representation that shows all the available options on a device | 21:58 |
| *** emagana has joined #openstack-cinder | 21:58 | |
| gary-smith | yup, that's all we were after | 21:58 |
| jungleboyj | smcginnis: Nice. | 21:59 |
| gary-smith | just a way of describing extra specs so an admin could have some guidance directly from the drivers | 21:59 |
| jgriffith | gary-smith: yeah | 21:59 |
| *** Apoorva_ has joined #openstack-cinder | 22:01 | |
| smcginnis | jungleboyj: Sorry, first thing that came to my mind when gary-smith said lemme dig it out. :) | 22:01 |
| smcginnis | jungleboyj: What can I say, it's Friday... | 22:01 |
| *** thingee has joined #openstack-cinder | 22:02 | |
| jgriffith | smcginnis: HAAAAA OMG that's hillarious! | 22:02 |
| jungleboyj | smcginnis: Yes, it is. | 22:02 |
| jgriffith | smcginnis: good ol Blazing Saddles | 22:02 |
| smcginnis | jgriffith: A classic | 22:02 |
| *** mriedem1 has quit IRC | 22:02 | |
| jgriffith | awww.. prairie sh*& | 22:02 |
| *** mriedem has joined #openstack-cinder | 22:02 | |
| smcginnis | :) | 22:03 |
| jgriffith | LOL... oh dear, I had forggoten how wrong an hillarious this movie was | 22:03 |
| smcginnis | Mel Brooks is good for that. Started to watch Spaceballs with my kids. Didn't get too far before I rememeber it wasn't really a good kids movie. | 22:03 |
| jungleboyj | jgriffith: I was just watching some of the other quotes. Need to go watch it again. | 22:03 |
| gary-smith | smcginnis: ROFL! | 22:04 |
| jungleboyj | smcginnis: What? My kids loved it. | 22:04 |
| jungleboyj | Didn't think there was anything too bad in there. | 22:04 |
| smcginnis | jungleboyj: I have girls, so probably a little different. ;) | 22:04 |
| jgriffith | smcginnis: my favorite line though is still good ol Clint: http://goo.gl/t4No6s | 22:04 |
| jungleboyj | smcginnis: Ah, fair enough. | 22:04 |
| *** Apoorva has quit IRC | 22:04 | |
| jgriffith | alright.. stop.. I was working!!! | 22:05 |
| *** ronis_ has quit IRC | 22:05 | |
| smcginnis | Glad I could derail productivity for the afternoon! | 22:05 |
| *** muraiki has left #openstack-cinder | 22:05 | |
| smcginnis | jgriffith: You know you're going to look awfully silly... | 22:05 |
| *** lpetrut has quit IRC | 22:06 | |
| thingee | jgriffith: thanks for the example | 22:07 |
| jgriffith | thingee: oh.. hey | 22:09 |
| jgriffith | sure | 22:09 |
| *** kaufer has quit IRC | 22:09 | |
| anish | Should I file a bug if check-grenade-dsvm fails with ERROR: the main setup script run by this job failed - exit code: 1 | 22:10 |
| anish | (recheck hasn't finished yet) | 22:10 |
| jgriffith | anish: nope | 22:10 |
| jgriffith | anish: there's likely already a bug for that | 22:10 |
| anish | so the bug pointed out by elastic recheck has a different error | 22:11 |
| anish | hence, asking | 22:11 |
| *** scottda_ has quit IRC | 22:11 | |
| jungleboyj | anish: I think that just means that a failure happened in something that was called by that script. | 22:12 |
| jungleboyj | anish: See that often and the real cause of the failure has to be found down in the logs. | 22:12 |
| anish | aah, gotcha | 22:13 |
| jungleboyj | anish: :-) | 22:14 |
| anish | aah, different tests in bug and failure, but same failing function in backtrace | 22:15 |
| *** markstur_ has quit IRC | 22:18 | |
| *** vilobhmm has quit IRC | 22:19 | |
| *** vilobhmm_ has joined #openstack-cinder | 22:19 | |
| *** scottda_ has joined #openstack-cinder | 22:21 | |
| *** jungleboyj has quit IRC | 22:22 | |
| *** harlowja_ is now known as harlowja_away | 22:24 | |
| *** harlowja_away is now known as harlowja_ | 22:28 | |
| *** thingee has quit IRC | 22:28 | |
| *** eharney has quit IRC | 22:32 | |
| *** Mandell has quit IRC | 22:34 | |
| cebruns | smcginnis: LOL - classics! "You still here...?" | 22:35 |
| *** emagana has quit IRC | 22:37 | |
| *** emagana has joined #openstack-cinder | 22:38 | |
| *** emagana has quit IRC | 22:42 | |
| anish | erm, so for lvm+iscsi target to work, the target service has to be started somehow | 22:46 |
| anish | unlike devstack, I do not see the relevant code in cinder | 22:46 |
| anish | is this left upto packaging, or is it perhaps in a separate repo ? | 22:47 |
| *** hodos has joined #openstack-cinder | 22:48 | |
| *** emagana has joined #openstack-cinder | 22:52 | |
| *** Mandell has joined #openstack-cinder | 22:53 | |
| *** scottda_ has quit IRC | 22:55 | |
| *** dannywilson has quit IRC | 22:56 | |
| *** sgotliv__ has joined #openstack-cinder | 22:56 | |
| *** patrickeast has quit IRC | 22:57 | |
| *** scottda_ has joined #openstack-cinder | 22:58 | |
| *** thingee has joined #openstack-cinder | 23:02 | |
| openstackgerrit | Mikhail Khodos proposed openstack/cinder: Fix Nexenta NFS driver mounts https://review.openstack.org/149380 | 23:05 |
| *** r-daneel has quit IRC | 23:06 | |
| *** jungleboyj has joined #openstack-cinder | 23:06 | |
| *** mriedem1 has joined #openstack-cinder | 23:07 | |
| thingee | jgriffith: Honestly I don't know if I can make progress on this spec. | 23:07 |
| thingee | jgriffith: I keep thinking I'm aligned with what you're thinking | 23:08 |
| thingee | jgriffith: but I have a much different idea of this I guess | 23:08 |
| *** junhongl has quit IRC | 23:08 | |
| jgriffith | thingee: gimmie a minute, I'll write up and example, see if it helps. | 23:08 |
| jgriffith | thingee: and then if you guys want I'll just leave to you guys | 23:08 |
| jgriffith | I don't want to stop or slow down something that I don't feel strongly about | 23:09 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support https://review.openstack.org/85847 | 23:09 |
| *** mriedem has quit IRC | 23:09 | |
| *** scottda_ has quit IRC | 23:10 | |
| *** tbarron has quit IRC | 23:10 | |
| *** scottda_ has joined #openstack-cinder | 23:11 | |
| *** primechuck has quit IRC | 23:12 | |
| anish | finally ! | 23:12 |
| anish | jgriffith: now rebased AND jenkins approved https://review.openstack.org/#/c/147758/ | 23:12 |
| *** thangp has quit IRC | 23:12 | |
| anish | thingee: I had to bring a noop function that you wanted dropped fyi, because abstract | 23:13 |
| thingee | anish: that didn't make any sense to me. I might be missing context | 23:13 |
| *** scottda__ has joined #openstack-cinder | 23:13 | |
| *** scottda_ has quit IRC | 23:14 | |
| anish | thingee: no I agree with your original suggestion (wrt my patch) but now it's an abstractmethod called through the common code | 23:15 |
| anish | hence required again | 23:15 |
| *** scottda__ has quit IRC | 23:16 | |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-series: Allow scheduling by disk https://review.openstack.org/155915 | 23:22 |
| *** sgotliv__ has quit IRC | 23:23 | |
| *** sgotliv has joined #openstack-cinder | 23:23 | |
| *** mriedem1 has quit IRC | 23:27 | |
| *** zigo has quit IRC | 23:27 | |
| *** IanGovett has quit IRC | 23:28 | |
| *** zigo has joined #openstack-cinder | 23:28 | |
| jgriffith | thingee: hemna_ gary-smith kmartin https://gist.github.com/j-griffith/fd27297d79e0bb2c3e3d | 23:28 |
| jgriffith | that's what I've been trying to describe. Gives the info, but it's lighter weight, a little more structured | 23:28 |
| *** mriedem has joined #openstack-cinder | 23:29 | |
| jgriffith | keep out all the extra garbage around descriptions, keys and values... just describe the capabilities and that's it | 23:29 |
| *** xyang has quit IRC | 23:29 | |
| jgriffith | leave it do docs to go into details | 23:29 |
| *** sigmavirus24 is now known as sigmavirus24_awa | 23:30 | |
| jgriffith | scottda: I'd like your feedback as well: https://gist.github.com/j-griffith/fd27297d79e0bb2c3e3d | 23:33 |
| hemna_ | so | 23:33 |
| openstackgerrit | xing-yang proposed openstack/cinder: Kilo Consistency Group API update https://review.openstack.org/144561 | 23:34 |
| hemna_ | the entire point of our original spec was to make creating volume types self contained and easy for admins, specifically for horizon users. | 23:34 |
| *** mriedem has quit IRC | 23:34 | |
| hemna_ | which includes name, description, types for each key | 23:34 |
| hemna_ | so that when someone is going through the new horizon wizard it's all there for what's running in cinder. | 23:34 |
| *** mriedem has joined #openstack-cinder | 23:35 | |
| thingee | hemna_: if I'm understand jgriffith correctly, we can still do that, but it would be up to the cinder-api to take this information and parse it and make it "pretty" or in json schema for horizon to consume | 23:35 |
| hemna_ | so where does description for vendor keys come from then? | 23:36 |
| hemna_ | john's gist has vendor keys, but no desc for them. | 23:36 |
| hemna_ | at no point were we thinking of modifying how the scheduler worked or looked at capabilities reported. | 23:36 |
| hemna_ | we simply wanted a way to give information to horizon users to create volume types. | 23:37 |
| hemna_ | john's gist is nice though | 23:38 |
| thingee | ideally it would come from the driver. I can't speak for jgriffith though | 23:38 |
| hemna_ | it's just lacking desc. | 23:38 |
| hemna_ | thingee, that's what he had planned in our original spec | 23:38 |
| thingee | hemna_: I get that. | 23:38 |
| hemna_ | the schema for the supported extra specs came directly from the driver itself. | 23:38 |
| thingee | hemna_: If I had to guess jgriffith doesn't really care about that part, so it was just neglected | 23:39 |
| *** mriedem has quit IRC | 23:39 | |
| hemna_ | heh | 23:39 |
| hemna_ | that's the only part we wanted. | 23:39 |
| hemna_ | :P | 23:39 |
| hemna_ | everything else just seems to be mass confusion and making things hard. | 23:39 |
| hemna_ | dunno | 23:39 |
| hemna_ | I'm sorry if we've made this so difficult | 23:40 |
| hemna_ | I think a few beers are in order :) | 23:40 |
| thingee | I think sleep is in order for me and cancelling any traveling | 23:41 |
| *** mriedem has joined #openstack-cinder | 23:41 | |
| *** MasterPiece has quit IRC | 23:42 | |
| *** hemna_ is now known as hemnafk | 23:42 | |
| *** superdan has quit IRC | 23:44 | |
| *** dansmith has joined #openstack-cinder | 23:45 | |
| jgriffith | hemna: did you read what I wrote? | 23:51 |
| jgriffith | hemna: just FYI, I'll -2 any spec or patch that includes description for every possible key | 23:51 |
| jgriffith | and BTW, all of this info is expected to come from the driver | 23:52 |
| thingee | jgriffith: is this purely because of the MQ data being passed around? | 23:52 |
| jgriffith | MQ data? | 23:52 |
| jgriffith | ohh | 23:52 |
| thingee | message queue limit issues that DuncanT raised | 23:52 |
| *** hodos has quit IRC | 23:53 | |
| jgriffith | thingee: no it's because I think it's crazy to include custom documentation in ever driver for every option | 23:53 |
| jgriffith | thingee: and the problem statement was "expose valid keys" | 23:53 |
| jgriffith | thingee: the DS I propose does that | 23:53 |
| jgriffith | but I guess it's not what they want, so ok | 23:53 |
| jgriffith | I'll leave it up to the rest of you | 23:53 |
| thingee | I guess I envisioned the driver asking the backend itself. I think that's a wonderful thing for the driver to ask the backend, what are you capable of doing, set a standard way to parse it and pass it back up. | 23:53 |
| jgriffith | thingee: that's exactly what i just posted FWIW | 23:54 |
| *** markvoelker has quit IRC | 23:54 | |
| jgriffith | thingee: meaning that's exactly where that data comes from | 23:54 |
| jgriffith | thingee: where the data structure is built and returned | 23:54 |
| *** markvoelker has joined #openstack-cinder | 23:55 | |
| jgriffith | seems to be a lot of confusion on this whole thing | 23:55 |
| thingee | yeah but mine had descriptions and all that. It was a way for the administrator to deploy a backend, have cinder setup to talk to it, then ask for the current capabilities. Create extra specs based on that information. Horizon could take advantage of it so it doesn't have to be done in the cli or rest calls. | 23:55 |
| jgriffith | seems pretty simple and straight forward to me | 23:55 |
| jgriffith | thingee: rigth, but that's what makes things an absolute mess IMO | 23:55 |
| jgriffith | thingee: it's not Cinder's job to document vendor features in it's code IMO | 23:55 |
| jgriffith | thingee: I'm fine with "helping" admins with Cinder options and making sure they know what's available | 23:56 |
| thingee | cinder is not documenting it though. Cinder gets a request via cinder-api to ask the backend. Cinder just gets back the description and all that data from the backend. It's not hardcoded in the driver. | 23:56 |
| thingee | it's just a pass through | 23:56 |
| jgriffith | but I'm not using Cinder as an HP "command help" vehicle | 23:56 |
| jgriffith | or a a documentation over RPC method | 23:56 |
| jgriffith | thingee: really? This is just weird to me | 23:57 |
| jgriffith | thingee: I'm really surprised there's even any push back here to be honest | 23:57 |
| thingee | Maybe users are crazy then. When I talk to people, they don't want to go through the vendor's cli or dashboard. They want to just talk to cinder or go through horizon. | 23:57 |
| jgriffith | thingee: hemna so for example, why don't you go propose a patch to Nova that exposes ALL KVM options and their description | 23:58 |
| jgriffith | see how that goes over | 23:58 |
| jgriffith | thingee: wait... you're not talking about the same thing are you? | 23:58 |
| *** hodos|2 has joined #openstack-cinder | 23:58 | |
| jgriffith | thingee: you're saying you're being told that they want to see documentation for driver features in the code and in the dashboard? | 23:58 |
| thingee | I never said documentation in the code. | 23:59 |
| *** markvoelker has quit IRC | 23:59 | |
| jgriffith | thingee: let's slow down a sec and back up maybe.... | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!