-
Notifications
You must be signed in to change notification settings - Fork 146
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
K8SPSMDB-807: Allow customizing replset name with custom config #1252
Conversation
* fix * fix sharding --------- Co-authored-by: Andrii Dema <[email protected]>
--------- Co-authored-by: Ege Güneş <[email protected]> Co-authored-by: Viacheslav Sarzhan <[email protected]>
wait_backup_agent $cluster-shard7-0 | ||
wait_backup_agent $cluster-shard7-1 | ||
wait_backup_agent $cluster-shard7-2 | ||
backup_name_aws="backup-aws-s3" |
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.
Why do we use only even shard numbers?
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.
the CR is from a customer environment, following their setup
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 remember it. Oks. Could we add comment about it?
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.
sure, what do you want to see in comment?
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.
As for me, It would be nice to add an explanation why we use this shards configuration
endpointUrl: https://storage.googleapis.com | ||
image: percona/percona-server-mongodb:4.4.10-11 | ||
imagePullPolicy: Always | ||
mongod: |
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.
@egegunes, we do not have mongod at all now. That is why you have a problem with this test.
e2e-tests/functions
Outdated
IMAGE_MONGOD_CHAIN=${IMAGE_MONGOD_CHAIN:-$' | ||
perconalab/percona-server-mongodb-operator:main-mongod4.4 | ||
perconalab/percona-server-mongodb-operator:main-mongod5.0 | ||
perconalab/percona-server-mongodb-operator:main-mongod6.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.
We use this var in mongod-major-upgrade test.
pkg/psmdb/mongos.go
Outdated
|
||
container.LivenessProbe.Exec.Command[0] = "/opt/percona/mongodb-healthcheck" | ||
container.ReadinessProbe.Exec.Command[0] = "/opt/percona/mongodb-healthcheck" |
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 it is better to add for >= 1.15.0
commit: 5d9ea86 |
CHANGE DESCRIPTION
Problem:
There are two problems:
cfg
.Solution:
Implemented a solution to override replset name using custom configuration:
CHECKLIST
Jira
Needs Doc
) and QA (Needs QA
)?Tests
compare/*-oc.yml
)?Config/Logging/Testability