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
Minimal local mode #148
Minimal local mode #148
Conversation
|
docker/local/Dockerfile
Outdated
RUN apk add --allow-untrusted otelcol-contrib_0.90.1_linux_arm64.apk | ||
|
||
# == Install Node Deps == | ||
RUN apk add nodejs npm yarn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
better to use nvm use
to lock node version
docker/local/Dockerfile
Outdated
|
||
# == Install Vector Deps == | ||
RUN apk add curl | ||
RUN curl --proto '=https' --tlsv1.2 -sSfL https://sh.vector.dev | bash -s -- -y |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we also want to specify vector version (
hyperdx/.github/workflows/main.yml
Line 28 in 843cfa3
curl -sSfL --proto '=https' --tlsv1.2 https://packages.timber.io/vector/0.34.0/vector-0.34.0-x86_64-unknown-linux-musl.tar.gz | tar xzf - -C vector --strip-components=2 |
docker/local/Dockerfile
Outdated
RUN apk add mongodb yaml-cpp=0.6.2-r2 | ||
|
||
# == Install Redis == | ||
RUN apk add redis |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version ?
docker/local/build.sh
Outdated
--build-context ingestor=./docker/ingestor \ | ||
--build-context local=./docker/local \ | ||
--build-context api=./packages/api \ | ||
--build-context app=docker-image://ghcr.io/hyperdxio/hyperdx:1.3.0-app \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this intentional ? (app context)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes so we can avoid adding a builder step in the already tiring docker file 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense. I think we probably want to read the version from .env
(to pull the latest)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's true - now that i typed it out, i'll probably need to add the builder step here anyways in this dockerfile since we'll need to have a specific bundle for local mode for a few small changes. so for now this is really just a placeholder.
@@ -27,6 +27,7 @@ | |||
"jsonwebtoken": "^9.0.0", | |||
"lodash": "^4.17.21", | |||
"minimist": "^1.2.7", | |||
"mongodb": "^6.3.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is supposed to be dev dep ? (for migration script)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's actually a peer dep of connect-mongo
, not sure why it was working before: https://github.com/jdesboeufs/connect-mongo/blob/master/package.json#L61
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
from mongoose 🤷♂️ . also the peer seems to be ^4.1.0
. anyway as long as it doesn't introduce deps issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome work! Just double check the dev dependency
c7fdbd2
to
99a144a
Compare
99a144a
to
e48aaa7
Compare
|
||
# == Clickhouse/Base Image == | ||
|
||
FROM clickhouse/clickhouse-server:23.11.1-alpine AS clickhouse_base |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would suggest to align the ch version with the prebuilt and dev envs (v23.8.8)
|
||
# == Install Vector Deps == | ||
RUN apk add curl | ||
RUN curl --proto '=https' --tlsv1.2 -sSfL https://sh.vector.dev | bash -s -- -y | ||
RUN mkdir -p vector | ||
RUN curl -sSfL --proto '=https' --tlsv1.2 https://packages.timber.io/vector/0.34.0/vector-0.34.0-x86_64-unknown-linux-musl.tar.gz | tar xzf - -C vector --strip-components=2 && \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v0.35.0
Not ready for prime time yet, but does accomplish the basics of #7
A few things to iron out: