8000 chore: enable automatic approval of dependabot PRs by willmurphyscode · Pull Request #213 · anchore/stereoscope · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore: enable automatic approval of dependabot PRs #213

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

Merged
merged 1 commit into from
Jan 18, 2024

Conversation

willmurphyscode
Copy link
Contributor

To reduce toil in this repo, enable dependabot PRs to be automatically approved, but not merged. They are not automatically merged because if the default GitHub token is used to automatically merge a PR, the resulting commit will not trigger workflows on main. Rather than generate a more potent token, just automatically review them, which reduces toil by eliminating several clicks and page loads for maintainers who are trying to merge dependabot PRs.

Copied from anchore/grype-db#222.

To reduce toil in this repo, enable dependabot PRs to be automatically
approved, but not merged. They are not automatically merged because if
the default GitHub token is used to automatically merge a PR, the
resulting commit will not trigger workflows on main. Rather than
generate a more potent token, just automatically review them, which
reduces toil by eliminating several clicks and page loads for
maintainers who are trying to merge dependabot PRs.

Signed-off-by: Will Murphy <will.murphy@anchore.com>
Copy link

Benchmark Test Results

Benchmark results from the latest changes vs base branch
latest: Pulling from library/ubuntu
tar: Option --mtime: Treating date 'UTC 2019-09-16' as 2019-09-16 00:00:00
goos: linux
goarch: amd64
pkg: github.com/anchore/stereoscope/pkg/file
cpu: AMD EPYC 7763 64-Core Processor                
ctr: 
           │ ./.tmp/benchmark-6a62d45.txt │
           │            sec/op            │
TarIndex-2                   34.50µ ± ∞ ¹
¹ need >= 6 samples for confidence interval at level 0.95

           │ ./.tmp/benchmark-6a62d45.txt │
           │             B/op             │
TarIndex-2                  5.560Ki ± ∞ ¹
¹ need >= 6 samples for confidence interval at level 0.95

           │ ./.tmp/benchmark-6a62d45.txt │
           │          allocs/op           │
TarIndex-2                    93.00 ± ∞ ¹
¹ need >= 6 samples for confidence interval at level 0.95

pkg: github.com/anchore/stereoscope/test/integration
                                      │ ./.tmp/benchmark-6a62d45.txt │
                                      │            sec/op            │
SimpleImage_GetImage/docker-archive-2                   1.210m ± ∞ ¹
SimpleImage_GetImage/podman-2                           20.99m ± ∞ ¹
geomean                                                 5.041m
¹ need >= 6 samples for confidence interval at level 0.95

                                      │ ./.tmp/benchmark-6a62d45.txt │
                                      │             B/op             │
SimpleImage_GetImage/docker-archive-2                  325.2Ki ± ∞ ¹
SimpleImage_GetImage/podman-2                          435.2Ki ± ∞ ¹
geomean                                                376.2Ki
¹ need >= 6 samples for confidence interval at level 0.95

                                      │ ./.tmp/benchmark-6a62d45.txt │
                                      │          allocs/op           │
SimpleImage_GetImage/docker-archive-2                   2.659k ± ∞ ¹
SimpleImage_GetImage/podman-2                           2.754k ± ∞ ¹
geomean                                                 2.706k
¹ need >= 6 samples for confidence interval at level 0.95

ctr: failed to dial "/run/containerd/containerd.sock": connection error: desc = "transport: error while dialing: dial unix /run/containerd/containerd.sock: connect: permission denied"
                                                   │ ./.tmp/benchmark-6a62d45.txt │
                                                   │            sec/op            │
SimpleImage_FetchSquashedContents/docker-archive-2                   17.94µ ± ∞ ¹
SimpleImage_FetchSquashedContents/podman-2                           17.90µ ± ∞ ¹
geomean                                                              17.92µ
¹ need >= 6 samples for confidence interval at level 0.95

                                                   │ ./.tmp/benchmark-6a62d45.txt │
                                                   │             B/op             │
SimpleImage_FetchSquashedContents/docker-archive-2                  2.648Ki ± ∞ ¹
SimpleImage_FetchSquashedContents/podman-2                          2.648Ki ± ∞ ¹
geomean                                                             2.648Ki
¹ need >= 6 samples for confidence interval at level 0.95

                                                   │ ./.tmp/benchmark-6a62d45.txt │
                                                   │          allocs/op           │
SimpleImage_FetchSquashedContents/docker-archive-2                    21.00 ± ∞ ¹
SimpleImage_FetchSquashedContents/podman-2                            21.00 ± ∞ ¹
geomean                                                               21.00
¹ need >= 6 samples for confidence interval at level 0.95

@willmurphyscode willmurphyscode merged commit eb656fc into main Jan 18, 2024
@willmurphyscode willmurphyscode deleted the chore/dependabot-automation branch January 18, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0