Skip to content

CI Update - Remove System.Private.ServiceModel package #10164

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

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

gewarren
Copy link
Contributor

Build.Reason:Manual by Genevieve Warren
Build.Url:https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=441514&view=results
source_repo.branch:remove-systemprivateservicemodel
source_repo.url:https://apidrop.visualstudio.com/_git/binaries

Remove System.Private.ServiceModel package from .NET Standard 2.0. See #10157 (comment).

 Build.Reason:Manual by Genevieve Warren
 Build.Url:https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=441514&view=results
 source_repo.branch:remove-systemprivateservicemodel
 source_repo.url:https://apidrop.visualstudio.com/_git/binaries
Copy link

Learn Build status updates of commit 6b8114c:

⚠️ Validation status: warnings

File Status Preview URL Details
xml/System.IdentityModel.Claims/WindowsClaimSet.xml ⚠️Warning View Details
xml/System.Runtime.Diagnostics/EventTraceActivity.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.Runtime/TimeoutHelper.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/ClientWebSocketFactory.xml ⚠️Warning View Details
xml/System.ServiceModel.Channels/CommunicationPool`2.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/CommunicationPool`2+EndpointConnectionPool.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/CommunicationPool`2+EndpointConnectionPool+PoolIdleConnectionPool.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/CommunicationPool`2+IdleConnectionPool.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/ConnectionPool.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/ConnectionPoolRegistry.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/IAsyncRequest.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/IConnection.xml ⚠️Warning View Details
xml/System.ServiceModel.Channels/IConnectionInitiator.xml ⚠️Warning View Details
xml/System.ServiceModel.Channels/IConnectionOrientedConnectionSettings.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/IConnectionOrientedTransportChannelFactorySettings.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/IConnectionOrientedTransportFactorySettings.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/IdlingCommunicationPool`2.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/IdlingCommunicationPool`2+IdleTimeoutEndpointConnectionPool.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/IdlingCommunicationPool`2+IdleTimeoutEndpointConnectionPool+IdleTimeoutIdleConnectionPool.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/IMessageSource.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/IRequestBase.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/ITcpChannelFactorySettings.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/ITransportFactorySettings.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/MaxMessageSizeStream.xml ⚠️Warning n/a (file deleted or renamed) Details
xml/System.ServiceModel.Channels/RequestChannel.xml ⚠️Warning n/a (file deleted or renamed) Details

This comment lists only the first 25 files in the pull request.

xml/System.IdentityModel.Claims/WindowsClaimSet.xml

  • [Warning: file-not-redirected - See documentation] File xml/System.IdentityModel.Claims/WindowsClaimSet.xml with URL /dotnet/api/system.identitymodel.claims.windowsclaimset.trycreatewindowssidclaim was deleted without redirection. To avoid broken links, add a redirection.

xml/System.Runtime.Diagnostics/EventTraceActivity.xml

  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime.Diagnostics/EventTraceActivity.xml with URL /dotnet/api/system.runtime.diagnostics.eventtraceactivity was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime.Diagnostics/EventTraceActivity.xml with URL /dotnet/api/system.runtime.diagnostics.eventtraceactivity.activityid was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime.Diagnostics/EventTraceActivity.xml with URL /dotnet/api/system.runtime.diagnostics.eventtraceactivity.-ctor was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime.Diagnostics/EventTraceActivity.xml with URL /dotnet/api/system.runtime.diagnostics.eventtraceactivity.empty was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime.Diagnostics/EventTraceActivity.xml with URL /dotnet/api/system.runtime.diagnostics.eventtraceactivity.getactivityidfromthread was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime.Diagnostics/EventTraceActivity.xml with URL /dotnet/api/system.runtime.diagnostics.eventtraceactivity.getfromthreadorcreate was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime.Diagnostics/EventTraceActivity.xml with URL /dotnet/api/system.runtime.diagnostics.eventtraceactivity.name was deleted without redirection. To avoid broken links, add a redirection.

xml/System.Runtime/TimeoutHelper.xml

  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.add was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.-ctor was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.divide was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.elapsedtime was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.frommilliseconds was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.getcancellationtoken was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.getcancellationtokenasync was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.istoolarge was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.maxwait was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.min was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.originaltimeout was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.remainingtime was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.subtract was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.throwifnegativeargument was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.throwifnonpositiveargument was deleted without redirection. To avoid broken links, add a redirection.
  • [Warning: file-not-redirected - See documentation] File xml/System.Runtime/TimeoutHelper.xml with URL /dotnet/api/system.runtime.timeouthelper.tomilliseconds was deleted without redirection. To avoid broken links, add a redirection.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@gewarren gewarren merged commit 6c6e9e6 into main Jul 26, 2024
3 checks passed
@gewarren gewarren deleted the remove-private-package branch July 26, 2024 18:58
@@ -3,7 +3,7 @@
<TypeSignature Language="ILAsm" Value=".class public auto ansi beforefieldinit SynchronizedCollection`1&lt;T&gt; extends System.Object implements class System.Collections.Generic.ICollection`1&lt;!T&gt;, class System.Collections.Generic.IEnumerable`1&lt;!T&gt;, class System.Collections.Generic.IList`1&lt;!T&gt;, class System.Collections.ICollection, class System.Collections.IEnumerable, class System.Collections.IList" />
<TypeSignature Language="DocId" Value="T:System.Collections.Generic.SynchronizedCollection`1" />
<TypeSignature Language="VB.NET" Value="Public Class SynchronizedCollection(Of T)&#xA;Implements ICollection(Of T), IEnumerable(Of T), IList, IList(Of T)" />
<TypeSignature Language="F#" Value="type SynchronizedCollection&lt;'T&gt; = class&#xA; interface IList&lt;'T&gt;&#xA; interface ICollection&lt;'T&gt;&#xA; interface seq&lt;'T&gt;&#xA; interface IEnumerable&#xA; interface IList&#xA; interface ICollection" FrameworkAlternate="net-8.0;netframework-4.6;netframework-4.6.1;netframework-4.6.2;netframework-4.7;netframework-4.7.1;netframework-4.7.2;netframework-4.8;netframework-4.8.1;netstandard-2.0" />
<TypeSignature Language="F#" Value="type SynchronizedCollection&lt;'T&gt; = class&#xA; interface IList&lt;'T&gt;&#xA; interface ICollection&lt;'T&gt;&#xA; interface seq&lt;'T&gt;&#xA; interface IEnumerable&#xA; interface IList&#xA; interface ICollection" FrameworkAlternate="net-8.0;netframework-4.6;netframework-4.6.1;netframework-4.6.2;netframework-4.7;netframework-4.7.1;netframework-4.7.2;netframework-4.8;netframework-4.8.1" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something still seems a bit off here. SynchronizedCollection is available for netstandard-2.0 in the 4.10.3 version of System.ServiceModel.Primitives. netstandard-2.0 support will be coming back in the latest packages soon.

<TypeSignature Language="DocId" Value="T:System.IdentityModel.Selectors.SecurityTokenResolver" />
<TypeSignature Language="VB.NET" Value="Public MustInherit Class SecurityTokenResolver" FrameworkAlternate="net-8.0;netframework-3.0;netframework-3.5;netframework-4.0;netstandard-2.0" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does it mean for netstandard-2.0 to be removed here? SecurityTokenResolver is available for netstandard2.0 from System.ServiceModel.Primitives in 4.10.3

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The package indexer tool only considers the latest stable (and latest preview version, if it's newer), which for System.ServiceModel.Primitives is 8.0.0. Is there a need to index 4.10.3 as well?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For now there is. We've extended support because we removed netstandard2.0 support from 8.0.0 and turns out it was more important than we were told so we're bringing it back. For now the 4.10.3 package is what you use if you need netstandard2.0 support.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, I'll special case it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mconnew I'm having a hard time ingesting the System.ServiceModel.Primitives 4.10.3 package for .NET Standard. I tried with both the lib and ref assemblies, and both times got:

##[error]mdoc: Mono.Documentation.MDocAssemblyException: Error caching System.ServiceModel, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089 from D:\a_work\1\s\source_repo\dotnet\netstandard-2.0\System.ServiceModel.dll ---> Mono.Documentation.MDocException: Failed to resolve type 'System.ServiceModel.BasicHttpBinding' ---> Mono.Cecil.AssemblyResolutionException: Failed to resolve assembly: 'System.ServiceModel.Http, Version=4.10.3.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'
at Mono.Documentation.Updater.Frameworks.MDocBaseResolver.Resolve(AssemblyNameReference name, ReaderParameters parameters, IEnumerable`1 filesToIgnore)

Should I also ingest this package for the documentation? https://www.nuget.org/packages/System.ServiceModel.Http/4.10.3

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update: I added System.ServiceModel.Http, but now getting an error for a dependency on System.ServiceModel.Duplex:

##[error]mdoc: Mono.Documentation.MDocAssemblyException: Error caching System.ServiceModel, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089 from D:\a_work\1\s\source_repo\dotnet\netstandard-2.0\System.ServiceModel.dll ---> Mono.Documentation.MDocException: Failed to resolve type 'System.ServiceModel.CallbackBehaviorAttribute' ---> Mono.Cecil.AssemblyResolutionException: Failed to resolve assembly: 'System.ServiceModel.Duplex, Version=4.10.3.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'

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

Successfully merging this pull request may close these issues.

3 participants