-
Couldn't load subscription status.
- Fork 9
Create a plugin to add a WAL Receiver pod #74
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
base: main
Are you sure you want to change the base?
Conversation
| RUN --mount=type=cache,target=/root/.cache/go-build ./scripts/build.sh | ||
|
|
||
| # Step 2: build the image to be actually run | ||
| FROM golang:1-alpine |
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 what CNPG does? Or could we go distroless?
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 can do distroless, cnpg uses gcr.io/distroless/static-debian12:nonroot
| // Plugin parameter keys | ||
| const ( | ||
| ImageParam = "image" // string | ||
| ReplicationHostParam = "replicationHost" // Required: primary host |
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.
isn't that implicit in CNPG?
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.
CNPG also takes an optional image name
| ) | ||
|
|
||
| const ( | ||
| defaultImage = "ghcr.io/cloudnative-pg/postgresql:16" |
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.
can we have an image with just the wal reciever?
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 can try to make that, yeah
| "--port", "5432", | ||
| "--username", "postgres", | ||
| "--no-password", | ||
| "--verbose", |
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 should probbaly depend on log level
No description provided.