Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue with k8s.io/docs/tutorials/configuration/configure-redis-using-configmap/ #14428

Closed
Gyeah11 opened this issue May 20, 2019 · 3 comments · Fixed by #14448
Closed

Issue with k8s.io/docs/tutorials/configuration/configure-redis-using-configmap/ #14428

Gyeah11 opened this issue May 20, 2019 · 3 comments · Fixed by #14448

Comments

@Gyeah11
Copy link

Gyeah11 commented May 20, 2019

Problem:

They code that is shown in website from page :
https://raw.githubusercontent.com/kubernetes/website/master/content/en/examples/pods/config/redis-pod.yaml

Is different if I type the below commands from tutorial :

curl -OL https://k8s.io/examples/pods/config/redis-pod.yaml
cat <>./kustomization.yaml
resources:

  • redis-pod.yaml
    EOF

Actually the problem is that content of the file at URL on https://k8s.io/.... is buggy , fails because "image: kubernetes/redis:v1" does not exist .

Proposed Solution:

Modify the curl command as below in order to get the correct yaml file.

curl -OL https://raw.githubusercontent.com/kubernetes/website/master/content/en/examples/pods/config/redis-pod.yaml

Page to Update:
https://kubernetes.io/docs/tutorials/configuration/configure-redis-using-configmap/

N/A

If you need additional information I can be contacted at [email protected] .
Thanks

@sethmccombs
Copy link
Contributor

PR - #14448

@thecrudge
Copy link
Contributor

/close

This issue is fixed with the above PR

@k8s-ci-robot
Copy link
Contributor

@thecrudge: Closing this issue.

In response to this:

/close

This issue is fixed with the above PR

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants