-
Notifications
You must be signed in to change notification settings - Fork 456
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disable flaky CI and E2E tests #5502
Disable flaky CI and E2E tests #5502
Conversation
@@ -29,7 +29,7 @@ public class RuntimeInfoProviderTest | |||
static readonly IEntityStore<string, ModuleState> RestartStateStore = new Mock<IEntityStore<string, ModuleState>>().Object; | |||
static readonly IRestartPolicyManager RestartManager = new Mock<IRestartPolicyManager>().Object; | |||
|
|||
[Integration] | |||
[Integration(Skip = "Flaky")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit : Can we provide reasoning in comments as to what component/method call is flaky?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am going to make PBIs with some contextual info. For most of these I don't even have much context other than sometimes it fails and it is a hard to pin down error. Right now I am prioritizing getting all the tests disabled so our pipelines are green. Then will circle back and organize the information.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's make sure we have work items to re-enable these before they are merged.
Discussed with Bear and he is helping to log work items after. |
These tests are flaky so disabling. Some of these are redundant given coverage in other areas. So we can honestly delete them when needed. (Thinking the
RemovePropertySuccess
andSendMessageBatchTest
tests)