zrhoffman opened a new issue, #7329: URL: https://github.com/apache/trafficcontrol/issues/7329
<!-- ************ STOP!! ************ If this issue identifies a security vulnerability, DO NOT submit it! Instead, contact the Apache Traffic Control Security Team at [email protected] and follow the guidelines at https://apache.org/security regarding vulnerability disclosure. - For *SUPPORT QUESTIONS*, use the #traffic-control channel on the ASF slack (https://s.apache.org/tc-slack-request) or the Traffic Control Users mailing list (send an email to [email protected] to subscribe). - Before submitting, please **SEARCH GITHUB** for a similar issue or PR * https://github.com/apache/trafficcontrol/issues * https://github.com/apache/trafficcontrol/pulls --> <!-- Do not submit security vulnerabilities or support requests here - see above --> ## This Bug Report affects these Traffic Control components: <!-- delete all those that don't apply --> - Automation TP and TPv2 tests<!-- Please specify which (GitHub Actions, Docker images, Ansible Roles, etc.) --> ## What did you do? <!-- Describe how the bug happens --> Made a migration that created a new role ## What did you expect to happen? <!-- Describe what the behavior would be without the bug --> TP and TPv2 tests, which try to create a user with the `admin` role should pass ## What happened instead? <!-- If the current behavior is a bug, please provide the *STEPS TO REPRODUCE* and include the applicable TC version. --> The tests failed because both the TP and TPv2 tests contain SQL queries that assume that the `admin` role has id `1`: https://github.com/apache/trafficcontrol/blob/bbd101957e4445130fb03eafab286a998a7be614/.github/actions/tp-integration-tests/entrypoint.sh#L62-L63 https://github.com/apache/trafficcontrol/blob/bbd101957e4445130fb03eafab286a998a7be614/.github/actions/tp-integration-tests/entrypoint.sh#L62-L63 However, since migrations run before DB seeding, as of #7309, the `trouter` role had ID `1` on new ATC deployments (though this will no longer be the case once that migration is squashed in ATC 8). -- 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]
