-
Notifications
You must be signed in to change notification settings - Fork 820
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 1.7.0-rc #1662
Release 1.7.0-rc #1662
Conversation
Build Failed 😱 Build Id: 366c0948-c881-4f18-841f-c40e635db565 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
10min timeout on Unit tests. Dug through and found the following tests in the stacktrace. Just looking for patterns long term, A successful pass is: 00:03:16, so a longer timeout isn't required.
|
Build Failed 😱 Build Id: c13e9ecb-52d2-4f6a-a4cb-440068600033 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
@@ -1343,7 +1343,7 @@ spec: | |||
serviceAccountName: agones-controller | |||
containers: | |||
- name: agones-controller | |||
image: "gcr.io/agones-images/agones-controller:1.7.0" | |||
image: "gcr.io/agones-images/agones-controller:1.7.0-rc" |
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.
Instead of changing this to 1.8.0 after the 1.7 release, would it be better to use 1.8.0-dev (or something)? Then the RC CL would change dev -> rc and the release would drop the extension. WDYT?
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.
I'd be happy with that. Just need to tweak the release checklist to include the process.
[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 |
Build Failed 😱 Build Id: 20ab8922-8244-4aaa-bf24-9a5060b071a5 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Same unit test freeze. Different spot???
|
Build Failed 😱 Build Id: 870dea81-e286-40c2-b18b-ec8975572245 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Different flake
Nope, that was a rate limit that retry fixed. Nevermind. |
Build Succeeded 👏 Build Id: 5df714e6-55cf-4980-b085-b2b0c06f4daa 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 release
What this PR does / Why we need it:
Release of 1.7.0 Release candidate
Which issue(s) this PR fixes:
None.
Special notes for your reviewer:
None.