mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joseph Wu <jos...@mesosphere.io>
Subject Re: Review Request 69514: Added gauge metric for operator event stream subscribers.
Date Mon, 10 Dec 2018 22:24:19 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69514/
-----------------------------------------------------------

(Updated Dec. 10, 2018, 2:24 p.m.)


Review request for mesos, Benno Evers, Benjamin Mahler, Gastón Kleiman, and Greg Mann.


Changes
-------

Rename!!!  No more `active_` prefix, to avoid having to answer the question of what an `inactive_`
subscriber could be.


Bugs: MESOS-9258
    https://issues.apache.org/jira/browse/MESOS-9258


Repository: mesos


Description (updated)
-------

This metric "master/operator_event_stream_subscribers" returns
the total number of subscribers to the master's operator event stream.


Diffs (updated)
-----

  docs/monitoring.md 00c6ea94bcb73746aef740236632ede123f5b534 
  docs/operator-http-api.md 6edf361047ee5915514da892c6d885b74a510a72 
  src/master/master.cpp ae5b24093156a6ba8ffa3e94a46e600eb4def5fa 
  src/master/metrics.hpp f6bb89da9394bfe469bc690ff2de66824e994098 
  src/master/metrics.cpp f69ed52c1c89912e7a5d26cb9409f5959b09111a 
  src/tests/api_tests.cpp edde48a4b4b46c9f47bc44de389c97b77322b8e8 
  src/tests/master_tests.cpp 3bff7a13bb8013ef91b72c14ace23ee94bc190e9 


Diff: https://reviews.apache.org/r/69514/diff/3/

Changes: https://reviews.apache.org/r/69514/diff/2-3/


Testing
-------

```
make check

src/mesos-tests --gtest_filter="*MaxEventStreamSubscribers*" --gtest_repeat=-1 --gtest_break_on_failure
```


Thanks,

Joseph Wu


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message