-
Notifications
You must be signed in to change notification settings - Fork 69
Add development time yarn db:clear:all script for easy analytics data clearing #1041
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
Conversation
|
Oops! Looks like you forgot to update the changelog. When updating CHANGELOG.md, please consider the following:
|
infrastructure/clear-all-data.sh
Outdated
| POSTGRES_DB="events" | ||
| EVENTS_MIGRATOR_ROLE="events_migrator" | ||
| EVENTS_APP_ROLE="events_app" | ||
| ANALYTICS_POSTGRES_USER=${ANALYTICS_POSTGRES_USER:-"events_analytics"} |
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's the role of this user in deployed environments or even locally? I just see it being dropped if it exists
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.
This user should be the one Metabase uses to access the analytics database. In real deployments, you'd have two different databases completely with a different set of creds
|
I got the following error after merging develop and running the command: And postgres is up and running when I ran this |
feb0190 to
adf4fae
Compare
My output right now: |
| -e POSTGRES_DB="${POSTGRES_DB}" \ | ||
| -e EVENTS_MIGRATOR_ROLE="${EVENTS_MIGRATOR_ROLE}" \ | ||
| -e EVENTS_APP_ROLE="${EVENTS_APP_ROLE}" \ | ||
| -e ANALYTICS_POSTGRES_USER="${ANALYTICS_POSTGRES_USER}" \ |
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 e2e repo needs any changes regarding this? I mean this will be different for each feature environment right?
Description
Clearly describe what has been changed. Include relevant context or background.
Explain how the issue was fixed (if applicable) and the root cause.
Link this pull request to the GitHub issue (and optionally name the branch
ocrvs-<issue #>)Checklist