Skip to content
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

fix oldumps #6253

Merged
merged 3 commits into from
Mar 10, 2022
Merged

fix oldumps #6253

merged 3 commits into from
Mar 10, 2022

Conversation

mekarpeles
Copy link
Member

@mekarpeles mekarpeles commented Mar 7, 2022

@cclauss for you to review + update pls

These changes may not clean up files correctly after running -- we may want to add this part to the cron rather than this file (so when the cron is kicked off, it cleans up if the step is necessary). We don't want to remove the files the top of this script because it will negate the work of saving our place and gracefully restarting if we hit + fix an error

One error we had was around admin collections permission for the /olsystem ia.ini role user s3 account (which has since been fixed). It needed permission for ol_export

Q: Can we (e.g. using xauthn make a request to see what our permissions are as a e.g. regressions rest)
cc: @jimbonator

Technical

Testing

Stakeholders

@cclauss @cdrini

scripts/oldump.sh Outdated Show resolved Hide resolved
@mekarpeles mekarpeles mentioned this pull request Mar 7, 2022
2 tasks
@mekarpeles mekarpeles added the Priority: 1 Do this week, receiving emails, time sensitive, . [managed] label Mar 7, 2022
@mekarpeles
Copy link
Member Author

Reviewed w/ @cclauss

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 1 Do this week, receiving emails, time sensitive, . [managed]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants