✨ Capture catalog content timestamp in ClusterCatalogStatus

[rashmigottipati]

fixes #306 out of the epic #239

[codecov]

Codecov Report

Attention: Patch coverage is 64.28571% with 5 lines in your changes missing coverage. Please review.

Project coverage is 34.27%. Comparing base (1213f56) to head (de3210f).

Files	Patch %	Lines
...rnal/controllers/core/clustercatalog_controller.go	62.50%	1 Missing and 2 partials ⚠️
api/core/v1alpha1/zz_generated.deepcopy.go	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #334      +/-   ##
==========================================
+ Coverage   34.13%   34.27%   +0.13%     
==========================================
  Files          15       15              
  Lines         665      674       +9     
==========================================
+ Hits          227      231       +4     
- Misses        414      417       +3     
- Partials       24       26       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[grokspawn]

It won't let me put a comment on it, but at https://github.com/operator-framework/catalogd/pull/334/files#diff-d446855f2187e6d61c4719c3202fcb21e4aea9b09e09f9e1d01419f341d8fc9fR51
can you update the print directive along the lines of

//+kubebuilder:printcolumn:name=UnpackedTime,type=date,JSONPath=`.metadata.catalogContentTimestamp

Totally flexible about the name ... but Age doesn't seem right here to me.

[everettraven]

Looks good to me based on my understanding of the goals of this PR.

[rashmigottipati]

Merging this PR since it was approved. upgrade-e2e will fail until a new release of catalogd is cut with the gvk updates.