Skip to content

qa/cephtool: remove/deprecate proxy cache-tier mode#35139

Merged
neha-ojha merged 2 commits intoceph:masterfrom
smithfarm:wip-proxy-is-removed
May 21, 2020
Merged

qa/cephtool: remove/deprecate proxy cache-tier mode#35139
neha-ojha merged 2 commits intoceph:masterfrom
smithfarm:wip-proxy-is-removed

Conversation

@smithfarm
Copy link
Contributor

@smithfarm smithfarm commented May 19, 2020

NOTE: to be backported via https://tracker.ceph.com/issues/45514


Fixes: https://tracker.ceph.com/issues/45614
Fixes: a0a3ed3
Signed-off-by: Nathan Cutler ncutler@suse.com

@smithfarm
Copy link
Contributor Author

smithfarm commented May 19, 2020

@smithfarm smithfarm requested a review from jdurgin May 19, 2020 20:21
@smithfarm
Copy link
Contributor Author

smithfarm commented May 19, 2020

The 'proxy' cache-mode is deprecated and can no longer be used.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
@smithfarm smithfarm force-pushed the wip-proxy-is-removed branch 3 times, most recently from b648142 to 21647ff Compare May 20, 2020 16:08
@smithfarm
Copy link
Contributor Author

smithfarm commented May 20, 2020

@smithfarm smithfarm force-pushed the wip-proxy-is-removed branch from 21647ff to cc36fe2 Compare May 20, 2020 20:41
@smithfarm
Copy link
Contributor Author

@smithfarm
Copy link
Contributor Author

@neha-ojha @tchaikov @jdurgin This fixes the test broken by a0a3ed3

The "proxy" and "forward" cache-tier modes have been completely removed,
so it's sufficient to test once that they cannot be set.

Fixes: a0a3ed3
Signed-off-by: Nathan Cutler <ncutler@suse.com>
@smithfarm smithfarm force-pushed the wip-proxy-is-removed branch from cc36fe2 to 4f0cecb Compare May 21, 2020 11:42
@smithfarm
Copy link
Contributor Author

smithfarm commented May 21, 2020

pass http://pulpito.ceph.com/smithfarm-2020-05-21_11:42:36-rados:singleton-bluestore-wip-proxy-is-removed-distro-basic-smithi/

@neha-ojha
Copy link
Member

No cephtool related failures in rados run: http://pulpito.ceph.com/nojha-2020-05-21_02:01:24-rados-wip-35139-distro-basic-smithi/

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.

3 participants