Skip to content

Conversation

@wfurt
Copy link
Member

@wfurt wfurt commented Apr 14, 2022

contributes to #67353. While the underlying issue is in gss-ntlm (and was fixed upstream) it will take while to make it to all supported distributions.
This change effectively disables NTLM tests on Linux with OpenSSL 3.0.
This should allow us to move forward with #67345 and add test queue (like Ubuntu 22) with OpenSSL 3.

cc: @omajid

@wfurt wfurt added the test-enhancement Improvements of test source code label Apr 14, 2022
@wfurt wfurt requested a review from a team April 14, 2022 18:45
@wfurt wfurt self-assigned this Apr 14, 2022
@ghost ghost added the area-System.Net label Apr 14, 2022
@ghost
Copy link

ghost commented Apr 14, 2022

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

contributes to #67353. While the underlying issue is in gss-ntlm (and was fixed upstream) it will take while to make it to all supported distributions.
This change effectively disables NTLM tests on Linux with OpenSSL 3.0.
This should allow us to move forward with #67345 and add test queue (like Ubuntu 22) with OpenSSL 3.

cc: @omajid

Author: wfurt
Assignees: wfurt
Labels:

area-System.Net, test-enhancement

Milestone: -

@omajid
Copy link
Member

omajid commented Apr 14, 2022

Should we file an issue tracking to re-enable these tests when gss-ntlmssp is fixed?

@wfurt
Copy link
Member Author

wfurt commented Apr 14, 2022

I will flip #67353 to Disabled test once merged.
e.g. this change does not fix it

@wfurt wfurt merged commit b6c61ae into dotnet:main Apr 15, 2022
@wfurt
Copy link
Member Author

wfurt commented Apr 15, 2022

Let me know if you see more troubles on Ubuntu 22 @omajid

@wfurt wfurt deleted the gss branch April 15, 2022 02:37
@ghost ghost locked as resolved and limited conversation to collaborators May 15, 2022
@karelz karelz added this to the 7.0.0 milestone Jul 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Net test-enhancement Improvements of test source code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants