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

Add hack/release.sh to do releasing automatically #34

Merged
merged 1 commit into from
Jan 10, 2019

Conversation

cofyc
Copy link
Member

@cofyc cofyc commented Jan 10, 2019

No description provided.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jan 10, 2019
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jan 10, 2019
@cofyc cofyc force-pushed the release branch 2 times, most recently from e379c84 to 9b4f506 Compare January 10, 2019 12:44
@cofyc
Copy link
Member Author

cofyc commented Jan 10, 2019

@msau42
This is a handy script to do releasing.

docker tag v2.3.0
./hack/release.sh

will build and release quay.io/cofyc/local-volume-provisioner:latest and quay.io/cofyc/local-volume-provisioner:v2.3.0.

I suggest to do stable version releasing manually for now, because even if we configured a prow job to build/push image automatically, we need to check later again to make sure image are built and pushed successfully. If failed because of temporary issues (e.g. network), we need to delete/create tag again to trigger prow job to rerun.

One thing I think it is useful is to automate unstable/canary image build on master branch.

What do you think?

echo "info: building is skipped"
fi

echo "info: pushing $image"
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add a check somewhere here that if a stable image has already been pushed, we should not override it?

Copy link
Member Author

Choose a reason for hiding this comment

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

@msau42
Copy link
Contributor

msau42 commented Jan 10, 2019

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 10, 2019
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cofyc, msau42

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit d149a95 into kubernetes-sigs:master Jan 10, 2019

if ! is_stable_version "$VERSION"; then
echo "error: VERSION '$VERSION' is not stable version, skipped pushing as latest image"
exit 1
Copy link
Contributor

Choose a reason for hiding this comment

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

do we want to exit with error here? will this cause canary build jobs to fail?

Copy link
Member Author

Choose a reason for hiding this comment

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

You are right. Job will fail, I will update it when configuring job in prow.

@cofyc cofyc deleted the release branch April 28, 2019 06:56
davidmccormick pushed a commit to davidmccormick/sig-storage-local-static-provisioner that referenced this pull request Mar 31, 2021
Add hack/release.sh to do releasing automatically
dobsonj pushed a commit to dobsonj/sig-storage-local-static-provisioner that referenced this pull request Feb 7, 2022
…ncy-openshift-4.10-local-storage-static-provisioner

Updating local-storage-static-provisioner images to be consistent with ART
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants