| opendevreview | Goutham Pacha Ravi proposed openstack/manila-tempest-plugin master: [DNM] Test IPv6 wildcard ::/0 in CephFS NFS https://review.opendev.org/c/openstack/manila-tempest-plugin/+/988824 | 06:27 |
|---|---|---|
| hong-p[m] | gouthamr: carloss Sorry for the late. I agree with treating this as a bugfix rather than a new spec, sounds like the right call. | 08:51 |
| hong-p[m] | I think simply adding availability_zone to _get_share_search_options() may not be enough to make the filter work. The filtering might need to be done against availability_zone_id internally, since that's what the DB layer operates on. If we want to support filtering by availability_zone name, some additional lookup logic would be needed. | 08:51 |
| hong-p[m] | I also noticed that the docs show availability_zone is supported as a sort_key, but in practice it only works with availability_zone_id. Do you think that inconsistency should be treated as a bug as well? | 08:51 |
| hong-p[m] | And just to make sure I'm on the same page since we're going with a bugfix approach, should I just abandon the spec I prepared? Also, for the non-existent AZ case, are we aligned on returning the full list and preserving the current behavior, similar to how services handles it? And should I go ahead and take ownership of this bugfix? thanks. | 08:52 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila-tempest-plugin master: [DNM] Test IPv6 wildcard ::/0 in CephFS NFS https://review.opendev.org/c/openstack/manila-tempest-plugin/+/988824 | 15:00 |
| opendevreview | Ivan Anfimov proposed openstack/manila-ui master: Sync releasenotes tox env with previous zuul job command https://review.opendev.org/c/openstack/manila-ui/+/988858 | 17:59 |
| opendevreview | Ivan Anfimov proposed openstack/manila-ui master: Sync releasenotes tox env with previous zuul job command https://review.opendev.org/c/openstack/manila-ui/+/988858 | 18:00 |
| opendevreview | Ivan Anfimov proposed openstack/manila-ui master: Sync releasenotes tox env with previous zuul job command https://review.opendev.org/c/openstack/manila-ui/+/988858 | 18:02 |
| opendevreview | Ivan Anfimov proposed openstack/manila-ui master: Sync releasenotes tox env with previous zuul job command https://review.opendev.org/c/openstack/manila-ui/+/988858 | 18:02 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!