From ef7941791576cca31acc12232cad402c6ef5fdd5 Mon Sep 17 00:00:00 2001
From: Yifan Xiong <yifan.xiong@microsoft.com>
Date: Wed, 8 Jan 2020 19:43:35 +0800
Subject: [PATCH] Remove owner reference for priority class

Remove framework owner reference for priority class, ref #4117.
---
 src/rest-server/src/models/v2/job/k8s.js | 28 ------------------------
 1 file changed, 28 deletions(-)

diff --git a/src/rest-server/src/models/v2/job/k8s.js b/src/rest-server/src/models/v2/job/k8s.js
index 9a85323a74..51304d44a5 100644
--- a/src/rest-server/src/models/v2/job/k8s.js
+++ b/src/rest-server/src/models/v2/job/k8s.js
@@ -606,33 +606,6 @@ const createPriorityClass = async (frameworkName, priority) => {
   }
 };
 
-const patchPriorityClassOwner = async (frameworkName, frameworkUid) => {
-  try {
-    const headers = {...launcherConfig.requestHeaders};
-    headers['Content-Type'] = 'application/merge-patch+json';
-    await axios({
-      method: 'patch',
-      url: launcherConfig.priorityClassPath(`${encodeName(frameworkName)}-priority`),
-      headers,
-      httpsAgent: apiserver.ca && new Agent({ca: apiserver.ca}),
-      data: {
-        metadata: {
-          ownerReferences: [{
-            apiVersion: launcherConfig.apiVersion,
-            kind: 'Framework',
-            name: encodeName(frameworkName),
-            uid: frameworkUid,
-            controller: false,
-            blockOwnerDeletion: false,
-          }],
-        },
-      },
-    });
-  } catch (error) {
-    logger.warn('Failed to patch owner reference for priority class', error);
-  }
-};
-
 const deletePriorityClass = async (frameworkName) => {
   try {
     await axios({
@@ -850,7 +823,6 @@ const put = async (frameworkName, config, rawConfig) => {
   }
   // do not await for patch
   auths.length && patchSecretOwner(frameworkName, response.data.metadata.uid);
-  patchPriorityClassOwner(frameworkName, response.data.metadata.uid);
 };
 
 const execute = async (frameworkName, executionType) => {