-
Notifications
You must be signed in to change notification settings - Fork 799
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
Release Automation: Generate version of website to push #2808
Conversation
Build Failed 😱 Build Id: 25bea67f-3990-4b5b-966a-3d0ea2e4790b To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
4c11eda
to
0bae03c
Compare
Build Succeeded 👏 Build Id: 42a96d16-579a-45d1-8dac-44469f28c3ae 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:
|
Build Succeeded 👏 Build Id: 104050c4-d4f8-4150-8b5b-c4f63aceaec8 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:
|
baseVersion := flag.String("version", "", "the base version to be taken in as input for release") | ||
flag.Parse() | ||
v, _ := semver.Parse(*baseVersion) | ||
releaseVersion := fmt.Sprintf("%d.%d.%d", v.Major, v.Minor-1, v.Patch) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ooooh! So we missed a small part of the requirements in the ticket:
(replace . with -)
But we can do this here! Use -
rather than .
, and we are good to go!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated it.
With the notes above - you should be able to deploy a 1-27-0 service on App Engine - which would be neat! |
7cc7ff0
to
78d0dc9
Compare
Build Failed 😱 Build Id: bd7dcbe7-b1b2-42e7-94dd-33270eb29c1c To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: 59f039ff-b9cf-48fc-867a-921887e6e065 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:
|
This looks good! Only thing extra to add, would be to update https://github.com/googleforgames/agones/blob/main/docs/governance/templates/release_issue.md to use this process instead! |
I will update it. |
ffda96a
to
d200862
Compare
Build Failed 😱 Build Id: 6c1471bc-8a95-4720-8d8e-cc9efc759ef6 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
cc81e4e
to
f990245
Compare
Build Failed 😱 Build Id: 64aea425-271b-4fea-9603-3a07c1c53338 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: e1d9d635-5483-4ce4-b737-f1decd57e60f 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:
|
Build Succeeded 👏 Build Id: 86127a99-4cec-4deb-b421-5ad9e1c70606 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:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! just missing a licence, and it's good to go 👍🏻
@@ -0,0 +1,5 @@ | |||
module github.com/agones/agones/build/scripts/previousversion | |||
|
|||
go 1.19 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! I think this is cleaner, what do you think?
929592e
to
5433101
Compare
Build Failed 😱 Build Id: 92440934-a724-4289-aab7-a5527a223a2c To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: db1d8292-ceff-402b-8c82-62dd86a1cef6 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:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good!
If you feel confident, we can merge this now, so it's ready for you to use during the 1.28.0 stable release. (will need to move it out of draft).
Also, thanks for squashing all the commits! 😃
Build Succeeded 👏 Build Id: 1e893758-9cfe-4f5b-9559-c972e35436be 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:
|
Build Succeeded 👏 Build Id: 03f05599-bb49-4a52-a894-15647087cc1b 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:
|
Yes, I feel confident. Was just thinking of taking it together in the next RC and release together. And may be merge it after tomorrow's release. What's your take? |
02eef3f
to
e026b82
Compare
Added GOFLAGS argument and Apache header license Fixes for version data. initial commit for automating version testing version changes testing version changes testing version changes testing version changes issue-2457 changes issue-2457 release version automation automate version Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Debug automation using docker Debug automation using docker Debug automation using docker Debug automation using docker Debug automation using docker Debug automation using docker Debug automation using docker lint error fixed review changes Resolved Merge conflicts updated go mod file Added GOFLAGS argument and Apache header license Added GOFLAGS argument and Apache header license Added GOFLAGS argument and Apache header license Added GOFLAGS argument and Apache header license Fixes for version data. testing version changes testing version changes testing version changes testing version changes issue-2457 changes issue-2457 release version automation automate version Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Testing automation in docker Debug automation using docker Debug automation using docker Debug automation using docker Debug automation using docker Debug automation using docker Debug automation using docker Debug automation using docker lint error fixed review changes Resolved Merge conflicts updated go mod file Added GOFLAGS argument and Apache header license Added GOFLAGS argument and Apache header license initial commit for automating version
e026b82
to
03c17ef
Compare
Build Failed 😱 Build Id: 50f43740-426e-40f8-9a6a-5fe5495679ff To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Failed 😱 Build Id: 6699c4bf-5315-4b8d-9a4c-c020908069f7 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: 2247c6fa-6c26-4126-aa16-769d8a53aa27 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:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved.
@mangalpalli can you move this out of draft, and we can merge it in 👍🏻
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mangalpalli, markmandel 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 |
What type of PR is this?
kind feature
What this PR does / Why we need it:
We need to have a make target to push a new version of the website during a release to capture a snapshot before we make changes. But we didn't have one that created the proper version string for the release and required to change the version string manually.
Which issue(s) this PR fixes:
We created a make target that can generate and format the proper version string for the release.
Closes #2457
Special notes for your reviewer: