opendevreview | Merged openstack/manila master: NetApp ONTAP: REST transition - Data motion related functions https://review.opendev.org/c/openstack/manila/+/871104 | 00:40 |
---|---|---|
opendevreview | Merged openstack/manila master: NetApp ONTAP: REST transition - DHSS True functions https://review.opendev.org/c/openstack/manila/+/871105 | 00:46 |
opendevreview | Felipe Rodrigues proposed openstack/manila master: [NetApp] Configure VLAN/MTU through subnet metadata https://review.opendev.org/c/openstack/manila/+/869708 | 01:29 |
felipe_rodrigues | gouthamr: I fixed what you proposed. Tested in my env, it should work. Fixed uts too. Waiting for community CI, so we can trigger the NetApp CI (probably it will pass, since the "skip if no subnet metadata" code is the same as before and the tests always run without the metadata) | 01:35 |
gouthamr | felipe_rodrigues: hey thanks! there’s an issue if the user doesn’t provide an MTU.. a unit test case for this could help… | 02:38 |
gouthamr | if you fix it up during your day, carloss can merge on my behalf… that’s my only concern, rest LGTM | 02:38 |
opendevreview | Merged openstack/manila master: [NetApp] Configure VLAN/MTU through subnet metadata https://review.opendev.org/c/openstack/manila/+/869708 | 08:00 |
opendevreview | Merged openstack/manila master: tox4 needs allowlist_externals for fast8 https://review.opendev.org/c/openstack/manila/+/874645 | 08:01 |
opendevreview | kiran pawar proposed openstack/manila master: Fix db query for service cleanup. https://review.opendev.org/c/openstack/manila/+/875002 | 11:12 |
opendevreview | kiran pawar proposed openstack/manila master: Fix db query for service cleanup. https://review.opendev.org/c/openstack/manila/+/875002 | 12:34 |
opendevreview | Merged openstack/manila master: [CI] Bump timeout for the migrations test case https://review.opendev.org/c/openstack/manila/+/874800 | 13:30 |
*** chuanm6 is now known as chuanm | 20:19 | |
opendevreview | Merged openstack/manila master: Fix exception in share transfer accept routine https://review.opendev.org/c/openstack/manila/+/874715 | 21:07 |
opendevreview | Merged openstack/manila master: Add 'state' column in 'services' table. https://review.opendev.org/c/openstack/manila/+/859301 | 23:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!