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

changed: update podman service documentation around network dns #1210

Merged
merged 1 commit into from
Feb 13, 2023

Conversation

s-fairchild
Copy link
Contributor

@s-fairchild s-fairchild commented Feb 4, 2023

What

Update podman documentation
Add network creation.
Update examples of manaully starting podman socket to start as a background job.

Tested with podman version

podman version
Client:       Podman Engine
Version:      4.3.1
API Version:  4.3.1
Go Version:   go1.19.2
Built:        Fri Nov 11 15:01:27 2022
OS/Arch:      linux/amd64

Why

The current documentation doesn't work due to podman's default podman network lacking dns. Node's fail to resolve other node names during installation/initialization.

podman network inspect podman -f '{{ .DNSEnabled }}'
false

The examples using podman system service --time=0 starts a podman socket initiated from that command. I think it's more useful to launch in the background rather than assuming user's will launch another shell.

Implications

None that I'm aware of, documentation only.

@iwilltry42
Copy link
Member

Hi @s-fairchild , thank you for this PR!
Can you please add the Podman version(s) that you tested this with, so it'll be easier for future readers to justify if this will help them?

@s-fairchild
Copy link
Contributor Author

s-fairchild commented Feb 10, 2023

@iwilltry42 Certainly! That's a good idea. I've added the podman version (4.3.1) I tested with to the top of the page.

Copy link
Member

@iwilltry42 iwilltry42 left a comment

Choose a reason for hiding this comment

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

LGTM

Add network creation.
Update page to show latest version of podman tested with.

The default "podman" network comes with dns disabled.
```bash
podman network inspect podman -f '{{ .DNSEnabled }}'
false
```

This causes multi node cluster creation to fail because nodes cannot
resolve other node hostnames on the podman network.
Creating k3d network with dns enabled resolves this issue.

Update examples of manaully starting podman socket to start as a background job.

The examples using `podman system service --time=0` starts a podman
socket initiated from that command. I think it's more useful to launch
in the background rather than assuming user's will launch another shell.
@iwilltry42
Copy link
Member

Thank you!

@iwilltry42 iwilltry42 changed the title Update podman service documentation changed: update podman service documentation around network dns Feb 13, 2023
@iwilltry42 iwilltry42 merged commit b225fb1 into k3d-io:main Feb 13, 2023
@s-fairchild s-fairchild deleted the podman-docs-update branch February 13, 2023 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants