-
Notifications
You must be signed in to change notification settings - Fork 823
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
Updates upgrade test to install multiple versions of Agones on a cluster in succession #3982
Conversation
6cdd881
to
8cbdf19
Compare
Build Failed 😭 Build Id: e6a67137-8741-46d9-bf54-575f827ed7ad Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
8cbdf19
to
b298c1f
Compare
Build Succeeded 🥳 Build Id: b524d267-46fb-48c3-a73a-82b21475f6a5 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:
|
61d3028
to
8646c26
Compare
Build Failed 😭 Build Id: 1eb2a7d8-315b-49b0-810b-114622e792d1 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 🥳 Build Id: abc17acf-69fb-4b48-be9f-6107b66aa9ba 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:
|
8646c26
to
079c7ab
Compare
Build Succeeded 🥳 Build Id: f229b7a3-beaa-481f-871b-0e59d6dc6c59 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:
|
079c7ab
to
ee46947
Compare
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. |
Build Failed 😭 Build Id: 55c81a37-6d4e-48a7-8a04-0be02a505bd6 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
cdf3ff4
to
ee46947
Compare
Build Succeeded 🥳 Build Id: a332445f-7bf8-4b9c-8c8f-55e29842b801 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:
This creates a basic config walker that installs all versions of Agones that are valid for a cluster on a particular Kubernetes versions. One valid version of Agones is installed, and the controller waits until the Helm release (Agones installation) is ready (deployed) before installing the next version.
Which issue(s) this PR fixes:
Working on #3795
Special notes for your reviewer:
The
upgrade-test-runner
Job runs a pod that is effectively a controller uses Helm to install Agones into theagones-system
namespace on the same cluster. This Job needs to have all of the permissions of both Helm and Agones in order to use and create those resources.Wait groups do not work for waiting on Kubernetes or Helm, so we use
wait.PollUntilContextTimeout
instead, similar to other e2e tests in Agones.There are a number of
//TODO
s in the main file denoting pieces that are not included in this particular PR, but will need to be included in future PRs.To run:
REGISTRY
set to your artifact registry$(REGISTRY)
value in upgradeTest.yaml currently needs to be manually set to your registrykubectl apply -f permissions.yaml
for reading Pod and Node and installing Agoneskubectl apply -f versionMap.yaml
to load the configmapkubectl apply -f upgradeTest.yaml
to run the test runnerresource.labels.container_name="upgrade-test-controller"
kubectl get po -n agones-system
to see the Agones pods. These are expected to crash loop a couple times when running on an Autopilot cluster as the Autopilot cluster needs to scale up to accommodate the pods.kubectl get jobs --watch
to see if the job is still running.helm history agones -n agones-system
. Example for a Kubernetes version 1.29 cluster: