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

HealthCheckLoop Never invoked in C# SDK #1583

Closed
stevenflautner opened this issue May 24, 2020 · 3 comments · Fixed by #1596
Closed

HealthCheckLoop Never invoked in C# SDK #1583

stevenflautner opened this issue May 24, 2020 · 3 comments · Fixed by #1596
Labels
area/user-experience Pertaining to developers trying to use Agones, e.g. SDK, installation, etc kind/bug These are bugs.
Milestone

Comments

@stevenflautner
Copy link

What happened:
Agones Game Server becomes Unhealthy because Health is never signaled by the SDK.
HealthCheckLoop function is never invoked in Agones Game Server SDK for C#

What you expected to happen:
HealthCheckLoop function should be invoked by the SDK in AgonesSDK.cs and since it being private there's no way to call it externally.

		private async Task HealthCheckLoop()
		{
			while(!cts.IsCancellationRequested)
			{
				if(HealthEnabled)
					await HealthAsync();
				await Task.Delay(HealthInterval * 1000);
			}
		}

How to reproduce it (as minimally and precisely as possible):
Use the AgonesSDK for C#

Anything else we need to know?:

Environment:

  • Agones version: SDK 1.0.2
  • Kubernetes version (use kubectl version):
  • Cloud provider or hardware configuration:
  • Install method (yaml/helm):
  • Troubleshooting guide log(s):
  • Others:
@stevenflautner stevenflautner added the kind/bug These are bugs. label May 24, 2020
@aLekSer
Copy link
Collaborator

aLekSer commented May 25, 2020

There is a lack of SDK conformance tests as well as Unit Tests for this functionality. So there is a need to fix C# SDK.

@aLekSer
Copy link
Collaborator

aLekSer commented May 26, 2020

For now you can call HealthAsync.

internal async Task<Status> HealthAsync()

HealthCheckLoop is redundant, I could not find such functionality in Golang SDK.

@markmandel
Copy link
Member

@Davidnovarro this is a bug against the C# SDK. #1598 is a PR against the Unity SDK, they are different SDKs.

@markmandel markmandel added this to the 1.7.0 milestone Jun 30, 2020
@markmandel markmandel added the area/user-experience Pertaining to developers trying to use Agones, e.g. SDK, installation, etc label Jun 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/user-experience Pertaining to developers trying to use Agones, e.g. SDK, installation, etc kind/bug These are bugs.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants