-
Notifications
You must be signed in to change notification settings - Fork 955
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cherry pick for 0.8.5 #2785
Cherry pick for 0.8.5 #2785
Conversation
* fix worker cache when set option Signed-off-by: zwwhdls <[email protected]> * update changelog in chart Signed-off-by: zwwhdls <[email protected]> * fix unittest Signed-off-by: zwwhdls <[email protected]> --------- Signed-off-by: zwwhdls <[email protected]> Signed-off-by: cheyang <[email protected]>
* fix multi cache dir Signed-off-by: zwwhdls <[email protected]> * fix unit test --------- Signed-off-by: zwwhdls <[email protected]> Signed-off-by: cheyang <[email protected]>
…unt points (fluid-cloudnative#2703) * Clean up broken mount point when NodeStageVolume Signed-off-by: dongyun.xzh <[email protected]> * Check mount point aliveness when NodePublishVolume Signed-off-by: dongyun.xzh <[email protected]> * Clean up broken mount point when NodeStageVolume Signed-off-by: dongyun.xzh <[email protected]> * Fix cleaning logic Signed-off-by: dongyun.xzh <[email protected]> --------- Signed-off-by: dongyun.xzh <[email protected]> Signed-off-by: cheyang <[email protected]>
…#2718) * Prettify error log message when calling NodePublishVolume Signed-off-by: dongyun.xzh <[email protected]> * Prettify error logs when calling helm-related funcs Signed-off-by: dongyun.xzh <[email protected]> * Use instead `errors.As` Signed-off-by: dongyun.xzh <[email protected]> * Use instead `errors.As` Signed-off-by: dongyun.xzh <[email protected]> * Set higher log level for helm exec Signed-off-by: dongyun.xzh <[email protected]> --------- Signed-off-by: dongyun.xzh <[email protected]> Signed-off-by: cheyang <[email protected]>
Signed-off-by: cheyang <[email protected]>
b2c7323
to
ef547e2
Compare
You have successfully added a new SonarCloud configuration ``. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab. |
Signed-off-by: cheyang <[email protected]>
Signed-off-by: cheyang <[email protected]>
Kudos, SonarCloud Quality Gate passed! |
/test fluid-e2e-nightly |
/test fluid-e2e-080 |
2 similar comments
/test fluid-e2e-080 |
/test fluid-e2e-080 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/override fluid-e2e-nightly
/lgtm
@TrafalgarZZZ: Overrode contexts on behalf of TrafalgarZZZ: fluid-e2e-nightly In response to this:
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/test-infra repository. |
/approve |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: TrafalgarZZZ The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?
fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews