Skip to content
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

ci: set proper parameters for the tar step #4696

Merged
merged 6 commits into from
Feb 17, 2021

Conversation

kuisathaverat
Copy link
Contributor

The archive of the system-tests file is falling for a while, checking the tar step is using a parameter that is deprecated, because of that does not find the files to tar.

@kuisathaverat kuisathaverat added bug ci automation Team:Automation Label for the Observability productivity team labels Feb 8, 2021
@kuisathaverat kuisathaverat requested review from a team February 8, 2021 12:08
@kuisathaverat kuisathaverat self-assigned this Feb 8, 2021
@apmmachine
Copy link
Contributor

apmmachine commented Feb 8, 2021

💚 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

Expand to view the summary

Build stats

  • Build Cause: Branch indexing

  • Start Time: 2021-02-17T02:50:23.562+0000

  • Duration: 39 min 5 sec

  • Commit: 07b690a

Test stats 🧪

Test Results
Failed 0
Passed 4496
Skipped 123
Total 4619

Trends 🧪

Image of Build Times

Image of Tests

Steps errors 1

Expand to view the steps failures

Test Sync
  • Took 3 min 24 sec . View more details on here
  • Description: ./.ci/scripts/sync.sh

@kuisathaverat kuisathaverat marked this pull request as draft February 8, 2021 15:11
@codecov-io
Copy link

codecov-io commented Feb 8, 2021

Codecov Report

Merging #4696 (b1b9e81) into master (9ad7072) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4696   +/-   ##
=======================================
  Coverage   76.43%   76.43%           
=======================================
  Files         165      165           
  Lines       10083    10083           
=======================================
  Hits         7707     7707           
  Misses       2376     2376           
Impacted Files Coverage Δ
...ack/apm-server/aggregation/txmetrics/aggregator.go 93.36% <0.00%> (ø)

@kuisathaverat kuisathaverat marked this pull request as ready for review February 8, 2021 18:03
@kuisathaverat kuisathaverat merged commit c88d8fc into elastic:master Feb 17, 2021
kuisathaverat added a commit to kuisathaverat/apm-server that referenced this pull request Feb 17, 2021
* fix: set proper parameters for the tar step

* fix: move everything to the build folder
kuisathaverat added a commit to kuisathaverat/apm-server that referenced this pull request Feb 17, 2021
* fix: set proper parameters for the tar step

* fix: move everything to the build folder
kuisathaverat added a commit to kuisathaverat/apm-server that referenced this pull request Feb 17, 2021
* fix: set proper parameters for the tar step

* fix: move everything to the build folder
kuisathaverat added a commit that referenced this pull request Feb 18, 2021
* fix: set proper parameters for the tar step

* fix: move everything to the build folder
kuisathaverat added a commit that referenced this pull request Feb 18, 2021
* fix: set proper parameters for the tar step

* fix: move everything to the build folder
v1v added a commit to v1v/apm-server that referenced this pull request Feb 22, 2021
…chemas-to-agents

* upstream/master: (111 commits)
  Introduce metricset.name (elastic#4857)
  processor/otel: test service.version handling (elastic#4853)
  docs: Add PHP agent information to shared docs (elastic#4740)
  Script for faster development workflow (elastic#4731)
  Update to elastic/beats@1b31c26 (elastic#4763)
  backport: add 7.12 to .backportrc.json (elastic#4807)
  backport: enable auto-merge on backport PRs (elastic#4777)
  Support for Node.js profiles (elastic#4728)
  docs: readds .NET link (elastic#4764)
  [DOCS] Fixes URLs on Secure communication with APM Agents page (elastic#4746)
  ci: set proper parameters for the tar step (elastic#4696)
  docs: add 7.11.1 release notes (elastic#4727)
  Disable sourcemap upload endpoint when data streams enabled (elastic#4735)
  Add service name to dataset field (elastic#4674)
  Update to elastic/beats@ba423212a660 (elastic#4733)
  sampling: require a default policy (elastic#4729)
  processor/otel: add unit test for span status (elastic#4734)
  Add support for consuming OTLP/gRPC metrics (elastic#4722)
  [apmpackage] Add config options supported in ESS (elastic#4690)
  Use the apm-server version everywhere* (elastic#4725)
  ...
v1v added a commit to v1v/apm-server that referenced this pull request Feb 22, 2021
…te-schema-json-1

* upstream/master: (111 commits)
  Introduce metricset.name (elastic#4857)
  processor/otel: test service.version handling (elastic#4853)
  docs: Add PHP agent information to shared docs (elastic#4740)
  Script for faster development workflow (elastic#4731)
  Update to elastic/beats@1b31c26 (elastic#4763)
  backport: add 7.12 to .backportrc.json (elastic#4807)
  backport: enable auto-merge on backport PRs (elastic#4777)
  Support for Node.js profiles (elastic#4728)
  docs: readds .NET link (elastic#4764)
  [DOCS] Fixes URLs on Secure communication with APM Agents page (elastic#4746)
  ci: set proper parameters for the tar step (elastic#4696)
  docs: add 7.11.1 release notes (elastic#4727)
  Disable sourcemap upload endpoint when data streams enabled (elastic#4735)
  Add service name to dataset field (elastic#4674)
  Update to elastic/beats@ba423212a660 (elastic#4733)
  sampling: require a default policy (elastic#4729)
  processor/otel: add unit test for span status (elastic#4734)
  Add support for consuming OTLP/gRPC metrics (elastic#4722)
  [apmpackage] Add config options supported in ESS (elastic#4690)
  Use the apm-server version everywhere* (elastic#4725)
  ...
kuisathaverat added a commit that referenced this pull request Mar 8, 2021
* fix: set proper parameters for the tar step

* fix: move everything to the build folder
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation bug ci Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants