| opendevreview | Suyeon Cha proposed openstack/openstacksdk master: Implemented force_delete for Share object. Passes unit and funcional tests. https://review.opendev.org/c/openstack/openstacksdk/+/890259 | 00:17 |
|---|---|---|
| opendevreview | Suyeon Cha proposed openstack/openstacksdk master: Added reset_status and force_delete methods to the shares resource under Shared File Systems. Functional & unit tests pass. https://review.opendev.org/c/openstack/openstacksdk/+/890260 | 00:17 |
| opendevreview | Grace Park proposed openstack/openstacksdk master: snapshot metadata https://review.opendev.org/c/openstack/openstacksdk/+/890264 | 03:04 |
| opendevreview | Anvi Joshi proposed openstack/openstacksdk master: Add support for managing and unmanaging shares in the shared file systems service https://review.opendev.org/c/openstack/openstacksdk/+/888940 | 04:01 |
| opendevreview | Anvi Joshi proposed openstack/openstacksdk master: Add support for managing and unmanaging shares in the shared file systems service https://review.opendev.org/c/openstack/openstacksdk/+/888940 | 04:05 |
| opendevreview | Merged openstack/openstacksdk master: Fix manila access rules functional tests https://review.opendev.org/c/openstack/openstacksdk/+/889997 | 07:38 |
| oschwart | Hey folks, is this channel still active? | 11:14 |
| frickler | oschwart: depends on your definition of active, if you have something to say about sdk or osc, there is no better one. but also be patient while waiting for an answer | 11:25 |
| opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: [WIP] Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | 11:41 |
| opendevreview | Omer Schwartz proposed openstack/openstacksdk master: Add support to Octavia (load_balancer) amphora get stats https://review.opendev.org/c/openstack/openstacksdk/+/889296 | 14:15 |
| opendevreview | Omer Schwartz proposed openstack/openstacksdk master: Add support to Octavia (load_balancer) amphora get stats https://review.opendev.org/c/openstack/openstacksdk/+/889296 | 14:17 |
| oschwart | frickler: sure, thanks for your reply. I asked a question a week ago and then went on PTO for a week, so I am not sure whether I got an answer to my question | 14:19 |
| oschwart | Anyway I am giving it another try, if I will need some help I will help you guys in this channel. Thanks again | 14:20 |
| oschwart | s/help/ask | 14:20 |
| frickler | oschwart: the channel is logged at https://meetings.opendev.org/irclogs/%23openstack-sdks/ , but it seems there was no answer to your question | 14:56 |
| oschwart | frickler: thanks | 15:00 |
| stephenfin | gtema: o/ Could you give me admin on https://review.opendev.org/admin/groups/33c03b5e000a75ffc476c51d484c5111648d399c,members so I can make the necessary changes discussed at the forum? | 15:11 |
| gtema | stephenfin - I can't give you admin - I myself have no priv to change admins, only gerrit admins (looking here also at frickler) can do so | 15:18 |
| tonyb | It looks like any member of that group is already an admin of the group | 15:20 |
| frickler | hmm, me fails to see where admins are defined, guess I need to turn my super powers on | 15:20 |
| frickler | oh, good point, is this a self-managed group like most other groups? | 15:21 |
| gtema | No clue, since I was also not able to see where the admins are managed | 15:22 |
| tonyb | https://review.opendev.org/admin/groups/33c03b5e000a75ffc476c51d484c5111648d399c looks like it? | 15:22 |
| gtema | tonyb - you are maybe right | 15:23 |
| gtema | stephenfin - I added you there as well | 15:23 |
| gtema | but that is not the right thing, it makes no sense if any member becomes admin | 15:23 |
| tonyb | I think for the most part that "admin" can only add/remove users from the group and that's safe given the trust releationships in the community | 15:25 |
| * tonyb has no ideas what chages were discussed at the forum | 15:25 | |
| frickler | gtema: we could make openstacksdk-core owner I guess? | 15:26 |
| gtema | yeah, this makes much more sense in my eyes. Thanks | 15:26 |
| frickler | fungi: ^^ can you confirm that the owner change would need to be done by a gerrit admin and that it makes sense in this case? (group of "auxillary sdk cores" that shouldn't be self owned) | 15:29 |
| stephenfin | tonyb: We agreed that we would add various service core groups (e.g. cinder-core) to openstacksdk-service-core. That group has +2 permissions for SDK (though intentionally not +W) | 15:30 |
| stephenfin | On a related note, I'm doing a cleanup of stale members of both openstacksdk-core and python-openstackclient-core. Will post to openstack-discuss about it shortly | 15:33 |
| tonyb | stephenfin: Ahh okay. | 15:34 |
| tonyb | gtema: I'd say yes you'd been another group and the corresponding ACL to gove them +2 but not +W | 15:34 |
| * tonyb adds caveat that he hasn't looked at the existing gerrit ACLs | 15:35 | |
| fungi | frickler: gtema: any member of a group can change the owner to themselves or to another group they're a member of | 15:42 |
| gtema | fungi - thanks, did that now | 15:43 |
| gtema | logical things are not so logical on the first sight | 15:44 |
| frickler | oh, I didn't know that, either, nice | 15:47 |
| stephenfin | gtema: rosmaita is proposing adding core groups directly to the ACL config file in project-config rather than as included groups in the openstacksdk-service-core group. Advantage would be that it's config driven rather than needing us to do in the UI, but it does remove the need for the openstacksdk-service-core group. Have you any preference? | 15:49 |
| rosmaita | gtema: got a question for you ... i am following up on the Forum consensus to add (for example) cinder-core to openstacksdk-service-core (and same for the cli) | 15:50 |
| rosmaita | there are 2 ways to do this: | 15:50 |
| rosmaita | 1 - in gerrit UI, a member of openstacksdk-service-core manually adds cinder-core | 15:50 |
| rosmaita | 2 - the project (cinder in this case) pushes a patch to project-config giving review powers -2..+2 to group cinder-core | 15:50 |
| rosmaita | which would you prefer? seems like 2 would be less work for you, though you will have to +1 the project-config patch | 15:50 |
| stephenfin | jinx | 15:50 |
| rosmaita | yeah, what stefenfin said | 15:50 |
| rosmaita | tell you what, i will post a patch to project-config and you can -1 it if you prefer the manual gerrit UI method | 15:52 |
| stephenfin | +1 | 15:52 |
| gtema | hmm, interesting. I am always for "avoiding" UI | 15:52 |
| gtema | so lets go with project-config stuff | 15:52 |
| rosmaita | https://review.opendev.org/c/openstack/project-config/+/890346 | 15:53 |
| gtema | yeah, I like that | 15:54 |
| rosmaita | ok, cool | 15:55 |
| rosmaita | i can send a message to openstack-discuss informing the other projects to do the same | 15:55 |
| stephenfin | rosmaita: thanks for following up on that and apologies for dropping the ball | 15:55 |
| stephenfin | yes, please | 15:55 |
| rosmaita | np, you have a lot on your plate | 15:55 |
| tonyb | rosmaita: nice | 15:58 |
| rosmaita | :) | 15:58 |
| rosmaita | Zuul does not like my patch ... i sorted the groups logically instead of alphabetically | 16:05 |
| rosmaita | stephenfin: gtema: will need you to renew your +1s on https://review.opendev.org/c/openstack/project-config/+/890346 | 16:12 |
| fungi | it's merged now, should deploy in short order | 17:29 |
| opendevreview | Suyeon Cha proposed openstack/openstacksdk master: reset_state, force_delete for Share, ShareSnapshot https://review.opendev.org/c/openstack/openstacksdk/+/888311 | 17:29 |
| opendevreview | Suyeon Cha proposed openstack/openstacksdk master: reset_state, force_delete for Share, ShareSnapshot https://review.opendev.org/c/openstack/openstacksdk/+/888311 | 17:31 |
| opendevreview | Suyeon Cha proposed openstack/openstacksdk master: reset_state, force_delete for Share, ShareSnapshot https://review.opendev.org/c/openstack/openstacksdk/+/888311 | 17:34 |
| opendevreview | Anvi Joshi proposed openstack/openstacksdk master: Add support for managing and unmanaging shares in the shared file systems service https://review.opendev.org/c/openstack/openstacksdk/+/888940 | 18:42 |
| opendevreview | Omer Schwartz proposed openstack/openstacksdk master: Add support to Octavia (load_balancer) amphora get stats https://review.opendev.org/c/openstack/openstacksdk/+/889296 | 18:42 |
| fungi | gtema: another acl addition https://review.opendev.org/890365 | 21:10 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!