Skip to content

MK8S-71 - platform-disk-management-agent: Initialize repository#1

Merged
vdaviot merged 5 commits intomainfrom
improvement/MK8S-71-initialize-repository
Mar 9, 2026
Merged

MK8S-71 - platform-disk-management-agent: Initialize repository#1
vdaviot merged 5 commits intomainfrom
improvement/MK8S-71-initialize-repository

Conversation

@vdaviot
Copy link
Copy Markdown
Collaborator

@vdaviot vdaviot commented Feb 9, 2026

Summary

Initialize the platform-disk-management-agent operator repository using Operator SDK with kubebuilder v4 scaffolding.

  • Repository bootstrap: Scaffold the project with operator-sdk init and operator-sdk create api, generating the base boilerplate (Makefile, Dockerfile, go.mod, PROJECT, cmd/main.go, etc.)
  • DiscoveredPhysicalDisk CRD: Define a new v1alpha1 custom resource under the metalk8s.scality.com API group with Spec and Status subresource, along with admin/editor/viewer RBAC roles and a stub reconciler
  • DaemonSet deployment model: Convert the default Deployment-based manager into a DaemonSet so the agent runs on every node; remove leader election resources (role + role binding) that are unnecessary in this mode
  • Kustomize configuration: Set up full kustomize overlays for CRDs, RBAC, manager, metrics, Prometheus monitoring, network policies, and scorecard
  • CI pipelines: Add GitHub Actions workflows for linting (golangci-lint), unit tests, and e2e tests (using kind)
  • Dev environment: Include a devcontainer configuration with post-install script for a ready-to-use development setup

@vdaviot vdaviot marked this pull request as draft February 9, 2026 14:04
@vdaviot vdaviot marked this pull request as ready for review February 11, 2026 11:02
@TeddyAndrieux
Copy link
Copy Markdown

TeddyAndrieux commented Feb 24, 2026

LGTM but to me scaffolding has to be re-done to change the name from platform-disk-management-agent to disk-management-agent

nit: Also if you may mention the operator-sdk command that has been ran in the commit body could be cool :)

Comment thread config/manager/kustomization.yaml Outdated
Comment thread config/default/metrics_service.yaml
Comment thread config/default/kustomization.yaml
Comment thread config/default/kustomization.yaml
@vdaviot
Copy link
Copy Markdown
Collaborator Author

vdaviot commented Feb 25, 2026

scaffolding has to be re-done to change the name from platform-disk-management-agent to disk-management-agent

I might change that at the end if that's ok for you @TeddyAndrieux, this way I don't have to chain merge/update in each PRs

@vdaviot vdaviot force-pushed the improvement/MK8S-71-initialize-repository branch from e956213 to 825e8f3 Compare March 9, 2026 13:47
Signed-off-by: Valentin Daviot <[email protected]>
@vdaviot vdaviot force-pushed the improvement/MK8S-71-initialize-repository branch from 825e8f3 to 4c1d35f Compare March 9, 2026 13:55
@vdaviot vdaviot merged commit e939723 into main Mar 9, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants