karmada-io
/
karmada
mirror of https://github.com/karmada-io/karmada.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Remove hardcoded condition type. (#166) 

Signed-off-by: RainbowMango <renhongcai@huawei.com>
This commit is contained in:
Hongcai Ren 2021-02-05 17:25:55 +08:00 committed by GitHub
parent 3305a54b11
commit c5502fc434
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 10 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
cmd/scheduler/main.go
View File

@ -12,7 +12,6 @@ func main() {
logs.InitLogs()
defer logs.FlushLogs()
// TODO: remove dependency on apiserver
stopChan := apiserver.SetupSignalHandler()
if err := app.NewSchedulerCommand(stopChan).Execute(); err != nil {

24
pkg/controllers/execution/execution_controller.go
View File

@ -6,7 +6,7 @@ import (
"k8s.io/apimachinery/pkg/api/errors"
"k8s.io/apimachinery/pkg/api/meta"
v1 "k8s.io/apimachinery/pkg/apis/meta/v1"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
"k8s.io/client-go/kubernetes"
"k8s.io/client-go/tools/record"
@ -90,8 +90,8 @@ func (c *Controller) syncWork(propagationWork *policyv1alpha1.PropagationWork) (
// isResourceApplied checking weather resource has been dispatched to member cluster or not
func (c *Controller) isResourceApplied(propagationWorkStatus *policyv1alpha1.PropagationWorkStatus) bool {
for _, condition := range propagationWorkStatus.Conditions {
if condition.Type == "Applied" {
if condition.Status == v1.ConditionTrue {
if condition.Type == policyv1alpha1.WorkApplied {
if condition.Status == metav1.ConditionTrue {
return true
}
}
@ -191,7 +191,7 @@ func (c *Controller) syncToClusters(cluster *v1alpha1.Cluster, propagationWork *
return err
}
clusterObj, err := clusterDynamicClient.DynamicClientSet.Resource(dynamicResource).Namespace(workload.GetNamespace()).Get(context.TODO(), workload.GetName(), v1.GetOptions{})
clusterObj, err := clusterDynamicClient.DynamicClientSet.Resource(dynamicResource).Namespace(workload.GetNamespace()).Get(context.TODO(), workload.GetName(), metav1.GetOptions{})
if err != nil {
klog.Errorf("Failed to get resource %v from member cluster, err is %v ", workload.GetName(), err)
return err
@ -235,16 +235,12 @@ func (c *Controller) removeFinalizer(propagationWork *policyv1alpha1.Propagation
// updateAppliedCondition update the Applied condition for the given PropagationWork
func (c *Controller) updateAppliedCondition(propagationWork *policyv1alpha1.PropagationWork) error {
currentTime := v1.Now()
propagationWorkApplied := "Applied"
appliedSuccess := "AppliedSuccess"
resourceApplied := "Success sync resource in member cluster"
newPropagationWorkAppliedCondition := v1.Condition{
Type: propagationWorkApplied,
Status: v1.ConditionTrue,
Reason: appliedSuccess,
Message: resourceApplied,
LastTransitionTime: currentTime,
newPropagationWorkAppliedCondition := metav1.Condition{
Type: policyv1alpha1.WorkApplied,
Status: metav1.ConditionTrue,
Reason: "AppliedSuccessful",
Message: "Manifest has been successfully applied",
LastTransitionTime: metav1.Now(),
}
propagationWork.Status.Conditions = append(propagationWork.Status.Conditions, newPropagationWorkAppliedCondition)
err := c.Client.Status().Update(context.TODO(), propagationWork)

1
pkg/util/cluster.go
View File

@ -14,7 +14,6 @@ import (
// IsClusterReady tells whether the cluster status in 'Ready' condition.
func IsClusterReady(clusterStatus *v1alpha1.ClusterStatus) bool {
for _, condition := range clusterStatus.Conditions {
// TODO(RainbowMango): Condition type should be defined in API, and after that update this hard code accordingly.
if condition.Type == v1alpha1.ClusterConditionReady {
if condition.Status == metav1.ConditionTrue {
return true