zrhoffman edited a comment on pull request #6669: URL: https://github.com/apache/trafficcontrol/pull/6669#issuecomment-1071510527
> We could - or we could figure out what cases it's covering that aren't covered by existing automation and absorb those. > On that note, I have no idea what those are. I'm truly puzzled why it's failing (and inconsistently, too) in this CiaB extension and not in general. Whatever we do with this test suite, we'll need to figure that out. Part of it is that t3c and Traffic Monitor are running in CDN in a Box, so they are hitting endpoints while the API tests are running > I'll merge it if you want want > but at some point we gotta figure out why they fail for us locally and then either fix them or figure out what needs to be done to other tests to make sure these don't cover a gap somewhere else. agree > So you could add that to this PR if you want The way I see it, there are a few things we could add: - The aforementioned GHA for TO API Tests in CiaB but only runs when `infrastructure/cdn-in-a-box/traffic_ops_integration_test/` is modified but can also be run using `workflow_dispatch` (button) - Junit parsing that annotates each line of code that had a failure - Consider adding t3c or traffic monitor to the GHA? I don't know what else I could add for any of those that would be relevant to this PR, though. Additional stuff seems better-suited for a different PR -- 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]
