Skip to content
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

Unblocks Agones release PR by waiting for either the Agones dev version or release version #4078

Merged
merged 2 commits into from
Jan 3, 2025

Conversation

igooch
Copy link
Collaborator

@igooch igooch commented Jan 2, 2025

What type of PR is this?

/kind hotfix

What this PR does / Why we need it:

The current release PR #4077 is blocked by the upgrade tests. The Logs Explorer showed the error PollUntilContextTimeout timed out while attempting upgrade to Agones version 1.46.0-dev-633ac4f. which is consistent with the code

agones/test/upgrade/main.go

Lines 316 to 328 in 8166704

// Wait for the helm release to install. Waits the same amount of time as the Helm timeout.
var helmStatus string
err = wait.PollUntilContextTimeout(ctx, 10*time.Second, Timeout, true, func(_ context.Context) (done bool, err error) {
helmStatus = checkHelmStatus(config.agonesVersion)
if helmStatus == "deployed" {
return true, nil
}
return false, nil
})
if err != nil || helmStatus != "deployed" {
log.Fatalf("PollUntilContextTimeout timed out while attempting upgrade to Agones version %s. Helm Status %s",
config.agonesVersion, helmStatus)
}
that is waiting for the "dev" chart to be installed. However, the release PR changes the chart from agones-1.46.0-dev to agones-1.46.0, so the test waits indefinitely.

This PR updates the waiting code to wait for either agones-{DevVersion}-dev or agones-{DevVersion}.

Which issue(s) this PR fixes:

NA

Special notes for your reviewer:

@github-actions github-actions bot added kind/hotfix Hotfixes for issues against release size/XS labels Jan 2, 2025
@igooch igooch requested a review from markmandel January 2, 2025 22:09
// Remove the commit sha from the DevVersion i.e. from 1.46.0-dev-7168dd3 to 1.46.0-dev
// Remove the commit sha from the DevVersion i.e. from 1.46.0-dev-7168dd3 to 1.46.0-dev or 1.46.0
// for the case of this test running during a new Agones version release PR.
agonesRelease := ""
Copy link
Collaborator

Choose a reason for hiding this comment

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

(non blocking) in the future, may want to use a library like https://github.com/blang/semver (which we use in a few util scripts) to work with semantic versions, rather than regex things.

@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: 2a484667-6651-45b6-ad8a-3dccb9a095fb

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@github-actions github-actions bot added the size/S label Jan 2, 2025
@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: af26185f-e1d2-4a03-9a27-d121a5a33a00

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/4078/head:pr_4078 && git checkout pr_4078
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.46.0-dev-5925c64

@igooch igooch merged commit 146fc31 into googleforgames:main Jan 3, 2025
4 checks passed
@igooch igooch deleted the release-hotfix branch January 3, 2025 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/hotfix Hotfixes for issues against release size/S size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants