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.

Sign up for GitHub

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

Jump to bottom

Lower user limit soft warning to 5K #27034

Merged
merged 7 commits into from
May 21, 2024
Merged

Lower user limit soft warning to 5K #27034

merged 7 commits into from
May 21, 2024

Conversation

marianunez
Copy link
Member

@marianunez marianunez commented May 16, 2024 •
edited

Summary

Lower soft user limit warning to 5K for TE and E0.

Ticket Link

https://mattermost.atlassian.net/browse/MM-58307

Release Note

Adjusted safety limit error message to show when users exceed 5,000 in compiled Team Edition and Enterprise Edition deployments when enterprise scale and access control automation features are unavailable. ERROR_SAFE_LIMITS_EXCEEDED.

@mm-cloud-bot
Copy link

@marianunez: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

I understand the commands that are listed here

@marianunez marianunez added the 2: Dev Review Requires review by a developer label May 16, 2024
@marianunez marianunez added this to the v9.9.0 milestone May 16, 2024
@mm-cloud-bot mm-cloud-bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed do-not-merge/release-note-label-needed labels May 16, 2024
@amyblais amyblais added the CherryPick/Approved Meant for the quality or patch release tracked in the milestone label May 16, 2024
@harshilsharma63
Copy link
Member

/update-branch

@amyblais amyblais added the Docs/Needed Requires documentation label May 17, 2024
@devinbinnie devinbinnie added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a developer labels May 17, 2024
@amyblais amyblais removed the request for review from M-ZubairAhmed May 17, 2024 13:31
@amyblais
Copy link
Member

/update-branch

@amyblais
Copy link
Member

/update-branch

@mattermost-build
Copy link
Contributor

Error trying to update the PR.
Please do it manually.

@amyblais
Copy link
Member

@marianunez There are some merge conflicts here.

@amyblais amyblais added the AutoMerge Used by Mattermod to merge PR automatically label May 21, 2024
@mattermost-build
Copy link
Contributor

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

@mattermost-build
Copy link
Contributor

Trying to auto merge this PR.

@mattermost-build mattermost-build merged commit 72fcedd into master May 21, 2024
46 checks passed
@mattermost-build
Copy link
Contributor

Pull Request successfully merged
SHA: 72fcedd

@mattermost-build mattermost-build deleted the lower-user-limit-warning branch May 21, 2024 17:41
@mattermost-build
Copy link
Contributor

Cherry pick is scheduled.

@mattermost-build mattermost-build removed the AutoMerge Used by Mattermod to merge PR automatically label May 21, 2024
mattermost-build pushed a commit that referenced this pull request May 21, 2024
@marianunez @mattermost-build
Lower user limit soft warning to 5K (#27034)
55aefdc 
Automatic Merge

(cherry picked from commit 72fcedd)
@mattermost-build mattermost-build mentioned this pull request May 21, 2024
Merged
@mattermost-build mattermost-build added CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone and removed CherryPick/Approved Meant for the quality or patch release tracked in the milestone labels May 21, 2024
@amyblais amyblais added the Changelog/Done Required changelog entry has been written label May 21, 2024
mattermost-build added a commit that referenced this pull request May 21, 2024
@mattermost-build
Merge branch 'release-9.9' into automated-cherry-pick-of-mattermost-#…
0ba0478 
…27034-upstream-release-9.9
mattermost-build added a commit that referenced this pull request May 21, 2024
@mattermost-build
Lower user limit soft warning to 5K (#27034) (#27088)
bee0d3a 
Automatic Merge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@harshilsharma63 harshilsharma63 harshilsharma63 approved these changes

@devinbinnie devinbinnie devinbinnie approved these changes

Assignees
No one assigned
Labels
4: Reviews Complete All reviewers have approved the pull request Changelog/Done Required changelog entry has been written CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone Docs/Needed Requires documentation release-note Denotes a PR that will be considered when it comes time to generate release notes.
Projects
None yet
Milestone
v9.9.0
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@marianunez @mm-cloud-bot @harshilsharma63 @amyblais @mattermost-build @devinbinnie