Skip to content
This repository was archived by the owner on Oct 28, 2024. It is now read-only.

Conversation

@v1v
Copy link
Member

@v1v v1v commented May 11, 2020

What does this PR do?

Enable the test-infra for the beats-ci workers:

The test-infra is just a quite minimalistic so far.

Why is it important?

Add more confidence what's the required infra looks like since it will evaluate the CI workers with the test-infra framework. Same framework as we are using for the apm-ci workers.

Related issues

Caused by https://github.com/elastic/observability-robots/issues/110
Requires #524

Tasks

Follow-ups

  • Add more test coverage.
  • How to report if any issues and who to

@v1v v1v self-assigned this May 11, 2020
@v1v v1v added the automation label May 11, 2020
@botelastic botelastic bot added the ci label May 11, 2020
Copy link
Member Author

Choose a reason for hiding this comment

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

So far the list of workers are:

Copy link
Contributor

Choose a reason for hiding this comment

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

I have asked for labels on the MacOS workers to know if they have docker and if they can sign binaries

@ghost
Copy link

ghost commented May 11, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

Test stats 🧪

Test Results
Failed 0
Passed 499
Skipped 18
Total 517

v1v and others added 5 commits May 11, 2020 18:39
…-pipeline-library into feature/beats-workers-test-infra

* 'feature/beats-workers-test-infra' of github.com:v1v/apm-pipeline-library:
  test: use a different port (#523)
…kers-test-infra

* upstream/master:
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.96
  fix: null build description (#528)
  test: apm-ci test-infra for windows (#527)
  ci: notify other build status (#521)
  ci: CI approval aborted rather than failed (#522)
  ci: avoid template when no PRs (#526)
  test: move to apm-ci folder (#524)
@v1v v1v marked this pull request as ready for review May 13, 2020 14:27
@v1v v1v added the test label May 13, 2020
Copy link
Contributor

@cachedout cachedout left a comment

Choose a reason for hiding this comment

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

👍

@v1v v1v merged commit 77f8a4f into elastic:master May 14, 2020
@v1v v1v deleted the feature/beats-workers-test-infra branch May 14, 2020 13:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants