mgr/dashboard: Language dropdown box is partly hidden on login page
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
mgr/cephadm: try to get FQDN for configuration files
Reviewed-by: Adam King <adking@redhat.com>
Reviewed-by: Michael Fritch <mfritch@suse.com>
Reviewed-by: Patrick Seidensal <pseidensal@suse.com>
Reviewed-by: Paul Cuzner <pcuzner@redhat.com>
Reviewed-by: Volker Theile <vtheile@suse.com>
The default pids-limit (docker 4096/podman 2048) prevent some
customization from working (http threads on RGW) or limits the number
of luns per iscsi target.
Fixes: https://tracker.ceph.com/issues/52898
Signed-off-by: Teoman ONAY <tonay@redhat.com>
Now, one session cannot support concurrent and it will lead to crash.
So there are mutil session using. At same time, it also can improve
the performance.
Fixes: https://tracker.ceph.com/issues/54361
Signed-off-by: Feng Hualong <hualong.feng@intel.com>
Commit 08df6e0fd0 ("qa/workunits/rbd: expand LevelSpec parsing
coverage") didn't account for images with a separate data pool. This
was missed because of small-cache-pool.yaml breakage.
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
cls/rgw: rgw_dir_suggest_changes detects race with completion
Reviewed-by: Matt Benjamin <mbenjami@redhat.com>
Reviewed-by: Mark Kogan <mkogan@redhat.com>
Reviewed-by: J. Eric Ivancich <ivancich@redhat.com>
rgw: Update "CEPH_RGW_DIR_SUGGEST_LOG_OP" for remove entries
Reviewed-by: Casey Bodley <cbodley@redhat.com>
Reviewed-by: Adam C. Emerson <aemerson@redhat.com>
Reviewed-by: Matt Benjamin <mbenjami@redhat.com>
There are two different ways to get the host address. From the
inventory of the mgr object directly or via the `_inventory_get_addr`
method of `CephadmService`. Update the code in order to use the
`_inventory_get_addr` method only.
Signed-off-by: Tatjana Dehler <tdehler@suse.com>
It's needed to addresses a FTBFS due to the Seastar's
no-locking-when-throwing hack.
Tags: seastar submodule
Signed-off-by: Radosław Zarzyński <rzarzyns@redhat.com>
It's necessary since 710a1bfdc02202fe9e59df8ea31de5b82b893fb4
in Seastar.
This change is a part of ongoing upgrade of Seastar which will
be completed in a follow-up PR, after merging another change
with the Seastar's upstream.
Signed-off-by: Radosław Zarzyński <rzarzyns@redhat.com>
iovec have unsigned length (size_t) and before this patch the
total length was computed by adding iovec's length to a signed
length variable (ssize_t). While the code checked if the resulting
length was negative on overflow, the case where length is positive
after overflow was not checked. This patch fixes the overflow check
by changing length to unsigned size_t.
Additionally, this patch fixes the case where some iovecs have been
added to the bufferlist and the aio completion has been blocked, but
adding an additional iovec fails because of overflow. This leads to
the UserBufferDeleter trying to unblock the completion on destruction
of the bufferlist but asserting because the completion was never
armed. We avoid this by first computing the total length and checking
for overflows and iovcnt before adding them to the bufferlist.
Signed-off-by: Jonas Pfefferle <pepperjo@japf.ch>
This PR changes the name "master" to "main" so
that builds (and, I assume, a great many other
things) will not fail.
Signed-off-by: Zac Dover <zac.dover@gmail.com>
On review, this constraint was correct--it does reliably prevent
same-cycle re-runs when a lc threads rendezvous on a bucket.
Also, for concurrent (or stale) and already processed buckets,
remember to advance head past the corresponding buckets.
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
Restore (and robustify) the assertion that, in general, each bucket
shard should be processed once per scheduling cycle.
If the prior cycle did not finish, processing in the current cyhcle
will continue from the marker where the last cycle left off.
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
