Friday, 2017-03-03

*** crushil has joined #openstack-manila00:02
*** gcb has quit IRC00:19
*** mkoderer__ has quit IRC00:31
*** mtanino_ has quit IRC00:33
*** mtanino has joined #openstack-manila00:34
*** ganso has quit IRC00:35
*** furlongm has quit IRC00:36
*** mtanino has quit IRC01:31
*** kaisers__ has joined #openstack-manila02:22
*** kaisers_ has quit IRC02:25
*** catintheroof has joined #openstack-manila02:30
openstackgerritTom Barron proposed openstack/manila master: Update test requirement  https://review.openstack.org/44018602:30
openstackgerritPeter Wang proposed openstack/manila master: Handle ssl for VNX manila driver  https://review.openstack.org/44036402:37
*** crushil has quit IRC02:38
*** tuanluong has joined #openstack-manila03:16
*** bswartz has quit IRC03:33
*** bswartz has joined #openstack-manila03:36
*** crushil has joined #openstack-manila03:53
*** furlongm has joined #openstack-manila03:58
*** cknight has joined #openstack-manila04:11
*** shausy has joined #openstack-manila04:25
*** gouthamr has quit IRC04:45
*** crushil has quit IRC04:58
*** catintheroof has quit IRC05:00
*** cknight has quit IRC05:04
*** kaisers__ has quit IRC05:12
openstackgerritPeter Wang proposed openstack/manila master: Handle ssl for VNX manila driver  https://review.openstack.org/44036405:39
openstackgerritPeter Wang proposed openstack/manila master: Handle ssl for VNX manila driver  https://review.openstack.org/44036405:48
*** nkrinner_afk is now known as nkrinner05:59
*** cknight has joined #openstack-manila06:05
*** tinwood has quit IRC06:05
*** tinwood has joined #openstack-manila06:06
*** cknight has quit IRC06:09
*** tuanluong has quit IRC07:15
*** david-lyle has quit IRC07:16
*** david-lyle has joined #openstack-manila07:16
*** pcaruana has joined #openstack-manila07:41
*** cknight has joined #openstack-manila07:52
*** cknight has quit IRC07:57
*** lpetrut has joined #openstack-manila07:59
*** lpetrut has quit IRC08:05
*** akapil has joined #openstack-manila08:33
*** david-lyle has quit IRC09:13
*** david-lyle has joined #openstack-manila09:13
*** a-pugachev has joined #openstack-manila09:16
*** david-lyle_ has joined #openstack-manila09:30
*** david-lyle has quit IRC09:31
openstackgerritPeter Wang proposed openstack/manila master: Handle ssl for VNX manila driver  https://review.openstack.org/44036409:34
*** cknight has joined #openstack-manila09:41
*** cknight has quit IRC09:46
*** dsariel has quit IRC09:51
*** dsariel has joined #openstack-manila10:04
vponomaryovtbarron: yes, what, did you think, is used in our "gate-manila-tempest-minimal-dsvm-dummy-ubuntu-xenial" job? Also, it is easy just to get and look at manila.conf in any of this job results10:15
openstackgerritMerged openstack/manila-ui master: Correct the actual path for setting DJANGO_SETTINGS_MODULE  https://review.openstack.org/43960610:29
*** lpetrut has joined #openstack-manila10:32
*** ganso has joined #openstack-manila10:40
*** akapil has quit IRC11:01
*** a-pugachev has quit IRC11:19
vponomaryovnew day new CI breakage ))11:23
gansovponomaryov: omg11:23
*** tommylikehu_ has joined #openstack-manila11:25
*** tommylikehu_ has quit IRC11:25
*** akapil has joined #openstack-manila11:25
*** tommylikehu_ has joined #openstack-manila11:25
*** a-pugachev has joined #openstack-manila11:26
*** tommylikehu_ has quit IRC11:28
*** cknight has joined #openstack-manila11:29
tbarronvponomaryov: it is just "odd" to me b/c all the other files under manila/tests are for unit tests, not for tempest tests.11:34
*** cknight has quit IRC11:34
tbarronvponomaryov: or is that assertion that otherwise we only have stuff for unit tests under manila/tests false?11:34
vponomaryovtbarron: dummy driver can be used for unit tests too11:34
vponomaryovtbarron: it is used only for testing11:35
tbarronvponomaryov: also, I don't understand your point about "just look at manila.conf from the run", I'm looking right now and don't see anything that tells me that the file manila/tests/share/drivers/dummy.py is required11:35
tbarronyour change is fine, but for many looking at it it is "black magic"11:36
vponomaryovtbarron: http://logs.openstack.org/22/439222/7/check/gate-manila-tempest-minimal-dsvm-dummy-ubuntu-xenial/af84ded/logs/etc/manila/manila.conf.txt.gz11:36
tbarronvponomaryov: what am I not seeing there?11:37
vponomaryov"share_driver = manila.tests.share.drivers.dummy.DummyDriver"11:37
tbarronah11:38
tbarronthat path11:38
vponomaryovI just pointed out how could you answer your question yourself11:38
tbarronyes, thanks11:38
tbarronfor the record I still find that path counter-intuitive :D11:39
tbarronbut I see that it avoids duplication of the same code with the unit tests11:39
tbarronon another subject your replication change is probably fine but should be reviewed by people who understand replication better than I do at this point - e.g. gouthamr11:41
tbarronand ganso11:41
vponomaryovtbarron: it is not real driver, so, we do not store it in common driver dir11:41
tbarronvponomaryov: yeah but it's not a real unit test either :D11:41
tbarronit is also used for a real unit test11:41
vponomaryovtbarron: yeah, I know, need to poke gouthamr more often on such kind of stuff )11:41
tbarronok, moving on11:42
tbarrongate is failing b/c of docs build?11:42
vponomaryovyes ))11:42
* tbarron is waking up slowly this morning11:42
tbarronany leads?  requirements change triggered?11:42
openstackgerritMerged openstack/python-manilaclient master: Updated from global requirements  https://review.openstack.org/43935611:43
* tbarron tries to repro locally11:44
vponomaryovtbarron: I think so -> https://github.com/openstack/requirements/commit/cb64cdc5f22ebb9bbdedf0988f0a011455fee4e911:44
vponomaryovupper constraints were changed11:45
tbarronso it should happen locally if I rebuild my venv11:45
vponomaryovyes11:45
*** shausy has quit IRC12:02
*** tommylikehu_ has joined #openstack-manila12:04
*** a-pugachev has quit IRC12:28
*** a-pugachev has joined #openstack-manila12:29
*** a-pugachev has quit IRC12:30
*** a-pugachev has joined #openstack-manila12:42
*** porrua has joined #openstack-manila13:02
*** porrua_ has joined #openstack-manila13:04
arnewiebalckI have a k8s cluster with up to a few thousand pods that can act as Manila clients. If you have suggestions how to use them for Manila stress and scale testing, let me know :)13:05
*** porrua has quit IRC13:08
*** tommylikehu_ has quit IRC13:12
*** chlong__ has quit IRC13:14
*** tommylikehu_ has joined #openstack-manila13:15
*** tommylikehu_ has quit IRC13:16
*** tommylikehu_ has joined #openstack-manila13:17
*** tommylikehu_ has quit IRC13:21
*** xyang1 has joined #openstack-manila13:24
*** catintheroof has joined #openstack-manila13:31
*** david_1 has joined #openstack-manila13:33
*** dgonzalez_ has joined #openstack-manila13:33
*** databus23_ has joined #openstack-manila13:33
*** tommy_ has joined #openstack-manila13:33
*** tommy_ is now known as Guest8079913:33
*** databus23_ has quit IRC13:35
*** dgonzalez_ has quit IRC13:35
*** Guest80799 has quit IRC13:35
*** david_1 has quit IRC13:35
*** catintheroof has quit IRC13:39
*** databus23_ has joined #openstack-manila13:42
*** david_1 has joined #openstack-manila13:42
*** carthaca_ has quit IRC13:44
*** david_1 has quit IRC13:44
*** sapcc-bot has quit IRC13:44
*** databus23_ has quit IRC13:44
openstackgerritSilvan Kaiser proposed openstack/manila master: Send resize parameters in rpc as list in the Quobyte driver  https://review.openstack.org/43750113:44
*** david_2 has joined #openstack-manila13:44
*** databus23_ has joined #openstack-manila13:44
*** sapcc-bot has joined #openstack-manila13:44
*** carthaca_ has joined #openstack-manila13:44
*** databus23_ has quit IRC13:44
*** david_2 has quit IRC13:44
*** gouthamr has joined #openstack-manila13:57
*** tommylikehu_ has joined #openstack-manila14:04
*** dustins has joined #openstack-manila14:07
*** nkrinner is now known as nkrinner_afk14:08
*** tommylikehu_ has quit IRC14:19
*** chlong__ has joined #openstack-manila14:22
gansotbarron, gouthamr, bswartz, vponomaryov, markstur, xyang1: Hello folks. Could you please take a look at this backport? Thanks in advance https://review.openstack.org/#/c/43542414:22
*** tommylikehu_ has joined #openstack-manila14:23
*** pcaruana has quit IRC14:24
*** eharney has joined #openstack-manila14:28
*** crushil has joined #openstack-manila14:33
*** faizy has joined #openstack-manila14:42
openstackgerritTom Barron proposed openstack/manila master: Unblock gate failure on docs build  https://review.openstack.org/44115614:47
tbarronvponomaryov ganso gouthamr bswartz markstur toabctl xyang1 ^^ let's unblock gate14:48
vponomaryovtbarron: "leg hurts? - cut it!"14:52
tbarronvponomaryov: i'm looking for a followup to get todos back14:53
tbarrontodos aren't a "leg", maybe a little finger14:53
vponomaryov^_^14:53
bswartzcutting off fingers still sounds painful14:53
tbarronso is being stuck in gate14:54
bswartzso manila is the only project that uses this todo structure?14:54
vponomaryovtbarron: better to make this job non-voting14:54
vponomaryovbswartz: no14:54
tbarron"this" todo, with manila_toto extension, yes14:54
vponomaryovbswartz: cinder is not failing because it does not use upper-constraints as we do14:54
bswartzmaking a job nonvoting requires a project-config change and those take 24-48 hours to get reviewed typically14:54
tbarronit's copied from cinder which doesn't have it anymore14:54
bswartzwhich upper constraints are we using?14:55
bswartzthe one from requirements?14:55
vponomaryovhttps://github.com/openstack/manila/commit/c9e1a1aabc0ffb4b60b9327916ab9a65df98087a14:55
vponomaryovand yes, tbarron is correct, Cinder does not have it anymore -> https://github.com/openstack/cinder/commit/14f378a580db5508b81ed184490f52de535182e114:57
toabctlbswartz, yes14:58
toabctlbswartz, https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt14:58
bswartzwhy would anyone not want upper constraints?14:59
toabctlit is wrong in cinder if they do not use it15:00
toabctland btw. they use it15:00
smcginnisWe most certainly use constraints.15:00
vponomaryovyes, my fault, I was looking at old Cinder version15:01
smcginnisWhat does the constraints have to do with the todo extension?15:02
* smcginnis probably didn't read far enough back15:02
toabctlI guess there was a sphinx update which broke the todo15:02
vponomaryovright15:02
vponomaryovconfig opt that is used there is no more available15:02
vponomaryovhttp://logs.openstack.org/01/437501/3/check/gate-manila-docs-ubuntu-xenial/040e6f5/console.html#_2017-03-03_13_52_16_78657415:02
vponomaryovAttributeError: No such config value: todo_emit_warnings15:03
smcginnisBased on the output we were getting, the todo stuff was pretty much useless IMO. I'd recommend just removing it.15:03
openstackgerritTom Barron proposed openstack/manila master: Unblock gate failure on docs build  https://review.openstack.org/44115615:04
tbarronput a small finger back, appendix still removed15:04
gouthamrsurgeons wear red hats15:05
tbarrongouthamr: i wear my brown RDO hat ever since xyang said I looked like he who must not be named15:07
smcginnis:)15:07
tommylikehuhey vponomaryov, could you take a look at this patch: https://review.openstack.org/#/c/440155/ ?15:09
*** mtanino has joined #openstack-manila15:09
gouthamrtbarron: :D15:10
vponomaryovtbarron: it fails with other error now15:10
vponomaryovtbarron: have you tried it on your lab?15:10
tbarronvponomaryov: tox -edocs passes for me locally, that's what I tried15:12
*** ociuhandu has joined #openstack-manila15:12
*** tommylikehu has quit IRC15:13
*** tommylikehu has joined #openstack-manila15:13
vponomaryovtbarron: ~/manila/doc/source/devref/share.rst:21: ERROR: Unknown directive type "todo".15:14
tbarronvponomaryov: smcginnis I did say in my commit message that I also am cleaning up dangling cross-references but I'll pull those out and do them as a followup.15:14
smcginnistbarron: Oh, that's fine.15:15
vponomaryovtbarron: ~manila/doc/source/devref/fakes.rst:21: ERROR: Unknown directive type "todo".15:15
smcginnistbarron: Kind of skimmed over the commit message.15:15
tbarronvponomaryov: maybe I didn't clean up the workspace before trying the second version, will check15:15
vponomaryovtbarron: a billion of warnings now of that kind -> WARNING: Block quote ends without a blank line; unexpected unindent.15:16
tbarronvponomaryov: they don't break the doc build for me though15:17
vponomaryovtbarron: you already do "cleanup" ))15:17
tbarronvponomaryov: yeah, i'm not saying don't suppress or fix15:17
tbarronvponomaryov: but unblock gate first, then figure whether we care about the warning.  Do we?15:18
tbarronvponomaryov: I cleaned the dishes; you point out that the rest of the house is dirty.15:18
tbarronvponomaryov: good way to teach people not to clean the dishes :D15:19
vponomaryovtbarron: dirty? it is buried in RED WARNINGS )))15:19
bswartzwhere the warnings there before?15:20
tbarronso the red warnings are coming from the new version of sphinx, right?15:20
vponomaryovtbarron: could be15:20
tbarrondo we want to fix them or suppress them?15:20
tbarronvponomaryov: those warnings have nothing to do with removing the old manila todo extension15:21
vponomaryovtbarron: it is compatibility-fix15:21
vponomaryovtbarron: I would remove warnings too15:21
vponomaryovtbarron: the cause of them15:21
tbarronvponomaryov: go ahead, anyone in the family can help clean the house15:22
vponomaryovtbarron: I will add co-author15:22
vponomaryov))15:22
vponomaryov:-p15:22
tbarronvponomaryov: my plan was to ask the community whether we should fix the warnings or suppress them, then do a followup15:22
tbarronvponomaryov: you can make yourself main author, I just want to get gate unblocked15:22
vponomaryovtbarron: but not nice?15:23
vponomaryovugly and dirty? )15:23
bswartzsounds like it was ugly before15:23
tbarronvponomaryov: the house is dirty, agreed.  This patch doesn't *make* it dirty, it shows the dirt for what it is.15:23
*** ociuhandu has quit IRC15:27
tbarronbswartz: vponomaryov all ... Here is output from a run with this patch when doc/build has been cleared: http://paste.openstack.org/show/601342/15:29
vponomaryovtbarron: very strange, they do not appear from second attempt15:30
vponomaryovthey - errors and warnings15:30
tbarronvponomaryov: remove doc/builid15:30
bswartztbarron: looks like 90% of the ugliness is Hitachi's15:30
tbarrondoc/build before 2nd attempt15:30
bswartzo_O15:30
vponomaryovtbarron: yes, it solves, but why does it matter?15:30
bswartzis this some new kind of pep8 checker?15:31
tbarronvponomaryov: are you saying fix or suppress?15:31
vponomaryovtbarron: I am saying that "doc/build" should be refreshed15:31
bswartzoh it's like a style checker for doc strings15:31
tbarronvponomaryov: we probably should remove doc/build in tox.ini before doing the build, right?15:32
vponomaryovyes15:33
*** akapil has quit IRC15:34
gouthamrbswartz tbarron: Just came across this: https://review.openstack.org/#/c/433437/ -- exact concern as we had15:37
gouthamrstorage_availability_zone must really be for the storage, that way, we can configure more than one in one config file when using multi-backend.15:38
openstackgerritTom Barron proposed openstack/manila master: Unblock gate failure on docs build  https://review.openstack.org/44115615:40
tbarronwill look at some house cleanup next, just added the doc/build wipe15:40
bswartzgouthamr: this is a different thing15:41
bswartzs/this/that/15:41
bswartzit's related, but it's been part of the discussion a long time15:41
gouthamrbswartz: this is what i asked for, so we didn't need to setup another manila share service when using multi-backend and replication15:41
bswartzspecifically -- what if the cinder node and the storage controler aren't co-located in15:42
bswartzin the same failure domain15:42
gouthamryes, i recall the discussion we had regarding AZ for service vs AZ for storage15:43
gouthamrcurrently, they're coupled15:44
*** eharney has quit IRC15:48
*** eharney has joined #openstack-manila16:05
openstackgerritValeriy Ponomaryov proposed openstack/manila master: Unblock gate failure on docs build  https://review.openstack.org/44115616:21
vponomaryovtbarron: ^ I finished16:22
tbarronvponomaryov: looking16:22
* tbarron has been trying to figure out those unexpected unindent warnings16:23
*** porrua_ has quit IRC16:25
*** tommylikehu_ has quit IRC16:27
tbarronvponomaryov: our house is very clean, thank you.16:28
*** porrua has joined #openstack-manila16:28
tbarronbswartz: ganso gouthamr ^^^^^16:29
tbarronxyang1: markstur toabctl ^^ let's unblock gate now16:30
tbarronty16:32
*** porrua has quit IRC16:34
ganso:)16:35
smcginnisgouthamr: Not sure if this is interesting related to your earlier conversation: https://review.openstack.org/#/c/433437/16:39
gouthamrsmcginnis: yes it is! i was discussing with bswartz about needing that fix in manila too...16:40
smcginnisgouthamr: I think it makes sense now that multibackend is supported.16:40
bswartzsmcginnis: yes that change is a good one but I remain very upset about the abuses and misuses of "storage" availability zone vs compute availability zones16:41
bswartzI feel like communication to end users about these concepts is a mess16:42
smcginnisbswartz: Yeah, that's a whole 'nother issue.16:42
bswartztbarron vponomaryov: ty both for sorting out the docs job16:43
gouthamrsmcginnis: yes.. manila users hit the issue with the replication feature.. we recommend bringing up multiple share services with different config files..16:44
*** markstur has quit IRC16:44
gouthamrsmcginnis: turns out that isn't very flexible in certain distributions...16:45
gouthamrsmcginnis: it worked on devstack :D16:45
smcginnisgouthamr: Hah! :D16:45
smcginnisgouthamr: It's at a minimum a lot of extra work.16:46
gouthamrsmcginnis: true... and mostly a communication gap as bswartz points out... we'll fix it and document it.16:47
bswartzit's not a communication gap -- it's a design gap16:48
bswartzstorage availability zones don't don't actually have a *meaning* that's consistent across deployments16:49
bswartzas long as different people mean different things when they talk about it, no amount of communication will help16:49
*** crushil has quit IRC16:55
*** crushil has joined #openstack-manila16:56
*** faizy has quit IRC17:13
*** gouthamr has quit IRC17:15
*** gouthamr has joined #openstack-manila17:26
*** a-pugachev has quit IRC17:30
*** darrenc_ has joined #openstack-manila17:37
*** sage_ has joined #openstack-manila17:39
*** arnewiebalck_ has joined #openstack-manila17:42
*** vkmc- has joined #openstack-manila17:43
*** darrenc has quit IRC17:43
*** arnewiebalck has quit IRC17:43
*** csaba has quit IRC17:43
*** MikeG451 has quit IRC17:43
*** vkmc has quit IRC17:43
*** sage has quit IRC17:43
*** arnewiebalck_ is now known as arnewiebalck17:43
*** vkmc- is now known as vkmc17:43
*** vkmc has quit IRC17:43
*** vkmc has joined #openstack-manila17:43
*** csaba has joined #openstack-manila17:44
*** MikeG451 has joined #openstack-manila17:44
vponomaryovganso: stable/ocata requires this - > https://review.openstack.org/#/c/441253/17:48
gansovponomaryov: +1ed, but looking at LVM failure in my backport patch, it seems failures are related to LVM17:52
vponomaryovganso: no17:53
vponomaryovganso: it is completely the same as was in master17:53
vponomaryovganso: cannot connect/SSH because SSH service is not started on VM, because it hangs not having enough memory17:53
gansovponomaryov: failures in master did not even start the tests17:53
gansovponomaryov: oh the not having memory issue17:54
gansovponomaryov: I did not see that the image change was backported17:54
tbarronganso: the image just gets uploaded, irrespective of branch17:55
tbarronganso: then jobs running in gate for whatever branch pick it up17:55
gansotbarron: oh i see18:00
gansotbarron: thanks for clarifying18:01
*** breitz has quit IRC18:02
*** chlong__ has quit IRC18:04
*** lpetrut has quit IRC18:06
openstackgerritMerged openstack/manila master: Unblock gate failure on docs build  https://review.openstack.org/44115618:09
*** crushil has quit IRC18:15
*** crushil has joined #openstack-manila18:16
*** dustins is now known as dschoenb18:18
*** dschoenb is now known as dustins18:18
*** chlong__ has joined #openstack-manila18:20
*** breitz has joined #openstack-manila18:29
tbarrongouthamr: yeah, https://review.openstack.org/#/c/433437/ looks like what we were discussing.18:29
*** cknight has joined #openstack-manila18:44
*** cknight has quit IRC18:54
*** crushil has quit IRC18:54
*** crushil has joined #openstack-manila18:54
*** porrua has joined #openstack-manila18:55
*** lpetrut has joined #openstack-manila18:59
openstackgerritTom Barron proposed openstack/manila-ui master: Clean docs/build before building docs  https://review.openstack.org/44133919:04
openstackgerritTom Barron proposed openstack/python-manilaclient master: Clean docs/build before building docs  https://review.openstack.org/44134419:07
*** porrua has quit IRC19:28
*** chlong__ has quit IRC19:55
*** chlong__ has joined #openstack-manila20:09
*** dustins has quit IRC20:11
*** chlong__ has quit IRC20:57
*** dustins has joined #openstack-manila21:02
*** crushil has quit IRC21:05
*** crushil has joined #openstack-manila21:06
*** mtanino has quit IRC21:36
*** gouthamr has quit IRC22:01
openstackgerritVictoria Martinez de la Cruz proposed openstack/manila-image-elements master: Adds support for NFS Ganesha  https://review.openstack.org/41150022:02
*** eharney has quit IRC22:15
*** catintheroof has joined #openstack-manila22:29
*** crushil has quit IRC22:41
*** dustins has quit IRC22:48
openstackgerritOpenStack Proposal Bot proposed openstack/manila master: Updated from global requirements  https://review.openstack.org/43922222:51
openstackgerritOpenStack Proposal Bot proposed openstack/manila-ui master: Updated from global requirements  https://review.openstack.org/44087222:51
*** xyang1 has quit IRC23:11
*** david-lyle_ is now known as david-lyle23:29
*** david-lyle has quit IRC23:47
*** tommylikehu_ has joined #openstack-manila23:49
*** tommylikehu_ has quit IRC23:53
*** tommylikehu_ has joined #openstack-manila23:54
*** tommylikehu_ has quit IRC23:55
*** tommylikehu_ has joined #openstack-manila23:56

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