| *** jmlowe has joined #openstack-manila | 00:37 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Add manage/unmanage of shares in DHSS=True https://review.openstack.org/635831 | 00:52 |
|---|---|---|
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila-tempest-plugin master: Add functional tests for manage/unmanage of shares in DHSS=True https://review.openstack.org/638231 | 00:53 |
| *** mmethot has quit IRC | 01:27 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Add manage/unmanage of shares in DHSS=True https://review.openstack.org/635831 | 01:49 |
| *** kshitija has quit IRC | 04:17 | |
| *** jmlowe has quit IRC | 04:29 | |
| *** kshitija has joined #openstack-manila | 05:36 | |
| *** carthaca has joined #openstack-manila | 06:57 | |
| *** arne_wiebalck has joined #openstack-manila | 07:40 | |
| *** kopecmartin|off is now known as kopecmartin | 08:04 | |
| *** pcaruana has joined #openstack-manila | 08:25 | |
| *** tosky has joined #openstack-manila | 08:27 | |
| *** e0ne has joined #openstack-manila | 08:34 | |
| *** a-pugachev has joined #openstack-manila | 09:22 | |
| *** e0ne has quit IRC | 09:40 | |
| *** e0ne has joined #openstack-manila | 10:16 | |
| *** luizbag has joined #openstack-manila | 10:18 | |
| *** dviroel_ has joined #openstack-manila | 10:20 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: [NetApp] Add manage/unmanage of share servers https://review.openstack.org/636625 | 10:41 |
| *** ganso has joined #openstack-manila | 10:44 | |
| *** mauricio_ps has joined #openstack-manila | 10:56 | |
| *** e0ne has quit IRC | 11:10 | |
| *** e0ne has joined #openstack-manila | 11:11 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Add manage/unmanage implementation to Container Driver https://review.openstack.org/636626 | 11:13 |
| *** mauricio_ps has quit IRC | 11:14 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila-tempest-plugin master: Add functional tests for manage/unmanage of shares in DHSS=True https://review.openstack.org/638231 | 11:14 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Add manage/unmanage of shares in DHSS=True https://review.openstack.org/635831 | 11:14 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Refactor Container Driver https://review.openstack.org/637926 | 11:15 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Add manage/unmanage implementation to Container Driver https://review.openstack.org/636626 | 11:15 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: [NetApp] Add manage/unmanage of share servers https://review.openstack.org/636625 | 11:16 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila-tempest-plugin master: Add functional tests for manage/unmanage of shares in DHSS=True https://review.openstack.org/638231 | 11:16 |
| *** thgcorrea has joined #openstack-manila | 11:43 | |
| *** a-pugachev has quit IRC | 11:53 | |
| *** e0ne has quit IRC | 12:00 | |
| *** e0ne has joined #openstack-manila | 12:08 | |
| *** carlos_silva has joined #openstack-manila | 12:10 | |
| *** e0ne has quit IRC | 13:04 | |
| openstackgerrit | Merged openstack/manila-tempest-plugin master: Add tests for export location changes in APIv 2.47 https://review.openstack.org/628921 | 13:51 |
| *** mmethot has joined #openstack-manila | 14:09 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila-tempest-plugin master: Temporarily disable test_manage_invalid https://review.openstack.org/635885 | 14:14 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila-tempest-plugin master: Add functional tests for manage/unmanage of shares in DHSS=True https://review.openstack.org/638231 | 14:14 |
| *** jmlowe has joined #openstack-manila | 14:18 | |
| toabctl | tbarron, wasn't there a statement in the docs that the generic driver should not be used in production? I can't find it currently | 14:28 |
| tbarron | toabctl: I don't know of such a statement - Valeriy would disagree though I believe that as you know. | 14:30 |
| toabctl | yeah. me too. I'm repeating this whenever somebody asked. | 14:30 |
| tbarron | toabctl: we say that downstream in our distro | 14:31 |
| *** pcaruana has quit IRC | 14:31 | |
| tbarron | none of this is to discourage anyone from working to make it production quaality by the way | 14:31 |
| tbarron | but it's not where I'm paid to work and I think it would be a lot of work. | 14:31 |
| toabctl | yeah | 14:35 |
| toabctl | tbarron, but it seems to be currently the only way to export shares via samba in manila. or am I wrong? | 14:36 |
| tbarron | toabctl: well there are several CIFS back ends of course and "container driver" does samba though I wouldn't consider it production quality currently | 14:38 |
| tbarron | it could probably be *developed* into a production back end | 14:38 |
| tbarron | toabctl: the deal with our open source drivers is that they don't distinguish clearly between driver and back end | 14:39 |
| toabctl | yes | 14:39 |
| tbarron | toabctl: and the back end is on the data path | 14:39 |
| tbarron | toabctl: so to have production quality it must scale out and be fault tolerant | 14:39 |
| tbarron | toabctl: we worked in our distro to do this for cephfs with nfs and shared how we do it | 14:40 |
| tbarron | toabctl: I think container driver has promise for such an extension though | 14:40 |
| tbarron | toabctl: given our investment in cephfs I'd be more inclined to work there with samba if we had | 14:41 |
| tbarron | toabctl: considerable customer demand for it | 14:41 |
| toabctl | tbarron, same here. SES (SUSE Enterprise Storage) has a samba gateway and having a driver that uses that gateway would be something I would like to see. And that should work with any samba gateway (I guess RH has something similar) | 14:42 |
| tbarron | toabctl: we aren't supporting samba for manila currently | 14:42 |
| toabctl | tbarron, we == RH ? | 14:43 |
| tbarron | toabctl: though I think it is *supportable* | 14:43 |
| tbarron | toabctl: yes, sorry | 14:43 |
| toabctl | tbarron, I think same here (at SUSE) | 14:43 |
| tbarron | toabctl: Rober Sander has a talk about using Samba CTDB with CephFS | 14:44 |
| toabctl | tbarron, I think the way for ceph & ganesha NFS is very similar at SUSE and RH (at least what I know from the slides you sent me) | 14:44 |
| tbarron | sec | 14:44 |
| tbarron | toabctl: https://www.slideshare.net/Inktank_Ceph/ceph-day-berlin-unlimited-fileserver-with-samba-ctdb-and-cephfs | 14:45 |
| toabctl | ah. thx! | 14:45 |
| tbarron | toabctl: ^^^ his use of CTDB is pretty cool but I wouldn't do NFS the way he is b/c we need NFS state (4.x +) | 14:45 |
| tbarron | toabctl: he can get wasy with "simple" NFS HA b/c he only does NFS 3 w/o locking | 14:46 |
| *** yajushiSri has joined #openstack-manila | 14:48 | |
| *** e0ne has joined #openstack-manila | 14:59 | |
| *** e0ne has quit IRC | 15:02 | |
| *** e0ne has joined #openstack-manila | 15:04 | |
| *** eharney has joined #openstack-manila | 15:04 | |
| *** zul has joined #openstack-manila | 15:12 | |
| *** pcaruana has joined #openstack-manila | 15:25 | |
| openstackgerrit | Merged openstack/manila-tempest-plugin master: Add tests for share type availability_zones extra-spec https://review.openstack.org/635703 | 15:46 |
| *** zul has quit IRC | 15:48 | |
| *** e0ne has quit IRC | 16:06 | |
| *** luizbag has quit IRC | 16:08 | |
| *** e0ne has joined #openstack-manila | 16:12 | |
| *** e0ne has quit IRC | 16:20 | |
| *** e0ne has joined #openstack-manila | 16:22 | |
| *** e0ne has quit IRC | 16:23 | |
| *** kopecmartin is now known as kopecmartin|off | 17:04 | |
| ganso | bswartz, tbarron, gouthamr, toabctl: Hi folks! Status update! The patches at https://wiki.openstack.org/wiki/Manila/SteinCycle#Manage.2FUnmanage_Share_Servers are pretty much *ready* at this point. I addressed all the latest concerns raised by tbarron and gouthamr on friday, the latest changes have already passed CI, they are all green. There is only a few improvements that I will do next that *DO NOT* change the end | 17:31 |
| ganso | result, so please, feel free to review the patches as soon as possible, as all the results are there (if they are not green on the latest rebased run, they are green on the previous before rebase, I have already identified the source of flakiness and I will be addressing that now). | 17:31 |
| ganso | What I will do next is 1) fix a very small concurrency issue in the Container driver that is causing it to fail sometimes, 2) enable the container manage tests in the container patch instead of in the core patch, and 3) I am going to change the dependency order of some patches in a way that makes more sense. It will not be different from any previous job result | 17:33 |
| tbarron | ganso: thanks for the update, hope to look this afternoon ... | 17:34 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Refactor Container Driver https://review.openstack.org/637926 | 17:36 |
| *** e0ne has joined #openstack-manila | 17:59 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Add manage/unmanage of shares in DHSS=True https://review.openstack.org/635831 | 18:04 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila-tempest-plugin master: Add functional tests for manage/unmanage of shares in DHSS=True https://review.openstack.org/638231 | 18:06 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Add manage/unmanage implementation to Container Driver https://review.openstack.org/636626 | 18:10 |
| *** ociuhandu has joined #openstack-manila | 18:11 | |
| ganso | tbarron, gouthamr, bswartz, toabctl: done #1, #2 and #3 =) just waiting for CI, but just reiterating, job results should not be any different. You will see container share_server_manage test runs on the container patch now instead of in the manila-tempest-plugin patch. That's the only thing that has changed | 18:13 |
| gouthamr | ganso: sure thanks, I don't see a reason to rush... | 18:14 |
| ganso | gouthamr: sorry, but I do. As I said in the weekly meeting, the closer we get to the deadline, the riskier it gets of the CI breaking due to external changes | 18:15 |
| ganso | gouthamr: our internal NetApp CI also breaks pretty much every day, every time for a different reason | 18:16 |
| ganso | gouthamr: and if it breaks and I cannot fix it in time to get a successful run if there are any comments to address in the NetApp driver patch, then I'd assume it would end up being left out of Stein | 18:17 |
| *** e0ne has quit IRC | 18:17 | |
| gouthamr | ganso: sigh, i maybe don't understand the pressure at hand. I'd just like us not to rush through reviewing massive code, although i appreciate your assurances that we are fine with the code | 18:21 |
| ganso | gouthamr: it is not as much as rush as the importance of merging it as soon as possible | 18:22 |
| ganso | gouthamr: for the reasons I described above | 18:22 |
| *** whoami-rajat has quit IRC | 18:24 | |
| ganso | gouthamr: Could you please clarify https://review.openstack.org/#/c/637926/4/manila/share/drivers/container/container_helper.py@128 please? | 18:27 |
| ganso | gouthamr: do you want to log.debug replaced by a log message? | 18:27 |
| ganso | gouthamr: or just concat the reason in the log.debug based on e.message ? | 18:27 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Refactor Container Driver https://review.openstack.org/637926 | 18:34 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila master: Add manage/unmanage implementation to Container Driver https://review.openstack.org/636626 | 18:35 |
| *** pcaruana has quit IRC | 18:56 | |
| *** whoami-rajat has joined #openstack-manila | 18:58 | |
| *** e0ne has joined #openstack-manila | 19:13 | |
| *** openstackgerrit has quit IRC | 19:23 | |
| *** eharney has quit IRC | 19:33 | |
| *** thgcorrea has quit IRC | 20:00 | |
| *** jmlowe has quit IRC | 20:08 | |
| *** e0ne has quit IRC | 20:22 | |
| *** jmlowe has joined #openstack-manila | 20:46 | |
| *** whoami-rajat has quit IRC | 21:04 | |
| *** eharney has joined #openstack-manila | 21:28 | |
| *** carlos_silva has quit IRC | 22:07 | |
| *** tosky has quit IRC | 23:28 | |
| *** dviroel_ has quit IRC | 23:29 | |
| *** eharney has quit IRC | 23:51 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!