Extend type of Chart ref to work with OCI protocol correctly

[NikPaushkin]

HTTP links are only allowed now here https://github.com/stackhpc/pyhelm3/blob/main/pyhelm3/models.py#L185. Could use simple str type or implement a custom one since Pydantic (and nobody tbh) doesn't know what OCI protocol is.

[NikPaushkin]

@mkjpryor Hey, do you accept any pull requests?

[mkjpryor]

@NikPaushkin We would welcome a PR to implement this!

Rather than accepting all strings, you should be able to define an OCI URI using stricturl in Pydantic, which can then be added to the union. Something like:

OciUrl = stricturl(allowed_schemes = {"oci"})

This would be my preferred method.

https://docs.pydantic.dev/1.10/usage/types/#urls

[nalyat]

@mkjpryor any feedback on the PR? Is there a reason not to merge this one? There's multiple PR's referencing the same issue(#4)

[isbalashov]

For anyone looking for a workaround consider monkeypatching Client.get_chart(). Here is what I came up with:

from pyhelm3 import Client, Chart
import typing as t
from typing_extensions import Annotated
from pydantic import (
    Field,
    DirectoryPath,
    FilePath,
    HttpUrl,
)

Name = Annotated[str, Field(pattern=r"^[a-z0-9-]+$")]
OCIPath = Annotated[str, Field(pattern=r"oci:\/\/*")]

class OCIChart(Chart):
    ref: t.Union[DirectoryPath, FilePath, HttpUrl, OCIPath, Name] = Field(
        ...,
    )

async def get_chart_oci(self, chart_ref, *, devel=False, repo=None, version=None):
    metadata = await self._command.show_chart(
        chart_ref,
        devel=devel,
        repo=repo,
        version=version
    )
    return OCIChart(
        _command=self._command,
        ref=chart_ref,
        repo=repo,
        metadata=metadata
    )


# Monkey patch the get_chart method of the Client class
Client.get_chart = get_chart_oci
client = Client()