Skip to content

Conversation

andyzhangx
Copy link
Member

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
cleanup: add more logging in blobfuse-proxy when blobfuse2 not found

Which issue(s) this PR fixes:

Fixes #

Requirements:

Special notes for your reviewer:

Release note:

none

@k8s-ci-robot k8s-ci-robot added 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 Aug 16, 2025
@k8s-ci-robot
Copy link
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

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 requested review from cvvz and feiskyer August 16, 2025 02:35
@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 Aug 16, 2025
@andyzhangx andyzhangx force-pushed the add-logging-blobfuse-proxy branch from 76e3ff1 to 8e7e1ba Compare August 16, 2025 02:56
@andyzhangx andyzhangx requested review from Copilot and removed request for feiskyer August 16, 2025 03:00
Copy link

@Copilot 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 adds enhanced logging to the blobfuse-proxy server to provide more diagnostic information when blobfuse2 executable is not found during mount operations.

Key Changes

  • Enhanced error handling in the MountAzureBlob function to detect "executable file not found" errors
  • Added PATH environment variable and command output to error messages for better debugging

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@andyzhangx
Copy link
Member Author

/retest

@andyzhangx andyzhangx merged commit 69883b2 into kubernetes-sigs:master Aug 17, 2025
21 of 22 checks passed
@andyzhangx
Copy link
Member Author

/cherrypick release-1.26

@k8s-infra-cherrypick-robot

@andyzhangx: new pull request created: #2131

In response to this:

/cherrypick release-1.26

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

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. 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.

3 participants