Skip to content

ARTEMIS-6030 Expose Lock Coordinator Status through management and CLI#6380

Merged
clebertsuconic merged 1 commit intoapache:mainfrom
clebertsuconic:lock-list
Apr 23, 2026
Merged

ARTEMIS-6030 Expose Lock Coordinator Status through management and CLI#6380
clebertsuconic merged 1 commit intoapache:mainfrom
clebertsuconic:lock-list

Conversation

@clebertsuconic
Copy link
Copy Markdown
Contributor

No description provided.

@clebertsuconic clebertsuconic changed the title ARTEMIS-6030 Explose Lock Coordinator Status through management and CLI ARTEMIS-6030 Expose Lock Coordinator Status through management and CLI Apr 23, 2026
@clebertsuconic clebertsuconic force-pushed the lock-list branch 2 times, most recently from ca51ae6 to 16180d3 Compare April 23, 2026 18:24
}
}

private void printStats(ActionContext context, JsonArray array) {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I changed it to always use the context instead of getActionContext()

import picocli.CommandLine;
import picocli.CommandLine.Command;

@Command(name = "lock", description = "Lock coordinators group. use 'help lock' for sub commands list", subcommands = {LockList.class})
Copy link
Copy Markdown
Contributor Author

@clebertsuconic clebertsuconic Apr 23, 2026

Choose a reason for hiding this comment

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

Just a heads up, I plan to add Stop, Start, Release soon, hence the group here.

@jbertram
Copy link
Copy Markdown
Contributor

My only comment here would be consider adding something to the docs so folks know these management options exist.

@clebertsuconic
Copy link
Copy Markdown
Contributor Author

@jbertram I will add docs on a separate commit. Before we release. I'm also adding stop and start. So I will add it to support the whole workflow.

That's to support failback like scenarios. And passive DRs. If I document thst now I will have to rewrite it later.

@clebertsuconic clebertsuconic merged commit 5478549 into apache:main Apr 23, 2026
6 checks passed
@clebertsuconic clebertsuconic deleted the lock-list branch April 24, 2026 15: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