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

Bucket replication config set, get, remove #934

Merged
merged 1 commit into from
Jun 2, 2021

Conversation

prakashsvmx
Copy link
Member

@prakashsvmx prakashsvmx commented May 26, 2021

Bucket replication set, get, remove APIs.

setBucketReplication(bucketName, replicationConfig={}, cb)
getBucketReplication(bucketName, cb)
removeBucketReplication(bucketName, cb)

Test Script:
test-replication.txt

Note: arn has to be generated manually and fed to the script to test. Steps are commented in the test script.
Place it inside of src/test and then npm run compile and then node src/test/test-replication.js

Note:
To run all functional tests locally the below must be exported and certificates as required:
export MINIO_KMS_KES_ENDPOINT=https://play.min.io:7373;export MINIO_KMS_KES_KEY_FILE=root.key;export MINIO_KMS_KES_CERT_FILE=root.cert;export MINIO_KMS_KES_KEY_NAME=my-minio-key;

@prakashsvmx prakashsvmx self-assigned this May 26, 2021
@prakashsvmx prakashsvmx changed the title Bucket replication set, get, remove Bucket replication config set, get, remove May 26, 2021
@prakashsvmx prakashsvmx force-pushed the bucket-replication-apis branch 2 times, most recently from 4d52939 to 4027c1b Compare May 27, 2021 07:57
Copy link
Contributor

@kannappanr kannappanr left a comment

Choose a reason for hiding this comment

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

LGTM

@kannappanr kannappanr merged commit d44419e into minio:master Jun 2, 2021
@prakashsvmx prakashsvmx deleted the bucket-replication-apis branch February 10, 2022 16:53
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