fix/existing-bugs #1

Merged

litelotus merged 7 commits from fix/existing-bugs into main

2026-05-28 10:07:18 +00:00

litelotus commented

2026-05-28 10:01:04 +00:00

Owner

No description provided.

litelotus added 6 commits

2026-05-28 10:01:04 +00:00

fix: nil map panics, missing interface method, unused error, and nil pointer dereferences 820d30c09c

- ingress.go: initialize annotation maps with make() instead of var (nil map write panic)
- ingress.go: attach converted paths to HTTP rule spec (paths were computed but discarded)
- ingress_resource_path.go: rename GetPrefix() to GetPathType() to satisfy ToolkitIngressPath interface
- service.go: handle error from GetPulumiServicePortArray() before using result
- longhorn.go: initialize annotation map with make(), add nil check for opt.Annotations
- deployment.go: remove unused imports from empty stub

refactor: replace Longhorn annotation-based volumes with StorageClass abstraction d810dedadb

Longhorn no longer supports PVC annotations for volume configuration.
Replace with a proper StorageClass-based approach:

- Remove longhorn.go (annotation-based volume config)
- Add longhorn_storage_class.go: ToolkitLonghornStorageClass with parameters
  for replicas, data locality, encryption, fsType, migratable, stale timeout,
  and recurring job selectors (backup/snapshot schedules)
- Add pvc.go: generic ToolkitPersistentVolumeClaim that references a StorageClass
- Add storage_size.go: shared StorageSize type

StorageClass defaults to reclaimPolicy=Retain and allowVolumeExpansion=true.

chore: add placeholder for Longhorn RecurringJob CRD abstraction 24f2b0042d

feat: add deployment, configmap, secret, and mount abstractions 8282e78161

- pods/deployment.go: ToolkitDeploymentOptions with auto-generated selectors,
  multi-container support, init containers, env (pulumi.StringInput values),
  envFrom (inject all keys from secret/configmap), volume mounts, resource
  limits, command/args overrides. Volumes are auto-deduced from mount sources.
- pods/mount.go: MountSource interface + Mount struct. Any toolkit resource
  (PVC, Secret, ConfigMap) that implements MountableVolumeName/Type can be
  mounted with just a path.
- core/configmap.go: ToolkitConfigMapOptions with pulumi.StringMapInput data.
- core/secret.go: ToolkitSecretOptions with pulumi.StringMapInput data,
  configurable type, satisfies MountSource.
- volumes/pvc.go: CreatedToolkitPersistentVolumeClaim now satisfies MountSource.

All inputs that could come from other Pulumi resources accept StringInput/
StringMapInput rather than plain strings.

feat: add OpenBao secret injection via pod annotations 82a6576737

- core/bao_secret.go: ToolkitBaoSecretOptions with path, role, template,
  file permission, and post-render command. Satisfies MountSource with
  type 'bao'. No Pulumi resource created — just a reference handle.
- pods/deployment.go: detects bao-type mounts, generates vault.hashicorp.com
  agent-inject annotations on the pod template instead of volumes/mounts.
  Supports multiple bao secrets per pod, custom templates, file paths,
  permissions, and post-render commands.

Usage:
  baoSecret := core.NewBaoSecret(core.ToolkitBaoSecretOptions{
      Name: "env",
      Path: "hermes-agent/data/env",
      Role: "hermes-agent",
  })
  // Then in deployment containers:
  Mounts: []pods.Mount{{Source: baoSecret, Path: "/vault/secrets/env"}}

docs: add Go doc comments to all exported types, methods, and constants 6c8fad65f5

litelotus added 1 commit

2026-05-28 10:03:49 +00:00

chore: fixing longhorn volume conflict 74d80418ff

litelotus merged commit b340e96dcb into main

2026-05-28 10:07:18 +00:00

litelotus referenced this pull request from a commit

2026-05-28 10:07:19 +00:00

Merge pull request 'fix/existing-bugs' (#1) from fix/existing-bugs into main

litelotus deleted branch fix/existing-bugs

2026-05-28 10:08:55 +00:00