Code review comment for ~blake-rouse/maas:pools-modified-tracking

Revision history for this message
Alberto Donato (ack) wrote :

The mechanism implemented here seems quite generic.
Maybe we could add a "resource_type" field so that a single table would track all type of resources that need sync.

Manager methods would then filter on that type (and the pg notification would also carry the type).

This way we don't need a different table and notification handler for each resource we cover with rbac.

review: Needs Information

« Back to merge proposal