Skip to content

Commit 90e0e48

Browse files
Revert "feat: allow EDA credential fields to link to external Secret … (#1352)
This reverts commit 09db042 - #1349 Requested by productization, it needs to wait until have packages for awx-plugins <!-- Mandatory: Provide a clear, concise description of the changes and their purpose --> <!-- If applicable, provide a link to the issue that is being addressed --> <!-- What is being changed? --> <!-- Why is this change needed? --> <!-- How does this change address the issue? --> <!-- Does this change introduce any new dependencies, blockers or breaking changes? --> <!-- How it can be tested? -->
1 parent 09db042 commit 90e0e48

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

46 files changed

+65
-3534
lines changed

poetry.lock

Lines changed: 3 additions & 530 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

poetry.toml

Lines changed: 0 additions & 4 deletions
This file was deleted.

pyproject.toml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -71,18 +71,6 @@ django-flags = "^5.0.13"
7171
insights-analytics-collector = "^0.3.2"
7272
distro = "^1.9.0"
7373
dispatcherd = { version = "v2025.05.19", extras = ["pg_notify"] }
74-
awx-plugins-core = { version = "^0.0.1a10", extras = [
75-
"credentials-aim",
76-
"credentials-aws-secretsmanager-credential",
77-
"credentials-azure-kv",
78-
"credentials-centrify-vault-kv",
79-
"credentials-conjur",
80-
"credentials-github-app",
81-
"credentials-hashivault-kv",
82-
"credentials-hashivault-ssh",
83-
"credentials-thycotic-dsv",
84-
"credentials-thycotic-tss"
85-
]}
8674

8775

8876
[tool.poetry.group.test.dependencies]

src/aap_eda/analytics/utils.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
from aap_eda.conf import application_settings
2727
from aap_eda.conf.registry import ANALYTICS_GATHER_INTERVAL
2828
from aap_eda.core import enums, models
29-
from aap_eda.core.utils.credentials import get_resolved_secrets
29+
from aap_eda.core.utils.credentials import inputs_from_store
3030

3131
logger = logging.getLogger("aap_eda.analytics")
3232

@@ -225,7 +225,7 @@ def generate_token() -> ServiceToken:
225225
def _get_credential_value(field: str, *setting_envs: Tuple[Any, str]) -> str:
226226
credentials = _get_analytics_credentials()
227227
for credential in credentials:
228-
inputs = get_resolved_secrets(credential)
228+
inputs = inputs_from_store(credential.inputs.get_secret_value())
229229
if value := inputs.get(field):
230230
return value
231231

@@ -314,7 +314,7 @@ def get_analytics_interval() -> int:
314314
def get_analytics_interval_if_exist(credential: models.EdaCredential) -> int:
315315
if credential.credential_type.kind != get_auth_mode():
316316
return 0
317-
inputs = get_resolved_secrets(credential)
317+
inputs = inputs_from_store(credential.inputs.get_secret_value())
318318
return inputs.get("gather_interval", 0)
319319

320320

src/aap_eda/api/exceptions.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -118,10 +118,3 @@ class InvalidEventStreamSource(APIException):
118118
"Configuration Error: Event Stream source could not be "
119119
"updated in ruleset"
120120
)
121-
122-
123-
class ExternalSMSError(APIException):
124-
status_code = status.HTTP_503_SERVICE_UNAVAILABLE
125-
default_detail = (
126-
"External SMS Error: not able to fetch secrets from external SMS"
127-
)

src/aap_eda/api/filters/__init__.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
ActivationInstanceFilter,
1818
ActivationInstanceLogFilter,
1919
)
20-
from .credential_input_source import CredentialInputSourceFilter
2120
from .credential_type import CredentialTypeFilter
2221
from .decision_environment import DecisionEnvironmentFilter
2322
from .eda_credential import EdaCredentialFilter
@@ -36,7 +35,6 @@
3635
# credential type
3736
"CredentialTypeFilter",
3837
"EdaCredentialFilter",
39-
"CredentialInputSourceFilter",
4038
# decision_environment
4139
"DecisionEnvironmentFilter",
4240
# activation instance

src/aap_eda/api/filters/credential_input_source.py

Lines changed: 0 additions & 34 deletions
This file was deleted.

src/aap_eda/api/serializers/__init__.py

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -25,18 +25,10 @@
2525
)
2626
from .auth import JWTTokenSerializer, LoginSerializer, RefreshTokenSerializer
2727
from .config import ConfigSerializer
28-
from .credential_input_source import (
29-
CredentialInputSourceCreateSerializer,
30-
CredentialInputSourceReferenceSerializer,
31-
CredentialInputSourceRefSerializer,
32-
CredentialInputSourceSerializer,
33-
CredentialInputSourceUpdateSerializer,
34-
)
3528
from .credential_type import (
3629
CredentialTypeCreateSerializer,
3730
CredentialTypeRefSerializer,
3831
CredentialTypeSerializer,
39-
CredentialTypeTestSerializer,
4032
)
4133
from .decision_environment import (
4234
DecisionEnvironmentCreateSerializer,
@@ -48,7 +40,6 @@
4840
EdaCredentialCopySerializer,
4941
EdaCredentialCreateSerializer,
5042
EdaCredentialSerializer,
51-
EdaCredentialTestSerializer,
5243
EdaCredentialUpdateSerializer,
5344
)
5445
from .event_stream import EventStreamInSerializer, EventStreamOutSerializer

src/aap_eda/api/serializers/activation.py

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
EDA_SERVER_VAULT_LABEL,
2828
SOURCE_MAPPING_ERROR_KEY,
2929
)
30-
from aap_eda.api.exceptions import ExternalSMSError, InvalidEventStreamSource
30+
from aap_eda.api.exceptions import InvalidEventStreamSource
3131
from aap_eda.api.serializers.decision_environment import (
3232
DecisionEnvironmentRefSerializer,
3333
)
@@ -46,11 +46,8 @@
4646
from aap_eda.api.vault import encrypt_string
4747
from aap_eda.core import models, validators
4848
from aap_eda.core.enums import DefaultCredentialType, ProcessParentType
49-
from aap_eda.core.exceptions import CredentialPluginError, ParseError
50-
from aap_eda.core.utils.credentials import (
51-
get_resolved_secrets,
52-
get_secret_fields,
53-
)
49+
from aap_eda.core.exceptions import ParseError
50+
from aap_eda.core.utils.credentials import get_secret_fields
5451
from aap_eda.core.utils.k8s_service_name import create_k8s_service_name
5552
from aap_eda.core.utils.rulebook import (
5653
build_source_list,
@@ -150,10 +147,8 @@ def _update_extra_vars_from_eda_credentials(
150147
schema_inputs = eda_credential.credential_type.inputs
151148
injectors = eda_credential.credential_type.injectors
152149
secret_fields = get_secret_fields(schema_inputs)
153-
try:
154-
user_inputs = get_resolved_secrets(eda_credential)
155-
except CredentialPluginError as err:
156-
raise ExternalSMSError(str(err))
150+
151+
user_inputs = yaml.safe_load(eda_credential.inputs.get_secret_value())
157152

158153
if creating and any(key in user_inputs for key in secret_fields):
159154
vault_data.password_used = True

0 commit comments

Comments
 (0)