-
Notifications
You must be signed in to change notification settings - Fork 8
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
Azure/personal/dns warning #78
Conversation
6202902
to
a12d9da
Compare
src/mountscript.sh
Outdated
if is_present_in_etc_hosts "$nfs_host"; then | ||
eecho "Mount failed!" | ||
eecho "Detected entry for $nfs_host in /etc/hosts." | ||
eecho "[Action Required]: Remove or comment out the entry for $nfs_host in /etc/hosts for MOUNT to work." | ||
exit 1 | ||
fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we need to be very careful that we don't raise a red flag, so let's first resolve the IP and then check for the entire
IP[[:space:]]+fqdn so that we only complain when user has set the resolver config such that the address is resolving from /etc/hosts.
lib/common.sh
Outdated
eecho "[FATAL] Detected entry $ipv4_addr $hname in /etc/hosts." | ||
eecho "$hname is resolving into $ipv4_addr from /etc/hosts." | ||
eecho "Remove the entry for "$hname" in /etc/hosts to ensure proper handling of endpoint IP address changes by AZNFS." | ||
return 1 | ||
else | ||
wecho "[FATAL] Detected entry $ipv4_addr $hname in /etc/hosts." 1>/dev/null | ||
wecho "Remove the entry for "$hname" in /etc/hosts to ensure proper handling of endpoint IP address changes by AZNFS." 1>/dev/null |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove dot at the end of each log, we don't add dot at the end of logs in other places, isn't it?
also why do you want to suppress log coming on stdout at L242 243, let it come on stdout
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here, I have suppressed the this log, as otherwise value of new_ip in aznfswatchdog gets updated to this error log
L 315: new_ip=$(resolve_ipv4 "$l_host" "false").
No description provided.