Skip to content

Conversation

@d-gubert
Copy link
Member

@d-gubert d-gubert commented Jul 28, 2025

Proposed changes (including videos or screenshots)

Decrease the log level in the /apps/installed endpoint when called with includeClusterStatus but deployment is not multi instance, to prevent verbosity by default.

Issue(s)

Steps to test or reproduce

Further comments

ALT-29

@changeset-bot
Copy link

changeset-bot bot commented Jul 28, 2025

⚠️ No Changeset found

Latest commit: af21ef9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Jul 28, 2025

Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

@d-gubert d-gubert added this to the 7.9.0 milestone Jul 28, 2025
@github-actions
Copy link
Contributor

PR Preview Action v1.6.2

🚀 View preview at
https://RocketChat.github.io/Rocket.Chat/pr-preview/pr-36549/

Built to branch gh-pages at 2025-07-28 17:02 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@codecov
Copy link

codecov bot commented Jul 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.71%. Comparing base (409ef45) to head (af21ef9).
⚠️ Report is 1 commits behind head on release-7.9.0.

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                @@
##           release-7.9.0   #36549      +/-   ##
=================================================
- Coverage          65.71%   65.71%   -0.01%     
=================================================
  Files               3172     3172              
  Lines             105574   105574              
  Branches           20036    20031       -5     
=================================================
- Hits               69381    69377       -4     
- Misses             33521    33527       +6     
+ Partials            2672     2670       -2     
Flag Coverage Δ
e2e 58.10% <ø> (+<0.01%) ⬆️
unit 70.97% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@d-gubert d-gubert marked this pull request as ready for review July 28, 2025 17:58
@MartinSchoeler MartinSchoeler added the stat: QA assured Means it has been tested and approved by a company insider label Jul 28, 2025
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Jul 28, 2025
@kodiakhq kodiakhq bot merged commit 61605f0 into release-7.9.0 Jul 28, 2025
53 checks passed
@kodiakhq kodiakhq bot deleted the regression/change-apps-cluster-warning-to-debug branch July 28, 2025 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: QA assured Means it has been tested and approved by a company insider stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants