These are the items that must be done by the developer and by reviewers before the change is ready to merge. Please strikeout any items that are not applicable, but don't delete them
[ ] Developer Checks
[x] Full build with unit tests and fmt and vet checks
[ ] Unit tests added / updated
[x] No unlicensed images, no third-party code (such as from StackOverflow)
[ ] Integration tests added / updated
[x] Manual testing done (and described)
[ ] Product sweep run and passed
[ ] Developer wiki updated (and linked to this description)
[ ] Reviewer Checks
[ ] Merge request description clear and understandable
[ ] Developer checklist items complete
[ ] Functional code review (how is the code written)
[ ] Architectural review (does the code try to do the right thing, in the right way)
Intent
Add extra logging to
addTarget
, see related commits https://github.com/turbonomic/turbo-api/pull/19 https://github.com/turbonomic/turbo-go-sdk/pull/159Background
Customer sees duplicate Kubeturbo targets. We cannot reproduce the problem in-house. We need extra logging to track the problem.
Testing
Checklist
These are the items that must be done by the developer and by reviewers before the change is ready to merge. Please
strikeoutany items that are not applicable, but don't delete themAudience
(@ mention any
review/...
groups or people that should be aware of this merge request)