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

[BUG] Microsoft.Rest.ClientRuntime Dependency Loop #18024

Closed
TheOnlyWei opened this issue Jan 16, 2021 · 11 comments
Closed

[BUG] Microsoft.Rest.ClientRuntime Dependency Loop #18024

TheOnlyWei opened this issue Jan 16, 2021 · 11 comments
Assignees
Labels
AutoRest Runtime Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. issue-addressed Workflow: The Azure SDK team believes it to be addressed and ready to close.

Comments

@TheOnlyWei
Copy link
Contributor

TheOnlyWei commented Jan 16, 2021

Describe the bug
Please provide the description of issue you're seeing.

Installing the following package:
https://www.nuget.org/packages/Microsoft.Rest.ClientRuntime/

NOTE: the command is wrong, the parameter -Version should be -RequiredVersion.

After entering the command, I get a dependency loop error:

PS C:\Users\Administrator\Desktop\dotnet\hybrid-compute-dotnet-manage-vm> Install-Package Microsoft.Rest.ClientRuntime -RequiredVersion 2.3.22

The package(s) come(s) from a package source that is not marked as trusted.
Are you sure you want to install software from 'nuget.org'?
[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "N"): A
Install-Package : Dependency loop detected for package 'Microsoft.Rest.ClientRuntime'.
At line:1 char:1
+ Install-Package Microsoft.Rest.ClientRuntime -RequiredVersion 2.3.22
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : Deadlock detected: (Microsoft.Rest.ClientRuntime:String) [Install-Package], Exception
    + FullyQualifiedErrorId : DependencyLoopDetected,Microsoft.PowerShell.PackageManagement.Cmdlets.InstallPackage

Expected behavior
What is the expected behavior?

Should not have an error.

Actual behavior (include Exception or Stack Trace)
What is the actual behavior?

There was an error.

To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

  1. Install-Package Microsoft.Rest.ClientRuntime -RequiredVersion 2.3.22

Environment:

  • Name and version of the Library package used: Microsoft.Rest.ClientRuntime 2.3.22
  • Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): Windows Server 2019 Datacenter
  • IDE and version : [e.g. Visual Studio 16.3] Visual Studio Code
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Jan 16, 2021
@jsquire jsquire added AutoRest Runtime Mgmt This issue is related to a management-plane library. needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team labels Jan 16, 2021
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Jan 16, 2021
@jsquire
Copy link
Member

jsquire commented Jan 16, 2021

Thank you for your feedback. Tagging and routing to the team member best able to assist.

@YalinLi0312
Copy link
Member

@jsquire It looks is Microsoft.Rest.ClientRuntime, why it is a question for management package?

@jsquire jsquire assigned pakrym and AlexGhiondea and unassigned allenjzhang Jan 26, 2021
@jsquire jsquire added Client This issue points to a problem in the data-plane of the library. and removed Mgmt This issue is related to a management-plane library. labels Jan 26, 2021
@jsquire
Copy link
Member

jsquire commented Jan 26, 2021

@YalinLi0312: That's a mistake on may part. Apologies for the confusion.

@ghost ghost added the no-recent-activity There has been no recent activity on this issue. label Mar 12, 2021
@ghost
Copy link

ghost commented Mar 12, 2021

Hi @TheOnlyWei. There hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by removing the no-recent-activity label. Otherwise, we'll close this out in 7 days.

@ghost ghost removed the no-recent-activity There has been no recent activity on this issue. label Mar 13, 2021
@ghost
Copy link

ghost commented Mar 13, 2021

Hi @TheOnlyWei. There was a mistake and this issue was unintentionally flagged as a stale pull request. The label has been removed and the issue will remain active; no action is needed on your part. Apologies for the inconvenience.

@AlexGhiondea
Copy link
Contributor

@TheOnlyWei there is a newer version of the package -- are you still seeing this?

@AlexGhiondea AlexGhiondea added the needs-author-feedback Workflow: More information is needed from author to address the issue. label Dec 17, 2021
@ghost ghost removed the needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team label Dec 17, 2021
@AlexGhiondea AlexGhiondea added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Dec 17, 2021
@ghost ghost added the no-recent-activity There has been no recent activity on this issue. label Dec 24, 2021
@ghost
Copy link

ghost commented Dec 24, 2021

Hi, we're sending this friendly reminder because we haven't heard back from you in 7 days. We need more information about this issue to help address it. Please be sure to give us your input. If we don't hear back from you within 14 days of this comment the issue will be automatically closed. Thank you!

@TheOnlyWei
Copy link
Contributor Author

@AlexGhiondea Yes, I get the same error in 2.3.23 (the latest version I found).

Install-Package: Dependency loop detected for package 'Microsoft.Rest.ClientRuntime'.

@ghost ghost added needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team and removed needs-author-feedback Workflow: More information is needed from author to address the issue. no-recent-activity There has been no recent activity on this issue. labels Dec 25, 2021
@pakrym
Copy link
Contributor

pakrym commented Dec 27, 2021

There are two versions of Install-Package commands. One exists inside Package Manager Console in VisualStudio and works just fine:

image

The other is from PackageManagement Powershell module and is intended for general-purpose powershell packages, not .NET packages. You are ending up using this version. It's unable to handle .NET-specific package authoring.

When using VSCOde I recommend using dotnet package add to install the package.

dotnet add package Microsoft.Rest.ClientRuntime --version 2.3.22

@pakrym pakrym added the issue-addressed Workflow: The Azure SDK team believes it to be addressed and ready to close. label Dec 27, 2021
@ghost
Copy link

ghost commented Dec 27, 2021

Hi @TheOnlyWei. Thank you for opening this issue and giving us the opportunity to assist. We believe that this has been addressed. If you feel that further discussion is needed, please add a comment with the text “/unresolve” to remove the “issue-addressed” label and continue the conversation.

@ghost ghost removed the needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team label Dec 27, 2021
@ghost
Copy link

ghost commented Jan 3, 2022

Hi @TheOnlyWei, since you haven’t asked that we “/unresolve” the issue, we’ll close this out. If you believe further discussion is needed, please add a comment “/unresolve” to reopen the issue.

@ghost ghost closed this as completed Jan 3, 2022
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-net that referenced this issue Mar 22, 2022
Review request for Microsoft.ContainerService to add version 2022-02-01 (Azure#18324)

* Adds base for updating Microsoft.ContainerService from version stable/2022-01-01 to version 2022-02-01

* Updates readme

* Updates API version in new specs and examples

* Update readmes for the 2022-02-01 dev branch of container service (Azure#17887)

* update readme

* remove useless -only tags

* Fix violated rule R2026 for 2022-02-01 managedCluster swagger (Azure#18024)

* fix allof

* add missing type

* GA alias minor version (Azure#18038)

* GA alias minor version

* polish more details on kubernetesVersion and currentKubernetesVersion

* Update specification/containerservice/resource-manager/Microsoft.ContainerService/stable/2022-02-01/managedClusters.json

minor revision

Co-authored-by: Matthew Christopher <[email protected]>

Co-authored-by: Matthew Christopher <[email protected]>

* add missing type object (Azure#18115)

* Revert "GA alias minor version (Azure#18038)" (Azure#18291)

This reverts commit 3a99cda3cb48e05c74923f5467c0737014322b0f.

* Align modifications of several common definitions with 2022-02-02-preview for container service 2022-02-01 (Azure#18216)

* sync modification

* fix ref

Co-authored-by: Jianping Zeng <[email protected]>
Co-authored-by: Matthew Christopher <[email protected]>
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
AutoRest Runtime Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. issue-addressed Workflow: The Azure SDK team believes it to be addressed and ready to close.
Projects
None yet
Development

No branches or pull requests

6 participants