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

Move distributed transaction test to OleTxTests #74747

Merged
merged 1 commit into from
Aug 29, 2022

Conversation

roji
Copy link
Member

@roji roji commented Aug 29, 2022

Test-only PR to stabilize the CI builds. The skipped test is because of #74745, the fix for that is ready as well (but separated from this unrelated test-only stabilization).

NonMsdtcPromoterTests.PSPENonMsdtcGetPromoterTypeMSDTC was triggering an MSDTC distributed transaction on Windows, but without the proper checks/resiliency. Moved to OleTxTests.

Fixes #74170

NonMsdtcPromoterTests.PSPENonMsdtcGetPromoterTypeMSDTC was triggering
an MSDTC distributed transaction on Windows, but without the proper
checks/resiliency. Moved to OleTxTests.

Fixes dotnet#74170
@roji
Copy link
Member Author

roji commented Aug 29, 2022

/backport to release/7.0

@ghost ghost assigned roji Aug 29, 2022
@github-actions
Copy link
Contributor

Started backporting to release/7.0: https://github.com/dotnet/runtime/actions/runs/2948112352

@roji
Copy link
Member Author

roji commented Aug 29, 2022

Test failures are unrelated, merging.

@roji roji merged commit 012f474 into dotnet:main Aug 29, 2022
@roji roji deleted the DistributedTransactionTest branch August 29, 2022 19:04
@ghost ghost locked as resolved and limited conversation to collaborators Sep 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Distributed transactions: AccessViolationException on arm64
2 participants