From 84e5b67d256af310fcb363ce5873d78d993c7138 Mon Sep 17 00:00:00 2001 From: Kiara Grouwstra Date: Sun, 11 May 2025 17:58:25 +0200 Subject: [PATCH] account for 285 --- panel/src/panel/views.py | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/panel/src/panel/views.py b/panel/src/panel/views.py index 341a9135..ec649576 100644 --- a/panel/src/panel/views.py +++ b/panel/src/panel/views.py @@ -23,6 +23,10 @@ class Index(TemplateView): template_name = 'index.html' +class Index(TemplateView): + template_name = 'index.html' + + class AccountDetail(LoginRequiredMixin, DetailView): model = User template_name = 'account_detail.html' @@ -109,6 +113,12 @@ class DeploymentStatus(ConfigurationForm): # pass in form info to our deployment "DEPLOYMENT": config.json() } + # XXX should we not log this if it may show proxmox credentials from `.envrc`s? + # those could instead be passed as sensitive TF vars, but that would not address this. + logger.debug("env: %s", env) + cwd = f"{settings.repo_dir}/launch" + # direnv wants this run upfront, and chaining in subprocess feels awkward + subprocess.check_call(["direnv", "allow"], cwd=cwd) cmd = [ "nix", "develop", @@ -119,9 +129,6 @@ class DeploymentStatus(ConfigurationForm): "apply", "test", ] - deployment_result = subprocess.run( - cmd, - cwd=settings.repo_dir, - env=env, - ) + deployment_result = subprocess.run(cmd, cwd=cwd, env=env) + logger.debug("deployment_result: %s", deployment_result) return deployment_result, config