Wednesday, 2015-03-04

*** kaufer has quit IRC00:01
*** annashen has joined #openstack-cinder00:03
*** ebalduf has joined #openstack-cinder00:04
*** ho has joined #openstack-cinder00:04
openstackgerritJay Bryant proposed openstack/cinder: Namespace updates for _i18n and imageutils mods
openstackgerritPatrick East proposed openstack/cinder: Add CHAP support to PureISCSIDriver
openstackgerritPatrick East proposed openstack/cinder: Add DB table for driver specific data
*** hemna is now known as hemnafk00:06
*** ebalduf has quit IRC00:08
*** akerr is now known as akerr_away00:13
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Remove unused options: iscsi_num_targets, iser_num_targets
mtaninoanish: ^^00:14
*** akerr_away is now known as akerr00:16
*** patrickeast_ has joined #openstack-cinder00:19
*** Yogi1 has joined #openstack-cinder00:19
*** patrickeast__ has joined #openstack-cinder00:20
*** patri728 has joined #openstack-cinder00:20
*** patri586 has joined #openstack-cinder00:20
anishmtanino: yep, looking at it now00:25
mtaninoanish: thanks. I think that method is deadcode in your target.00:25
*** markvoelker has quit IRC00:26
anishmtanino: kinda. our limit is 102400:27
*** lpabon has joined #openstack-cinder00:28
*** lpabon has quit IRC00:28
*** akerr has quit IRC00:28
mtaninoanish: _ensure_iscsi_targets in your target is never called. so I just removed it in this patch along with removing two config options.00:32
anishyes. I implemented it based on tgt code, which may not have been the correct approach00:36
*** annashen has quit IRC00:38
anishmtanino: I added the iser guys as well for some feedback00:42
mtaninoanish: thank you.00:44
*** tsekiyam_ has joined #openstack-cinder00:46
*** mtanino has quit IRC00:50
*** tsekiyama has quit IRC00:50
*** tsekiyam_ has quit IRC00:51
*** topshare has joined #openstack-cinder00:53
*** Yogi1 has quit IRC00:57
*** topshare_ has joined #openstack-cinder00:57
*** topshare has quit IRC01:00
*** tbarron has joined #openstack-cinder01:00
*** Mandell has quit IRC01:04
*** Mandell has joined #openstack-cinder01:05
*** ebalduf has joined #openstack-cinder01:05
openstackgerritThang Pham proposed openstack/cinder: Switch to oslo_versionedobjects
anishjungleboyj: replied re
*** annegentle has quit IRC01:08
*** markvoelker has joined #openstack-cinder01:09
*** tbarron has quit IRC01:09
*** ebalduf has quit IRC01:09
*** Mandell has quit IRC01:09
*** tbarron has joined #openstack-cinder01:10
*** jaypipes has quit IRC01:11
jungleboyjanish: Ok, can you update the commit message?01:12
*** annegentle has joined #openstack-cinder01:12
*** markvoelker has quit IRC01:14
*** kmartin has quit IRC01:15
anishjungleboyj: I can but I'll wait for mtanino to do it since it's his patchset01:15
*** dannywilson has quit IRC01:16
jungleboyjOk.  Sounds good.  Thanks.01:16
*** Mandell has joined #openstack-cinder01:16
*** davechen has joined #openstack-cinder01:17
*** Mandell has quit IRC01:17
*** Mandell has joined #openstack-cinder01:19
*** annegentle has quit IRC01:19
*** IanGovett1 has joined #openstack-cinder01:20
*** IanGovett has quit IRC01:21
*** Apoorva has quit IRC01:24
*** Apoorva has joined #openstack-cinder01:24
*** scottda_ has joined #openstack-cinder01:29
*** _cjones_ has quit IRC01:29
*** _cjones_ has joined #openstack-cinder01:32
*** scottda_ has quit IRC01:32
*** scottda_ has joined #openstack-cinder01:36
*** Mandell has quit IRC01:37
*** gugl3 has quit IRC01:38
*** cebruns has joined #openstack-cinder01:38
*** cdelatte has joined #openstack-cinder01:40
*** rcallawa has quit IRC01:41
*** scottda_ has quit IRC01:43
*** pradip_vedams has quit IRC01:47
*** pradip_vedams has joined #openstack-cinder01:47
*** vilobhmm has quit IRC01:48
openstackgerritharsh mishra proposed openstack/cinder: Fix for  inconsistent cinder-services state change
*** _cjones_ has quit IRC01:49
*** longboard has joined #openstack-cinder01:52
*** IanGovett1 has quit IRC01:55
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Remove unused options: iscsi_num_targets, iser_num_targets
*** Mandell has joined #openstack-cinder01:58
*** jamielennox is now known as jamielennox|away01:59
*** leeantho has quit IRC01:59
*** Tross has quit IRC02:07
*** markvoelker has joined #openstack-cinder02:10
*** markvoelker has quit IRC02:15
*** patrickeast has quit IRC02:19
*** patrickeast__ has quit IRC02:20
*** patri728 has quit IRC02:20
*** patri586 has quit IRC02:20
*** patrickeast_ has quit IRC02:20
*** annegentle has joined #openstack-cinder02:20
*** annegentle has quit IRC02:25
*** markvoelker has joined #openstack-cinder02:27
*** Mandell has quit IRC02:29
*** markvoelker has quit IRC02:33
*** jamielennox|away is now known as jamielennox02:37
*** kaisers has joined #openstack-cinder02:40
*** kaisers1 has quit IRC02:42
*** longboard has quit IRC02:43
*** Longgeek has joined #openstack-cinder02:47
openstackgerritwanghao proposed openstack/cinder: Get volume from db again after driver upload image
*** jamielennox is now known as jamielennox|away02:52
*** anish has quit IRC02:53
*** jamielennox|away is now known as jamielennox02:55
openstackgerritJeegn Chen proposed openstack/cinder: More error handling on VNX migration failure.
*** cdelatte has quit IRC02:56
*** jamielennox is now known as jamielennox|away02:56
*** jamielennox|away is now known as jamielennox02:57
*** annegentle has joined #openstack-cinder03:05
*** scottda_ has joined #openstack-cinder03:06
*** annegentle has quit IRC03:07
*** jcru has joined #openstack-cinder03:07
*** scottda_ has quit IRC03:09
*** BharatK has joined #openstack-cinder03:09
openstackgerritDave Chen proposed openstack/python-cinderclient: Add commands for modifying image metadata
*** Apoorva has quit IRC03:12
openstackgerritDave Chen proposed openstack/python-cinderclient: Add commands for modifying image metadata
*** scottda_ has joined #openstack-cinder03:17
openstackgerritThang Pham proposed openstack/cinder: Switch get_all_snapshots to use objects
openstackgerritThang Pham proposed openstack/cinder: Switch get_all_snapshots to use objects
*** harlowja_ is now known as harlowja_away03:21
*** ebalduf has joined #openstack-cinder03:27
*** markvoelker has joined #openstack-cinder03:28
*** patrickeast has joined #openstack-cinder03:31
*** patrickeast has quit IRC03:33
*** markvoelker has quit IRC03:33
*** scottda_ has quit IRC03:38
openstackgerritThang Pham proposed openstack/cinder: Switch get_all_snapshots to use objects
openstackgerritSasikanth Eda proposed openstack/cinder: Change default value of gpfs_images_share_mode to None
*** changbl has joined #openstack-cinder03:48
*** tbarron has quit IRC03:53
*** vilobhmm has joined #openstack-cinder03:57
openstackgerritxing-yang proposed openstack/python-cinderclient: Create Consistency Group from CG Snapshot CLI
*** haomaiwang has joined #openstack-cinder04:04
*** ho__ has joined #openstack-cinder04:06
*** ho has quit IRC04:06
openstackgerritJohn Griffith proposed openstack/cinder: Make objects behave more like our old dictionaries
*** rushil has joined #openstack-cinder04:12
*** dannywilson has joined #openstack-cinder04:13
*** pradipta has joined #openstack-cinder04:14
*** dannywilson has quit IRC04:17
*** rushil has quit IRC04:20
*** adam_g has quit IRC04:20
*** _cjones_ has joined #openstack-cinder04:20
*** _cjones_ has quit IRC04:22
*** kaufer has joined #openstack-cinder04:24
*** guitarzan has quit IRC04:24
*** kaufer has quit IRC04:26
*** kaufer has joined #openstack-cinder04:26
*** adam_g has joined #openstack-cinder04:28
*** markvoelker has joined #openstack-cinder04:29
*** mgagne has quit IRC04:31
*** ho has joined #openstack-cinder04:34
*** markvoelker has quit IRC04:34
*** ho__ has quit IRC04:35
openstackgerritharsh mishra proposed openstack/cinder: Fix for  inconsistent cinder-services state change
*** vilobhmm has quit IRC04:44
*** mgagne has joined #openstack-cinder04:45
*** mgagne is now known as Guest7571104:45
*** anish has joined #openstack-cinder04:45
*** openstackgerrit has quit IRC04:46
*** _cjones_ has joined #openstack-cinder04:50
*** anuragpalsule has joined #openstack-cinder04:50
*** openstackgerrit has joined #openstack-cinder04:53
*** _cjones_ has quit IRC04:54
*** ebalduf has quit IRC04:57
*** bkopilov has joined #openstack-cinder05:05
*** vilobhmm has joined #openstack-cinder05:11
*** vilobhmm1 has joined #openstack-cinder05:16
openstackgerritHuang Zhiteng proposed openstack/cinder: Generic Volume Migration Support - Proof-of-Concept (DO NO MERGE)
*** vilobhmm has quit IRC05:19
*** xyang has joined #openstack-cinder05:20
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add multiattach capabilities injection
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support
*** Longgeek has quit IRC05:28
*** BharatK has quit IRC05:28
*** markvoelker has joined #openstack-cinder05:31
winston-djbernard: ping05:32
*** jamielennox is now known as jamielennox|away05:32
winston-djbernard: FYI, I leave some comment in your generic volume migration support change, and put up one example here:  FYI.05:33
winston-djbernard: sorry it took me so long to finally get a time to do this.  hopefully it helps a bit.05:34
*** openstack has joined #openstack-cinder05:36
*** esker has joined #openstack-cinder05:40
*** esker has quit IRC05:41
*** BharatK has joined #openstack-cinder05:45
*** jcru has quit IRC05:47
openstackgerrityatin proposed openstack/cinder: Added more options while uploading volume to glance
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex
*** vilobhmm1 has quit IRC06:07
*** Longgeek has joined #openstack-cinder06:08
nikesh_vedamswant to know some information on manage/unmanage feature06:09
nikesh_vedamsis there anyone online06:09
nikesh_vedamswho knows manage/unmanage06:09
*** eharney has quit IRC06:10
*** eharney has joined #openstack-cinder06:10
anishyou could proly take a look at
*** lpetrut has joined #openstack-cinder06:18
*** yamada-h has quit IRC06:21
*** yamada-h has joined #openstack-cinder06:23
*** topshare_ has quit IRC06:28
*** markvoelker has joined #openstack-cinder06:32
*** topshare has joined #openstack-cinder06:33
*** topshare_ has joined #openstack-cinder06:35
*** topshare has quit IRC06:38
*** markvoelker has quit IRC06:38
*** nkrinner has joined #openstack-cinder06:41
*** xyang has quit IRC06:48
*** kaufer has quit IRC06:50
*** nlevinki has joined #openstack-cinder06:50
*** deepakcs has joined #openstack-cinder07:05
openstackgerritDave Chen proposed openstack/cinder: Role based properties protection
openstackgerritDave Chen proposed openstack/cinder: Policies based properties protection
*** krishna_vedams has joined #openstack-cinder07:15
*** thrawn01 has quit IRC07:17
*** aniket has joined #openstack-cinder07:19
*** eikke has quit IRC07:19
*** dulek has joined #openstack-cinder07:21
*** Longgeek has quit IRC07:21
openstackgerritrajiv proposed openstack/python-cinderclient: cinderclient accepts arguments after metadata without -- separator
*** david-lyle_afk has joined #openstack-cinder07:33
*** Longgeek has joined #openstack-cinder07:34
*** markvoelker has joined #openstack-cinder07:34
*** TobiasE has joined #openstack-cinder07:35
*** topshare_ has quit IRC07:39
*** markvoelker has quit IRC07:39
*** topshare has joined #openstack-cinder07:43
*** sgotliv has quit IRC07:43
*** Ilja has joined #openstack-cinder07:46
DuncanTnikesh_vedams: What info do you want? and are you keeping good notes of what you're learning to improve our docs?07:47
*** afazekas has quit IRC07:48
openstackgerritMarc Koderer proposed openstack/cinder: Adapt volume manager to new driver structure
*** openstackgerrit has quit IRC07:49
*** openstackgerrit has joined #openstack-cinder07:50
*** anshul has joined #openstack-cinder07:50
*** anshul has quit IRC07:50
*** anshul has joined #openstack-cinder07:51
openstackgerritVincent Hou proposed openstack/cinder: A patch for test. Disregard it.
openstackgerritVincent Hou proposed openstack/cinder: A patch for test. Disregard it.
*** ronis has joined #openstack-cinder08:18
*** chlong has quit IRC08:20
*** e0ne has joined #openstack-cinder08:20
*** topshare has quit IRC08:29
-openstackstatus- NOTICE: Zuul check queue stuck due to reboot maintenance window at one of our cloud providers - no need to recheck changes at the moment, they won't move forward.08:29
*** ChanServ changes topic to "Zuul check queue stuck due to reboot maintenance window at one of our cloud providers - no need to recheck changes at the moment, they won't move forward."08:29
*** topshare has joined #openstack-cinder08:30
openstackgerritVincent Hou proposed openstack/cinder: A patch for test. Disregard it.
*** markvoelker has joined #openstack-cinder08:35
*** Miouge has joined #openstack-cinder08:38
*** pradipta has quit IRC08:39
*** markvoelker has quit IRC08:39
krishna_vedamsRecenlty we did a commit for blueprint "Add iSCSI SCST Target support to cinder" and merged08:41
*** _afezekas|pub has joined #openstack-cinder08:42
*** e0ne has quit IRC08:42
*** alecv has joined #openstack-cinder08:42
*** karimb has joined #openstack-cinder08:42
krishna_vedamsAfter code merge happen, is there any thing remaining to do ?08:42
krishna_vedamscan anybody please suggest me?08:43
*** lpetrut has quit IRC08:43
openstackgerritVincent Hou proposed openstack/cinder: A patch for test. Disregard it.
*** jordanP has joined #openstack-cinder08:48
*** tshefi has quit IRC08:50
*** jistr has joined #openstack-cinder08:50
nikesh_vedamsDuncanT: yes i have notes08:52
nikesh_vedamsDuncanT: my doubt about manage/unmanage is clear from this08:53
DuncanTnikesh_vedams: Improvements to the cinder wiki pages for developers are very welcome. These are hard to start for those of us who've been working on the code for years, since we don't know what info newcomers want.08:53
DuncanTnikesh_vedams: Looking at the patch now08:53
DuncanTnikesh_vedams: Ok, that patch adds a horizon interface to the manage/unmanage functionality that has been in cinder for some time. What are you unsure about? Unmanage causes cinder to forget about a volume, while leaving it on the backend storage. Manage imports a reference to an existing volume into cinder, where it can then be used with the full range of08:56
DuncanTcinder commands - clone, snap migrate, etc08:56
*** kaisers1 has joined #openstack-cinder08:58
*** ho has quit IRC08:58
*** kaisers has quit IRC08:58
nikesh_vedamsDuncanT: after seeing this patch message its clear now,earlier before asking question i havenot seen this patch08:59
DuncanTnikesh_vedams: Ah, I see. Thanks. If anything useful can be added to the wiki that would have made figuring out each of the driver functions easier, please start the page and fill in the details you have. Myself and others will be glad to add more detail09:00
nikesh_vedamsDuncanT: yes i will start,what is the procedure to change wiki09:00
DuncanTnikesh_vedams: Either find a page that starts to cover what you want and edit it to add more, or start a new page and link to it off the main cinder pages09:01
DuncanTnikesh_vedams: Anybody can edit, it keeps the history so we can tidy up mistakes09:01
*** nibalizer has quit IRC09:02
DuncanTWiki is down at the moment due to a maintance window at rackspace09:05
DuncanTShould be back shorty09:05
openstackgerritVincent Hou proposed openstack/cinder: Test.
*** openstack has joined #openstack-cinder15:28 [freenode-info] channel trolls and no channel staff around to help? please check with freenode support:
*** scottda has joined #openstack-cinder15:33
*** scottda has quit IRC15:34
*** nlevinki has quit IRC15:36
*** kazum has joined #openstack-cinder15:39
*** kazum has quit IRC15:40
*** scottda has joined #openstack-cinder15:40
*** kazum has joined #openstack-cinder15:41
*** teruaki has joined #openstack-cinder15:41
*** rhedlind has quit IRC15:42
*** hemnafk is now known as hemna15:44
*** pradipm has quit IRC15:44
*** frankm has joined #openstack-cinder15:46
*** dulek has joined #openstack-cinder15:49
*** timcl has quit IRC15:49
*** dustins_ has quit IRC15:50
*** tbarron has quit IRC15:52
*** tbarron has joined #openstack-cinder15:52
*** esker has quit IRC15:55
*** tbarron has quit IRC15:55
*** tbarron has joined #openstack-cinder15:56
*** deepakcs has quit IRC15:56
*** bswartz has joined #openstack-cinder15:57
*** dustins has joined #openstack-cinder15:57
*** kmartin has joined #openstack-cinder15:58
*** tbarron has quit IRC15:58
*** tbarron has joined #openstack-cinder15:58
*** eikke has joined #openstack-cinder16:02
*** Bsony has quit IRC16:03
*** dustins has quit IRC16:04
*** tsekiyama has joined #openstack-cinder16:05
*** jungleboyj has joined #openstack-cinder16:12
j_kinganyone mind if I take ?16:15
openstackLaunchpad bug 1298135 in Cinder "Cinder should handle token expiration for long ops" [Medium,Confirmed]16:15
openstackgerritGorka Eguileor proposed openstack/cinder: Make lio iSCSI changes persistent to avoid lost
*** timcl has joined #openstack-cinder16:18
e0nej_king: i think, you could just to assign this issue you yourself16:18
*** annegentle has quit IRC16:21
*** annegentle has joined #openstack-cinder16:22
*** asselin__ has joined #openstack-cinder16:25
*** jaypipes has joined #openstack-cinder16:26
*** Ilja has quit IRC16:26
*** asselin_ has quit IRC16:29
openstackgerritPetrut Lucian proposed openstack/cinder: SMBFS: Add minimum qemu-img version requirement
*** dulek has quit IRC16:32
*** Ilja has joined #openstack-cinder16:37
*** Redcavalier has joined #openstack-cinder16:38
RedcavalierHi, I was suggested to come here, regarding this bug :
openstackLaunchpad bug 1348811 in Cinder "RBD + Nova migration fails because imports an existing disk" [High,Confirmed]16:40
RedcavalierBasically, this is very critical for us to have fixed before we send our setup into production. I was wondering if there was any internal patch, anything that was not yet sent for code review?16:41
*** jwcroppe has quit IRC16:41
*** jwcroppe has joined #openstack-cinder16:41
*** rwsu-afk is now known as rwsu16:42
RedcavalierI know that jbernard said he may have a look, so I'm wondering if there is any update on that.16:42
jbernardRedcavalier: give me a few mins, cinder meeting is right now16:43
*** julim has quit IRC16:44
*** julim has joined #openstack-cinder16:45
mtreinishdoes someone want to +A: it already has 3 +2s? Or am I missing something?16:45
*** jwcroppe has quit IRC16:46
*** _cjones_ has joined #openstack-cinder16:47
*** sgotliv has quit IRC16:51
*** Ilja has quit IRC16:51
*** Ilja has joined #openstack-cinder16:52
openstackgerritRajini Ram proposed openstack/cinder: Convert all eqlx tests from mox to mock
*** Yogi11 has quit IRC16:54
*** anshul has quit IRC16:54
*** asselin__ has quit IRC16:56
*** ronis__ has quit IRC16:56
*** Ilja1 has joined #openstack-cinder16:56
*** teruaki has left #openstack-cinder16:57
*** sgotliv has joined #openstack-cinder16:58
*** Ilja has quit IRC16:59
*** patrickeast has quit IRC16:59
jbernardmight it be better to raise volume migration at summit?17:00
jbernardRedcavalier: ok, hit me17:00
jbernardRedcavalier: so you're saying that just resizing an rbd-backed instance causes error17:01
jungleboyjjbernard: Yes, we will need to.17:01
jungleboyjwinston-d: What did you need to talk about?17:01
jbernardjungleboyj: ok, ill submit a proposal17:01
jbernardjungleboyj: thanks17:01
winston-dThis bug here:
openstackLaunchpad bug 1406703 in Cinder "Deleting VM with an attached volume during copy-volume-to-image causes the volume remains in-use state" [Undecided,In progress] - Assigned to Abhijeet Malawade (abhijeet-malawade)17:01
jungleboyjjbernard: Migration in general needs to be discussed.17:01
Redcavalierjbernard, yup, because openstack tries to migrate it to another node.17:01
jbernardyeah, it's becoming clear that opinions are yet well formed17:02
hemnawinston-d, I wonder if that bug goes away if my multiattach patch lands17:02
hemnawinston-d, as we track individual attachments now17:02
winston-dDoes anyone feel the same as I do about Cinder shouldn't allow uploading a 'in-use' vol to image?17:02
jbernardRedcavalier: when you say migrate, you're referring to the instance, right?17:02
hemnawinston-d, +117:02
DuncanTwinston-d: Yes, yes I do17:02
jbernardwinston-d: +117:02
jbernardwinston-d: unless there's some usecase im unable to visualize17:03
hemnawinston-d, I'm not sure what the user expects in that case.  data will be out of sync in the image17:03
*** kazum has quit IRC17:03
Redcavalieryea, the instance moving from compute 1 to compute 2. It looks like openstack tries to move its block device as if it was epehmeral storage on the computes disks.17:03
winston-dhemna: yeah, maybe, but for those backends doesn't support multiattach, doing such will corrupt data.17:03
DuncanTwinston-d: That's a clear bug, same with backup - a slow copy of a live block device will almost always break stuff17:03
hemnawinston-d, yah17:03
eharneywinston-d: the driver could create a temporary snapshot and upload it from that... doesn't seem unreasonable to do at some point17:03
*** dustins has joined #openstack-cinder17:03
*** dustins has quit IRC17:04
*** NarenNarendra has joined #openstack-cinder17:04
jbernardRedcavalier: ok, ive not tested that particular case17:04
DuncanTeharney: Backup of snapshot is being looked at as a first class explicit operation in L. We could use the same code to do upload-of-snap I think17:04
NarenNarendraHello Cinder team...17:04
jbernardRedcavalier: and the gate doesn't either, so that could have easily slipped through17:04
winston-dwait, so we are *OK* to allow upload-to-image for a 'in-use' vol, just want to make sure vol stats got to correct state like the bug mentioned?17:04
winston-dOr, we shouldn't do that util we have sth like upload-of-snap?17:05
*** bswartz has quit IRC17:05
DuncanTwinston-d: We shouldn't allow upload of in-use, in my oppinion17:05
*** tbarron has quit IRC17:06
winston-dDuncanT: +217:06
*** tbarron has joined #openstack-cinder17:06
DuncanTwinston-d: Implement upload-snap in L, explicit is better than implicit and the implicit snap is really easy to mess up in the driver17:06
winston-dDuncanT: agree with that too.17:06
jbernardRedcavalier: in general, shared storage is a relatively new concept to nova, so i have no doubt there are still some bugs ther17:06
Redcavalierjbernard, ok. For us it'S really important to be able to migrate/resize VMs before we move into production, so we're a bit stressed to have it working as soon as possible.17:07
jbernardRedcavalier: what you describe is technically a nova bug17:07
winston-dDuncanT, hemna, eharney, please chime in in bup report if you feel like it:
openstackLaunchpad bug 1406703 in Cinder "Deleting VM with an attached volume during copy-volume-to-image causes the volume remains in-use state" [Undecided,In progress] - Assigned to Abhijeet Malawade (abhijeet-malawade)17:07
hemnawinston-d, ok17:07
winston-dwe can open another bug if needed.17:07
winston-dhemna: thx17:07
aarefievjungleboyj: hi17:07
jbernardRedcavalier: but i can still try to reproduce and possilbly provide a fix if time allows17:07
*** lpetrut has quit IRC17:08
NarenNarendraHi Cinder team! Pardon my perhaps novice IRC etiquette here... I am working with the OpenStack Product WorkGroup. We had a Kilo midcycle meetup back in January. One of the immediate tasks that we have picked is to speak with each of the project teams/PTLs on data gathering. Questions to address are:17:08
NarenNarendra1. What are you delivering for Kilo?17:08
NarenNarendra2. What do you plan on delivering for Liberty cycle? (we understand confidence is lower with time).17:08
NarenNarendra3. What do you plan on delivering for M cycle? (we understand confidence is lower with time).17:08
NarenNarendra4. How can the product WG help?17:08
NarenNarendraWould one of you be kind to address these?17:08
davechenthingee: hi,17:09
davechenthingee: are you around?17:09
jungleboyjaarefiev: What is up?17:10
aarefievjungleboyj: do we have plans to switch to oslo.middleware in k17:10
jungleboyjThe library instead of the olso-incubator?17:10
*** lpetrut has joined #openstack-cinder17:11
winston-djbernard: hey, about the example here:  what's your concern?17:11
hemnaaarefiev, I hope not at this point.17:11
aarefievjungleboyj: e0ne asked me to make patch for grenade to update paste api17:11
jbernardwinston-d: honestly i like it17:11
jungleboyjaarefiev: We already have the patch out there for that?17:11
winston-djbernard: if you still time to look at it, i'm fine, we can discuss later17:12
jbernardwinston-d: i just want to make sure there's some agreement, so im not losing time17:12
jungleboyjhemna: That one is a smaller impact that the scheduler.17:12
winston-djbernard: i'll try to get avishay to look at it, and hopefully jgriffith17:12
hemnajungleboyj, url ?17:12
jungleboyjTrying to find it.17:12
aarefievjungleboyj: not yet17:12
*** e0ne is now known as e0ne_17:12
*** e0ne_ is now known as e0ne17:12
jbernardwinston-d: that would be great, thanks for that17:13
DuncanTNarenNarendra: One think I think we could do with guidance/help on is what features of ours are actually being used in the wild, rather than what vendors tell us. A recent post on the operators list suggests most operators don't understand a huge chunk of our features17:13
hemnajungleboyj, honestly, I'm a bit worried with lots of oslo_ migrations this late in the game.17:13
hemnajust due to past experience17:13
DuncanTNarenNarendra: That can go under question (4)17:13
*** jistr has quit IRC17:13
winston-dDuncanT: good point17:13
DuncanTNarenNarendra: I suspect the answer to (3) is going to be pretty much unknown, given past experience17:13
jungleboyjhemna: Agreed, but I believe we have kept middleware up to date.  This is just using the library version instead.17:14
hemnajungleboyj, it's not passing jenkins at the moment.17:14
hemnano warm and fuzzies17:14
jungleboyjhemna: Yes, that is what aarefiev is trying to fix.17:14
-openstackstatus- NOTICE: Issue solved, gate slowly digesting accumulated changes17:14
winston-djbernard: i'm pretty confident avishay is(will be) on board, I need to hear feedback from jgriffith17:14
DuncanTNarenNarendra: Thingee should be able to give you an answer to (1) and (2), if you haven't heard something soon I'll try to throw together a summary of what I know of17:14
*** alecv has quit IRC17:14
jungleboyjhemna: It is a grenade problem with package versions.17:15
jbernardwinston-d: kk, ill get to work17:15
hemnawinston-d, are you ok with xyang's feedback on the multiattach injection ?17:15
NarenNarendraDuncanT: thanks a bunch!17:15
*** davechen has quit IRC17:15
jbernardwinston-d: should i try to hit the k3 cutoff, or is that unrealisitic?17:15
hemnaI'd really like to get some other folks testing the multiattach patch to get it to land this week if possible.17:15
NarenNarendraDuncanT: pls try to answer 1 and 2 also17:15
jungleboyjhemna: I agree the scheduler change is dangerous, but this one I am less concerned about.17:16
NarenNarendraDuncanT: I have tried to reach out Thingee for the last couple of weeks over multiple channels... it looks like he may to too busy rounding up things for Kilo17:16
hemnathat one17:16
DuncanTNarenNarendra: Actually, it might be best to start an etherpad and let others chime in too.
winston-djbernard: i'll try my best to help. honestly i feel guilty about not providing enough help on this for past few months, it should be landed already if i had more time to look closer to it.17:17
jungleboyjaarefiev: So, if you can get the patch passing Jenkins that would be greatly apprecaited.17:17
winston-dhemna: not really, i would like to talk to her if possible.17:18
hemnawinston-d, ok17:18
winston-dxyang: around?17:18
DuncanTNarenNarendra: I'll try to fill in what I can over the next few hours (I need to drop offline for an hour or so momentarily), and I'll encourage others to chime in too17:18
hemnashe seemed it was necessary17:18
hemnaor desired17:18
NarenNarendraDuncanT: that will also work great! can you pls help me circulate that with the Cinder team? I can watch the etherpad for updates. Perhaps, give a deadline of one week from now for all updates?17:18
*** nellysmitt has quit IRC17:18
NarenNarendraDuncanT: thanks so much! I appreciate it a lot17:19
aarefievjungleboyj: I think there will the problem, patches will block each other17:19
winston-dhemna: the injection is only needed when a volume is created with 'multiattach' flag in vol metadata, while not in type extra specs, right.17:19
DuncanTNarenNarendra: That seems reasonable. A week and a day means we can bring it up at the next weekly meeting for any final details, plus we'll be after code feature freeze so the answers will be more reliable17:19
hemnathe multiattach flag is on the volume itself17:19
*** jwcroppe has joined #openstack-cinder17:20
aarefievjungleboyj: my fix and switch to oslo lib17:20
winston-dhemna: but that logic we are talking about is for creating a consistency group17:20
NarenNarendraDuncanT: thats fine.. until next Friday, 03/13 is also fine17:20
NarenNarendraDuncanT: you can bring this up in the next project mtg on next wed too17:20
winston-dhemna: yes, but the flag was persistent into cinder as volume metadata, right?17:20
DuncanTNarenNarendra: Excellent. Keep an eye on that etherpad17:20
NarenNarendraDuncanT: awesome awesome! thanks so much!17:20
jungleboyjaarefiev: Can you add it to ?17:20
hemnatechnically it's not in volume metadata, but part of the volume itself.17:21
jungleboyjSo that they don't block each other.17:21
winston-dhemna: oh? where do we save it to?17:21
hemnalike 4017:22
aarefievjungleboyj: how do that? there are in diff projects17:22
hemnaline 4017:22
hemnait's a column in the volume table17:22
DuncanTRight, I'm off for a while, back later. Leave a message after the tone and all that...17:22
winston-dhemna: ok, so a new column in vol table. that's fine.17:23
*** e0ne is now known as e0ne_17:23
winston-dhemna: but we don't have a new column for ConsistencyGroup table, right?17:23
hemnaso the _get_weighted_candidates_group is for scheduling/filtering on CG create ?17:23
jungleboyjaarefiev: Didn't understand that.17:23
hemnawinston-d, correct17:23
winston-dhemna: and ConsistencyGroup doesn't have metadata table.17:24
jungleboyjaarefiev: There must be a way to resolve this as others have dealt with this problem in Grenade before.17:24
aarefievjungleboyj: me too)17:24
winston-dhemna: so my point is, unless there is a way to tell Cinder, please create a CG that has a multiattach flag, we don't need that logic in scheduler.17:24
aarefievjungleboyj: I don't know such way17:24
*** jwcroppe_ has joined #openstack-cinder17:25
*** jwcroppe has quit IRC17:25
jungleboyjaarefiev: Have you tried asking on the openstack-infra channel?17:25
hemnaok I can remove it17:25
hemnaxyang,  ?17:25
aarefievjungleboyj: no17:25
jungleboyjaarefiev: I would try asking there.  I thought e0ne knew how to resolve this based on our last discussion.17:26
winston-dhemna: the only way to do that, now, is to create a CG with a type, that has 'multiattach' flag in extra specs. and that has already be taken care of by scheduler itself, no extra logic is needed.17:26
hemnaok yah that makes sense17:26
hemnaI'll put up another patch removing it.17:26
aarefievjungleboyj: thanks17:27
*** sgotliv has quit IRC17:27
winston-dhemna: sure17:27
*** sgotliv has joined #openstack-cinder17:27
winston-dxyang: hopefully you can see the discussion above and will be able to ack when you are available17:27
*** jwcroppe_ has quit IRC17:28
jungleboyjaarefiev: Thank you for looking into it.17:28
*** aarefiev has quit IRC17:28
*** jwcroppe has joined #openstack-cinder17:28
*** aarefiev has joined #openstack-cinder17:28
*** aarefiev has quit IRC17:29
nikesh_vedamswhat is unit of size returned by manage_existing_get_size17:29
*** Apoorva has joined #openstack-cinder17:29
*** aarefiev has joined #openstack-cinder17:29
xyangwinston-d: hi17:32
*** aarefiev has quit IRC17:32
*** aarefiev has joined #openstack-cinder17:32
xyangwinston-d: did you see my comments in the patch?  I think that is also needed for CG because scheduler needs to choose one backend that can serve all volume types17:32
winston-dxyang: yeah I saw that.17:33
winston-dxyang: if multiattach is part of type extra spec, we don't need such logic in scheduler.17:33
xyangwinston-d: you are saying it will take care of itself?17:34
xyangwinston-d: basically we want to be able to find a backend that support multiattach for CG as well17:34
winston-dxyang: your extra logic for consistency group is there only because one can request creating a CG with a type that doesn't have 'cg=supported' in extra spec17:35
xyangwinston-d: ok, that is true.  so why do we need extra logic for create volume with multiattach?17:36
winston-dxyang: now, the create CG request itself can be considered as the 'cg=supported' flag, so filter scheduler needs to inject a flag to extra spec if type definiton doesn't have it.17:36
winston-dxyang: for creating a CG with multiattach flag, how should the user specify multiattach flag for this CG, if not using type extra spec?17:38
*** narenhn has joined #openstack-cinder17:38
*** ndipanov has quit IRC17:38
xyangwinston-d: yes, should be same as create volume.  so if you want the volume created to support multiattach, you need to add that to extra spec17:38
*** leeantho has joined #openstack-cinder17:38
xyangwinston-d: that's why I'm confused why that is not needed in that code path17:39
winston-dxyang: if that's possible, then we need the same logic as cg=supported injected here. otherwise, scheduler takes care of type extra spec17:39
*** NarenNarendra has quit IRC17:40
winston-dxyang: no, in hemna's change, you can create a volume with multiattach flag *WITHOUT* usng a type that has multiattach=true in extra spec17:40
xyangwinston-d: so it seems to me if that logic is needed for create volume, then it should be needed for create CG17:40
xyangwinston-d: ok, maybe I'm confused with the need of that patch17:41
*** e0ne has joined #openstack-cinder17:41
*** e0ne has quit IRC17:41
winston-dxyang: question is, can i create a CG with multiattach flag *WITHOUT* multiattach=true in type extra spec?17:41
winston-dxyang: yes, or no?17:42
xyangwinston-d: I think you do have to pass that info in extra spec.17:43
xyangwinston-d: we don't have a flag to pass to CG directly17:44
winston-dxyang: then scheduler takes care of that already.17:44
*** afazekas has quit IRC17:44
winston-dxyang: do you remember why you add that extra logic for CG in filter scheduler?17:45
winston-dsame idea17:45
xyangwinston-d: yes, that is for the CG create command to succeed without that in extra specs17:45
winston-dxyang: exactly, CG create command itself present a need to inject the flag, without every type having to be updated.17:46
*** patrickeast has joined #openstack-cinder17:47
winston-dxyang: so you agree that hemna doesn't need that logic in get_weighted_cg_candidates() for multiattach?17:47
hemnaI have the patch ready to submit :P17:48
hemnajust give me the word17:48
xyangwinston-d: so for CG to pick a backend that supports multiattach, it requires multiattach to be in extra specs, I mean user has to define it17:48
winston-dxyang: yes17:49
*** akerr is now known as akerr_away17:51
nikesh_vedamsxyang : hi17:51
xyangnikesh_vedams: hi17:52
hemnaxyang, so are we good?17:53
hemnacan I submit my updated patchset ?17:53
*** anshul has joined #openstack-cinder17:53
nikesh_vedamsxyang: we are working on CI,but one merged code is breaking our scst merged patch we have reported bug in this patch
*** akerr_away is now known as akerr17:55
*** afazekas has joined #openstack-cinder17:57
*** patrickeast_ has joined #openstack-cinder17:58
*** patrickeast__ has joined #openstack-cinder17:58
*** patri251 has joined #openstack-cinder17:58
*** patri498 has joined #openstack-cinder17:58
hemnaxyang, ?17:59
*** pschaef has quit IRC18:00
*** erlon has joined #openstack-cinder18:00
xyanghemna: hi, I think it is fine.  looks like a doc item now18:01
hemnaok thanks, I'll push the review up18:01
*** patri461 has joined #openstack-cinder18:01
xyanghemna: so when you create a volume, you don't need that in extra specs, but when you create CG, you need it18:01
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add multiattach capabilities injection
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support
*** patri582 has joined #openstack-cinder18:01
nikesh_vedamsjgriffith: there?18:02
*** annashen has joined #openstack-cinder18:03
openstackgerritRajini Ram proposed openstack/cinder: Convert all eqlx tests from mox to mock
*** karimb has quit IRC18:04
*** winston-d is now known as winston-d_zZZ18:04
nikesh_vedamsxyang: thanks,is anyone else online for one more +2 :)18:05
*** Yogi11 has joined #openstack-cinder18:10
*** Mandell has joined #openstack-cinder18:14
*** rushil has joined #openstack-cinder18:15
*** leeantho has quit IRC18:17
*** narenhn has quit IRC18:19
*** NarenNarendra has joined #openstack-cinder18:19
*** bswartz has joined #openstack-cinder18:19
*** NarenNarendra has quit IRC18:19
*** harlowja_away is now known as harlowja_18:19
*** jordanP has quit IRC18:20
*** e0ne has joined #openstack-cinder18:22
*** vilobhmm has joined #openstack-cinder18:24
*** Yogi11 has quit IRC18:25
*** vilobhmm1 has joined #openstack-cinder18:25
*** leeantho has joined #openstack-cinder18:25
*** Yogi1 has joined #openstack-cinder18:26
*** leeantho has quit IRC18:26
*** diemt has joined #openstack-cinder18:26
nikesh_vedamsif i am correct,size of cinder volume is shown in integer GBs,so in manage_existing_get_size if size of volume is say 4.7 GB,what should we return18:26
*** leeantho has joined #openstack-cinder18:26
nikesh_vedams4 or 518:27
*** vilobhmm has quit IRC18:29
*** afazekas has quit IRC18:31
*** ronis__ has joined #openstack-cinder18:32
mgagnejgriffith: does cinder handle xExceededLimit exception from SolidFire? =)18:33
jbernardnikesh_vedams: 5 would be my guess18:33
mgagnejgriffith: my point is that there is no logs in cinder-volume about the lack of capacity. But I'm not sure if there should be any.18:33
*** thingee has joined #openstack-cinder18:36
thingeeleeantho: going to check out the docs today for
thingeeafter that will you remove the -1 workflow?18:40
leeanthothingee, yes once I get some feedback on the content I can remove the -1 workflow tag18:41
nikesh_vedamsthingee: morning18:43
jungleboyjthingee: When you have a chance, can you remove your -2 from here given that the Nova side went in:  Thanks.18:44
thingeejungleboyj: done18:47
jungleboyjthingee: Thank you!18:47
*** sgotliv has quit IRC18:48
jungleboyjhemna: Can you also take a look ^^18:48
hemnajungleboyj, ok I gave my feedback18:49
hemnathe system check ifs kinda suck18:50
hemnaI think it'd be better to have a way of automatically building a platform specific targeted connector18:50
jungleboyjhemna: Thanks for the feedback.18:50
hemnaso we don't start pooping if systemX, elseif systemY, etc etc18:50
jungleboyjhemna: Yeah, but that isn't something that is going to happen for Kilo.18:51
hemnaon the upside, I think everyone uses the factory to build the correct Connector18:51
hemnawe could modify the factory to build the right one for the right platform18:52
hemnabut at a minimum the os-brick patch needs to exist as well.18:52
jungleboyjhemna: Do you want it included in that same patch>?18:52
*** fischerw has joined #openstack-cinder18:53
hemnajungleboyj, and if you see any other reviews that touch cinder/brick/initiator, the same changes need to exist in os-brick as well.18:53
hemnajungleboyj, I don't think you can, because os-brick is a separate repository18:53
jungleboyjhemna: I realized that as I was typing that.18:53
jungleboyjhemna: So here:
*** alecv has joined #openstack-cinder18:54
jungleboyjOk.  Cool.18:54
*** xyang has quit IRC18:55
jungleboyjYou approve this one if we get the other one pushed up as well.18:55
*** xyang has joined #openstack-cinder18:55
*** dustins_ has joined #openstack-cinder18:57
*** dustins_ is now known as dustins18:58
hemnaI don't think it would be too hard to make a platform specific Connector object18:59
hemnaand just test for system.machine() in the factory18:59
hemnaI think that's a bit cleaner18:59
nikesh_vedamslooks like ietadm also missing in
hemnain fact I think the changes would be easy to do19:00
hemnatiny refactoring in the FibreChannelConnector to add a hook for a child class to extend19:01
*** alexpilotti_ has joined #openstack-cinder19:01
hemnain connect_volume and disconnect_volume()19:01
hemnaand a test in the factory to build the correct platform specific Connector19:01
jungleboyjhemna: Ok, if you think that is cleaner ... do you want to propose that in the review then?19:02
*** timcl1 has joined #openstack-cinder19:02
hemnaI was about to add comments about it in the review19:02
nikesh_vedamswhat should i use in help of iscsi_helper for ietadm for
hemnagive me a few19:02
jungleboyjhemna: Great, I will push them and try to get that turned around.19:03
*** alexpilotti has quit IRC19:03
*** alexpilotti_ is now known as alexpilotti19:03
*** bswartz has quit IRC19:05
*** e0ne is now known as e0ne_19:05
*** e0ne_ has quit IRC19:05
*** timcl has quit IRC19:05
*** rushil has quit IRC19:06
hemnaok hopefully that's clear.19:07
*** dannywilson has joined #openstack-cinder19:07
*** bswartz has joined #openstack-cinder19:09
*** dannywilson has quit IRC19:09
*** dannywilson has joined #openstack-cinder19:10
*** dannywilson has quit IRC19:10
*** dannywilson has joined #openstack-cinder19:11
jungleboyjhemna: Let me look.19:12
*** lpabon has joined #openstack-cinder19:15
*** ChanServ changes topic to "The OpenStack Block Storage Project Cinder | The New Kids On the Block |"19:15
*** e0ne has joined #openstack-cinder19:16
*** e0ne is now known as e0ne_19:16
mtaninonikesh_vedams: Please add this "ietadm for iSCSI Enterprise Target"19:16
*** Ilja1 has quit IRC19:18
*** Mandell has quit IRC19:19
jungleboyje0ne_: Did aarefiev get hold of you?19:19
*** e0ne_ is now known as e0ne19:20
e0nejungleboyj: not sure that i anderstood correctly your message19:21
openstackgerritBilly Olsen proposed openstack/cinder: Refactoring attach volume api to use taskflow
* e0ne needs to improve english19:21
jungleboyje0ne: aarefiev Was asking me about issues getting the patch for paste.api into Grenade to fix the oslo.messaging issue.19:22
e0nejungleboyj: yep. i read chat history19:22
jungleboyje0ne: He was concerned about how the two can be checked in when they are in different repositories but he thought they depended upon each oother.19:22
jungleboyje0ne: Ok.19:22
*** nkrinner has joined #openstack-cinder19:23
e0nejungleboyj: good question. unfortunately, i don't know the answer yet19:23
*** Bsony has joined #openstack-cinder19:24
jungleboyje0ne: Ok, I didn't either .  Referred him to the the infra channel.19:24
*** nkrinner has quit IRC19:24
*** nkrinner has joined #openstack-cinder19:24
*** BharatK has quit IRC19:24
*** aix has quit IRC19:24
e0nejungleboyj: agree. we defenatly need to ask infra team19:24
jungleboyje0ne: Ok, good, I sent him the right direction.19:26
openstackgerritPetrut Lucian proposed openstack/cinder: Fix fetch_to_volume_format if vhd is requested
openstackgerritPetrut Lucian proposed openstack/cinder: Move vhd/x resize logic to image utils
openstackgerritPetrut Lucian proposed openstack/cinder: Fix qemu-img info vhd format issue
openstackgerritPetrut Lucian proposed openstack/cinder: Add image subformat option to fetch_to_volume_format function
openstackgerritPetrut Lucian proposed openstack/cinder: Add support for caching images fetched from Glance
openstackgerritPetrut Lucian proposed openstack/cinder: SMBFS: Add image caching support
openstackgerritPetrut Lucian proposed openstack/cinder: Windows iSCSI: Add image caching support
*** angela-s has joined #openstack-cinder19:28
*** Mandell has joined #openstack-cinder19:28
jgriffithmgagne: it does not19:31
jgriffithmgagne: that *should* get proopogated if hit, but I'd have to look and verify19:32
mgagnejgriffith: I got a No valid host found error though19:32
jgriffithmgagne: yeah, that's what I'd expect19:32
jgriffithmgagne: I'd expect the sched to say "no" before it got to us19:33
jgriffithmgagne: but that's likely not right either, until something is done about effective etc19:33
*** lpetrut has quit IRC19:33
mgagnejgriffith: oh, is SF reporting volume capacity left?19:33
jgriffithmgagne: indeed19:33
e0nejungleboyj: i've got some converation in #openstack-infra19:33
jgriffithmgagne: free_capacity19:34
jungleboyje0ne: Ok.  Need me to join?19:34
e0nejungleboyj: no19:34
jungleboyje0ne: Ok.19:34
e0nejungleboyj: it's very easy19:34
mgagnejgriffith: this could explain why cinder-volume didn't log it as it's a "normal" behavior: no space left, go somewhere else.19:34
jungleboyje0ne: Sweet.19:34
jgriffithmgagne: exactly... "piss off" :)19:34
jgriffithmgagne: Dave's not here19:34
e0nejungleboyj, aarefiev: please, follow this guideline for your fix to grenade
*** ebalduf has quit IRC19:35
e0nejungleboyj: one more noob question from me19:37
jungleboyje0ne: Thanks for finding that.19:37
jungleboyje0ne: Noob?  You aren't a noob.  ;-)19:37
*** sgotliv has joined #openstack-cinder19:37
e0nejungleboyj: what is the right way to add a note for just edit a wiki page or we ve got some repo gor it?19:37
jungleboyje0ne: That is a good question.  I should know that but don't remember the answer.  jgriffith Should know.19:39
e0neneed to go sleep19:39
thingeejgriffith: I've asked you a few times about this now and feel like you're ignoring me. Can you please either remove your -2 or respond to this
e0nei failed and scheduled car service for 9am for tomorrow :(19:39
mgagnee0ne: on a side note, that's a problem if it's auto-generated from a repo and there is no note about that fact19:39
jungleboyje0ne: :-(  Hope it isn't major service.19:40
e0nejungleboyj: i just want to install fog lights. but it's tooo early for me19:40
jungleboyje0ne: Oh, good, fun service.19:41
jungleboyjthingee: Do you know how to add to the release notes?19:42
thingeejungleboyj: it's usually a wiki page. You just edit it19:42
e0nebb tomorrow. have a good day!19:42
jungleboyje0ne: Have a good night.19:42
* jungleboyj face palms19:43
jungleboyjYeah, we can edit that.  Thanks thingee19:43
jungleboyjNext I ask hemna how to configure the storwize driver.19:43
nikesh_vedamsmtanino: thanks19:44
openstackgerritnikeshmahalka proposed openstack/cinder: Choices for iscsi_helper does not contain scstadmin.
mtaninonikesh_vedams: sure.19:45
hemnajungleboyj, volume_driver=cinder.volume.drivers.san.hp.hp_3par_fc.HP3PARFCDriver19:45
jungleboyjhemna: Sweet, thanks.  I will pass that along to our customer .... hey wait a minute ...19:46
hemnawhy does my driver not work ?!19:46
nikesh_vedamsmtanino: is it good now19:46
*** r-daneel has joined #openstack-cinder19:47
nikesh_vedamsjgriffith : morning19:48
*** Bsony has quit IRC19:48
nikesh_vedamsthingee : morning19:48
thingeenikesh_vedams: tip, just ask for what you need instead of pinging me.19:49
thingeenikesh_vedams: I don't respond to pings19:49
*** Ilja has joined #openstack-cinder19:49
*** anshul has quit IRC19:51
*** crose has joined #openstack-cinder19:52
hemnaicmp blocked.19:52
nikesh_vedamsthingee:sure,just to inform that we are working on CI for scst and was breaking our scst code so uploaded fix for this
*** e0ne is now known as e0ne_19:55
*** EmilienM is now known as EmilienM|afk19:56
*** harlowja_ has quit IRC19:57
*** ronis__ has quit IRC19:57
thingeenikesh_vedams: excellent!19:57
*** devlaps has joined #openstack-cinder19:58
*** harlowja has joined #openstack-cinder20:01
*** e0ne_ has quit IRC20:01
*** jaypipes has quit IRC20:01
openstackgerritTom Swanson proposed openstack/cinder: REST API removed option fails snapshot creation
*** lpetrut has joined #openstack-cinder20:02
*** jaypipes has joined #openstack-cinder20:02
SwansonI spaced making the branch in the gerrit commit bit.  How should I fix this?20:02
*** lpetrut has left #openstack-cinder20:03
*** e0ne has joined #openstack-cinder20:05
*** anshul has joined #openstack-cinder20:07
*** Mandell has quit IRC20:08
*** _cjones_ has quit IRC20:08
openstackgerritTom Swanson proposed openstack/cinder: Dell SC API change fails snapshot creation in Dell driver
jungleboyjSwanson: He he.  I used to do taht all the time.20:11
*** alecv has quit IRC20:11
SwansonI scrolled the workflow page a bit too fast this time.20:12
*** timcl1 has quit IRC20:13
*** dustins has quit IRC20:13
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements
mtaninohemna: thank you for your comment. I will add deprecated comments and add warning message when these configs will be used.20:15
*** tbarron has quit IRC20:16
*** timcl has joined #openstack-cinder20:16
*** annashen has quit IRC20:17
*** timcl has quit IRC20:19
*** tbarron has joined #openstack-cinder20:22
openstackgerritTom Swanson proposed openstack/cinder: Dell SC API change fails snapshot creation
*** timcl has joined #openstack-cinder20:23
*** _cjones_ has joined #openstack-cinder20:25
*** gmurphy has joined #openstack-cinder20:26
*** tbarron has quit IRC20:26
openstackgerritYAMADA Hideki proposed openstack/cinder: Add missing copy_volume_to_image method to Sheepdog driver
*** rushil has joined #openstack-cinder20:28
*** logan2 has joined #openstack-cinder20:29
*** kmartin has quit IRC20:29
*** chlong has joined #openstack-cinder20:30
*** tbarron has joined #openstack-cinder20:33
jungleboyjSwanson: Did you figure it out.  Sorry, manager walked in.20:34
*** ebalduf has joined #openstack-cinder20:36
*** timcl1 has joined #openstack-cinder20:38
*** Mandell has joined #openstack-cinder20:38
*** madskier has joined #openstack-cinder20:38
*** timcl has quit IRC20:38
Swansonjungleboyj: yeah, I think so.20:38
SwansonZuul seems broken.  450 in check.  And why did it run python26 against my driver and mine alone!?!?20:39
jungleboyjSwanson: Ok, sorry, I didn't respond.  Bascially you just need to do a 'git checkout -b <branchname>' in master before doing anything else.20:39
jungleboyjIf your review is still on master, it doesn't really matter, just need to change it locally so you don't lose it.20:39
smcginnisSwanson: Zuul got backed up last night. Still trying to catch up.20:40
SwansonThat's what I did.  Seems fine now.20:40
angela-sthingee: May I ask when BPs which were rejected for K can be resubmitted to L?  Or is this documented on wiki and I'm just not seeing it?  Is this a topic that is discussed at the summit? Thanks.20:40
jungleboyjSwanson: Good.20:40
*** ebalduf has quit IRC20:41
Swansonjungleboyj: Thanks!20:42
jungleboyjSwanson: Welcome.  I just started editing in the wrong branch and couldn't figure out why my other changes weren't there.  *Sigh*20:42
jungleboyjAnd they still aren't *Sigh*20:43
*** akerr has quit IRC20:43
*** Yogi1 has quit IRC20:44
*** esker has joined #openstack-cinder20:44
thingeeangela-s: it is not documented anywhere.20:47
thingeeangela-s: very likely you can look at gerrit reviews for cinder that are open and marked with a topic branch bp/*20:47
thingeethat have a -2 at this point20:47
angela-sthingee: yes, we have 2 BPs in this state.  trying to determine the date at which we can move forward with those for L.  I was assuming that we have to wait until L is master.  is that the correct assumption?20:51
*** crose has quit IRC20:53
*** crose has joined #openstack-cinder20:53
thingeeangela-s: yes eventually we'll propose a patch that will switch things to L.20:54
openstackgerritnikeshmahalka proposed openstack/cinder: Two choices for iscsi_helper is missing
*** annashen has joined #openstack-cinder20:54
thingeeWe will until march 19th + whatever time needed for RCs20:54
*** Bsony has joined #openstack-cinder20:55
*** kmartin has joined #openstack-cinder20:55
angela-sthingee: Ok, thanks for the confirmation.  i've not been to a summit before, but will attend Vancouver.  Are BP proposals discussed at the summit?20:55
thingeeangela-s: if there is discussion really needed, sure20:56
thingeethere are multiple kinds of sessions that happen though20:56
thingeeangela-s: are you on the openstack dev ML?20:56
thingeeangela-s: these are the different kinds of sessions that will exist
thingeeangela-s: and here's where I talked about proposals
thingeefor cinder20:58
angela-sthingee: ok.  besides the BPs, we have a couple of topics for discussion, i will post those to the etherpad for the April 27 meeting.  thanks for the info!20:58
thingeeangela-s: np20:59
*** dustins has joined #openstack-cinder20:59
*** Bsony has quit IRC20:59
*** jungleboyj has quit IRC20:59
*** kmartin has quit IRC21:00
*** angela-s_ has joined #openstack-cinder21:01
*** angela-s has quit IRC21:03
*** nkrinner has quit IRC21:05
*** Longgeek has quit IRC21:10
*** e0ne has quit IRC21:14
*** cdelatte has quit IRC21:14
*** alexpilotti has quit IRC21:17
*** Anticimex has quit IRC21:21
*** sandywalsh has quit IRC21:23
*** sandywalsh has joined #openstack-cinder21:24
*** kmartin has joined #openstack-cinder21:26
*** Anticimex has joined #openstack-cinder21:27
*** rushil has quit IRC21:28
*** patrickeast has quit IRC21:29
hemnamtanino, ok cool, so you get my point about deprecating the config options?21:30
*** Redcavalier has quit IRC21:31
mtaninohemna: yes. I understand it. I will add same code of lio_initiator_iqns config option in my patch.21:31
*** rushil has joined #openstack-cinder21:31
*** patri582 has quit IRC21:31
mtaninohemna: This option shows warning message when it is uded.21:32
hemnamtanino, ok cool.21:32
*** patrickeast__ has quit IRC21:32
*** patrickeast_ has quit IRC21:32
hemnamtanino, we can remove it in L.  we just need to at least deprecate options for a release afaik.21:32
*** patri251 has quit IRC21:32
*** patri498 has quit IRC21:32
mtaninohemna: I got it.21:32
hemnamtanino, thanks man.21:33
*** patri461 has quit IRC21:33
*** patrickeast has joined #openstack-cinder21:34
*** patrickeast_ has joined #openstack-cinder21:35
*** patrickeast__ has joined #openstack-cinder21:35
*** patri255 has joined #openstack-cinder21:35
*** patri910 has joined #openstack-cinder21:35
*** patri169 has joined #openstack-cinder21:35
*** patri733 has joined #openstack-cinder21:35
Swansonjungleboyj: thanks for the review!21:35
*** bswartz has quit IRC21:41
*** EmilienM|afk is now known as EmilienM21:43
*** bswartz has joined #openstack-cinder21:46
*** madskier has quit IRC21:46
*** dustins_ has joined #openstack-cinder21:47
*** dustins has quit IRC21:48
*** dustins_ has quit IRC21:49
*** rushil has quit IRC21:49
*** dhellmann has quit IRC21:52
*** rushil has joined #openstack-cinder21:55
*** dhellmann has joined #openstack-cinder21:55
*** rushil has quit IRC21:56
*** dhellmann has quit IRC21:56
*** dhellmann has joined #openstack-cinder21:57
*** rushil has joined #openstack-cinder21:57
*** bswartz has quit IRC21:58
*** harlowja has quit IRC21:58
*** rushil has quit IRC22:02
*** patrickeast has quit IRC22:06
*** esker has quit IRC22:09
*** patrickeast_ has quit IRC22:09
*** patrickeast__ has quit IRC22:09
*** patri169 has quit IRC22:10
*** patri910 has quit IRC22:10
*** patri255 has quit IRC22:10
*** annegentle has quit IRC22:10
*** patri733 has quit IRC22:10
*** patrickeast has joined #openstack-cinder22:11
*** annegentle has joined #openstack-cinder22:11
*** patrickeast has quit IRC22:12
*** patrickeast has joined #openstack-cinder22:12
*** chlong has quit IRC22:12
*** patrickeast_ has joined #openstack-cinder22:13
*** patrickeast__ has joined #openstack-cinder22:13
*** patri582 has joined #openstack-cinder22:13
*** patri455 has joined #openstack-cinder22:13
*** patri696 has joined #openstack-cinder22:13
*** angela-s_ has quit IRC22:13
*** jamielennox|away is now known as jamielennox22:14
*** anshul has quit IRC22:20
*** e0ne has joined #openstack-cinder22:22
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Make unused iscsi_num_targets, iser_num_targets configs as deprecated
*** thangp has quit IRC22:25
*** tbarron has quit IRC22:27
*** tbarron has joined #openstack-cinder22:27
*** harlowja has joined #openstack-cinder22:28
*** Mandell has quit IRC22:29
*** e0ne has quit IRC22:30
*** mriedem is now known as mriedem-away22:30
*** patrickeast___ has joined #openstack-cinder22:31
*** tbarron has quit IRC22:32
*** diemt has quit IRC22:33
Swansonhemna: Thanks!22:33
anishmtanino: I thought you were only deprecating iser_num_targets ?22:39
mtaninoanish: No. both iscsi_num_targets and iser_num_targets22:40
mtaninoanish: Do you still need iscsi_num_targets?22:40
*** erlon has quit IRC22:41
anishmtanino: nope, my understanding from hemna's comments was only iser needed to be deprecated22:41
hemnaanish, any config options that existed in a previous release.22:43
hemnaif you want to remove a config option, and it existed in a prior release, it needs to first be deprecated for a release, before removal.22:44
mtaninohemna: I updated the patch. If you have chance, please visit again.
anishhemna: gotcha, thanks22:46
openstackgerritAnthony Lee proposed openstack/cinder: Adding manage/unmanage support for LeftHand driver
*** ebalduf has joined #openstack-cinder22:47
*** Anticimex has quit IRC22:49
*** ebalduf has quit IRC22:51
*** Anticimex has joined #openstack-cinder22:51
*** asselin_ has joined #openstack-cinder22:52
*** asselin_ has quit IRC22:57
*** Mandell has joined #openstack-cinder22:57
*** crose has quit IRC22:57
*** eharney has quit IRC23:00
*** NarenNarendra has joined #openstack-cinder23:00
*** annegentle has quit IRC23:04
*** alonmarx has quit IRC23:04
*** alonmarx has joined #openstack-cinder23:05
hemnamtanino, can you do a driver cert run w/ the IET target configured for lvm ?23:06
*** alexpilotti has joined #openstack-cinder23:06
hemnaI don't think we've thought about the CI implications of the target drivers23:06
hemnathey really should be CI'd separately, since they fundamentally change the capability of a driver23:07
mtaninohemna: OK. I can post cert test.23:07
hemnamtanino, I think that'd be great.  thank you!23:07
*** jaypipes has quit IRC23:09
mtaninohemna: A comment from jgriffith, we don't need CI for IET because this driver is reference driver.23:11
hemnawell lvm is a reference driver23:11
hemnabut it can be changed based upon which target is enabled23:11
hemnaand that changes the capability of lvm23:11
mtaninohemna: but your concern is understandable.23:12
hemnasince lvm is a reference driver, if anything it should be tested more rigorously than anything23:12
hemnaincluding all of the possible target configurations that lvm can have.23:12
openstackgerritAnthony Lee proposed openstack/cinder: HP 3par driver filter and evaluator function
hemnamtanino, and iet is not the default target for lvm23:12
hemnaso it actually never gets tested anywhere.23:12
mtaninohemna: correct...23:13
hemnahence the request for the driver cert results at a minimum23:13
mtaninohemna: personally, I tried cert test and all tests were passed.23:14
mtaninoso I will try again and post the result23:14
hemnaperfect, post the results. :)23:14
mtaninousing latest patch set23:14
anishmtanino: maybe you want to open the bug for cert results now and update commit msg23:15
anish(jenkins is slowwww today)23:15
mtaninoanish: sure23:15
mtaninoand make the bug invalid23:15
anishthat way, jenins will be done by the time you actually update with results :)23:15
anishnever understood why a change restricted to the commit msg forces a recheck (if I am remembering this correctly), seems unnecessary23:16
*** kaufer has quit IRC23:16
*** ganso_ has quit IRC23:17
hemnamtanino, ok I added this as an agenda item for next week's meeting.23:17
hemnawe need to talk about it as a cinder team.23:17
hemnawhat to do about CI for the different target objects.23:17
mtaninohemna: sure. thank you.23:18
*** jungleboyj has joined #openstack-cinder23:18
anishhemna: add my name to the ping list please (or point me to where I can do this)23:20
hemnajust login and edit :)23:20
*** jcru has joined #openstack-cinder23:22
anishcopying stuff over from nova23:22
anishgrr, why is one of them wiki/CinderMeetings and the other wiki/Meetings/Nova23:22
hemnaum, one is for Nova and the other is for Cinder ?23:23
anishI mean23:25
anishwhy CinderMeetings instead of Meetings/Cinder23:25
hemnano idea23:25
*** annegentle has joined #openstack-cinder23:25
hemnabecause we want to be difficult ?23:25
SwansonNova's not the boss of us!23:26
*** alexpilotti has quit IRC23:26
*** jcru has quit IRC23:26
*** annegentle has quit IRC23:32
*** annegentle has joined #openstack-cinder23:33
*** IanGovett has quit IRC23:39
*** bswartz has joined #openstack-cinder23:39
openstackgerritJulie Gravel proposed openstack/cinder: Validate volume type extra specs
*** chlong has joined #openstack-cinder23:42
*** annegentle has quit IRC23:43
*** annegentle has joined #openstack-cinder23:43
anishcan someone take a look at and ?23:44
anishthey're very small changes I'm hoping to get in before the deadline23:44
*** chlong has quit IRC23:46
*** jungleboyj has quit IRC23:46
*** annashen has quit IRC23:46
*** chlong has joined #openstack-cinder23:46
hemnaanish, careful what you ask for :P23:48
*** annegentle has quit IRC23:48
*** ebalduf has joined #openstack-cinder23:48
anishhemna: you too :) there's a giant chain of irc messages on why six should not be used23:50
hemnayah that's what I thought as well23:50
anishthe end conclusion was if you're using , instead of %, six is not required irrespective of log level23:52
*** ebalduf has quit IRC23:53
hemnayah I think that wrapping happens inside the call then23:54
hemnayou aren't building a string, so don't wrap it.23:54
anish% without six can issues (I forget what but xyang/smcginnis/jgriffith know the deets) but was discussed here as well
hemnayah because % is trying to construct a string23:54
hemnaand if you don't wrap it, the exception object doesn't get decoded correctly into a string.23:55
*** EmilienM is now known as EmilienM|afk23:56
*** annegentle has joined #openstack-cinder23:56
openstackgerritAnish Bhatt proposed openstack/cinder: Simplify cxt test_create_export() unit test
*** annegentle has quit IRC23:57
anishhemna: added an explanation ^23:57
*** annegentle has joined #openstack-cinder23:57
hemnaanish, ok thank you.23:58
*** annegentle has quit IRC23:58
hemnajust remember the commit messages are all you have sometimes when looking through git commits locally.23:58
hemnaand something that explains the patch is all that you have to go on at times.23:58
anishgotcha, the new one commit message is much more self explanatory23:59

Generated by 2.14.0 by Marius Gedminas - find it at!