Added ObservedGeneration to solrcloud status#722
Added ObservedGeneration to solrcloud status#722RavinaChidambaram wants to merge 4 commits intoapache:mainfrom
Conversation
Signed-off-by: RavinaChidambaram <cravina.19@gmail.com>
gerlowskija
left a comment
There was a problem hiding this comment.
I left one in-line question, but otherwise this LGTM. It seems to bring out status a little closer into line with best practices.
I'm ambivalent on the "lastUpdated" timestamp field that was described in the issue description. It seems like there's less strong of a precedent around that field, and it's maybe more subject to some of the "false-signal" concerns I expressed in #650. But I'm not against it - if others think it's worthwhile and are willing to add it, I'll hold off merging for a bit. But otherwise I'll add a Chart.yaml change-entry and merge this sometime in the next few days!
Thanks @RavinaChidambaram !
api/v1beta1/solrcloud_types.go
Outdated
| BackupRepositoriesAvailable map[string]bool `json:"backupRepositoriesAvailable,omitempty"` | ||
|
|
||
| // ObservedGeneration represents the most recent generation observed for this SolrCloud. | ||
| ObservedGeneration int64 `json:"observedGeneration"` |
There was a problem hiding this comment.
[Q] Are there any edge-cases where there won't be an observedGeneration, say, immediately after creating a SolrCloud but before the operator has created any of the resources backing it?
Assuming that case exists, I guess the 'observedGeneration' would just be '0' in that case, and that's probably fine unless it'd be better to make the field an optional *int64?
|
Thanks for the feedback and approval. I’m currently working on considering other scenarios as well around Also lets just address |
Signed-off-by: RavinaChidambaram <cravina.19@gmail.com>
Signed-off-by: RavinaChidambaram <cravina.19@gmail.com>
Signed-off-by: RavinaChidambaram <cravina.19@gmail.com>
Implemented #650
Have added observed generation to the status of the solrcloud CR to track the transition of the solrcloud.