Github user foxish commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19468#discussion_r145271340
  
    --- Diff: pom.xml ---
    @@ -2649,6 +2649,13 @@
         </profile>
     
         <profile>
    +      <id>kubernetes</id>
    +      <modules>
    +        <module>resource-managers/kubernetes/core</module>
    --- End diff --
    
    We expected it would be a multi-module 
(https://github.com/apache-spark-on-k8s/spark/tree/branch-2.2-kubernetes/resource-managers/kubernetes).
 The other modules being - configuration files for the docker images and 
integration tests. The docker files are pretty much static configuration files, 
so, moving that instead to a directory like `conf/kubernetes/` might make sense 
also. I'm not sure where the integration tests will be eventually. 
    
    cc @mccheah @ssuchter


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to