Skip to content

Yc csi driver large volume support#6

Merged
ulrihaa merged 6 commits intomasterfrom
yc-csi-driver-large-volume-support
Jan 21, 2026
Merged

Yc csi driver large volume support#6
ulrihaa merged 6 commits intomasterfrom
yc-csi-driver-large-volume-support

Conversation

@ulrihaa
Copy link
Copy Markdown
Contributor

@ulrihaa ulrihaa commented Dec 9, 2025

What this PR does?
Adds support for creating disks with a non-standard block size

How was it tested?
Manually assembling and replacing the image link in containers. Next, create disks with standard and non-standard block sizes.

Adding support for non-standard disk blocks
Creating a disk with a non-standard block size
Comment thread .github/workflows/test.yaml Outdated

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium test

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}

Copilot Autofix

AI 4 months ago

The best way to fix this problem is by explicitly specifying a permissions block restricting the GITHUB_TOKEN to the least privilege required. In this workflow, the steps only need to check repository contents; write access is unnecessary. Thus, the minimal required permissions are contents: read. This can be set at the workflow root (applies to all jobs) or inside the specific job config if more granularity is required. As there is only one job, adding the following at the workflow root after the name: line is optimal (as per convention and examples):

permissions:
  contents: read

Make this addition immediately after the name: Run tests line (line 15), before the on: block.

No imports, methods, or further changes are required.


Suggested changeset 1
.github/workflows/test.yaml

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
--- a/.github/workflows/test.yaml
+++ b/.github/workflows/test.yaml
@@ -13,6 +13,8 @@
 # limitations under the License.
 
 name: Run tests
+permissions:
+  contents: read
 
 on:
   pull_request: {}
EOF
@@ -13,6 +13,8 @@
# limitations under the License.

name: Run tests
permissions:
contents: read

on:
pull_request: {}
Copilot is powered by AI and may make mistakes. Always verify output.
Comment thread .github/workflows/test.yaml
Comment thread pkg/services/controller/controller.go Outdated
Comment thread pkg/services/controller/controller.go
@ulrihaa ulrihaa merged commit d34f2d6 into master Jan 21, 2026
4 checks passed
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