-
Notifications
You must be signed in to change notification settings - Fork 829
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
Allow env var overrides for e2e tests #1566
Conversation
Build Failed 😱 Build Id: ae8021ad-7d0e-4bd9-8240-7ba4e316725b To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Doh, broke a thing:
I'll come back to this later. |
The ordering of env var parsing meant you couldn't override test values such as image with an environment variable, even though the intent of the code seems to indicate that is the desired result. This change allows the use of environment variables can now be used!
Build Failed 😱 Build Id: 99d7d42b-4804-41fe-aad4-8e8d1c5f52d8 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Looks like it might be a bit of a flake. Might want to put a retry on that operation. |
Build Succeeded 👏 Build Id: 67710271-2678-44ee-ae8a-d4d75df5e9fa 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:
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: markmandel, roberthbailey The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The ordering of env var parsing meant you couldn't override test values such as image with an environment variable, even though the intent of the code seems to indicate that is the desired result. This change allows the use of environment variables can now be used!
What type of PR is this?
/kind bug
What this PR does / Why we need it:
The ordering of env var parsing meant you couldn't override test values such as image with an environment variable, even though the intent of the code seems to indicate that is the desired result.
This change allows the use of environment variables can now be used!
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
None