Skip to content

Conversation

@jarrpa
Copy link
Contributor

@jarrpa jarrpa commented Oct 15, 2018

This is a relatively naive attempt to implement some functionality I'd like to have for my CSI driver development, namely the ability to specify VolumeParameters for my test volumes.

Signed-off-by: Jose A. Rivera jarrpa@redhat.com

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Oct 15, 2018
@k8s-ci-robot k8s-ci-robot requested review from lpabon and pohly October 15, 2018 20:50
@k8s-ci-robot k8s-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 15, 2018
@jarrpa
Copy link
Contributor Author

jarrpa commented Oct 16, 2018

@pohly @lpabon @saad-ali PTAL

@lpabon
Copy link
Member

lpabon commented Oct 18, 2018

Talked with @jarrpa offline. We will be splitting up the PR into two.

@jarrpa jarrpa changed the title Add volume parameters and roles to sanity tests Add volume parameters to sanity tests Oct 23, 2018
@jarrpa
Copy link
Contributor Author

jarrpa commented Oct 23, 2018

@lpabon Commit and PR updated as discussed. PTAL.

Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
Copy link
Contributor

@pohly pohly left a comment

Choose a reason for hiding this comment

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

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 24, 2018
@jarrpa
Copy link
Contributor Author

jarrpa commented Oct 24, 2018

@pohly @lpabon @saad-ali So how do I get the approved label? :)

@pohly
Copy link
Contributor

pohly commented Oct 25, 2018

@jarrpa I can do that now.

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jarrpa, pohly

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details 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 the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 25, 2018
@k8s-ci-robot k8s-ci-robot merged commit 1213743 into kubernetes-csi:master Oct 25, 2018
suneeth51 pushed a commit to suneeth51/csi-test that referenced this pull request Sep 11, 2019
Add volume parameters to sanity tests
pohly added a commit to pohly/csi-test that referenced this pull request Jan 26, 2021
1d60e77 Merge pull request kubernetes-csi#131 from pohly/kubernetes-1.20-tag
9f10459 prow.sh: support building Kubernetes for a specific version
fe1f284 Merge pull request kubernetes-csi#121 from kvaps/namespace-check
8fdf0f7 Merge pull request kubernetes-csi#128 from fengzixu/master
1c94220 fix: fix a bug of csi-sanity
a4c41e6 Merge pull request kubernetes-csi#127 from pohly/fix-boilerplate
ece0f50 check namespace for snapshot-controller
dbd8967 verify-boilerplate.sh: fix path to script
9289fd1 Merge pull request kubernetes-csi#125 from sachinkumarsingh092/optional-spelling-boilerplate-checks
ad29307 Make the spelling and boilerplate checks optional
5f06d02 Merge pull request kubernetes-csi#124 from sachinkumarsingh092/fix-spellcheck-boilerplate-tests
48186eb Fix spelling and boilerplate errors
71690af Merge pull request kubernetes-csi#122 from sachinkumarsingh092/include-spellcheck-boilerplate-tests
981be3f Adding spelling and boilerplate checks.
2bb7525 Merge pull request kubernetes-csi#117 from fengzixu/master
3b6d17b Merge pull request kubernetes-csi#118 from pohly/cloud-build-timeout
9318c6c cloud build: double the timeout, now 1 hour
4ab8b15 use the tag to replace commit of csi-test
5d74e45 change the csi-test import path to v4
7dcd0a9 upgrade csi-test to v4.0.2
86ff580 Merge pull request kubernetes-csi#116 from andyzhangx/export-image-name
c3a9662 allow export image name and registry name

git-subtree-dir: release-tools
git-subtree-split: 1d60e77
stmcginnis pushed a commit to stmcginnis/csi-test that referenced this pull request Oct 9, 2024
…al-spelling-boilerplate-checks

Make the spelling and boilerplate checks optional
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/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants