-
Notifications
You must be signed in to change notification settings - Fork 14
add harbor cluster reconcile logic freamwork. #21
add harbor cluster reconcile logic freamwork. #21
Conversation
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.
A suggestion left
2.complete the harborcluster reconcile logic.
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.
comments left
2.use pointer ref for harborcluster
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.
comments left
) | ||
|
||
// HarborClusterReconciler reconciles a HarborCluster object | ||
type HarborClusterReconciler struct { | ||
client.Client | ||
ServiceGetter | ||
Log logr.Logger | ||
Scheme *runtime.Scheme |
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.
maybe we should add the event record, e.p: Recorder record.EventRecorder
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.
LGTM
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.
LGTM
No description provided.