Skip to content

Commit

Permalink
Add confirmation for additional stanchion stats items
Browse files Browse the repository at this point in the history
  • Loading branch information
@shino
shino committed Jul 28, 2015
1 parent 1fca610 commit 6d41f7d
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion riak_test/tests/stats_test.erl
View file
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,11 @@ confirm_initial_stats(stanchion, UserConfig, Port) ->
confirm_stat_count(Stats, <<"user_create_one">>, 1),
confirm_stat_count(Stats, <<"riakc_put_cs_user_one">>, 1),

?assert(proplists:is_defined(<<"memory_atom">>, Stats)),
?assert(proplists:is_defined(<<"waiting_time_mean">>, Stats)),
?assert(proplists:is_defined(<<"waiting_time_median">>, Stats)),
?assert(proplists:is_defined(<<"waiting_time_95">>, Stats)),
?assert(proplists:is_defined(<<"waiting_time_99">>, Stats)),
?assert(proplists:is_defined(<<"waiting_time_100">>, Stats)),
?assert(proplists:is_defined(<<"sys_process_count">>, Stats)),
?assert(proplists:is_defined(<<"webmachine_mochiweb_active_sockets">>, Stats)),
ok.
Expand Down

6 comments on commit 6d41f7d

@kuenishi
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@borshop
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from kuenishi
at 6d41f7d

@borshop
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

basho/riak_cs/feature/stanchion-stats-test = 6d41f7d merged ok, testing candidate = c6b9620

@borshop
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging basho/riak_cs/feature/stanchion-stats-test = 6d41f7d into borshop-integration-1199-feature/stanchion-stats-test

@borshop
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1417

@borshop
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding develop to borshop-integration-1199-feature/stanchion-stats-test = c6b9620

Please sign in to comment.