Conversation
I'm opening this pull-request also to start the discussion. In |
The present code is missing the logic to create/instantiate the FAS client |
''' | ||
|
||
if not hasattr(_cache, 'backend'): | ||
cache.configure(**config['fmn.rules.cache']) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need a reference to the config
object here.
Well, what are the other ACLs? 'watch'? If a user has a watch ACL then they probably want fmn to notify them also. OTOH, this could be expanded to be more configurable. |
…t and forward the configuration
Fair enough yeah :) |
Let's test it! Is there a package out there that is owned by infra-sig but not by one of us? |
Bochecha is part of the infra-sig group but has no ACLs on any of the package the group has I think |
Ah, this doesn't actually work. The code you added gets called by |
I was fearing this. I'll try to think of the opposite approach |
...just like we do for pkgdb ownership info. This relies on the code introduced in fedora-infra/fmn.rules#16.
No description provided.