This repository has been archived by the owner on Nov 1, 2023. It is now read-only.
Replies: 1 comment 2 replies
-
I think this is a great idea, would it be only enabled for check-pr? |
Beta Was this translation helpful? Give feedback.
2 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Minor problem:
Currently we store all errors/exception in AppInsights
Check-pr uses that to validate that there were no issues in the run (it is also very ugly check, since there is bunch of filtering to avoid test failure due to Azure specific errors). Also adds "time delay" to validate whether there were issues
Writing tests that interact with the system - will also require accessing app insights to validate that there were no failures
I see one obvious solution :
Add azure table and write errors and exceptions in there
And then query the table (maybe even add REST API to the service to get that info from the service side)
Beta Was this translation helpful? Give feedback.
All reactions