redpanda icon indicating copy to clipboard operation
redpanda copied to clipboard

Call GetLicenseInfo before SetLicense

Open pvsune opened this issue 3 years ago • 0 comments
trafficstars

Cover letter

SetLicense should now be idempotent (i.e. no logs created if trying to load same license) REF. This PR adds another check in the client (i.e. operator) of AdminAPI to check license info before loading it. This PR also adds improvement in instantiating AdminAPIClient only once in the Cluster controller and passing it to funcs.

Backport Required

  • [ ] not a bug fix
  • [ ] issue does not exist in previous branches
  • [ ] papercut/not impactful enough to backport
  • [ ] v22.3.x
  • [ ] v22.2.x
  • [ ] v22.1.x

UX changes

None

Release notes

Improvements

  • Call GetLicenseInfo in the operator before SetLicense to add extra check and avoid unnecessary logs in Redpanda

pvsune avatar Nov 14 '22 15:11 pvsune