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

cargohold compat target for testing different S3-like buckets #1210

Merged
merged 1 commit into from
Sep 23, 2020

Conversation

jschaul
Copy link
Member

@jschaul jschaul commented Sep 22, 2020

If testing against S3, minio, and others, this allows to run

make compat-minio
make compat-s3
...

Requires a compat folder (currently symlinked to a private repository containing some credentials)

Copy link
Contributor

@mheinzel mheinzel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Introduces more conditionals/complexity in integration.sh, but that should be worth it. Before merging, let me just make sure it works for me.

run spar "" ${orange}
run federator "" ${blue}
if [[ $INTEGRATION_CARGOHOLD_ONLY_COMPAT -eq 1 ]]; then
source "${CARGOHOLD_COMPAT_CONFIG_FOLDER}/env.sh"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could also do relevant parts of check_prerequisites here, but not a big deal now that this is not run routinely.

Copy link
Contributor

@mheinzel mheinzel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was able to run it successfully!

@jschaul jschaul merged commit 8daf281 into develop Sep 23, 2020
@jschaul jschaul deleted the cargohold-integration branch September 23, 2020 14:50
@jschaul jschaul mentioned this pull request Oct 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants