| opendevreview | Yian Zong proposed openstack/manila master: Adds a new Manila driver for Dell PowerStore storage backend https://review.opendev.org/c/openstack/manila/+/885974 | 05:10 |
|---|---|---|
| opendevreview | Hervé Beraud proposed openstack/puppet-manila stable/yoga: Prepare new stable/yoga release https://review.opendev.org/c/openstack/puppet-manila/+/889948 | 11:59 |
| opendevreview | Caique Mello proposed openstack/manila master: [NetApp] SVM migration get progress https://review.opendev.org/c/openstack/manila/+/889737 | 12:31 |
| opendevreview | Helena proposed openstack/manila master: [NetApp] Fix BadRequest from share-server-migration-get-progress https://review.opendev.org/c/openstack/manila/+/889747 | 12:54 |
| Uggla | Hello, fyi currently playing with the manila client and the lock api | 13:51 |
| Uggla | carloss, gouthamr see ^ | 13:51 |
| carloss | o/ Uggla nice! | 13:52 |
| Uggla | So far it works fine. I cannot remove a shared, I previously locked. | 13:52 |
| carloss | ack - the scenario we didn't manage to stress much yet is: locking using the service role | 13:54 |
| carloss | but if you have some feedback on something, please let us know and we can act on it | 13:54 |
| Uggla | carloss, note I have configured the service token in between nova and manila. I hope it is done correctly. I'll check that when I'll be a bit more familiar with the new cli and sdk options | 13:58 |
| carloss | Uggla: ack, thanks! | 14:00 |
| Uggla | if I remember well if a request is passed with a service token, then the lock context will be service right ? | 14:07 |
| Uggla | carloss, if you have a moment, could you rebase the sdk https://review.opendev.org/c/openstack/openstacksdk/+/889519 | 14:39 |
| opendevreview | Merged openstack/puppet-manila stable/yoga: Prepare new stable/yoga release https://review.opendev.org/c/openstack/puppet-manila/+/889948 | 15:21 |
| Uggla | gouthamr, carloss I'm not completely sure because I have done so many tests today, but it seems "openstack share access list <share>" is not working from the client but ok from sdk. https://paste.opendev.org/show/b2mzUx7wlJ15OMamjTyG/ | 15:58 |
| carloss | > carloss, if you have a moment, could you rebase the sdk https://review.opendev.org/c/openstack/openstacksdk/+/889519 | 15:59 |
| carloss | Uggla sure! I'll do that | 15:59 |
| Uggla | however the openstack share access delete ... is working well. | 15:59 |
| carloss | > gouthamr, carloss I'm not completely sure because I have done so many tests today, but it seems "openstack share access list <share>" is not working from the client but ok from sdk. https://paste.opendev.org/show/b2mzUx7wlJ15OMamjTyG/ | 15:59 |
| carloss | ran into that issue as well... I believe it's an implementation thing | 15:59 |
| carloss | fixed it locally, will submit a new PS | 15:59 |
| gouthamr | for retrieving the access rule, you'll need a share id as well? | 16:00 |
| Uggla | carloss, cool if you encountered it as well | 16:00 |
| Uggla | gouthamr, yes | 16:00 |
| Uggla | at least the client requires it. | 16:00 |
| Uggla | this client issue gave me some troubles. Anyway I will continue on Monday with a fresh mind. I wish you a nice WE. | 16:03 |
| carloss | gouthamr: I believe it's an issue after the count implementation. Before, when doing locks.get_all, we'd be returning only the list of locks. Now we are retrieving two values: locks and the count | 16:04 |
| gouthamr | ah | 16:04 |
| carloss | the patch that is upstream still doesn't fix that | 16:04 |
| carloss | then it fails to retrieve the list of access rules for a share. I have run into that issue yesterday, while implementing functional tests | 16:05 |
| carloss | I was running more tests to ensure there aren't more issues before submitting a new PS | 16:05 |
| carloss | but I have fixed it already | 16:05 |
| carloss | just need to have the most recent PS on gerrit | 16:05 |
| gouthamr | ++ | 16:07 |
| gouthamr | i was trying to simplify the db method rather than introducing multiple methods | 16:08 |
| gouthamr | but if that's problematic, we can fix that too | 16:08 |
| carloss | nah, I think it's good gouthamr - I believe that behavior is the same across other resources retrieving the count | 16:11 |
| gouthamr | carloss ack | 16:12 |
| carloss | > if I remember well if a request is passed with a service token, then the lock context will be service right ? | 18:08 |
| carloss | yes :) - sorry, just saw this | 18:08 |
| opendevreview | Helena proposed openstack/manila master: Modify a query to resolve the slowness to list share servers. https://review.opendev.org/c/openstack/manila/+/884592 | 20:05 |
| opendevreview | Carlos Eduardo proposed openstack/manila master: Allow restricting access rules fields and deletion https://review.opendev.org/c/openstack/manila/+/887725 | 20:29 |
| carloss | Uggla: sorry for the late ping (I expect you to see it only on Monday), but I have rebased the openstacksdk change and proposed the fix to the access list issue in the manila code... if you get a chance, please take a look | 20:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!