bh90210 / dron8s

Yet another Kubernetes plugin for Drone using dynamic Server Side Apply to achieve --server-side parity for your CI-CD pipelines
http://plugins.drone.io/bh90210/dron8s
The Unlicense
20 stars 5 forks source link

Document conflict condition #24

Closed Chronostasys closed 3 years ago

Chronostasys commented 3 years ago

Hi, I'm using dron8s plugin in my drone to update my services in k8s cluster. I found that when I trigger cd on a deployment previously created by kubectl client, a conflict error will occur(like the image below).
image
And it took me some time to find out I have to transfer ownership to dron8s to avoid this error(reading this doc)
However, I can't find the field manager's name in dron8s documentation and have to search it in the drone8s source code.
https://github.com/bh90210/dron8s/blob/03fc616ea0bc8a612ee3ae1b95f9c4c2a385ffa2/main.go#L155
I think it might be better to document it in the project readme, or add a force-conflict flag in the ssa request. It may save a lot of time for people who want to migrate existing deployment to dron8s.

bh90210 commented 3 years ago

thank you so much for opening the ticket, I will look into to asap!