[GitHub] [dolphinscheduler] rickchengx commented on pull request #12358: [Fix-12356][k8s] fix the null exception when submitting k8s task plugin

[GitBox]

rickchengx commented on PR #12358:
URL: 
https://github.com/apache/dolphinscheduler/pull/12358#issuecomment-1278750817

   > Hi. Did you check that t_ds_k8s is not used?
   
   Hi, @kezhenxu94 , thanks for the comment.
   
   * step 1: create cluster
   <img width="1456" alt="截屏2022-10-13 16 03 56" 
src="https://user-images.githubusercontent.com/38122586/195538282-f36e1fbd-fb2e-4867-8741-7913d2c4f4d4.png";>
   
   * step 2: create k8s namespace
   <img width="1453" alt="截屏2022-10-13 16 04 01" 
src="https://user-images.githubusercontent.com/38122586/195538294-6107919b-24c1-4e5a-80d6-e0ab85bb38da.png";>
   
   After my check, `t_ds_cluster` is inserted when creating cluster (step 1) 
and `t_ds_k8s_namespace` is inserted when creating k8s namespace (step 2).
   
   And `t_ds_k8s` is empty (this is also the reason of null exception because 
it tries to get config from an empty table).
   
   
   Furthermore, the k8s cluster info (including `k8s config`) is inserted to 
the table `t_ds_cluster` (not `t_ds_k8s`) as below:
   
   
https://github.com/apache/dolphinscheduler/blob/1e792186d0b1e660dc2299b46827a46c59d393f6/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/ClusterServiceImpl.java#L122-L126
   
   
   
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@dolphinscheduler.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org