search

cloudfoundry / cf-crd-explorations

Apache License 2.0
3 stars 2 forks source link

Explore: How to support the `updated_at` timestamp field on CF resources. #47

Open gnovv opened 3 years ago

gnovv commented 3 years ago

Context / Background Currently the "updated_at" field for the cfshim GET API endpoints is blank

Desired outcomes A clear and concise description of what you want to happen.

Acceptance Criteria / Scenarios A set of criteria by which the issue can be considered closed.

Dev Notes

Tim was considering: One thing I was thinking of was piggybacking off of the time field within the managedFields part of the metadata https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.21/#managedfieldsentry-v1-meta

gnovv commented 3 years ago

Based on a conversations it seems like the "updated_at" field is not hugely important to be perfectly accurate.

It seems fine to take the easiest approach of capturing the latest timestamp in the managedFields of the Kuberentes resource. @akrishna90 and I wrote a generic function to extract the latest timestamp and added it to all the presenter formatting functions.

If it turns out that there is a requirement for something more robust, we can reopen this issue.