feat: Add sentry-admin.sh tool #2594
Conversation
azaslavsky
force-pushed
the
azaslavsky/sentry-admin
branch
from
d2e2909
to
2e911c8
Compare
| @@ -0,0 +1,50 @@ | |||
| #!/usr/bin/env bash | |||
There was a problem hiding this comment.
I don't think we should have two instances of backup in integration tests if it takes too long. It already takes over 25 minutes to run integration tests and this adds to that overhead
There was a problem hiding this comment.
Removed this 2nd backup test, and modified the original to use sentry-admin.sh instead. We'll modify the docs once this PR lands.
sentry-admin.sh
Outdated
There was a problem hiding this comment.
If we have this sentry-admin tool, I think we can delete the /scripts/backup.sh and /scripts/restore.sh, thoughts?
There was a problem hiding this comment.
Along the lines of that, I do think reset.sh could be incorporated here, but that can be done in a follow up PR.
There was a problem hiding this comment.
I'd like to keep them for now, since users may be relying on them. We can deprecate-then-delete them in future releases.
Agree on reset.sh.
azaslavsky
force-pushed
the
azaslavsky/sentry-admin
branch
from
4380d23
to
efa2901
Compare
sentry-admin.shis just a bash wrapper around some admin commands that are already technically exposed in thewebcontainer, though a bit hard to find today. This command just forwards thesentrycommand-line administration tool out from the container onto the host machine, slightly editing the help docs to make it a bit easier to use in the process.