Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
# SPDX-FileCopyrightText: 2026 - Canonical Ltd
# SPDX-License-Identifier: Apache-2.0

"""Inspur AS13000 backend for Sunbeam storage."""
109 changes: 109 additions & 0 deletions sunbeam-python/sunbeam/storage/backends/inspuras13000/backend.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,109 @@
# SPDX-FileCopyrightText: 2026 - Canonical Ltd
# SPDX-License-Identifier: Apache-2.0

"""AS13000 backend implementation using base step classes."""

import logging
from typing import Annotated, Literal

from pydantic import Field
from rich.console import Console

from sunbeam.core.manifest import StorageBackendConfig
from sunbeam.storage.base import StorageBackendBase
from sunbeam.storage.models import SecretDictField

LOG = logging.getLogger(__name__)
console = Console()


class Inspuras13000Config(StorageBackendConfig):
"""Configuration model for AS13000 backend.

This model includes ALL configuration options for the backend.
Additional configuration can be managed dynamically through the charm.
"""

# Mandatory connection parameters
san_ip: Annotated[str, Field(description="IP address of SAN controller")]
san_login: Annotated[
str,
Field(description="Username for SAN controller"),
SecretDictField(field="san-login"),
]
san_password: Annotated[
str,
Field(description="Password for SAN controller"),
SecretDictField(field="san-password"),
]
as13000_token_available_time: Annotated[
str,
Field(
alias="as13000-token-available-time",
description="The effective time of token validity in seconds.",
),
SecretDictField(field="as13000-token-available-time"),
]

# Optional backend configuration
protocol: Annotated[
Literal["iscsi"] | None,
Field(description="Protocol selector: iscsi."),
] = None
as13000_ipsan_pools: Annotated[
str | None,
Field(
description="The Storage Pools Cinder should use, a comma separated list."
),
] = None
as13000_meta_pool: Annotated[
str | None,
Field(
description="The pool which is used as a meta pool when creating a volume."
),
] = None
san_thin_provision: Annotated[
bool | None,
Field(description="Use thin provisioning for SAN volumes?"),
] = None
use_multipath_for_image_xfer: Annotated[
bool | None,
Field(description="Enable multipathing for image transfer operations."),
] = None


class Inspuras13000Backend(StorageBackendBase):
"""AS13000 backend implementation."""

backend_type = "inspuras13000"
display_name = "AS13000"
generally_available = True

@property
def charm_name(self) -> str:
"""Return the charm application name."""
return "cinder-volume-inspuras13000"

@property
def charm_channel(self) -> str:
"""Return the default charm channel."""
return "latest/edge"

@property
def charm_revision(self) -> str | None:
"""Return a pinned charm revision, if any."""
return None

@property
def charm_base(self) -> str:
"""Return the target base for this charm."""
return "ubuntu@24.04"

@property
def supports_ha(self) -> bool:
"""Whether this backend supports HA deployments."""
return False

def config_type(self) -> type[StorageBackendConfig]:
"""Return the configuration model type for this backend."""
return Inspuras13000Config
12 changes: 11 additions & 1 deletion sunbeam-python/tests/unit/sunbeam/storage/backends/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
from sunbeam.storage.backends.dellpowerstore.backend import DellPowerstoreBackend
from sunbeam.storage.backends.dellsc.backend import DellSCBackend
from sunbeam.storage.backends.hitachi.backend import HitachiBackend
from sunbeam.storage.backends.inspuras13000.backend import Inspuras13000Backend
from sunbeam.storage.backends.purestorage.backend import PureStorageBackend


Expand All @@ -29,19 +30,28 @@ def dellsc_backend():
return DellSCBackend()


@pytest.fixture
def inspuras13000_backend():
"""Provide an Inspur AS13000 backend instance."""
return Inspuras13000Backend()


@pytest.fixture
def dellpowerstore_backend():
"""Provide a Dell PowerStore backend instance."""
return DellPowerstoreBackend()


@pytest.fixture(params=["hitachi", "purestorage", "dellsc", "dellpowerstore"])
@pytest.fixture(
params=["hitachi", "purestorage", "dellsc", "inspuras13000", "dellpowerstore"]
)
def any_backend(request):
"""Parametrized fixture that provides each backend type."""
backends = {
"hitachi": HitachiBackend(),
"purestorage": PureStorageBackend(),
"dellsc": DellSCBackend(),
"inspuras13000": Inspuras13000Backend(),
"dellpowerstore": DellPowerstoreBackend(),
}
return backends[request.param]
16 changes: 14 additions & 2 deletions sunbeam-python/tests/unit/sunbeam/storage/backends/test_common.py
Original file line number Diff line number Diff line change
Expand Up @@ -157,13 +157,18 @@ def backend(self, any_backend):


def test_all_backends_have_unique_types(
hitachi_backend, purestorage_backend, dellsc_backend, dellpowerstore_backend
hitachi_backend,
purestorage_backend,
dellsc_backend,
inspuras13000_backend,
dellpowerstore_backend,
):
"""Test that all backends have unique type identifiers."""
backends = [
hitachi_backend,
purestorage_backend,
dellsc_backend,
inspuras13000_backend,
dellpowerstore_backend,
]
types = [b.backend_type for b in backends]
Expand All @@ -173,13 +178,18 @@ def test_all_backends_have_unique_types(


def test_all_backends_have_unique_charm_names(
hitachi_backend, purestorage_backend, dellsc_backend, dellpowerstore_backend
hitachi_backend,
purestorage_backend,
dellsc_backend,
inspuras13000_backend,
dellpowerstore_backend,
):
"""Test that all backends have unique charm names."""
backends = [
hitachi_backend,
purestorage_backend,
dellsc_backend,
inspuras13000_backend,
dellpowerstore_backend,
]
charm_names = [b.charm_name for b in backends]
Expand All @@ -196,6 +206,7 @@ def test_all_backends_have_unique_charm_names(
("hitachi", "hitachi"),
("purestorage", "purestorage"),
("dellsc", "dellsc"),
("inspuras13000", "inspuras13000"),
("dellpowerstore", "dellpowerstore"),
],
)
Expand All @@ -211,6 +222,7 @@ def test_backend_types_match_expected(any_backend, backend_type, expected_type):
("hitachi", "cinder-volume-hitachi"),
("purestorage", "cinder-volume-purestorage"),
("dellsc", "cinder-volume-dellsc"),
("inspuras13000", "cinder-volume-inspuras13000"),
("dellpowerstore", "cinder-volume-dellpowerstore"),
],
)
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,81 @@
# SPDX-FileCopyrightText: 2026 - Canonical Ltd
# SPDX-License-Identifier: Apache-2.0

"""Tests for Inspur AS13000 backend."""

import pytest
from pydantic import ValidationError

from sunbeam.storage.models import SecretDictField
from tests.unit.sunbeam.storage.backends.test_common import BaseBackendTests


class TestInspuras13000Backend(BaseBackendTests):
"""Tests for Inspur AS13000 backend."""

@pytest.fixture
def backend(self, inspuras13000_backend):
"""Provide Inspur AS13000 backend instance."""
return inspuras13000_backend

def test_backend_type_is_inspuras13000(self, backend):
"""Test that backend type is 'inspuras13000'."""
assert backend.backend_type == "inspuras13000"

def test_charm_name_is_inspuras13000_charm(self, backend):
"""Test that charm name is cinder-volume-inspuras13000."""
assert backend.charm_name == "cinder-volume-inspuras13000"

def test_config_has_required_fields(self, backend):
"""Test that AS13000 config has required fields."""
fields = backend.config_type().model_fields
for field in (
"san_ip",
"san_login",
"san_password",
"as13000_token_available_time",
"protocol",
):
assert field in fields, f"Required field {field} not found in config"

def test_credentials_are_secret(self, backend):
"""Test that credential-like fields are marked as secrets."""
config_class = backend.config_type()
for field_name in ("san_login", "san_password", "as13000_token_available_time"):
field = config_class.model_fields.get(field_name)
assert field is not None
assert any(isinstance(m, SecretDictField) for m in field.metadata), (
f"{field_name} should be marked as secret"
)


class TestInspuras13000ConfigValidation:
"""Test AS13000 config validation behavior."""

def test_protocol_rejects_invalid_value(self, inspuras13000_backend):
"""Test that protocol rejects values other than iscsi."""
config_class = inspuras13000_backend.config_type()
with pytest.raises(ValidationError):
config_class.model_validate(
{
"san-ip": "192.168.1.1",
"san-login": "admin",
"san-password": "secret",
"as13000-token-available-time": "3600",
"protocol": "fc",
}
)

def test_protocol_accepts_iscsi(self, inspuras13000_backend):
"""Test that protocol accepts iscsi."""
config_class = inspuras13000_backend.config_type()
config = config_class.model_validate(
{
"san-ip": "192.168.1.1",
"san-login": "admin",
"san-password": "secret",
"as13000-token-available-time": "3600",
"protocol": "iscsi",
}
Comment thread
ahmad-can marked this conversation as resolved.
)
assert config.protocol == "iscsi"
Loading