-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Update release workflow to include new docker images #2108
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2108 +/- ##
===========================================
- Coverage 83.51% 57.99% -25.52%
===========================================
Files 100 100
Lines 8088 8088
===========================================
- Hits 6755 4691 -2064
- Misses 1333 3397 +2064
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
/lgtm |
New changes are detected. LGTM label has been removed. |
…ase workflows. Add/update docker images for lambda feature server, FTS, and java feature server Signed-off-by: Danny Chiao <danny@tecton.ai>
Signed-off-by: Danny Chiao <danny@tecton.ai>
Signed-off-by: Danny Chiao <danny@tecton.ai>
Signed-off-by: Danny Chiao <danny@tecton.ai>
e2f9597
to
2c67d87
Compare
Signed-off-by: Danny Chiao <danny@tecton.ai>
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: adchia, pyalex, tsotnet The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: Danny Chiao <danny@tecton.ai>
Signed-off-by: Danny Chiao <danny@tecton.ai>
What this PR does / why we need it:
There are new feature transformation server, lambda feature server, and java feature server docker images. This adds convenience commands in the Makefile + merges/updates our release workflow to build + push these images to GCR/dockerhub.
This also removes a workflow that builds a new FTS image + tags against the commit. We don't use this (the helm charts don't point to these either).
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: