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

Fix: Orleans.Hosting.KubernetesHosting: System.MissingMethodException #7346

Merged
merged 1 commit into from Oct 25, 2021

Conversation

highlyunavailable
Copy link
Contributor

@highlyunavailable highlyunavailable commented Oct 22, 2021

Due to kubernetes-client/csharp#681 in C# Kubernetes clients newer than 6.0.2 (e.g. when using KubeOps in conjunction with Orleans in the same app), the following error is thrown when using Orleans.Hosting.KubernetesHosting:

System.MissingMethodException: Method not found: 'System.Net.Http.DelegatingHandler k8s.KubernetesClientConfiguration.CreateWatchHandler()'.
    at Orleans.Hosting.KubernetesHostingExtensions.UseKubernetesHosting(IServiceCollection services, Action`1 configureOptions)

This fixes the issue by removing the offending line, which is no longer needed, and upgrading the referenced Kubernetes client.

Microsoft Reviewers: Open in CodeFlow

Due to kubernetes-client/csharp#681 in C# Kubernetes clients newer than 6.0.2, the following error is thrown when using `Orleans.Hosting.KubernetesHosting`:

```
Unhandled exception. System.MissingMethodException: Method not found: 'System.Net.Http.DelegatingHandler k8s.KubernetesClientConfiguration.CreateWatchHandler()'.
    at Orleans.Hosting.KubernetesHostingExtensions.UseKubernetesHosting(IServiceCollection services, Action`1 configureOptions)
```

This fixes the issue by removing the offending line, which is no longer needed.
@dnfadmin
Copy link

dnfadmin commented Oct 22, 2021

CLA assistant check
All CLA requirements met.

@highlyunavailable
Copy link
Contributor Author

Not sure what I can do about those test failures, looks like DynamoDB tests failed, which I haven't touched.

@ReubenBond
Copy link
Member

Thanks, Ann, there is something flaky about the DynamoDB tests which I'm not sure about. They're meant to be skipped when run on GitHub. Regardless, it's not a concern for this PR. Thanks for the contribution

@ReubenBond ReubenBond merged commit 4718d48 into dotnet:main Oct 25, 2021
@ReubenBond ReubenBond added the Needs: backport PRs which should be backported to a (prior) release branch label Oct 25, 2021
@highlyunavailable highlyunavailable deleted the upgrade-kuberentes-client branch October 25, 2021 21:24
benjaminpetit pushed a commit to benjaminpetit/orleans that referenced this pull request Oct 29, 2021
…dotnet#7346)

Due to kubernetes-client/csharp#681 in C# Kubernetes clients newer than 6.0.2, the following error is thrown when using `Orleans.Hosting.KubernetesHosting`:

```
Unhandled exception. System.MissingMethodException: Method not found: 'System.Net.Http.DelegatingHandler k8s.KubernetesClientConfiguration.CreateWatchHandler()'.
    at Orleans.Hosting.KubernetesHostingExtensions.UseKubernetesHosting(IServiceCollection services, Action`1 configureOptions)
```

This fixes the issue by removing the offending line, which is no longer needed.
ReubenBond pushed a commit that referenced this pull request Oct 29, 2021
…7364)

* Fix: Orleans.Hosting.KubernetesHosting: System.MissingMethodException (#7346)

Due to kubernetes-client/csharp#681 in C# Kubernetes clients newer than 6.0.2, the following error is thrown when using `Orleans.Hosting.KubernetesHosting`:

```
Unhandled exception. System.MissingMethodException: Method not found: 'System.Net.Http.DelegatingHandler k8s.KubernetesClientConfiguration.CreateWatchHandler()'.
    at Orleans.Hosting.KubernetesHostingExtensions.UseKubernetesHosting(IServiceCollection services, Action`1 configureOptions)
```

This fixes the issue by removing the offending line, which is no longer needed.

* Fix build for netstandard2.0

Co-authored-by: Ann Wright <highlyunavailable@gmail.com>
@github-actions github-actions bot locked and limited conversation to collaborators Dec 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Needs: backport PRs which should be backported to a (prior) release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants