-
-
Notifications
You must be signed in to change notification settings - Fork 219
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
Support Nouveau in CHT Deploy script and upgrades and docs #9707
Comments
@jkuester or @sugat009 - do we know how a deployment looks when it has both a clustered couch deployment AND Nouveau? That is - if there's 3 couch containers running, is there 1 Nouveau container or 3 Nouveau containers? cc @dianabarsan |
😓 Mokhtar might know the answer to that question, but unfortunately I do not have any hard answers yet. Definitely something we need to confirm. |
Gotcha! I'll do a naive POC to see what happens on the latest CI branch but with multi-node couch compose file to see if I can find out anything. |
Ah - I don't think CI has ever passed, so the only way to POC is in dev mode. I'll see what I can do! I'll also see if CouchDB docs have any info. |
@jkuester or @sugat009 - are there steps on how to run Nouveau against a developer setup? Otherwise, I can't find an answer through a POC (b/c I can't bootstrap the system in the first place) and don't see Nouveau setup instructions involving a cluster anywhere - I'm hopping in there Slack server to see what I can find!
|
Good news! Jan Lehnardt (CouchDB dev and PMC chair) reports:
So we can very likely proceed with just the one instance. I've asked a follow up per the "Fan out" comment if we'd deploy a cluster of Nouveau nodes or something 🤔 Ok - confirmed! we can safely use 1:N Nouveau instances \o/ The only possible limitation is performance - which will be directly based on how Lucene performs. I think it's safe to keep it as is (default to "just another container") and we can scale it vertically or horizontally if needed in the future. Robert Newson @ Apache replied in our Slack thread:
|
cc @dianabarsan 👆 |
Thanks for the update @mrjones-plip ! Great news! |
Describe the issue
Current, the CHT k8s Deploy script and k8s upgrade service is not aware of the upcoming CHT Core Nouveau feature.
Describe the improvement you'd like
We should update the deploy script such that a happy path new deployment works when we ship CHT 4.x with Nouveau. Additionally, upgrade paths from prior versions of k8s based deployments should account for this change to ensure services don't break. This may be just documentation improvements or it maybe code additions or helm chart additions.
The text was updated successfully, but these errors were encountered: