Skip to content

Conversation

@pingsutw
Copy link
Member

Tracking issue

NA

Why are the changes needed?

Failed to run the task since k8s injects too many env vars into the pod spec

This pull request makes a small update to the pod specification in the Kubernetes plugin machinery. The change explicitly disables service links in the generated pod specification to improve pod security and reduce unnecessary environment variables.

  • Kubernetes Pod Spec:
    • Explicitly sets EnableServiceLinks to false in the podSpec to prevent automatic injection of service environment variables into pods.

What changes were proposed in this pull request?

How was this patch tested?

Labels

Please add one or more of the following labels to categorize your PR:

  • added: For new features.
  • changed: For changes in existing functionality.
  • deprecated: For soon-to-be-removed features.
  • removed: For features being removed.
  • fixed: For any bug fixed.
  • security: In case of vulnerabilities

This is important to improve the readability of release notes.

Setup process

Screenshots

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

Docs link

Signed-off-by: Kevin Su <pingsutw@apache.org>
Copy link
Contributor

@pmahindrakar-oss pmahindrakar-oss left a comment

Choose a reason for hiding this comment

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

Can you update the PR title

@pingsutw pingsutw changed the title test Always set enableServiceLinks to false Jan 30, 2026
@pingsutw
Copy link
Member Author

updated

Signed-off-by: Kevin Su <pingsutw@apache.org>
@pingsutw pingsutw merged commit efa4f95 into v2 Jan 30, 2026
4 checks passed
@pingsutw pingsutw deleted the enableServiceLinks-false branch January 30, 2026 22:53
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.

3 participants