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

feat: OBS-5 - add docker-compose #12

Merged
merged 15 commits into from
Feb 9, 2024
Merged

feat: OBS-5 - add docker-compose #12

merged 15 commits into from
Feb 9, 2024

Conversation

mfiedorowicz
Copy link
Member

@mfiedorowicz
chore: ignore Go coverage.txt
eb62bb3 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - allow diode-server/docker/netbox/env
4708acf 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - allow diode-server/docker/netbox/env/
1e91b23 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - add docker compose for netbox
8a79404 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - allow diode-server/docker/diode/env/
22e576b 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - add base docker compose for diode (redis+redis streams)
5ef8388 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - ignore diode-server/build
742ecd5 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - add diode server version
03c01e1 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - add diode services into docker compose
c25f6a9 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - move docker-compose.netbox
0554e9f 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - add Makefile for linting, testing, building and running…
12354b9 
… docker-compose

Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - format Makefile
b700047 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz mfiedorowicz self-assigned this Feb 9, 2024
@mfiedorowicz mfiedorowicz requested a review from natm as a code owner February 9, 2024 11:18
@linear Linear
Copy link

linear bot commented Feb 9, 2024

OBS-5 Add docker-compose for diode

Add docker-compose for running diode components and their dependencies:

  • distributor service
  • ingester service
  • reconciler service
  • redis
  • redis-streams

We should also be able to spin up docker-compose for running diode and netbox all together.

jit-ci[bot]
jit-ci bot reviewed Feb 9, 2024
View reviewed changes
Copy link

@jit-ci jit-ci bot left a comment

Choose a reason for hiding this comment

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

❌ Jit has detected 1 important finding in this PR that you should review.
The finding is detailed below as a comment.
It’s highly recommended that you fix this security issue before merge.

diode-server/docker/Dockerfile Show resolved Hide resolved
@mfiedorowicz
feat: OBS-5 - dockerfile use nonroot
5d8e6e2 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - dockerfile fix build path
f06a16a 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz
feat: OBS-5 - version comment
55cd12a 
Signed-off-by: Michal Fiedorowicz <mfiedorowicz@netboxlabs.com>
@mfiedorowicz mfiedorowicz merged commit cdcdac7 into develop Feb 9, 2024
11 checks passed
@mfiedorowicz mfiedorowicz deleted the OBS-5-docker-compose branch February 9, 2024 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jit-ci jit-ci[bot] jit-ci[bot] left review comments

@natm natm Awaiting requested review from natm

Assignees

@mfiedorowicz mfiedorowicz

Labels
build diode-server go internal python
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mfiedorowicz