Skip to content

Conversation

mjs
Copy link
Contributor

@mjs mjs commented Jun 23, 2025

Community Contribution License

All community contributions in this pull request are licensed to the project maintainers
under the terms of the Apache 2 license.
By creating this pull request I represent that I have the right to license the
contributions to the project maintainers under the Apache 2 license.

Description

Includes the remote bucket ARN in the default output of mc replicate list.

Sample output:
image

Motivation and Context

The replicate resync start command requires the remote bucket ARN. This is currently difficult to obtain and requires picking through the output of replicate list --json. Worse still, the JSON output doesn't contain the human-readable host for the bucket, making it difficult to know that you've got the correct one.

This change solves the issue by including the ARN in the default list output.

How to test this PR?

Set up a bucket replication rule and run mc replicate list <host>/<bucket>.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Optimization (provides speedup with no functional changes)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • Fixes a regression (If yes, please add commit-id or PR # here)
  • Unit tests added/updated
  • Internal documentation updated
  • Create a documentation update request here

This documentation will get somewhat simpler with this change: https://min.io/docs/minio/linux/administration/bucket-replication/server-side-replication-resynchronize-remote.html

I'll submit a doc update request if I know this PR is going to be approved.

Copy link
Contributor

@shtripat shtripat left a comment

Choose a reason for hiding this comment

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

lgtm

The "replicate resync start" command requires the remote bucket ARN.
This is currently difficult to obtain and requires picking through the
output of "replicate list --json". Worse still, the JSON output doesn't
contain the human-readable host for the bucket, making it difficult to
know that you've got the correct one.

This change solves the issue by including the ARN in the default list
output.
@mjs mjs force-pushed the replicate-list-arn branch from 5d2cd37 to 903c9d2 Compare June 23, 2025 20:49
@harshavardhana harshavardhana merged commit 039be57 into minio:master Jun 25, 2025
5 checks passed
@mjs mjs deleted the replicate-list-arn branch June 25, 2025 22:28
dormanze added a commit to dormanze/mc that referenced this pull request Jul 14, 2025
* 'ping' of https://github.com/dormanze/mc: (44 commits)
  mv:support set tag and checksum (minio#5133)
  ping: support ping a specified node and remove duplicate port printing (minio#5132)
  feat: add `--storage-class` option to `mc put` subcommand (minio#5228)
  mirror and cp support set the maximum number of threads (minio#5220)
  Some paths like url2Stat will send resource with backslashes on windows (minio#5226)
  Include ARN in "replicate list" output (minio#5215)
  Update build version (minio#5218)
  fix: add checks for nil HTTP in admin-trace matches func (minio#5216)
  remove fips 'mc' container Dockerfile
  Fix deletion of previously sync'd buckets and objects during mc-mirror (minio#5205)
  Add STATUS column to batch job list table output (minio#5202)
  fix: remove check if lock config exists before set (minio#5203)
  removing config cmd (minio#5201)
  fix: support chained mirror properly for delete propagation minio#4558 (minio#5197)
  Change support profile defaults (minio#5198)
  Explicitly set file permissions for admin-cluster-{bucket,iam}-export… (minio#5194)
  Add support for catalog batch job type (minio#5150)
  Add `idp openid accesskey` commands (minio#5182)
  feat: add deletemarker info for batch status (minio#5191)
  mirror: skip status message from excluded buckets (minio#5190)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants