Bug #18438

Allow adjusting notification checking period

Added by Thomas McKay over 1 year ago. Updated 9 days ago.

Status:Closed
Priority:Normal
Assignee:matan werbner
Category:Notifications
Target version:1.16.0
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link: Found in Releases:
Pull request:https://github.com/theforeman/foreman/pull/4520

Description

Checking for notifications every 10 seconds seems excessive.

At a minimum I'd like a way to increase that period on a per-organization basis.

Another idea would be to have the time between checks increase with idle activity. Start at 10 seconds but if no user interaction with UI, begin to bump up the interval to some max (perhaps every one minute?). This should be configurable as well.


Related issues

Related to Foreman - Feature #19535: Silence notification Rails logs New 05/15/2017
Duplicated by Foreman - Refactor #18961: Reduce frequency of notification polling Duplicate 03/20/2017

Associated revisions

Revision b8c9069d
Added by matan werbner about 1 year ago

Fixes #18438 - control notifications polling

History

#1 Updated by Gail Steiger over 1 year ago

I think it would be more worthwhile to implement websockets sooner rather than setting up an interim configuration mechanism.

#2 Updated by Ohad Levy over 1 year ago

Gail, websockets is probably not feasible in the near/immediate future

#3 Updated by Ohad Levy over 1 year ago

  • Duplicated by Refactor #18961: Reduce frequency of notification polling added

#4 Updated by The Foreman Bot about 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4520 added

#5 Updated by matan werbner about 1 year ago

  • Status changed from Ready For Testing to New
  • Assignee set to matan werbner
  • Pull request deleted (https://github.com/theforeman/foreman/pull/4520)

#6 Updated by Dominic Cleal about 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4520 added

#7 Updated by Lukas Zapletal about 1 year ago

#8 Updated by matan werbner about 1 year ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#9 Updated by Marek Hulán about 1 year ago

  • Legacy Backlogs Release (now unused) set to 240

Also available in: Atom PDF