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

SDK Server ignores custom GameServer configuration file in local mode #1508

Closed
cadfoot opened this issue May 1, 2020 · 2 comments · Fixed by #1509
Closed

SDK Server ignores custom GameServer configuration file in local mode #1508

cadfoot opened this issue May 1, 2020 · 2 comments · Fixed by #1509
Labels
good first issue These are great first issues. If you are looking for a place to start, start here! help wanted We would love help on these issues. Please come help us! kind/bug These are bugs.
Milestone

Comments

@cadfoot
Copy link
Contributor

cadfoot commented May 1, 2020

What happened:
SDK Server ignores custom GameServer configuration file in local mode

What you expected to happen:
SDK Server initializes with provided GameServer configuration when running locally

How to reproduce it (as minimally and precisely as possible):

  1. Create a GameServer config (health check is disabled in this example)
apiVersion: "agones.dev/v1"
kind: GameServer
metadata:
  generateName: "simple-udp-"
spec:
  ports:
  - name: default
    portPolicy: Dynamic
    containerPort: 7654
  health:
    disabled: true
  1. Run SDK Server locally with created config
sdk-server.windows.amd64.exe --local -f gameserver.yaml
  1. Request game server information
curl -GET "http://localhost:9358/gameserver" -H "accept: application/json"
  1. Notice that health check is still enabled (formatted the output for readability)
{
  "object_meta": {
    ...
  },
  "spec": {
    "health": {
      "period_seconds": 3,
      "failure_threshold": 5,
      "initial_delay_seconds": 10
    }
  },
  "status": {
    ...
  }
}

Anything else we need to know?:
Reproduced on versions above 1.3.0 (did not test RCs).

Environment:

  • Agones version: 1.5.0
  • Kubernetes version (use kubectl version):
  • Cloud provider or hardware configuration:
  • Install method (yaml/helm):
  • Troubleshooting guide log(s):
  • Others:
@cadfoot cadfoot added the kind/bug These are bugs. label May 1, 2020
@markmandel markmandel added good first issue These are great first issues. If you are looking for a place to start, start here! help wanted We would love help on these issues. Please come help us! labels May 1, 2020
@markmandel
Copy link
Member

markmandel commented May 1, 2020

Looking at the code, I don't think we actually support -f like the docs say we do.
Sorry, looks like we do:
https://github.com/googleforgames/agones/blob/master/cmd/sdk-server/main.go#L271

But would still like to know the answer to below.

Does this bug still occur if you use --file ?

If it does, can you also share the local SDK server's log output, would be useful.

@markmandel
Copy link
Member

Looks like I can replicate on my end. Should get a PR in shortly.

You can download a working local sdk executable from the PR when it's ready too.

markmandel added a commit to markmandel/agones that referenced this issue May 1, 2020
Subtle bug in variable assignment caused the sdk server to not load a
GameServer yaml when passed in.

Closes googleforgames#1508
markmandel added a commit that referenced this issue May 1, 2020
Subtle bug in variable assignment caused the sdk server to not load a
GameServer yaml when passed in.

Closes #1508

Co-authored-by: Alexander Apalikov <[email protected]>
@markmandel markmandel added this to the 1.6.0 milestone May 1, 2020
aLekSer added a commit to aLekSer/agones that referenced this issue May 4, 2020
Subtle bug in variable assignment caused the sdk server to not load a
GameServer yaml when passed in.

Closes googleforgames#1508

Co-authored-by: Alexander Apalikov <[email protected]>
ilkercelikyilmaz pushed a commit to ilkercelikyilmaz/agones that referenced this issue Oct 23, 2020
Subtle bug in variable assignment caused the sdk server to not load a
GameServer yaml when passed in.

Closes googleforgames#1508

Co-authored-by: Alexander Apalikov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue These are great first issues. If you are looking for a place to start, start here! help wanted We would love help on these issues. Please come help us! kind/bug These are bugs.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants