Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update images #234

Merged
merged 1 commit into from
Mar 6, 2020
Merged

Conversation

xing-yang
Copy link
Collaborator

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
This PR updates images.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. release-note-none Denotes a PR that doesn't merit a release note. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jan 7, 2020
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: xing-yang

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 added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 7, 2020
@yuxiangqian
Copy link
Contributor

is this ready? @xing-yang

@yuxiangqian
Copy link
Contributor

/assign @yuxiangqian

@xing-yang
Copy link
Collaborator Author

I'll need to update the image to 2.0.1.

@xing-yang xing-yang force-pushed the update_images_v2.0.0 branch from fbabdb2 to 2bde75f Compare March 5, 2020 14:33
@xing-yang
Copy link
Collaborator Author

@yuxiangqian updated the image to 2.0.1. PTAL.

@xing-yang xing-yang changed the title WIP: Update images Update images Mar 5, 2020
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 5, 2020
@xing-yang xing-yang force-pushed the update_images_v2.0.0 branch from 2bde75f to 5d72243 Compare March 5, 2020 14:50
@@ -72,7 +72,7 @@ spec:
serviceAccount: csi-snapshotter
containers:
- name: csi-provisioner
image: quay.io/k8scsi/csi-provisioner:v1.5.0-rc1
image: quay.io/k8scsi/csi-provisioner:v1.5.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

In the other repos, we use "latest" tag so we don't need to remember to update it every release

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure. I want to backport this to release-2.0 branch. Do you suggest to submit a PR directly to the release-2.0 branch?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I guess this is a strange example because in other sidecar repos, the deployment is not a complete/workable spec and the expectation is to go to the hostpath driver repo to look at a real example with fixed versions

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We want to provide an example here so people can test create snapshot easily without deploying a full blown setup as in the hostpath driver repo. I can add a note here for people to look at that repo for full testing of all features.

@msau42
Copy link
Collaborator

msau42 commented Mar 6, 2020

This is fine but I'm concerned about future maintenance burden to remember to update this every single release.
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 6, 2020
@k8s-ci-robot k8s-ci-robot merged commit 2890f4a into kubernetes-csi:master Mar 6, 2020
Sneha-at added a commit to Sneha-at/external-snapshotter that referenced this pull request Sep 7, 2023
984feec Merge pull request kubernetes-csi#234 from siddhikhapare/csi-tools
1f7e605 fixed broken links of testgrid dashboard

git-subtree-dir: release-tools
git-subtree-split: 984feec
humblec added a commit to humblec/external-snapshotter that referenced this pull request Sep 8, 2023
984feec Merge pull request kubernetes-csi#234 from siddhikhapare/csi-tools
1f7e605 fixed broken links of testgrid dashboard

git-subtree-dir: release-tools
git-subtree-split: 984feec
humblec added a commit to humblec/external-snapshotter that referenced this pull request Oct 21, 2023
984feec Merge pull request kubernetes-csi#234 from siddhikhapare/csi-tools
1f7e605 fixed broken links of testgrid dashboard

git-subtree-dir: release-tools
git-subtree-split: 984feec
andyzhangx pushed a commit to andyzhangx/external-snapshotter that referenced this pull request Jun 5, 2024
fixed broken links of testgrid dashboard
# for free to join this conversation on GitHub. Already have an account? # 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. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants