Skip to content

qa/suites/rados/thrash-old-clients: use cephadm#32377

Merged
liewegas merged 13 commits intoceph:masterfrom
liewegas:wip-old-clients-cephadm
Dec 23, 2019
Merged

qa/suites/rados/thrash-old-clients: use cephadm#32377
liewegas merged 13 commits intoceph:masterfrom
liewegas:wip-old-clients-cephadm

Conversation

@liewegas
Copy link
Member

  • deploy cluster with cephadm so we can run a octopus+ cluster and also
    install client packages that are ancient.
  • move client.2 back onto the third node, since packages no longer
    conflict.

Signed-off-by: Sage Weil sage@redhat.com

@liewegas liewegas force-pushed the wip-old-clients-cephadm branch from d03fdeb to 1d2cdbe Compare December 20, 2019 16:15
@liewegas liewegas force-pushed the wip-old-clients-cephadm branch 4 times, most recently from 3c8ae89 to fa3a59a Compare December 21, 2019 18:51
@liewegas liewegas requested a review from a team as a code owner December 21, 2019 18:56
@liewegas liewegas force-pushed the wip-old-clients-cephadm branch from 7356aa6 to 964d7b5 Compare December 21, 2019 23:56
If it's a v1 addr on the old default 6789 port, print it as a legacy-style
addr (no v1: prefix) so that legacy code can parse it.

Signed-off-by: Sage Weil <sage@redhat.com>
- deploy cluster with cephadm so we can run a octopus+ cluster and also
  install client packages that are ancient.
- move client.2 back onto the third node, since packages no longer
  conflict.
- test on centos 7.x (i picked 6), since the old releases all built on
  that release.

Signed-off-by: Sage Weil <sage@redhat.com>
Due to 6f5fb95

Signed-off-by: Sage Weil <sage@redhat.com>
Jewel and Hammer don't have python-rgw tho.

Signed-off-by: Sage Weil <sage@redhat.com>
Mostly just lifted from ceph.py

Signed-off-by: Sage Weil <sage@redhat.com>
Old distros don't have packages for bionic.

Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas force-pushed the wip-old-clients-cephadm branch 3 times, most recently from abe4c1c to ff12757 Compare December 23, 2019 00:20
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas force-pushed the wip-old-clients-cephadm branch from 3a7457f to a48f7c6 Compare December 23, 2019 20:01
@liewegas
Copy link
Member Author

This is passing almost all of the tests now (vs none in master). The last todo item i see is cleaning up the ceph-objecstore-tool code... working on that now, but I think what we have here so far should go in. The ceph-objectstore-tool piece is a big cleanup.

Signed-off-by: Sage Weil <sage@redhat.com>
ceph.py does this and ceph_manager.py kill_osd relies on it.

Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
This is more explicit.  More importantly, the 'mon update' command
can't handle an "ip:port"; it wants either a CIDR, bare IP, or addrvec.

Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas force-pushed the wip-old-clients-cephadm branch from 7c1cb2c to c14234e Compare December 23, 2019 22:35
liewegas added a commit that referenced this pull request Dec 23, 2019
* refs/pull/32377/head:
	qa/suites/rados/thrash-old-clients: configure mons in terms of addrvecs
	qa/suites/rados/thrash-old-clients: hammer: fix package list
	qa/tasks/cephadm: set .conf to cluster config object
	qa/tasks/cephadm: archive /var/log/ceph logs too (not just cluster dir)
	qa/tasks/cephadm: client keyring
	qa/tasks/cephadm: setup thrashers ctx item
	qa/tasks/ceph_manager: asok commands via cephadm shell
	qa/suites/rados/thrash-old-clients: stick to el7
	qa/tasks/cephadm: check cluster log; support log-whitelist
	qa/suites/rados/thrash-old-clienets: python-foo to python3-foo
	qa/suites/rados/thrash-old-clients: add new exclude_packages
	qa/suites/rados/thrash-old-clients: use cephadm
	mon/ConfigMonitor: make legacy mon addr/port parseable by legacy code

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
@liewegas liewegas merged commit c14234e into ceph:master Dec 23, 2019
@liewegas liewegas deleted the wip-old-clients-cephadm branch December 23, 2019 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants