Merge pull request #179 from adrienjt/different-default-priority

Different default priority

pkg/model/delegatepod/model.go

@@ -1,5 +1,5 @@
 /*
- * Copyright 2020 The Multicluster-Scheduler Authors.
+ * Copyright 2023 The Multicluster-Scheduler Authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -78,6 +78,9 @@ func MakeDelegatePod(proxyPod *corev1.Pod) (*v1alpha1.PodChaperon, error) {
 		delegatePod.Spec.SchedulerName = common.CandidateSchedulerName
 	}
 
+	// support different default priority in target cluster
+	delegatePod.Spec.Priority = nil
+
 	return delegatePod, nil
 }
 

test/e2e/e2e.sh

@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
 #
-# Copyright 2022 The Multicluster-Scheduler Authors.
+# Copyright 2023 The Multicluster-Scheduler Authors.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -65,6 +65,9 @@ webhook_ready 1 admiralty multicluster-scheduler-controller-manager multicluster
 # unfortunately, we can't use kubectl to taint nodes with node.kubernetes.io/network-unavailable
 # some system defaulting admission controller overwriting the reserved taint?
 
+# fix GH issue #152: different default priority in target cluster
+k 2 apply -f test/e2e/priorityclass.yaml
+
 argo_test 1 2
 follow_test 1 2
 logs_test 1 2

test/e2e/priorityclass.yaml

@@ -0,0 +1,6 @@
+apiVersion: scheduling.k8s.io/v1
+kind: PriorityClass
+metadata:
+  name: default
+value: 1000
+globalDefault: true