Skip to content

feat: add checksum for config in dataservice and gateway deployments#159

Open
avaya09 wants to merge 1 commit intomainfrom
fix/checksum-support
Open

feat: add checksum for config in dataservice and gateway deployments#159
avaya09 wants to merge 1 commit intomainfrom
fix/checksum-support

Conversation

@avaya09
Copy link
Contributor

@avaya09 avaya09 commented Feb 24, 2026

  • Introduced checksum/config annotation in both dataservice and gateway deployment templates to ensure proper configuration updates and cache invalidation.

- Introduced checksum/config annotation in both dataservice and gateway deployment templates to ensure proper configuration updates and cache invalidation.
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds checksum/config annotations to both the gateway and dataservice deployment pod templates to ensure pods are automatically restarted when the shared configuration changes. The checksum is calculated based on the gateway/config.yaml file, which contains the shared environment configuration (Secret or ConfigMap) consumed by both services.

Changes:

  • Added checksum/config annotation to gateway deployment pod template
  • Added checksum/config annotation to dataservice deployment pod template

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
charts/portkey-gateway/templates/gateway/deployment.yaml Adds checksum annotation to trigger pod restarts when shared config changes
charts/portkey-gateway/templates/dataservice/deployment.yaml Adds checksum annotation to trigger pod restarts when shared config changes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@narengogi narengogi requested a review from sk-portkey February 27, 2026 09:29
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