Distinguish unresolved from resolved-empty required resources
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 42/100
Piste de recherche
Commencez par lire les APIs request.get_required_resource et get_required_resources ainsi que leurs docstrings, puis comparez le comportement analogue de require_schema/get_required_schema. Définissez les cas pris en charge unresolved, absent et present, et confirmez la forme d’API choisie avec des tests ; le travail est terminé lorsque les appelants peuvent distinguer ces états sans dupliquer les vérifications d’appartenance.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
What problem are you facing?
request.get_required_resource (and get_required_resources) returns None/[] in two semantically different cases: the requirement hasn't been resolved by Crossplane yet, and the requirement resolved but matched nothing. The only way to tell them apart is to also check name in req.required_resources, which the get_required_resources docstring documents as a caveat the caller must handle by hand.
This distinction matters whenever a function gates on an external resource. "Not resolved yet" is transient — wait and let Crossplane re-call. "Resolved, matched nothing" is a real state — the resource was deleted, or its reference is a typo — and usually wants a different response (e.g. a warning, or holding steady rather than waiting). Functions end up reimplementing the same in-then-get dance. We hit it twice in one project: once gating a hydration Job on an auth Secret, and once gating replica placement on a referenced cache being resolvable.
How could this Function help solve your problem?
A small helper that classifies a required resource into the three states, so callers don't each re-derive it from the in check. Roughly:
class Resolution(enum.Enum):
UNRESOLVED = "unresolved" # Crossplane hasn't fetched it yet (key absent)
ABSENT = "absent" # resolved, but nothing matched
PRESENT = "present" # resolved and found
def resolve_required(req, name) -> tuple[Resolution, dict | None]: ...
A list variant (and possibly the same for require_schema/get_required_schema, which has the analogous distinction) would round it out. Naming/placement and whether this is an enum vs. a small result type are open — happy to send a PR once there's a preferred shape.
- Langage dominant
- Python
- Étoiles
- 12
- Forks
- 15
- Merge moyen
- 1 j 12 h
- PR mergées (30 j)
- 7
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de crossplane/function-sdk-python
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
crossplane/function-sdk-python#199 · 4 commentaires · 3 réactions ·
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
crossplane/function-sdk-python#129 · 1 commentaire ·
-
Dependency Dashboard Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 10/100
-
enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
crossplane/function-sdk-python#5 · 1 commentaire ·
Toutes les issues de crossplane/function-sdk-python
Issues similaires
-
essnmx good first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 95/100
-
[Feature] 奇物选择添加优先级 Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
syfoud/Simulated_Scepter#174 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Giskard-AI/giskard-oss#2840 · 1 commentaire ·
-
A claim comment carrying the issue number is silently declined while the workflow reports success Ouvertearea: repo bug perceived difficulty: 2
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
yeti-platform/yeti#1380 ·