Skip to content

cleanup: refine mount time out error message#1023

Merged
andyzhangx merged 1 commit intokubernetes-csi:masterfrom
andyzhangx:refine-mountTimeout-error
Dec 25, 2025
Merged

cleanup: refine mount time out error message#1023
andyzhangx merged 1 commit intokubernetes-csi:masterfrom
andyzhangx:refine-mountTimeout-error

Conversation

@andyzhangx
Copy link
Copy Markdown
Member

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
cleanup: refine mount time out error message

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 kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. release-note-none Denotes a PR that doesn't merit a release note. labels Dec 25, 2025
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: andyzhangx

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 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. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Dec 25, 2025
@andyzhangx andyzhangx requested review from Copilot and removed request for gnufied and mauriciopoppe December 25, 2025 08:34
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refines the mount timeout error message in the NFS CSI driver to provide more detailed and actionable information when mount operations timeout. The changes improve observability by including the source, target path, and timeout duration in the error message.

Key Changes:

  • Introduced a named constant for the mount timeout value (110 seconds)
  • Enhanced the timeout error message to include source volume, target path, and timeout duration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pkg/nfs/nodeserver.go
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 20502074527

Details

  • 2 of 4 (50.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 78.582%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/nfs/nodeserver.go 2 4 50.0%
Totals Coverage Status
Change from base Build 20361374728: -0.1%
Covered Lines: 1108
Relevant Lines: 1410

💛 - Coveralls

@andyzhangx andyzhangx merged commit 57bf760 into kubernetes-csi:master Dec 25, 2025
23 of 24 checks passed
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. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. 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