Skip to content

[FIXED] Fix lack of check for priority groups on push consumers#7053

Merged
neilalexander merged 1 commit intomainfrom
add-check-for-consumer-groups
Jul 11, 2025
Merged

[FIXED] Fix lack of check for priority groups on push consumers#7053
neilalexander merged 1 commit intomainfrom
add-check-for-consumer-groups

Conversation

@Jarema
Copy link
Copy Markdown
Member

@Jarema Jarema commented Jul 11, 2025

Signed-off-by: Tomasz Pietrek tomasz@synadia.com

@Jarema Jarema requested a review from a team as a code owner July 11, 2025 12:00
@Jarema Jarema requested a review from ripienaar July 11, 2025 12:00
Copy link
Copy Markdown
Contributor

@ripienaar ripienaar left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Tomasz Pietrek <tomasz@synadia.com>
@Jarema Jarema force-pushed the add-check-for-consumer-groups branch from 0db4815 to bad2dc9 Compare July 11, 2025 12:14
@ripienaar ripienaar requested a review from neilalexander July 11, 2025 12:31
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

Copy link
Copy Markdown
Member

@neilalexander neilalexander left a comment

Choose a reason for hiding this comment

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

Can we please add a unit test that proves we now can't set this configuration?

@Jarema
Copy link
Copy Markdown
Member Author

Jarema commented Jul 11, 2025

@neilalexander it is added here:

{"Push consumer with Priority Group", nc, "TEST", "PUSH_WITH_POLICY", []string{"A"}, PriorityOverflow, "subject", &ApiError{ErrCode: uint16(JSConsumerPushWithPriorityGroupErr)}},

Copy link
Copy Markdown
Member

@neilalexander neilalexander left a comment

Choose a reason for hiding this comment

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

Apologies, I misinterpreted that on the review. LGTM!

@neilalexander neilalexander merged commit 6932e7b into main Jul 11, 2025
111 of 114 checks passed
@neilalexander neilalexander deleted the add-check-for-consumer-groups branch July 11, 2025 14:02
@wallyqs wallyqs changed the title Fix lack of check for priority groups on push consumers [FIXED] Fix lack of check for priority groups on push consumers Jul 24, 2025
neilalexander added a commit that referenced this pull request Jul 25, 2025
Includes the following:

- #7031
- #7033
- #7034
- #7035
- #7036
- #7040
- #7043
- #7045
- #7047
- #7046
- #7050
- #7051
- #7052
- #7053
- #7061
- #7063
- #7064
- #7065
- #7066
- #7070
- #7072
- #7080
- #7026
- #6728
- #7074
- #7089
- #7095
- #7087
- #7094
- #7096
- #7099

Signed-off-by: Neil Twigg <neil@nats.io>
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