-
Notifications
You must be signed in to change notification settings - Fork 29
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
[COR-155] 'coredb install' to install operator #57
Conversation
- name: Check CRD is updated in the yaml directory | ||
run: | | ||
set -xe | ||
cargo run --bin crdgen > yaml/crd.yaml | ||
git diff --exit-code yaml/crd.yaml |
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.
this part will fail if CRD definition is changed but yaml directory not updated
coredb-operator/Dockerfile
Outdated
FROM clux/muslrust:stable AS builder | ||
|
||
COPY . . | ||
|
||
RUN cargo build --release | ||
|
||
FROM alpine:latest | ||
|
||
RUN adduser -D nonroot | ||
|
||
COPY --chown=nonroot:nonroot --from=builder target/x86_64-unknown-linux-musl/release/controller /app/controller | ||
COPY --chown=nonroot:nonroot --from=builder target/x86_64-unknown-linux-musl/release/crdgen /app/crdgen | ||
|
||
USER nonroot | ||
|
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.
Although rust compiles to a binary, by default that binary has dependencies in the system like openssl and libc. If we build with MUSL we can avoid that - like clux did in the controller example.
@@ -86,7 +86,7 @@ spec: | |||
serviceAccountName: coredb-controller | |||
containers: | |||
- name: coredb-controller | |||
image: localhost:5001/controller:ac593af6da990c911652c6e35fb777f36498d6dc | |||
image: sjmiller609/coredb-operator: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.
replace this with quay when ian is back
* wip restart event * change unhandled message * add sts restart fn * handle restart event * add test * fix test * handle reporting state event * cleanup * minor refactor
coredb-install-demo.mov