Conversation
|
|
||
| def get_docker_cache_config_and_mirrors(layout, cluster_config): | ||
| """ | ||
| generate hived config from layout.yaml and config.yaml |
There was a problem hiding this comment.
These comments are incorrect.
| logger = get_logger(__name__) | ||
|
|
||
|
|
||
| def get_docker_cache_config_and_mirrors(layout, cluster_config): |
There was a problem hiding this comment.
I think this function can be removed. During installation, using config.yml + services-configuration.yaml.template is enough.
| return { "skus": skus } | ||
|
|
||
|
|
||
| def get_docker_cache_config_and_mirrors(layout, cluster_config): |
There was a problem hiding this comment.
I think this function can be removed. During installation, using config.yml + services-configuration.yaml.template is enough.
| service_port: 5000 | ||
| service_nodeport: 30500 | ||
| remote_url: https://registry-1.docker.io | ||
| registry-htpasswd: dGVzdDokMnkkMDUkRUZiaWphaHovMHl4UC5xMFk1VW52TzljU2hHMThCRzM3QzBHNFhoRmFtTXdTUXdQLjBqQi4KCg== No newline at end of file |
There was a problem hiding this comment.
Is it a default passwd? What is the exact meaning of this string?
There was a problem hiding this comment.
It's test test processed by htpasswd tool and base64 encoded, commented out in new commit.
src/drivers/deploy/service.yaml
Outdated
|
|
||
| prerequisite: | ||
| - cluster-configuration | ||
| - drivers |
There was a problem hiding this comment.
Please remove this.
You can ignore services with cluster-type: yarn because they are not supported now.
src/zookeeper/deploy/service.yaml
Outdated
|
|
||
| prerequisite: | ||
| - cluster-configuration | ||
| - docker-cache |
There was a problem hiding this comment.
You can ignore services with cluster-type: yarn because they are not supported now.
contrib/kubespray/quick-start/services-configuration.yaml.template
Outdated
Show resolved
Hide resolved
| - name: run add_docker_cache_config.py script to add docker cache config | ||
| ansible.builtin.command: | ||
| argv: | ||
| - sudo |
There was a problem hiding this comment.
I notice there's "become: true" already. So you can remove this sudo.
There was a problem hiding this comment.
ok, I will update this
6 participants
close #5219