Skip to content

More shortcuts#151

Open
kverma12 wants to merge 1 commit into
dgkanatsios:mainfrom
kverma12:master
Open

More shortcuts#151
kverma12 wants to merge 1 commit into
dgkanatsios:mainfrom
kverma12:master

Conversation

@kverma12

Copy link
Copy Markdown

adding more short cuts..
Adding 1.19 above version shortcuts

@dgkanatsios dgkanatsios left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the PR, some comments + clarification questions!

Comment thread a.core_concepts.md

```bash
kubectl create namespace mynamespace
kubectl create ns mynamespace

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since it's the same command as the one in the previous line, can you add it with an explanatory comment?

Comment thread a.core_concepts.md
kubectl run nginx --image=nginx --restart=Never --dry-run=client -n mynamespace -o yaml > pod.yaml
```

Above version 1.19, no need of --restart=Never. Use the following:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe you mean, "version 1.19 and later"?

Comment thread a.core_concepts.md
kubectl run nginx --image=nginx --restart=Never --dry-run=client -n mynamespace -o yaml > pod.yaml
```

Above version 1.19, no need of --restart=Never. Use the following:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also explain why --restart=Never is not needed?

Comment thread a.core_concepts.md

```bash
kubectl create -f pod.yaml -n mynamespace
kubectl apply if pod.yaml -n mynamespace

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mean apply -f?

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 this pull request may close these issues.

2 participants