summaryrefslogtreecommitdiffstats
path: root/qa/tasks/mgr (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #58052 from jamiepryde/isal-defaultYuri Weinstein9 days1-3/+2
|\ | | | | | | | | | | | | core: Change the default plugin for Ceph erasure coded pools from Jerasure to ISA-L Reviewed-by: Afreen Misbah <afreen23.git@gmail.com> Reviewed-by: Mark Nelson <mnelson@redhat.com>
| * qa: Change default erasure code profile from jerasure to isaJamie Pryde2024-12-171-3/+2
| | | | | | | | Signed-off-by: Jamie Pryde <jamiepry@uk.ibm.com>
* | Merge pull request #60798 from rhcs-dashboard/edit-configuration-at-runtimeafreen232024-12-171-0/+12
|\ \ | | | | | | | | | | | | mgr/dashboard: Administration > Configuration > rgw_dns_name and rgw_dns_s3website_name not updatable via dashboard Reviewed-by: Afreen Misbah <afreen@ibm.com>
| * | mgr/dashboard: Administration > Configuration > Some of the config options ↵Naman Munet2024-12-161-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | are not updatable at runtime Fixes: https://tracker.ceph.com/issues/68976 Fixes Includes: 1) by-passing 'can_update_at_runtime' flag for 'rgw' related configurations as the same can be updated at runtime via CLI. Also implemented a warning popup for user to make force edit to rgw related configurations. 2) when navigated to Administration >> Configuration, modified configuration will be seen as we see in cli "ceph config dump", instead of configuration with filter level:basic Signed-off-by: Naman Munet <naman.munet@ibm.com>
* | | Merge pull request #60635 from cah-hbaum/fix-subuser-creation-via-dashboardafreen232024-12-161-2/+2
|\ \ \ | |/ / |/| | | | | | | | mgr: fix subuser creation via dashboard Reviewed-by: Afreen Misbah <afreen@ibm.com>
| * | mgr: fix subuser creation via dashboardHannes Baum2024-12-121-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Subusers couldn't be created through the dashboard, because the get call was overwritten with Python magic due to it being the function under the HTTP call. The get function was therefore split into an "external" and "internal" function, whereas one can be used by functions without triggering the magic. Since the user object was then returned correctly, json.loads could be removed. Signed-off-by: Hannes Baum <hannes.baum@cloudandheat.com>
* | | Merge pull request #60511 from rhcs-dashboard/health-test-fixNizamudeen A2024-11-113-7/+20
|\ \ \ | |/ / |/| | | | | | | | qa/dashboard: fix tasks.mgr.dashboard.test_health.HealthTest Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
| * | qa/dashboard: fix test_list_enabled_module failureNizamudeen A2024-11-083-5/+20
| | | | | | | | | | | | | | | | | | | | | | | | Check the ports availability and go for a new port if the current one is not available Fixes: https://tracker.ceph.com/issues/62972 Signed-off-by: Nizamudeen A <nia@redhat.com>
| * | qa/dashboard: fix tasks.mgr.dashboard.test_health.HealthTestNizamudeen A2024-10-281-3/+1
| | | | | | | | | | | | | | | | | | | | | as per: https://tracker.ceph.com/issues/47612#note-14 Fixes: https://tracker.ceph.com/issues/47612 Signed-off-by: Nizamudeen A <nia@redhat.com>
* | | mgr/zabbix: remove deprecated moduleErnesto Puerta2024-10-281-7/+0
| | | | | | | | | | | | | | | | | | | | | | | | This (already deprecated) module is removed as a side-effect of the deprecation and removal of the `restful` module. Fixes: https://tracker.ceph.com/issues/47066 Signed-off-by: Ernesto Puerta <epuertat@redhat.com>
* | | mgr/restful: remove deprecated moduleErnesto Puerta2024-10-281-2/+2
|/ / | | | | | | | | | | | | | | | | | | | | | | | | Detailed changes: * Remove `restful` mgr module dir, * Remove Python depedencies (`pecan`, `werkzeug`) from ceph.spec and debian control, * Remove docs, * Remove associated QA tests, * Update vstart. Fixes: https://tracker.ceph.com/issues/47066 Signed-off-by: Ernesto Puerta <epuertat@redhat.com>
* | mgr/dashboard: introduce server side pagination for osdsNizamudeen A2024-10-111-2/+3
| | | | | | | | | | Fixes: https://tracker.ceph.com/issues/56511 Signed-off-by: Nizamudeen A <nia@redhat.com>
* | Merge pull request #58456 from rhcs-dashboard/auth2-ssoPedro Gonzalez Gomez2024-09-161-2/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | mgr/dashboard: Add SSO through oauth2 protocol Reviewed-by: afreen23 <NOT@FOUND> Reviewed-by: Ernesto Puerta <epuertat@redhat.com> Reviewed-by: Nizamudeen A <nia@redhat.com> Reviewed-by: Redouane Kachach <rkachach@redhat.com>
| * | mgr/dashboard: add SSO through oauth2 protocolPedro Gonzalez Gomez2024-09-161-2/+4
| | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/66900 Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
* | | qa/tests: fix test_list_enabled_modules timeout errorNizamudeen A2024-09-041-1/+3
|/ / | | | | | | | | | | | | | | | | | | This test deals with enabling/disabling the modules. The assumption I have is after enabling the module test will wait for an active mgr but its not able to find it in time and it fails. so taking inspiration from https://github.com/ceph/ceph/pull/58995/commits/6c7253be6f6fbfa6faed7a539cb78847fec04580 adding retries and logs to see if that's the case Fixes: https://tracker.ceph.com/issues/62972 Signed-off-by: Nizamudeen A <nia@redhat.com>
* | qa/mgr/dashboard: fix test race conditionErnesto Puerta2024-08-052-13/+41
| | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/66844 Signed-off-by: Ernesto Puerta <epuertat@redhat.com>
* | Merge pull request #58386 from YiteGu/dashboard-rename-apiNizamudeen A2024-07-121-0/+12
|\ \ | | | | | | | | | | | | | | | mgr/dashboard: add cephfs rename REST API Reviewed-by: Dhairya Parmar <dparmar@redhat.com> Reviewed-by: Nizamudeen A <nia@redhat.com>
| * | mgr/dashboard: add cephfs rename REST APIyite.gu2024-07-041-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | Introduce rename for the cephfs REST API controller, we can rename the existing file or directory by it. Fixes: https://tracker.ceph.com/issues/66797 Signed-off-by: Yite Gu <yitegu0@gmail.com>
* | | Merge pull request #58224 from Serrini/fix-66490-mainNizamudeen A2024-07-111-0/+13
|\ \ \ | |/ / |/| | | | | | | | mgr/dashboard: add restful api for creating crush rule with type of 'erasure' Reviewed-by: Nizamudeen A <nia@redhat.com>
| * | mgr/dashboard: add restful api for creating crush rule with type of 'erasure'sunlan2024-07-101-0/+13
| | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/66490 Signed-off-by: sunlan <sunlan@asiainfo.com>
* | | Merge PR #53503 into mainPatrick Donnelly2024-06-251-0/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/53503/head: qa: add tests for `mds last-seen` command doc/cephfs: add documentation for `mds last-seen` PendingReleaseNotes: add note on last-seen command mon/MDSMonitor: add command to lookup when mds was last seen mon/MDSMonitor: set birth time on FSMap during encode pybind/mgr/dashboard: show context diff for openapi check Reviewed-by: Venky Shankar <vshankar@redhat.com>
| * | | mon/MDSMonitor: set birth time on FSMap during encodePatrick Donnelly2024-06-211-0/+1
| |/ / | | | | | | | | | | | | | | | | | | So we can begin to answer questions like: when did we last see an MDS? Fixes: https://tracker.ceph.com/issues/62849 Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
* | | Merge pull request #57401 from kamoltat/wip-ksirivad-fix-65826Yuri Weinstein2024-06-251-1/+9
|\ \ \ | | | | | | | | | | | | | | | | qa/tasks/mgr/test_progress.py: deal with pre-exisiting pool Reviewed-by: Laura Flores <lflores@redhat.com>
| * | | qa/tasks/mgr/test_progress.py: deal with pre-exisiting poolKamoltat2024-05-101-1/+9
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Currently, the test will fail we try to remove a pool that was not created by test_progress.py if we use the function `remove_pool` from class CephManager because it tracks the pools in a dictionary and there is an assert to check if the pool exists in the dictionary. Solution: Therefore, add a case where if there is a pool that was not created by the test, then we would delete it through raw command instead of using `remove_pool` from CephManager Fixes: https://tracker.ceph.com/issues/65826 Signed-off-by: Kamoltat <ksirivad@redhat.com>
* | | mgr/dashboard: select default daemon based on the default zonegroupNizamudeen A2024-06-211-33/+38
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | if multisite is configured, the default daemon needs to be selected based on the default zonegroup. Otherwise dashboard gives you incorrect details when doing the period commit The issue occurs when you do a period update --commit and you reload one of the block page, the api assigns the zonegroup of the second gateway because for a moment, the first gateway reflects the period changes... This is not true because the default zonegroup is of the previous active gateway but even though the back-end correctly says the active zonegroup, the dashboard api says it wrongly. Fixes: https://tracker.ceph.com/issues/66394 Signed-off-by: Nizamudeen A <nia@redhat.com>
* | pybind/rbd: expand what get_parent_image_spec() returnsIlya Dryomov2024-06-131-2/+12
|/ | | | | | | Propagate everything there is in rbd_linked_image_spec_t and rbd_snap_spec_t structures returned by the C API. Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
* Merge PR #56997 into mainPatrick Donnelly2024-04-303-7/+101
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/56997/head: pybind/mgr: disable sqlite3/python autocommit qa/tasks/mgr: add tests for sqlite autocommit qa/tasks/vstart_runner: run daemons in foreground qa/tasks/vstart_runner: add missing poll method qa/suites/rados/mgr: add cli/devicehealth tasks qa: reorganize mgr unit tests qa: use position-independent link qa: add missing terminating newline pybind/mgr: add killpoint for sqlite3 database setup mgr: allow specifying module option level mon/MgrMonitor: promote standby when unsetting down flag mon/MgrMonitor: only drop active if exists Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
| * qa/tasks/mgr: add tests for sqlite autocommitPatrick Donnelly2024-04-292-7/+100
| | | | | | | | | | | | | | That autocommit is properly turned off and that commits via context managers work as expected. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
| * mgr: allow specifying module option levelPatrick Donnelly2024-04-291-0/+1
| | | | | | | | | | | | Some are for development purposes and should be filtered out by the dashboard. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
* | qa/tasks: vstart_runner: introduce --config-modeLeonid Usov2024-04-251-4/+4
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new mode of the vstart_runner allows for passing paths to yaml configs that will be merged and then run just as the teuthology would do it. Building on the standard run method we can even pass "-" as the config name and provide one on the stdin like python3 ../qa/tasks/vstart_runner.py --config-mode "-" << END tasks: - quiescer: quiesce_factor: 0.5 min_quiesce: 10 max_quiesce: 10 initial_delay: 5 cancelations_cap: 2 paths: - a - b - c - waiter: on_exit: 100 END This commit does the minimum to allow testing of the quiescer, but it also lays the groundwork for running arbitrary configs. The cornerstone of the approach is to inject our local implementations of the main fs suite classes. To be able to do that, some minor refactoring was required in the corresponding modules: the standard classes were renamed to have a *Base suffix, and the former class name without the suffix is made a module level variable initialized with the *Base implementation. This refactoring is meant to be backward compatible. Signed-off-by: Leonid Usov <leonid.usov@ibm.com>
* qa: update dashboard schema for mon_statusPatrick Donnelly2024-03-191-1/+2
| | | | Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
* mds/quiesce: resolve the quiesce cluster at the mds monitorLeonid Usov2024-03-041-1/+3
| | | | Signed-off-by: Leonid Usov <leonid.usov@ibm.com>
* Merge PR #52258 into mainVenky Shankar2024-02-281-0/+14
|\ | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/52258/head: client: check mds down status bofore getting mds_gid_t from mdsmap mgr/dashboard: allow sending back error status code fetching clients fails Reviewed-by: Xiubo Li <xiubli@redhat.com> Reviewed-by: Venky Shankar <vshankar@redhat.com> Reviewed-by: Rishabh Dave <ridave@redhat.com> Reviewed-by: Patrick Donnelly <pdonnell@redhat.com> Reviewed-by: Nizamudeen A <nia@redhat.com> Reviewed-by: Dhairya Parmar <dparmar@redhat.com>
| * client: check mds down status bofore getting mds_gid_t from mdsmapyite.gu2024-02-261-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Get mds_gid_t from up of MDSMap, will cause to core dump if target mds_rank_t does not exist up map: from: src/mds/MDSMap.h const auto& get_info(mds_rank_t m) const { return mds_info.at(up.at(m)); } reproduct: 1. ceph fs fail <fs_name> 2. curl -X GET "example.com:8080/api/cephfs/1/clients" ... up.at(m) will cause to core dump. Fixes: https://tracker.ceph.com/issues/61844 Signed-off-by: Yite Gu <yitegu0@gmail.com>
* | tasks/.../test_erasure_code_profile: assertSubset in test_create_pluginSamuel Just2024-02-041-1/+1
|/ | | | | | Newly added profile options may break this test otherwise. Signed-off-by: Samuel Just <sjust@redhat.com>
* Revert "crush: add multistep retry rules"Samuel Just2024-01-261-1/+1
| | | | | | | This PR was merged by accident before it was ready. Let's revert for now and open a new PR. Signed-off-by: Samuel Just <sjust@redhat.com>
* Merge pull request #55096 from athanatos/sjust/for-review/wip-crush-msrYuri Weinstein2024-01-261-1/+1
|\ | | | | | | | | crush: add multistep retry rules Reviewed-by: Laura Flores <lflores@redhat.com>
| * tasks/.../test_erasure_code_profile: assertSubset in test_create_pluginSamuel Just2024-01-091-1/+1
| | | | | | | | | | | | Newly added profile options may break this test otherwise. Signed-off-by: Samuel Just <sjust@redhat.com>
* | qa: test devicehealth legacy load of deleted snap objPatrick Donnelly2024-01-101-0/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Failure without fix looks like: 2023-12-21T16:05:55.737+0000 7fbe585b0700 0 [devicehealth DEBUG root] loading object ABC_DEADB33F_FA 2023-12-21T16:05:55.737+0000 7fbe585b0700 -1 log_channel(cluster) log [ERR] : Unhandled exception from module 'devicehealth' while running on mgr.x: [errno 2] RADOS object not found (Failed to operate read op for oid ABC_DEADB33F_FA) 2023-12-21T16:05:55.737+0000 7fbe585b0700 -1 devicehealth.serve: 2023-12-21T16:05:55.737+0000 7fbe585b0700 -1 Traceback (most recent call last): File "/home/pdonnell/ceph/src/pybind/mgr/devicehealth/module.py", line 394, in serve self._do_serve() File "/home/pdonnell/ceph/src/pybind/mgr/mgr_module.py", line 524, in check return func(self, *args, **kwargs) File "/home/pdonnell/ceph/src/pybind/mgr/devicehealth/module.py", line 354, in _do_serve finished_loading_legacy = self.check_legacy_pool() File "/home/pdonnell/ceph/src/pybind/mgr/devicehealth/module.py", line 326, in check_legacy_pool if self._load_legacy_object(ioctx, obj.key): File "/home/pdonnell/ceph/src/pybind/mgr/devicehealth/module.py", line 300, in _load_legacy_object ioctx.operate_read_op(op, oid) File "rados.pyx", line 3723, in rados.Ioctx.operate_read_op rados.ObjectNotFound: [errno 2] RADOS object not found (Failed to operate read op for oid ABC_DEADB33F_FA) Credit to Greg Farnum for postulating the cause. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
* | qa: allow failing whatever the active mgr isPatrick Donnelly2024-01-101-2/+5
| | | | | | | | | | | | Vanilla `mgr fail` will just fail the active. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
* | qa: add unit tests for MgrMap down flagPatrick Donnelly2024-01-102-0/+34
|/ | | | Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
* qa: add automate_balance to dashboard qa schemaPatrick Donnelly2023-12-121-1/+2
| | | | Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
* Merge pull request #51942 from trociny/wip-61599Rishabh Dave2023-09-061-1/+2
|\ | | | | | | | | | | mds: optionally forbid to use standby for another fs as last resort Reviewed-by: Ernesto Puerta <epuertat@redhat.com> Reviewed-by: Rishabh Dave <ridave@redhat.com>
| * qa/mgr/dashboard: add refuse_standby_for_another_fs flag to mdsmap schemaMykola Golub2023-07-261-1/+2
| | | | | | | | Signed-off-by: Mykola Golub <mykola.golub@clyso.com>
* | Merge pull request #51227 from jsoref/spelling-dashboardNizamudeen A2023-08-172-2/+2
|\ \ | | | | | | | | | | | | | | | | | | dashboard: Fix spelling Reviewed-by: Ernesto Puerta <epuertat@redhat.com> Reviewed-by: Nizamudeen A <nia@redhat.com> Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
| * | dashboard: fix spelling errorsJosh Soref2023-08-092-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * access * availability * dashboard * depth * dimless * evaluation * executing * existing * facts * gigabytes * idempotent * independent * initial * inventory * managed * must not * notification * notifications * orchestrator * previously * promises * purging * queried * repetitive * split * subdirectories * tenant * the * timestamp * transformed * unavailable * visibility * yourself Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
* | | mon, qa: issue pool application warning even if pool is emptyPrashant D2023-08-011-0/+2
|/ / | | | | | | | | | | | | | | | | | | | | | | | | Ceph status fail to report pool application warning if the pool is empty. Report pool application warning even if pool has 0 objects stored in it. Add POOL_APP_NOT_ENABLED cluster warnings to log-ignorelist to fix rados suite. Fixes: https://tracker.ceph.com/issues/57097 Signed-off-by: Prashant D <pdhange@redhat.com>
* | Merge pull request #52218 from YiteGu/cephfs-get-statfsNizamudeen A2023-07-251-0/+57
|\ \ | | | | | | | | | | | | | | | | | | | | | mgr/dashboard: CephFS statfs REST API Reviewed-by: Avan Thakkar <athakkar@redhat.com> Reviewed-by: Dhairya Parmar <dparmar@redhat.com> Reviewed-by: Nizamudeen A <nia@redhat.com> Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
| * | mgr/dashboard: CephFS statfs REST APIyite.gu2023-07-141-0/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | Introduce statfs for the CephFS REST API controller, we can easily get statfs of the specified path by it, it returns the used bytes, used files and used subdirs. Fixes: https://tracker.ceph.com/issues/61883 Signed-off-by: Yite Gu <yitegu0@gmail.com>
* | | Merge pull request #50291 from batrick/i55606Yuri Weinstein2023-07-142-0/+42
|\ \ \ | |/ / |/| | | | | | | | pybind/mgr: reopen database handle on blocklist Reviewed-by: Ramana Raja <rraja@redhat.com>