Skip to content

feat(ci): gha and blacksmith #4647

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

Conversation

danielgak
Copy link
Contributor

@danielgak danielgak commented May 1, 2025

Description

Ports the Pull Request jobs from CircleCI to GitHub Actions on Blacksmith runners.

The multi-region test-server job remains in CircleCI as the only job (along with its dependencies get-version and build-postgres-docker-image) triggered by Pull Requests.

Commits to main, hotfix* and testing* branches will run on CircleCI as usual.

Copy link

linear bot commented May 1, 2025

@danielgak danielgak force-pushed the daniel/web-3263-test-yarn-install-caching-in-github-actions branch 4 times, most recently from 2559f07 to 8d3fc9b Compare May 1, 2025 15:27
@danielgak danielgak force-pushed the daniel/web-3263-test-yarn-install-caching-in-github-actions branch from 8d3fc9b to 08469af Compare May 1, 2025 15:50
@danielgak danielgak changed the title feat(ci): trying out gha yarn cache WIP (ci): trying out gha yarn cache May 2, 2025
@danielgak danielgak marked this pull request as draft May 14, 2025 06:05
@danielgak danielgak changed the title WIP (ci): trying out gha yarn cache WIP (ci): trying out gha May 19, 2025
@danielgak danielgak force-pushed the daniel/web-3263-test-yarn-install-caching-in-github-actions branch 2 times, most recently from 31b2f98 to 337e4bd Compare May 19, 2025 10:07
@danielgak danielgak force-pushed the daniel/web-3263-test-yarn-install-caching-in-github-actions branch from 337e4bd to 8638fbe Compare May 19, 2025 10:14
@danielgak
Copy link
Contributor Author

Anchors are not supported actions/runner#1182

@danielgak danielgak changed the title WIP (ci): trying out gha ci: trying out gha May 19, 2025
@danielgak danielgak changed the title ci: trying out gha feat(ci): gha and blacksmith May 19, 2025
Copy link

codecov bot commented May 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.66%. Comparing base (3a2564d) to head (8c08069).
Report is 40 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (3a2564d) and HEAD (8c08069). Click for more details.

HEAD has 7 uploads less than BASE
Flag BASE (3a2564d) HEAD (8c08069)
8 1
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #4647       +/-   ##
===========================================
- Coverage   70.31%   58.66%   -11.66%     
===========================================
  Files         578      109      -469     
  Lines       25014     3970    -21044     
  Branches     3729      488     -3241     
===========================================
- Hits        17589     2329    -15260     
+ Misses       6257     1613     -4644     
+ Partials     1168       28     -1140     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@danielgak danielgak force-pushed the daniel/web-3263-test-yarn-install-caching-in-github-actions branch from e5cfc17 to d29b0c1 Compare May 19, 2025 11:59
@danielgak danielgak force-pushed the daniel/web-3263-test-yarn-install-caching-in-github-actions branch 2 times, most recently from 700f0d6 to 529621d Compare May 19, 2025 12:52
@danielgak danielgak marked this pull request as ready for review June 6, 2025 16:29
@danielgak danielgak requested a review from iainsproat June 6, 2025 16:30
Comment on lines 182 to 186
- docker-build-postgres-container:
context: *build-context
filters: *filters-build
filters: *filters-allow-all
requires:
- get-version
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this one still runs after get-version, so it can be used by multiregion

Comment on lines +63 to +70
- test-server-multiregion:
context:
- speckle-server-licensing
- stripe-integration
- speckle-server-codecov
filters: *filters-allow-all
requires:
- docker-build-postgres-container
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this one still runs after docker-build-postgres-container

@@ -126,66 +136,64 @@ workflows:
filters: *filters-allow-all
Copy link
Contributor Author

Choose a reason for hiding this comment

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

get-version is still needed in circle ci

iainsproat
iainsproat previously approved these changes Jun 6, 2025
@@ -22,4 +22,4 @@ COPY --link --from=builder /aiven-extras/aiven_extras.so /usr/local/lib/postgres

EXPOSE 5432

CMD ["postgres"]
CMD ["postgres", "-c", "max_connections=1000", "-c", "wal_level=logical" ]
Copy link
Contributor

Choose a reason for hiding this comment

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

In CircleCI multiregion test we no longer strictly require the line command: -c 'max_connections=1000' -c 'wal_level=logical' if this is present here.

But the command in CircleCI replaces the entire CMD statement in the Dockerfile, so no need to make any changes.

uses: useblacksmith/build-push-action@v1
with:
push: false
tags: speckle/speckle-test-deployment:${{ inputs.IMAGE_VERSION_TAG }} # TODO: fix tag,
Copy link
Contributor

Choose a reason for hiding this comment

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

TODO?

@danielgak danielgak merged commit 295f02a into main Jun 9, 2025
28 of 30 checks passed
@danielgak danielgak deleted the daniel/web-3263-test-yarn-install-caching-in-github-actions branch June 9, 2025 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants