fixes, minor config change
authorIan Kelling <iank@fsf.org>
Sun, 26 May 2024 21:34:24 +0000 (17:34 -0400)
committerIan Kelling <iank@fsf.org>
Sun, 26 May 2024 21:34:24 +0000 (17:34 -0400)
fai/config/distro-install-common/ethusb-static
wrt-setup-local

index c55b93dd63d23e7092ddb18e072fd9f7df994299..fc31af17798203ea40301dd1ec1d2fd166cd4dcb 100755 (executable)
@@ -30,27 +30,19 @@ shopt -s inherit_errexit 2>/dev/null ||: # ignore fail in bash < 4.4
 set -eE -o pipefail
 trap 'echo "$0:$LINENO:error: \"$BASH_COMMAND\" returned $?" >&2' ERR
 
-[[ $EUID == 0 ]] || exec sudo -E "${BASH_SOURCE[0]}" "$@"
-
 m() { printf "%s\n" "$*";  "$@"; }
 
 ## begin arg parsing ##
 
 force=false
-conf_only=false
-comment='# iank file id: ethusb-dhcp-v1'
 off=false
 while [[ $1 ]]; do
   case $1 in
-    -c)
-      conf_only=true
-      ;;
     -f)
       force=true
       ;;
     off)
       off=true
-      comment='# iank file id: ethusb-static-v1'
       ;;
     *)
       echo "$0: error unexpected argument: $1" >&2
@@ -65,14 +57,77 @@ done
 
 shopt -s nullglob
 
-# we already configured the interface once, afterwards, comment and
-# uncomment to enable/disable. This makes it so we don't depend on /p
-# being mounted.
+wiredx=1
+
+# device that has an eth0, but we aren't using it because it is
+# broken. We could just hardcode a mac comparison with `cat
+# /sys/class/net/eth0/address` but this is cooler.
+if [[ -e  /sys/class/net/eth0 ]]; then
+  bus_info=$(ethtool -i eth0 | awk '$1 == "bus-info:" { print $2 }')
+  if [[ $bus_info != usb* ]]; then
+    wiredx=2
+  fi
+fi
+
+eth_dev=eth$(( wiredx - 1 ))
+
+nm_con=$(nmcli device show $eth_dev | \
+           awk '$1 == "GENERAL.CONNECTION:" {out=$2; for(i=3;i<=NF;i++){out=out" "$i}; print out}' ||:)
+
+if [[ ! $nm_con ]]; then
+  nm_con="Wired connection $wiredx"
+fi
+
 
-conf=/etc/NetworkManager/system-connections/ethusb-static.nmconnection
-if ! $force && [[ -s $conf ]] && grep -qFx "$comment" $conf; then
-  # we already ran successfully in the past to set things this way, so
-  # do nothing.
+con_exists=false
+if nmcli con | grep -q "^$nm_con " &>/dev/null; then
+  con_exists=true
+fi
+
+declare -a args
+if $off; then
+
+  if ! $con_exists; then
+    echo "warning: no existing connection: $nm_con found in output of nmcli con"
+    exit 0
+  fi
+
+
+  tmpstr=$(nmcli con show "$nm_con" 2>/dev/null | sort -r | awk '$1 == "ipv4.method:" || $1 == "ipv4.addresses:" || $1 == "ipv4.gateway:" || $1 == "ipv4.dns:" || $1 == "GENERAL.STATE:" {print $2}' )
+  {
+    read -r ipv4_method
+    read -r ipv4_gateway
+    read -r ipv4_dns
+    read -r ipv4_addresses
+    read -r state
+  }<<<"$tmpstr"
+
+  reup=false
+  if [[ $state == activated ]]; then
+    reup=true
+  fi
+
+  if [[ $ipv4_method != auto ]]; then
+    args+=(ipv4.method auto)
+  fi
+  if [[ $ipv4_addresses != -- ]]; then
+    args+=(-ipv4.addresses "$ipv4_addresses")
+  fi
+  if [[ $ipv4_dns != -- ]]; then
+    args+=(-ipv4.dns "$ipv4_dns")
+  fi
+  if [[ $ipv4_gateway != -- ]]; then
+    # undocumented in t11 man nmcli. guessed randomly
+    args+=(ipv4.gateway 0.0.0.0)
+  fi
+  if (( ${#args[@]} >= 1 )); then
+    m nmcli con mod "$nm_con" "${args[@]}"
+    if $reup; then
+      m nmcli con up "$nm_con"
+    fi
+  else
+    echo "$0: found expected state, nothing to do."
+  fi
   exit 0
 fi
 
@@ -82,14 +137,17 @@ if [[ $(dig +short @10.2.0.1 -x 10.2.0.2 2>&1 ||:) == kd.b8.nz. ]] \
   #  we are at_home=true
 
   while read -r ip_suf host mac; do
+    if [[ ! $ip_suf || $ip_suf == \#* ]]; then
+      continue
+    fi
     if [[ $mac != usb ]]; then
       continue
     fi
     if [[ $host == ${HOSTNAME}c ]]; then
 
-      net_info="address1=10.2.0.$ip_suf/16,10.2.0.1
-dns=8.8.8.4;8.8.8.8;"
-
+      ip=10.2.0.$ip_suf/16
+      gateway=10.2.0.1
+      dns=8.8.8.4,8.8.8.8
       break
     fi
   done </p/c/host-info
@@ -102,47 +160,64 @@ else
   if ! type -p dig &>/dev/null; then
     apt-get install dig
   fi
-  ip=$(dig +short @192.168.0.25 $HOSTNAME.office.fsf.org)
-  net_info="address1=$ip/24,192.168.0.1
-dns=192.168.0.10;192.168.0.25;"
+  myip=$(dig +short @192.168.0.25 $HOSTNAME.office.fsf.org)
+  if [[ ! $ip ]]; then
+    echo "$0: error: didnt detect home network and failed to get office ip"
+    exit 1
+  fi
+  dns=192.168.0.10,192.168.0.25
+  gateway=192.168.0.1
+  ip=$myip/24
 fi
 
-wiredx=
-
-# device that has an eth0, but we aren't using it because it is
-# broken. We could just hardcode a mac comparison with `cat
-# /sys/class/net/eth0/address` but this is cooler.
-if [[ -e  /sys/class/net/eth0 ]]; then
-  bus_info=$(ethtool -i eth0 | awk '$1 == "bus-info:" { print $2 }')
-  if [[ $bus_info != usb* ]]; then
-    wiredx=2
+if ! $force && $con_exists; then
+  current=$(nmcli con show "$nm_con" 2>/dev/null | sort -r | awk '$1 == "ipv4.method:" || $1 == "ipv4.addresses:" || $1 == "ipv4.gateway:" || $1 == "ipv4.dns:" {print $2}')
+  expected="manual
+$gateway
+$dns
+$ip"
+  if [[ $current == "$expected" ]]; then
+    echo "$0: found expected state, nothing to do."
+    exit 0
   fi
 fi
 
-ethx=$(( wiredx - 1 ))
+m nmcli con mod 'Wired connection 1' ipv4.method manual ipv4.addresses $ip ipv4.gateway $gateway ipv4.dns $dns
 
+state=$(nmcli con show "$nm_con" 2>/dev/null | awk '$1 == "GENERAL.STATE:" {print $2}')
+if [[ $state == activated ]]; then
+  m nmcli con up "$nm_con"
+fi
 
 
-uuid=$(nmcli con show "Wired connection $wiredx" 2>/dev/null | awk '$1 == "connection.uuid:" {print $2}' ||:)
-if [[ ! $uuid ]]; then
-  # just a uuid that nm generated for me at some point
-  uuid=0da4c614-6a3c-3ad2-8d4b-c6eebe0814c3
-fi
+# example of down cli
+#nmcli con mod 'Wired connection 1' ipv4.method auto -ipv4.addresses 10.2.0.9/16 ipv4.gateway 0.0.0.0 -ipv4.dns "8.8.8.4,8.8.8.8"
 
 
-# This template is the result of running, for example
+# FYI: the result of running, for example
 # nmcli con mod "Wired connection 1" \
+  #         ipv4.method manual \
   #       ipv4.addresses "10.2.0.23/24" \
   #       ipv4.gateway "10.2.0.1" \
   #       ipv4.dns "8.8.8.4,8.8.8.8"
 
-# which creates a fille named "Wired connection 1.nmconnection",
-# below. I see no reason to keep the same file name, or a bunch of
+# creates a fille named "/etc/NetworkManager/system-connections/Wired connection 1.nmconnection",
+# below.
+#
+# The nmcli man page says you should just edit files in that dir and
+# then run nmcli con reload to reread them all to load your changes, but
+# I've found that to be unreliable, the systemd journal would say
+# something like "reload happened" then nothing would change in the
+# connect that the file clearly modifies, so I switched over to using
+# the command line and just ignoring those files.
+#
+# I see no reason to keep the same file name, or a bunch of
 # setting that seem irrelevant, and empty sections don't seem to do
 # anything according to the man page.
+#
 
 # [connection]
-# id=Wired connection 2
+# id=Wired connection 1
 # uuid=b0fb7694-dfe6-31a1-81fa-7c17b61515a7
 # type=ethernet
 # interface-name=eth1
@@ -160,45 +235,3 @@ fi
 # method=auto
 
 # [proxy]
-
-{
-  cat <<EOF
-[connection]
-id=Wired connection $wiredx
-uuid=$uuid
-type=ethernet
-interface-name=eth$ethx
-
-[ipv4]
-EOF
-  if $off; then
-    cat <<'EOF'
-method=auto
-EOF
-  else
-    cat <<EOF
-$net_info
-method=manual
-EOF
-  fi
-} | install -T -m0600 /dev/stdin $conf
-
-if ! $conf_only; then
-  state=$(nmcli con show $uuid 2>/dev/null | awk '$1 == "GENERAL.STATE:" {print $2}' ||:)
-
-  reup=false
-  if [[ $state == activated ]]; then
-    reup=true
-  fi
-
-  m nmcli con reload
-
-  if $reup; then
-    m nmcli con down $uuid
-    m nmcli con up $uuid
-  fi
-fi
-
-if ! grep -F "$comment" $conf; then
-  printf "%s\n" "$comment" >>$conf
-fi
index 527e4a8e72dab75f1252662f93a2087ea5bab763..092e486a401f8c75ff3b84035814e4c7295f0d3e 100755 (executable)
@@ -617,7 +617,7 @@ EOF
 esac
 
 {
-  /root/cmc-firewall-data
+  /root/cmc-firewall-data
   cat <<EOF
 ## begin no external dns for ziva
 config rule
@@ -791,7 +791,7 @@ config redirect
  option src              wan
  option src_dport        80
  option dest             lan
- option dest_ip          $l.7
+ option dest_ip          $l.2
  option proto            tcp
 config rule
  option src              wan
@@ -804,7 +804,7 @@ config redirect
  option src              wan
  option src_dport        443
  option dest             lan
- option dest_ip          $l.7
+ option dest_ip          $l.2
  option proto            tcp
 config rule
  option src              wan
@@ -1032,7 +1032,7 @@ EOF
 
 
   {
-    /root/ptr-data
+    /root/ptr-data
     cat  <<EOF
 
 local-data-ptr: "10.2.0.1 cmc.b8.nz"
@@ -1110,7 +1110,7 @@ mkdir -p /mnt/usb/tftpboot
 
 {
   # generated with host-info-update
-  /root/dnsmasq-data
+  /root/dnsmasq-data
   cat <<EOF
 # no dns
 port=0