-
Notifications
You must be signed in to change notification settings - Fork 204
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
feat(ci): gha and blacksmith #4647
Conversation
2559f07
to
8d3fc9b
Compare
8d3fc9b
to
08469af
Compare
31b2f98
to
337e4bd
Compare
337e4bd
to
8638fbe
Compare
Anchors are not supported actions/runner#1182 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
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. 🚀 New features to boost your workflow:
|
e5cfc17
to
d29b0c1
Compare
700f0d6
to
529621d
Compare
- docker-build-postgres-container: | ||
context: *build-context | ||
filters: *filters-build | ||
filters: *filters-allow-all | ||
requires: | ||
- get-version |
There was a problem hiding this comment.
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
- test-server-multiregion: | ||
context: | ||
- speckle-server-licensing | ||
- stripe-integration | ||
- speckle-server-codecov | ||
filters: *filters-allow-all | ||
requires: | ||
- docker-build-postgres-container |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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
utils/postgres/Dockerfile
Outdated
@@ -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" ] |
There was a problem hiding this comment.
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.
.github/workflows/builds.yml
Outdated
uses: useblacksmith/build-push-action@v1 | ||
with: | ||
push: false | ||
tags: speckle/speckle-test-deployment:${{ inputs.IMAGE_VERSION_TAG }} # TODO: fix tag, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO?
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*
andtesting*
branches will run on CircleCI as usual.