| *** brinzhang has joined #openstack-manila | 01:23 | |
| *** brinzhang_ has quit IRC | 01:26 | |
| *** esker has joined #openstack-manila | 01:29 | |
| *** brinzhang_ has joined #openstack-manila | 01:55 | |
| *** brinzhang has quit IRC | 01:58 | |
| *** esker has quit IRC | 02:12 | |
| *** maaritamm has joined #openstack-manila | 03:41 | |
| *** maaritamm has quit IRC | 03:42 | |
| *** brinzhang has joined #openstack-manila | 03:49 | |
| *** brinzhang_ has quit IRC | 03:52 | |
| openstackgerrit | haixin proposed openstack/manila master: Extend share will go through scheduler https://review.opendev.org/700393 | 03:55 |
|---|---|---|
| *** esker has joined #openstack-manila | 04:00 | |
| *** esker has quit IRC | 04:00 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila-tempest-plugin master: Change to use neutron api to create floating ip https://review.opendev.org/668597 | 04:35 |
| *** maaritamm has joined #openstack-manila | 04:46 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila-tempest-plugin master: Change to use neutron api to create floating ip https://review.opendev.org/668597 | 05:22 |
| *** brinzhang has quit IRC | 05:57 | |
| *** brinzhang has joined #openstack-manila | 05:58 | |
| *** maaritamm has left #openstack-manila | 06:13 | |
| *** maaritamm has joined #openstack-manila | 06:18 | |
| *** maaritamm has left #openstack-manila | 06:22 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-manila master: Prepare Ussuri RC1 https://review.opendev.org/722582 | 06:24 |
| *** brinzhang_ has joined #openstack-manila | 06:30 | |
| *** brinzhang_ has quit IRC | 06:32 | |
| *** brinzhang has quit IRC | 06:33 | |
| *** tosky has joined #openstack-manila | 07:33 | |
| openstackgerrit | haixin proposed openstack/manila master: Extend share will go through scheduler https://review.opendev.org/700393 | 07:52 |
| *** brinzhang has joined #openstack-manila | 09:03 | |
| *** brinzhang_ has joined #openstack-manila | 09:06 | |
| *** brinzhang has quit IRC | 09:09 | |
| *** brinzhang_ has quit IRC | 10:25 | |
| openstackgerrit | Shi Yan proposed openstack/manila-tempest-plugin master: Change to use neutron api to create floating ip https://review.opendev.org/668597 | 11:40 |
| *** felipe_rodrigues has quit IRC | 11:51 | |
| *** eharney has joined #openstack-manila | 12:18 | |
| *** vhari has quit IRC | 12:25 | |
| openstackgerrit | Sean McGinnis proposed openstack/manila master: Add py38 package metadata https://review.opendev.org/722700 | 13:24 |
| openstackgerrit | Sean McGinnis proposed openstack/python-manilaclient master: Add py38 package metadata https://review.opendev.org/722748 | 13:26 |
| *** maaritamm has joined #openstack-manila | 14:31 | |
| dviroel | gouthamr: let me know when u have some minutes to discuss about those tempest tests | 14:55 |
| gouthamr | dviroel: o/ sup? | 14:58 |
| dviroel | gouthamr: hey, bad news :P | 15:03 |
| dviroel | gouthamr: in order to have a test with different backends, without specifying an az, we'll need to fix this: https://review.opendev.org/#/c/709697/12/manila/scheduler/drivers/filter.py@186 | 15:03 |
| gouthamr | dviroel: i anticipated that, you'll still use pool_name, won't you? | 15:04 |
| gouthamr | dviroel: unless your pool_name is the same across backends | 15:04 |
| dviroel | gouthamr: yes ^ | 15:05 |
| dviroel | gouthamr: can't guarantee that will land on the another backend | 15:05 |
| gouthamr | dviroel: ack.. | 15:08 |
| dviroel | gouthamr: the test will not be deterministic since will depend how pools are named | 15:08 |
| dviroel | gouthamr: how we should proceed? | 15:08 |
| dviroel | gouthamr: for sure we need a fix for 'backend_name' | 15:08 |
| dviroel | filtering | 15:09 |
| dviroel | gouthamr: was thinking on letting those 2 other tests that are working (same backends, different pool; different azs) | 15:09 |
| dviroel | provide a fix and latter add the test that is missing. | 15:10 |
| gouthamr | dviroel: popping out the share backend name was deliberate, correct | 15:10 |
| dviroel | gouthamr: releasing tempest with those two tests proposed by andrebeltrami (with the refactoring that you proposed) is acceptable? | 15:13 |
| dviroel | gouthamr: the manila fix will need to backported to ussuri and a new test added for different backends. | 15:13 |
| dviroel | I'm considering that this fix isn't a reason for a new RC | 15:14 |
| gouthamr | dviroel: can you even fix this? | 15:14 |
| dviroel | gouthamr: the fix would be just to continue to pop out when dealing with replication, not share from snapshot | 15:15 |
| gouthamr | dviroel: hmmm, that wouldn't make sense, no? we're not letting people change share types when creating from snapshot; and constraining with share_backend_name doesn't make sense here | 15:18 |
| dviroel | gouthamr: yes, you're right | 15:18 |
| dviroel | gouthamr: oh. are you saying that that 'share_backend_name' pop out shouldn't exist at all? | 15:20 |
| gouthamr | dviroel: nope, i don't believe a fix is needed in manila to test this use case deterministically, the test doesn't align with user expectations for this feature imo | 15:21 |
| dviroel | gouthamr: yes, its actually the way that we found to do it. User may just select the destination AZ that they want | 15:22 |
| gouthamr | dviroel: yep, or let the scheduler find the best place | 15:23 |
| dviroel | gouthamr: this is another think that is not true, yet ^. I think that was not discussed on the reviews. Creating from snapshot will always have a destination AZ | 15:26 |
| dviroel | gouthamr: was implemented on this way before and the current implementation still set the same AZ from the source share if the user doesn't specify one | 15:27 |
| gouthamr | dviroel: that sorta makes sense? | 15:27 |
| dviroel | gouthamr: yesterday I was wondering that may not | 15:28 |
| dviroel | gouthamr: Was wondering why we keep this behavior. | 15:29 |
| dviroel | https://review.opendev.org/#/c/709697/12/manila/share/api.py@197 | 15:29 |
| dviroel | gouthamr: there are lot of details on the change and we may let this pass without discussing. | 15:30 |
| dviroel | gouthamr: we might consider as a issue and provide a fix for that too if everybody agrees | 15:30 |
| dviroel | gouthamr: so, we are open to discuss and fix. We just figure it out this points yesterday and dealing with tempest | 15:32 |
| dviroel | s/and/when | 15:32 |
| gouthamr | dviroel: hmmm, backends in the same AZ do weigh higher, correct? | 15:33 |
| dviroel | gouthamr: yes | 15:34 |
| gouthamr | dviroel: okay, then it's fine to stop sending the AZ, and let the scheduler decide | 15:35 |
| dviroel | gouthamr: yeah | 15:42 |
| openstackgerrit | Merged openstack/manila master: Add py38 package metadata https://review.opendev.org/722700 | 15:43 |
| gouthamr | tbarron_ dviroel carloss: need this https://review.opendev.org/#/c/722531/ for all the train backports, please take a look when possible | 16:04 |
| tbarron_ | k | 16:04 |
| dviroel | gouthamr: have been already there. look good | 16:05 |
| dviroel | voted | 16:05 |
| tbarron_ | I was just waiting to see that it works, and ity does. | 16:05 |
| tbarron_ | "it" does too | 16:06 |
| gouthamr | great ty guys :) | 16:06 |
| dviroel | gouthamr: so, going back to this 3 tests: https://etherpad.opendev.org/p/netapp-shr-from-snap-another-pool | 16:06 |
| tbarron_ | gouthamr: thanks for that fix! | 16:06 |
| dviroel | gouthamr: can't think on how to proceed with test 2. | 16:07 |
| gouthamr | dviroel: for now lets drop test 2 - and proceed with 1 and 3 | 16:08 |
| dviroel | gouthamr: ack, andrebeltrami should be finishing the refactoring right now. | 16:08 |
| gouthamr | dviroel: if the behavior of test 2 is needed, pool names have to be unique, maybe you can do that on your test clusters | 16:08 |
| dviroel | gouthamr: yes, by retrieving extra backend infos and so on | 16:09 |
| dviroel | we are finishing that CI configuration presented on the the collab review | 16:10 |
| dviroel | this will help us on testing all scenarios | 16:10 |
| gouthamr | nice | 16:11 |
| gouthamr | thanks for looking at https://review.opendev.org/#/c/668597/ | 16:11 |
| gouthamr | it'd be nice to get this one into this release too, since its a minor but pretty useful change | 16:11 |
| gouthamr | i'm confused why neutron client behaves differently in only one of the jobs | 16:12 |
| gouthamr | huh | 16:14 |
| dviroel | there is more than one | 16:14 |
| gouthamr | https://opendev.org/openstack/tempest/src/branch/master/tempest/scenario/manager.py#L959-L968 | 16:15 |
| dviroel | that is failing on the change | 16:15 |
| gouthamr | you're right, the zfsonlinux job also failed with the same errors | 16:16 |
| gouthamr | but, cephfs-nfs and lvm both passed | 16:16 |
| gouthamr | (cephfs-nfs failed elsewhere) | 16:17 |
| dviroel | oh | 16:20 |
| gouthamr | weird, http://paste.openstack.org/show/792682/ | 16:24 |
| gouthamr | same request, different responses :| | 16:24 |
| gouthamr | https://docs.openstack.org/api-ref/network/v2/?expanded=list-ports-detail#list-ports doesn't list fixed_ip, | 16:27 |
| gouthamr | dviroel: ty for noticing, lemme see if i can fix that | 16:28 |
| dviroel | gouthamr: np, tks for working on this | 16:29 |
| dviroel | weird that its working on the other job | 16:31 |
| dviroel | gouthamr: is this somehow related? | 16:34 |
| dviroel | "If the ip-substring-filtering extension is enabled, the Neutron API supports IP address substring filtering on the fixed_ips attribute. If you specify an IP address substring (ip_address_substr) in an entry of the fixed_ips attribute, the Neutron API will list all ports that have an IP address matching the substring." | 16:34 |
| gouthamr | could be, but we're using "fixed_ip" and sending it as a string, in our case, its always None... we don't enable that extension explicitly, lemme check if its the default, for starters, i'm attempting to conform and filter with fixed_ips instead of fixed_ip, and do that only if an ip is provided (we never do) | 16:36 |
| *** maaritamm has quit IRC | 16:37 | |
| gouthamr | kinda makes me think we ought to, especially when plugging in the storage network | 16:37 |
| gouthamr | it probably is fine for now because of nova preserving the nic order we request with | 16:37 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila-tempest-plugin master: Fix floatingip allocation https://review.opendev.org/668597 | 17:06 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila-tempest-plugin master: Fix floatingip allocation https://review.opendev.org/668597 | 17:28 |
| *** logan- has quit IRC | 17:30 | |
| *** logan- has joined #openstack-manila | 17:42 | |
| openstackgerrit | Merged openstack/python-manilaclient master: Add py38 package metadata https://review.opendev.org/722748 | 17:57 |
| openstackgerrit | Andre Beltrami proposed openstack/manila-tempest-plugin master: Functional tests for create share from snapshot different pools/azs https://review.opendev.org/709702 | 19:58 |
| *** toabctl has quit IRC | 20:00 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila stable/train: [stable/train only] Pin neutron-tempest-plugin https://review.opendev.org/722531 | 20:03 |
| -openstackstatus- NOTICE: The Zuul project gating service is reporting new patches in merge conflict erroneously due to a configuration error, fix in progress | 20:20 | |
| dviroel | vkmc: ^ | 20:25 |
| vkmc | yep | 20:25 |
| vkmc | :) | 20:25 |
| openstackgerrit | Daniel Tapia proposed openstack/manila-tempest-plugin master: Update shrink scenario tests https://review.opendev.org/715758 | 21:04 |
| openstackgerrit | Daniel Tapia proposed openstack/manila-tempest-plugin master: Update shrink scenario tests https://review.opendev.org/715758 | 21:05 |
| openstackgerrit | Daniel Tapia proposed openstack/manila-tempest-plugin master: Update shrink scenario tests https://review.opendev.org/715758 | 21:06 |
| vkmc | zuul is down | 21:24 |
| vkmc | right? | 21:24 |
| lseki | vkmc: yep, it seems so :\ | 22:05 |
| openstackgerrit | Daniel Tapia proposed openstack/manila-tempest-plugin master: Update shrink scenario tests https://review.opendev.org/715758 | 22:39 |
| -openstackstatus- NOTICE: the This Zuul outage was taken as an opportunity to perform an impromptu maintenance for changing our service deployment model; any merge failures received from Zuul between 19:40 and 20:20 UTC were likely in error and those changes should be rechecked; any patches uploaded between 20:55 and 22:45 UTC were missed entirely by Zuul and should also be rechecked to get fresh test results | 22:57 | |
| *** tosky has quit IRC | 23:14 | |
| *** vhari has joined #openstack-manila | 23:36 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/python-manilaclient master: [ussuri][goal] Change contributor guide https://review.opendev.org/723056 | 23:50 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila-tempest-plugin master: Fix floatingip allocation https://review.opendev.org/668597 | 23:52 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!