-
Notifications
You must be signed in to change notification settings - Fork 810
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
Adds Counters and Lists REST API Conformance Tests #3546
Adds Counters and Lists REST API Conformance Tests #3546
Conversation
…quest as the REST API can request to change the capacity, count, and countdiff all in one request. Also updates the SDK Server to follow the same change update pattern as the local SDK server of first updating the capacity, then setcount, then countdiff. Fixes a bug in the local SDK server to properly update the setcount.
…omatic complexity
Build Succeeded 👏 Build Id: 221552aa-bb48-4434-a403-12d4598e8070 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
Build Failed 😱 Build Id: 4093cbe4-4ce8-4245-a704-114f6ca49a04 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: f8c60d1d-fed1-4f81-a576-c9ab2cb7227c The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
What type of PR is this?
/kind feature
What this PR does / Why we need it:
GetCounter
,UpdateCounter
,GetList
,UpdateList
,AddListValue
andRemoveListValue
which are different from the client methods in the Arbitrary Counts and Lists for GameServers, SDKs and Allocation #2716 that the Go SDK uses and other language SDKs will use.)Which issue(s) this PR fixes:
Working on #2716
Special notes for your reviewer: