| *** lseki has quit IRC | 00:09 | |
| *** dviroel_ has quit IRC | 01:17 | |
| *** senrique__ has joined #openstack-manila | 01:19 | |
| *** whoami-rajat has joined #openstack-manila | 01:29 | |
| *** vh_ has joined #openstack-manila | 02:00 | |
| *** senrique__ has quit IRC | 02:04 | |
| openstackgerrit | Merged openstack/manila master: NetApp cDOT assume disabled compression on empty result https://review.openstack.org/619570 | 02:08 |
|---|---|---|
| *** senrique__ has joined #openstack-manila | 02:20 | |
| *** senrique__ has quit IRC | 02:27 | |
| *** vh_ has quit IRC | 02:43 | |
| openstackgerrit | Merged openstack/manila stable/rocky: Fix driver filter to not check share_backend_name https://review.openstack.org/643767 | 03:44 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila stable/queens: Fix driver filter to not check share_backend_name https://review.openstack.org/644723 | 03:54 |
| *** e0ne has joined #openstack-manila | 07:17 | |
| *** pcaruana has joined #openstack-manila | 07:20 | |
| *** e0ne has quit IRC | 07:27 | |
| *** tosky has joined #openstack-manila | 07:29 | |
| *** e0ne has joined #openstack-manila | 07:42 | |
| *** e0ne has quit IRC | 08:15 | |
| openstackgerrit | Merged openstack/manila master: Check all_tenants value in security_service api https://review.openstack.org/620197 | 09:58 |
| openstackgerrit | Helen Walsh proposed openstack/manila stable/rocky: VMAX manila - deprecate old tags correctly https://review.openstack.org/644809 | 10:12 |
| openstackgerrit | Merged openstack/manila master: Check all_tenants value in share_networks api https://review.openstack.org/618991 | 10:14 |
| openstackgerrit | Merged openstack/puppet-manila master: Prepare Stein RC1 https://review.openstack.org/643826 | 10:15 |
| openstackgerrit | Helen Walsh proposed openstack/manila stable/queens: VMAX manila doc - use of correct VMAX tags https://review.openstack.org/644814 | 10:29 |
| openstackgerrit | Helen Walsh proposed openstack/manila stable/queens: VMAX manila doc - SSL Support https://review.openstack.org/644815 | 10:33 |
| openstackgerrit | Helen Walsh proposed openstack/manila stable/queens: Manila VMAX docs - clarify driver_handles_share_servers https://review.openstack.org/644819 | 10:35 |
| *** e0ne has joined #openstack-manila | 10:45 | |
| *** carlos_silva has joined #openstack-manila | 11:06 | |
| toabctl | tbarron, gouthamr is anybody still using run_tests.sh ? | 11:07 |
| openstackgerrit | Thomas Bechtold proposed openstack/manila master: Drop run_tests.sh and tools/colorizer.py https://review.openstack.org/644826 | 11:10 |
| *** vh_ has joined #openstack-manila | 11:15 | |
| *** vh_ has left #openstack-manila | 11:18 | |
| *** vh_ has joined #openstack-manila | 11:19 | |
| openstackgerrit | Helen Walsh proposed openstack/manila stable/queens: Manila VMAX docs - clarify snapshot support https://review.openstack.org/644829 | 11:22 |
| *** thgcorrea has joined #openstack-manila | 11:27 | |
| *** pcaruana has quit IRC | 11:38 | |
| *** dviroel_ has joined #openstack-manila | 11:44 | |
| openstackgerrit | Helen Walsh proposed openstack/manila stable/queens: Manila VMAX docs - improve pre-configurations on VMAX section https://review.openstack.org/644838 | 11:52 |
| openstackgerrit | Helen Walsh proposed openstack/manila stable/queens: Manila VMAX docs - differences between quotas https://review.openstack.org/644840 | 11:56 |
| *** e0ne has quit IRC | 12:07 | |
| *** pcaruana has joined #openstack-manila | 12:16 | |
| *** eharney has joined #openstack-manila | 12:42 | |
| *** rraja has joined #openstack-manila | 12:48 | |
| *** rraja has quit IRC | 12:54 | |
| tbarron | toabctl: no run_tests.sh is deprecated | 12:55 |
| tbarron | toabctl: has been for several releases | 12:56 |
| toabctl | tbarron, oki. I created already a changeset to remove it to see what the CI says | 12:56 |
| toabctl | https://review.openstack.org/644826 | 12:56 |
| tbarron | toabctl: did you grep through our doc to see if we still mention it anywhere? | 12:56 |
| toabctl | I didn't find anything | 12:57 |
| *** jmlowe has quit IRC | 12:58 | |
| tbarron | toabctl: thanks | 13:00 |
| *** lseki has joined #openstack-manila | 13:09 | |
| *** thgcorrea has quit IRC | 13:15 | |
| *** e0ne has joined #openstack-manila | 13:16 | |
| openstackgerrit | Merged openstack/manila stable/queens: Fix driver filter to not check share_backend_name https://review.openstack.org/644723 | 13:28 |
| *** senrique__ has joined #openstack-manila | 13:42 | |
| *** senrique__ has quit IRC | 13:44 | |
| *** enriquetaso has joined #openstack-manila | 13:44 | |
| *** thgcorrea has joined #openstack-manila | 13:48 | |
| *** jgrosso has joined #openstack-manila | 13:56 | |
| *** eharney has quit IRC | 14:03 | |
| openstackgerrit | Merged openstack/python-manilaclient master: Update master for stable/stein https://review.openstack.org/644164 | 14:04 |
| openstackgerrit | Merged openstack/python-manilaclient stable/stein: Update .gitreview for stable/stein https://review.openstack.org/644162 | 14:13 |
| openstackgerrit | Merged openstack/manila stable/queens: Destroy type quotas when a share type is deleted https://review.openstack.org/644585 | 14:14 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila stable/pike: Destroy type quotas when a share type is deleted https://review.openstack.org/644877 | 14:37 |
| openstackgerrit | Merged openstack/manila stable/rocky: VMAX manila - deprecate old tags correctly https://review.openstack.org/644809 | 14:45 |
| openstackgerrit | Merged openstack/manila stable/queens: VMAX manila doc - use of correct VMAX tags https://review.openstack.org/644814 | 14:45 |
| openstackgerrit | Merged openstack/manila stable/queens: VMAX manila doc - SSL Support https://review.openstack.org/644815 | 14:45 |
| openstackgerrit | Merged openstack/manila stable/queens: Manila VMAX docs - clarify driver_handles_share_servers https://review.openstack.org/644819 | 14:45 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila stable/pike: Fix driver filter to not check share_backend_name https://review.openstack.org/644886 | 14:52 |
| tbarron | carlos_silva: lseki: how is it going with https://review.openstack.org/#/c/644560/ ? | 14:58 |
| *** jmlowe has joined #openstack-manila | 15:00 | |
| *** enriquetaso has quit IRC | 15:08 | |
| carlos_silva | lseki: is going to propose a new patch in few hours | 15:15 |
| carlos_silva | we are waiting for our internal CI output before creating a new patch | 15:16 |
| carlos_silva | s/a new patch/a new upstream patch | 15:18 |
| carlos_silva | tbarron lseki changed some things in manila-tempest-plugin and the changes seems effective | 15:24 |
| tbarron | carlos_silva: lseki cool, just checking :) | 15:28 |
| *** eharney has joined #openstack-manila | 15:40 | |
| openstackgerrit | Lucio Seki proposed openstack/manila-tempest-plugin master: Delete servers before deleting its share network https://review.openstack.org/644905 | 15:54 |
| openstackgerrit | Carlos Eduardo proposed openstack/manila master: Fix server delete attempt along with share net deletion https://review.openstack.org/644560 | 16:01 |
| tbarron | carlos_silva: lseki fingers crossed :) | 16:02 |
| lseki | tbarron, carlos_silva great! | 16:04 |
| carlos_silva | :) | 16:07 |
| *** enriquetaso has joined #openstack-manila | 16:11 | |
| openstackgerrit | jacky06 proposed openstack/python-manilaclient master: Update hacking version https://review.openstack.org/641399 | 16:12 |
| openstackgerrit | Merged openstack/python-manilaclient stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein https://review.openstack.org/644163 | 16:21 |
| *** jmlowe has quit IRC | 16:21 | |
| *** altlogbot_2 has joined #openstack-manila | 16:40 | |
| *** altlogbot_2 has quit IRC | 16:41 | |
| *** jmlowe has joined #openstack-manila | 16:48 | |
| *** thgcorrea has quit IRC | 16:55 | |
| *** thgcorrea has joined #openstack-manila | 16:57 | |
| *** e0ne has quit IRC | 17:07 | |
| *** altlogbot_1 has joined #openstack-manila | 17:24 | |
| *** altlogbot_1 has quit IRC | 17:24 | |
| *** altlogbot_2 has joined #openstack-manila | 17:26 | |
| *** altlogbot_2 has quit IRC | 17:28 | |
| *** altlogbot_0 has joined #openstack-manila | 17:29 | |
| *** raissa has joined #openstack-manila | 17:31 | |
| openstackgerrit | Sofia Enriquez proposed openstack/manila-specs master: Manila support in python-openstackclient https://review.openstack.org/644218 | 17:55 |
| openstackgerrit | Lucio Seki proposed openstack/manila-tempest-plugin master: Delete servers before deleting its share network https://review.openstack.org/644905 | 18:00 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila stable/pike: Destroy type quotas when a share type is deleted https://review.openstack.org/644877 | 18:00 |
| *** enriquetaso has quit IRC | 18:02 | |
| openstackgerrit | Carlos Eduardo proposed openstack/manila master: Fix server delete attempt along with share net deletion https://review.openstack.org/644560 | 18:09 |
| *** e0ne has joined #openstack-manila | 18:17 | |
| *** kopecmartin is now known as kopecmartin|off | 18:19 | |
| *** irclogbot_1 has quit IRC | 18:21 | |
| openstackgerrit | Tom Barron proposed openstack/manila master: Only allow IP access type for CephFS NFS https://review.openstack.org/640185 | 18:23 |
| *** irclogbot_3 has joined #openstack-manila | 18:24 | |
| *** irclogbot_3 has quit IRC | 18:27 | |
| *** irclogbot_3 has joined #openstack-manila | 18:28 | |
| *** amito has quit IRC | 18:31 | |
| *** amito has joined #openstack-manila | 18:33 | |
| tbarron | gouthamr: carlos_silva lseki bswartz carthaca ganso jgrosso toabctl xyang here are our outstanding reviews on master: | 18:35 |
| tbarron | https://review.openstack.org/#/q/status:open+project:openstack/manila+branch:master | 18:35 |
| tbarron | The only one left that we're trying to get in to rc1 is: https://review.openstack.org/#/c/644560/ | 18:35 |
| tbarron | which depends on manila-tempest-plugin fix https://review.openstack.org/#/c/644905/ | 18:36 |
| tbarron | i'm ok with the changes from code inspection | 18:38 |
| gouthamr | tbarron: ack | 18:38 |
| tbarron | vkmc: gouthamr ^^ have you had a chance to look at the tempest-plugin changes proposed? | 18:39 |
| gouthamr | tbarron: although, i want to ask a question of ganso about what he thought about this... | 18:39 |
| gouthamr | tbarron: technically, when a share server is unmanaged, you should be able to delete the share network, and not delete the unmanaged share server... | 18:39 |
| gouthamr | tbarron: don't understand entirely *why* we'd want to stop users from removing it.. | 18:40 |
| gouthamr | is it because of port allocations from neutron that need to go away? | 18:40 |
| tbarron | gouthamr: that was my assumption | 18:41 |
| gouthamr | that'd be wrong, wouldn't it... i mean, we don't own those ports, the administrator does.. | 18:42 |
| gouthamr | i'll just recheck that code path, afaiu, case 1) Share server is created by manila, but admin unmanaged a share - so, share server isn't going to be deleted... admin needs to do it, it makes sense to keep the share network around, unless the administrator wants to unmanage or delete the share server first... | 18:45 |
| tbarron | if we delete the share network then it frees up the allocations so we no longer have neutron reservation of those resources | 18:45 |
| gouthamr | a lesser privileged user in the project may want to cleanup the share network, but can't because there's a share server they don't know about that is blocking the share network deletion | 18:45 |
| tbarron | gouthamr: ok follow up with ganso and if there's an issue here file a bug | 18:48 |
| gouthamr | i think in this situation, we can prevent the share network deletion but let the error message be a little more user friendly: i.e, "there may be shares on this share network that we don't know about. Please contact your administrator to clarify" | 18:48 |
| tbarron | gouthamr: oh, you are suggesting a tweak. | 18:48 |
| tbarron | gouthamr: post it right away then. | 18:49 |
| tbarron | gouthamr: we have a couple of hours while https://review.openstack.org/#/c/644560/ and https://review.openstack.org/#/c/644905/ make their way through the system | 18:49 |
| tbarron | gouthamr: we can look at that tweak in parallel | 18:50 |
| gouthamr | ack | 18:50 |
| tbarron | once we merge these though I plan to propose rc1 | 18:50 |
| tbarron | other projects have already released ... | 18:50 |
| tbarron | don't want to get caught in a logjam | 18:50 |
| tbarron | any bug fixes are fair to merge until rc1 is proposed though | 18:51 |
| tbarron | If there's nothing else on rc1 let's talk backports since I want to propose stable/* releases this week as well | 18:52 |
| *** e0ne has quit IRC | 18:53 | |
| tbarron | we don't have anything pending on stable/rocky any more: https://review.openstack.org/#/q/status:open+project:openstack/manila+branch:stable/rocky | 18:53 |
| gouthamr | ack on rocky | 18:54 |
| tbarron | stable queens has no critical backports: https://review.openstack.org/#/q/status:open+project:openstack/manila+branch:stable/queens | 18:54 |
| tbarron | there's a couple doc backports and one can debate whether they should be backported or not but they're harmless | 18:55 |
| gouthamr | ack on queens as well, the ones you workflowed don't need to be in a release; i.e, they'll update https://docs.openstack.org/manila/queens | 18:55 |
| tbarron | gouthamr: right | 18:55 |
| tbarron | stable/pike is going to go EOM soon | 18:56 |
| gouthamr | https://review.openstack.org/#/q/status:open+project:openstack/manila+branch:stable/pike has a couple of my backports that would be nice before marking EM | 18:56 |
| gouthamr | EOM != EM :) | 18:56 |
| tbarron | yeah | 18:56 |
| tbarron | picky picky picky | 18:56 |
| gouthamr | :P | 18:57 |
| tbarron | but correct! | 18:57 |
| tbarron | Let's see if we can get those two merged | 18:57 |
| tbarron | gouthamr: actually I'd like to merge https://review.openstack.org/#/c/635887/ as well | 18:58 |
| tbarron | it's test-only but might as well include it in the last release | 18:59 |
| gouthamr | tbarron: ack, so we can fix the tempest run within the ceph devstack plugin? | 18:59 |
| tbarron | gouthamr: that's a separate fix, will need to be done in branchelss devstack-plugin-ceph | 19:00 |
| *** altlogbot_0 has quit IRC | 19:00 | |
| tbarron | gouthamr: to have it run tempest in the way pike wants on pike and the way it currently runs on more recent branches | 19:00 |
| tbarron | ugly | 19:00 |
| gouthamr | tbarron: ack, i wonder if you can write a condition on ZUUL_BRANCH and run tox -eplugin instead of "tempest run" | 19:01 |
| tbarron | but this fix will get devstack completing instead of failing b/c of conflicting ceph versions | 19:01 |
| tbarron | gouthamr: maybe, I'll have to pick your brain | 19:01 |
| gouthamr | tbarron: +1, W'ed | 19:01 |
| tbarron | gouthamr: thanks | 19:01 |
| *** altlogbot_3 has joined #openstack-manila | 19:02 | |
| tbarron | OK, I think that does it for backports? | 19:02 |
| tbarron | I'll start cutting releases after we see the stuff we mentioned merge (and after my workday meetings). | 19:03 |
| gouthamr | yes | 19:03 |
| tbarron | gouthamr: thanks for the help! | 19:03 |
| gouthamr | thank you tbarron | 19:03 |
| gouthamr | we might need to work on the rc1 blocker a bit more.. the latest change breaks tempest for the dummy driver | 19:03 |
| *** altlogbot_3 has quit IRC | 19:06 | |
| *** altlogbot_0 has joined #openstack-manila | 19:07 | |
| *** jmlowe has quit IRC | 19:15 | |
| *** irclogbot_3 has quit IRC | 19:16 | |
| *** irclogbot_2 has joined #openstack-manila | 19:17 | |
| *** raissa has quit IRC | 19:26 | |
| *** irclogbot_2 has quit IRC | 19:30 | |
| *** jmlowe has joined #openstack-manila | 19:30 | |
| *** irclogbot_3 has joined #openstack-manila | 19:31 | |
| *** altlogbot_0 has quit IRC | 19:34 | |
| *** altlogbot_3 has joined #openstack-manila | 19:35 | |
| ganso | gouthamr: hey | 19:39 |
| ganso | gouthamr: what is that you would like to ask? | 19:39 |
| ganso | gouthamr: I was in a meeting, saw the notification just now | 19:40 |
| *** eharney has quit IRC | 19:50 | |
| *** thgcorrea has quit IRC | 19:59 | |
| gouthamr | ganso: hey, thanks for the ping.. was in a meeting myself | 20:00 |
| gouthamr | ganso: have you seen the bug: https://bugs.launchpad.net/manila/+bug/1820118 | 20:01 |
| openstack | Launchpad bug 1820118 in Manila "Manila unexpectedly attempts to delete a share server when deleting a share network" [Medium,In progress] - Assigned to Carlos Eduardo (silvacarlose) | 20:01 |
| ganso | gouthamr: yes | 20:01 |
| gouthamr | ganso: and the fix proposed: https://review.openstack.org/#/c/644905/ | 20:01 |
| ganso | gouthamr: no | 20:02 |
| * ganso looks | 20:02 | |
| gouthamr | ganso: cool, please review it if possible.. i feel it leaks out the share server abstraction a bit | 20:02 |
| *** eharney has joined #openstack-manila | 20:02 | |
| ganso | gouthamr: why | 20:02 |
| gouthamr | ganso: i'd like to know what you think, and how we should handle it.. | 20:03 |
| gouthamr | ganso: by default, in our perception of how this is used... non-privileged users can delete share networks since they can create them | 20:03 |
| *** vh_ has quit IRC | 20:03 | |
| gouthamr | ganso: but now we have something that an admin does (unmanaging a share / managing a share server into the user's share network) that prevents them from deleting their share networks | 20:04 |
| ganso | gouthamr: I agree with https://review.openstack.org/#/c/644560, but the error message should be user-facing. Users don't know what share servers are | 20:04 |
| gouthamr | ganso: yes, 1) i think the error message can be more appropriate... | 20:05 |
| openstackgerrit | Merged openstack/manila master: Drop run_tests.sh and tools/colorizer.py https://review.openstack.org/644826 | 20:05 |
| openstackgerrit | Merged openstack/manila stable/queens: Manila VMAX docs - clarify snapshot support https://review.openstack.org/644829 | 20:05 |
| openstackgerrit | Merged openstack/manila stable/queens: Manila VMAX docs - improve pre-configurations on VMAX section https://review.openstack.org/644838 | 20:05 |
| openstackgerrit | Merged openstack/manila stable/queens: Manila VMAX docs - differences between quotas https://review.openstack.org/644840 | 20:05 |
| ganso | gouthamr: I don't agree with the test patch. I don't think the core needs to be changed (as the patch is doing), but a separate test should be created to test the new error message | 20:07 |
| gouthamr | ganso: 2) i'm thinking where this might go wrong.. maybe this stuff can be clarified by documentation somewhere... unmanaging a share from a share server or having a managed share server on a share network does not allow users to delete the underlying share network | 20:07 |
| ganso | gouthamr: and the test patch should depend on the fix. Nothing else should change | 20:08 |
| ganso | gouthamr: /the core needs to be changed/the base class needs to be changed | 20:08 |
| ganso | gouthamr: and I think it is better for us to agree on the proper solution here first before -1'ing the patch | 20:09 |
| gouthamr | ganso: yes, unsure if lseki, carlos_silva are still around | 20:10 |
| ganso | gouthamr: I don't see a problem with #2 you mentioned above | 20:10 |
| carlos_silva | yeah, I'm here | 20:10 |
| ganso | gouthamr: as long as the error message is clear | 20:10 |
| ganso | gouthamr: something along the lines of "there may be shares on this share network that we don't know about. Please contact your administrator to clarify" as you suggested should be fine | 20:11 |
| lseki | ganso: > I don't think the base class needs to be changed | 20:12 |
| lseki | I needed to change it to make sure that the share-servers are deleted before the share-network upon testclass teardown | 20:12 |
| gouthamr | ganso: cool, that works for me... i think it'd be prudent to also log a warning message in the share manager when a share server is managed or when a share is unmanaged that the share network will not be deleted unless the associated share server is unmanaged or deleted | 20:13 |
| lseki | otherwise it would raise the exception carlos_silva's patch added, sporadically | 20:13 |
| gouthamr | ganso: hmmm, i suggested they handle this cleanup in the base class, do you think we should rather do this everywhere the conditions above are true? (share unmanaged from share server, or share server is managed) | 20:14 |
| ganso | lseki: it isn't a problem for any of the current tests. And by making sure you delete the share server in all of the manage/unmanage tests (where is_auto_deletable will be False) will be making sure of that. The change as you proposed affects all tests and then all other tests will stop testing the deletion of share servers when deleting a share network (when all share servers are is_auto_deletable = True) | 20:14 |
| ganso | gouthamr: yes | 20:15 |
| ganso | gouthamr: I don't think the warning is needed, but it is ok to add | 20:15 |
| *** eharney has quit IRC | 20:22 | |
| openstackgerrit | Merged openstack/manila stable/pike: Only run the needed services for CephFS jobs https://review.openstack.org/635887 | 20:22 |
| lseki | ganso: gouthamr we realized the need to adapt the functional tests when several tests failed while cheking carlos_silva's patch, including those tests that don't manage/unmanage things at all (such as AvailabilityZonesTest, ShareLimitsTest, etc). | 20:27 |
| ganso | why? why carlos_silva's change broke other tests where is_auto_deletable=True ? | 20:28 |
| lseki | ganso my suspect is that teardown of those test classes is trying to delete the share-network before share-servers are deleted | 20:30 |
| ganso | lseki: then that problem would already be there and unrelated to carlos_silva's changes | 20:30 |
| lseki | ganso yes, but the error was not being raised, until carlos_silva's change started to block it at the api | 20:31 |
| ganso | lseki: perhaps this needs further invesgation then, as it wasn't supposed to happen | 20:33 |
| lseki | and by forcing the share-servers' deletion before the share-network deletion, modifying the base class, those errors stopped appearing | 20:34 |
| carlos_silva | ganso: you can check the tests output before lseki changes here: http://13.56.179.158/logs/19/1019/3/downstream-functional/manila-cDOT-ss/418763a/console.html.gz | 20:35 |
| ganso | lseki: what if you delete the share server within the test manila_tempest_tests.tests.api.admin.test_share_manage.ManageNFSShareTest ? | 20:36 |
| ganso | lseki: you are probably going to need 3 patches | 20:37 |
| ganso | lseki: 1) to change how share servers are deleted on any test that causes is_auto_deletable to be Fals | 20:37 |
| ganso | 2) carlos_silva's changes | 20:38 |
| ganso | 3) new test that validates carlos_silva's changes | 20:38 |
| lseki | > what if you delete the share server within the test ... | 20:39 |
| lseki | ganso: it would make the ManageNFSShareTest tests pass, but probably keep the other tests (that doesn't manage/unmanage at all) failing | 20:39 |
| ganso | lseki: I think it is worth a shot. The reason other tests which seem unrelated are failing is most likely because they are sharing the same share_network as the ManageNFSTest class | 20:40 |
| ganso | lseki: I didn't check the code of those tests to see if they are creating a share_network of their own, but I would assume they are not | 20:42 |
| *** irclogbot_3 has quit IRC | 20:45 | |
| *** irclogbot_0 has joined #openstack-manila | 20:47 | |
| *** e0ne has joined #openstack-manila | 20:52 | |
| *** altlogbot_3 has quit IRC | 20:53 | |
| *** altlogbot_1 has joined #openstack-manila | 20:55 | |
| openstackgerrit | Carlos Eduardo proposed openstack/manila master: Fix server delete attempt along with share net deletion https://review.openstack.org/644560 | 20:58 |
| openstackgerrit | Soledad Kuczala proposed openstack/manila master: Adding documentation for user messages in Manila https://review.openstack.org/645010 | 21:14 |
| *** eharney has joined #openstack-manila | 21:19 | |
| openstackgerrit | Soledad Kuczala proposed openstack/manila master: Adding documentation for user messages in Manila https://review.openstack.org/645010 | 21:19 |
| lseki | ganso: yeah, I tried with one of positive tests here and your approach worked | 21:24 |
| lseki | so I'll change every single test that unmanages share/share-server and delete the share-server manually before the test ends | 21:25 |
| *** jgrosso has quit IRC | 21:25 | |
| lseki | and remove the change in the base class | 21:27 |
| lseki | and cross fingers hoping that all other tests will continue passing | 21:27 |
| *** whoami-rajat has quit IRC | 21:29 | |
| *** e0ne has quit IRC | 22:00 | |
| *** pcaruana has quit IRC | 22:11 | |
| openstackgerrit | Merged openstack/manila stable/pike: Destroy type quotas when a share type is deleted https://review.openstack.org/644877 | 22:26 |
| openstackgerrit | Merged openstack/manila stable/pike: Fix driver filter to not check share_backend_name https://review.openstack.org/644886 | 22:38 |
| openstackgerrit | Tom Barron proposed openstack/manila master: Only allow IP access type for CephFS NFS https://review.openstack.org/640185 | 22:45 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila master: DNM: Test Python3 LVM Job https://review.openstack.org/623061 | 23:01 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila master: DNM: Test Python3 LVM Job https://review.openstack.org/623061 | 23:04 |
| *** tosky has quit IRC | 23:31 | |
| openstackgerrit | Carlos Eduardo proposed openstack/manila master: Fix server delete attempt along with share net deletion https://review.openstack.org/644560 | 23:41 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!