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

Add Github Actions workflow to build and publish Docker images #18

Merged
merged 3 commits into from
May 15, 2023

Conversation

martinholters
Copy link
Contributor

@martinholters martinholters commented May 15, 2023

This builds docker images and publishes them at ghcr.io/pvtom/rscp2mqtt (see ghcr.io/martinholters/rscp2mqtt where I've tested this at). Images both with and without InfluxDB support are build. Pushes to the main branch are tagged main and main-with-influxdb, repectively. Similar for tags, i.e. git tag v3.2 will be tagged as v3.2 and v3.2-with-influxdb, but they are also tagged with latest and latest-with-influxdb.

Pull requests also have the images built (to verify they at least build fine), but they are not pushed to the registry. Would it be preferred to also push them for easier testing?

The option of running via docker should probably by mentioned in the README, but before promising anything there, I'd like to see this actually working.

@pvtom, if you have Github actions enabled in the project setting - which I think is the default - you should be able to approve the run here and have images built (but not pushed as said).

@martinholters martinholters marked this pull request as draft May 15, 2023 12:39
@martinholters martinholters force-pushed the mh/gha branch 2 times, most recently from 71c1221 to f6c63e6 Compare May 15, 2023 13:19
@martinholters martinholters marked this pull request as ready for review May 15, 2023 13:36
@martinholters martinholters mentioned this pull request May 15, 2023
@pvtom pvtom merged commit 2ebb438 into pvtom:main May 15, 2023
@martinholters martinholters deleted the mh/gha branch May 15, 2023 16:19
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.

2 participants