kong icon indicating copy to clipboard operation
kong copied to clipboard

feat(clustering) add cert exp date for clustering dp endpoint

Open artomal opened this issue 3 years ago • 7 comments

Summary

Proposing to add a new field to /clustering/data-planes endpoint which is data plane certificate expiration date. I think it is important information to have in one place which would be convenient knowing that if you run a Hybrid setup and have a bunch of data planes and if you are in PKI mode you would like to know which data planes are online and when their certificate are going to expire or maybe have already expired.

Today we have only 2 options to keep track of that which is:

  • Keeping track of certificate expiration dates in a separate system which is stored during certificate generation.
  • Finding out that information from the control plane logs which would indicate that certificate has already expired and connection is broken.

Full changelog

  • Added new field cert_exp_date

artomal avatar Apr 14 '21 02:04 artomal

Hi Arturas, I have just changed the base branch used here to master - as we are getting rid of next. Apologies for the disruption, and let me know if you need help with a potential rebase.

gszr avatar Apr 15 '21 18:04 gszr

Hi Arturas, I have just changed the base branch used here to master - as we are getting rid of next. Apologies for the disruption, and let me know if you need help with a potential rebase.

Hey GS! No, that is good, thank you for letting me know! ;)

artomal avatar Apr 15 '21 20:04 artomal

@artomal any updates on this, should this be marked as ready for review?

dndx avatar Apr 28 '21 17:04 dndx

@artomal any updates on this, should this be marked as ready for review?

Yes, sorry for a delay ;) thanks for pinging me!

artomal avatar May 04 '21 16:05 artomal

👍 We can also to expose it in prometheus metrics but it's a different story. Maybe a warning message if you are starting Kong DP with an expired cert is also useful for debugging.

Is there a ticket to track making this a metric. It has been identified as a missing piece of critical observability as long as the cert renewal process is manual for customers.

wyndigo avatar Aug 26 '21 21:08 wyndigo

Ping @fffonion ^^

dndx avatar Sep 01 '21 17:09 dndx

@wyndigo we created a card to track adding this functionality into Prometheus internally. I removed the comment from @fffonion because it contained internal information.

dndx avatar Sep 01 '21 17:09 dndx

Closing this due to lack of activity. Please re-open if needed.

hbagdi avatar Oct 25 '22 21:10 hbagdi