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

docs: initial github container registry integration guide #7088

Merged
merged 9 commits into from
Apr 26, 2024

Conversation

lukemarsden
Copy link
Contributor

@lukemarsden lukemarsden commented Apr 12, 2024

An initial guide to how to push to container images into GHCR from Dagger running in GitHub Actions.

Signed-off-by: Luke Marsden <me@lukemarsden.net>
Signed-off-by: Luke Marsden <me@lukemarsden.net>
Signed-off-by: Luke Marsden <me@lukemarsden.net>
@lukemarsden
Copy link
Contributor Author

CI failure seems unrelated to this change?

@vikram-dagger
Copy link
Contributor

I've not reviewed in depth but a few questions off the top:

  • Why don't we incorporate this into the existing GitHub Actions integrations page? This would avoid duplicating the GHA workflow. Having two workflow examples risks user confusion, as it's then not clear which version is canonical.
  • The file should be called ....go instead of ...go.txt. You could place it in a separate subdir of snippets and include the dagger.gen.go and go.mod files, and then you would not need to import dagger in the Go code. Example structure here.

@vikram-dagger
Copy link
Contributor

CI failure seems unrelated to this change?

CI error is failed to get edge: inconsistent graph state which does indeed seem unrelated.

Co-authored-by: vikram-dagger <112123850+vikram-dagger@users.noreply.github.com>
Signed-off-by: lukemarsden <me@lukemarsden.net>
@lukemarsden
Copy link
Contributor Author

Yeah, happy to move it into the GHA page as a sub-section if you think that's best!

@vikram-dagger
Copy link
Contributor

@lukemarsden OK to push changes directly to your PR?

@lukemarsden
Copy link
Contributor Author

lukemarsden commented Apr 17, 2024 via email

Signed-off-by: Vikram Vaswani <vikram@dagger.io>
Signed-off-by: Luke Marsden <me@lukemarsden.net>
@lukemarsden
Copy link
Contributor Author

Updated the module reference to point to daggerverse/dagger-ghcr-demo instead of lukemarsden/dagger-ghcr-demo

This LGTM

@vikram-dagger vikram-dagger self-assigned this Apr 25, 2024
@vikram-dagger
Copy link
Contributor

Ping @kpenfound for review

@vikram-dagger vikram-dagger self-requested a review April 25, 2024 04:52
Copy link
Contributor

@kpenfound kpenfound left a comment

Choose a reason for hiding this comment

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

LGTM!

@vikram-dagger vikram-dagger merged commit 379e812 into dagger:main Apr 26, 2024
31 checks passed
vikram-dagger added a commit to vikram-dagger/dagger that referenced this pull request May 3, 2024
* docs: initial github container registry integration guide

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* docs: fix link to main.go

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* docs: hopefully convince linter this isn't a real go file

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* Update docs/current_docs/integrations/snippets/ghcr-workflow.yml

Co-authored-by: vikram-dagger <112123850+vikram-dagger@users.noreply.github.com>
Signed-off-by: lukemarsden <me@lukemarsden.net>

* Integrates examplein to GitHub Actions integ page

Signed-off-by: Vikram Vaswani <vikram@dagger.io>

* rename

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* bump tag

Signed-off-by: Luke Marsden <me@lukemarsden.net>

---------

Signed-off-by: Luke Marsden <me@lukemarsden.net>
Signed-off-by: lukemarsden <me@lukemarsden.net>
Signed-off-by: Vikram Vaswani <vikram@dagger.io>
Co-authored-by: vikram-dagger <112123850+vikram-dagger@users.noreply.github.com>
Co-authored-by: Vikram Vaswani <vikram@dagger.io>
vikram-dagger added a commit to vikram-dagger/dagger that referenced this pull request May 3, 2024
* docs: initial github container registry integration guide

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* docs: fix link to main.go

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* docs: hopefully convince linter this isn't a real go file

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* Update docs/current_docs/integrations/snippets/ghcr-workflow.yml

Co-authored-by: vikram-dagger <112123850+vikram-dagger@users.noreply.github.com>
Signed-off-by: lukemarsden <me@lukemarsden.net>

* Integrates examplein to GitHub Actions integ page

Signed-off-by: Vikram Vaswani <vikram@dagger.io>

* rename

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* bump tag

Signed-off-by: Luke Marsden <me@lukemarsden.net>

---------

Signed-off-by: Luke Marsden <me@lukemarsden.net>
Signed-off-by: lukemarsden <me@lukemarsden.net>
Signed-off-by: Vikram Vaswani <vikram@dagger.io>
Co-authored-by: vikram-dagger <112123850+vikram-dagger@users.noreply.github.com>
Co-authored-by: Vikram Vaswani <vikram@dagger.io>
vikram-dagger added a commit to vikram-dagger/dagger that referenced this pull request May 3, 2024
* docs: initial github container registry integration guide

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* docs: fix link to main.go

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* docs: hopefully convince linter this isn't a real go file

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* Update docs/current_docs/integrations/snippets/ghcr-workflow.yml

Co-authored-by: vikram-dagger <112123850+vikram-dagger@users.noreply.github.com>
Signed-off-by: lukemarsden <me@lukemarsden.net>

* Integrates examplein to GitHub Actions integ page

Signed-off-by: Vikram Vaswani <vikram@dagger.io>

* rename

Signed-off-by: Luke Marsden <me@lukemarsden.net>

* bump tag

Signed-off-by: Luke Marsden <me@lukemarsden.net>

---------

Signed-off-by: Luke Marsden <me@lukemarsden.net>
Signed-off-by: lukemarsden <me@lukemarsden.net>
Signed-off-by: Vikram Vaswani <vikram@dagger.io>
Co-authored-by: vikram-dagger <112123850+vikram-dagger@users.noreply.github.com>
Co-authored-by: Vikram Vaswani <vikram@dagger.io>
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