feat: Add Knative Service support to Reloader #969
Conversation
- Add Knative serving client to kube clients - Implement CRUD operations for Knative services - Add rolling upgrade support for Knative services - Update RBAC permissions for serving.knative.dev/services - Add comprehensive test coverage for Knative functionality - Update documentation with Knative service support - Support both env-var and annotation reload strategies - Add patch template support for Knative services Resolves the need for automatic reload of Knative services when ConfigMaps or Secrets change, extending Reloader's capabilities to serverless workloads.
- Restored all original documentation sections that were accidentally removed - Added KnativeService to the Mermaid workflow diagram - Maintained all contributing guidelines, license info, and help sections - Safe for open source repository
| - apiGroups: | ||
| - "" | ||
| resources: | ||
| - events | ||
| verbs: | ||
| - create | ||
| - patch | ||
| {{- if .Values.reloader.enableHA }} |
There was a problem hiding this comment.
@yg1996 what is this change about?
There was a problem hiding this comment.
@rasheedamir This change isn't needed for Knative support specifically - it's just for better service reliability.
The {{ if .Values.reloader.enableHA }} condition adds RBAC permissions for leader election when running multiple Reloader replicas in High Availability mode. This is completely separate from the Knative functionality.
What it does:
Grants permissions to manage Kubernetes leases for leader election
Only when enableHA: true is set
Allows running multiple Reloader instances without conflicts
Why it's separate from Knative:
Knative support works fine with single replica (default setup)
HA is purely for operational reliability/redundancy
The actual Knative integration uses different RBAC permissions (serving.knative.dev/services)
So you can use Knative support without enabling HA - this is just an optional enhancement for production environments that want multiple Reloader instances for better availability.
There was a problem hiding this comment.
@rasheedamir Still waiting for approval
There was a problem hiding this comment.
@yg1996 thanks for the patience; team is pretty occupied and we will look into it asap
Resolves the need for automatic reload of Knative services when ConfigMaps or Secrets change, extending Reloader's capabilities to serverless workloads.
Closes #493