Friday, 2015-09-04

« Thursday, 2015-09-03 Index Saturday, 2015-09-05 »
*** cknight has quit IRC00:00
ganso_bswartz: the job finished... exactly the same error message00:05
openstackgerritBen Swartzlander proposed openstack/manila: Dummy change (do not merge)  https://review.openstack.org/22035400:06
bswartzDoh!00:06
bswartzforgot the most important part00:06
openstackgerritBen Swartzlander proposed openstack/manila: Dummy change (do not merge)  https://review.openstack.org/22035400:08
openstackgerritCsaba Henk proposed openstack/manila: glusterfs: directory mapped share layout  https://review.openstack.org/21529300:11
openstackgerritCsaba Henk proposed openstack/manila: glusterfs/common: refactor GlusterManager  https://review.openstack.org/21492100:11
openstackgerritCsaba Henk proposed openstack/manila: glusterfs*: factor out common parts  https://review.openstack.org/21446200:11
openstackgerritCsaba Henk proposed openstack/manila: glusterfs/layout: add layout base classes  https://review.openstack.org/21517200:11
openstackgerritCsaba Henk proposed openstack/manila: glusterfs: volume mapped share layout  https://review.openstack.org/21517300:11
openstackgerritCsaba Henk proposed openstack/manila: glusterfs-native: cut back on redundancy  https://review.openstack.org/21502100:11
*** sgotliv has quit IRC00:43
*** jasonsb has quit IRC00:45
bswartzganso_: indeed we're broken again00:52
*** BharatK has joined #openstack-manila00:55
bswartzganso_: it's something w/ devstack and manila00:56
*** mmartin781 has joined #openstack-manila00:57
*** mmartin78 has quit IRC00:58
ganso_bswartz: :\00:59
ganso_bswartz: I will spend the time coding more tempest tests01:00
ganso_bswartz: and fixing bugs, looks like cknight has found one01:00
*** mtanino has quit IRC01:02
*** tbarron_afk is now known as tbarron01:08
openstackgerritCsaba Henk proposed openstack/manila: glusterfs: directory mapped share layout  https://review.openstack.org/21529301:18
openstackgerritCsaba Henk proposed openstack/manila: glusterfs: volume mapped share layout  https://review.openstack.org/21517301:18
*** jasonsb has joined #openstack-manila01:45
*** xyang1 has quit IRC01:47
*** ChanServ changes topic to "The gate is broken again"02:37
*** kambiz has quit IRC02:42
*** kambiz has joined #openstack-manila02:45
*** bswartz has quit IRC02:49
*** bswartz has joined #openstack-manila02:50
*** BharatK has quit IRC03:00
*** cknight has joined #openstack-manila03:11
*** openstackgerrit_ has joined #openstack-manila03:11
*** cknight1 has joined #openstack-manila03:12
*** cknight has quit IRC03:15
*** bswartz has quit IRC03:28
*** tbarron has quit IRC03:28
*** tbarron has joined #openstack-manila03:32
*** bswartz has joined #openstack-manila03:34
*** akerr has joined #openstack-manila03:44
*** akerr has quit IRC03:44
*** openstackgerrit_ has quit IRC04:14
*** cknight1 has quit IRC04:37
*** deepakcs has joined #openstack-manila04:49
*** haomaiwa_ has quit IRC04:57
*** jasonsb has quit IRC05:10
*** jasonsb has joined #openstack-manila05:10
*** shausy has joined #openstack-manila05:17
*** shausy has quit IRC05:39
*** BharatK has joined #openstack-manila05:40
*** shausy has joined #openstack-manila05:40
*** nkrinner has joined #openstack-manila05:58
*** shausy has quit IRC06:00
*** shausy has joined #openstack-manila06:32
*** ociuhandu has quit IRC06:44
openstackgerritValeriy Ponomaryov proposed openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041507:00
*** vponomaryov1 has joined #openstack-manila07:03
vponomaryov1https://bugs.launchpad.net/manila/+bug/149213307:04
openstackLaunchpad bug 1492133 in Manila "manila devstack no more compatible" [Critical,In progress] - Assigned to Valeriy Ponomaryov (vponomaryov)07:04
vponomaryov1fix for our gates ^ https://review.openstack.org/#/c/220415/07:04
vponomaryov1mkoderer: ^07:05
*** lpetrut has joined #openstack-manila07:07
openstackgerritValeriy Ponomaryov proposed openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041507:39
mkoderervponomaryov1: nice one07:41
*** vponomaryov1 has quit IRC07:55
*** u_glide1 has joined #openstack-manila08:08
lpetrutvponomaryov1: thanks for sending the devstack plugin patch! I was just about to send a similar one as I bumped into the same issue on the CI08:35
*** izadorozhna has joined #openstack-manila08:51
*** akerr has joined #openstack-manila08:53
*** akerr_ has joined #openstack-manila08:54
*** akerr has quit IRC08:58
*** akerr_ has quit IRC09:04
*** rraja has joined #openstack-manila09:09
*** ociuhandu has joined #openstack-manila09:51
openstackgerritValeriy Ponomaryov proposed openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041510:01
openstackgerritValeriy Ponomaryov proposed openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041510:40
openstackgerritValeriy Ponomaryov proposed openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041510:41
openstackgerritValeriy Ponomaryov proposed openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041511:20
openstackgerritValeriy Ponomaryov proposed openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041511:34
*** shausy has quit IRC11:35
*** JoseMello has joined #openstack-manila11:50
openstackgerritValeriy Ponomaryov proposed openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041512:04
*** ociuhandu has quit IRC12:06
bswartzvponomaryov: why 7 patchsets for the devstack plugin?12:08
bswartzare you unsure that the current patch fixes the devstack problem?12:09
vponomaryovbswartz: devstack was switched to keystone v3, manilaclient keystone-v2-specific12:09
bswartzyeah I saw what the problem was last night when csaba pointed it out12:09
*** martyturner has joined #openstack-manila12:10
bswartzbut are you able to test the fixes with your devstack before pushing upstream?12:10
bswartzu_glide, u_glide1: pong12:11
vponomaryovbswartz: that is why I updated lots of times12:11
vponomaryovbswartz: found problem - uploaded changed thing a soon12:11
vponomaryovs/soon/ so on/12:11
bswartzare you confident that patchset 7 will work?12:11
*** ociuhandu has joined #openstack-manila12:12
vponomaryovbswartz: you mean number 8? =)12:12
vponomaryovwill answer in two-three minutes12:13
bswartzerr yes12:14
bswartzpatchset 812:14
vponomaryovbswartz: fail again, will be number 912:14
vponomaryovAuthorizationFailure: Authorization failed: The resource could not be found. (HTTP 404)12:15
vponomaryovfor operation of manila share type creation12:15
bswartzvponomaryov: why git review while you're still testing the patch? doesn't that waste jenkins resources?12:15
lpetrutHi guys. it seems to me that the method checking if a driver supports snapshots is flawed12:15
vponomaryovlpetrut: there are couple of bugs12:16
vponomaryovlpetrut: already in LP12:16
bswartzlpetrut: it autodetects, but your driver can override the detection I think12:16
vponomaryovlpetrut: https://bugs.launchpad.net/manila/+bug/1491345 and https://bugs.launchpad.net/manila/+bug/149134612:17
openstackLaunchpad bug 1491345 in Manila "'snapshot_support' attr is absent in share payload" [High,New] - Assigned to Valeriy Ponomaryov (vponomaryov)12:17
openstackLaunchpad bug 1491346 in Manila "lost import of translation func in snapshots API" [High,New] - Assigned to Valeriy Ponomaryov (vponomaryov)12:17
lpetrutGot it, I'll check it out. The idea is that it doesn't mean that if the class does not specifically override those methods, it does not support them. For example, in our case, those methods are just inherited from the generic driver.12:17
vponomaryovlpetrut: set this explicitly12:17
vponomaryovlpetrut: what is the problem?12:17
vponomaryovlpetrut: in that case no bug12:18
lpetrutgreat, I was just wondering if the drivers are 'allowed' to do this12:18
vponomaryovlpetrut: sure12:18
*** gouthamr has joined #openstack-manila12:23
*** dzamboni has joined #openstack-manila12:26
*** tpsilva has joined #openstack-manila12:31
*** alyson_ has joined #openstack-manila12:39
openstackgerritValeriy Ponomaryov proposed openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041512:41
*** porrua has joined #openstack-manila12:41
openstackgerritJulia Varlamova proposed openstack/manila-ui: Add availability zones support to manila-ui  https://review.openstack.org/21667412:43
*** aorourke_ has quit IRC12:52
*** dustins has joined #openstack-manila12:57
openstackgerritValeriy Ponomaryov proposed openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041512:57
*** nkrinner has quit IRC13:06
*** nkrinner has joined #openstack-manila13:08
openstackgerritValeriy Ponomaryov proposed openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041513:14
*** esker has quit IRC13:15
*** esker has joined #openstack-manila13:15
*** jasonsb has quit IRC13:20
*** esker has quit IRC13:20
*** haomaiwang has joined #openstack-manila13:21
vponomaryovto_all: our gates blocker fix ready for review - https://review.openstack.org/#/c/220415/1113:28
*** cknight has joined #openstack-manila13:30
openstackgerritPetrut Lucian proposed openstack/manila: Enhance base driver checking if a method was implemented  https://review.openstack.org/22051113:42
lpetrutvponomaryov: How about this? https://review.openstack.org/#/c/220511/13:43
openstackgerritAlyson proposed openstack/manila: Adds retry function to HNAS driver  https://review.openstack.org/21985113:43
*** eharney has joined #openstack-manila13:44
lpetrutvponomaryov: It's simpler and this way, it may be used in other scenarios.13:44
vponomaryov lpetrut: need ad unit tests13:45
vponomaryov lpetrut: to cover case of inheritance of a driver by another driver13:45
vponomaryovs/ad/add/13:45
lpetrutsure, just wanted to make sure you're ok with this approach :)13:46
vponomaryovlpetrut: also "hasattr(parent, method)" should stay13:46
lpetrutwhy?13:47
lpetrutoh, got it13:47
vponomaryovlpetrut: method can be absent13:47
vponomaryovlpetrut: usign this we have more strict logic13:47
lpetrutwell, we can simply check like hasattr(self, method). we don't need to think about the parent13:48
openstackgerritValeriy Ponomaryov proposed openstack/manila: Add possibility to setup password for generic driver  https://review.openstack.org/21978813:48
vponomaryovlpetrut: it was copy-paste13:49
vponomaryovlpetrut: main idea in "hasattr()" thing13:49
lpetrutah, got it. great, I'll update this13:50
lpetrutthanks!13:50
vponomaryovbswartz, cknight: updated password setting for generic driver - https://review.openstack.org/219788 and set on top of devstack fix - https://review.openstack.org/#/c/220415/13:50
vponomaryovbswartz, cknight: so, all existing commit changes can be rechecked13:51
vponomaryovdevstack fix works, proof - https://jenkins07.openstack.org/job/gate-manila-tempest-dsvm-neutron/1079/console13:51
ganso_vponomaryov: :D13:52
vponomaryovit was third day of rushing gates, and they finally overcame it!13:53
*** rraja has quit IRC13:53
u_glide1thanks vponomaryov!13:55
u_glide1fix works on my local lab13:55
*** esker has joined #openstack-manila13:56
*** esker has quit IRC13:56
*** esker has joined #openstack-manila13:56
*** eharney has quit IRC13:57
*** eharney has joined #openstack-manila13:58
*** haomaiwang has quit IRC14:01
*** haomaiwang has joined #openstack-manila14:01
ameadeu_glide1: https://review.openstack.org/#/c/215343/14:04
ameadeu_glide1: I had put in a comment about my thoughts on your 2 points14:04
*** ayma has joined #openstack-manila14:06
*** deepakcs has quit IRC14:06
*** xyang1 has joined #openstack-manila14:12
ganso_vponomaryov: I submitted a simple tempest test yesterday https://review.openstack.org/#/c/220278/ it is running on jenkins now14:12
ganso_vponomaryov: I remember jenkins job timing out some time ago, with this test it can take around 2-3 minutes longer14:13
ganso_vponomaryov: is it plausible to increase job timeout?14:13
*** nkrinner has quit IRC14:15
*** eharney has quit IRC14:15
u_glide1ameade: I have responded in review14:18
u_glide1ameade: if you have any additional questions, please fill free to ask14:19
vponomaryovganso_: job timeouts are set in CI config14:20
ganso_vponomaryov: yes, can they be increased?14:21
vponomaryovganso_:yes, timeout can be increased, but it can be dissallowed14:22
vponomaryovganso_: need to try14:22
vponomaryovganso_: but why do you think we need it?14:22
vponomaryovganso_: our tests running in parallel14:22
*** mtanino has joined #openstack-manila14:22
openstackgerritMerged openstack/puppet-manila: acceptance: enable debug & verbosity for OpenStack logs  https://review.openstack.org/21350914:23
ganso_vponomaryov: I am not sure yet if we will need it, I just wanted to know if it was possible to increase, we will need to observe how long it will take with this test and the one and I want to submit, which is similar to your scenario tests that create instance, mount, write data... those take a very long time14:24
ganso_vponomaryov: btw I am facing this problem running the scenario test:14:25
ganso_Traceback (most recent call last):14:25
ganso_      File "tempest/scenario/test_share_basic_ops.py", line 53, in setUp14:25
ganso_        if img["name"] == CONF.share.image_with_share_tools:14:25
ganso_    TypeError: string indices must be integers14:25
vponomaryovganso_: it is with tempest plugin ?14:25
*** eharney has joined #openstack-manila14:26
vponomaryovganso_: I mean recent change14:26
ganso_vponomaryov: I printed the output and it seems the response is a bit different, like if the test code was outdated14:26
ganso_vponomaryov: no, without recent change, I believe tempest plugin has not merged14:26
vponomaryovganso_: strange, but CI does not have such problem14:26
ganso_vponomaryov: yea, I am waiting for a full run to see if CI will have this problem14:27
ganso_vponomaryov: it is very weird14:27
vponomaryovganso_: oh!14:29
vponomaryovganso_: it works when result is 200 Ok14:29
vponomaryovganso_: but if you get some server error, you get there not a list, but an error14:29
vponomaryovganso_: so, you have some problem getting list of images, service answers with error14:29
vponomaryovganso_: so, just try get list of images by hands or print its output14:30
ganso_vponomaryov: but I got 200 OK http://paste.openstack.org/show/445459/14:30
ganso_vponomaryov: o_O14:31
vponomaryovganso_: you sure that you used same API version as Tempest?14:31
ganso_vponomaryov: you mean like my image-client version is outdated or tempest is outdated?14:32
vponomaryovganso_: yes14:32
ganso_vponomaryov: it is possible, let me try a git pull on both14:32
vponomaryovganso_: print returning result in tempest14:32
vponomaryovganso_: because I suspect that "img" in your case is string of an one symbol14:33
vponomaryovganso_: and "images" is string of many symbols with error description14:33
ganso_vponomaryov: updated glance, python-glanceclient, tempest, restarted services... still same error14:40
vponomaryovshow result14:41
ganso_vponomaryov: looks like I need to updated my lab14:41
ganso_vponomaryov: hold on a sec14:41
vponomaryovganso_: exactly var "images" what is there14:41
vponomaryovthat on line 5214:41
vponomaryovline 50, images = self.images_client.list_images()14:41
*** _crobertsrh is now known as crobertsrh14:44
ganso_vponomaryov: http://paste.openstack.org/show/445482/14:45
vponomaryovganso_: why do you use "str" translation?14:47
vponomaryovganso_: print type(images)14:47
vponomaryovganso_: print images14:47
vponomaryovprint images[0]14:48
ganso_vponomaryov: I like to see dictionary contents in raw format14:48
vponomaryovgreat, but you have error exactly with string14:48
ganso_vponomaryov: sorry, I am not following, what error?14:49
vponomaryovganso_: iterating "images" you get strings14:50
vponomaryovganso_: that is unexpected14:50
vponomaryovganso_: but "images" should be a list of dics14:50
vponomaryovs/dics/dicts/14:50
openstackgerritPetrut Lucian proposed openstack/manila: Enhance base driver checking if a method was implemented  https://review.openstack.org/22051114:52
openstackgerritPetrut Lucian proposed openstack/manila: Enhance base driver checking if a method was implemented  https://review.openstack.org/22051115:00
*** haomaiwang has quit IRC15:01
*** haomaiwang has joined #openstack-manila15:01
ganso_vponomaryov: looks like scenario tests are not run in generic driver's CI15:04
ganso_vponomaryov: also, looks like it is missing libraries required to mount: http://logs.openstack.org/78/220278/2/check/gate-manila-tempest-dsvm-neutron-multibackend/221cee2/logs/screen-m-shr.txt.gz15:04
vponomaryov ganso_: scenario tests have its own job15:06
vponomaryovgate-manila-tempest-dsvm-neutron-scenario15:06
ganso_vponomaryov: thanks, I did not notice they were moved15:07
vponomaryovganso_: moved?15:07
vponomaryovganso_: they always were separate15:07
ganso_vponomaryov: some time ago they were in the same job, if I am not mistaken15:07
*** mtanino has quit IRC15:07
vponomaryovganso_: mistaken15:07
ganso_vponomaryov: yea, sorry... I confused with HDS CI which was running in the same job15:08
ganso_vponomaryov: does the CI has nfs libraries required to mount a NFS share?15:09
ganso_ s/has/have15:09
vponomaryovganso_: I guess thing that fails is only migration tests, because you try to use "nfs" things on host of devstack15:09
ganso_vponomaryov: yes, it is intended15:10
vponomaryovganso_: need to install it with our devstack15:10
vponomaryovganso_: but need also add env var set enables/disables it15:10
vponomaryovganso_: both, migration tests and installation of NFS stuff to host machine15:10
ganso_vponomaryov: it is coming in next patch, added "migration_enabled" in CONF, set in tempest.conf15:11
bswartzvponomaryov: 220415 is nearly through the gate15:11
ganso_vponomaryov: but default is true15:11
vponomaryov ganso_: default should be False and defined to True only for CI15:12
vponomaryov ganso_: in pre_test_hook15:12
vponomaryov ganso_: or post_test_hook15:12
vponomaryov ganso_: or in devstack15:12
ganso_vponomaryov: I set it to true because if I did set to false I know I would not be able to test on Jenkins without additional changes on CI, like you mentioned15:12
vponomaryov=)15:12
vponomaryovganso_: no15:12
vponomaryovganso_: only changes to our repo is required15:13
ganso_vponomaryov: no changes to CI?15:13
vponomaryovganso_: only manila repo15:13
vponomaryovganso_: https://github.com/openstack/manila/blob/master/contrib/ci/post_test_hook.sh15:13
ganso_vponomaryov: great! I will include the change in next patch15:15
vponomaryovbswartz: https://review.openstack.org/#/c/219788/ requires one more update, sorry15:15
vponomaryovbswartz: wrong name of opt defined15:16
bswartzvponomaryov: it passed jenkins though15:16
vponomaryovbswartz: no15:16
bswartzvponomaryov: you saying it got lucky again?15:16
vponomaryovbswartz: scenarios test failed as was15:16
bswartzI'm not concerned about the scenario tests as much as working around the unstable neutron bug15:17
openstackgerritValeriy Ponomaryov proposed openstack/manila: Add possibility to setup password for generic driver  https://review.openstack.org/21978815:18
openstackgerritMerged openstack/manila: Use devstack functions for registering Manila  https://review.openstack.org/22041515:18
vponomaryovbswartz: I mean scenario tests are criterion15:18
vponomaryovbswartz: they shown that password was not applied15:18
vponomaryovbswartz: now it is ok15:18
bswartzso we should not merge 219788 until we see scenario tests pass?15:18
vponomaryovbswartz: and scenario tests are expected to pass guaranteed15:19
vponomaryovbswartz: they will15:19
vponomaryovbswartz: and we can15:19
vponomaryovbswartz: patch-set 9 sets proper opts for passwords15:19
vponomaryovbswartz: I just did not see this commit passing gates because gates were broken15:20
bswartzI'll watch that one in zuul15:20
vponomaryovbswartz: gates unblocked!15:20
vponomaryovleft this one15:20
bswartzzuul hasn't reacted yet to you patchset 915:20
bswartzthere it goes15:21
bswartzit should remove patchset 8 from the gate momentarily and start testing 915:21
bswartzI'll watch outcome of scenario test15:21
vponomaryovbswartz: also, we can merge it even if scenario tests fail, but for some other reason15:22
bswartzlol15:22
vponomaryovbswartz: not related to target bug15:22
bswartzlet's hope that doesn't occur15:22
vponomaryov))15:22
vponomaryovhope15:22
bswartzI was hoping we could make scenario tests vote later today15:22
bswartzif they are once again stable, then it is the right thing to do15:23
bswartzalthough just for our own sanity it would be wise to hold off until features merge15:23
vponomaryovbswartz: but it is infra-team-dependent thing15:26
vponomaryovbswartz: to make job voting15:27
bswartzyes I know15:27
bswartzlet's do it next wek15:27
bswartzassuming tests pass15:27
vponomaryovbswartz: next week? but when is deadline for merging features?15:29
bswartzI haven't set it yet15:30
bswartzI need to see something else merge15:30
bswartzafter 219788 merges I'll pick a time and announce on ML15:30
bswartzit will probably be Monday late or Tuesday early15:31
bswartzdustins: ping15:38
*** pgbridge has joined #openstack-manila15:43
*** mtanino has joined #openstack-manila15:44
*** jasonsb has joined #openstack-manila15:49
*** BharatK has quit IRC15:51
openstackgerritValeriy Ponomaryov proposed openstack/manila: Add possibility to setup password for generic driver  https://review.openstack.org/21978815:57
*** haomaiwang has quit IRC16:01
*** haomaiwang has joined #openstack-manila16:01
*** gouthamr has quit IRC16:02
openstackgerritPetrut Lucian proposed openstack/manila: Enhance base driver checking if a method was implemented  https://review.openstack.org/22051116:04
openstackgerritValeriy Ponomaryov proposed openstack/manila: Add share hooks  https://review.openstack.org/20166916:04
*** gouthamr has joined #openstack-manila16:14
*** gouthamr_ has joined #openstack-manila16:15
*** gouthamr has quit IRC16:19
*** timcl has quit IRC16:21
*** esker has quit IRC16:29
openstackgerritMerged openstack/manila: Updated from global requirements  https://review.openstack.org/21949416:32
dustinsbswartz: pong16:34
*** bswartz has quit IRC16:37
vponomaryovbswartz: are you OK merging Windows driver without passed tests for share creation?16:40
*** timcl has joined #openstack-manila16:40
*** eharney has quit IRC16:47
vponomaryovcsaba: it is time to rebasse your https://review.openstack.org/#/c/214462/ against master16:48
*** eharney has joined #openstack-manila16:48
ganso_vponomaryov: what job runs DHSS false in CI?16:52
vponomaryovganso_: gate-manila-tempest-dsvm-neutron-no-share-servers16:52
ganso_vponomaryov: thanks16:52
ganso_vponomaryov: do you think it is a good idea to create a multibackend variant to run migration tests under DHSS = false?16:53
vponomaryovganso_: we need two backends for migration tests16:58
vponomaryovganso_: it is exactly multiback-end16:58
ganso_vponomaryov: gate-manila-tempest-dsvm-neutron-no-share-servers is single backend16:59
ganso_vponomaryov: http://logs.openstack.org/78/220278/2/check/gate-manila-tempest-dsvm-neutron-no-share-servers/55c2116/console.html17:00
vponomaryovganso_: I know17:00
vponomaryovganso_: I am ok if it is multibackend17:00
*** haomaiwang has quit IRC17:01
*** haomaiwang has joined #openstack-manila17:01
ganso_vponomaryov: but change it to multibackend or create one more job?17:01
*** openstackgerrit_ has joined #openstack-manila17:02
vponomaryov ganso_: one more, I guess, will be overkill17:08
ganso_vponomaryov: I agree17:09
ganso_vponomaryov: I am looking at editing post_hook file, my change would be something like "if multibackend is enabled, enable migration", this change would be the same as using multibackend config17:10
vponomaryovganso_: there is similar tests, see https://github.com/openstack/manila/blob/master/contrib/tempest/tempest/api/share/admin/test_multi_backend.py17:11
ganso_vponomaryov: it makes sense to move my test code to that class, right?17:12
vponomaryovganso_: no, this one tests multibackendency itself17:12
vponomaryovganso_: it is example of dependency for having several backends17:13
csabavponomaryov: OK17:15
*** eharney has quit IRC17:15
*** ayma has quit IRC17:16
*** ayma has joined #openstack-manila17:16
vponomaryovganso_: about your issue with scenario tests17:17
vponomaryovganso_: I suspect you use latst tempest17:18
csabavponomaryov: do we still need to rebase over https://review.openstack.org/219788 / depend on Ibfd83b6abdfeec328019246a372363cada53869e ?17:18
vponomaryovs/latst/latest/17:18
ganso_vponomaryov: I saw this line: # Go to Tempest dir and checkout stable commit to avoid possible17:18
ganso_# incompatibilities for plugin stored in Manila repo.17:18
ganso_TEMPEST_COMMIT="489f5e62"  # 15 June, 201517:18
ganso_vponomaryov: so I need to revert my tempest, latest has this error, correct?17:19
vponomaryovcsaba: bswartz was recommending to use "Depends-On: %change-id%" things, not rebase17:19
vponomaryov ganso_: it is incompatibility, not error17:19
vponomaryov ganso_: yes, need to use same thing as Ci does17:19
csabavponomaryov: OK, so as of the current resubmission atop of master, should I have any "Depends-On: %change-id%" spec?17:20
*** bswartz has joined #openstack-manila17:21
vponomaryovcsaba: I do not insist, you can, it will allow your change avoid instability bug17:21
openstackgerritMerged openstack/manila: Add Windows SMB share driver  https://review.openstack.org/20015417:21
ganso_vponomaryov: ok, thanks, I will retest, I was about to run devstack again17:22
ganso_vponomaryov: please let me know when CI host has NFS libs so I can recheck my patch17:22
vponomaryovganso_: ?17:22
ganso_vponomaryov: I also got a newer patch ready but there is no point in submitting if it will fail due to nfs libraries missing17:23
ganso_vponomaryov: my tempest test does not run in Jenkins because devstack host does not have NFS libraries installed17:23
vponomaryovganso_: so update %manila%/devstack/plugin.sh file17:23
ganso_vponomaryov: or should I "sudo apt-get install nfs-common" from post_test_hook?17:23
ganso_vponomaryov: oh, that can be done from there17:24
ganso_vponomaryov: nice, thanks17:24
vponomaryovganso_: but keep in mind that not only ubuntu can be used17:24
vponomaryovganso_: there are several distros17:24
vponomaryovganso_: in CI we use only ubuntu - yes, but your update should consider it17:24
vponomaryovganso_: and try install packages considering host distro17:25
openstackgerritCsaba Henk proposed openstack/manila: glusterfs: directory mapped share layout  https://review.openstack.org/21529317:25
openstackgerritCsaba Henk proposed openstack/manila: glusterfs/common: refactor GlusterManager  https://review.openstack.org/21492117:25
openstackgerritCsaba Henk proposed openstack/manila: glusterfs*: factor out common parts  https://review.openstack.org/21446217:25
openstackgerritCsaba Henk proposed openstack/manila: glusterfs/layout: add layout base classes  https://review.openstack.org/21517217:25
openstackgerritCsaba Henk proposed openstack/manila: glusterfs: volume mapped share layout  https://review.openstack.org/21517317:25
openstackgerritCsaba Henk proposed openstack/manila: glusterfs-native: cut back on redundancy  https://review.openstack.org/21502117:25
*** ChanServ changes topic to "The gate is unbroken. New deadline for features: 1200 UTC 8/8"17:26
bswartzderp17:26
*** ChanServ changes topic to "The gate is fixed! New deadline for features: 1200 UTC 8/8"17:26
bswartzfixed sounds better than unbroken17:26
csababswartz: whats 8/8 ? do you mean it's the date with a one-off typo?17:27
ganso_bswartz: lol17:28
ganso_deadline was 1 month ago, we are all doomed17:29
bswartzDOH17:29
*** ChanServ changes topic to "The gate is fixed! New deadline for features: 1200 UTC 8 Sept"17:30
*** timcl1 has joined #openstack-manila17:31
*** timcl has quit IRC17:31
*** eharney has joined #openstack-manila17:31
*** esker has joined #openstack-manila17:34
*** esker has quit IRC17:34
*** esker has joined #openstack-manila17:35
vponomaryovbswartz: we are about to be broken again, see - https://review.openstack.org/#/c/220532/17:38
vponomaryovbswartz; so, I am going to upload preventing change17:39
vponomaryovbswartz: to be compatible with any case17:39
openstackgerritValeriy Ponomaryov proposed openstack/manila: Make our devstack plugin independt from default Identity API version  https://review.openstack.org/22060117:44
openstackgerritValeriy Ponomaryov proposed openstack/manila: Make devstack plugin independent from default Identity API version  https://review.openstack.org/22060117:45
*** timcl1 has quit IRC17:45
*** crobertsrh has quit IRC17:45
*** crobertsrh has joined #openstack-manila17:46
openstackgerritCsaba Henk proposed openstack/manila: glusterfs: directory mapped share layout  https://review.openstack.org/21529317:48
*** crobertsrh has quit IRC17:48
vponomaryovbswartz, cknight, markstur, toabctl, xyang1: please look at https://review.openstack.org/220601 - it is preventing commit from our gates breakage17:49
vponomaryovbswartz, cknight, markstur, toabctl, xyang1:  https://review.openstack.org/#/c/220532/ is our time bomb17:49
vponomaryovu_glide: ^17:52
*** aorourke has joined #openstack-manila17:54
vponomaryovalso stability fix can be merged - https://review.openstack.org/#/c/219788/17:56
*** devlaps has joined #openstack-manila17:56
vponomaryov toabctl, markstur: thatnks for fast action, also ^17:56
*** haomaiwang has quit IRC18:01
*** timcl has joined #openstack-manila18:01
*** haomaiwang has joined #openstack-manila18:01
ganso_cknight: Could you please remove -1 from https://review.openstack.org/#/c/179803/8, the change is included in https://review.openstack.org/#/c/179790/18:01
vponomaryovbswartz, cknight, markstur, toabctl, xyang1: sorry to bother you with lots of things, but https://review.openstack.org/#/c/219788/ is used as dependency for plenty of commits and ready for merge18:01
*** ociuhandu has quit IRC18:02
marksturvponomaryov, No way. Really?18:02
marksturvponomaryov, :)   Hard to believe anything gets thru Jenkins today18:03
marksturvponomaryov, Thanks for hammering away at it18:03
ganso_markstur: lol18:03
toabctl:)18:03
toabctlvponomaryov: on the way to the gate18:03
cknightganso_: Yes18:04
vponomaryovthanks!18:04
cknightvponomaryov: good to see scenario tests passing, thanks!18:04
vponomaryovcknight: "he is alive!"18:05
marksturdevstack is dead.  Long live devstack.18:05
cknightvponomaryov: :-)  And you're probably half asleep.  Thanks for working late to fix us.18:05
vponomaryovcknight: yes, a little bit =) Three "funny"days18:07
vponomaryovour treat is on last lap - https://review.openstack.org/#/c/220532/18:09
vponomaryovs/treat/threat/18:09
marksturI thought "treat" was sarcastic18:10
ganso_markstur: I thought so too lol18:10
vponomaryovas in templates - "emphasize correct one" =)18:12
openstackgerritMerged openstack/manila: devstack plug-in to reflect new manila-ui plug-in  https://review.openstack.org/21991118:15
cknightganso_: There was one string fix I was looking for in https://review.openstack.org/#/c/179803/818:16
ganso_cknight: sure, in a sec!18:17
openstackgerritRodrigo Barbieri proposed openstack/python-manilaclient: Add Share Migration support to Manila Client  https://review.openstack.org/17980318:19
cknightganso_: Thanks, looks good.18:20
bswartzvponomaryov, u_glide, cknight, markstur, toabctl, xyang1: please remember that when merging the chains of commits for CGs and GlusterFS, it's better to workflow them in reverse order, so they all go through the gate at one time18:20
ganso_cknight: great! thanks!18:21
*** lpetrut has quit IRC18:25
*** timcl1 has joined #openstack-manila18:31
*** timcl has quit IRC18:32
*** ayma has quit IRC18:35
openstackgerritMerged openstack/manila: Remove instances of "infinite" capacity from Manila  https://review.openstack.org/21415518:36
xyang1bswartz: If A depends on B, you want to workflow A first?  Is that what you mean by reverse order?18:48
bswartzxyang1: yes18:48
xyang1bswartz: That will fail to merge though18:49
bswartzzuul won't actually do anything until the earliest change in the chain gets workflowed18:49
bswartzit knows to wait18:49
xyang1bswartz: Ok18:49
bswartzif you workflow A first, then B, they both go into the gate together18:49
openstackgerritMerged openstack/manila: Add possibility to setup password for generic driver  https://review.openstack.org/21978818:52
*** haomaiwang has quit IRC19:01
*** 5EXAB2JHN has joined #openstack-manila19:01
*** resker has joined #openstack-manila19:06
*** esker has quit IRC19:10
openstackgerritClinton Knight proposed openstack/manila: Add functional tests for Manila consistency groups  https://review.openstack.org/21534619:10
openstackgerritClinton Knight proposed openstack/manila: Consistency groups in NetApp cDOT drivers  https://review.openstack.org/21534719:10
openstackgerritClinton Knight proposed openstack/manila: Scheduler changes for consistency groups  https://review.openstack.org/21534419:10
openstackgerritClinton Knight proposed openstack/manila: Add Consistency Groups API  https://review.openstack.org/21534519:10
openstackgerritClinton Knight proposed openstack/manila: Add DB changes for consistency-groups  https://review.openstack.org/21534319:10
openstackgerritClinton Knight proposed openstack/manila: Consistency Group Support for the Generic Driver  https://review.openstack.org/21989119:10
*** gouthamr_ has quit IRC19:10
*** ociuhandu has joined #openstack-manila19:12
*** vponomaryov has quit IRC19:12
*** lpetrut has joined #openstack-manila19:17
cknightxyang1: ^^ We've addressed your comments in 215347 and 219891.  Alex is still working on some of the others.19:23
*** porrua has quit IRC19:23
xyang1cknight: Sure, I am taking a look19:24
cknightxyang1: Thanks.  Alex will have his stuff up shortly.19:24
xyang1cknight: I added some comments in 215343, did you see it?19:25
cknightxyang1: Yes.  ameade did the core work for CG, and he's responding to comments from you and u_glide now.19:26
xyang1cknight: Ok19:26
cknightxyang1: Igor asked for alembic unit tests, so that's taking a bit longer.19:26
openstackgerritMerged openstack/manila: Make devstack plugin independent from default Identity API version  https://review.openstack.org/22060119:40
ameadexyang1: https://review.openstack.org/#/c/215345/4/manila/api/v1/cgsnapshots.py trying to think of what messages to add here, the defaults aren't good enough?19:43
xyang1ameade: Which line?19:43
ameadexyang1: as per your comments19:44
xyang1ameade: Let me check19:44
xyang1ameade: Let me add some msg there, just to give more context of the exception19:46
ameadexyang1: thanks, i'm almost done with everything then i'll get new versions up19:47
xyang1ameade: Ok19:47
xyang1ameade: I added suggested msg there19:50
ameadexyang1: thanks, having a mental block atm19:50
xyang1ameade: :)19:50
*** lpetrut has quit IRC19:53
*** 5EXAB2JHN has quit IRC20:01
*** haomaiwang has joined #openstack-manila20:01
*** openstackgerrit_ has quit IRC20:03
*** openstackgerrit_ has joined #openstack-manila20:04
*** dzamboni has quit IRC20:12
*** cknight1 has joined #openstack-manila20:13
*** eduardo has quit IRC20:13
*** xyang1_ has joined #openstack-manila20:14
openstackgerritMerged openstack/puppet-manila: acceptance: bump trusty packaging to trusty-proposed/liberty  https://review.openstack.org/21983120:16
*** xyang1 has quit IRC20:16
*** cknight has quit IRC20:16
*** eikke has quit IRC20:16
*** xyang1_ is now known as xyang120:16
*** eikke has joined #openstack-manila20:16
*** eharney has quit IRC20:17
*** dustins has quit IRC20:21
*** dustins has joined #openstack-manila20:22
*** timcl1 has quit IRC20:26
*** lpetrut has joined #openstack-manila20:41
*** openstackgerrit_ has quit IRC20:47
*** openstackgerrit_ has joined #openstack-manila20:47
*** openstackgerrit_ has quit IRC20:48
*** openstackgerrit_ has joined #openstack-manila20:48
*** eharney has joined #openstack-manila20:50
*** openstackgerrit_ has quit IRC20:53
*** bswartz has quit IRC20:55
*** lpetrut has quit IRC20:59
*** jasonsb has quit IRC21:00
*** haomaiwang has quit IRC21:01
*** jasonsb has joined #openstack-manila21:01
*** haomaiwang has joined #openstack-manila21:01
*** mriedem has joined #openstack-manila21:03
*** tpsilva has quit IRC21:03
mriedemanyone seen this? http://logs.openstack.org/34/219934/1/check/gate-manilaclient-dsvm-neutron-functional/d072339/logs/devstacklog.txt.gz#_2015-09-04_03_30_21_01021:03
mriedemopenstack service create: error: unrecognized arguments: --type=share21:03
mriedemah https://bugs.launchpad.net/manila/+bug/149213321:05
openstackLaunchpad bug 1492133 in Manila "manila devstack plugin no more compatible" [Critical,Fix committed] - Assigned to Valeriy Ponomaryov (vponomaryov)21:05
*** mriedem has left #openstack-manila21:07
openstackgerritSebastien Badia proposed openstack/puppet-manila: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987221:16
*** dustins has quit IRC21:16
*** bswartz has joined #openstack-manila21:17
*** jasonsb_ has joined #openstack-manila21:18
*** mmartin78 has joined #openstack-manila21:18
*** mmartin781 has quit IRC21:19
openstackgerritMerged openstack/manila: glusterfs*: factor out common parts  https://review.openstack.org/21446221:19
*** jasonsb has quit IRC21:21
*** cknight1 has quit IRC21:34
*** cknight has joined #openstack-manila21:34
*** xyang1 has quit IRC21:35
ganso_cknight: hi, you are probably very busy with CG, I would like to remind you to send me the Share migration bug details when you get some spare time. Thanks21:48
cknightganso_: Hi, Rodrigo.  Yes, and yes!21:49
openstackgerritMerged openstack/manila: Add share hooks  https://review.openstack.org/20166921:59
*** JoseMello has quit IRC21:59
*** haomaiwang has quit IRC22:01
*** 17SADHVEF has joined #openstack-manila22:01
*** alyson_ has quit IRC22:10
*** martyturner has quit IRC22:13
*** marcusvrn_ has quit IRC22:35
*** 17SADHVEF has quit IRC23:01
*** haomaiwang has joined #openstack-manila23:01
*** eharney has quit IRC23:11
*** jasonsb_ has quit IRC23:17
openstackgerritClinton Knight proposed openstack/manila: Add functional tests for Manila consistency groups  https://review.openstack.org/21534623:52
openstackgerritClinton Knight proposed openstack/manila: Consistency groups in NetApp cDOT drivers  https://review.openstack.org/21534723:52
openstackgerritClinton Knight proposed openstack/manila: Scheduler changes for consistency groups  https://review.openstack.org/21534423:52
openstackgerritClinton Knight proposed openstack/manila: Add Consistency Groups API  https://review.openstack.org/21534523:52
openstackgerritClinton Knight proposed openstack/manila: Add DB changes for consistency-groups  https://review.openstack.org/21534323:52
openstackgerritClinton Knight proposed openstack/manila: Consistency Group Support for the Generic Driver  https://review.openstack.org/21989123:52
« Thursday, 2015-09-03 Index Saturday, 2015-09-05 »

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