use env vars for bedrock guardrail vars

This commit is contained in:
Dinesh Yeduguru 2024-11-07 20:15:42 -08:00
parent 98c09323a9
commit 932b524449
2 changed files with 3 additions and 16 deletions

View file

@ -71,18 +71,6 @@ def pytest_addoption(parser):
default=None, default=None,
help="Specify the safety model to use for testing", help="Specify the safety model to use for testing",
) )
parser.addoption(
"--bedrock-guardrail-id",
action="store",
default=None,
help="Specify the guard rail ID to use for testing bedrock",
)
parser.addoption(
"--bedrock-guardrail-version",
action="store",
default=None,
help="Specify the guard rail version to use for testing bedrock",
)
SAFETY_MODEL_PARAMS = [ SAFETY_MODEL_PARAMS = [

View file

@ -15,6 +15,7 @@ from llama_stack.providers.inline.safety.meta_reference import (
SafetyConfig, SafetyConfig,
) )
from llama_stack.providers.remote.safety.bedrock import BedrockSafetyConfig from llama_stack.providers.remote.safety.bedrock import BedrockSafetyConfig
from llama_stack.providers.tests.env import get_env_or_fail
from llama_stack.providers.tests.resolver import resolve_impls_for_test_v2 from llama_stack.providers.tests.resolver import resolve_impls_for_test_v2
from ..conftest import ProviderFixture, remote_stack_fixture from ..conftest import ProviderFixture, remote_stack_fixture
@ -120,10 +121,8 @@ async def safety_stack(inference_model, safety_model, request):
elif provider_type == "remote::together": elif provider_type == "remote::together":
shield_config["model"] = safety_model shield_config["model"] = safety_model
elif provider_type == "remote::bedrock": elif provider_type == "remote::bedrock":
identifier = request.config.getoption("--bedrock-guardrail-id", None) identifier = get_env_or_fail("BEDROCK_GUARDRAIL_IDENTIFIER")
shield_config["guardrailVersion"] = request.config.getoption( shield_config["guardrailVersion"] = get_env_or_fail("BEDROCK_GUARDRAIL_VERSION")
"--bedrock-guardrail-version", None
)
# Create shield # Create shield
shield = Shield( shield = Shield(