Skip to content

Conversation

tinswzy
Copy link
Contributor

@tinswzy tinswzy commented Sep 29, 2025

#43638

Resolve issue accessing GCP Cloud Storage with ak/sk pr#51
upgrade wp v0.1.7

@sre-ci-robot sre-ci-robot added size/S Denotes a PR that changes 10-29 lines. area/dependency Pull requests that update a dependency file labels Sep 29, 2025
@mergify mergify bot added the dco-passed DCO check passed. label Sep 29, 2025
@sre-ci-robot
Copy link
Contributor

[ci-v2-notice]
Notice: We are gradually rolling out the new ci-v2 system.

  • Legacy CI jobs remain unaffected, you can just ignore ci-v2 if you don't want to run it.
  • Additional "ci-v2/*" checkers will run for this PR to ensure the new ci-v2 system is working as expected.
  • For tests that exist in both v1 and v2, passing in either system is considered PASS.

To rerun ci-v2 checks, comment with:

  • /ci-rerun-code-check // for ci-v2/code-check
  • /ci-rerun-build // for ci-v2/build
  • /ci-rerun-ut-integration // for ci-v2/ut-integration
  • /ci-rerun-ut-go // for ci-v2/ut-go
  • /ci-rerun-ut-cpp // for ci-v2/ut-cpp
  • /ci-rerun-ut // for all ci-v2/ut-integration, ci-v2/ut-go, ci-v2/ut-cpp

If you have any questions or requests, please contact @zhikunyao.

@mergify mergify bot added the kind/bug Issues or changes related a bug label Sep 29, 2025
Copy link

codecov bot commented Sep 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.99%. Comparing base (c9f01a7) to head (d6f610a).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #44636   +/-   ##
=======================================
  Coverage   83.99%   83.99%           
=======================================
  Files         504      504           
  Lines       76806    76806           
=======================================
  Hits        64513    64513           
  Misses      12293    12293           
Components Coverage Δ
Client ∅ <ø> (∅)
Core 83.99% <ø> (ø)
Go ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tinswzy
Copy link
Contributor Author

tinswzy commented Sep 29, 2025

/ci-rerun-ut-go

@tinswzy
Copy link
Contributor Author

tinswzy commented Sep 29, 2025

/ci-rerun-ut-integration

Copy link
Collaborator

@czs007 czs007 left a comment

Choose a reason for hiding this comment

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

/lgtm

@sre-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: czs007, tinswzy

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

@tinswzy
Copy link
Contributor Author

tinswzy commented Sep 30, 2025

/ci-rerun-ut-go

1 similar comment
@tinswzy
Copy link
Contributor Author

tinswzy commented Sep 30, 2025

/ci-rerun-ut-go

@tinswzy tinswzy force-pushed the fix_wp_access_gcp_using_aksk branch from b154c34 to d6f610a Compare September 30, 2025 09:03
@sre-ci-robot sre-ci-robot removed the lgtm label Sep 30, 2025
@sre-ci-robot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved area/dependency Pull requests that update a dependency file dco-passed DCO check passed. kind/bug Issues or changes related a bug size/S Denotes a PR that changes 10-29 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants