Closed ericvaandering closed 10 years ago
Comment by huangch on Fri Mar 23 15:58:30 2012
"user_group" is supplied in transfer requests but not in deletion requests. For deletion, we have to find the group information through block_replicas ... I am working on it.
Closed by magini on Thu Dec 20 03:14:32 2012
Comment by magini on Thu Dec 20 03:14:32 2012
Hi,
I believe that this was released a few months ago, now deletion mails include the group data managers in cc, closing
Cheers N.
Original Savannah ticket 92911 reported by huangch on Fri Mar 23 05:23:49 2012.
For transfer requests, we correctly look up the set of managers for the group that is making the request. For deletion requests, we do not do that yet, because no group is attached to the request. So, no group-data-managers are notified that their data is going to be deleted(*).
We need, for deletion requests, to explicitly look up who owns each data-item that is being deleted, at each site, and add them to the notification. Given that the same data may be owned by different groups at different sites, this needs to be tested carefully.
The affected code is in PHEDEX::Core::Mail::_send_request_create_email
(*) Note that until this week, no group-data-managers were actually defined in SiteDB, but that's another matter :-)