Skip to content

Add reporting to Tinybird for new GitHub Actions pipelines #12610

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 10 commits into from
May 16, 2025

Conversation

silv-io
Copy link
Member

@silv-io silv-io commented May 12, 2025

Motivation

This PR is part of the initiative to switch to GitHub Actions in the test suite of this repository. It adds reporting to tinybird of the workflow and test results.

Changes

  • Add workflow status reporting from main and mamr pipelines
  • Add pytest single test result publishing for all test suites

Testing

Run the different workflows (full, acceptance, mamr) and make sure the tinybird report goes through. Also check the dashboard for existence of the workflows

TODO

What's left to do:

  • Adapt the tinybird severity scale to take in the GHA pipelines instead of CircleCI after establishing the new pipelines as stable
  • Enable the guard to only report on master before merging

@silv-io silv-io force-pushed the add-tinybird-reporting-gha branch from 4d8a98c to 782eb58 Compare May 12, 2025 18:46
@silv-io silv-io force-pushed the add-tinybird-reporting-gha branch from 782eb58 to 2842b33 Compare May 13, 2025 10:12
Copy link

github-actions bot commented May 13, 2025

S3 Image Test Results (AMD64 / ARM64)

  2 files  ± 0    2 suites  ±0   8m 21s ⏱️ -10s
495 tests + 7  445 ✅ + 7   50 💤 ±0  0 ❌ ±0 
990 runs  +14  890 ✅ +14  100 💤 ±0  0 ❌ ±0 

Results for commit c5fe2bf. ± Comparison against base commit 9b6ef35.

♻️ This comment has been updated with latest results.

@silv-io silv-io changed the title Add tinybird reporting gha Add reporting to Tinybird for new GitHub Actions pipelines May 13, 2025
@silv-io silv-io force-pushed the add-tinybird-reporting-gha branch from 2842b33 to 7603f71 Compare May 13, 2025 10:25
@silv-io silv-io added the semver: patch Non-breaking changes which can be included in patch releases label May 13, 2025
Copy link

github-actions bot commented May 13, 2025

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 41m 4s ⏱️ + 1m 49s
4 433 tests +8  4 049 ✅ +7  384 💤 +1  0 ❌ ±0 
4 435 runs  +8  4 049 ✅ +7  386 💤 +1  0 ❌ ±0 

Results for commit c5fe2bf. ± Comparison against base commit 9b6ef35.

♻️ This comment has been updated with latest results.

@silv-io silv-io force-pushed the add-tinybird-reporting-gha branch from 298817a to 43e700e Compare May 14, 2025 07:20
@silv-io silv-io force-pushed the add-tinybird-reporting-gha branch from eee53ef to 6c3ab45 Compare May 14, 2025 11:40
@silv-io silv-io marked this pull request as ready for review May 14, 2025 12:26
@@ -152,6 +154,24 @@ jobs:
make publish || echo "dev release failed (maybe it is already published)"
fi

push-to-tinybird:
name: Push Workflow Status to Tinybird
if: always() # && github.ref == 'refs/heads/master'
Copy link
Contributor

@k-a-il k-a-il May 15, 2025

Choose a reason for hiding this comment

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

why is this part # && github.ref == 'refs/heads/master' commented out? do you plan to enable it for the master branch only once we're ready to switch to GitHub Actions?

Copy link
Member Author

Choose a reason for hiding this comment

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

was too eager to merge right now and forgot to revert those changes, will do a quick follow-up PR 😄

Copy link
Contributor

@k-a-il k-a-il left a comment

Choose a reason for hiding this comment

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

Nice job 🚀 Great to see reporting to tinybird in the main pipeline

@silv-io silv-io merged commit 7b25203 into master May 16, 2025
53 of 54 checks passed
@silv-io silv-io deleted the add-tinybird-reporting-gha branch May 16, 2025 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants