Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

selector not generated for Deployment when upgrade extensions/v1beta1 to apps/v1 #34

Closed
isaactl opened this issue Aug 4, 2020 · 4 comments
Labels
question Further information is requested

Comments

@isaactl
Copy link

isaactl commented Aug 4, 2020

background

we have a old version(extensions/v1beta1) of configuration for Deployment that didn't set selector field, this has been running in k8s 1.14. While migrating to k8s 1.16, we found extensions/v1beta1 has deprecated. for the new version 'apps/v1', the selector field is now a required field according to schema. than i use this plugin to upgrade configuration in cluster

issue

although we have the apiVersion upgraded, but the selector field is not generated, and cause some issues to do further helm upgrade

Any thoughts, ideas will be appreciated!

@hickeyma
Copy link
Collaborator

hickeyma commented Aug 4, 2020

@isaactl If you update the chart to contain the selector, does it then work when you do an upgrade? If not what is the error output?

@hickeyma hickeyma added the question Further information is requested label Aug 4, 2020
@isaactl
Copy link
Author

isaactl commented Aug 5, 2020

hi @hickeyma yes, in that case, i can do the upgrade without an issue.
my scenario is fetch the manifest from k8s and do the upgrade based on that manifest(for example, update the image field only). since the selector is missing, the update manifest become invalid. i've tried kubectl convert, it does generate the selector. i can certainly add this field by myself, just wondering if mapkubeapis can also support this? thanks!

@hickeyma
Copy link
Collaborator

hickeyma commented Aug 5, 2020

@isaactl It is good that you are able to progress and not blocked.

i can certainly add this field by myself, just wondering if mapkubeapis can also support this?

It is a good point that you raise. I would however prefer keeping the plugin to do the sole job of updating release APIs. I want to avoid introducing additional capabilities as there are numerous other updates that could be added like status etc. The problem with this is that the code base would balloon and a limit would be found on what could be supported or not. The plugin should be simple.

Hope this answers your questions.

@isaactl
Copy link
Author

isaactl commented Aug 5, 2020

@hickeyma i understand keep this plugin simple is important, so i'll close this issue. thanks for the reply!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants