[ 
https://issues.apache.org/jira/browse/YUNIKORN-638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17330929#comment-17330929
 ] 

Amit Sharma commented on YUNIKORN-638:
--------------------------------------

[~yuchaoran2011] & [~wwei], I can pick this up. 

Question: there are a lot of similar opportunities to allow properties to be 
configurable. Is the general direction to use [Scheduler 
Conf|https://github.com/apache/incubator-yunikorn-k8shim/blob/9f0e86fb03b9991d38fde1bd8b6c0f17006e906c/pkg/conf/schedulerconf.go#L53]
 for this purpose?

Or do we split the properties based on Performance, Environment etc. 

> Make placeholder image configurable
> -----------------------------------
>
>                 Key: YUNIKORN-638
>                 URL: https://issues.apache.org/jira/browse/YUNIKORN-638
>             Project: Apache YuniKorn
>          Issue Type: Improvement
>          Components: shim - kubernetes
>            Reporter: Chaoran Yu
>            Assignee: Amit Sharma
>            Priority: Minor
>
> The placeholder image is currently hard-coded as a constant at 
> https://github.com/apache/incubator-yunikorn-k8shim/blob/v0.10.0/pkg/common/constants/constants.go#L55.
>  In many sectors and enterprises, it's common to have restricted internet 
> access. When replacing the placeholder image with something else, the entire 
> k8shim image also needs to be rebuilt. It's more inconvenient when different 
> deployment environments (dev/test, staging and prod) can't access images in 
> another environment. 
> It would be better if the placeholder image can be configured in the Helm 
> chart: 
> https://github.com/apache/incubator-yunikorn-release/blob/master/helm-charts/yunikorn/values.yaml.
>  That would make CI/CD easier too



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to