*** ociuhandu has joined #openstack-manila | 00:04 | |
*** yangyapeng has quit IRC | 00:06 | |
*** baojg has joined #openstack-manila | 00:16 | |
*** cknight has joined #openstack-manila | 00:22 | |
*** kaisers1 has joined #openstack-manila | 00:35 | |
*** bswartz has joined #openstack-manila | 00:36 | |
*** kaisers has quit IRC | 00:38 | |
*** baojg has quit IRC | 00:42 | |
*** chlong_ has joined #openstack-manila | 01:12 | |
*** chlong has joined #openstack-manila | 01:12 | |
*** yangyapeng has joined #openstack-manila | 01:20 | |
*** zengyingzhe_ has quit IRC | 01:33 | |
*** wangqun has joined #openstack-manila | 01:41 | |
*** cknight has quit IRC | 02:03 | |
*** wangqun_ has joined #openstack-manila | 02:09 | |
*** akshai_ has quit IRC | 02:09 | |
*** wangqun has quit IRC | 02:09 | |
*** cknight has joined #openstack-manila | 02:46 | |
*** StraubTW has quit IRC | 02:49 | |
openstackgerrit | zhongjun proposed openstack/python-manilaclient: Add share_type filter to pool_list https://review.openstack.org/368528 | 03:05 |
---|---|---|
*** ociuhandu has quit IRC | 03:25 | |
*** baojg has joined #openstack-manila | 03:43 | |
*** cknight has quit IRC | 03:45 | |
*** baojg has quit IRC | 03:47 | |
openstackgerrit | Yingzhe Zeng proposed openstack/manila: Fix huawei driver username/password encoding bug https://review.openstack.org/368538 | 03:48 |
openstackgerrit | Yingzhe Zeng proposed openstack/manila: Fix huawei driver username/password encoding bug https://review.openstack.org/367103 | 03:52 |
openstackgerrit | Nguyen Phuong An proposed openstack/manila: [api-ref] Remove temporary block in conf.py https://review.openstack.org/367726 | 03:58 |
openstackgerrit | Vijay Ladani proposed openstack/manila: HPE 3PAR: file share support of AD in devref https://review.openstack.org/366045 | 04:08 |
*** gouthamr has joined #openstack-manila | 04:19 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/manila: Adjust doc about threading https://review.openstack.org/367773 | 04:25 |
*** martyt has joined #openstack-manila | 04:30 | |
*** jcsp has joined #openstack-manila | 04:31 | |
*** kaisers has joined #openstack-manila | 04:39 | |
*** martyt has quit IRC | 04:57 | |
*** lpetrut has joined #openstack-manila | 05:02 | |
*** chlong_ has quit IRC | 05:03 | |
*** chlong has quit IRC | 05:03 | |
*** chlong_ has joined #openstack-manila | 05:16 | |
*** chlong has joined #openstack-manila | 05:21 | |
*** senk has joined #openstack-manila | 05:21 | |
*** chlong has quit IRC | 05:25 | |
*** wangqun_ has quit IRC | 05:27 | |
*** senk has quit IRC | 05:45 | |
*** senk has joined #openstack-manila | 05:46 | |
*** senk has quit IRC | 05:47 | |
*** lpetrut has quit IRC | 05:50 | |
openstackgerrit | Ha Van Tu proposed openstack/manila: [api-ref] Refactor consistency group API https://review.openstack.org/367711 | 05:57 |
*** tries_ has joined #openstack-manila | 05:57 | |
*** houming has joined #openstack-manila | 05:57 | |
*** lpetrut has joined #openstack-manila | 06:03 | |
*** chlong_ has quit IRC | 06:05 | |
openstackgerrit | Ha Van Tu proposed openstack/manila: [api-ref] Refactor manila extension API https://review.openstack.org/368561 | 06:15 |
*** nherciu has joined #openstack-manila | 06:19 | |
openstackgerrit | zhongjun proposed openstack/python-manilaclient: Add the ability to check tenant quota usages https://review.openstack.org/368567 | 06:25 |
openstackgerrit | Yingzhe Zeng proposed openstack/manila: huawei driver default create thin type share https://review.openstack.org/351475 | 06:25 |
*** lpetrut has quit IRC | 06:27 | |
*** zengyingzhe has joined #openstack-manila | 06:27 | |
*** lpetrut has joined #openstack-manila | 06:31 | |
*** nkrinner_afk is now known as nkrinner | 06:45 | |
*** senk has joined #openstack-manila | 06:46 | |
*** pcaruana has joined #openstack-manila | 06:49 | |
openstackgerrit | Ha Van Tu proposed openstack/manila: [api-ref] Refactor limits and services API https://review.openstack.org/368581 | 06:53 |
*** gouthamr has quit IRC | 07:01 | |
*** gouthamr has joined #openstack-manila | 07:02 | |
*** gouthamr has quit IRC | 07:03 | |
*** lpetrut has quit IRC | 07:07 | |
openstackgerrit | Yingzhe Zeng proposed openstack/manila: Fix dedup/compression description in doc https://review.openstack.org/368598 | 07:33 |
*** akapil has joined #openstack-manila | 07:38 | |
*** akapil has quit IRC | 07:38 | |
*** akapil has joined #openstack-manila | 07:39 | |
*** akapil has quit IRC | 07:43 | |
*** akapil has joined #openstack-manila | 07:43 | |
openstackgerrit | zhongjun proposed openstack/python-manilaclient: Add share_type filter to pool_list https://review.openstack.org/368528 | 07:51 |
*** martyt has joined #openstack-manila | 07:58 | |
*** akapil_ has joined #openstack-manila | 07:59 | |
*** akapil_ has joined #openstack-manila | 07:59 | |
*** akapil has quit IRC | 08:02 | |
*** martyt has quit IRC | 08:02 | |
*** jcsp has quit IRC | 08:05 | |
*** aovchinnikov has joined #openstack-manila | 08:08 | |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: [DNM] More container driver tests https://review.openstack.org/366740 | 08:08 |
openstackgerrit | Ha Van Tu proposed openstack/manila: [api-ref] Refactor limits and services API https://review.openstack.org/368581 | 08:11 |
*** rraja has joined #openstack-manila | 08:16 | |
openstackgerrit | Ha Van Tu proposed openstack/manila: [api-ref] Refactor limits and services API https://review.openstack.org/368581 | 08:37 |
*** lpetrut has joined #openstack-manila | 08:42 | |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: [DNM] More container driver tests https://review.openstack.org/366740 | 08:48 |
*** akapil_ has quit IRC | 09:00 | |
openstackgerrit | Ha Van Tu proposed openstack/manila: [api-ref] Refactor manila extension API https://review.openstack.org/368561 | 09:04 |
*** openstackgerrit has quit IRC | 09:04 | |
*** openstackgerrit has joined #openstack-manila | 09:05 | |
*** hoonetorg has joined #openstack-manila | 09:05 | |
*** zhugaoxiao has quit IRC | 09:16 | |
*** zhugaoxiao has joined #openstack-manila | 09:17 | |
*** akapil has joined #openstack-manila | 09:18 | |
openstackgerrit | Tatiana Ovchinnikova proposed openstack/manila-ui: Use parse_isotime filters for Share Replicas table columns https://review.openstack.org/368664 | 09:20 |
openstackgerrit | Ha Van Tu proposed openstack/manila: [api-ref] Refactor limits and services API https://review.openstack.org/368581 | 09:21 |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: [DNM] More container driver tests https://review.openstack.org/366740 | 09:28 |
*** jcsp has joined #openstack-manila | 09:31 | |
*** houming has quit IRC | 09:31 | |
*** jcsp has quit IRC | 09:39 | |
*** akapil has quit IRC | 09:52 | |
*** akapil has joined #openstack-manila | 10:13 | |
*** akapil_ has joined #openstack-manila | 10:15 | |
*** akapil has quit IRC | 10:19 | |
*** kaisers has quit IRC | 10:20 | |
*** kaisers has joined #openstack-manila | 10:33 | |
openstackgerrit | Tatiana Ovchinnikova proposed openstack/manila-ui: Fix Manage Replicas button availability https://review.openstack.org/368726 | 10:35 |
*** yangyapeng has quit IRC | 10:40 | |
*** ganso has joined #openstack-manila | 10:55 | |
*** hoonetorg has quit IRC | 11:03 | |
*** kaisers has quit IRC | 11:06 | |
zhongjun_ | tbarron: hi :) | 11:08 |
openstackgerrit | Valeriy Ponomaryov proposed openstack/python-manilaclient: Add support of Availability Zones API https://review.openstack.org/368746 | 11:09 |
zhongjun_ | tbarron: Is it ok for IPv6 feature code that not change the our roll-your-own validation IPv4 for IPv4. Please see this link again,thanks: https://review.openstack.org/#/c/312321/26/manila/api/v1/shares.py | 11:12 |
openstackgerrit | Valeriy Ponomaryov proposed openstack/python-manilaclient: Add support of Availability Zones API https://review.openstack.org/368746 | 11:14 |
openstackgerrit | yuyafei proposed openstack/python-manilaclient: Add __ne__ built-in function https://review.openstack.org/355157 | 11:22 |
tbarron | zhongjun_: that's OK with me as long as we do the IPv4 routine as well, can be separate patch. I'll say so in the review. | 11:32 |
openstackgerrit | Bin Zhou proposed openstack/manila: Modify use of assertTrue(A in B) https://review.openstack.org/368767 | 11:34 |
*** sandanar has joined #openstack-manila | 11:42 | |
*** rooneym has joined #openstack-manila | 11:55 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila-ui: Read list of AZs using manila's API instead of nova's https://review.openstack.org/367992 | 11:55 |
*** eharney has quit IRC | 11:57 | |
*** sandanar has quit IRC | 12:01 | |
*** cknight has joined #openstack-manila | 12:05 | |
zhongjun_ | tbarron: Thanks for review again | 12:06 |
tbarron | zhongjun_: no problem, I guess we have to wait for O to merge these though | 12:08 |
openstackgerrit | Alyson proposed openstack/manila: Add cleanup to create from snap in Manila HNAS driver https://review.openstack.org/356087 | 12:09 |
*** alyson_ has joined #openstack-manila | 12:09 | |
zhongjun_ | tbarron: yes, we could merge the first patch early in Ocata. :) | 12:16 |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: [DNM] More container driver tests https://review.openstack.org/366740 | 12:17 |
*** akapil_ has quit IRC | 12:24 | |
openstackgerrit | zhongjun proposed openstack/manila: Add IPv6 ACL support in Manila https://review.openstack.org/312321 | 12:24 |
*** akapil has joined #openstack-manila | 12:25 | |
openstackgerrit | zhongjun proposed openstack/manila: Add IPv6 ACL support in Manila https://review.openstack.org/312321 | 12:26 |
*** porrua has joined #openstack-manila | 12:26 | |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: [DNM] More container driver tests https://review.openstack.org/366740 | 12:28 |
*** porrua_ has joined #openstack-manila | 12:28 | |
*** jcsp has joined #openstack-manila | 12:29 | |
*** porrua has quit IRC | 12:31 | |
openstackgerrit | Alyson proposed openstack/manila: Fix access rules for managed shares in HSP driver https://review.openstack.org/366310 | 13:02 |
ganso | vponomaryov, cknight, markstur, bswartz: Could you please review when you have some spare time? https://review.openstack.org/365136 thanks in advance | 13:04 |
ganso | tbarron: ^ | 13:05 |
vponomaryov | ganso: looking | 13:07 |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: [DNM] More container driver tests https://review.openstack.org/366740 | 13:10 |
*** eharney has joined #openstack-manila | 13:15 | |
*** akerr has joined #openstack-manila | 13:16 | |
*** openstackgerrit has quit IRC | 13:19 | |
*** openstackgerrit has joined #openstack-manila | 13:20 | |
vponomaryov | ganso: ping | 13:20 |
ganso | vponomaryov: pong | 13:20 |
vponomaryov | ganso: Hello Rodrigo | 13:21 |
ganso | vponomaryov: Hi Valeriy | 13:21 |
vponomaryov | ganso: what exactly did you want to say here: https://review.openstack.org/#/c/366856/2/doc/source/devref/share_back_ends_feature_support_mapping.rst ? | 13:21 |
ganso | vponomaryov: we could map all drivers that are known to support host-assisted migration | 13:22 |
vponomaryov | ganso: isn't host-assisted approach is driver-agnostic? | 13:22 |
*** dustins has joined #openstack-manila | 13:22 | |
ganso | vponomaryov: the approach is, but it does not guarantee that all drivers support it. See my comment in the reno of your other patch | 13:22 |
vponomaryov | which one? | 13:23 |
ganso | vponomaryov: https://review.openstack.org/#/c/366833 | 13:23 |
*** jcsp has quit IRC | 13:24 | |
*** jcsp has joined #openstack-manila | 13:24 | |
vponomaryov | ganso: actually, in perfect world, we would report such capability | 13:24 |
vponomaryov | ganso: until we do not do this, we should not distinguish it | 13:25 |
*** kaisers has joined #openstack-manila | 13:25 | |
vponomaryov | anyhow in text | 13:25 |
ganso | vponomaryov: oh, right... I see what you mean now, you're mapping what is in code | 13:25 |
vponomaryov | ganso: so, right now it is called "we can try, but no guarantee" | 13:26 |
vponomaryov | ganso: so, it is driver-agnostic | 13:26 |
vponomaryov | ganso: as of now | 13:26 |
*** akshai has joined #openstack-manila | 13:27 | |
ganso | vponomaryov: updated the comment | 13:28 |
*** cknight has quit IRC | 13:30 | |
*** cknight has joined #openstack-manila | 13:31 | |
*** porrua_ has quit IRC | 13:38 | |
*** yangyapeng has joined #openstack-manila | 13:39 | |
*** jcsp has quit IRC | 13:42 | |
*** ociuhandu has joined #openstack-manila | 13:46 | |
openstackgerrit | Clinton Knight proposed openstack/manila: NetApp cDOT driver autosupport broken https://review.openstack.org/368856 | 13:48 |
*** jcsp has joined #openstack-manila | 14:01 | |
vponomaryov | ganso: in https://review.openstack.org/#/c/366856/2/doc/source/devref/share_back_ends_feature_support_mapping.rst I use values that are available via API | 14:03 |
vponomaryov | ganso: so, people looking at this table understand what values of migration-start args could be used with it | 14:04 |
vponomaryov | ganso: and things you propose me to do are confusing to me, and, I suspect, with be confusing for lots of others too | 14:04 |
vponomaryov | ы.цшер.цшдд. | 14:05 |
vponomaryov | s/with/will/ | 14:05 |
vponomaryov | ganso: same here: https://review.openstack.org/#/c/366833/1/releasenotes/notes/add-share-migration-support-in-zfsonlinux-driver-88e6da5692b50810.yaml | 14:09 |
vponomaryov | ganso: I see that you are insisting on saying "support of 'host-assisted' migration in share driver", and exactly this is confusing to me, it is up-to-host only. So, it should be attribute of data-service, not share driver | 14:11 |
vponomaryov | ganso: that is why saying "share migration in FOO driver" I mean that driver has implemented interfaces of share migration | 14:12 |
vponomaryov | ganso: no more, no less. Host-assisted migration depends on data-service and driver-assisted on share driver. | 14:13 |
vponomaryov | ganso: so, I guess, you need someone else from core team to weigh in | 14:14 |
*** porrua has joined #openstack-manila | 14:14 | |
*** nherciu has quit IRC | 14:23 | |
*** jcsp has quit IRC | 14:23 | |
*** kaisers has quit IRC | 14:25 | |
*** baojg has joined #openstack-manila | 14:27 | |
*** aovchinnikov has quit IRC | 14:30 | |
*** jcsp has joined #openstack-manila | 14:30 | |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: [DNM] More container driver tests https://review.openstack.org/366740 | 14:30 |
*** akshai has quit IRC | 14:37 | |
*** senk has quit IRC | 14:38 | |
*** akshai has joined #openstack-manila | 14:38 | |
*** hoonetorg has joined #openstack-manila | 14:38 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila-ui: Fix display of Delete share replica button https://review.openstack.org/368880 | 14:39 |
*** gouthamr has joined #openstack-manila | 15:09 | |
*** nkrinner is now known as nkrinner_afk | 15:20 | |
*** baojg has quit IRC | 15:20 | |
openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: Manila install guide: Fix wrong instructions https://review.openstack.org/359491 | 15:37 |
*** kaisers has joined #openstack-manila | 15:40 | |
openstackgerrit | Goutham Pacha Ravi proposed openstack/python-manilaclient: Add reno for availability-zone-list https://review.openstack.org/368935 | 15:43 |
openstackgerrit | Merged openstack/python-manilaclient: Add support of Availability Zones API https://review.openstack.org/368746 | 15:47 |
*** nherciu has joined #openstack-manila | 15:48 | |
*** pcaruana has quit IRC | 15:56 | |
vponomaryov | bswartz: ping | 16:00 |
vponomaryov | bswartz: could you please make a 1.12 tag for manilaclient -> https://github.com/openstack/python-manilaclient/releases ? | 16:01 |
vponomaryov | bswartz: it is required for manila_ui | 16:01 |
*** lpetrut has quit IRC | 16:04 | |
*** rraja has quit IRC | 16:05 | |
*** porrua has quit IRC | 16:30 | |
*** kaisers has quit IRC | 16:33 | |
*** senk has joined #openstack-manila | 16:39 | |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Fix Share Migration not allowing access https://review.openstack.org/365136 | 16:45 |
*** zhongjun_ has quit IRC | 16:52 | |
*** zhongjun_ has joined #openstack-manila | 16:53 | |
*** porrua has joined #openstack-manila | 16:53 | |
*** akshai has quit IRC | 16:57 | |
*** akapil_ has joined #openstack-manila | 16:57 | |
*** akshai has joined #openstack-manila | 16:58 | |
*** akapil has quit IRC | 17:01 | |
*** Suyi_ has joined #openstack-manila | 17:02 | |
*** akapil_ has quit IRC | 17:02 | |
*** akshai has quit IRC | 17:14 | |
*** harlowja has joined #openstack-manila | 17:19 | |
openstackgerrit | Merged openstack/python-manilaclient: Add reno for availability-zone-list https://review.openstack.org/368935 | 17:20 |
*** harlowja has quit IRC | 17:43 | |
*** harlowja has joined #openstack-manila | 17:45 | |
*** senk has quit IRC | 17:46 | |
*** senk has joined #openstack-manila | 17:47 | |
*** senk has quit IRC | 17:48 | |
*** akerr has quit IRC | 17:51 | |
*** eharney_ has joined #openstack-manila | 17:56 | |
*** eharney has quit IRC | 17:57 | |
*** eharney_ is now known as eharney | 17:57 | |
*** kaisers has joined #openstack-manila | 18:00 | |
*** kaisers has quit IRC | 18:01 | |
*** kaisers_ has joined #openstack-manila | 18:01 | |
*** akshai has joined #openstack-manila | 18:04 | |
*** rraja has joined #openstack-manila | 18:11 | |
openstackgerrit | Merged openstack/manila: NetApp cDOT driver autosupport broken https://review.openstack.org/368856 | 18:32 |
bswartz | ganso: I don't think I agree with this bug at all | 18:42 |
ganso | bswartz: I just posted a comment | 18:43 |
bswartz | We never discussed mounting the destination of a share before the migration was completed | 18:43 |
ganso | bswartz: wait, what are you talking about? | 18:43 |
bswartz | https://bugs.launchpad.net/manila/+bug/1619788 | 18:44 |
openstack | Launchpad bug 1619788 in Manila "Share migration does not allow_access before migration_complete" [High,In progress] - Assigned to Rodrigo Barbieri (rodrigo-barbieri2010) | 18:44 |
ganso | bswartz: what bug are you referring to | 18:44 |
ganso | bswartz: oh that one | 18:44 |
ganso | bswartz: yes we did, in the midcycle | 18:44 |
bswartz | the problem is that for "writable" migrations, the data will continue to change until you call the migration complete | 18:45 |
bswartz | so there is no point to mounting the destination | 18:45 |
ganso | bswartz: yes, but for the purposes of validating the metadata preservation, he does not need to validate all the files | 18:45 |
*** lpetrut has joined #openstack-manila | 18:49 | |
bswartz | who asked for this? | 18:53 |
*** lpetrut has quit IRC | 18:53 | |
bswartz | the bug reporters for both bugs is you | 18:54 |
bswartz | https://bugs.launchpad.net/manila/+bug/1594922 | 18:54 |
openstack | Launchpad bug 1594922 in Manila "Share migration tempest test fail on driver optimized migration" [High,Fix released] - Assigned to Rodrigo Barbieri (rodrigo-barbieri2010) | 18:54 |
bswartz | ^ original bug | 18:54 |
ganso | bswartz: yes | 18:56 |
ganso | bswartz: so, I implemented this before, in the patch that merged before FF | 18:56 |
bswartz | I guess this just isn't a use case that I think is important | 18:56 |
ganso | bswartz: partially implemented, in fact | 18:56 |
ganso | bswartz: but it had a bug | 18:56 |
bswartz | If you don't trust that the cloud is migrating your data correctly, then you have bigger problems | 18:57 |
bswartz | yeah I understand that the change already merged and this change fixes a bug in that change | 18:57 |
ganso | bswartz: https://github.com/openstack/manila/blob/master/manila/share/api.py#L1351 | 18:57 |
ganso | bswartz: that condition does not work | 18:57 |
bswartz | still I debate the value of the original change | 18:57 |
*** lpetrut has joined #openstack-manila | 18:58 | |
ganso | bswartz: what is the disadvantage of having it? | 18:58 |
bswartz | it seems to address a fairly narrow use case and possibly introduces significant problems for drivers that can't do it | 18:58 |
bswartz | what about drivers for which the destination is not accessible until after the migration completes? | 18:58 |
ganso | bswartz: according to the spec, this is only "whenever possible" | 18:58 |
ganso | bswartz: gouthamr asked the same question | 18:59 |
bswartz | any time we do a "whenever possible" kind of feature though it creates lots of confusion | 18:59 |
ganso | bswartz: my response is that, if some drivers cannot do it, the export location will just not show up | 18:59 |
bswartz | because users won't have a way to know whether the feature is usable on any given cloud | 18:59 |
ganso | bswartz: humm, I implemented how we had previously discussed in the midcycle... if we do not want it now, then I should remove this capability completely and fix that condition in allow_access | 19:00 |
*** senk has joined #openstack-manila | 19:00 | |
ganso | bswartz: ok so, is it best to remove? | 19:01 |
bswartz | for newton it's best to just fix the bug and leave it as is | 19:02 |
ganso | bswartz: ok | 19:02 |
bswartz | We could revisit the capability in ocata and change it though | 19:02 |
bswartz | but I was asking who requested it though because that's who should make the case for not removing it | 19:02 |
gouthamr | it does make sense, in case of generic migration | 19:02 |
ganso | bswartz: I am remember exactly, but I am 100% sure it was asked during midcycle | 19:03 |
ganso | gouthamr: right, we possibly that host-assisted migration in mind when we discussed at the midcycle | 19:03 |
ganso | gouthamr: possibly *had | 19:04 |
*** pcaruana has joined #openstack-manila | 19:04 | |
*** lpetrut has quit IRC | 19:05 | |
gouthamr | ganso: yes.. if optimized, i presume, for most drivers, it may not be easy to get export locations on the destination | 19:05 |
*** xyang1 has joined #openstack-manila | 19:05 | |
gouthamr | ganso: specially if the source has to remain writable, as bswartz pointed out above | 19:05 |
ganso | bswartz, gouthamr: ok, let's revisit this in Ocata | 19:06 |
ganso | bswartz, gouthamr: we will probably be able to resolve this question better when we have more drivers supporting driver-assisted migration | 19:06 |
gouthamr | ganso: +1 | 19:06 |
*** senk has quit IRC | 19:07 | |
ganso | gouthamr: in NetApp case, is it possible to provide the export location? | 19:07 |
gouthamr | ganso: nope.. | 19:07 |
ganso | gouthamr: hummm, ok. THanks! | 19:07 |
bswartz | ganso: in ZFS case I'm not sure we'll always be able to either | 19:08 |
bswartz | after we implement writable migration that is | 19:08 |
gouthamr | bswartz: with the current implementation, you might be able to ... stop replication (i.e, break relationship) after phase 1 is complete, and push a snap delta at the beginning of phase 2.. | 19:11 |
*** nherciu has quit IRC | 19:11 | |
gouthamr | not sure if its worth the effort though.. | 19:11 |
bswartz | gouthamr: the more important question is why would we bother? | 19:22 |
gouthamr | bswartz: as ganso mentioned earlier, maybe it was requested for the fallback case -> but then with the checksum implementation he did in newton, not sure its necessary.. | 19:24 |
bswartz | people need to trust manila's migration implementation | 19:24 |
ganso | gouthamr: checksum does not cover metadata | 19:24 |
bswartz | if they don't trust it then they're better off rsyncing their own data | 19:24 |
gouthamr | true... | 19:25 |
gouthamr | ganso: when are you planning on adding docs? :P would be nice to cover these aspects in the documentation.. | 19:25 |
ganso | gouthamr: once spec is merged | 19:27 |
ganso | gouthamr: now it needs to be updated based on this discussion | 19:27 |
gouthamr | ganso: hmmm, sure. will review and post any final comments today | 19:27 |
*** dsariel has joined #openstack-manila | 19:29 | |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Fix allow_access and deny_access error message https://review.openstack.org/365136 | 19:36 |
*** StraubTW has joined #openstack-manila | 19:38 | |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Fix allow_access and deny_access error message https://review.openstack.org/365136 | 19:40 |
openstackgerrit | Alyson proposed openstack/manila: Fix access rules for managed shares in HSP driver https://review.openstack.org/366310 | 19:45 |
openstackgerrit | Alyson proposed openstack/manila: Fix access rules for managed shares in HSP driver https://review.openstack.org/366310 | 19:53 |
*** lpetrut has joined #openstack-manila | 19:59 | |
ganso | bswartz: could you please weigh in at this discussion? https://review.openstack.org/#/c/366833/ | 20:02 |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Improve Share Migration tempest tests https://review.openstack.org/366324 | 20:04 |
bswartz | ganso: wow this is a pretty minor debate | 20:08 |
bswartz | we're debating the wording of a release note? | 20:08 |
ganso | bswartz: yes | 20:08 |
bswartz | I can see the merit of both arguments | 20:08 |
bswartz | I guess I lean towards your statement | 20:08 |
bswartz | I'll vote -1 | 20:09 |
ganso | bswartz: there may be another patch that introduces some changes to make ZFS work on host-assisted migration. So that patch would have a similar release note, and in that case, this one would be confusing | 20:09 |
*** porrua has quit IRC | 20:11 | |
ganso | bswartz: There's also this debate: https://review.openstack.org/#/c/366856/ | 20:11 |
ganso | bswartz: possibly more relevant than a release note | 20:11 |
* bswartz sighs | 20:15 | |
bswartz | I keep wanting that page to disappear and it keeps expanding | 20:15 |
bswartz | I guess I need to spend some time on migrating (hah) the content of that page to openstack-manuals | 20:16 |
openstackgerrit | Merged openstack/manila-ui: Read list of AZs using manila's API instead of nova's https://review.openstack.org/367992 | 20:19 |
ganso | bswartz: lol | 20:19 |
ganso | bswartz: I like that page :( | 20:19 |
bswartz | there are multiple reasons to get rid of it | 20:20 |
bswartz | the simplest is that it's the center of some horrible merge conflicts | 20:20 |
bswartz | however the more important reason is that the source code isn't the right place for that kind of information | 20:21 |
bswartz | it's ultimately admin-facing information and belongs in the docs, not the devref | 20:21 |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila-specs: Add spec for Share Migration Newton improvements https://review.openstack.org/315707 | 20:29 |
*** lpetrut has quit IRC | 20:31 | |
*** eharney has quit IRC | 20:35 | |
*** dsariel has quit IRC | 20:37 | |
*** cknight has quit IRC | 20:37 | |
*** akapil has joined #openstack-manila | 20:41 | |
*** akapil has quit IRC | 20:45 | |
*** pcaruana has quit IRC | 20:46 | |
*** rraja has quit IRC | 20:46 | |
*** lpetrut has joined #openstack-manila | 20:49 | |
*** akshai has quit IRC | 20:56 | |
*** MikeG451 has quit IRC | 20:59 | |
*** MikeG451 has joined #openstack-manila | 21:00 | |
*** kaisers_ has quit IRC | 21:12 | |
*** kaisers has joined #openstack-manila | 21:14 | |
*** kaisers has quit IRC | 21:18 | |
*** dustins has quit IRC | 21:51 | |
*** alyson_ has quit IRC | 21:59 | |
*** zigo has quit IRC | 22:08 | |
*** martyt has joined #openstack-manila | 22:10 | |
*** zigo has joined #openstack-manila | 22:11 | |
*** zigo is now known as Guest41388 | 22:11 | |
*** Guest41388 is now known as zigo_ | 22:17 | |
*** ociuhandu has quit IRC | 22:20 | |
*** StraubTW has quit IRC | 22:21 | |
ganso | bswartz: ping | 22:35 |
*** martyt has quit IRC | 22:45 | |
*** openstackgerrit has quit IRC | 22:48 | |
*** openstackgerrit has joined #openstack-manila | 22:49 | |
*** kaisers has joined #openstack-manila | 23:03 | |
*** lpetrut has quit IRC | 23:03 | |
*** kaisers has quit IRC | 23:07 | |
openstackgerrit | Jay Mehta proposed openstack/manila: Correctly calling setup_server() calls in unit-tests https://review.openstack.org/360088 | 23:10 |
*** martyt has joined #openstack-manila | 23:11 | |
*** martyt has quit IRC | 23:15 | |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila-specs: Add spec for Share Migration Newton improvements https://review.openstack.org/315707 | 23:17 |
*** xyang1 has quit IRC | 23:20 | |
*** akshai has joined #openstack-manila | 23:50 | |
*** akshai_ has joined #openstack-manila | 23:53 | |
*** akshai has quit IRC | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!