Skip to content
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

Create stargate Dockerfile and GH Publish action #189

Merged
merged 2 commits into from
Sep 29, 2020

Conversation

trevor-scheer
Copy link
Member

This commit introduces a Dockerfile for building a stargate image.
Currently this is building from an in-flight PR (#135), which we
can update as soon as it lands on main.

The additional GH action publishes the container to the new
GitHub Container Registry. For now, this requires a PAT (as noted
in the comments). We should remove this in favor of GITHUB_TOKEN
as soon as it's supported.

This commit introduces a Dockerfile for building a stargate image.
Currently this is building from an in-flight PR (#135), which we
can update as soon as it lands on `main`.

The additional GH action publishes the container to the new
GitHub Container Registry. For now, this requires a PAT (as noted
in the comments). We should remove this in favor of GITHUB_TOKEN
as soon as it's supported.
Copy link
Contributor

@Enrico2 Enrico2 left a comment

Choose a reason for hiding this comment

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

This won't work anymore if (once?) #186 is merged to #135

Do you want to base this on #186 and assume stargate-actix-web will be the one we use? The Dockerfile location can remain the same, just the build should build the binary in integrations/stargate-actix-web

stargate/Dockerfile Outdated Show resolved Hide resolved
Co-authored-by: Jesse Rosenberger <git@jro.cc>
Copy link
Member

@abernix abernix left a comment

Choose a reason for hiding this comment

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

This won't work anymore if (once?) #186 is merged to #135
Do you want to base this on #186 and assume stargate-actix-web will be the one we use?

While true, that makes #186 a blocker of this PR unnecessarily.

Instead, let's have #186 make the appropriate change — when finished/ratified — to this file so this can merge right away.

@trevor-scheer trevor-scheer merged commit 4b02293 into main Sep 29, 2020
@trevor-scheer trevor-scheer deleted the trevor/publish_stargate_on_release branch September 29, 2020 14:20
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.

None yet

3 participants