ceph: factor out get/broadcast ceph versions to ceph::services

which also removes some dead code
(the my $local_last_version variable was never used)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Dominik Csapak 2020-03-16 13:55:59 +01:00 committed by Thomas Lamprecht
parent 59807d1e3b
commit 0496138e44
3 changed files with 33 additions and 22 deletions

View File

@ -74,10 +74,8 @@ __PACKAGE__->register_method ({
version => PVE::Cluster::get_node_kv("ceph-version"),
};
if (defined(my $vers = PVE::Cluster::get_node_kv("ceph-versions"))) {
$res->{node} = {
map { eval { $_ => decode_json($vers->{$_}) } } keys %$vers
};
if (defined(my $vers = PVE::Ceph::Services::get_ceph_versions())) {
$res->{node} = $vers;
}
for my $type ( qw(mon mgr mds) ) {

View File

@ -47,6 +47,36 @@ sub broadcast_ceph_services {
}
}
sub broadcast_ceph_versions {
my ($version, $buildcommit, $vers_parts) = PVE::Ceph::Tools::get_local_version(1);
if ($version) {
# FIXME: remove with 7.0 - for backward compat only
PVE::Cluster::broadcast_node_kv("ceph-version", $version);
my $node_versions = {
version => {
str => $version,
parts => $vers_parts,
},
buildcommit => $buildcommit,
};
PVE::Cluster::broadcast_node_kv("ceph-versions", encode_json($node_versions));
}
}
sub get_ceph_versions {
my $res;
if (defined(my $vers = PVE::Cluster::get_node_kv("ceph-versions"))) {
$res = {
map { eval { $_ => decode_json($vers->{$_}) } } keys %$vers
};
}
return $res;
}
sub get_cluster_service {
my ($type) = @_;

View File

@ -465,24 +465,7 @@ sub update_ceph_metadata {
PVE::Ceph::Services::broadcast_ceph_services();
my ($version, $buildcommit, $vers_parts) = PVE::Ceph::Tools::get_local_version(1);
my $local_last_version = PVE::Cluster::get_node_kv('ceph-versions');
if ($version) {
# FIXME: remove with 7.0 - for backward compat only
PVE::Cluster::broadcast_node_kv("ceph-version", $version);
my $node_versions = {
version => {
str => $version,
parts => $vers_parts,
},
buildcommit => $buildcommit,
};
PVE::Cluster::broadcast_node_kv("ceph-versions", encode_json($node_versions));
}
PVE::Ceph::Services::broadcast_ceph_versions();
}
sub update_sdn_status {