-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add data_layer support #230
base: main
Are you sure you want to change the base?
Conversation
docs/chiadatalayer.md
Outdated
spec: | ||
chia: | ||
caSecretName: chiaca-secret # A kubernetes Secret containing certificate authority files | ||
``` |
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.
Need more docs.
app.kubernetes.io/managed-by: kustomize | ||
name: chiadatalayer-sample | ||
spec: | ||
# TODO(user): Add fields here |
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.
Should make a better sample here even though the samples aren't really advertised anywhere.
8b4e621
to
9e7aebc
Compare
} | ||
|
||
// ChiaDataLayerSpecChia defines the desired state of Chia component configuration | ||
type ChiaDataLayerSpecChia struct { |
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.
What settings would be valuable for this? Off the top of my head, it would be cool to be able to add or delete store IDs from the builtin wallet in the yaml config. But doing that I believe would likely require config sidecar be developed that runs alongside the data_layer container in the Pod and makes all the RPC requests for managing stores from the yaml config for a ChiaDatalayer resource.
Maybe there's another option? But I don't think using the chia-docker entrypoint script works as I believe the data_layer/wallet RPC needs to be up.
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.
If we go that far with developing the config sidecar, it may as well support all of the actions from https://docs.chia.net/guides/datalayer-user-guide/
Config sidecar will likely be developed after this PR. Just standing up the instance and managing the wallet manually by exec-ing into the chia container as a MVP.
No description provided.