Merge pull request ceph#59332 from afreen23/nvmeof-group-mtls

mgr/dashboard: Add group field in nvmeof service form

Reviewed-by: Afreen Misbah <[email protected]>

Author: afreen23

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

@@ -105,6 +105,30 @@
           </div>
         </div>
 
+        <!-- Group Name -->
+        <div class="form-group row"
+             *ngIf="serviceForm.controls.service_type.value === 'nvmeof'">
+          <label class="cd-col-form-label required"
+                 for="group">
+            <span i18n>Group Name</span>
+          </label>
+          <div class="cd-col-form-input">
+            <div class="input-group">
+              <input id="group"
+                     class="form-control"
+                     type="text"
+                     formControlName="group"
+                     (change)="onNvmeofGroupChange($event.target.value)">
+            </div>
+            <cd-help-text i18n>
+              The name of the gateway group.
+            </cd-help-text>
+            <span class="invalid-feedback"
+                  *ngIf="serviceForm.showError('service_id', frm, 'required')"
+                  i18n>This field is required.</span>
+          </div>
+        </div>
+
         <!-- Service id -->
         <div class="form-group row"
              *ngIf="serviceForm.controls.service_type.value !== 'snmp-gateway'">

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

@@ -392,6 +392,7 @@ x4Ea7kGVgx9kWh5XjWz9wjZvY49UKIT5ppIAWPMbLl3UpfckiuNhTA==
         formHelper.setValue('service_type', 'nvmeof');
         formHelper.setValue('service_id', 'svc');
         formHelper.setValue('pool', 'xyz');
+        formHelper.setValue('group', 'abc');
       });
 
       it('should submit nvmeof', () => {
@@ -401,7 +402,8 @@ x4Ea7kGVgx9kWh5XjWz9wjZvY49UKIT5ppIAWPMbLl3UpfckiuNhTA==
           service_id: 'svc',
           placement: {},
           unmanaged: false,
-          pool: 'xyz'
+          pool: 'xyz',
+          group: 'abc'
         });
       });
 
@@ -412,6 +414,10 @@ x4Ea7kGVgx9kWh5XjWz9wjZvY49UKIT5ppIAWPMbLl3UpfckiuNhTA==
       it('should throw error when there is no pool', () => {
         formHelper.expectErrorChange('pool', '', 'required');
       });
+
+      it('should throw error when there is no group', () => {
+        formHelper.expectErrorChange('group', '', 'required');
+      });
     });
 
     describe('should test service smb', () => {

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

@@ -195,6 +195,7 @@ export class ServiceFormComponent extends CdForm implements OnInit {
           })
         ]
       ],
+      group: [null, Validators.required],
       // RGW
       rgw_frontend_port: [null, [CdValidators.number(false)]],
       realm_name: [null],
@@ -522,6 +523,7 @@ export class ServiceFormComponent extends CdForm implements OnInit {
               break;
             case 'nvmeof':
               this.serviceForm.get('pool').setValue(response[0].spec.pool);
+              this.serviceForm.get('group').setValue(response[0].spec.group);
               break;
             case 'rgw':
               this.serviceForm
@@ -778,10 +780,13 @@ export class ServiceFormComponent extends CdForm implements OnInit {
       ?.pool_name;
     if (defaultRbdPool) {
       this.serviceForm.get('pool').setValue(defaultRbdPool);
-      this.serviceForm.get('service_id').setValue(defaultRbdPool);
     }
   }
 
+  onNvmeofGroupChange(groupName: string) {
+    this.serviceForm.get('service_id').setValue(groupName);
+  }
+
   requiresServiceId(serviceType: string) {
     return ['mds', 'rgw', 'nfs', 'iscsi', 'nvmeof', 'smb', 'ingress'].includes(serviceType);
   }
@@ -837,6 +842,7 @@ export class ServiceFormComponent extends CdForm implements OnInit {
         break;
       case 'nvmeof':
         this.serviceForm.get('pool').disable();
+        this.serviceForm.get('group').disable();
         break;
     }
   }
@@ -920,6 +926,9 @@ export class ServiceFormComponent extends CdForm implements OnInit {
         break;
 
       case 'nvmeof':
+        serviceSpec['pool'] = values['pool'];
+        serviceSpec['group'] = values['group'];
+        break;
       case 'iscsi':
         serviceSpec['pool'] = values['pool'];
         break;

src/pybind/mgr/dashboard/frontend/src/app/shared/models/service.interface.ts

@@ -31,6 +31,7 @@ export interface CephServiceAdditionalSpec {
   monitor_port: number;
   virtual_interface_networks: string[];
   pool: string;
+  group: string;
   rgw_frontend_ssl_certificate: string;
   ssl: boolean;
   ssl_cert: string;