apache/airflow

Use Kubernetes-recommended labels for airflow helm chart

Open

#34 048 ouverte le 3 sept. 2023

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)Python (16 781 forks)batch import
area:helm-chartgood first issuekind:feature

Métriques du dépôt

Stars
 (44 809 stars)
Métriques de merge PR
 (Merge moyen 7j 18h) (834 PRs mergées en 30 j)

Description

Description

Kubernetes offers a list of recommended labels for grouping resource objects. The prefix app.kubernetes.io differentiates these recommended labels from custom labels.

Why is it needed? Enhancing readability and maintainability

References https://helm.sh/docs/chart_best_practices/labels/#standard-labels https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/#labels

Use case/motivation

https://github.com/apache/airflow/blob/913eff6759bf89d635ef1df1a0db23c157e48c4b/chart/templates/webserver/webserver-deployment.yaml#L36-L40

The above labels would be changed as follows:

app.kubernetes.io/part-of: airflow
app.kubernetes.io/component: webserver
app.kubernetes.io/instance: {{ .Release.Name }}
helm.sh/chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
app.kubernetes.io/managed-by: {{ .Release.Service }}

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

Guide contributeur

Direction de recherche
Ce problème nécessite la mise à jour des étiquettes dans les modèles du chart Helm d'Airflow pour utiliser les étiquettes recommandées par Kubernetes. Plus précisément, modifiez le fichier `chart/templates/webserver/webserver deployment.yaml` (et les fichiers similaires dans le chart) pour remplacer les étiquettes personnalisées existantes par les étiquettes standard `app.kubernetes.io/*`, comme indiqué dans la description du problème. Assurez vous de la cohérence avec les bonnes pratiques de Helm et testez le chart localement avec `helm lint` et `helm install dry run`.
Stack technique
kubernetespython
Domaine
devopsinfrastructurecloud
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
KubernetesHelm
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70