zrhoffman commented on code in PR #6802:
URL: https://github.com/apache/trafficcontrol/pull/6802#discussion_r888171875


##########
traffic_ops/app/db/create_tables.sql:
##########
@@ -37,6 +37,7 @@ CREATE EXTENSION IF NOT EXISTS plpgsql WITH SCHEMA pg_catalog;
 

Review Comment:
   Most of the roles look fixed, it looks like the only role still missing 
perms is the `portal` role:
   
   ```diff
   -5   ASN:READ        A_TIMESTAMP
   -5   ASYNC-STATUS:READ       A_TIMESTAMP
   -5   CACHE-GROUP:READ        A_TIMESTAMP
   -5   CAPABILITY:READ A_TIMESTAMP
   -5   CDN:READ        A_TIMESTAMP
   -5   CDN-SNAPSHOT:READ       A_TIMESTAMP
   -5   COORDINATE:READ A_TIMESTAMP
   -5   DELIVERY-SERVICE:READ   A_TIMESTAMP
    5   DELIVERY-SERVICE-SAFE:UPDATE    A_TIMESTAMP
   -5   DELIVERY-SERVICE:UPDATE A_TIMESTAMP
   -5   DIVISION:READ   A_TIMESTAMP
   -5   DS-REQUEST:CREATE       A_TIMESTAMP
   -5   DS-REQUEST:DELETE       A_TIMESTAMP
   -5   DS-REQUEST:READ A_TIMESTAMP
   -5   DS-REQUEST:UPDATE       A_TIMESTAMP
   -5   DS-SECURITY-KEY:READ    A_TIMESTAMP
   -5   FEDERATION:READ A_TIMESTAMP
   -5   FEDERATION-RESOLVER:READ        A_TIMESTAMP
   -5   ISO:READ        A_TIMESTAMP
   -5   JOB:CREATE      A_TIMESTAMP
   -5   JOB:DELETE      A_TIMESTAMP
   -5   JOB:READ        A_TIMESTAMP
   -5   JOB:UPDATE      A_TIMESTAMP
   -5   LOG:READ        A_TIMESTAMP
   -5   MONITOR-CONFIG:READ     A_TIMESTAMP
   -5   ORIGIN:READ     A_TIMESTAMP
   -5   PARAMETER:READ  A_TIMESTAMP
   -5   PHYSICAL-LOCATION:READ  A_TIMESTAMP
   -5   PLUGIN-READ     A_TIMESTAMP
   -5   PROFILE:READ    A_TIMESTAMP
   -5   REGION:READ     A_TIMESTAMP
   -5   ROLE:READ       A_TIMESTAMP
   -5   SERVER-CAPABILITY:READ  A_TIMESTAMP
   -5   SERVER-CHECK:READ       A_TIMESTAMP
   -5   SERVER:READ     A_TIMESTAMP
   -5   SERVICE-CATEGORY:READ   A_TIMESTAMP
   -5   STAT:CREATE     A_TIMESTAMP
   -5   STATIC-DN:READ  A_TIMESTAMP
   -5   STAT:READ       A_TIMESTAMP
   -5   STATUS:READ     A_TIMESTAMP
   -5   STEERING:CREATE A_TIMESTAMP
   -5   STEERING:DELETE A_TIMESTAMP
   -5   STEERING:READ   A_TIMESTAMP
   -5   STEERING:UPDATE A_TIMESTAMP
   -5   TENANT:READ     A_TIMESTAMP
   -5   TOPOLOGY:READ   A_TIMESTAMP
   -5   TRAFFIC-VAULT:READ      A_TIMESTAMP
   -5   TYPE:READ       A_TIMESTAMP
   -5   USER:READ       A_TIMESTAMP
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to