-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[maven-release-plugin] prepare release kubernetes-1.27.3
- Loading branch information
Showing
1 changed file
with
2 additions
and
2 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6813ac7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As this is a last commit of the latest release and there is no option to create an issue in this project, i must express my frustration in the comment.
So nothing personal. Just hoping that it will reach someone.
What's up with "label" and "containerTemplate"?
If those are deprecated why are they still in a majority of examples?
What is the replacement for both?
Using "label" does not work properly any more. It breaks pod deployment with just redeploying a pod template from some previous build of any job that uses same label for the pod. How can I set a static prefix for the pod label now that it not the job name? It does not seem to be possible now.
With containerTemplate we ware able to set the version of the container. I can not find any alternative with using yamlFile for the template and overwriting some container parameters. If it was deprecated in favour of yaml parameter, how about making it work in combination with yamlFile first and than deprecating what was working?
6813ac7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume you use declarative since the warnings have been added there.
label
has been deprecated for long. Omit it, and your pod templates will use a generated label instead, unique to your build.There is a regression being handled for the problem you described.
About your use case requiring
containerTemplate
, do I understand right that you are providing some kind of template usingyamlFile
where you patch some fields (like image) usingcontainerTemplate
field?