]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
qa/standalone/mon: remove --mon-inital-members setting
authorKamoltat <ksirivad@redhat.com>
Wed, 30 Nov 2022 22:51:02 +0000 (22:51 +0000)
committerKamoltat <ksirivad@redhat.com>
Fri, 9 Dec 2022 15:43:45 +0000 (15:43 +0000)
commit4d8b0c29bf6a8295ae793a555d26fd48f8a178ae
tree6c80408ec23c845e3c0db7c84df40a47a1338fc8
parent767a4be12d64c42f5857cf390bb1d868cab96c72
qa/standalone/mon: remove --mon-inital-members setting

Problem:

--mon-initial-members does nothing but causes monmap
to populate ``removed_ranks`` because the way we start
monitors in standalone tests uses ``run_mon $dir $id ..``
on each mon. Regardless of --mon-initial-members=a,b,c, if
we set --mon-host=$MONA,$MONB,$MONC (which we do every single tests),
everytime we run a monitor (e.g.,run mon.b) it will pre-build
our monmap with

```
noname-a=mon.noname-a addrs v2:127.0.0.1:7127/0,
b=mon.b addrs v2:127.0.0.1:7128/0,
noname-c=mon.noname-c addrs v2:127.0.0.1:7129/0,
```

Now, with --mon-initial-members=a,b,c we are letting
monmap know that we should have initial members name:
a,b,c, which we only have `b` as a match. So what
``MonMap::set_initial_members`` do is that it will
remove noname-a and noname-c which will
populate `removed_ranks`.

Solution:

remove all instances of --mon-initial-members
in the standalone test as it has no impact on
the nature of the tests themselves.

Fixes: https://tracker.ceph.com/issues/58132
Signed-off-by: Kamoltat <ksirivad@redhat.com>
qa/standalone/ceph-helpers.sh
qa/standalone/mon/misc.sh
qa/standalone/mon/mon-bind.sh
qa/standalone/mon/mon-handle-forward.sh