Tuesday, 2019-02-05

*** _alastor_ has quit IRC00:11
*** tosky has quit IRC00:14
*** sdake has quit IRC00:25
*** sdake has joined #openstack-cinder00:29
*** sdake has quit IRC00:30
*** N3l1x has joined #openstack-cinder00:32
*** Nel1x has quit IRC00:32
*** TxGirlGeek has quit IRC00:44
*** sdake has joined #openstack-cinder00:57
*** sdake has quit IRC01:03
*** whoami-rajat has joined #openstack-cinder01:18
*** sdake has joined #openstack-cinder01:19
*** ircuser-1 has joined #openstack-cinder01:21
*** sdake has quit IRC01:27
*** sdake has joined #openstack-cinder01:35
*** sdake has quit IRC01:52
*** sdake has joined #openstack-cinder02:21
*** Dinesh_Bhor has joined #openstack-cinder02:32
*** rchurch_ has joined #openstack-cinder02:35
*** rchurch has quit IRC02:36
*** Dinesh_Bhor has quit IRC02:58
*** psachin has joined #openstack-cinder03:04
*** Dinesh_Bhor has joined #openstack-cinder03:04
*** sdake has quit IRC03:17
*** sdake has joined #openstack-cinder03:18
whoami-rajatjungleboyj:  i just added a topic yesterday, hope we get some time to discuss that too.03:34
*** sdake has quit IRC03:35
*** dave-mccowan has quit IRC03:44
jungleboyjOk, what did you add?03:48
jungleboyjwhoami-rajat: ^^03:48
*** Dinesh_Bhor has quit IRC03:54
whoami-rajatjungleboyj:  Discuss way of keeping track of features/code affecting other projects03:55
whoami-rajatthis one03:55
*** sdake has joined #openstack-cinder03:57
jungleboyjwhoami-rajat:  Ok, when is a good time for you to discuss that?03:57
whoami-rajatjungleboyj: if it is possible then can we do it at mid time before/after lunch, but i see the timings are filled up so i'm fine with any timing.04:02
jungleboyjAre you planning to connect remotely?04:03
whoami-rajatjungleboyj: yes.04:03
jungleboyjwhoami-rajat:  If so, when we have some extra time just bring it up.  We can usually fit topics in.04:03
whoami-rajatjungleboyj: sure, thanks!04:04
jungleboyjWelcome.  Look forward to chatting with you tomorrow.04:04
whoami-rajatjungleboyj: same here :)04:05
*** v12aml has quit IRC04:08
*** v12aml has joined #openstack-cinder04:14
*** sdake has quit IRC04:16
*** gkadam__ has joined #openstack-cinder04:17
*** udesale has joined #openstack-cinder04:17
*** bhagyashris_ has joined #openstack-cinder04:39
*** Dinesh_Bhor has joined #openstack-cinder04:42
*** sdake has joined #openstack-cinder04:44
*** lpetrut has joined #openstack-cinder04:46
*** arunkant_ has quit IRC04:54
*** dims has quit IRC04:58
*** N3l1x has quit IRC05:19
*** lpetrut has quit IRC05:23
*** sdake has quit IRC06:03
*** markvoelker has joined #openstack-cinder06:45
*** Luzi has joined #openstack-cinder06:48
*** dims has joined #openstack-cinder06:48
*** dims has quit IRC06:52
*** dims has joined #openstack-cinder06:54
*** dims has quit IRC06:59
*** dims has joined #openstack-cinder07:02
*** abhishekk has joined #openstack-cinder07:04
*** markvoelker has quit IRC07:18
*** e0ne has joined #openstack-cinder07:33
*** pcaruana has joined #openstack-cinder07:45
*** tkajinam has quit IRC08:06
*** markvoelker has joined #openstack-cinder08:15
*** lpetrut has joined #openstack-cinder08:21
*** lpetrut has joined #openstack-cinder08:22
*** Luzi_ has joined #openstack-cinder08:25
*** Luzi has quit IRC08:27
*** lpetrut has quit IRC08:28
*** markvoelker has quit IRC08:49
*** tosky has joined #openstack-cinder08:50
*** lennyb has quit IRC08:55
*** lennyb has joined #openstack-cinder08:59
*** ociuhandu has quit IRC09:03
*** Luzi_ has quit IRC09:07
*** Luzi has joined #openstack-cinder09:07
*** ociuhandu has joined #openstack-cinder09:24
*** ganso has joined #openstack-cinder09:33
*** Dinesh_Bhor has quit IRC09:35
*** Dinesh_Bhor has joined #openstack-cinder09:36
*** luizbag has joined #openstack-cinder09:43
*** markvoelker has joined #openstack-cinder09:46
*** bhagyashris_ has quit IRC09:53
openstackgerritRajat Dhasmana proposed openstack/cinder master: Raise appropriate exception  https://review.openstack.org/63489409:57
*** v12aml has quit IRC10:15
*** v12aml has joined #openstack-cinder10:16
*** markvoelker has quit IRC10:19
*** ociuhandu_ has joined #openstack-cinder10:28
*** ociuhandu has quit IRC10:31
*** erlon has joined #openstack-cinder10:45
*** gnufied has quit IRC11:05
*** Dinesh_Bhor has quit IRC11:11
*** gnufied has joined #openstack-cinder11:11
*** Dinesh_Bhor has joined #openstack-cinder11:13
*** markvoelker has joined #openstack-cinder11:15
*** davidsha has joined #openstack-cinder11:27
*** udesale has quit IRC11:28
*** markvoelker has quit IRC11:48
*** _hemna has quit IRC11:51
*** abhishekk has quit IRC12:00
openstackgerritSofia Enriquez proposed openstack/cinder master: Support Incremental Backup Completion In RBD  https://review.openstack.org/62794112:10
*** enriquetaso has joined #openstack-cinder12:15
*** samc-bbc has quit IRC12:32
*** avishay has joined #openstack-cinder12:34
*** e0ne has quit IRC12:41
*** avishay has quit IRC12:48
*** gkadam__ has quit IRC12:58
*** sdake has joined #openstack-cinder13:05
*** mriedem has joined #openstack-cinder13:11
*** jmlowe has quit IRC13:17
*** abishop_ has quit IRC13:22
enriquetasojungleboyj, o/ how are u? have you defined a "communication channel" for the Stein Mid-Cycle?13:26
*** Dinesh_Bhor has quit IRC13:30
*** jmlowe has joined #openstack-cinder13:34
*** erlon has quit IRC13:41
*** e0ne has joined #openstack-cinder13:42
smcginnisenriquetaso: We are just getting set up and should have a link to join soon.13:44
smcginnisHe's currently running back and forth to the front door to let people in. :)13:44
enriquetasosmcginnis, hahaha :D  thanks!13:45
whoami-rajatI hope for a hangout + live stream link like last time :)13:49
*** mvkr has quit IRC13:52
*** sdake has quit IRC13:54
smcginnisStill trying to get things set up. Might be another 10-15 minutes yet.14:03
*** sdake has joined #openstack-cinder14:04
*** rosmaita has joined #openstack-cinder14:04
*** rosmaita has left #openstack-cinder14:05
*** rosmaita has joined #openstack-cinder14:06
*** eharney has joined #openstack-cinder14:07
*** psachin has quit IRC14:09
openstackgerritMatt Riedemann proposed openstack/cinder master: Drop nova-multiattach job  https://review.openstack.org/60698514:12
mriedemkickstarting ^ again14:15
mriedemthis time it should be good14:15
smcginnisThanks mriedem14:15
*** markvoelker has joined #openstack-cinder14:16
*** sdake has quit IRC14:17
jungleboyjHangouts meeting:14:18
*** mvkr has joined #openstack-cinder14:20
*** erlon has joined #openstack-cinder14:24
jungleboyjCoffee just arrived.  Will start after that.14:25
erlonjungleboyj, how is the webcasting going?14:26
* erlon has lost connection on the last hour14:26
erlonso, might have lost any conversation about it14:26
smcginniserlon: Link is in https://etherpad.openstack.org/p/cinder-stein-mid-cycle-planning14:27
smcginnisLine 14714:27
smcginnisWe will get started in just a couple minutes, so you haven't missed anything yet.14:27
whoami-rajaterlon: not yet started. you can join the hangout link above.14:27
erlonsmcginnis, nice thanks14:27
jungleboyjErlon, going to start the webcast in a minute.14:28
jungleboyjThey are setting up stuff here.14:29
*** udesale has joined #openstack-cinder14:32
jungleboyjLive stream is here:  https://www.youtube.com/watch?v=yqu88SvzSJ814:34
*** sdake has joined #openstack-cinder14:34
*** pckizer has quit IRC14:38
whoami-rajatrunning py3 jobs https://review.openstack.org/#/c/616748/14:40
*** avishay has joined #openstack-cinder14:42
avishaygeguileo: Hey.  I have a question about os-brick... Seems like I found a bug14:42
geguileoavishay: :-(14:43
geguileodon't look!!!14:43
avishaySo I wanted to look at the potential volume paths and make sure they don't exist before the connect14:44
avishayI was please to see that extend_volume already does that14:44
*** sdake has quit IRC14:45
avishayBut it just takes whatever is from connection_properties, which if I understand correctly is fine if we have target_portals, but if we need to do discovery, we won't have all paths in connection_properties14:46
avishaygeguileo: right?14:46
*** pckizer has joined #openstack-cinder14:46
*** sdake has joined #openstack-cinder14:48
*** markvoelker has quit IRC14:49
whoami-rajatsmcginnis:  i was working on the upgrade checks base framework in other projects, i can implement the checks if noone is working on it currently?14:51
jungleboyjCan people hear on the live stream?14:54
whoami-rajatjungleboyj: yes14:55
*** sdake has quit IRC14:56
jungleboyjOk.  I have it using the mic on the camera accidentally.  Will try to get it changed on the next recording.14:56
*** ociuhandu_ has quit IRC14:58
*** ociuhandu has joined #openstack-cinder14:58
*** Nel1x has joined #openstack-cinder14:59
*** _hemna has joined #openstack-cinder14:59
*** arunkant has joined #openstack-cinder15:02
*** sdake has joined #openstack-cinder15:04
*** udesale has quit IRC15:06
avishayhemna: _hemna Hey!  What's up?  Perhaps you know the answer to my question above about extend_volume in os-brick?  (I saw you wrote it)15:11
*** Luzi has quit IRC15:12
openstackgerritBrian Rosmaita proposed openstack/python-brick-cinderclient-ext master: Add python 3.6 unit test job  https://review.openstack.org/61674815:13
_hemnawhat's up?15:14
_hemnaI wrote that ages and ages ago15:14
_hemnanot sure if anyone is really using it15:14
avishayOK.  Thanks!15:15
jungleboyjgeguileo:  You around?  Walt is here and wanted to chat about Cinderlib.15:28
geguileojungleboyj: about to go into a meeting, but sure15:29
jungleboyjEric says it is just your team meeting.  Come join us.15:29
jungleboyjgeguileo: https://hangouts.google.com/call/c2H8hkBRkJFCWJ_HC5ShAAEE15:30
geguileojungleboyj: I'm in15:30
jungleboyjwoojay:  Is the sound in the live stream better now?15:37
woojayyeah.  definitely.  thank you15:37
jungleboyjAwesome.  Got the right Mic working this time.15:37
geguileojungleboyj: I don't know where in the Etherpad to add it: https://twitter.com/shenitzky/status/108768619257178931315:42
*** vishakha has joined #openstack-cinder15:45
*** markvoelker has joined #openstack-cinder15:46
*** jgriffith has quit IRC15:54
*** jgriffit1 has joined #openstack-cinder15:54
*** jgriffit1 is now known as jgriffith15:55
*** avishay has quit IRC15:59
*** e0ne has quit IRC16:05
*** TxGirlGeek has joined #openstack-cinder16:09
*** markvoelker has quit IRC16:19
*** arunkant_ has joined #openstack-cinder16:23
*** davidsha has quit IRC16:23
*** arunkant has quit IRC16:25
openstackgerritSean McGinnis proposed openstack/cinder master: Remove legacy-tempest-dsvm-full-drbd-devstack job  https://review.openstack.org/63496916:28
mriedemtrying to help with the tooz connection errors in the gate https://review.openstack.org/63497016:30
mriedemsmcginnis: does the libvirt drbd volume driver in nova need to be deprecated as well? https://review.openstack.org/#/c/256292/6/nova/virt/libvirt/driver.py16:32
mriedemvolume type of 'drbd'16:32
mriedemi guess in cinder that's the DrbdManageDrbdDriver in 'remote' mode16:34
smcginnismriedem: I believe so.16:35
*** openstackgerrit has quit IRC16:35
smcginniswoojay: ^^16:36
woojayYeah didn't know we loose one somewhere else.  Let me look at it for one sec please.16:37
mriedemwoojay: this is the nova driver https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume/drbd.py16:37
*** jmlowe has quit IRC16:44
woojaymriedem: I believe I still need that nova driver for the LINSTOR driver, that will ultimately replace DRBDManage driver for DRBD.16:44
mriedemok as long as the 'drbd' volume type is not going away then i think we're ok16:48
woojayk, thank you.16:49
*** gnufied has quit IRC16:50
erlonneed to drop16:51
smcginnisThanks erlon16:52
*** _alastor_ has joined #openstack-cinder16:52
*** mvkr has quit IRC16:53
*** erlon has quit IRC16:57
enriquetasosee you after lunch!17:07
*** enriquetaso has quit IRC17:11
*** markvoelker has joined #openstack-cinder17:16
*** enriquetaso has joined #openstack-cinder17:20
*** _hemna has quit IRC17:23
*** jmlowe has joined #openstack-cinder17:26
*** gnufied has joined #openstack-cinder17:35
*** luizbag_ has joined #openstack-cinder17:35
*** luizbag has quit IRC17:35
*** jmlowe has quit IRC17:36
*** enriquetaso has quit IRC17:44
*** markvoelker has quit IRC17:48
*** arunkant__ has joined #openstack-cinder17:56
*** jmlowe has joined #openstack-cinder17:57
*** arunkant_ has quit IRC18:00
*** enriquetaso has joined #openstack-cinder18:05
*** dims has quit IRC18:06
*** arunkant__ has quit IRC18:06
*** enriquetaso has quit IRC18:10
*** ociuhandu has quit IRC18:11
*** sdake has quit IRC18:11
*** ociuhandu has joined #openstack-cinder18:13
*** ociuhandu has quit IRC18:17
*** dims has joined #openstack-cinder18:26
*** Nel1x has quit IRC18:29
jungleboyjWe are recongregating.18:30
smcginnisWe are napping.18:31
jungleboyjHangouts restarted:  https://hangouts.google.com/call/c2H8hkBRkJFCWJ_HC5ShAAEE18:33
*** _hemna has joined #openstack-cinder18:33
*** markvoelker has joined #openstack-cinder18:46
*** itlinux has joined #openstack-cinder18:49
*** sdake has joined #openstack-cinder18:52
whoami-rajatjungleboyj:  what is the notes section for https://docs.openstack.org/cinder/latest/reference/support-matrix.html ?19:00
*** luizbag_ has quit IRC19:00
*** openstackgerrit has joined #openstack-cinder19:04
openstackgerritEric Harney proposed openstack/cinder master: Update rbd thin-provisioning in support-matrix  https://review.openstack.org/63500919:04
*** enriquetaso has joined #openstack-cinder19:07
*** vishakha has quit IRC19:12
*** dims has quit IRC19:16
*** markvoelker has quit IRC19:19
*** dims has joined #openstack-cinder19:25
*** pcaruana has quit IRC19:27
*** erlon has joined #openstack-cinder19:29
*** ociuhandu has joined #openstack-cinder19:30
*** ociuhandu has quit IRC19:35
openstackgerritGorka Eguileor proposed openstack/cinder master: Import the cinderlib code  https://review.openstack.org/62066919:41
openstackgerritGorka Eguileor proposed openstack/cinder master: Add cinderlib docs  https://review.openstack.org/62067019:41
openstackgerritGorka Eguileor proposed openstack/cinder master: Add cinderlib functional tests  https://review.openstack.org/62067119:41
*** erlon has quit IRC19:54
*** ociuhandu has joined #openstack-cinder19:59
*** e0ne has joined #openstack-cinder20:03
*** mvkr has joined #openstack-cinder20:09
*** markvoelker has joined #openstack-cinder20:16
*** enriquetaso has quit IRC20:19
*** jmlowe has quit IRC20:29
whoami-rajate0ne:  could you check again https://review.openstack.org/#/c/616748/2 ?20:36
e0newhoami-rajat: will do in 15 mins20:36
whoami-rajate0ne: sure20:37
whoami-rajatjungleboyj:  do we need to start implementing missing commands in OSC or holding it for now?20:41
jungleboyjwhoami-rajat:  We want to start implementing as many as possible.20:41
whoami-rajatjungleboyj: ok20:43
smcginnishemna: https://releases.openstack.org/reference/using.html#requesting-a-release20:44
*** markvoelker has quit IRC20:48
smcginniseharney: https://bugs.launchpad.net/cinder/+bug/175235521:02
openstackLaunchpad bug 1752355 in Cinder "Configure an NFS storage back end in Cinder" [Undecided,Incomplete]21:02
smcginnisShouldn't packages handle that?21:02
woojayjungleboyj: Thank you for the youtube cast.21:09
openstackgerritMerged openstack/cinder master: Update rbd thin-provisioning in support-matrix  https://review.openstack.org/63500921:10
openstackgerritSean McGinnis proposed openstack/cinder master: Update install docs to match default NFS config  https://review.openstack.org/63504121:12
*** itlinux has quit IRC21:29
openstackgerritSofia Enriquez proposed openstack/cinder master: Support Incremental Backup Completion In RBD  https://review.openstack.org/62794121:31
openstackgerritRajat Dhasmana proposed openstack/python-brick-cinderclient-ext master: Add python 3.6 unit test job  https://review.openstack.org/61674821:32
*** e0ne has quit IRC21:33
*** _hemna has quit IRC21:39
*** sdake has quit IRC21:39
openstackgerritRajat Dhasmana proposed openstack/cinder master: Add Python 3.6 version in setup.cfg  https://review.openstack.org/63504921:40
*** markvoelker has joined #openstack-cinder21:46
openstackgerritEric Fried proposed openstack/cinder master: DNM: Demonstrate driver sorting in support matrix  https://review.openstack.org/63505121:46
*** mchlumsky has quit IRC21:52
*** ganso has quit IRC21:57
*** efried has joined #openstack-cinder22:06
efriedsmcginnis: I'm sure I'm being a doofus here.22:06
smcginnisefried: Well that's odd. Not the output I was getting locally.22:07
efriedsmcginnis: Was it actually sorted, or was it just sorted *differently*?22:08
efriedsmcginnis: What's your PYTHONHASHSEED locally?22:08
efriedIt looks to me like you may actually have to sort implementations by driver.title22:08
* efried tries...22:10
smcginnisI think it should work based on http://git.openstack.org/cgit/openstack/sphinx-feature-classification/tree/sphinx_feature_classification/support_matrix.py#n28122:11
* smcginnis notices line 312 is redundant with 281.22:11
efriedsmcginnis: But you're not sorting matrix.drivers. You're sorting feature.implementations22:11
smcginnisTrue, but I saw the same behavior.22:12
smcginnisTrying to repro it now.22:12
efriedsmcginnis: I don't know offhand what feature.implementations corresponds to, versus matrix.drivers. But I'm guessing (about to confirm) that you could say22:14
efriedsorted(features.implementations, key=lambda x: matrix.drivers[x].title)22:14
efriedand it would work. Whether that's doing extra that you don't need, I can't say...22:14
efriedsmcginnis: FWIW, building locally off of cinder master with nothing changed gives me the proper sorting. So I'm betting it's the fact that our local envs aren't setting PYTHONHASHSEED by default.22:16
smcginnisefried: Hmm, sorts fine locally for me.22:16
efriedsmcginnis: Yeah, try turning off your sphinx patch and see if it still does :)22:17
smcginnisI'm trying using the PYTHONHASHSEED from the gate job.22:18
*** markvoelker has quit IRC22:20
smcginnisLocal output is sorted using the same PYTHONHASHSEED. Trying unpatches support-matrix code now.22:21
smcginnisI wonder if it is checking out the right commit, but then installing the released version - http://logs.openstack.org/51/635051/1/check/openstack-tox-docs/0ab7ec4/job-output.txt.gz#_2019-02-05_21_52_11_86700322:23
efriedYeah, I was just looking at that. That would suck.22:25
efriedI can't get it to *not* sort locally, *without* your change, *with* a nonzero hash seed. So the hash seed isn't the problem. And unless we figure out what's causing sorting to happen locally, means we'll have to debug it in the gate :(22:26
efriedsmcginnis: ima try my above fix on top of yours22:27
*** sdake has joined #openstack-cinder22:27
smcginnisefried: OK, that might be best. I saw the same sorted behavior without the patch. Cleaned out build output and rerunning using the same hash seed once more time.22:28
smcginnisReally concerns me that we can't reproduce this behavior locally. Not sure what would cause a difference if it's not the hash seed.22:29
smcginnisYep, still sorted.22:29
openstackgerritEric Fried proposed openstack/cinder master: DNM: Demonstrate driver sorting in support matrix  https://review.openstack.org/63505122:32
efriedsmcginnis: ^ with new dep22:32
efriedsmcginnis: I've run into this before, but I don't remember what the cause was.22:32
*** sdake has quit IRC22:34
*** sdake has joined #openstack-cinder22:36
smcginnisefried: New patch isn't sorted either. :(22:42
smcginnisI suppose I could add some javascript to the css to just sort the list.22:46
efriedye gods no22:52
efriedyeah, this one is unsorted in a totally different way. Again making me wonder if it's not properly honoring the patch22:53
smcginnisefried: I would be sure of that if it wasn't for different results locally.22:53
*** tkajinam has joined #openstack-cinder22:55
*** erlon has joined #openstack-cinder23:03
efriedsmcginnis: The keys to the feature.implementations dict are the same as the keys to the matrix.drivers dict. Which are things like 'driver.datacore', not the actual title, meaning the sorting isn't exactly alphabetical (which I noticed e.g. where "Generic NFS Reference Driver (NFS)" comes in the middle of the Ns) BUT your original fix *should* be working fine.23:05
efried(Pdb) sorted(matrix.drivers) == sorted(feature.implementations)23:06
smcginnisefried: Adding some debug logs, I see the same where the full module path in "keys" is sorted as I would like to have them.23:06
smcginnisSo I guess something later is randomizing the bullet_list entries.23:06
efriedor your fix is working, and the gate isn't honoring the dependency for some reason.23:07
smcginnisIt's a simple fix he said... It will just take a few minutes he said...  :)23:07
smcginnisefried: I suppose we could try merging it and see, but if that's the case, it wouldn't show up until an actual release is done.23:08
efriedsmcginnis: f it, I'm going to approve it based on the fact that it should be working, and then once you've gone through all the pain of releasing and updating l-c and all that crap, we'll see if it really worked or not.23:08
efriedwhich seems... stupid.23:08
smcginnisefried: I would feel much much better if I could reproduce things locally. *shrug*23:09
efriedOr we could go gripe in -infra and see whether they can confirm or refute that the dep is or is not being honored.23:09
smcginnisI was thinking of adding Stephen F to the review. He's done a lot of sphinx extensions, so maybe if I add him and we give it until tomorrow he might come back with a reason.23:09
*** erlon has quit IRC23:10
efriedthis really has nothing to do with sphinx.23:11
efriedthough he might know things about why the sorting seems to be happening locally.23:12
efriedActually, it looks to me like the sorting isn't *exactly* the same.23:12
efriedDid you check it exhaustively or just glance through it?23:12
smcginnisMy suspicion is that the sphinx bullet_list is randomized somehow internally.23:13
smcginnisWhat do you mean "check it exhaustively"?23:13
smcginnisIIRC, python 3 added list randomization so folks would stop assuming things would always be in the right order.23:14
efriedI mean that, in my debug session where I'm building locally, the unsorted and sorted versions are *close*, but not *identical*.23:15
efriedSo if a guy were just skimming, it might look like it was really sorted, but it ain't.23:15
efriedi.e. I may have a legit local repro.23:15
*** markvoelker has joined #openstack-cinder23:16
efriedsmcginnis: booyah, confirmed.23:17
efriedAt least in my local env, the ordering starts to be different at23:17
efriedDell EMC VMAX Af (250F, 450F, 850F, 950F) Storage Driver (FC, iSCSI): complete23:17
efriedDell EMC VMAX3 (100K, 200K, 400K) Storage Driver (iSCSI, FC): complete23:17
efriedLet me instate your fix and run it again...23:17
openstackgerritMerged openstack/cinder master: Fix keystone auth config in install doc  https://review.openstack.org/63008123:17
openstackgerritMerged openstack/cinder master: api-ref: modify 'keys' description  https://review.openstack.org/56621523:17
smcginnisefried: VMAX3 should sort after VMAX23:19
efriedsmcginnis: Remember, it's sorting on the keys, not the titles.23:19
efriedIf you want it to sort properly on the titles, you'll have to change it in both places.23:19
efriedfor now I think we'll be happy just to get 'em sorting the same.23:20
smcginnisAhh, right.23:20
efriedas each other.23:20
smcginnisSo really what we want is what you had with sorting on the title value, not the key.23:20
smcginnisI think I've learned this lesson before. Multiple times.23:21
openstackgerritAngela Smith proposed openstack/cinder master: REST connector for Brocade zone driver  https://review.openstack.org/61710023:21
efriedsmcginnis: Confirmed.23:24
efriedsmcginnis: So your fix does work, and I can prove it locally (I just can't prove it in the gate with a dep <fume>)23:24
efriedsmcginnis: I can +2 this... unless you want to do the extra thing to sort by title in both places?23:24
efriedand with that, I'm outta here.23:31
*** rosmaita has left #openstack-cinder23:31
smcginnisefried: Thanks, I think we actually do want your change. I'll update my patch with your changes.23:34
openstackgerritSam Morrison proposed openstack/cinder master: Include availability-zone of a pool in get-pools API request  https://review.openstack.org/59986623:36
*** eharney has quit IRC23:43
*** imacdonn has quit IRC23:48
*** imacdonn has joined #openstack-cinder23:49
*** markvoelker has quit IRC23:50

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!