Skip to content
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

Sharding incorrectly reports being added to shard #302

Closed
MiaAltieri opened this issue Nov 6, 2023 · 1 comment
Closed

Sharding incorrectly reports being added to shard #302

MiaAltieri opened this issue Nov 6, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@MiaAltieri
Copy link
Contributor

Steps to reproduce

juju deploy ./*charm --config role="config-server" config-server-one -n2
juju deploy ./*charm --config role="shard" shard-one -n2
juju deploy ./*charm --config role="shard" shard-two -n2
juju integrate config-server-one:config-server shard-one:sharding
juju integrate config-server-one:config-server shard-two:sharding
juju remove-relation config-server-one:config-server shard-two:sharding

Expected behavior

the last unit of shard-two to report active and running

Actual behavior

the last unit of shard-two reports it is still being added to the cluster

why this is happening

This is happening because we check if a shard has been added by relation data

proposed fix

Actually ask mongodb if the shard has been added instead of checking a boolean flag

@MiaAltieri MiaAltieri added the bug Something isn't working label Nov 6, 2023
Copy link

github-actions bot commented Nov 6, 2023

@MiaAltieri MiaAltieri mentioned this issue Nov 6, 2023
MiaAltieri added a commit that referenced this issue Nov 6, 2023
## Issue
We have several bugs in the sharding implementation, fix them
#297 
#301 
#302
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant