Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Fail t3c int tests if migrations fail, and print migration output #7778

Merged
rimashah25 merged 1 commit into from
Sep 6, 2023
Merged

Fail t3c int tests if migrations fail, and print migration output #7778

rimashah25 merged 1 commit into from
Sep 6, 2023

Conversation

@zrhoffman
Copy link
Member

@zrhoffman zrhoffman commented Sep 6, 2023 •
edited by rimashah25
Loading

Which Traffic Control components are affected by this PR?

  • Cache-Config Tests

What is the best way to verify this PR?

If this is a bugfix, which Traffic Control versions contained the bug?

PR submission checklist

rimashah25
rimashah25 approved these changes Sep 6, 2023
View reviewed changes
Copy link
Contributor

@rimashah25 rimashah25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code changed LGTM

@rimashah25 rimashah25 merged commit e9b559b into apache:master Sep 6, 2023
@zrhoffman zrhoffman deleted the t3c-migrations-fail branch September 6, 2023 16:42
@zrhoffman zrhoffman added tests related to tests and/or testing infrastructure cache-config Cache config generation improvement The functionality exists but it could be improved in some way. labels Sep 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@rimashah25 rimashah25 rimashah25 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cache-config Cache config generation improvement The functionality exists but it could be improved in some way. tests related to tests and/or testing infrastructure

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@zrhoffman @rimashah25