Skip to content

Modify doc and init jobs.#225

Merged
pavlelee merged 2 commits into
tkestack:mainfrom
TencentCloudADP:iaas_cos_0616
Jun 16, 2026
Merged

Modify doc and init jobs.#225
pavlelee merged 2 commits into
tkestack:mainfrom
TencentCloudADP:iaas_cos_0616

Conversation

@yuzhengtao-tx

Copy link
Copy Markdown
Contributor

No description provided.

@pavlelee pavlelee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

发现一个阻塞问题,先不合并:

  • incubator/adp/charts/deploy-pre-check/templates/job.yaml:184-188 新增 COS_STS_KEY 后,紧跟着模板里原有的孤立 value: {{ .Values.global.components.s3.cos.subPath | quote }} 会被 YAML 解析成同一个 env item 的重复 value key,并覆盖 COS_STS_KEY 的真实值。
  • 我本地用 PR head 渲染验证过:--set global.components.s3.cos.stsKey=sts-key 后,解析出的 COS_STS_KEY 不是 sts-key,而是 public;默认未配置 stsKey 时也会解析成 public。这样 check.py 会误认为已开启 STS 模式,从而跳过 SecretId/SecretKey 校验。

建议移除这行孤立的 value,或把 COS_STS_KEY 放到不会被覆盖的位置,并增加一次渲染解析验证。

@yuzhengtao-tx

Copy link
Copy Markdown
Contributor Author

发现一个阻塞问题,先不合并:

  • incubator/adp/charts/deploy-pre-check/templates/job.yaml:184-188 新增 COS_STS_KEY 后,紧跟着模板里原有的孤立 value: {{ .Values.global.components.s3.cos.subPath | quote }} 会被 YAML 解析成同一个 env item 的重复 value key,并覆盖 COS_STS_KEY 的真实值。
  • 我本地用 PR head 渲染验证过:--set global.components.s3.cos.stsKey=sts-key 后,解析出的 COS_STS_KEY 不是 sts-key,而是 public;默认未配置 stsKey 时也会解析成 public。这样 check.py 会误认为已开启 STS 模式,从而跳过 SecretId/SecretKey 校验。

建议移除这行孤立的 value,或把 COS_STS_KEY 放到不会被覆盖的位置,并增加一次渲染解析验证。

已经fix

@pavlelee pavlelee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pavlelee pavlelee merged commit e11b4bb into tkestack:main Jun 16, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants