Couchbase Server

Poor man's alerts can be 'prematurely eaten' sometimes

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 1.8.0
  • Fix Version/s: 1.8.1
  • Component/s: ns_server
  • Security Level: Public
  • Labels:
    None

Description

Anything that calls menelaus_alerts_srv:fetch_alerts will reset current list of alerts.

Particularly it was possible for some other smart client or moxi to 'eat' alerts. Which would incorrectly mark them as shown to user and not effectively shown to user.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Thuan Nguyen added a comment -
Integrated in github-ns-server-2-0 #349 (See [http://qa.hq.northscale.net/job/github-ns-server-2-0/349/])
    explicitly consume poor man's alerts. MB-5275 (Revision 2cfb9473a69bfff634698893d54eab673ada09a3)

     Result = SUCCESS
Aliaksey Kandratsenka :
Files :
* src/menelaus_web.erl
* priv/public/js/hooks.js
* src/menelaus_web_alerts_srv.erl
* CHANGES
* priv/public/js/app.js
Show
Thuan Nguyen added a comment - Integrated in github-ns-server-2-0 #349 (See [ http://qa.hq.northscale.net/job/github-ns-server-2-0/349/ ])     explicitly consume poor man's alerts. MB-5275 (Revision 2cfb9473a69bfff634698893d54eab673ada09a3)      Result = SUCCESS Aliaksey Kandratsenka : Files : * src/menelaus_web.erl * priv/public/js/hooks.js * src/menelaus_web_alerts_srv.erl * CHANGES * priv/public/js/app.js
Hide
Permalink
Aleksey Kondratenko added a comment -
Done
Show
Aleksey Kondratenko added a comment - Done
Hide
Permalink
Aleksey Kondratenko added a comment -
I disagree _a lot_ with recent subject change. We really had problems with moxi 'eating' alerts. And causing them to not be shown to user.
Show
Aleksey Kondratenko added a comment - I disagree _a lot_ with recent subject change. We really had problems with moxi 'eating' alerts. And causing them to not be shown to user.
Hide
Permalink
Farshid Ghods added a comment -
thanks. can you please help in modifying the subject to make it easier to end user understand what was fixed. ( what alerts were not displayed and under what circumstances )
IMHP "poor man alert" is not descriptive enough for our end user
Show
Farshid Ghods added a comment - thanks. can you please help in modifying the subject to make it easier to end user understand what was fixed. ( what alerts were not displayed and under what circumstances ) IMHP "poor man alert" is not descriptive enough for our end user
Hide
Permalink
Aleksey Kondratenko added a comment -
Any alerts were could be potentially eaten. Poor man's alerts is how we call them. I have no idea about how docs call them
Show
Aleksey Kondratenko added a comment - Any alerts were could be potentially eaten. Poor man's alerts is how we call them. I have no idea about how docs call them

People

  • Assignee:
    Aleksey Kondratenko
    Reporter:
    Aleksey Kondratenko
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: