See here: https://phab-01.wmflabs.org/T127
There was no action by herald. Maybe one reason could be T117663.
Description
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Invalid | None | T117665 Bugs at security options at phabricator labs | |||
Invalid | None | T117664 [Phabricator Labs] Ops Access Request don't work due to Puppet overwriting config |
Event Timeline
That's because Herald rules are not synced from the Phabricator production instance to random Labs instances. So I don't consider this a bug per se.
https://phab-01.wmflabs.org/herald/rule/1/ exists. And I got an own instance with that too, and imported the actual security code from the wikimedia git, so the code is unique.Normaly, herald should do the same actions, and other features, like NDA or Securtiy works.
Well the security extension isn't even present nor configured on Phab-01 (which creates the Herald rules). Known issue.
I guess it's the event listener, which don't work? Is there a easy way to fix this? This feature is useful for some of my tests, so for me it would be useful, if this bug gets fixed fast.
I'm still not sure if we want the security extension on Phab-01 but if a Phabricator admin wants to do it its just:
- Making sure the security extension is checked out in /srv/phab/libext/security (it should be with a puppet run)
- Setting security extension to load: sudo bin/config set load-libraries '["/srv/phab/libext/Sprint/src","/srv/phab/libext/security/src"]'
- And configuring the event listeners: sudo bin/config set events.listeners '["SecurityPolicyEventListener"]'
Only point 3 was missing. But now, there is a new error:
>>> UNRECOVERABLE FATAL ERROR <<< Call to a member function getPHID() on a non-object /srv/phab/libext/security/src/policy/WMFSecurityPolicy.php:171 ┻━┻ ︵ ¯\_(ツ)_/¯ ︵ ┻━┻
The SRE-Access-Requests exists (my instance, not phab-01), but I get this bug.
This is the second problem: Puppet overwrites this config. Can you help me to protect this setting from against overwriting?
I guess we can close this, because the event listeners are gone now. Closing this as invalid, because we don't have a status 'obsolete'.