Merge pull request ceph#64477 from gbregman/main

mgr/cephadm/nvmeof: Increase the default limit of max_namespaces

Author: gbregman

src/pybind/mgr/cephadm/tests/test_services.py

@@ -387,8 +387,8 @@ def test_nvmeof_config(self, _get_name, _run_cephadm, cephadm_module: CephadmOrc
 max_namespaces_with_netmask = 1000
 max_subsystems = 128
 max_hosts = 2048
-max_namespaces = 2048
-max_namespaces_per_subsystem = 256
+max_namespaces = 4096
+max_namespaces_per_subsystem = 512
 max_hosts_per_subsystem = 128
 subsystem_cache_expiration = 5
 

src/pybind/mgr/dashboard/controllers/nvmeof.py

@@ -179,7 +179,7 @@ def get(self, nqn: str, gw_group: Optional[str] = None, traddr: Optional[str] =
             "Create a new NVMeoF subsystem",
             parameters={
                 "nqn": Param(str, "NVMeoF subsystem NQN"),
-                "max_namespaces": Param(int, "Maximum number of namespaces", True, 1024),
+                "max_namespaces": Param(int, "Maximum number of namespaces", True, 4096),
                 "enable_ha": Param(bool, "Enable high availability"),
                 "gw_group": Param(str, "NVMeoF gateway group", True, None),
             },
@@ -188,7 +188,7 @@ def get(self, nqn: str, gw_group: Optional[str] = None, traddr: Optional[str] =
         @NvmeofCLICommand("nvmeof subsystem add", model.RequestStatus)
         @convert_to_model(model.SubsystemStatus)
         @handle_nvmeof_error
-        def create(self, nqn: str, enable_ha: bool = True, max_namespaces: int = 1024,
+        def create(self, nqn: str, enable_ha: bool = True, max_namespaces: int = 4096,
                    gw_group: Optional[str] = None, traddr: Optional[str] = None):
             return NVMeoFClient(gw_group=gw_group, traddr=traddr).stub.create_subsystem(
                 NVMeoFClient.pb2.create_subsystem_req(

src/pybind/mgr/dashboard/frontend/src/app/ceph/block/nvmeof-subsystems-details/nvmeof-subsystems-details.component.spec.ts

@@ -27,7 +27,7 @@ describe('NvmeofSubsystemsDetailsComponent', () => {
       subtype: 'NVMe',
       nqn: 'nqn.2001-07.com.ceph:1720603703820',
       namespace_count: 1,
-      max_namespaces: 256
+      max_namespaces: 4096
     };
     component.permissions = new Permissions({
       grafana: ['read']

src/pybind/mgr/dashboard/frontend/src/app/ceph/block/nvmeof-subsystems-form/nvmeof-subsystems-form.component.spec.ts

@@ -80,7 +80,7 @@ describe('NvmeofSubsystemsFormComponent', () => {
     });
 
     it(`should give error on max_namespaces greater than ${MAX_NAMESPACE}`, () => {
-      formHelper.setValue('max_namespaces', 2000);
+      formHelper.setValue('max_namespaces', 6000);
       component.onSubmit();
       formHelper.expectError('max_namespaces', 'max');
     });

src/pybind/mgr/dashboard/frontend/src/app/shared/api/nvmeof.service.ts

@@ -6,7 +6,7 @@ import { Observable, of as observableOf } from 'rxjs';
 import { catchError, mapTo } from 'rxjs/operators';
 import { CephServiceSpec } from '../models/service.interface';
 
-export const MAX_NAMESPACE = 1024;
+export const MAX_NAMESPACE = 4096;
 
 export type GatewayGroup = CephServiceSpec;
 

src/pybind/mgr/dashboard/openapi.yaml

@@ -8559,7 +8559,7 @@ paths:
                   description: NVMeoF gateway group
                   type: string
                 max_namespaces:
-                  default: 1024
+                  default: 4096
                   description: Maximum number of namespaces
                   type: integer
                 nqn:

src/python-common/ceph/deployment/service_spec.py

@@ -1407,8 +1407,8 @@ def __init__(self,
                  max_namespaces_with_netmask: Optional[int] = 1000,
                  max_subsystems: Optional[int] = 128,
                  max_hosts: Optional[int] = 2048,
-                 max_namespaces: Optional[int] = 2048,
-                 max_namespaces_per_subsystem: Optional[int] = 256,
+                 max_namespaces: Optional[int] = 4096,
+                 max_namespaces_per_subsystem: Optional[int] = 512,
                  max_hosts_per_subsystem: Optional[int] = 128,
                  subsystem_cache_expiration: Optional[int] = 5,
                  server_key: Optional[str] = None,