| openstackgerrit | Merged openstack/puppet-manila master: Add Puppet Litmus https://review.opendev.org/749116 | 00:58 |
|---|---|---|
| *** LinPeiWen has joined #openstack-manila | 01:08 | |
| *** vishalmanchanda has joined #openstack-manila | 01:58 | |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: Add share server migration https://review.opendev.org/747043 | 02:02 |
| openstackgerrit | Douglas Viroel proposed openstack/python-manilaclient master: Add commands for share server migration https://review.opendev.org/747204 | 02:02 |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: [NetApp] Add support for share server migration https://review.opendev.org/747048 | 02:08 |
| dviroel | gouthamr: o/ | 02:44 |
| dviroel | gouthamr: you were investigating these error earlier? ERROR: InvocationError for command /home/zuul/src/opendev.org/openstack/manila/.tox/pep8/bin/flake8 (exited with code 1) | 02:45 |
| dviroel | or something similar | 02:45 |
| gouthamr | dviroel: https://review.opendev.org/#/c/743137/ | 02:47 |
| gouthamr | looks like zuul ate my recheck | 02:48 |
| dviroel | lol | 02:48 |
| gouthamr | dviroel: but you can workflow it if you like, it'll unblock further patches | 02:49 |
| dviroel | gouthamr: done. I'll do my rechecks tomorrow | 02:53 |
| gouthamr | dviroel: thanks! | 02:53 |
| gouthamr | dviroel: goodnight! | 02:53 |
| dviroel | gouthamr: ty | 02:54 |
| dviroel | gouthamr: goodnight | 02:54 |
| openstackgerrit | Merged openstack/manila master: [goal] Migrate tox based testing to ubuntu focal https://review.opendev.org/743137 | 06:21 |
| openstackgerrit | Merged openstack/manila stable/stein: [Glusterfs] Fix delete share, mount point not disconnected https://review.opendev.org/750182 | 07:31 |
| *** tosky has joined #openstack-manila | 07:32 | |
| *** maaritamm has joined #openstack-manila | 08:04 | |
| openstackgerrit | Lin PeiWen proposed openstack/manila stable/rocky: [Glusterfs] Fix delete share, mount point not disconnected https://review.opendev.org/750183 | 08:16 |
| *** toabctl has quit IRC | 10:00 | |
| *** toabctl has joined #openstack-manila | 10:03 | |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: Add share server migration https://review.opendev.org/747043 | 10:29 |
| openstackgerrit | Douglas Viroel proposed openstack/python-manilaclient master: Add commands for share server migration https://review.opendev.org/747204 | 10:40 |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: Add share server migration https://review.opendev.org/747043 | 11:52 |
| *** raildo has joined #openstack-manila | 11:52 | |
| openstackgerrit | Liron Kuchlani proposed openstack/manila-tempest-plugin master: Remove unused "_ping_host_from_export_location" method https://review.opendev.org/750971 | 12:12 |
| openstackgerrit | Merged openstack/manila stable/rocky: [Glusterfs] Fix delete share, mount point not disconnected https://review.opendev.org/750183 | 12:17 |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: Improve IPv6 documentation for access rules https://review.opendev.org/745232 | 12:36 |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: Improve IPv6 documentation for access rules https://review.opendev.org/745232 | 13:20 |
| openstackgerrit | Merged openstack/manila master: Update user doc index https://review.opendev.org/745292 | 14:52 |
| *** andrebeltrami has joined #openstack-manila | 14:56 | |
| *** felipe_rodrigues has joined #openstack-manila | 15:09 | |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: Add share server migration for Dummy driver https://review.opendev.org/747054 | 15:30 |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: [Container driver] Adds share and share server migration https://review.opendev.org/740831 | 15:32 |
| openstackgerrit | Lukas Piwowarski proposed openstack/devstack-plugin-ceph master: Create tox environment to build pdf documentation https://review.opendev.org/751036 | 15:45 |
| openstackgerrit | Merged openstack/python-manilaclient master: Implement OSC share quota commands https://review.opendev.org/703992 | 15:52 |
| openstackgerrit | Tom Barron proposed openstack/manila-ui master: Make ``cover`` job non-voting https://review.opendev.org/751042 | 16:24 |
| vkmc | tbarron, xD | 16:42 |
| vkmc | https://bugs.launchpad.net/manila-ui/+bug/1895172 | 16:42 |
| openstack | Launchpad bug 1895172 in manila-ui "Improve testing coverage for manila-ui/api/manila.py" [High,Triaged] - Assigned to Victoria Martinez de la Cruz (vkmc) | 16:42 |
| openstackgerrit | Maari Tamm proposed openstack/python-manilaclient master: Implement the first half of OSC share snapshots commands https://review.opendev.org/738691 | 16:53 |
| * gouthamr if anyone else here is wondering, its deliberate that we're not including OSC in the release notes - we're not shadow launching it, but we wanted to achieve a certain level of parity with manilaclient before calling it ready - it may be a poor decision, we can discuss that - as tbarron would say, smack me with a 2x4 and let me know! | 16:56 | |
| openstackgerrit | Andre Beltrami proposed openstack/manila-tempest-plugin master: Add tests for share server migration https://review.opendev.org/747157 | 16:57 |
| tbarron | vkmc: improving coverage would be good (so i'm all for the bug) but your patch doesn't cause any significant regression | 17:08 |
| tbarron | false precison | 17:09 |
| tbarron | precision | 17:09 |
| vkmc | tbarron, I know | 17:10 |
| vkmc | if we can get it merged and then add the test coverage after ff | 17:10 |
| vkmc | that would be | 17:10 |
| vkmc | glorious | 17:10 |
| openstackgerrit | Merged openstack/manila master: Improve IPv6 documentation for access rules https://review.opendev.org/745232 | 17:44 |
| openstackgerrit | Andre Beltrami proposed openstack/manila-tempest-plugin master: Add tests for share server migration https://review.opendev.org/747157 | 17:46 |
| *** gmann is now known as gmann_afk | 18:11 | |
| gouthamr | vkmc carloss tbarron: manila-ui won't be released until rc - so we have time to wrap up reviews on the user messages feature | 18:19 |
| tbarron | gouthamr: ack | 18:19 |
| carloss | ack | 18:20 |
| gouthamr | thanks for all the work so far... we'll try to get it in asap, but, no need to rush in case you need time :) | 18:20 |
| openstackgerrit | Merged openstack/manila-ui master: Make ``cover`` job non-voting https://review.opendev.org/751042 | 18:22 |
| tbarron | andrebeltrami: dviroel questions on https://review.opendev.org/#/c/740532/ | 18:43 |
| tbarron | what is the user experience if | 18:43 |
| vkmc | gouthamr++ | 18:44 |
| tbarron | #1) they use a share type with netapp:adaptive_qos_policy_group=... and do not have cluster scoped account? | 18:44 |
| dviroel | #1) It will fail on volume creation in the current versions of ONTAP | 18:45 |
| dviroel | #1) it also fails with qos-policy-group today =/ | 18:46 |
| tbarron | ok, that was my followup | 18:47 |
| tbarron | we could theoretically do some check and handle it more gracefully but at least it's consistent | 18:47 |
| gouthamr | how does it fail qos-policy-group today? | 18:47 |
| dviroel | gouthamr: vsadmin cant modify qos-policy-group for a volume | 18:48 |
| dviroel | ::> volume modify -volume test -qos-policy-group ENG | 18:49 |
| gouthamr | dviroel: sure, but qos policies can't be applied when vsadmin too, https://github.com/openstack/manila/blob/6f58af1ae39b4151db5feaa72deecafb9365ca16/manila/share/drivers/netapp/dataontap/cluster_mode/lib_base.py#L352-L357 | 18:49 |
| dviroel | gouthamr: true, it can be avoided too. Michael concerns were that this might change in the next releases of ONTAP, and create/modify a volume with a pre-created policy will work with vsadmin | 18:53 |
| dviroel | but not create a new qos-policy-group | 18:53 |
| gouthamr | dviroel: ack, that part i'd leave it to you folks to convey via documentation - it is simple though to ensure you have the check now | 18:55 |
| tbarron | so that's an argument for making two checks instead of one, so you can adjust each independently depending on ontap version, right? | 18:55 |
| gouthamr | and remove it when you have a version that it is supported in? | 18:55 |
| tbarron | #2) what is the user experience when using such a share type and a DHSS=True back end? | 18:56 |
| openstackgerrit | Felipe Rodrigues proposed openstack/manila master: [NetApp] Enables configuring NFS transfer limits https://review.opendev.org/746568 | 18:56 |
| openstackgerrit | Merged openstack/python-manilaclient master: Implement the first half of OSC share snapshots commands https://review.opendev.org/738691 | 18:56 |
| gouthamr | tbarron: this new thing has no relevance to the older qos specs - although in theory could have been built on top of it :) | 18:58 |
| gouthamr | tbarron: +1 - dviroel: that check can easily be performed, no? | 18:58 |
| tbarron | gouthamr: understood, i was just wondering if the failure expreiences were similar | 18:58 |
| dviroel | #2) it fails on volume creation too. The information about error will be on available in log messages. | 18:59 |
| gouthamr | dviroel: if its not compatible with replication and DHSS=True, maybe you can fail provisioning when those are used together? | 18:59 |
| tbarron | ok that's #2 and #3) but fair nuff, my question is whether these can be checked and the ontap error avoided | 19:00 |
| dviroel | gouthamr: tbarron: yes, there are some minor fixes that could provide a better user experience, but its up to Michael to provide them at this moment, since we are working hard on the other ones. | 19:02 |
| dviroel | gouthamr: tbarron: so its up to you folks, to hold this one and wait for improvements if you guys think that is better | 19:02 |
| tbarron | dviroel: ty | 19:09 |
| dviroel | gouthamr: tbarron if you guys think that we can land in this way and provide fixes later, its ok for me too | 19:13 |
| tbarron | dviroel: that's what I said on the review | 19:13 |
| dviroel | tbarron: tks tom | 19:14 |
| *** maaritamm has left #openstack-manila | 20:26 | |
| *** openstackgerrit has quit IRC | 20:36 | |
| dviroel | tbarron: gouthamr we have dummy and container running and passing for share server migration tests here: https://review.opendev.org/#/c/747157/ | 20:36 |
| tbarron | dviroel: kk, will look again in a bit :D | 20:59 |
| dviroel | :) | 21:00 |
| *** openstackgerrit has joined #openstack-manila | 21:20 | |
| openstackgerrit | Victoria Martinez de la Cruz proposed openstack/manila master: Totalcount returned by pagination query is wrong https://review.opendev.org/703025 | 21:20 |
| *** raildo has quit IRC | 21:30 | |
| dviroel | vkmc: carloss is working on this one ^ | 21:30 |
| vkmc | dviroel, oh cool | 21:30 |
| vkmc | I rebased it only | 21:30 |
| vkmc | :) | 21:30 |
| dviroel | vkmc: hope to fix those issues soon | 21:30 |
| vkmc | perfect | 21:30 |
| vkmc | I'll continue looking into other patches | 21:30 |
| carloss | yep... hopefully I'll submit an updated version by tomorrow | 21:30 |
| dviroel | carloss++ | 21:31 |
| vkmc | carloss++ | 21:32 |
| *** vishalmanchanda has quit IRC | 22:21 | |
| openstackgerrit | Merged openstack/manila-tempest-plugin master: Remove unused "_ping_host_from_export_location" method https://review.opendev.org/750971 | 22:23 |
| openstackgerrit | Douglas Viroel proposed openstack/manila master: Add share server migration for Dummy driver https://review.opendev.org/747054 | 22:24 |
| *** tosky has quit IRC | 22:42 | |
| *** gmann_afk is now known as gmann | 22:49 | |
| openstackgerrit | Merged openstack/manila master: Add share server migration for Dummy driver https://review.opendev.org/747054 | 23:10 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!