-
Notifications
You must be signed in to change notification settings - Fork 4
Add build yaml #119
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
base: main
Are you sure you want to change the base?
Add build yaml #119
Conversation
This PR simply uses the same casing for the FROM and AS statements, as recommended by [Docker](https://docs.docker.com/go/dockerfile/rule/from-as-casing/)
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.
We need to add shorter tags as to allow staging deployments to be isolated.
We have a hard 63-char limit to Kubernetes Namespace names, as well as for Ingress URLs 🤷 .
#123 deals with this by adding a truncated version of $github.sha
as tag.
Adding an additional, shorter tag to the artifact will allow easier deployment-time configurations e.g., Ingress URLs. This will greatly simplify visualization for also debugging
!n8n-deploy-archive-node-api |
!n8n-destroy-archive-node-api |
Adds a
build.yaml
workflow triggered by PRs to the main branch that will bundle the branch into a docker build and upload the artifact to GCP.