fix: failing tests and change operator-container to only go if ci works

Author: ayuskauskas

.github/workflows/operator-container.yaml

@@ -28,13 +28,6 @@
 name: Build and push operator container image
 
 on:
-  pull_request:
-    branches:
-      - main
-    paths:
-      - operator/**/*.go
-      - containers/operator.Dockerfile
-      - .github/workflows/operator-container.yaml
   push:
     branches:
       - main
@@ -44,6 +37,10 @@ on:
       - operator/**/*.go
       - containers/operator.Dockerfile
       - .github/workflows/operator-container.yaml
+  workflow_run:
+    workflows: ["Operator unit and functional tests"]
+    types:
+      - completed
 
 # NOTE: we may want to switch to matrix build for multi-platform support if this is taking too long
 # https://docs.docker.com/build/ci/github-actions/multi-platform/#distribute-build-across-multiple-runners

agent/Makefile

@@ -39,7 +39,7 @@ test: venv ## Test using hatch, prints coverage and outputs a report to coverage
 
 .PHONY: license-fmt
 license-fmt: ## Run add license header to code.
-	python3 scripts/format_license.py --root-dir . --license-file ../LICENSE
+	python3 ../scripts/format_license.py --root-dir . --license-file ../LICENSE
 
 .PHONY: fmt
 fmt: license-fmt ## Run go fmt against code.

operator/Makefile

@@ -122,7 +122,7 @@ license-check: go-licenses ## Run go-licenses check against code.
 
 .PHONY: license-fmt
 license-fmt: ## Run add license header to code.
-	python3 scripts/format_license.py --root-dir . --license-file ../LICENSE
+	python3 ../scripts/format_license.py --root-dir . --license-file ../LICENSE
 
 .PHONY: fmt
 fmt: license-fmt ## Run go fmt against code.