Skip to content

Commit 3e5e85a

Browse files
committed
Revert "mgr/orchestrator: require "group" field for nvmeof specs"
This reverts commit f6d552d. It was decided by the nvmeof team to stick with defaulting to an empty string rather than forcing the users onto other non-empty names when they upgrade Signed-off-by: Adam King <[email protected]>
1 parent 15cbae0 commit 3e5e85a

File tree

2 files changed

+2
-7
lines changed

2 files changed

+2
-7
lines changed

src/pybind/mgr/orchestrator/module.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1866,7 +1866,6 @@ def _apply_oauth2_proxy(self,
18661866
@_cli_write_command('orch apply nvmeof')
18671867
def _apply_nvmeof(self,
18681868
pool: str,
1869-
group: str,
18701869
placement: Optional[str] = None,
18711870
unmanaged: bool = False,
18721871
dry_run: bool = False,
@@ -1878,9 +1877,8 @@ def _apply_nvmeof(self,
18781877
raise OrchestratorValidationError('unrecognized command -i; -h or --help for usage')
18791878

18801879
spec = NvmeofServiceSpec(
1881-
service_id=f'{pool}.{group}',
1880+
service_id=pool,
18821881
pool=pool,
1883-
group=group,
18841882
placement=PlacementSpec.from_string(placement),
18851883
unmanaged=unmanaged,
18861884
preview_only=dry_run

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

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1382,7 +1382,7 @@ def __init__(self,
13821382
#: ``name`` name of the nvmeof gateway
13831383
self.name = name
13841384
#: ``group`` name of the nvmeof gateway
1385-
self.group = group
1385+
self.group = group or ''
13861386
#: ``enable_auth`` enables user authentication on nvmeof gateway
13871387
self.enable_auth = enable_auth
13881388
#: ``state_update_notify`` enables automatic update from OMAP in nvmeof gateway
@@ -1472,9 +1472,6 @@ def validate(self) -> None:
14721472
if not self.pool:
14731473
raise SpecValidationError('Cannot add NVMEOF: No Pool specified')
14741474

1475-
if not self.group:
1476-
raise SpecValidationError('Cannot add NVMEOF: No group specified')
1477-
14781475
if self.enable_auth:
14791476
if not all([self.server_key, self.server_cert, self.client_key,
14801477
self.client_cert, self.root_ca_cert]):

0 commit comments

Comments
 (0)