Merge pull request #25 from small-hack/fix-scripts-path

clean up create_user script and location of script

Author: jessebot

charts/home-assistant/Chart.yaml

@@ -15,7 +15,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.7.2
+version: 0.7.3
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Home Assistant versions can be found here:

charts/home-assistant/README.md

@@ -1,6 +1,6 @@
 # home-assistant
 
-![Version: 0.7.2](https://img.shields.io/badge/Version-0.7.2-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 2024.4.1](https://img.shields.io/badge/AppVersion-2024.4.1-informational?style=flat-square)
+![Version: 0.7.3](https://img.shields.io/badge/Version-0.7.3-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 2024.4.1](https://img.shields.io/badge/AppVersion-2024.4.1-informational?style=flat-square)
 
 A Helm chart for home assistant on Kubernetes
 

…rts/home-assistant/config/create_user.py …ts/home-assistant/scripts/create_user.pycharts/home-assistant/config/create_user.py renamed to charts/home-assistant/scripts/create_user.py charts/home-assistant/config/create_user.py renamed to charts/home-assistant/scripts/create_user.py

@@ -1,3 +1,4 @@
+# Author: https://github.com/jessebot
 import json
 from os import environ as env
 import requests
@@ -6,6 +7,21 @@
 class RunHomeAssistantOnboarding():
     """
     Runs through home assistant onboarding to create a new user and disable registration
+    Required environment variables:
+
+    INTERNAL_URL   - if using k8s: name of k8s service, or if local you can use IP
+                     like this http://your-ip-here:8124
+    EXTERNAL_URL   - the url you use to access home assistant. if local, may be
+                     the same as INTERNAL_URL if using k8s with ingress, this is
+                     your ingress host, such as https://ha.yourhostname.tld/
+    ADMIN_PASSWORD - string of password for initial owner user, not required, recommended
+                     default: "b33pB00p.d4Doop"
+
+    # optional environment variables
+    DEBUG        - print all api responses, WARNING includes sensitive data. set to "true"
+    ADMIN_NAME     - name of the first owner user, default: 'admin'
+    ADMIN_USERNAME - login username of first owner user, default: 'admin'
+    ADMIN_LANGUAGE - 2 character string of initial owner user's language, default "en"
 
     home assistant user onboarding urls found here:
     https://github.com/home-assistant/core/tree/dev/homeassistant/components/onboarding/views.py
@@ -14,7 +30,7 @@ def __init__(self):
         self.headers = {
           'Content-Type': 'application/json',
         }
-        self.base_url = f"http://{env.get('SERVICE', 'home-assistant:8124')}"
+        self.base_url = f"http://{env.get('INTERNAL_URL', 'localhost:8124')}"
         self.external_url = env.get('EXTERNAL_URL', '')
         self.debug = env.get('DEBUG', False)
 
@@ -33,7 +49,7 @@ def run_integration_config(self) -> dict:
         """
         runs the integration config step of onboarding via the home assistant api
 
-        may not be working 🤷
+        may not be working 🤷, however this doesn't break the onboarding.
         """
         integration_url = f"{self.base_url}/api/onboarding/integration"
         print(f"We're going to post to {integration_url} for integration config")
@@ -68,7 +84,7 @@ def create_user(self) -> dict:
           "client_id": client_id,
           "name": env.get('ADMIN_NAME', 'admin'),
           "username": env.get('ADMIN_USERNAME', 'admin'),
-          "password": env.get('ADMIN_PASSWORD', 'test'),
+          "password": env.get('ADMIN_PASSWORD', "b33pB00p.d4Doop"),
           "language": env.get('ADMIN_LANGUAGE', 'en')
         })
 

charts/home-assistant/templates/create-user-configmap.yaml

@@ -9,5 +9,5 @@ metadata:
     "helm.sh/hook-weight": "-5"
 data:
   create_user.py: |-
-{{ .Files.Get "config/create_user.py" | indent 4 }}
+{{ .Files.Get "scripts/create_user.py" | indent 4 }}
 {{- end }}