Skip to content

Commit 7a54ba3

Browse files
committed
mgr/dashboard: Left navigation improvements
Fixes: https://tracker.ceph.com/issues/62366 Signed-off-by: cloudbehl <[email protected]>
1 parent 28cb4d1 commit 7a54ba3

File tree

7 files changed

+277
-164
lines changed

7 files changed

+277
-164
lines changed

src/pybind/mgr/dashboard/frontend/cypress/e2e/ui/navigation.po.ts

Lines changed: 38 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -6,33 +6,18 @@ export class NavigationPageHelper extends PageHelper {
66
};
77

88
navigations = [
9-
{ menu: 'NFS', component: 'cd-error' },
10-
{
11-
menu: 'Object Gateway',
12-
submenus: [
13-
{ menu: 'Gateways', component: 'cd-rgw-daemon-list' },
14-
{ menu: 'Users', component: 'cd-rgw-user-list' },
15-
{ menu: 'Buckets', component: 'cd-rgw-bucket-list' }
16-
]
17-
},
189
{ menu: 'Dashboard', component: 'cd-dashboard' },
1910
{
2011
menu: 'Cluster',
2112
submenus: [
13+
{ menu: 'Pools', component: 'cd-pool-list' },
2214
{ menu: 'Hosts', component: 'cd-hosts' },
23-
{ menu: 'Physical Disks', component: 'cd-error' },
24-
{ menu: 'Monitors', component: 'cd-monitor' },
25-
{ menu: 'Services', component: 'cd-error' },
2615
{ menu: 'OSDs', component: 'cd-osd-list' },
27-
{ menu: 'Configuration', component: 'cd-configuration' },
16+
{ menu: 'Physical Disks', component: 'cd-error' },
2817
{ menu: 'CRUSH map', component: 'cd-crushmap' },
29-
{ menu: 'Manager Modules', component: 'cd-mgr-module-list' },
30-
{ menu: 'Ceph Users', component: 'cd-crud-table' },
31-
{ menu: 'Logs', component: 'cd-logs' },
32-
{ menu: 'Alerts', component: 'cd-prometheus-tabs' }
18+
{ menu: 'Monitors', component: 'cd-monitor' }
3319
]
3420
},
35-
{ menu: 'Pools', component: 'cd-pool-list' },
3621
{
3722
menu: 'Block',
3823
submenus: [
@@ -41,7 +26,41 @@ export class NavigationPageHelper extends PageHelper {
4126
{ menu: 'iSCSI', component: 'cd-iscsi' }
4227
]
4328
},
44-
{ menu: 'File Systems', component: 'cd-cephfs-list' }
29+
{
30+
menu: 'Object',
31+
submenus: [
32+
{ menu: 'Overview', component: 'cd-rgw-overview-dashboard' },
33+
{ menu: 'Buckets', component: 'cd-rgw-bucket-list' },
34+
{ menu: 'Users', component: 'cd-rgw-user-list' },
35+
{ menu: 'Multi-site', component: 'cd-rgw-multisite-details' },
36+
{ menu: 'Gateways', component: 'cd-rgw-daemon-list' },
37+
{ menu: 'NFS', component: 'cd-error' }
38+
]
39+
},
40+
{
41+
menu: 'File',
42+
submenus: [
43+
{ menu: 'File Systems', component: 'cd-cephfs-list' },
44+
{ menu: 'NFS', component: 'cd-error' }
45+
]
46+
},
47+
{
48+
menu: 'Observability',
49+
submenus: [
50+
{ menu: 'Logs', component: 'cd-logs' },
51+
{ menu: 'Alerts', component: 'cd-prometheus-tabs' }
52+
]
53+
},
54+
{
55+
menu: 'Administration',
56+
submenus: [
57+
{ menu: 'Services', component: 'cd-error' },
58+
{ menu: 'Upgrade', component: 'cd-error' },
59+
{ menu: 'Ceph Users', component: 'cd-crud-table' },
60+
{ menu: 'Manager Modules', component: 'cd-mgr-module-list' },
61+
{ menu: 'Configuration', component: 'cd-configuration' }
62+
]
63+
}
4564
];
4665

4766
getVerticalMenu() {

src/pybind/mgr/dashboard/frontend/src/app/app-routing.module.ts

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -124,31 +124,31 @@ const routes: Routes = [
124124
path: 'ceph-users',
125125
component: CRUDTableComponent,
126126
data: {
127-
breadcrumbs: 'Cluster/Ceph Users',
127+
breadcrumbs: 'Administration/Ceph Users',
128128
resource: '[email protected]'
129129
}
130130
},
131131
{
132132
path: 'cluster/user/create',
133133
component: CrudFormComponent,
134134
data: {
135-
breadcrumbs: 'Cluster/Ceph Users/Create',
135+
breadcrumbs: 'Administration/Ceph Users/Create',
136136
resource: '[email protected]'
137137
}
138138
},
139139
{
140140
path: 'cluster/user/import',
141141
component: CrudFormComponent,
142142
data: {
143-
breadcrumbs: 'Cluster/Ceph Users/Import',
143+
breadcrumbs: 'Administration/Ceph Users/Import',
144144
resource: '[email protected]'
145145
}
146146
},
147147
{
148148
path: 'cluster/user/edit',
149149
component: CrudFormComponent,
150150
data: {
151-
breadcrumbs: 'Cluster/Ceph Users/Edit',
151+
breadcrumbs: 'Administration/Ceph Users/Edit',
152152
resource: '[email protected]'
153153
}
154154
},
@@ -169,7 +169,7 @@ const routes: Routes = [
169169
section_info: 'Orchestrator',
170170
header: 'Orchestrator is not available'
171171
},
172-
breadcrumbs: 'Cluster/Services'
172+
breadcrumbs: 'Administration/Services'
173173
},
174174
children: [
175175
{
@@ -213,7 +213,7 @@ const routes: Routes = [
213213
},
214214
{
215215
path: 'configuration',
216-
data: { breadcrumbs: 'Cluster/Configuration' },
216+
data: { breadcrumbs: 'Administration/Configuration' },
217217
children: [
218218
{ path: '', component: ConfigurationComponent },
219219
{
@@ -231,7 +231,7 @@ const routes: Routes = [
231231
{
232232
path: 'logs',
233233
component: LogsComponent,
234-
data: { breadcrumbs: 'Cluster/Logs' }
234+
data: { breadcrumbs: 'Observability/Logs' }
235235
},
236236
{
237237
path: 'telemetry',
@@ -240,7 +240,7 @@ const routes: Routes = [
240240
},
241241
{
242242
path: 'monitoring',
243-
data: { breadcrumbs: 'Cluster/Alerts' },
243+
data: { breadcrumbs: 'Observability/Alerts' },
244244
children: [
245245
{ path: '', redirectTo: 'active-alerts', pathMatch: 'full' },
246246
{
@@ -297,7 +297,7 @@ const routes: Routes = [
297297
section_info: 'Orchestrator',
298298
header: 'Orchestrator is not available'
299299
},
300-
breadcrumbs: 'Cluster/Upgrade'
300+
breadcrumbs: 'Administration/Upgrade'
301301
},
302302
children: [
303303
{
@@ -321,7 +321,7 @@ const routes: Routes = [
321321
// Mgr modules
322322
{
323323
path: 'mgr-modules',
324-
data: { breadcrumbs: 'Cluster/Manager Modules' },
324+
data: { breadcrumbs: 'Administrator/Manager Modules' },
325325
children: [
326326
{
327327
path: '',
@@ -339,7 +339,7 @@ const routes: Routes = [
339339
// Pools
340340
{
341341
path: 'pool',
342-
data: { breadcrumbs: 'Pools' },
342+
data: { breadcrumbs: 'Cluster/Pools' },
343343
loadChildren: () => import('./ceph/pool/pool.module').then((m) => m.RoutedPoolModule)
344344
},
345345
// Block
@@ -352,7 +352,7 @@ const routes: Routes = [
352352
{
353353
path: 'cephfs',
354354
canActivate: [FeatureTogglesGuardService],
355-
data: { breadcrumbs: 'File Systems' },
355+
data: { breadcrumbs: 'File/File Systems' },
356356
children: [
357357
{ path: '', component: CephfsListComponent },
358358
{
@@ -380,7 +380,7 @@ const routes: Routes = [
380380
header: 'The Object Gateway Service is not configured'
381381
},
382382
breadcrumbs: true,
383-
text: 'Object Gateway',
383+
text: 'Object',
384384
path: null
385385
},
386386
loadChildren: () => import('./ceph/rgw/rgw.module').then((m) => m.RoutedRgwModule)

0 commit comments

Comments
 (0)