build(deps): bump go.uber.org/goleak from 1.2.1 to 1.3.0 #48262

dependabot · 2023-11-03T04:32:08Z

Bumps go.uber.org/goleak from 1.2.1 to 1.3.0.

Release notes

Sourced from go.uber.org/goleak's releases.

v1.3.0

Fixed

Built-in ignores now match function names more accurately. They will no longer ignore stacks because of file names that look similar to function names. (#112)

Added

Add an IgnoreAnyFunction option to ignore stack traces that have the provided function anywhere in the stack. (#113)

Ignore testing.runFuzzing and testing.runFuzzTests alongside other already-ignored test functions (testing.RunTests, etc). (#105)

Changed

Miscellaneous CI-related fixes. (#103, #108, #114)

Changelog

Sourced from go.uber.org/goleak's changelog.

1.3.0

Fixed

Built-in ignores now match function names more accurately. They will no longer ignore stacks because of file names that look similar to function names. (#112)

Added

Add an IgnoreAnyFunction option to ignore stack traces that have the provided function anywhere in the stack. (#113)

Ignore testing.runFuzzing and testing.runFuzzTests alongside other already-ignored test functions (testing.RunTests, etc). (#105)

Changed

Miscellaneous CI-related fixes. (#103, #108, #114)

Commits

31095c6 Release v1.3.0 (#115)
5643445 Revert "Make maxSleep and maxRetires configurable when building options (#94)...
7b4998f options: Add IgnoreAnyFunction (#113)
66916c2 README: Fix build status badge (#114)
ecabcf9 ignores: Don't use strings.Contains (#112)
91de685 stack: Parse all functions (#111)
25cbb67 internal/stack: Use control flow for state (#110)
f995fdb ci: Use golangci-lint for linting (#108)
03cb6e6 Document incompatibility of VerifyNone with t.Parallel (#107)
3e8744f Ignore testing.runFuzzing and testing.runFuzzTests (#105)
Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

@dependabot rebase will rebase this PR
@dependabot recreate will recreate this PR, overwriting any edits that have been made to it
@dependabot merge will merge this PR after your CI passes on it
@dependabot squash and merge will squash and merge this PR after your CI passes on it
@dependabot cancel merge will cancel a previously requested merge and block automerging
@dependabot reopen will reopen this PR if it is closed
@dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
@dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
@dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
@dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
@dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

What problem does this PR solve?

Issue Number: close None
Problem Summary:

What is changed and how it works?

Check List

Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No need to test
- I checked and no code files have been changed.

Side effects

Performance regression: Consumes more CPU
Performance regression: Consumes more Memory
Breaking backward compatibility

Documentation

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.

None

Bumps [go.uber.org/goleak](https://github.com/uber-go/goleak) from 1.2.1 to 1.3.0. - [Release notes](https://github.com/uber-go/goleak/releases) - [Changelog](https://github.com/uber-go/goleak/blob/master/CHANGELOG.md) - [Commits](uber-go/goleak@v1.2.1...v1.3.0) --- updated-dependencies: - dependency-name: go.uber.org/goleak dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>

ti-chi-bot · 2023-11-03T04:32:18Z

Hi @dependabot[bot]. Thanks for your PR.

I'm waiting for a pingcap member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

tiprow · 2023-11-03T04:32:23Z

Hi @dependabot[bot]. Thanks for your PR.

PRs from untrusted users cannot be marked as trusted with /ok-to-test in this repo meaning untrusted PR authors can never trigger tests themselves. Collaborators can still trigger tests on the PR using /test all.

I understand the commands that are listed here.

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.

Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>

hawkingrei · 2023-11-03T05:33:05Z

/ok-to-test

codecov · 2023-11-03T05:49:26Z

Codecov Report

Merging #48262 (a72c66e) into master (035ad5c) will increase coverage by 1.3709%.
Report is 3 commits behind head on master.
The diff coverage is n/a.

Additional details and impacted files

@@               Coverage Diff                @@
##             master     #48262        +/-   ##
================================================
+ Coverage   71.4980%   72.8690%   +1.3709%     
================================================
  Files          1402       1425        +23     
  Lines        406253     413251      +6998     
================================================
+ Hits         290463     301132     +10669     
+ Misses        95970      93181      -2789     
+ Partials      19820      18938       -882

Flag	Coverage Δ
integration	`42.8773% <ø> (?)`
unit	`71.4902% <ø> (-0.0079%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
dumpling	`53.9874% <ø> (ø)`
parser	`∅ <ø> (∅)`
br	`48.6978% <ø> (-4.2180%)`	⬇️

ti-chi-bot · 2023-11-03T06:28:25Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Defined2014, hawkingrei

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:

~~OWNERS~~ [Defined2014,hawkingrei]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

ti-chi-bot · 2023-11-03T06:28:29Z

[LGTM Timeline notifier]

Timeline:

2023-11-03 05:33:25.481665468 +0000 UTC m=+3190403.068775597: ☑️ agreed by hawkingrei.
2023-11-03 06:28:28.403345961 +0000 UTC m=+3193705.990456106: ☑️ agreed by Defined2014.

hawkingrei · 2023-11-03T06:35:39Z

/retest

dependabot bot added the dependencies Pull requests that update a dependency file label Nov 3, 2023

ti-chi-bot bot added do-not-merge/needs-linked-issue do-not-merge/needs-tests-checked do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Nov 3, 2023

ti-chi-bot bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. needs-ok-to-test labels Nov 3, 2023

update

a72c66e

Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>

hawkingrei added the skip-issue-check label Nov 3, 2023

ti-chi-bot bot removed the do-not-merge/needs-linked-issue label Nov 3, 2023

ti-chi-bot bot added ok-to-test and removed needs-ok-to-test labels Nov 3, 2023

hawkingrei approved these changes Nov 3, 2023

View reviewed changes

ti-chi-bot bot added approved needs-1-more-lgtm labels Nov 3, 2023

hawkingrei requested a review from Defined2014 November 3, 2023 05:46

Defined2014 approved these changes Nov 3, 2023

View reviewed changes

ti-chi-bot bot added lgtm and removed needs-1-more-lgtm labels Nov 3, 2023

ti-chi-bot bot merged commit 4061df3 into master Nov 3, 2023
14 of 16 checks passed

ti-chi-bot bot deleted the dependabot/go_modules/go.uber.org/goleak-1.3.0 branch November 3, 2023 07:48

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

build(deps): bump go.uber.org/goleak from 1.2.1 to 1.3.0 #48262

build(deps): bump go.uber.org/goleak from 1.2.1 to 1.3.0 #48262

dependabot bot commented on behalf of github Nov 3, 2023 •

edited by hawkingrei

ti-chi-bot bot commented Nov 3, 2023

tiprow bot commented Nov 3, 2023

hawkingrei commented Nov 3, 2023

codecov bot commented Nov 3, 2023 •

edited

ti-chi-bot bot commented Nov 3, 2023

ti-chi-bot bot commented Nov 3, 2023

hawkingrei commented Nov 3, 2023

build(deps): bump go.uber.org/goleak from 1.2.1 to 1.3.0 #48262

build(deps): bump go.uber.org/goleak from 1.2.1 to 1.3.0 #48262

Conversation

dependabot bot commented on behalf of github Nov 3, 2023 • edited by hawkingrei

v1.3.0

Fixed

Added

Changed

1.3.0

Fixed

Added

Changed

What problem does this PR solve?

What is changed and how it works?

Check List

Release note

ti-chi-bot bot commented Nov 3, 2023

tiprow bot commented Nov 3, 2023

hawkingrei commented Nov 3, 2023

codecov bot commented Nov 3, 2023 • edited

Codecov Report

ti-chi-bot bot commented Nov 3, 2023

ti-chi-bot bot commented Nov 3, 2023

[LGTM Timeline notifier]

hawkingrei commented Nov 3, 2023

dependabot bot commented on behalf of github Nov 3, 2023 •

edited by hawkingrei

codecov bot commented Nov 3, 2023 •

edited