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

action: build and test on Linux GH runners #1995

Merged
merged 8 commits into from
May 22, 2023
Merged

Conversation

v1v
Copy link
Member

@v1v v1v commented Jan 27, 2023

What

Enable GH actions for building/testing on Linux
Visualise test errors

Issues

Originally implemented in #1989

@v1v v1v added automation Team:Automation Label for the Observability productivity team labels Jan 27, 2023
@v1v v1v requested review from a team January 27, 2023 08:57
@v1v v1v self-assigned this Jan 27, 2023
@elastic-apm-tech elastic-apm-tech added this to In Progress in APM-Agents (OLD) Jan 27, 2023
@apmmachine
Copy link
Collaborator

apmmachine commented Jan 27, 2023

💚 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 previewSnapshots

Expand to view the summary

Build stats

  • Start Time: 2023-04-03T07:31:22.134+0000

  • Duration: 79 min 56 sec

Test stats 🧪

Test Results
Failed 0
Passed 6535
Skipped 141
Total 6676

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.

  • run benchmark tests : Run the benchmark test.

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

steps:
- uses: elastic/apm-pipeline-library/.github/actions/test-report@current
with:
artifact: test-results # artifact name
Copy link
Member

Choose a reason for hiding this comment

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

Linking for visibility: elastic/apm-pipeline-library#2063 these can quite noisy but hit an issue with reporting only failed.

Copy link
Member Author

Choose a reason for hiding this comment

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

#2019 is the one testing that particular test reporter action

Copy link
Member Author

Choose a reason for hiding this comment

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

although it won't test it since the action needs exist in the default main branch. I'll try with the ecs-logging-net repo

Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

🥳 LGTM! Please wait with merging while we release 1.20.0 today though.

run: .ci/linux/release.sh true

- name: Rustup
run: rustup default 1.59.0
Copy link
Member

Choose a reason for hiding this comment

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

@v1v the current build failures are because we now need rust 1.67.1

Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

LGTM, the current failure is intermittent and due to nuget's status

@Mpdreamz Mpdreamz merged commit fad8329 into main May 22, 2023
11 of 13 checks passed
APM-Agents (OLD) automation moved this from In Progress to Done May 22, 2023
@Mpdreamz Mpdreamz deleted the feature/github-action-linux branch May 22, 2023 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agent-dotnet automation Team:Automation Label for the Observability productivity team
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants