airbytehq / airbytehq/airbyte

Optional traffic policy fields and annotation parity for chart v2 Services

未關閉
#83,757 2 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視
autoteam community team/use
主要語言
Python
星號
22.1k
分支
5.4k
平均合併
5 小時
30 天內合併 PR
671

描述

The airbyte-platform PR template directs contributions here, so filing this as an issue with a ready patch.

## Proposed change

Adds optional, values-driven fields to every Service template in charts/v2/airbyte:

* `internalTrafficPolicy` (Cluster or Local)
* `externalTrafficPolicy` (rendered only for NodePort or LoadBalancer Services, as the API requires)
* `trafficDistribution` (for example `PreferClose`, Kubernetes 1.30+)
* optional `appProtocol` on port entries
* an `annotations` block for the two Services that currently lack one (temporal, stigg-sidecar), bringing them to parity with the other seven

All fields are optional and gated with `with`/`if`. A chart rendered without them is byte-identical to today.

## Why

Self-managed deployments on multi-AZ clusters want zone-aware routing for in-cluster traffic to Airbyte's Services to reduce cross-AZ latency and data transfer cost. These are standard Kubernetes Service fields, but none of the nine Service templates currently accept them, so the only option is patching Services after `helm upgrade`, which leaves cluster state unmanaged by the chart.

## How

Follows the chart's existing conventions: guarded blocks in the templates, `# --` helm-docs annotations for every new key in values.yaml, defaults empty. No changes to autogenerated files under templates/config.

A ready patch is available at https://github.com/airbytehq/airbyte-platform/compare/main...RafPe:airbyte-platform:chart-service-field-passthroughs; happy to open it as a PR wherever you prefer.

---
**Internal Tracking:** https://github.com/airbytehq/oncall/issues/13247

貢獻指南

開啟貢獻指南

研究方向

Start with the Service templates under charts/v2/airbyte and values.yaml. Compare their existing guarded conventions and helm-docs annotations, then render the chart with defaults and configured traffic-policy, appProtocol, and annotation values. Done means default output remains byte-identical and the optional fields render only where specified.

由索引模型根據 Issue 內容生成。

評估

技術堆疊
helm, kubernetes
領域
devops, infrastructure
Issue 類型
功能
難度
3/5
預估耗時
1-2 天
活躍度
冷清
描述清晰度
描述清楚
新手友好度
55/100

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。