Skip to content

Add direct-consumers flag to jsz monitoring endpoint#7543

Merged
neilalexander merged 1 commit intomainfrom
neil/jszdirectconsumers
Nov 11, 2025
Merged

Add direct-consumers flag to jsz monitoring endpoint#7543
neilalexander merged 1 commit intomainfrom
neil/jszdirectconsumers

Conversation

@neilalexander
Copy link
Copy Markdown
Member

It is currently not possible to see direct consumers that are used by sources or mirrors. This extends jsz to make it possible to see these direct consumers to help when diagnosing sourcing/mirroring issues.

Signed-off-by: Neil Twigg neil@nats.io

Signed-off-by: Neil Twigg <neil@nats.io>
@neilalexander neilalexander marked this pull request as ready for review November 11, 2025 13:12
@neilalexander neilalexander requested a review from a team as a code owner November 11, 2025 13:12
Accounts bool `json:"accounts,omitempty"`
Streams bool `json:"streams,omitempty"`
Consumer bool `json:"consumer,omitempty"`
DirectConsumer bool `json:"direct_consumer,omitempty"`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why a new field and not just include them in the results? They'd have Direct:true set?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think only if you have ?config=true set, otherwise you can't tell them apart.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ah, yeah good call though unforunate but suppose best we can do for now

Copy link
Copy Markdown
Member

@MauriceVanVeen MauriceVanVeen left a comment

Choose a reason for hiding this comment

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

LGTM

@neilalexander neilalexander merged commit e4b21f1 into main Nov 11, 2025
131 of 136 checks passed
@neilalexander neilalexander deleted the neil/jszdirectconsumers branch November 11, 2025 14:23
neilalexander added a commit that referenced this pull request Nov 12, 2025
Includes the following:

- #7535
- #7536
- #7538
- #7539
- #7440
- #7543

Signed-off-by: Neil Twigg <neil@nats.io>
neilalexander added a commit that referenced this pull request Nov 12, 2025
Includes the following:

- #7534
- #7535
- #7536
- #7537
- #7538
- #7539
- #7543
- #7456

Signed-off-by: Neil Twigg <neil@nats.io>
@alexbozhenko
Copy link
Copy Markdown
Member

@neilalexander should we include that in jsz as well, to include in audit gathers by default?

@alexbozhenko
Copy link
Copy Markdown
Member

oh, nevermind, it is there: nats-io/jsm.go#737

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.

4 participants