-
Notifications
You must be signed in to change notification settings - Fork 813
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
Update release checklist #3334
Update release checklist #3334
Conversation
Build Succeeded 👏 Build Id: fac3f538-3d13-46bc-aa60-95c00e4dfe66 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: 2b29c565-f4b0-4bed-ac56-76ec388aec3e 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:
|
I will follow the same approach for the helm commands that you mentioned in the other PR to create a .sh file for the Rust SDK. |
Build Succeeded 👏 Build Id: 89912b34-b8b1-42f3-bae2-bf1d859476f1 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: 7d3ece2a-2163-4a93-8f5a-8fc51c9bc766 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:
|
@markmandel, I've included a shell script and a target to run the necessary If any of the helm commands fail during the release process, the failed and remaining steps will be handled manually. Does this sound good? |
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 some ideas on naming and some possible cleanup 👍🏻
build/Makefile
Outdated
@@ -856,3 +856,10 @@ test-gen-crd-client: | |||
echo "$$diff_output"; \ | |||
fi | |||
|
|||
update-agones: $(ensure-build-image) |
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.
update-agones: $(ensure-build-image) | |
install-release: $(ensure-build-image) |
Maybe, as a better name?
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.
modified here and other locations as well
build/Makefile
Outdated
docker run -it --rm \ | ||
$(common_mounts) \ | ||
-w $(workdir_path) \ | ||
-e "KUBECONFIG=/root/.kube/$(kubeconfig_file)" \ |
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.
-e "KUBECONFIG=/root/.kube/$(kubeconfig_file)" \ |
Is this line required? I feel like it should be where it looks by default 🤔
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.
excluded this line
build/Makefile
Outdated
update-agones: $(ensure-build-image) | ||
docker run -it --rm \ | ||
$(common_mounts) \ | ||
-w $(workdir_path) \ |
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.
-w $(workdir_path) \ |
Given that this is all helm commands - probably doesn't need to be in a specific work dir?
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.
removed this line
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! Need to move it out of draft
though please 😄
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Kalaiselvi84, 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 |
Build Succeeded 👏 Build Id: 27290b95-99b1-4b65-b82d-1a6b38e6c62d 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:
|
New changes are detected. LGTM label has been removed. |
Build Failed 😱 Build Id: e8c63aa3-9d06-4742-a695-790fdfd78286 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: b41aeba1-2410-4e1b-af0d-391eb8e9804e 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/Makefile
Outdated
docker run -it --rm \ | ||
$(common_mounts) \ | ||
$(DOCKER_RUN_ARGS) \ | ||
$(build_tag) /go/src/agones.dev/agones/build/install-release.sh |
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.
Can you add a new blank line here?
Build Succeeded 👏 Build Id: df73b796-868e-45cc-a963-457a29017d8e 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: f3eed2fc-cde5-43ab-81a0-6b675329e0df 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: 6234402d-11c1-42ee-9e4b-6e540e9fc530 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 cleanup
What this PR does / Why we need it:
Which issue(s) this PR fixes:
Closes #
Special notes for your reviewer: