Skip to content

[8.4] update ironbank image product name (backport #8959)#8961

Merged
axw merged 1 commit into8.4from
mergify/bp/8.4/pr-8959
Aug 25, 2022
Merged

[8.4] update ironbank image product name (backport #8959)#8961
axw merged 1 commit into8.4from
mergify/bp/8.4/pr-8959

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Aug 25, 2022

This is an automatic backport of pull request #8959 done by Mergify.

Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

@jmlrt @axw @mergify
update ironbank image product name (#8959)
a2f578c 
This is required to automate the creation of the ironbank merge requests as the ubireleaser is using this field to compute the apm-server artifact url. For example it is now trying to retrieve https://artifacts.elastic.co/downloads/beats/apm-server-8.4.0-linux-x86_64.tar.gz instead of https://artifacts.elastic.co/downloads/apm-server/apm-server-8.4.0-linux-x86_64.tar.gz

Co-authored-by: Andrew Wilkins <axw@elastic.co>
(cherry picked from commit e751054)
@mergify mergify bot assigned jmlrt Aug 25, 2022
@ghost
Copy link
Copy Markdown

ghost commented Aug 25, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-08-25T01:27:58.430+0000

  • Duration: 26 min 33 sec

Test stats 🧪

Test Results
Failed 0
Passed 137
Skipped 1
Total 138

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • /package : Generate and publish the docker images.

  • /test windows : Build & tests on Windows.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@ghost
Copy link
Copy Markdown

ghost commented Aug 25, 2022

📚 Go benchmark report

Diff with the 8.4 branch

name                                                            old time/op    new time/op    delta
pkg:github.com/elastic/apm-server/internal/agentcfg goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/internal/decoder goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/internal/model/modelindexer goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/internal/processor/stream goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/internal/publish goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/aggregation/spanmetrics goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/aggregation/txmetrics goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/sampling goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/sampling/eventstorage goos:linux goarch:amd64
ReadEvents/json_codec/0_events-12                                  317ns ± 3%     309ns ± 1%  -2.50%  (p=0.032 n=5+5)
ReadEvents/json_codec/199_events-12                               1.35ms ± 1%    1.34ms ± 1%  -1.22%  (p=0.032 n=5+5)
IsTraceSampled/unsampled-12                                       78.7ns ± 1%    77.5ns ± 1%  -1.54%  (p=0.032 n=5+5)

name                                                            old alloc/op   new alloc/op   delta
pkg:github.com/elastic/apm-server/internal/agentcfg goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/internal/decoder goos:linux goarch:amd64
CompressedRequestReader/gzip_content_encoding-12                  45.6kB ± 0%    45.6kB ± 0%  -0.12%  (p=0.032 n=4+5)
pkg:github.com/elastic/apm-server/internal/model/modelindexer goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/internal/processor/stream goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/internal/publish goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/aggregation/spanmetrics goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/aggregation/txmetrics goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/sampling goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/sampling/eventstorage goos:linux goarch:amd64
WriteTransaction/json_codec_big_tx-12                             3.52kB ± 0%    3.52kB ± 0%  +0.03%  (p=0.029 n=4+4)

name                                                            old allocs/op  new allocs/op  delta
pkg:github.com/elastic/apm-server/internal/agentcfg goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/internal/decoder goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/internal/model/modelindexer goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/internal/processor/stream goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/internal/publish goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/aggregation/spanmetrics goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/aggregation/txmetrics goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/sampling goos:linux goarch:amd64
pkg:github.com/elastic/apm-server/x-pack/apm-server/sampling/eventstorage goos:linux goarch:amd64

report generated with https://pkg.go.dev/golang.org/x/perf/cmd/benchstat

@axw axw merged commit 9deaec4 into 8.4 Aug 25, 2022
@axw axw deleted the mergify/bp/8.4/pr-8959 branch August 25, 2022 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@jmlrt jmlrt jmlrt approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@jmlrt jmlrt

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jmlrt @axw