From ea485a6f28b8d1d783f351f5cc885c5b654c7643 Mon Sep 17 00:00:00 2001
From: Amir Yogev <ayogev@redhat.com>
Date: Mon, 15 Jun 2026 10:15:39 +0300
Subject: [PATCH 1/2] NO-ISSUE: Add AGENTS.md, CLAUDE.md, and Copilot
 instructions

Create AGENTS.md as tool-agnostic source of truth (HostLease
controller, Ironic integration, build/test commands, architecture).
Create CLAUDE.md with @AGENTS.md. Add Copilot instructions.

Assisted-by: Claude Code <noreply@anthropic.com>
Signed-off-by: Amir Yogev <ayogev@redhat.com>
---
 .github/copilot-instructions.md |  1 +
 AGENTS.md                       | 97 +++++++++++++++++++++++++++++++++
 2 files changed, 98 insertions(+)
 create mode 100644 .github/copilot-instructions.md
 create mode 100644 AGENTS.md

diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md
new file mode 100644
index 0000000..d71e174
--- /dev/null
+++ b/.github/copilot-instructions.md
@@ -0,0 +1 @@
+Read and follow the project's AGENTS.md for build commands, architecture, conventions, and coding standards.
diff --git a/AGENTS.md b/AGENTS.md
new file mode 100644
index 0000000..6a3700c
--- /dev/null
+++ b/AGENTS.md
@@ -0,0 +1,97 @@
+# host-management-openstack
+
+Kubernetes controller that manages bare metal hosts via OpenStack Ironic. Watches HostLease CRs (defined in bare-metal-fulfillment-operator) and reconciles power state with Ironic nodes.
+
+## Critical Rules
+
+- **Always `make manifests generate`** after modifying CRD-consuming types
+- **Always `make lint`** before committing — fix all golangci-lint issues
+- **Never edit** `config/crd/` or `zz_generated.deepcopy.go` — these are generated
+- Run `make lint test` before committing
+
+## Dev Environment
+
+```bash
+make build                     # Build manager binary to bin/manager
+make test                      # Unit tests (excludes e2e)
+make test-e2e                  # E2E tests with Kind cluster
+make lint                      # golangci-lint
+make lint-fix                  # Auto-fix lint issues
+make fmt                       # go fmt
+make vet                       # go vet
+
+make manifests                 # Generate RBAC manifests
+make generate                  # Generate DeepCopy methods
+
+make run                       # Run controller locally
+make install                   # Install CRDs into cluster
+make deploy IMG=<registry>/host-management-openstack:tag
+make undeploy
+
+make image-build IMG=<registry>/host-management-openstack:tag
+make image-push IMG=<registry>/host-management-openstack:tag
+
+# Pre-commit hooks
+pre-commit run --all-files
+```
+
+## Repository Structure
+
+```text
+host-management-openstack/
+├── cmd/
+│   └── main.go                # Controller entry point
+├── internal/
+│   ├── controller/            # HostLease reconciliation logic
+│   └── management/            # OpenStack Ironic client
+├── config/
+│   ├── default/               # Kustomize default overlay
+│   ├── manager/               # Manager deployment manifest
+│   ├── rbac/                  # Generated RBAC rules
+│   └── samples/               # Example HostLease CRs
+├── test/
+│   ├── e2e/                   # End-to-end tests
+│   └── utils/                 # Test utilities
+├── hack/                      # Boilerplate and helper scripts
+├── Makefile                   # Build, test, lint, deploy targets
+├── go.mod                     # Go 1.26, controller-runtime, gophercloud
+└── .golangci.yml              # Linter configuration
+```
+
+## Architecture
+
+Single controller watching HostLease CRs:
+
+```text
+HostLease CR (spec.poweredOn: true/false)
+  ↓ (reconcile)
+HostLease Controller
+  ↓ (OpenStack API)
+Ironic Node (power state)
+```
+
+- **HostLease types** are defined in `bare-metal-fulfillment-operator/api/v1alpha1/` — this repo consumes them
+- **Management client** (`internal/management/`) abstracts OpenStack Ironic interactions via gophercloud
+- Controller syncs `spec.poweredOn` with Ironic node power state, requeues on interval for drift detection
+- Handles auth reconnection on token expiry
+
+### Key Files
+
+| File | Purpose |
+|------|---------|
+| `internal/controller/hostlease_controller.go` | Reconciliation logic |
+| `internal/controller/hostlease_names.go` | Naming helpers |
+| `internal/management/client.go` | Management client interface |
+| `internal/management/openstack.go` | OpenStack Ironic implementation |
+
+## CI
+
+GitHub Actions (`.github/workflows/`):
+- **build-image.yaml** — runs tests, builds + pushes container image and manifests
+- **pre-commit.yaml** — pre-commit hooks + golangci-lint on PRs
+
+## Code Quality
+
+- **golangci-lint** with dupl, errcheck, ginkgolinter, goconst, gocyclo, revive, staticcheck, and more (see `.golangci.yml`)
+- **Pre-commit hooks**: trailing-whitespace, yamllint (strict, excludes `config/`), golangci-lint
+- **Tests**: Ginkgo v2 + Gomega with envtest

From 9778370d27d09b8b9e2299c31d773a84e973a7f2 Mon Sep 17 00:00:00 2001
From: Amir Yogev <ayogev@redhat.com>
Date: Mon, 15 Jun 2026 13:55:12 +0300
Subject: [PATCH 2/2] NO-ISSUE: Remove copilot-instructions.md from this PR

Assisted-by: Claude Code <noreply@anthropic.com>
Signed-off-by: Amir Yogev <ayogev@redhat.com>
---
 .github/copilot-instructions.md | 1 -
 1 file changed, 1 deletion(-)
 delete mode 100644 .github/copilot-instructions.md

diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md
deleted file mode 100644
index d71e174..0000000
--- a/.github/copilot-instructions.md
+++ /dev/null
@@ -1 +0,0 @@
-Read and follow the project's AGENTS.md for build commands, architecture, conventions, and coding standards.