Skip to content

feat(k8s): bundle support #4097

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

feat(k8s): bundle support #4097

wants to merge 5 commits into from

Conversation

DanielHougaard
Copy link
Member

@DanielHougaard DanielHougaard commented Jul 14, 2025

Description 📣

This PR adds bundle support to the Kubernetes Operator. This is a prerequisite for releasing to OperatorHub.

We had to rename from k8-operator -> infisical-operator for the generated output to match our Operator Hub release name. This is a hard requirement by their test suite.

I've created an initial release for operator hub, but after our initial release is out, we'll need to make a CI step to automate the release to operator hub.

Type ✨

  • Bug fix
  • New feature
  • Improvement
  • Breaking change
  • Documentation

@DanielHougaard DanielHougaard self-assigned this Jul 14, 2025
@maidul98
Copy link
Collaborator

maidul98 commented Jul 14, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

@DanielHougaard DanielHougaard marked this pull request as ready for review July 14, 2025 16:59
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Greptile Summary

This PR prepares the Kubernetes Operator for distribution through OperatorHub by implementing bundle support and standardizing naming conventions. The primary change is renaming all references from k8-operator to infisical-operator across the codebase to comply with OperatorHub's strict naming requirements. The changes include:

  1. Updating import paths in all Go files to use the new package name
  2. Modifying Kubernetes resource labels and metadata
  3. Adding bundle generation and validation support in the Makefile
  4. Creating a new script (fix-csv.sh) to customize the ClusterServiceVersion manifest

This change is infrastructural in nature and doesn't modify the core functionality of the operator. It's a necessary step to enable automated releases to OperatorHub and improve the operator's accessibility to Kubernetes users.

Confidence score: 4.5/5

  1. This PR is safe to merge as it primarily involves package renaming and build infrastructure changes
  2. High confidence score given because the changes are well-structured, consistent across files, and don't affect runtime behavior
  3. Files needing attention:
    • k8-operator/scripts/fix-csv.sh: New script that needs validation for correct CSV modification
    • k8-operator/Makefile: Verify bundle generation targets work as expected
    • k8-operator/controllers/infisicaldynamicsecret/infisicaldynamicsecret_controller.go: Contains TODO comments that should be addressed

48 files reviewed, 2 comments
Edit PR Review Bot Settings | Greptile

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.

2 participants