Skip to content
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

Reenable previously flaky Union_non_entity #25705

Merged
merged 1 commit into from
Aug 26, 2021
Merged

Conversation

roji
Copy link
Member

@roji roji commented Aug 25, 2021

Testing whether this will triggers weird flakiness

@roji
Copy link
Member Author

roji commented Aug 25, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@roji
Copy link
Member Author

roji commented Aug 25, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@smitpatel
Copy link
Contributor

This also happens locally. You can run tests in the loop in local machine.

@roji
Copy link
Member Author

roji commented Aug 25, 2021

That's also happening :)

@roji
Copy link
Member Author

roji commented Aug 25, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@roji
Copy link
Member Author

roji commented Aug 26, 2021

OK, I left EFCore.SqlServer.FunctionalTests running all night long in a loop without failures, and we have 4 successful CI runs here too. I propose we merge this - if we get this failure again we can always disable the test again and investigate.

@roji roji marked this pull request as ready for review August 26, 2021 09:12
@roji roji requested a review from a team August 26, 2021 09:12
@roji roji merged commit f6dbacc into release/6.0 Aug 26, 2021
@roji roji deleted the roji/ReenableFlakyTest branch August 26, 2021 09:19
@roji
Copy link
Member Author

roji commented Aug 26, 2021

🤞 🤞 🤞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants