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.

Sign up for GitHub

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

Jump to bottom

Docker-Integration fuer lokale Entwicklung und Github Packages/Dockerhub #30

Open
wants to merge 20 commits into
base: develop
Choose a base branch
from
Open

Docker-Integration fuer lokale Entwicklung und Github Packages/Dockerhub #30

wants to merge 20 commits into from

Conversation

janspeller
Copy link
Member

No description provided.

@janspeller janspeller changed the title Feature/docker Docker-Integration fuer lokale Entwicklung und Github Packages/Dockehub Apr 13, 2020
@thejockel
Copy link

Moin,
können wir den build auch für develop mit aufnehmen.
Grade wenn wir automatisch testen wollen, finde ich das für develop auch sehr wichtig.

@Schrolli91
Copy link
Member

Denke das sollte kein Problem sein.
Allerdings sehe ich die Notwendigkeit fürs automatische Testen nicht wirklich.
BOSWatch selbst wird ja schon jetzt getestet - ganz ohne Docker.

@janspeller janspeller changed the title Docker-Integration fuer lokale Entwicklung und Github Packages/Dockehub Docker-Integration fuer lokale Entwicklung und Github Packages/Dockerhub Apr 18, 2020
@janspeller janspeller self-assigned this Apr 18, 2020
@janspeller janspeller force-pushed the feature/docker branch 2 times, most recently from 337b974 to 3df108c Compare May 4, 2022 21:10
@janspeller
Copy link
Member Author

janspeller commented May 21, 2022 •
edited
Loading

@Schrolli91 Habe das ganze hier auch mal weiter gebracht. Bisher werden die Images noch auf mein "privates" Container-Repository geschoben:
ghcr.io/janspeller/bw3-core/client:latest // ghcr.io/janspeller/bw3-core/server:latest

Die Versionierung muss ggf. mit rein.

Außerdem sind bei frischen Installationen von pyyaml die Keys wohl case-sensitive. Das hatte ich angepasst, nachdem ich Ärger damit hatte.

Schau bei Gelegenheit mal drüber.

@Schrolli91
Copy link
Member

Schau ich mit im Laufe der Woche mal an - Danke dafür!

Schrolli91
Schrolli91 requested changes May 23, 2022
View reviewed changes
Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
@janspeller
Copy link
Member Author

@Schrolli91 Habe die Emails angepasst und das maintainer-label durch das authors-label aus der opencontainers specification ersetzt. (https://github.com/opencontainers/image-spec/blob/main/annotations.md)

@janspeller janspeller marked this pull request as ready for review May 23, 2022 21:59
Schrolli91
Schrolli91 requested changes May 24, 2022
View reviewed changes
Dockerfile

FROM build-base AS rtl_fm
ARG RTL_SDR_VERSION=0.6.0
Copy link
Member

Choose a reason for hiding this comment

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

Wieso sind die Versionsnummern weggefallen?
Das wurde damals ja extra so gemacht, weil es Probleme mit einer neueren Version gab.
So konnte man zum Build explizit vorgeben, was genutzt werden.
Same @ rtl_fm

Durch das ARG konnte man die Nummer dann nämlich auch zum Buildzeitpunkt übergeben.

Dockerfile
Comment on lines +26 to +27
RUN apk add libusb-dev libpulse
RUN pip3 install pyyaml
Copy link
Member

Choose a reason for hiding this comment

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

Wieso zweimal RUN direkt nacheinander?
Am besten beide Befehle in einen Verknüpfen.
Das reduziert die Container Größe.

@Schrolli91 Schrolli91 mentioned this pull request Dec 19, 2022
Closed
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Schrolli91 Schrolli91 Schrolli91 requested changes

@KoenigMjr KoenigMjr KoenigMjr approved these changes

Requested changes must be addressed to merge this pull request.

Assignees

@janspeller janspeller

Labels
Feature Status: In Progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@janspeller @thejockel @Schrolli91 @KoenigMjr