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

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

"""Veritas Access backend implementation using base step classes."""

import logging
from enum import StrEnum
from typing import Annotated

from pydantic import Field
from rich.console import Console

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

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


class Protocol(StrEnum):
"""Enumeration of valid protocol types."""

ISCSI = "iscsi"


class VeritasAccessConfig(StorageBackendConfig):
"""Configuration model for Veritas Access 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="Storage array management IP address or hostname.")
]
protocol: Annotated[
Protocol | None,
Field(description="Protocol selector: iscsi."),
] = None

# Optional backend configuration
vrts_lun_sparse: Annotated[
bool | None,
Field(description="Create sparse LUN."),
] = None

vrts_target_config: Annotated[
str | None,
Field(description="VA config file."),
] = None


class VeritasAccessBackend(StorageBackendBase):
"""Veritas Access backend implementation."""

backend_type = "veritasaccess"
display_name = "Veritas Access"
generally_available = True

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

@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 VeritasAccessConfig
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 @@ -9,6 +9,7 @@
from sunbeam.storage.backends.dellsc.backend import DellSCBackend
from sunbeam.storage.backends.hitachi.backend import HitachiBackend
from sunbeam.storage.backends.purestorage.backend import PureStorageBackend
from sunbeam.storage.backends.veritasaccess.backend import VeritasAccessBackend


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


@pytest.fixture
def veritasaccess_backend():
"""Provide a Veritas Access backend instance."""
return VeritasAccessBackend()


@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", "veritasaccess", "dellpowerstore"]
)
def any_backend(request):
"""Parametrized fixture that provides each backend type."""
backends = {
"hitachi": HitachiBackend(),
"purestorage": PureStorageBackend(),
"dellsc": DellSCBackend(),
"veritasaccess": VeritasAccessBackend(),
"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,
veritasaccess_backend,
dellpowerstore_backend,
):
"""Test that all backends have unique type identifiers."""
backends = [
hitachi_backend,
purestorage_backend,
dellsc_backend,
veritasaccess_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,
veritasaccess_backend,
dellpowerstore_backend,
):
"""Test that all backends have unique charm names."""
backends = [
hitachi_backend,
purestorage_backend,
dellsc_backend,
veritasaccess_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"),
("veritasaccess", "veritasaccess"),
("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"),
("veritasaccess", "cinder-volume-veritasaccess"),
("dellpowerstore", "cinder-volume-dellpowerstore"),
],
)
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
# SPDX-FileCopyrightText: 2026 - Canonical Ltd
# SPDX-License-Identifier: Apache-2.0

"""Tests for Veritas Access backend."""

import pytest
from pydantic import ValidationError

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


class TestVeritasAccessBackend(BaseBackendTests):
"""Tests for Veritas Access backend."""

@pytest.fixture
def backend(self, veritasaccess_backend):
"""Provide Veritas Access backend instance."""
return veritasaccess_backend

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

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

def test_display_name_is_veritas_access(self, backend):
"""Test that display name is user-friendly and consistent."""
assert backend.display_name == "Veritas Access"

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


class TestVeritasAccessConfigValidation:
"""Test Veritas Access config validation behavior."""

def test_protocol_rejects_invalid_value(self, veritasaccess_backend):
"""Test that protocol rejects values other than iscsi."""
config_class = veritasaccess_backend.config_type()
with pytest.raises(ValidationError):
config_class.model_validate(
{
"san-ip": "192.168.1.1",
"protocol": "fc",
}
)

def test_protocol_accepts_iscsi(self, veritasaccess_backend):
"""Test that protocol accepts iscsi."""
config_class = veritasaccess_backend.config_type()
config = config_class.model_validate(
{
"san-ip": "192.168.1.1",
"protocol": "iscsi",
}
)
assert config.protocol == "iscsi"
Loading