rgw: fix missing tenant prefix in bucket name during bucket link
Reviewed-by: Casey Bodley <cbodley@redhat.com>
Reviewed-by: J. Eric Ivancich <ivancich@redhat.com>
Before this commit, backport-resolve-issue would blindly grab the first tracker
URL it saw in the GitHub PR description. With this commit, it examines all the
tracker URLs it sees in the PR description and uses the first one that
corresponds to an issue in the Redmine Backport tracker.
Signed-off-by: Nathan Cutler <ncutler@suse.com>
The `navigateTo` method will check if a table exists on the table it has
been used to navigate to and if so, wait for the data of the table to
be loaded.
Fixes: https://tracker.ceph.com/issues/41349
Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
Needed to adapt the backend to return the type of the tree nodes as the
previous selector did not work reliably.
Fixes: https://tracker.ceph.com/issues/40693
Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
to prevent Proctator from hanging indefinitely when testing pages where
the feature toggles are retrieved.
Fixes: mgr/dashboard: Switch to async/await for E2E tests
Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
Ceph Dashboard replaces the original checkbox with a fancy replacement
using the pseudo selector "::before" inside the <label> tag. When the
fancy replacement checkbox is clicked, it actually triggers a click on
the label of the checkbox. `clickCheckbox` does the same.
Removed `moveClick` as it was used to perform an action that the user
would not be able to perform, namely clicking on the checkbox instead of
the checkboxes' label. It was also used for cases where it was not
necessary to be used for, hence the replacement `clickCheckbox` has been
introduced. It checks if the argument passed to it is of type `checkbox`
or `label`. If it is of type `label`, the label will be clicked. If it
is of type `checkbox`, the corresponding `label` is determined and
clicked. This replaces the old implementation with a solution that has
an easy to understand purpose. It also prevents every false application.
This method is designed to be used on all checkboxes of Ceph Dashboard.
Fixes: https://tracker.ceph.com/issues/40693
Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
and migrate `protractor.conf.js` to use async/await.
Fixes: https://tracker.ceph.com/issues/40693
Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
* check libfmt version instead of the exisistence of chrono.h, as
libfmt merged fmt/time.h into fmt/chrono.h since v 6.0, while in
libfmt v5.3, fmt/chrono.h did not provide all the necessary bits
for fmt::to_format(..., const tm&). see also
a939c75956
and https://github.com/fmtlib/fmt/blob/master/ChangeLog.rst
* use fmt API instead of libc API for getting `localtime()`, it's
more C++ friendly than plain `localtime_r()`
Signed-off-by: Kefu Chai <kchai@redhat.com>
test/librbd: set nbd timeout due to newer kernels defaulting it on
Reviewed-by: Mike Christie <mchristi@redhat.com>
Reviewed-by: Mykola Golub <mgolub@suse.com>
* v2 api: remove variadic futures. let's be in sync with upstream.
* warn if the returned future is ignored. this helps us to catch
bugs and to improve the readability of the code -- we need to
explicitly ignore a future if we want to. and ideally, we should
document the reason why a future is ignored.
Signed-off-by: Kefu Chai <kchai@redhat.com>
