From 99f20a8cb5bd8e4ae5b2563c1b6af8a5bc4abe7f Mon Sep 17 00:00:00 2001
From: Jason Hollis <400979+CodeBleu@users.noreply.github.com>
Date: Mon, 4 Dec 2023 17:15:51 -0500
Subject: [PATCH] fix(openbsd): services & build tool

    * Added net/activator for `ifconfig` that is used with OpenBSD
    * Added sysvinit scripts for OpenBSD
    * Updated build-on-openbsd tool with additional dependencies and
      better logic
---
 cloudinit/net/activators.py          | 29 ++++++++++++++++++
 setup.py                             |  6 ++++
 sysvinit/openbsd/cloudconfig.tmpl    | 37 ++++++++++++++++++++++
 sysvinit/openbsd/cloudfinal.tmpl     | 35 +++++++++++++++++++++
 sysvinit/openbsd/cloudinit.tmpl      | 35 +++++++++++++++++++++
 sysvinit/openbsd/cloudinitlocal.tmpl | 38 +++++++++++++++++++++++
 tools/build-on-openbsd               | 46 +++++++++++++++++++++++++---
 7 files changed, 221 insertions(+), 5 deletions(-)
 create mode 100755 sysvinit/openbsd/cloudconfig.tmpl
 create mode 100755 sysvinit/openbsd/cloudfinal.tmpl
 create mode 100755 sysvinit/openbsd/cloudinit.tmpl
 create mode 100755 sysvinit/openbsd/cloudinitlocal.tmpl

diff --git a/cloudinit/net/activators.py b/cloudinit/net/activators.py
index e69da40d371..141faf2c55c 100644
--- a/cloudinit/net/activators.py
+++ b/cloudinit/net/activators.py
@@ -102,6 +102,33 @@ def bring_down_interface(device_name: str) -> bool:
         return _alter_interface(cmd, device_name)
 
 
+class IfConfigActivator(NetworkActivator):
+    @staticmethod
+    def available(target=None) -> bool:
+        """Return true if ifconfig can be used on this system."""
+        expected = "ifconfig"
+        search = ["/sbin"]
+        return subp.which(expected, search=search, target=target)
+
+    @staticmethod
+    def bring_up_interface(device_name: str) -> bool:
+        """Bring up interface using ifconfig <dev> up.
+
+        Return True is successful, otherwise return False
+        """
+        cmd = ["ifconfig", device_name, "up"]
+        return _alter_interface(cmd, device_name)
+
+    @staticmethod
+    def bring_down_interface(device_name: str) -> bool:
+        """Bring up interface using ifconfig <dev> down.
+
+        Return True is successful, otherwise return False
+        """
+        cmd = ["ifconfig", device_name, "down"]
+        return _alter_interface(cmd, device_name)
+
+
 class NetworkManagerActivator(NetworkActivator):
     @staticmethod
     def available(target=None) -> bool:
@@ -220,6 +247,7 @@ def bring_down_interface(device_name: str) -> bool:
     "netplan",
     "network-manager",
     "networkd",
+    "ifconfig",
 ]
 
 NAME_TO_ACTIVATOR: Dict[str, Type[NetworkActivator]] = {
@@ -227,6 +255,7 @@ def bring_down_interface(device_name: str) -> bool:
     "netplan": NetplanActivator,
     "network-manager": NetworkManagerActivator,
     "networkd": NetworkdActivator,
+    "ifconfig": IfConfigActivator,
 }
 
 
diff --git a/setup.py b/setup.py
index bff18362498..e40d730e550 100644
--- a/setup.py
+++ b/setup.py
@@ -132,6 +132,11 @@ def render_tmpl(template, mode=None, is_yaml=False):
         for f in glob("sysvinit/netbsd/*")
         if is_f(f)
     ],
+    "sysvinit_openbsd": lambda: [
+        render_tmpl(f, mode=0o755)
+        for f in glob("sysvinit/openbsd/*")
+        if is_f(f)
+    ],
     "sysvinit_deb": lambda: [f for f in glob("sysvinit/debian/*") if is_f(f)],
     "sysvinit_openrc": lambda: [
         f for f in glob("sysvinit/gentoo/*") if is_f(f)
@@ -156,6 +161,7 @@ def render_tmpl(template, mode=None, is_yaml=False):
     "sysvinit": "etc/rc.d/init.d",
     "sysvinit_freebsd": "usr/local/etc/rc.d",
     "sysvinit_netbsd": "usr/local/etc/rc.d",
+    "sysvinit_openbsd": "etc/rc.d",
     "sysvinit_deb": "etc/init.d",
     "sysvinit_openrc": "etc/init.d",
     "systemd": pkg_config_read("systemd", "systemdsystemunitdir"),
diff --git a/sysvinit/openbsd/cloudconfig.tmpl b/sysvinit/openbsd/cloudconfig.tmpl
new file mode 100755
index 00000000000..29c324eb7c9
--- /dev/null
+++ b/sysvinit/openbsd/cloudconfig.tmpl
@@ -0,0 +1,37 @@
+## template:jinja
+#!/bin/ksh
+
+# PROVIDE: cloudconfig
+# REQUIRE: cloudinit
+# BEFORE: sshd
+
+
+daemon="cloud-init"
+daemon_execdir="{{prefix}}/bin"
+daemon_flags="modules --mode config"
+daemon_user=root
+
+. /etc/rc.d/rc.subr
+
+rc_bg="YES" # (undefined or "YES")
+rc_usercheck="YES" # (undefined or "NO")
+
+rc_start() {
+    if test -e /etc/cloud/cloud-init.disabled; then
+        echo -n "cloud-init is disabled via cloud-init.disabled file." | logger -t ${daemon}
+    else
+        echo -n "Starting..." | logger -t ${daemon}
+        rc_exec "${daemon_execdir}/${daemon} ${daemon_flags}"
+    fi
+}
+
+rc_check() {
+    pgrep -f "${daemon}" >/dev/null
+}
+
+rc_stop() {
+    echo -n "Stopping..." | logger -t ${daemon}
+    pkill -f "${daemon}" >/dev/null
+}
+
+rc_cmd "$1"
diff --git a/sysvinit/openbsd/cloudfinal.tmpl b/sysvinit/openbsd/cloudfinal.tmpl
new file mode 100755
index 00000000000..3a2ced806ad
--- /dev/null
+++ b/sysvinit/openbsd/cloudfinal.tmpl
@@ -0,0 +1,35 @@
+## template:jinja
+#!/bin/ksh
+
+# PROVIDE: cloudfinal
+# REQUIRE: LOGIN cloudconfig
+
+daemon="cloud-init"
+daemon_execdir="{{prefix}}/bin"
+daemon_flags="modules --mode final"
+daemon_user=root
+
+. /etc/rc.d/rc.subr
+
+rc_bg="YES" # (undefined or "YES")
+rc_usercheck="YES" # (undefined or "NO")
+
+rc_start() {
+    if test -e /etc/cloud/cloud-init.disabled; then
+        echo -n "cloud-init is disabled via cloud-init.disabled file." | logger -t ${daemon}
+    else
+        echo -n "Starting..." | logger -t ${daemon}
+        rc_exec "${daemon_execdir}/${daemon} ${daemon_flags}"
+    fi
+}
+
+rc_check() {
+    pgrep -f "${daemon}" >/dev/null
+}
+
+rc_stop() {
+    echo -n "Stopping..." | logger -t ${daemon}
+    pkill -f "${daemon}" >/dev/null
+}
+
+rc_cmd "$1"
diff --git a/sysvinit/openbsd/cloudinit.tmpl b/sysvinit/openbsd/cloudinit.tmpl
new file mode 100755
index 00000000000..f8b95e162b0
--- /dev/null
+++ b/sysvinit/openbsd/cloudinit.tmpl
@@ -0,0 +1,35 @@
+## template:jinja
+#!/bin/ksh
+
+# PROVIDE: cloudinit
+# REQUIRE: cloudinitlocal
+
+daemon="cloud-init"
+daemon_execdir="{{prefix}}/bin"
+daemon_flags="init"
+daemon_user=root
+
+. /etc/rc.d/rc.subr
+
+rc_bg="YES" # (undefined or "YES")
+rc_usercheck="YES" # (undefined or "NO")
+
+rc_start() {
+    if test -e /etc/cloud/cloud-init.disabled; then
+        echo -n "cloud-init is disabled via cloud-init.disabled file." | logger -t ${daemon}
+    else
+        echo -n "Starting..." | logger -t ${daemon}
+        rc_exec "${daemon_execdir}/${daemon} ${daemon_flags}"
+    fi
+}
+
+rc_check() {
+    pgrep -f "${daemon}" >/dev/null
+}
+
+rc_stop() {
+    echo -n "Stopping..." | logger -t ${daemon}
+    pkill -f "${daemon}" >/dev/null
+}
+
+rc_cmd "$1"
diff --git a/sysvinit/openbsd/cloudinitlocal.tmpl b/sysvinit/openbsd/cloudinitlocal.tmpl
new file mode 100755
index 00000000000..c58eb97561f
--- /dev/null
+++ b/sysvinit/openbsd/cloudinitlocal.tmpl
@@ -0,0 +1,38 @@
+## template:jinja
+#!/bin/ksh
+
+# PROVIDE: cloudinitlocal
+# REQUIRE: NETWORKING
+
+# After NETWORKING because we don't want staticroute to wipe
+# the route set by the DHCP client toward the meta-data server.
+
+daemon="cloud-init"
+daemon_execdir="{{prefix}}/bin"
+daemon_flags="init -l"
+daemon_user=root
+
+. /etc/rc.d/rc.subr
+
+rc_bg="YES" # (undefined or "YES")
+rc_usercheck="YES"
+
+rc_start() {
+    if test -e /etc/cloud/cloud-init.disabled; then
+        echo -n "cloud-init is disabled via cloud-init.disabled file." | logger -t ${daemon}
+    else
+        echo -n "Starting..." | logger -t ${daemon}
+        rc_exec "${daemon_execdir}/${daemon} ${daemon_flags}"
+    fi
+}
+
+rc_check() {
+    pgrep -f "${daemon}" >/dev/null
+}
+
+rc_stop() {
+    echo -n "Stopping..." | logger -t ${daemon}
+    pkill -f "${daemon}" >/dev/null
+}
+
+rc_cmd "$1"
diff --git a/tools/build-on-openbsd b/tools/build-on-openbsd
index fd038afa2d5..611353da3ec 100755
--- a/tools/build-on-openbsd
+++ b/tools/build-on-openbsd
@@ -2,6 +2,12 @@
 
 fail() { echo "FAILED:" "$@" 1>&2; exit 1; }
 
+PYTHON=${PYTHON:-python3}
+if ! command -v ${PYTHON} >/dev/null 2>&1; then
+    echo "Please install python first."
+    exit 1
+fi
+
 # Check dependencies:
 depschecked=/tmp/c-i.dependencieschecked
 pkgs="
@@ -16,12 +22,42 @@ pkgs="
    py3-setuptools
    py3-yaml
    sudo--
+   wget
 "
-[ -f "$depschecked" ] || pkg_add "${pkgs}" || fail "install packages"
 
-touch $depschecked
+[ -f $depschecked ] || echo "Installing the following packages: $pkgs"; output=$(pkg_add -zI $pkgs 2>&1)
+
+
+if  echo "$output" | grep -q -e "Can't find" -e "Ambiguous"; then
+    echo "Failed to find or install one or more packages"
+    echo "Failed Package(s):"
+    echo "$output"
+    exit 1
+else
+    echo Successfully installed packages
+    touch $depschecked
+
+    python3 setup.py build
+    python3 setup.py install -O1 --distro openbsd --skip-build --init-system sysvinit_openbsd
+
+    echo "Installation completed."
+    RC_LOCAL="/etc/rc.local"
+    RC_LOCAL_CONTENT="
 
-python3 setup.py build
-python3 setup.py install -O1 --distro openbsd --skip-build
+/usr/lib/cloud-init/ds-identify
 
-echo "Installation completed."
+cloud-init init --local
+
+cloud-init init
+
+cloud-init modules --mode config
+
+cloud-init modules --mode final
+"
+    if ! test -e $RC_LOCAL; then
+        echo "export PATH=$PATH:/usr/local/sbin:/usr/local/bin" >> $RC_LOCAL
+        echo "$RC_LOCAL_CONTENT" >> $RC_LOCAL
+    elif ! grep -Fq "cloud-init" $RC_LOCAL; then
+        echo "$RC_LOCAL_CONTENT" >> $RC_LOCAL
+    fi
+fi