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

Update deploy to build canary images for 1.0 #347

Merged
merged 2 commits into from
May 28, 2019

Conversation

ShyamsundarR
Copy link
Contributor

@ShyamsundarR ShyamsundarR commented May 3, 2019

and hence stop updating the same 1.0.0 image for every PR.

At some point we would need to decide how to release, 1.0.1.

Signed-off-by: ShyamsundarR srangana@redhat.com

@ShyamsundarR ShyamsundarR added the DNM DO NOT MERGE label May 3, 2019
@ShyamsundarR ShyamsundarR requested review from Madhu-1, leseb, rootfs and gman0 May 3, 2019 00:53
@ShyamsundarR
Copy link
Contributor Author

Should be merged once #346 is merged, which would be the final update for 1.0.0 image version.

@ShyamsundarR
Copy link
Contributor Author

@phlogistonjohn PTAL, thanks.

@humblec
Copy link
Collaborator

humblec commented May 7, 2019

@ShyamsundarR do we really need this PR now ? why I am asking is, we are going or planning to have a 1.0.x release soon. More or less, we are not planning to take any more PRs in v1.0 branch. @Madhu-1 what do you think ?

deploy.sh Outdated
elif [ "${TRAVIS_BRANCH}" == 'csi-v1.0' ]; then
export RBD_IMAGE_VERSION='v1.0.0'
export CEPHFS_IMAGE_VERSION='v1.0.0'
export ENV_RBD_IMAGE_VERSION='v1.0.1-canary'
Copy link
Collaborator

Choose a reason for hiding this comment

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

AFAIK we agreed to have only canary images tag for csi-v1.0 and also please send a PR to master branch

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We need to stop publishing images from 1.0 branch, what is the strategy for doing that?

Also please update these issues with the same, for users on these branches, #296 #345

The intention here is to have rolling images, such that users can stick to a particular version of the image rather than have the same image updated.

Your strategy seems to be to call master 1.0 and release canary from the same. I think we may need to inform users of this change and proceed, at present there is confusion on the messaging.

@ShyamsundarR
Copy link
Contributor Author

Updated canary tag to 1.0-canary rather than 1.0.1-canary

@ShyamsundarR ShyamsundarR changed the title Update deploy to build canary images for 1.0.1 Update deploy to build canary images for 1.0 May 9, 2019
and hence stop updating the same 1.0.0 image for every PR.

At some point we would need to decide how to release, 1.0.1.

Signed-off-by: ShyamsundarR <srangana@redhat.com>
@ShyamsundarR
Copy link
Contributor Author

Should be merged once #346 is merged, which would be the final update for 1.0.0 image version.

Removing DNM as the above is now merged.

@ShyamsundarR ShyamsundarR removed the DNM DO NOT MERGE label May 28, 2019
@ShyamsundarR
Copy link
Contributor Author

@Madhu-1 and @humblec PTAL

@mergify mergify bot merged commit 2623bce into ceph:csi-v1.0 May 28, 2019
@ShyamsundarR ShyamsundarR deleted the stop-image-update branch June 12, 2019 19:05
Nikhil-Ladha pushed a commit to Nikhil-Ladha/ceph-csi that referenced this pull request Aug 8, 2024
Syncing latest changes from upstream devel for ceph-csi
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants