Skip to content

Remove Puma notices from AdminArea banner

Qingyu Zhao requested to merge remove-puma-notices-from-admin-area-banner into master

What does this MR do?

These two notices are removed:

  • You are running Puma, which is currently experimental. More information is available in our documentation
  • Puma is running with a thread count above 1. Information on deprecated GitLab features in this configuration is available in the documentation

More background why they are removed:

Currently, we detect and display 3 messages(1st and 2nd are notices, the 3rd is a warning):

  1. Puma is the application server
  2. Puma multi-threaded(thread>1) in use
  3. Rugged enabled and multi-threaded(thread>1) Puma in use

Since most cases we expect Puma running with thread count > 1, so the 1st and 2nd messages will show: image

The 2nd message(notice) is a bit redundant, because: for those users who are not using Rugged, we actually hope they use multi-threaded Puma(thread count > 1); for those users who are using Rugged, we will show the 3rd message.

The 1st message(notice) was there during Puma is experimental. Now we consider Puma is official since Gitlab.com switched to Puma. What is more, the linked-to documentation mainly talks about Rugged + multithread Puma. So it actually does not help much unless the deployment is Rugged + multi-thread Puma in which case the 3rd message(warning) will show anyway.

So the notices(1st and 2nd messages) are not necessary. This MR remove Puma notices from AdminArea banner

Conformity

Edited by Qingyu Zhao

Merge request reports

Loading