| *** LinPeiWen has joined #openstack-manila | 00:56 | |
| *** ivan_lin has quit IRC | 03:37 | |
| *** LinPeiWen has quit IRC | 03:37 | |
| *** ivan_lin has joined #openstack-manila | 03:37 | |
| *** jmlowe has quit IRC | 05:56 | |
| *** jmlowe has joined #openstack-manila | 06:34 | |
| *** LinPeiWen has joined #openstack-manila | 06:35 | |
| *** ivan_lin has quit IRC | 07:11 | |
| *** LinPeiWen has quit IRC | 07:11 | |
| *** ivan_lin has joined #openstack-manila | 07:12 | |
| *** tosky has joined #openstack-manila | 07:36 | |
| *** LinPeiWen has joined #openstack-manila | 07:48 | |
| *** ivan_lin has quit IRC | 07:50 | |
| *** LinPeiWen80 has joined #openstack-manila | 08:21 | |
| *** lpetrut has joined #openstack-manila | 09:27 | |
| *** carloss has joined #openstack-manila | 10:38 | |
| simondodsley | Need some help with the `manila_tempest_plugin` tests. I have 7 tests that error with `Share network must be set when the driver_handles_share_servers is true` which makes no sense as the driver I am testing has DHSS set to false. | 13:07 |
|---|---|---|
| simondodsley | An example of a failing test is `test_create_shrink_and_write` | 13:07 |
| *** jv has joined #openstack-manila | 13:12 | |
| vkmc | simondodsley, can you share your tempest.conf? | 13:13 |
| vkmc | multitenancy_enabled = True <- you need this set to false if it's dhss=false | 13:17 |
| vkmc | more info here https://docs.openstack.org/manila/latest/contributor/tempest_tests.html | 13:18 |
| simondodsley | This is auto-generated from Software Factory which we use for our CI system https://www.irccloud.com/pastebin/eCof90mr/ | 13:19 |
| simondodsley | I have `multitenancy_enabled = False` | 13:20 |
| *** tosky_ has joined #openstack-manila | 13:27 | |
| *** tosky has quit IRC | 13:27 | |
| *** tosky_ is now known as tosky | 13:27 | |
| vkmc | hm | 13:31 |
| vkmc | simondodsley, default_share_type_name = dhss_false <- this share type has dhss = false too, right? | 13:31 |
| simondodsley | yes | 13:34 |
| vkmc | seems that error is raised here https://opendev.org/openstack/manila/src/branch/master/manila/api/v1/shares.py#L381-L388 | 13:35 |
| simondodsley | that is a default share type created by devstack | 13:35 |
| vkmc | well, the default share created by devstack has the name "default" | 13:36 |
| simondodsley | Yes - but that has DHSS = true | 13:36 |
| simondodsley | misused defauklt - it is a share type created by devstack | 13:36 |
| simondodsley | If I use `default` fir the share-type many other tests fail with the same error message. | 13:37 |
| vkmc | ah you are right, we create default, dhss_false and dhss_true | 13:37 |
| simondodsley | Seems that these 7 specific tests are failing for some other reason that is setting of this error message | 13:37 |
| *** vishalmanchanda has joined #openstack-manila | 13:37 | |
| vkmc | simondodsley, is this showing on the patch you have up for review in the queue? | 13:38 |
| simondodsley | yes | 13:38 |
| vkmc | cool, let me take a look | 13:38 |
| vkmc | it's odd, I'm seeing that we define all the other gates, being those dhss true or dhss false to use the default share type for tempest | 13:58 |
| vkmc | and it passes | 13:58 |
| vkmc | dviroel, o/ | 13:58 |
| vkmc | dviroel, maybe you can take a look? not sure what we are missing there | 13:58 |
| dviroel | vkmc: o/ | 13:59 |
| vkmc | dviroel, mostly scenario tests are failing for simondodsley | 13:59 |
| openstackgerrit | Merged openstack/devstack-plugin-ceph master: Changed minversion in tox to 3.18.0 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/790215 | 14:00 |
| dviroel | oh, let me take a look then | 14:00 |
| dviroel | this change here: https://review.opendev.org/c/openstack/manila/+/789384 ? | 14:00 |
| simondodsley | yes | 14:01 |
| vkmc | yes | 14:01 |
| vkmc | dviroel, the only difference with other gates is that simondodsley is using dhss_false share type in tempest conf | 14:02 |
| vkmc | dviroel, for some reason we use default for all the gates, being those dhss=false or dhss=true | 14:02 |
| * dviroel looking | 14:10 | |
| *** lpetrut has quit IRC | 14:22 | |
| * dviroel looking into tempest-plugin | 14:22 | |
| gouthamr | Simon simondodsley | 14:39 |
| gouthamr | can you share your local.conf file? | 14:39 |
| dviroel | sorry, in a meeting right now | 14:40 |
| gouthamr | we’ve some devstack plug-in opts to override the default share type and the specs associated with them - the plug-in should be creating three share types by default; but, we could be having a special preference to the one called “default” somehow | 14:42 |
| simondodsley | gouthamr: This is the `local.conf` https://www.irccloud.com/pastebin/Hgwe79j3/ | 14:47 |
| * gouthamr switches to workstation | 14:53 | |
| gouthamr | simondodsley: the plugin tries to read backend config and set the appropriate value for the extra-spec "driver_handles_share_servers": https://github.com/openstack/manila/blob/master/devstack/plugin.sh#L531 | 14:56 |
| gouthamr | simondodsley: in your case, the backend is being configured via post_config... | 14:57 |
| gouthamr | simondodsley: so that may be the issue here... | 14:58 |
| gouthamr | yeah.. http://openstack-logs.purestorage.com/84/789384/2/thirdparty-check/pure-devstack-manila-tempest-aio/0c94253/job-output.txt.gz#_2021-05-10_14_20_47_695870 | 15:00 |
| simondodsley | so you want it created in the `DEFAULT` section? Isn't that confusing? Isn't the whole point of the `post-config` to set up specific backend stanzas | 15:00 |
| simondodsley | Then I would not have to force the default share type to be something other than default as well | 15:01 |
| simondodsley | I guess I did it this way as that is the way we do it in the Cinder tempest tests and that is what I know | 15:02 |
| gouthamr | http://paste.openstack.org/show/805169/ | 15:03 |
| simondodsley | Yep - I'll change that and see what happens - thanks | 15:04 |
| vkmc | gouthamr++ | 15:12 |
| vkmc | thanks for following up | 15:12 |
| dviroel | gouthamr++ tks | 15:16 |
| vkmc | simondodsley, if that fixes it for you, could you open a doc bug? | 15:18 |
| vkmc | we could clarify this in the driver maintainers docs | 15:18 |
| gouthamr | if this works, we could add it to the driver maintainers doc we were planning | 15:22 |
| simondodsley | Sure | 15:33 |
| simondodsley | Didn't work - the devstack build fails with this `/opt/stack/devstack/.localrc.auto: line 56: MANILA_OPTGROUP_flashblade-1_driver_handles_share_servers=False: command not found` which makes no sense | 15:39 |
| simondodsley | could it be the hyphen in the backend name? | 15:39 |
| gouthamr | Oh - yes, likely | 15:39 |
| simondodsley | OK - I'll amend and try again then | 15:41 |
| openstackgerrit | Radosław Piliszek proposed openstack/devstack-plugin-ceph master: Bump up Ceph to Pacific and Ganesha to v3.5 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/787774 | 18:14 |
| *** afabio has joined #openstack-manila | 20:21 | |
| *** vishalmanchanda has quit IRC | 20:27 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: Bump up Ceph to Pacific and Ganesha to v3.5 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/787774 | 21:54 |
| *** iurygregory has quit IRC | 22:07 | |
| *** iurygregory has joined #openstack-manila | 22:11 | |
| *** afabio has quit IRC | 22:42 | |
| *** tosky has quit IRC | 22:57 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!