Skip to content

Conversation

miai10
Copy link
Contributor

@miai10 miai10 commented Aug 13, 2025

What type of PR is this?

Uncomment only one /kind <> line, press enter to put that in a new line, and remove leading whitespace from that line:

/kind breaking
/kind bug
/kind cleanup
/kind documentation
/kind feature
/kind hotfix
/kind release

What this PR does / Why we need it:

The number of items in a list are bounded to the hardcoded value of 1000. During load tests, this value seems too low so a mechanism to configure it was put in place using Helm values. The default is still 1000 to be backwards compatible.

Which issue(s) this PR fixes:

Closes #4192

Special notes for your reviewer:

@github-actions github-actions bot added kind/feature New features for Agones size/S labels Aug 13, 2025
@miai10
Copy link
Contributor Author

miai10 commented Aug 13, 2025

Using the make gen-install command returns an error because the .Values.agones.crds.gameserver.lists.maxItems value is nil. It is suppose to use the value from the values file. Not sure why is that, still investigating but let me know if somebody has an idea...

The complete error:

`root@2ddf739d200f:/workspaces/agones/build# make gen-install

mkdir -p ~/.kube/
mkdir -p /workspaces/agones/build//.gocache
mkdir -p /workspaces/agones/build//.config/gcloud
mkdir -p ~/.config/helm
mkdir -p ~/.cache/helm
make ensure-image IMAGE_TAG=agones-build:419ba293a7 BUILD_TARGET=build-build-image
make[1]: Entering directory '/workspaces/agones/build'
make[1]: Leaving directory '/workspaces/agones/build'
docker run --rm -v /workspaces/agones/build//.config/gcloud:/root/.config/gcloud -v ~/.kube/:/root/.kube -v ~/.config/helm:/root/.config/helm -v ~/.cache/helm:/root/.cache/helm -v /workspaces/agones:/go/src/agones.dev/agones -v /workspaces/agones/build//.gomod:/go/pkg/mod -v /workspaces/agones/build//.gocache:/root/.cache/go-build agones-build:419ba293a7 bash -c
'helm template agones-manual --namespace agones-system /go/src/agones.dev/agones/install/helm/agones
--set agones.controller.generateTLS=false
--set agones.allocator.generateTLS=false
--set agones.allocator.generateClientTLS=false
--set agones.crds.cleanupOnDelete=false
> /go/src/agones.dev/agones/install/yaml/install.yaml'
Error: template: agones/templates/crds/gameserverset.yaml:117:22: executing "agones/templates/crds/gameserverset.yaml" at <include "gameserver.schema" $data>: error calling include: template: agones/templates/crds/_gameserverspecschema.yaml:194:33: executing "gameserver.schema" at <.Values.agones.crds.gameserver.lists.maxItems>: nil pointer evaluating interface {}.agones`

@miai10
Copy link
Contributor Author

miai10 commented Aug 13, 2025

Using the make gen-install command returns an error because the .Values.agones.crds.gameserver.lists.maxItems value is nil. It is suppose to use the value from the values file. Not sure why is that, still investigating but let me know if somebody has an idea...

The complete error:

`root@2ddf739d200f:/workspaces/agones/build# make gen-install

mkdir -p ~/.kube/ mkdir -p /workspaces/agones/build//.gocache mkdir -p /workspaces/agones/build//.config/gcloud mkdir -p ~/.config/helm mkdir -p ~/.cache/helm make ensure-image IMAGE_TAG=agones-build:419ba293a7 BUILD_TARGET=build-build-image make[1]: Entering directory '/workspaces/agones/build' make[1]: Leaving directory '/workspaces/agones/build' docker run --rm -v /workspaces/agones/build//.config/gcloud:/root/.config/gcloud -v ~/.kube/:/root/.kube -v ~/.config/helm:/root/.config/helm -v ~/.cache/helm:/root/.cache/helm -v /workspaces/agones:/go/src/agones.dev/agones -v /workspaces/agones/build//.gomod:/go/pkg/mod -v /workspaces/agones/build//.gocache:/root/.cache/go-build agones-build:419ba293a7 bash -c 'helm template agones-manual --namespace agones-system /go/src/agones.dev/agones/install/helm/agones --set agones.controller.generateTLS=false --set agones.allocator.generateTLS=false --set agones.allocator.generateClientTLS=false --set agones.crds.cleanupOnDelete=false > /go/src/agones.dev/agones/install/yaml/install.yaml' Error: template: agones/templates/crds/gameserverset.yaml:117:22: executing "agones/templates/crds/gameserverset.yaml" at <include "gameserver.schema" $data>: error calling include: template: agones/templates/crds/_gameserverspecschema.yaml:194:33: executing "gameserver.schema" at <.Values.agones.crds.gameserver.lists.maxItems>: nil pointer evaluating interface {}.agones`

Thanks, @lacroixthomas for the help to fix this!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change is not related to my changes but this is how the new install.yaml file got regenerated

@miai10 miai10 changed the title WIP: Using agones.crds.gameserver.lists.maxItems instead of a hardcoded limit feat: use agones.crds.gameserver.lists.maxItems instead of a hardcoded limit Aug 14, 2025
@miai10 miai10 changed the title feat: use agones.crds.gameserver.lists.maxItems instead of a hardcoded limit feat: use gameservers.lists.maxItems instead of a hardcoded limit Aug 18, 2025
@lacroixthomas
Copy link
Collaborator

/gcbrun

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 50854079-a8d7-433c-a210-e8119671a0d3

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:

git fetch https://github.com/googleforgames/agones.git pull/4246/head:pr_4246 && git checkout pr_4246
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.52.0-dev-a2341e7

@miai10
Copy link
Contributor Author

miai10 commented Aug 25, 2025

@markmandel @lacroixthomas : let me know if this needs more work or we should merge it

@markmandel
Copy link
Collaborator

I'm back from devcom and Gamescom - I can peek at this shortly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New features for Agones size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Proposal] - Remove the 1000 entries limit for lists
4 participants