-
Notifications
You must be signed in to change notification settings - Fork 4
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
DB-drop-and-reset-write-down #2710
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #2710 +/- ##
===========================================
- Coverage 92.99% 92.75% -0.25%
===========================================
Files 219 235 +16
Lines 4482 5201 +719
Branches 385 452 +67
===========================================
+ Hits 4168 4824 +656
- Misses 242 290 +48
- Partials 72 87 +15
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Sentry.
|
# How to DROP existing DB and Recreate a fresh DB | ||
|
||
### Connecting to DB service | ||
First step is to connect to the instance DB (see [this](CloudFoundry-DB-Connection.md)). |
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.
Could you just move this to a section within that document please? Otherwise good write up!
After connection to the DB is made (the step above will make a psql connection), then the following Postgres commands have to run: | ||
|
||
1. List the DBs: `\l` | ||
2. find the associated DB name with instance. E.g: `tdp_db_dev_qasp` |
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.
Does it make sense to have a section or command on how to take a backup/dump?
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.
Please address both feedback items I commented.
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.
Thank you! LGTM
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.
thanks for the demo @raftmsohani 🚀
Summary of Changes
Provide a brief summary of changes
Pull request closes #_
How to Test
List the steps to test the PR
These steps are generic, please adjust as necessary.
Deliverables
More details on how deliverables herein are assessed included here.
Deliverable 1: Accepted Features
Checklist of ACs:
lfrohlich
and/oradpennington
confirmed that ACs are met.Deliverable 2: Tested Code
CodeCov Report
comment in PR)CodeCov Report
comment in PR)Deliverable 3: Properly Styled Code
Deliverable 4: Accessible
iamjolly
andttran-hub
using Accessibility Insights reveal any errors introduced in this PR?Deliverable 5: Deployed
Deliverable 6: Documented
Deliverable 7: Secure
Deliverable 8: User Research
Research product(s) clearly articulate(s):