-
Notifications
You must be signed in to change notification settings - Fork 6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pacific: mgr/dashboard: "Please expand your cluster first" shouldn't be shown if cluster is already meaningfully running #45044
Conversation
jenkins test dashboard |
jenkins test dashboard cephadm |
@nizamial09 visualTests/dashboard.vrt-spec.ts is failing here, do we need to backport #44956 to Pacific, too? See your comment #44951 (comment)
|
Made some changes in the applitools and its passing now. |
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
615171f
to
2cbd402
Compare
…if cluster is already meaningfully running This PR will assume that a cluster is already up and fully running. If this should not be the expected behaviour, deployment tools have to set 'INSTALLED' explicitly. Without this assumption it might happen that upgraded and fully running clusters, e.g. Octopus -> Pacific, will show the 'Expand Cluster' on first log in. cephadm will take care that the bootstrap phase will write the necessary key to show the 'Expand cluster' page. Fixes: https://tracker.ceph.com/issues/54215 Signed-off-by: Volker Theile <vtheile@suse.com> (cherry picked from commit 48fff60) Conflicts: src/cephadm/cephadm Signed-off-by: Volker Theile <vtheile@suse.com>
2cbd402
to
7a8371f
Compare
jenkins test dashboard |
jenkins test make check |
It seems the Pacific branch has been broken by another PR that is RBD related. |
jenkins test make check |
jenkins test dashboard |
@epuertat Can this be merged after all checks are successful? |
backport tracker: https://tracker.ceph.com/issues/54284
backport of #44951
parent tracker: https://tracker.ceph.com/issues/54215
this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh