Skip to content
This repository has been archived by the owner on Dec 11, 2022. It is now read-only.

Replica set check requires "clusterManager" database role #28

Open
3 tasks done
aldeed opened this issue Feb 10, 2020 · 2 comments
Open
3 tasks done

Replica set check requires "clusterManager" database role #28

aldeed opened this issue Feb 10, 2020 · 2 comments
Labels
bug For issues that describe a defect or regression in the released software help wanted For issues that have a clear solution described and are not currently prioritized core team work

Comments

@aldeed
Copy link
Contributor

aldeed commented Feb 10, 2020

Prerequisites

  • Are you running the latest version?
  • Are you able to consistently reproduce the issue?
  • Did you search the issue queue for existing issue? Search issues

Issue Description

Reaction Identity waits before starting up to confirm that the MongoDB replica set is available. This check fails if the database user does not have "clusterManager" database role. We should not require this role in a deployed situation, especially not in production.

Steps to Reproduce

  1. Start Reaction Identity Docker service with a MONGO_URL that specifies a user who does not have "clusterManager" database role.
  2. View the service logs. It will hang for 2 minutes waiting for replica set, and then show a permission error.

Solution

This check was added primarily for development environments, where we are creating a replica set on the fly and Reaction Identity (Meteor) will throw an error if it isn't ready yet (race condition).

We can wrap the check in an environment variable that we set by default in development but would need to be manually set in a deployed environment.

Versions

3.0.0

@aldeed aldeed self-assigned this Feb 10, 2020
@aldeed aldeed added bug For issues that describe a defect or regression in the released software help wanted For issues that have a clear solution described and are not currently prioritized core team work labels May 15, 2020
@aldeed aldeed removed their assignment May 21, 2020
@delagroove delagroove reopened this Aug 24, 2021
@brent-hoover
Copy link
Collaborator

@aldeed Since this project is deprecated are we okay to close this?

@aldeed
Copy link
Contributor Author

aldeed commented Jul 29, 2022

@zenweasel It's referring to this script, which might have been in some other projects, too. If you don't find similar code in any other projects, this should be okay to close.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug For issues that describe a defect or regression in the released software help wanted For issues that have a clear solution described and are not currently prioritized core team work
Projects
None yet
Development

No branches or pull requests

3 participants