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

[postfix] set hostname via docker-compose #4353

Open
wants to merge 1 commit into
base: staging
Choose a base branch
from

Conversation

feldsam
Copy link
Member

@feldsam feldsam commented Dec 1, 2021

@andryyy hi, what you think about this change? I like to prevent setting myhostname in main.cf file, which is git tracked and make my git repo not clean and I have to do chekout before update. I don't use alltimes update.sh, especially when there are small changes in only web. Yes, I know I can do "assume-unchaged", but this is cleaner in my opinion.

I tested it using docker-compose exec postfix-mailcow bash -c 'postconf|grep myhostname' and I also try to ping "postfix" hostname from SOGo, so it doesnt breaks internal networking and name resolution.

Signed-off-by: Kristian Feldsam feldsam@gmail.com

Prevent setting myhostname in main.cf file, which is git tracked

Signed-off-by: Kristian Feldsam <feldsam@gmail.com>
@feldsam feldsam changed the base branch from master to staging Dec 1, 2021
@andryyy
Copy link
Member

andryyy commented Dec 1, 2021

I think there was a major problem with this regarding DNS lookups.

How does acme-mailcow handle this?

How long have you been running it? Please don't tell me it's untested. :) That's for an issue then.

@feldsam
Copy link
Member Author

feldsam commented Dec 1, 2021

this is why I asking you about your opinion :) I tested it only few minutes :) I consider this as discussion with code example.

what doest acme have with postfix hostname?

@andryyy
Copy link
Member

andryyy commented Dec 2, 2021

@feldsam
Copy link
Member Author

feldsam commented Dec 2, 2021

hi, I tested using ping in Sogo and Dovecot and it working good

root@4dee60befa84:/# ping postfix
PING postfix(mailcowdockerized_postfix-mailcow_1.mailcowdockerized_mailcow-network (fd4d:6169:6c63:6f77::d)) 56 data bytes
64 bytes from mailcowdockerized_postfix-mailcow_1.mailcowdockerized_mailcow-network (fd4d:6169:6c63:6f77::d): icmp_seq=1 ttl=64 time=0.078 ms
64 bytes from mailcowdockerized_postfix-mailcow_1.mailcowdockerized_mailcow-network (fd4d:6169:6c63:6f77::d): icmp_seq=2 ttl=64 time=0.146 ms
64 bytes from mailcowdockerized_postfix-mailcow_1.mailcowdockerized_mailcow-network (fd4d:6169:6c63:6f77::d): icmp_seq=3 ttl=64 time=0.109 ms
^C
--- postfix ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 62ms
rtt min/avg/max/mdev = 0.078/0.111/0.146/0.027 ms
root@4dee60befa84:/# ping -4 postfix
PING postfix (172.22.1.253) 56(84) bytes of data.
64 bytes from mailcowdockerized_postfix-mailcow_1.mailcowdockerized_mailcow-network (172.22.1.253): icmp_seq=1 ttl=64 time=0.207 ms
64 bytes from mailcowdockerized_postfix-mailcow_1.mailcowdockerized_mailcow-network (172.22.1.253): icmp_seq=2 ttl=64 time=0.083 ms
64 bytes from mailcowdockerized_postfix-mailcow_1.mailcowdockerized_mailcow-network (172.22.1.253): icmp_seq=3 ttl=64 time=0.114 ms
64 bytes from mailcowdockerized_postfix-mailcow_1.mailcowdockerized_mailcow-network (172.22.1.253): icmp_seq=4 ttl=64 time=0.146 ms

when I try to ping FQDN it works also correctly

root@4dee60befa84:/# ping maildev.feldhost.cz
PING maildev.feldhost.cz (185.174.168.25) 56(84) bytes of data.
64 bytes from maildev.feldhost.cz (185.174.168.25): icmp_seq=1 ttl=64 time=0.073 ms
64 bytes from maildev.feldhost.cz (185.174.168.25): icmp_seq=2 ttl=64 time=0.122 ms
^C
--- maildev.feldhost.cz ping statistics ---
2 packets transmitted, 2 received, 0% packet loss, time 62ms
rtt min/avg/max/mdev = 0.073/0.097/0.122/0.026 ms
root@4dee60befa84:/# ping -6 maildev.feldhost.cz
PING maildev.feldhost.cz(maildev.fedlhost.cz (2a0b:a900::b9ff:feae:a819)) 56 data bytes
64 bytes from maildev.fedlhost.cz (2a0b:a900::b9ff:feae:a819): icmp_seq=1 ttl=64 time=0.057 ms
64 bytes from maildev.fedlhost.cz (2a0b:a900::b9ff:feae:a819): icmp_seq=2 ttl=64 time=0.099 ms
64 bytes from maildev.fedlhost.cz (2a0b:a900::b9ff:feae:a819): icmp_seq=3 ttl=64 time=0.104 ms
^C
--- maildev.feldhost.cz ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 41ms

@andryyy
Copy link
Member

andryyy commented Dec 2, 2021

@DerLinkman DerLinkman deleted the branch mailcow:staging Jan 21, 2022
@DerLinkman DerLinkman closed this Jan 21, 2022
@DerLinkman DerLinkman reopened this Jan 21, 2022
@feldsam feldsam force-pushed the staging branch 2 times, most recently from 2aead31 to 880a68d Compare Mar 22, 2022
@DerLinkman DerLinkman deleted the branch mailcow:staging May 5, 2022
@DerLinkman DerLinkman closed this May 5, 2022
@DerLinkman DerLinkman reopened this May 5, 2022
@milkmaker
Copy link
Contributor

milkmaker commented Jul 12, 2022

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@milkmaker milkmaker added the stale Please update the issue with current status, unclear if it's still open/needed. label Jul 12, 2022
@milkmaker milkmaker closed this Jul 20, 2022
@feldsam feldsam reopened this Jul 20, 2022
@milkmaker milkmaker closed this Jul 28, 2022
@feldsam feldsam reopened this Jul 28, 2022
@feldsam
Copy link
Member Author

feldsam commented Jul 28, 2022

I have to test it, need time

@milkmaker milkmaker removed the stale Please update the issue with current status, unclear if it's still open/needed. label Jul 28, 2022
@DerLinkman
Copy link
Contributor

DerLinkman commented Jul 29, 2022

I can and would join the test if you tell me on what I'll have to look for 😅

@DerLinkman DerLinkman added enhancement neverstale Bot doesn't mark the issue or PR as stale labels Sep 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement neverstale Bot doesn't mark the issue or PR as stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants