-
Notifications
You must be signed in to change notification settings - Fork 30
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
RHINENG-10032: fix repack job db user #1479
Conversation
Referenced Jiras: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1479 +/- ##
==========================================
+ Coverage 65.03% 65.06% +0.02%
==========================================
Files 114 114
Lines 7877 7877
==========================================
+ Hits 5123 5125 +2
+ Misses 2217 2215 -2
Partials 537 537
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
it might be possible that this won't help because admin
user is probably not a superuser and the option to use regular user seems broken in pg_repack reorg/pg_repack#223
Can we use default postgres
user? Try to look at how db migrations are performed, I think we're using the superuser there
|
340405c
to
2aea703
Compare
Secure Coding Practices Checklist GitHub Link
Secure Coding Checklist