The overrides will be loaded when we connect to the cluster via librados
and the current approach prevents running 'rbd help' without a running
cluster.
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
get_partition_dev() is a function called to get a block device path.
The @retry decorator implements a timeout mecanism as it exist a race
between the partition creation command and the actual presence of the
partition.
This function have a semantic issue.
The function is asked if /dev/sda6 exists but check if /sys/block/sda/sda6 exists.
Once /sys/block/sda/sda6 is populated within the timeout, the function returns "/dev/sda6".
But this doesn't check if "/dev/sda6" really exists, so a function
using this value fall into a race condition and fails too.
We do have this case inside the CI where timing are very variable
regarding the workload and VM's performance and even sometimes nasted VMs.
This patch offer to double-check before returning that the actual /dev/
entry actually exist once the BLOCKDIR (/sys) is populated. If not, the
@retry is triggered to let some time resolution this temporary situation.
If this state is permanent, an explicit error message is reported like :
/dev/sdb1 is not populated while /sys/block/sdb has it
Signed-off-by: Erwan Velu <erwan@redhat.com>
...not stderr. Seems more reasonable as this
command is essentially feeding back a measurement result,
rather than just commenting on its status
Fixes: http://tracker.ceph.com/issues/24022
Signed-off-by: John Spray <john.spray@redhat.com>
Telegraf is a agent for collecting and reporting metrics.
It has multiple inputs and can send data to various outputs like
for example InfluxDB or ElasticSearch.
This module works by using the socket_listener of Telegraf and can
send data over UDP, TCP and a local Unix Socket.
Signed-off-by: Wido den Hollander <wido@42on.com>
The bootstrap process cannot immediately quit if it notices the remote
image is not primary. Instead, it needs to continue if the local image is
still chained to the remote.
Fixes: http://tracker.ceph.com/issues/24009
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
the crypto_plugins target was defined in
src/crypto/isa-l/CMakeLists.txt, but this is only included
if(HAVE_INTEL AND HAVE_BETTER_YASM_ELF64 AND (NOT APPLE))
moving it out of the if() block allows the os target to depend on it
even if no plugins are built
Signed-off-by: Casey Bodley <cbodley@redhat.com>
This will force the use of prettier on frontend code,
bringing a more consistent formatting to all the code.
The current configuration will apply the prettier formatting,
on all staged files, during the precommit hook.
Signed-off-by: Tiago Melo <tmelo@suse.com>
mgr/dashboard: Use HTTPS in dev proxy configuration and HACKING.rst
Reviewed-by: Lenz Grimmer <lgrimmer@suse.com>
Reviewed-by: Ricardo Marques <rimarques@suse.com>
Reviewed-by: Tatjana Dehler <tdehler@suse.com>
Reviewed-by: Tiago Melo <tmelo@suse.com>
Now all error notifications are processed on the api-interceptor-service,
removing the need to handle them on every api request.
Signed-off-by: Tiago Melo <tmelo@suse.com>
- Do some polishing in the docs.
- Refactor RgwClient::is_service_online() method. The system is considered as online if the response structure is valid. The response content itself is not validated in this case.
- Relocate NoCredentialsException and derive it from RequestException.
Signed-off-by: Volker Theile <vtheile@suse.com>
(cherry picked from commit ad997b54c2)
If MDCache::rejoin_gather is empty and MDCache::rejoins_pending is true
when MDCache::process_imported_caps() calls maybe_send_pending_rejoins()
Both MDCache::rejoin_send_rejoins() and MDCache::process_imported_caps()
may call rejoin_gather_finish().
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
Fixes: http://tracker.ceph.com/issues/24047
