Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

cluster_info removal #212

Closed
wants to merge 1 commit into from

2 participants

@evanmcc

Remove cluster info from the release and applications stuff.
Remove the call from riak-admin.

This change also relies upon:
basho/riak_kv#393
basho/riak_search#130
basho/riak_core#226
basho/riak_pipe#54

@evanmcc evanmcc referenced this pull request in basho/riak_pipe
Closed

remove cluster_info #54

@jaredmorrow

This removal would be with the assumption that riaknostic will be included by default, correct?

@evanmcc

Kind of. The idea was that cluster info would join riaknostic in our package of optional diagnostic tools, but since then we've come to the conclusion that it would go into riaknostic, since too many customer have complained about not being able to use riaknostic, since they can't install additional stuff in their security-restricted environments.

@jaredmorrow

@evanmcc I'm going to assume this is going to be post-1.4 considering none of the other PR's are closed yet?

@evanmcc
@evanmcc evanmcc closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Sep 10, 2012
  1. @evanmcc

    cluster_info removal

    evanmcc authored
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 22 deletions.
  1. +0 −1  rebar.config
  2. +2 −19 rel/files/riak-admin
  3. +0 −2  rel/reltool.config
View
1  rebar.config
@@ -9,7 +9,6 @@
{erl_opts, [debug_info, fail_on_warning]}.
{deps, [
- {cluster_info, ".*", {git, "git://github.com/basho/cluster_info", {branch, "master"}}},
{riak_kv, ".*", {git, "git://github.com/basho/riak_kv", {branch, "master"}}},
{riak_search, ".*", {git, "git://github.com/basho/riak_search",
{branch, "master"}}},
View
21 rel/files/riak-admin
@@ -378,23 +378,6 @@ case "$1" in
$NODETOOL rpc riak_core_console ring_status $@
;;
- cluster[_-]info)
- if [ $# -lt 2 ]; then
- echo "Usage: $SCRIPT $1 <output_file> ['local' | <node> ['local' | <node>] [...]]"
- exit 1
- fi
-
- # Make sure the local node IS running
- RES=`$NODETOOL ping`
- if [ "$RES" != "pong" ]; then
- echo "Node is not running!"
- exit 1
- fi
- shift
-
- $NODETOOL rpc_infinity riak_kv_console cluster_info $@
- ;;
-
services)
$NODETOOL rpcterms riak_core_node_watcher services ''
;;
@@ -597,8 +580,8 @@ case "$1" in
echo "Usage: $SCRIPT { cluster | join | leave | backup | restore | test | "
echo " reip | js-reload | erl-reload | wait-for-service | "
echo " ringready | transfers | force-remove | down | "
- echo " cluster-info | member-status | ring-status | vnode-status |"
- echo " diag | status | transfer-limit | "
+ echo " member-status | ring-status | vnode-status | diag |"
+ echo " status | transfer-limit | "
echo " top [-interval N] [-sort reductions|memory|msg_q] [-lines N] }"
exit 1
;;
View
2  rel/reltool.config
@@ -24,7 +24,6 @@
riak_kv,
riak_search,
riak_api,
- cluster_info,
lager,
riak_control,
erlydtl,
@@ -40,7 +39,6 @@
{excl_sys_filters, ["^bin/.*",
"^erts.*/bin/(dialyzer|typer)"]},
{excl_archive_filters, [".*"]},
- {app, cluster_info, [{incl_cond, include}]},
{app, erlang_js, [{incl_cond, include}]},
{app, luke, [{incl_cond, include}]},
{app, ebloom, []},
Something went wrong with that request. Please try again.