Skip to content

[Chore] Move generateK8sTaskExecutionContext from AbstractParameters to K8sTaskParameters#17976

Merged
SbloodyS merged 3 commits intoapache:devfrom
ruanwenjun:dev_wenjun_polishK8sTaskExecutionContext
Feb 22, 2026
Merged

[Chore] Move generateK8sTaskExecutionContext from AbstractParameters to K8sTaskParameters#17976
SbloodyS merged 3 commits intoapache:devfrom
ruanwenjun:dev_wenjun_polishK8sTaskExecutionContext

Conversation

@ruanwenjun
Copy link
Copy Markdown
Member

Purpose of the pull request

Polish code. The generateK8sTaskExecutionContext method only needed by K8sTaskParameters

Brief change log

Remove generateK8sTaskExecutionContext method from AbstractParameters

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

Pull Request Notice

Pull Request Notice

If your pull request contains incompatible change, you should also add it to docs/docs/en/guide/upgrade/incompatible.md

@ruanwenjun ruanwenjun self-assigned this Feb 13, 2026
@ruanwenjun ruanwenjun added this to the 3.4.1 milestone Feb 13, 2026
Copy link
Copy Markdown
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

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

+1

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 60%)

See analysis details on SonarQube Cloud

@SbloodyS SbloodyS merged commit bf9a8b7 into apache:dev Feb 22, 2026
73 of 75 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants