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

Publish image to GitHub Container Registry #621

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

cimnine
Copy link
Collaborator

@cimnine cimnine commented Oct 18, 2021

Related Issue: n/a

New Behavior

I've extracted this from #604.

Contrast to Current Behavior

We only publish to quay.io and docker.io. But GitHub's container registry has a nice integration into GitHub, which is the home of NetBox Docker.

Discussion: Benefits and Drawbacks

I don't see no drawback, since we rely on GitHub anyway. I do see as advantage that we'd have one more fallback solution in place. Docker is now rate-limited and who knows what Red Hat plans with Quay. (To be fair, the same could be said for GHCR, but at least (hopefully) did GitHub already know what they put themselves into when they launched GHCR.)

Changes to the Wiki

none

Proposed Release Note Entry

NetBox Docker is now also published to GitHub's Container Registry: docker pull ghcr.io/netbox-community/netbox.

Double Check

  • I have read the comments and followed the PR template.
  • I have explained my PR according to the information in the comments.
  • My PR targets the develop branch.

@cimnine cimnine added the enhancement The issue describes an enhancement that we would like to implement in the future. label Oct 18, 2021
@cimnine cimnine self-assigned this Oct 18, 2021
@cimnine cimnine marked this pull request as ready for review October 18, 2021 10:50
@cimnine cimnine requested a review from tobiasge October 18, 2021 10:51
@cimnine cimnine changed the title Publish image to Docker images Publish image to GitHub Container Registry Oct 19, 2021
@cimnine cimnine merged commit 1e78ccd into netbox-community:develop Oct 19, 2021
@cimnine cimnine deleted the GHCR branch October 19, 2021 12:24
@anthr76
Copy link

anthr76 commented Nov 12, 2021

This was merged, but the images have not been yet made public.

@cimnine
Copy link
Collaborator Author

cimnine commented Nov 13, 2021

It's not yet released.

@tobiasge tobiasge mentioned this pull request Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement The issue describes an enhancement that we would like to implement in the future.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants