| openstackgerrit | OpenStack Proposal Bot proposed openstack/manila: Updated from global requirements https://review.openstack.org/221930 | 00:05 |
|---|---|---|
| *** akerr has joined #openstack-manila | 00:14 | |
| *** akerr has quit IRC | 00:14 | |
| *** u_glide has joined #openstack-manila | 00:17 | |
| openstackgerrit | Merged openstack/manila: Allow service image download to be skipped https://review.openstack.org/205222 | 00:44 |
| *** u_glide has quit IRC | 00:48 | |
| *** rushil has quit IRC | 01:00 | |
| *** cknight has joined #openstack-manila | 01:02 | |
| *** cknight1 has joined #openstack-manila | 01:04 | |
| *** cknight has quit IRC | 01:06 | |
| *** bill_az_ has quit IRC | 01:10 | |
| *** bswartz has quit IRC | 01:15 | |
| *** bswartz has joined #openstack-manila | 01:16 | |
| *** baojg has joined #openstack-manila | 01:17 | |
| *** baojg has quit IRC | 01:17 | |
| *** baojg has joined #openstack-manila | 01:19 | |
| *** haomaiwang has joined #openstack-manila | 01:29 | |
| *** haomaiwang has quit IRC | 02:01 | |
| *** haomaiwang has joined #openstack-manila | 02:01 | |
| *** jasonsb has joined #openstack-manila | 02:21 | |
| *** Zhongjun_ has joined #openstack-manila | 02:28 | |
| *** cknight1 has quit IRC | 02:52 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwa_ has joined #openstack-manila | 03:01 | |
| openstackgerrit | Merged openstack/manila: Fix description in Huawei driver https://review.openstack.org/221176 | 03:34 |
| *** bill_az has quit IRC | 03:59 | |
| *** haomaiwa_ has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-manila | 04:01 | |
| *** baojg has quit IRC | 04:02 | |
| *** esker has joined #openstack-manila | 04:24 | |
| *** ashokchandran has quit IRC | 04:31 | |
| *** ashokchandran has joined #openstack-manila | 04:32 | |
| *** esker has quit IRC | 04:33 | |
| *** Zhongjun_ has quit IRC | 04:37 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwa_ has joined #openstack-manila | 05:02 | |
| *** esker has joined #openstack-manila | 05:11 | |
| *** sgotliv_ has joined #openstack-manila | 05:11 | |
| *** esker has quit IRC | 05:19 | |
| *** shausy has joined #openstack-manila | 05:28 | |
| *** shausy has quit IRC | 05:32 | |
| *** sbjorn has quit IRC | 05:33 | |
| *** vbellur has quit IRC | 05:56 | |
| *** esker has joined #openstack-manila | 05:56 | |
| *** breitz has quit IRC | 05:58 | |
| *** breitz has joined #openstack-manila | 05:59 | |
| *** sbjorn has joined #openstack-manila | 06:00 | |
| *** haomaiwa_ has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-manila | 06:01 | |
| *** anubhav has joined #openstack-manila | 06:06 | |
| *** esker has quit IRC | 06:07 | |
| *** breitz has quit IRC | 06:22 | |
| *** esker has joined #openstack-manila | 06:25 | |
| *** esker has quit IRC | 06:31 | |
| *** vbellur has joined #openstack-manila | 06:31 | |
| *** zhongjun has joined #openstack-manila | 06:32 | |
| *** vbellur has quit IRC | 06:35 | |
| *** Zhongjun_ has joined #openstack-manila | 06:37 | |
| *** ashokchandran has quit IRC | 06:41 | |
| *** ashokchandran has joined #openstack-manila | 06:46 | |
| *** vbellur has joined #openstack-manila | 06:49 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-manila | 07:01 | |
| *** ashokchandran has quit IRC | 07:04 | |
| *** zhongjun22 has joined #openstack-manila | 07:09 | |
| *** zhongjun has quit IRC | 07:09 | |
| *** Zhongjun_ is now known as Zhongjun | 07:09 | |
| *** esker has joined #openstack-manila | 07:11 | |
| *** zhongjun22 has quit IRC | 07:15 | |
| *** breitz has joined #openstack-manila | 07:16 | |
| *** ashokchandran has joined #openstack-manila | 07:22 | |
| *** baojg has joined #openstack-manila | 07:22 | |
| *** ashokchandran has quit IRC | 07:23 | |
| *** esker has quit IRC | 07:24 | |
| *** ashokchandran has joined #openstack-manila | 07:29 | |
| *** deepakcs has joined #openstack-manila | 07:33 | |
| *** chen12 has joined #openstack-manila | 07:38 | |
| chen12 | hello, I have some questions about consistency-groups, anyone can help ? | 07:38 |
| *** esker has joined #openstack-manila | 07:41 | |
| *** baojg has quit IRC | 07:42 | |
| *** sac has quit IRC | 07:47 | |
| *** esker has quit IRC | 07:50 | |
| *** sac has joined #openstack-manila | 07:51 | |
| *** baojg has joined #openstack-manila | 07:51 | |
| *** baojg has quit IRC | 07:54 | |
| *** baojg has joined #openstack-manila | 07:58 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** ashokchandran has quit IRC | 08:01 | |
| *** ashokchandran has joined #openstack-manila | 08:01 | |
| *** haomaiwang has joined #openstack-manila | 08:01 | |
| *** esker has joined #openstack-manila | 08:10 | |
| *** baojg has quit IRC | 08:19 | |
| *** esker has quit IRC | 08:23 | |
| *** ashokchandran has quit IRC | 08:24 | |
| *** rraja has joined #openstack-manila | 08:25 | |
| *** ashokchandran has joined #openstack-manila | 08:34 | |
| *** vbellur has quit IRC | 08:37 | |
| *** vbellur has joined #openstack-manila | 08:54 | |
| *** Zhongjun has quit IRC | 08:57 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-manila | 09:01 | |
| *** ociuhandu has quit IRC | 09:10 | |
| *** ashokchandran has quit IRC | 09:12 | |
| *** nobse has joined #openstack-manila | 09:13 | |
| *** ashokchandran has joined #openstack-manila | 09:13 | |
| *** zhongjun has joined #openstack-manila | 09:16 | |
| *** nobse has quit IRC | 09:20 | |
| *** nobse has joined #openstack-manila | 09:20 | |
| *** zhongjun has quit IRC | 09:21 | |
| *** zhongjun has joined #openstack-manila | 09:22 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: Return share_type UUID instead of name in Share API https://review.openstack.org/220046 | 09:25 |
| *** baojg has joined #openstack-manila | 09:27 | |
| *** csaba3 is now known as csaba | 09:29 | |
| *** baojg has quit IRC | 09:29 | |
| *** zhongjun has quit IRC | 09:34 | |
| *** zhongjun has joined #openstack-manila | 09:36 | |
| *** lpetrut has joined #openstack-manila | 09:36 | |
| openstackgerrit | Merged openstack/manila: Updated from global requirements https://review.openstack.org/221930 | 09:39 |
| openstackgerrit | Petrut Lucian proposed openstack/manila: Enhance base driver checking if a method was implemented https://review.openstack.org/220511 | 09:54 |
| *** zhongjun22 has joined #openstack-manila | 09:55 | |
| *** zhongjun has quit IRC | 09:59 | |
| *** zhongjun22 has quit IRC | 10:00 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-manila | 10:01 | |
| *** zhongjun22 has joined #openstack-manila | 10:05 | |
| *** zhongjun22 has quit IRC | 10:11 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/puppet-manila: Fix rspec 3.x syntax https://review.openstack.org/222109 | 10:15 |
| *** zhongjun22 has joined #openstack-manila | 10:15 | |
| *** zhongjun22 has quit IRC | 10:20 | |
| *** zhongjun22 has joined #openstack-manila | 10:25 | |
| *** zhongjun22 has quit IRC | 10:31 | |
| openstackgerrit | Julia Varlamova proposed openstack/manila-ui: Add availability zones support to manila-ui https://review.openstack.org/216674 | 10:31 |
| *** zhongjun22 has joined #openstack-manila | 10:35 | |
| *** u_glide has joined #openstack-manila | 10:38 | |
| *** nobse has quit IRC | 10:40 | |
| *** zhongjun22 has quit IRC | 10:41 | |
| filipe | Hi, I can't start manila-share. it fails with 2015-09-10 12:22:47.512 25783 TRACE manila raise exception.BridgeDoesNotExist(bridge=bridge) | 10:45 |
| *** zhongjun22 has joined #openstack-manila | 10:45 | |
| filipe | my neutron service is configured to use linuxbridge driver | 10:45 |
| *** jcsp has joined #openstack-manila | 10:46 | |
| filipe | does manila support linuxbridge? if so how do I solve the issue? | 10:46 |
| *** zhongjun22 has quit IRC | 10:51 | |
| ganso_ | vponomaryov: ping | 10:55 |
| *** zhongjun22 has joined #openstack-manila | 10:55 | |
| openstackgerrit | zhaohua proposed openstack/manila: Huawei driver: fix reports reduplicate pools https://review.openstack.org/220799 | 10:57 |
| u_glide | toabctl: ping https://review.openstack.org/#/c/220046/ | 10:59 |
| u_glide | toabctl: Could you please review this patch? | 10:59 |
| *** zhongjun22 has quit IRC | 11:01 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-manila | 11:01 | |
| *** ociuhandu has joined #openstack-manila | 11:02 | |
| *** zhongjun22 has joined #openstack-manila | 11:05 | |
| *** ociuhandu_ has joined #openstack-manila | 11:08 | |
| *** ociuhandu has quit IRC | 11:08 | |
| *** ociuhandu_ is now known as ociuhandu | 11:08 | |
| *** zhongjun22 has quit IRC | 11:11 | |
| *** zhongjun22 has joined #openstack-manila | 11:15 | |
| *** zhongjun22 has quit IRC | 11:21 | |
| *** timcl has joined #openstack-manila | 11:21 | |
| *** timcl has left #openstack-manila | 11:21 | |
| vponomaryov | filipe: which driver exactly do you use? Link? also, are you sure that "bridge" indeed exist in system? Your Neutron works ok? | 11:24 |
| *** zhongjun22 has joined #openstack-manila | 11:25 | |
| vponomaryov | ganso_: pong | 11:25 |
| ganso_ | vponomaryov: hello :) | 11:25 |
| ganso_ | vponomaryov: Looks like they do not like our approach of renaming jobs: https://review.openstack.org/221880 | 11:26 |
| vponomaryov | ganso_: actually, there is proposed different approach for "renaming" not restricted "renaming" in general | 11:28 |
| vponomaryov | ganso_: he proposes you to make existing jobs as templates and then be able to use any substitution | 11:29 |
| ganso_ | vponomaryov: if I understood correctly what he suggested, it will look more organized, and should not make any practical changes, is this correct? | 11:29 |
| vponomaryov | ganso_: practical changes? it is exactly practical | 11:30 |
| ganso_ | vponomaryov: I mean, in the end we will have the same jobs, the yaml file will just look better | 11:30 |
| *** zhongjun22 has quit IRC | 11:30 | |
| vponomaryov | ganso_: use template for different similar jobs | 11:30 |
| vponomaryov | ganso_: yes, just a bit different look of our config file | 11:31 |
| *** deepakcs has quit IRC | 11:31 | |
| *** timcl has joined #openstack-manila | 11:31 | |
| ganso_ | vponomaryov: but in this part "Why rename? It seems both jobs should be defined." I thought he was suggesting having both "no-share-servers" and "no-share-servers-multibackend" jobs | 11:31 |
| vponomaryov | ganso_: template can solve it | 11:32 |
| vponomaryov | ganso_: but we do not really need both for scenario | 11:32 |
| ganso_ | vponomaryov: yes, but we do not want those 2 jobs variants running at the same time, do we? | 11:32 |
| ganso_ | vponomaryov: exactly | 11:32 |
| *** u_glide has quit IRC | 11:33 | |
| vponomaryov | ganso_: no no no, it does not mean that it will run two | 11:33 |
| vponomaryov | ganso_: we do not need template because we will use only one case | 11:33 |
| vponomaryov | ganso_: you can use it as argument | 11:33 |
| *** zhongjun22 has joined #openstack-manila | 11:33 | |
| vponomaryov | ganso_: we change one to another, we do not add new one | 11:33 |
| vponomaryov | ganso_: so, usage of template is improvement, that can be done in general and directly related to curren tgoal | 11:34 |
| ganso_ | vponomaryov: yes, that's what I wanted to confirm, I will try talking to him first | 11:34 |
| vponomaryov | ganso_: as we use same amount of jobs | 11:34 |
| vponomaryov | s/directly related/directly NOT related/ | 11:34 |
| vponomaryov | ganso_: commented there | 11:38 |
| ganso_ | vponomaryov: thanks, I am talking to him in infra channel | 11:41 |
| vponomaryov | ganso_: yeah, I was summoned by notify of my IRC app =) | 11:41 |
| *** vbellur has quit IRC | 11:43 | |
| *** ociuhandu has quit IRC | 11:51 | |
| *** anubhav has quit IRC | 11:52 | |
| openstackgerrit | Merged openstack/manila: Add SSL middleware to fix incorrect version host_url https://review.openstack.org/180483 | 11:58 |
| *** haomaiwang has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-manila | 12:01 | |
| *** eduardo has joined #openstack-manila | 12:03 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: Add 'snapshot_support' attr to share details https://review.openstack.org/221231 | 12:13 |
| *** tpsilva has joined #openstack-manila | 12:21 | |
| *** u_glide has joined #openstack-manila | 12:21 | |
| *** porrua has joined #openstack-manila | 12:22 | |
| *** jcsp has quit IRC | 12:28 | |
| *** vbellur has joined #openstack-manila | 12:32 | |
| *** haomaiwang has quit IRC | 12:35 | |
| *** bswartz has quit IRC | 12:36 | |
| *** dustins has joined #openstack-manila | 12:42 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Fix Share Migration tempest tests https://review.openstack.org/221916 | 12:44 |
| ganso_ | vponomaryov: Valeriy, I replied to your comment, could you please take a quick look at https://review.openstack.org/219851/ ? | 12:44 |
| *** xyang1 has joined #openstack-manila | 12:45 | |
| vponomaryov | ganso_: sure, one moment about https://review.openstack.org/#/c/221916/2..3/manila_tempest_tests/config.py | 12:46 |
| vponomaryov | ganso_: would be really great to move it up also =) | 12:46 |
| vponomaryov | ganso_: so all opts starting with "run_" will be located together | 12:47 |
| ganso_ | vponomaryov: lol ok just a sec | 12:50 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Fix Share Migration tempest tests https://review.openstack.org/221916 | 12:52 |
| ganso_ | vponomaryov: there you go :) | 12:53 |
| *** anubhav has joined #openstack-manila | 12:54 | |
| *** bill_az_ has joined #openstack-manila | 12:55 | |
| vponomaryov | ganso_: thanks =) | 12:55 |
| vponomaryov | ganso_: see https://review.openstack.org/219851/ | 12:59 |
| vponomaryov | added comments | 12:59 |
| *** akerr has joined #openstack-manila | 12:59 | |
| *** bswartz has joined #openstack-manila | 12:59 | |
| *** jcsp has joined #openstack-manila | 12:59 | |
| ganso_ | vponomaryov: thanks! Alyson will improve coverage :) | 13:00 |
| vponomaryov | ganso_: strange that none of reviewers did not point to cover job failure | 13:00 |
| ganso_ | vponomaryov: yes, that is something to improve on | 13:10 |
| *** martyturner has joined #openstack-manila | 13:12 | |
| *** cknight has joined #openstack-manila | 13:21 | |
| *** jcsp has joined #openstack-manila | 13:24 | |
| *** gouthamr has joined #openstack-manila | 13:37 | |
| *** vbellur has quit IRC | 13:38 | |
| *** zhongjun2 has joined #openstack-manila | 13:46 | |
| *** vbellur has joined #openstack-manila | 13:51 | |
| openstackgerrit | Andrew Kerr proposed openstack/manila: Add v2 Manila API path as base for microversions https://review.openstack.org/220221 | 13:58 |
| *** ktolstoy has joined #openstack-manila | 14:12 | |
| u_glide | bswartz, cknight, markstur, xyang1: ping https://review.openstack.org/#/c/211675/ | 14:12 |
| *** haomaiwang has joined #openstack-manila | 14:16 | |
| *** esker has joined #openstack-manila | 14:19 | |
| openstackgerrit | Merged openstack/puppet-manila: Fix rspec 3.x syntax https://review.openstack.org/222109 | 14:19 |
| *** esker has quit IRC | 14:23 | |
| *** esker has joined #openstack-manila | 14:23 | |
| *** u_glide has quit IRC | 14:24 | |
| *** u_glide has joined #openstack-manila | 14:26 | |
| *** anubhav has quit IRC | 14:29 | |
| *** eharney has joined #openstack-manila | 14:30 | |
| *** JoseMello has joined #openstack-manila | 14:35 | |
| *** ayma has joined #openstack-manila | 14:51 | |
| *** markstur_ has joined #openstack-manila | 14:59 | |
| *** haomaiwang has quit IRC | 15:00 | |
| *** Zhongjun has joined #openstack-manila | 15:00 | |
| *** lpabon has joined #openstack-manila | 15:00 | |
| *** filipe has quit IRC | 15:03 | |
| *** casusbelli has joined #openstack-manila | 15:03 | |
| *** timcl has quit IRC | 15:06 | |
| *** eharney has quit IRC | 15:06 | |
| *** eharney has joined #openstack-manila | 15:19 | |
| *** rushil has joined #openstack-manila | 15:21 | |
| *** rraja has quit IRC | 15:30 | |
| openstackgerrit | Merged openstack/manila: Fix usage of novaclient https://review.openstack.org/221687 | 15:32 |
| *** rraja has joined #openstack-manila | 15:32 | |
| *** ganso_ is now known as ganso | 15:33 | |
| *** rraja has quit IRC | 15:33 | |
| *** rraja has joined #openstack-manila | 15:37 | |
| *** u_glide_a has joined #openstack-manila | 15:41 | |
| openstackgerrit | Clinton Knight proposed openstack/manila: Missing check in ShareManager::manage_existing() https://review.openstack.org/222257 | 15:47 |
| *** timcl has joined #openstack-manila | 15:48 | |
| *** u_glide has quit IRC | 15:49 | |
| *** u_glide_a has quit IRC | 15:50 | |
| *** u_glide_a has joined #openstack-manila | 15:51 | |
| *** u_glide_a has quit IRC | 15:55 | |
| *** casusbelli has quit IRC | 16:01 | |
| *** lpabon has quit IRC | 16:02 | |
| openstackgerrit | Chuck Fouts proposed openstack/python-manilaclient: Add consistency group support to manila client https://review.openstack.org/215351 | 16:03 |
| *** rraja has quit IRC | 16:03 | |
| *** markstur_ has quit IRC | 16:04 | |
| *** jasonsb has quit IRC | 16:08 | |
| *** jasonsb has joined #openstack-manila | 16:08 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-manila: acceptance: switch RDO to CBS repositories https://review.openstack.org/222275 | 16:17 |
| *** jasonsb has quit IRC | 16:23 | |
| *** jasonsb has joined #openstack-manila | 16:27 | |
| *** lpetrut has quit IRC | 16:28 | |
| *** lpabon has joined #openstack-manila | 16:35 | |
| *** lpabon has quit IRC | 16:41 | |
| *** eharney has quit IRC | 16:47 | |
| *** JoseMello has quit IRC | 16:49 | |
| *** esker has quit IRC | 16:51 | |
| ganso | bswartz, cknight, markstur: https://review.openstack.org/#/c/221916/ | 16:51 |
| *** pgbridge has quit IRC | 16:58 | |
| *** lpetrut has joined #openstack-manila | 16:58 | |
| *** martyturner has quit IRC | 17:02 | |
| *** timcl1 has joined #openstack-manila | 17:03 | |
| *** martyturner has joined #openstack-manila | 17:03 | |
| *** timcl has quit IRC | 17:05 | |
| openstackgerrit | Alyson proposed openstack/manila: Adds retry function to HNAS driver https://review.openstack.org/219851 | 17:06 |
| *** ktolstoy has quit IRC | 17:08 | |
| *** esker has joined #openstack-manila | 17:09 | |
| *** ociuhandu has joined #openstack-manila | 17:09 | |
| *** lpetrut has quit IRC | 17:19 | |
| *** timcl1 has quit IRC | 17:20 | |
| *** eharney has joined #openstack-manila | 17:22 | |
| ganso | markstur: ping | 17:29 |
| markstur | ganso, Hi ganso | 17:30 |
| *** lpetrut has joined #openstack-manila | 17:30 | |
| ganso | markstur: hi Mark :) | 17:30 |
| ganso | markstur: Do you think it makes sense for the env variable default value to be True if Multibackend mode is true as well? | 17:30 |
| ganso | markstur: the CI maintainer will be able to disable through RUN_MANILA_MIGRATION_TESTS=False, but if not specified, default is True, makes sense? | 17:31 |
| markstur | Seems reasonable | 17:31 |
| markstur | Not sure at the moment what is perfect | 17:31 |
| markstur | I just want it to be easy to disable if not supported | 17:32 |
| ganso | markstur: yes, I guess only time will tell what the best setting is | 17:32 |
| markstur | ganso, Reviewers might also tell | 17:32 |
| *** tmckay has joined #openstack-manila | 17:32 | |
| ganso | markstur: I am a bit new to configuring Jenkins through scripts in manila repo, how exactly would I configure it to run Migration tests if the env variable default is false? | 17:33 |
| tmckay | Hi folks. Just hooked up manila ui in a horizon instance using these instructions https://github.com/openstack/manila-ui/blob/master/README.rst | 17:34 |
| tmckay | both from git | 17:34 |
| tmckay | Looks like "share type" dropdown is not populated on the create share form | 17:34 |
| tmckay | any ideas? | 17:34 |
| tmckay | I should mention that this is a horizon/manila from openstack tip, running against a Kilo packaged version of openstack (rhel osp) | 17:35 |
| markstur | ganso, I'm not sure about that. So your idea of defaulting to true for multi-backend -- when not specified -- might be the quick way. | 17:35 |
| tmckay | maybe they don't work together ... | 17:35 |
| *** martyturner has quit IRC | 17:36 | |
| *** erlon has joined #openstack-manila | 17:36 | |
| markstur | ganso, look at pre_test_hook.sh | 17:37 |
| markstur | ganso, maybe not | 17:38 |
| *** cknight1 has joined #openstack-manila | 17:38 | |
| *** cknight has quit IRC | 17:38 | |
| ganso | markstur: I am looking it at, I find it actually hard not to link with Multibackend... it would up either having a job with "migration" in the name, or defining it as True when multibackend is true | 17:40 |
| ganso | markstur: s/would up/would end up | 17:40 |
| *** eharney has quit IRC | 17:41 | |
| tmckay | hmm, I think maybe I don't have any types | 17:44 |
| markstur | ganso, project-config/jenkins/jobs/manila.yaml | 17:44 |
| markstur | ganso, It seems like generic is using default options so far, but it looks like non-default could be added there. I think VP asked you to make this off by default but on for generic. | 17:45 |
| markstur | ganso, I don't know if there is a better place to control it than manila.yaml. Checking the job name is kind of a hack. | 17:45 |
| ganso | markstur: yes, currently a standardized hack :\ | 17:46 |
| markstur | ganso, You don't need to do anything special for multibackend do you? You really just want it always on for generic and easy to enable for 3rd-party when implemented. Right? | 17:46 |
| ganso | markstur: easy way to disable for 3rd-party. yes | 17:47 |
| *** timcl has joined #openstack-manila | 17:47 | |
| markstur | ganso, Yes. Hacks-R-Us. It would be nice to control this without touching project-config. I wonder if anyone lurking knows the answer and isn't telling us. | 17:47 |
| * ganso looks around | 17:48 | |
| markstur | :) | 17:48 |
| ganso | markstur: I think I will proceed with what I had in mind, and see how it turns out | 17:48 |
| markstur | ganso, OK | 17:49 |
| ganso | markstur: I do not know if you have noticed, but line 47 of pre_test_hook kinda defeats the purpose of line 26 of post_test_hook | 17:55 |
| markstur | ganso, Does it make sense to also check for 'gate' in the name? | 17:56 |
| ganso | markstur: I don't think so, it could interfere with 3rd-party, like setting False by default for 3rd parties, we would also like 3rd parties to have it enabled at some point. This would cause 3rd-parties to have to enable them manually, instead of disabling manually. | 17:58 |
| *** timcl has quit IRC | 17:59 | |
| markstur | ganso, Well then you just set the default to true and your done and the 3rd-party CIs all break and get fixed. But I thought Valeriy said you should default to False | 17:59 |
| ganso | markstur: only 3rd-party CIs that have "multibackend" in the name break | 18:02 |
| markstur | ganso, I'd prefer gate prefix so 3rd-party CIs can use similar suffixes for their jobs. But if you prefer the multibackend check at least it gets us running (and is an existing hack) | 18:05 |
| markstur | ganso, I don't know if any 3rd party is running multi-backend yet. I just would like us to. | 18:05 |
| *** timcl has joined #openstack-manila | 18:05 | |
| *** martyturner has joined #openstack-manila | 18:06 | |
| *** Zhongjun has quit IRC | 18:07 | |
| *** eharney has joined #openstack-manila | 18:08 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Fix Share Migration tempest tests https://review.openstack.org/221916 | 18:08 |
| ganso | markstur: I can see that no 3rd-party has multibackend in the name, hehe | 18:09 |
| vponomaryov | ganso: any third-party CI can configure its setup on its own | 18:11 |
| vponomaryov | ganso: so, no direct dependency | 18:12 |
| markstur | vponomaryov, ganso Is there a list of words we are not allowed to say in our job names? | 18:12 |
| markstur | George Carlin anyone? | 18:12 |
| markstur | vponomaryov, ganso So it just seems to me that maybe "gate" prefix would indicate upstream tests and vendors would have other prefixes | 18:13 |
| ganso | markstur: lol what? | 18:13 |
| vponomaryov | markstur: none from GreatDictionaryOfRudeWords | 18:13 |
| markstur | so vendors don't break if they use common names like share-servers, multibackend, etc | 18:14 |
| markstur | ganso, I guess you are too young to remember George Carlin | 18:14 |
| ganso | markstur: yea never heard of him | 18:15 |
| *** dzamboni has joined #openstack-manila | 18:16 | |
| vponomaryov | markstur: he just do not watch TV maybe | 18:16 |
| markstur | vponomaryov, Do we want to just make the defaults be for generic and all 3rd-parties have to react? That makes it easy for ganso to enable | 18:18 |
| ganso | vponomaryov: I rarely do | 18:18 |
| markstur | vponomaryov, I thought I've seen you suggest that in the past, but I think you suggested disabling migration by default | 18:18 |
| vponomaryov | markstur: disable because it is not mandatory feature | 18:19 |
| vponomaryov | markstur: not because generic can run it | 18:19 |
| markstur | OK. | 18:19 |
| vponomaryov | markstur: also implemented in few drivers | 18:19 |
| markstur | So maybe the best way to set env for generic is in project-config. But none of us like doing that because it require infra approval | 18:19 |
| ganso | markstur: yes, it is being painful already to just rename the jobs | 18:20 |
| vponomaryov | markstur: we have hooks for it | 18:20 |
| markstur | So we can hack pre and/or post test hook to check for those "special" words | 18:20 |
| markstur | I just like the idea of using "gate" or "gate-manila" prefix to do special non-3rd-party settings | 18:21 |
| vponomaryov | markstur: just set your own env vars | 18:21 |
| vponomaryov | markstur: it is not hacking | 18:22 |
| markstur | creative coding | 18:22 |
| ganso | vponomaryov: but that usually involves your 3rd party CI breaking before the new env var is set | 18:22 |
| ganso | vponomaryov: I see as no big deal... but it happens | 18:23 |
| markstur | ganso, Does it make sense to skip that extra install based on migration enabled now too? | 18:27 |
| ganso | markstur: it does | 18:29 |
| ganso | markstur: good point | 18:29 |
| *** pgbridge has joined #openstack-manila | 18:34 | |
| ganso | markstur: if line 47 of pre_test_hook overwrites previously defined lines in localrc, then this is not going to work | 18:37 |
| openstackgerrit | Ben Swartzlander proposed openstack/manila: Remove HDS SOP driver due to lack of CI https://review.openstack.org/222317 | 18:37 |
| ganso | markstur: ok, nevermind, the default is going to be true, unless disabled | 18:38 |
| *** esker has quit IRC | 18:39 | |
| *** esker has joined #openstack-manila | 18:39 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Fix Share Migration tempest tests https://review.openstack.org/221916 | 18:45 |
| *** zhongjun22 has quit IRC | 18:52 | |
| openstackgerrit | Ben Swartzlander proposed openstack/manila: Remove IBM GPFS driver due to lack of CI https://review.openstack.org/222326 | 18:55 |
| bswartz | jasonsb: https://review.openstack.org/222317 | 19:03 |
| *** vbellur has quit IRC | 19:05 | |
| jasonsb | bswartz: hi sir | 19:07 |
| jasonsb | bswartz: would you prefer if i submitted the removal? | 19:07 |
| bswartz | jasonsb: hello! | 19:08 |
| bswartz | I tried to reach you yesterday and the day before | 19:08 |
| jasonsb | bswartz: i got your mail | 19:08 |
| jasonsb | bswartz: was checking internally | 19:08 |
| bswartz | is there any progress I don't know about? | 19:08 |
| jasonsb | sorry for awol | 19:08 |
| jasonsb | sadly no. very sorry about that | 19:09 |
| bswartz | well you can +1 the patch or simply comment in the review | 19:09 |
| bswartz | I'm very sad to see a driver removed | 19:09 |
| jasonsb | i was hoping for alot more | 19:09 |
| jasonsb | alot more community involvement | 19:09 |
| jasonsb | but the support wasn't there internally. | 19:10 |
| bswartz | make sure the folks at HDS understand that the driver can be brought back if a CI system is created | 19:10 |
| jasonsb | that is the plan | 19:10 |
| jasonsb | individuals exist who want to make that happen | 19:10 |
| jasonsb | i do want to thank you for your fantastic support | 19:11 |
| bswartz | jasonsb: if my removal patch leaves anything behind which should also be removed, you can -1 my patch and I'll fix it -- that will make the removal more clean and hopefully easier to revert in the future | 19:11 |
| jasonsb | welcoming community and everything you do | 19:11 |
| openstackgerrit | Merged openstack/python-manilaclient: Update path to subunit2html in post_test_hook https://review.openstack.org/219934 | 19:11 |
| jasonsb | ok, i'll double check to make sure it cleans up | 19:12 |
| *** tpsilva has quit IRC | 19:13 | |
| *** zhongjun2 has quit IRC | 19:13 | |
| jasonsb | you should get rid of hds directory too | 19:13 |
| jasonsb | but let me do a proper git fetch and go through | 19:13 |
| bswartz | jasonsb: iirc there are no directories in git | 19:15 |
| *** timcl has quit IRC | 19:15 | |
| jasonsb | bswartz: oh right | 19:16 |
| *** ociuhandu has quit IRC | 19:18 | |
| jasonsb | bswartz: i -1'd -> opts.py | 19:31 |
| bswartz | jasonsb: thanks | 19:31 |
| *** vbellur has joined #openstack-manila | 19:32 | |
| openstackgerrit | Ben Swartzlander proposed openstack/manila: Remove HDS SOP driver due to lack of CI https://review.openstack.org/222317 | 19:32 |
| alyson_ | bswartz: maybe exception's classes about SOP driver in exception.py? | 19:36 |
| bswartz | alyson_: ooh good point | 19:37 |
| bswartz | I should have done more grepping | 19:37 |
| openstackgerrit | Ben Swartzlander proposed openstack/manila: Remove HDS SOP driver due to lack of CI https://review.openstack.org/222317 | 19:39 |
| *** lpetrut has quit IRC | 19:44 | |
| *** ociuhandu has joined #openstack-manila | 19:46 | |
| bswartz | csaba: ping | 19:46 |
| bswartz | who here fully understands all of the ganesha code we have? | 19:46 |
| jasonsb | alyson_: yes, sop was in exception.py too. nice catch | 19:49 |
| openstackgerrit | Ben Swartzlander proposed openstack/manila: Remove IBM GPFS driver due to lack of CI https://review.openstack.org/222326 | 20:01 |
| markstur | bswartz, pong | 20:01 |
| cFouts | vponomaryov and markstur https://review.openstack.org/#/c/215351/ cg support for the client is ready for review again when you are ready | 20:20 |
| *** dustins has quit IRC | 20:26 | |
| *** dustins has joined #openstack-manila | 20:28 | |
| *** rhagarty__ has joined #openstack-manila | 20:37 | |
| *** lpetrut has joined #openstack-manila | 20:42 | |
| *** bill_az_ has quit IRC | 20:43 | |
| *** rhagarty__ has quit IRC | 20:43 | |
| *** rhagarty__ has joined #openstack-manila | 20:43 | |
| *** u_glide has joined #openstack-manila | 20:44 | |
| *** akerr has quit IRC | 20:45 | |
| *** rhagarty__ has quit IRC | 20:46 | |
| *** rhagarty__ has joined #openstack-manila | 20:47 | |
| *** bswartz has quit IRC | 20:50 | |
| *** lpetrut has quit IRC | 20:50 | |
| *** rhagarty__ has quit IRC | 20:54 | |
| *** rhagarty__ has joined #openstack-manila | 20:54 | |
| *** jasonsb has quit IRC | 20:55 | |
| cFouts | markstur and vponomaryov: thanks for +2s | 20:57 |
| *** dustins has quit IRC | 20:59 | |
| openstackgerrit | Merged openstack/manila: Huawei driver: fix reports reduplicate pools https://review.openstack.org/220799 | 21:02 |
| openstackgerrit | Merged openstack/manila: Add v2 Manila API path as base for microversions https://review.openstack.org/220221 | 21:03 |
| *** eharney has quit IRC | 21:07 | |
| *** porrua has quit IRC | 21:10 | |
| *** jasonsb has joined #openstack-manila | 21:14 | |
| *** bswartz has joined #openstack-manila | 21:17 | |
| *** martyturner has quit IRC | 21:27 | |
| *** sgotliv_ has quit IRC | 21:49 | |
| *** cknight1 has quit IRC | 21:54 | |
| openstackgerrit | Merged openstack/python-manilaclient: Implement Share Instances Admin API https://review.openstack.org/211675 | 21:55 |
| openstackgerrit | Merged openstack/python-manilaclient: Allow user to unset security service params https://review.openstack.org/214577 | 21:55 |
| *** gouthamr has quit IRC | 22:00 | |
| *** rushil has quit IRC | 22:02 | |
| *** dzamboni has quit IRC | 22:02 | |
| *** erlon has quit IRC | 22:08 | |
| *** alyson_ has quit IRC | 22:10 | |
| *** rushil has joined #openstack-manila | 22:11 | |
| *** pgbridge has quit IRC | 22:19 | |
| *** pgbridge has joined #openstack-manila | 22:20 | |
| markstur | ganso, ping | 22:21 |
| ganso | markstur: pong | 22:24 |
| markstur | ganso, So the empty migration tests runs with multi-backend of course. | 22:24 |
| markstur | but otherwise the migration tests are skipped | 22:24 |
| ganso | markstur: yes | 22:25 |
| markstur | I'm confused. What's the plan to run those? | 22:25 |
| ganso | markstur: for Jenkins to run we still need job rename patch to be approved by infra team | 22:25 |
| csaba | bswartz: pong | 22:25 |
| ganso | markstur: for Jenkins to run scenario tests, I mean | 22:26 |
| *** bill_az has joined #openstack-manila | 22:26 | |
| markstur | ganso, Your going to rename scenario to scenario-multibackend? | 22:26 |
| *** rhagarty__ has quit IRC | 22:26 | |
| ganso | markstur: yes https://review.openstack.org/#/c/221880/ | 22:26 |
| markstur | ganso, I see commit | 22:26 |
| markstur | ganso, So we should probably have the infra change land first so we can recheck to make sure this doesn't break our gate | 22:33 |
| markstur | ganso, Or maybe DependsOn... if that works with project configs (?) | 22:34 |
| ganso | markstur: good idea, possibly does work | 22:34 |
| ganso | markstur: I will try that tomorrow morning | 22:35 |
| ganso | markstur: thanks | 22:36 |
| markstur | ganso, If my -1 lingers ping me. I'll try to keep an eye on it, but sometimes things get buried. | 22:36 |
| ganso | markstur: no problem :) | 22:39 |
| *** porrua has joined #openstack-manila | 22:45 | |
| *** tmckay has left #openstack-manila | 22:45 | |
| *** porrua has quit IRC | 22:46 | |
| *** gouthamr has joined #openstack-manila | 23:02 | |
| *** ayma has quit IRC | 23:04 | |
| *** xyang1 has quit IRC | 23:05 | |
| *** rushil has quit IRC | 23:10 | |
| *** gouthamr_ has joined #openstack-manila | 23:20 | |
| *** gouthamr has quit IRC | 23:23 | |
| *** chen12 has quit IRC | 23:32 | |
| *** chen12 has joined #openstack-manila | 23:33 | |
| *** rhagarty__ has joined #openstack-manila | 23:35 | |
| *** chen12 has quit IRC | 23:38 | |
| *** rushil has joined #openstack-manila | 23:41 | |
| *** rhagarty__ has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!