Skip to content

Commit a68aef1

Browse files
authored
Merge pull request ceph#59859 from afreen23/wip-nvme-service-fixes
mgr/dashboard: Service form fixes for mTLS Reviewed-by: Pedro Gonzalez Gomez <[email protected]> Reviewed-by: Nizamudeen A <[email protected]>
2 parents 1a8a7e4 + 3bcbe50 commit a68aef1

File tree

3 files changed

+12
-12
lines changed

3 files changed

+12
-12
lines changed

src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/services/service-form/service-form.component.html

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1292,7 +1292,7 @@
12921292
</div>
12931293

12941294
<!-- root_ca_cert -->
1295-
<div *ngIf="serviceForm.controls.enable_auth.value"
1295+
<div *ngIf="serviceForm.controls.enable_mtls.value"
12961296
class="form-group row">
12971297
<label class="cd-col-form-label required"
12981298
for="root_ca_cert">
@@ -1312,7 +1312,7 @@
13121312
</div>
13131313

13141314
<!-- client_cert -->
1315-
<div *ngIf="serviceForm.controls.enable_auth.value"
1315+
<div *ngIf="serviceForm.controls.enable_mtls.value"
13161316
class="form-group row">
13171317
<label class="cd-col-form-label required"
13181318
for="client_cert">
@@ -1332,7 +1332,7 @@
13321332
</div>
13331333

13341334
<!-- client_key -->
1335-
<div *ngIf="serviceForm.controls.enable_auth.value"
1335+
<div *ngIf="serviceForm.controls.enable_mtls.value"
13361336
class="form-group row">
13371337
<label class="cd-col-form-label required"
13381338
for="client_key">
@@ -1352,7 +1352,7 @@
13521352
</div>
13531353

13541354
<!-- server_cert -->
1355-
<div *ngIf="serviceForm.controls.enable_auth.value"
1355+
<div *ngIf="serviceForm.controls.enable_mtls.value"
13561356
class="form-group row">
13571357
<label class="cd-col-form-label required"
13581358
for="server_cert">
@@ -1372,7 +1372,7 @@
13721372
</div>
13731373

13741374
<!-- server_key -->
1375-
<div *ngIf="serviceForm.controls.enable_auth.value"
1375+
<div *ngIf="serviceForm.controls.enable_mtls.value"
13761376
class="form-group row">
13771377
<label class="cd-col-form-label required"
13781378
for="server_key">

src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/services/service-form/service-form.component.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -769,7 +769,7 @@ x4Ea7kGVgx9kWh5XjWz9wjZvY49UKIT5ppIAWPMbLl3UpfckiuNhTA==
769769
formHelper.setValue('pool', 'rbd');
770770
formHelper.setValue('group', 'default');
771771
// mTLS disabled
772-
formHelper.setValue('enable_auth', false);
772+
formHelper.setValue('enable_mtls', false);
773773
component.onSubmit();
774774
expect(cephServiceService.update).toHaveBeenCalledWith({
775775
service_type: 'nvmeof',

src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/services/service-form/service-form.component.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -694,12 +694,12 @@ export class ServiceFormComponent extends CdForm implements OnInit {
694694
case 'nvmeof':
695695
this.serviceForm.get('pool').setValue(response[0].spec.pool);
696696
this.serviceForm.get('group').setValue(response[0].spec.group);
697-
this.serviceForm.get('enable_auth').setValue(response[0].spec.enable_auth);
698-
this.serviceForm.get('root_ca_cert').setValue(response[0].spec.root_ca_cert);
699-
this.serviceForm.get('client_cert').setValue(response[0].spec.client_cert);
700-
this.serviceForm.get('client_key').setValue(response[0].spec.client_key);
701-
this.serviceForm.get('server_cert').setValue(response[0].spec.server_cert);
702-
this.serviceForm.get('server_key').setValue(response[0].spec.server_key);
697+
this.serviceForm.get('enable_mtls').setValue(response[0].spec?.enable_auth);
698+
this.serviceForm.get('root_ca_cert').setValue(response[0].spec?.root_ca_cert);
699+
this.serviceForm.get('client_cert').setValue(response[0].spec?.client_cert);
700+
this.serviceForm.get('client_key').setValue(response[0].spec?.client_key);
701+
this.serviceForm.get('server_cert').setValue(response[0].spec?.server_cert);
702+
this.serviceForm.get('server_key').setValue(response[0].spec?.server_key);
703703
break;
704704
case 'rgw':
705705
this.serviceForm

0 commit comments

Comments
 (0)