Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-5275

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

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: 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.

      No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

        alkondratenko Aleksey Kondratenko (Inactive) created issue -
        Hide
        thuan 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 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
        alkondratenko Aleksey Kondratenko (Inactive) added a comment -

        Done

        Show
        alkondratenko Aleksey Kondratenko (Inactive) added a comment - Done
        alkondratenko Aleksey Kondratenko (Inactive) made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        farshid Farshid Ghods (Inactive) made changes -
        Summary Poor man's alerts can be 'prematurely eaten' sometimes some of the alerts such as moxi crashing are not displayed in the web console
        Hide
        alkondratenko Aleksey Kondratenko (Inactive) 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
        alkondratenko Aleksey Kondratenko (Inactive) 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.
        alkondratenko Aleksey Kondratenko (Inactive) made changes -
        Summary some of the alerts such as moxi crashing are not displayed in the web console Poor man's alerts can be 'prematurely eaten' sometimes
        Description Anything that calls menelaus_alerts_srv:fetch_alerts will reset current list of alerts. Causing e.g. moxi to eat alerts without actually showing anything to user. 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.
        Hide
        farshid Farshid Ghods (Inactive) 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 Farshid Ghods (Inactive) 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
        alkondratenko Aleksey Kondratenko (Inactive) 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
        alkondratenko Aleksey Kondratenko (Inactive) 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:
            alkondratenko Aleksey Kondratenko (Inactive)
            Reporter:
            alkondratenko Aleksey Kondratenko (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes