Skip to content
This repository has been archived by the owner on Oct 20, 2022. It is now read-only.

[Fix/Plugin] Remove metadata.resourceVersion from the applied resource #252

Merged
merged 2 commits into from
Sep 2, 2020

Conversation

erdrix
Copy link
Contributor

@erdrix erdrix commented Sep 1, 2020

Q A
Bug fix? yes
New feature? no
API breaks? no
Deprecations? no
Related tickets fixes #251
License Apache 2.0

What's in this PR?

This PR removes the metadata.resourceVersion field before applying the resource when the plugin is used.

Why?

Without this, once the plugin has performed an operation that requires an apply, we aren't able to apply the changes without removing the metadata.annotations.kubectl.kubernetes.io/last-applied-configuration field.

Checklist

  • Implementation tested
  • Logging code meets the guideline
  • User guide and development docs updated (if needed)
  • Append changelog

@erdrix erdrix added the bug Something isn't working label Sep 1, 2020
@erdrix erdrix self-assigned this Sep 1, 2020
@erdrix erdrix requested review from fdehay and cscetbon September 1, 2020 09:12
@erdrix erdrix merged commit bd7c5ef into Orange-OpenSource:master Sep 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kubectl apply fails after rolling restart triggered by plugin
2 participants