Skip to content

Add env option to all Agones component deployments - #4499

Merged
markmandel merged 7 commits into
agones-dev:mainfrom
lazy-slackoth:feature/add-custom-env-vars
Apr 27, 2026
Merged

markmandel merged 7 commits into
agones-dev:mainfrom
lazy-slackoth:feature/add-custom-env-vars

Conversation

@lazy-slackoth

Copy link
Copy Markdown
Contributor

What type of PR is this?

/kind feature

What this PR does / Why we need it:
Adds an env field to each Agones component in values.yaml (controller, extensions, ping, allocator, allocator.processor), allowing users to inject arbitrary environment variables into any deployment. The full Kubernetes env var spec is supported (value, valueFrom, secretKeyRef, configMapKeyRef, fieldRef, etc.). Custom vars are appended after the built-in ones.

@markmandel

Copy link
Copy Markdown
Member

I like it!

/gcbrun

@markmandel

Copy link
Copy Markdown
Member

Just a gentle bump to see if you would like to round out this PR?

@lazy-slackoth

Copy link
Copy Markdown
Contributor Author

Yes,sure
Going to update this week

@lazy-slackoth
lazy-slackoth force-pushed the feature/add-custom-env-vars branch from 4785c53 to c634e39 Compare April 22, 2026 08:25
@github-actions

Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@lazy-slackoth
lazy-slackoth force-pushed the feature/add-custom-env-vars branch 2 times, most recently from ab331fe to 6ffb6b1 Compare April 22, 2026 08:36
@github-actions

Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@lazy-slackoth
lazy-slackoth force-pushed the feature/add-custom-env-vars branch 2 times, most recently from 19f0236 to 7a05c96 Compare April 22, 2026 08:44
@github-actions

Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Alexander Taran added 2 commits April 22, 2026 11:49
Signed-off-by: Alexander Taran <a.taran@strikerz.inc>
Signed-off-by: Alexander Taran <a.taran@strikerz.inc>
@lazy-slackoth
lazy-slackoth force-pushed the feature/add-custom-env-vars branch from 7a05c96 to d1b609b Compare April 22, 2026 08:49
Signed-off-by: Alexander Taran <a.taran@strikerz.inc>
@lazy-slackoth

Copy link
Copy Markdown
Contributor Author

From my side, it seems to be ready

| `agones.controller.mutatingWebhook.disableCaBundle` | **Deprecated**. Use `agones.extensions.mutatingWebhook.disableCaBundle` instead. Disable ca-bundle so it can be injected by cert-manager | `false` |
| `agones.controller.allocationBatchWaitTime` | Wait time between each allocation batch when performing allocations in controller mode | `500ms` |
| `agones.controller.topologySpreadConstraints` | Ensures better resource utilization and high availability by evenly distributing Pods in the agones-system namespace | `{}` |
| `agones.controller.env` | Additional [environment variables](https://kubernetes.io/docs/tasks/inject-data-application/define-environment-variable-container/) to inject into the controller pod | `[]` |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the docs, but these need to be feature shortcoded:
https://agones.dev/site/docs/contribute/documentation-editing-contribution/#within-a-page

This is because docs publish immediately, not on release 👍ðŸŧ

You may also need to duplicate the whole table, since shortcodes don't work so great when in the middle of tables.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed

Alexander Taran added 2 commits April 27, 2026 11:03
@markmandel

Copy link
Copy Markdown
Member

/gcbrun

@markmandel
markmandel enabled auto-merge (squash) April 27, 2026 15:17
@markmandel

Copy link
Copy Markdown
Member

/gcbrun

@markmandel
markmandel merged commit b15b22f into agones-dev:main Apr 27, 2026
4 checks passed
@agones-bot

Copy link
Copy Markdown
Collaborator

Build Succeeded ðŸĨģ

Build Id: 6934191d-d33b-4d10-9355-eee73080c91b

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/4499/head:pr_4499 && git checkout pr_4499
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.58.0-dev-a1e6831

@lazy-slackoth
lazy-slackoth deleted the feature/add-custom-env-vars branch April 29, 2026 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants