-
Notifications
You must be signed in to change notification settings - Fork 80
feat(component):Add observedGeneration to represent Component #205
feat(component):Add observedGeneration to represent Component #205
Conversation
Add observedGeneration to represent Component Fix crossplane#91 Signed-off-by: zhuhuijun <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add a test for this feature
copy |
Add observedGeneration to represent Component Fix crossplane#205 Signed-off-by: zhuhuijun <[email protected]>
Add observedGeneration to represent Component Fix crossplane#205 Signed-off-by: zhuhuijun <[email protected]>
233659f
to
bc182db
Compare
pkg/controller/v1alpha2/applicationconfiguration/applicationconfiguration.go
Show resolved
Hide resolved
Add observedGeneration to represent Component Fix crossplane#205 Signed-off-by: zhuhuijun <[email protected]>
bc182db
to
263dd2a
Compare
pkg/controller/v1alpha2/applicationconfiguration/applicationconfiguration_test.go
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally LGTM, please fix the conflict, thanks!
Signed-off-by: zhuhuijun <[email protected]>
…ration Signed-off-by: zhuhuijun <[email protected]>
…ration Signed-off-by: zhuhuijun <[email protected]>
Signed-off-by: zhuhuijun <[email protected]>
@@ -228,6 +228,7 @@ func (r *OAMApplicationReconciler) Reconcile(req reconcile.Request) (result reco | |||
|
|||
// execute the posthooks at the end no matter what | |||
defer func() { | |||
updateObservedGeneration(ac) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An afterthought: I don't think this is the right place to update ObservedGeneration. Because this will always update even if the reconciliation failed.
Note that I will be fixing this shortly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reconcile failed also need to add observed
Add observedGeneration to represent Component
Fix #91
Signed-off-by: zhuhuijun [email protected]