Add SKIP_WIDGET_RENDERER to manifests#1935
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds the SKIP_WIDGET_RENDERER: "true" environment variable to all three Cloud Foundry manifest files to prevent instance crashes during deployments. The environment variable prevents the application from attempting to load the Rust widget renderer library, which was being removed by CF during deployments and causing failures.
Key Changes
- Added
SKIP_WIDGET_RENDERER: "true"to production, staging, and demo manifests - Ensures consistent configuration across all deployment environments
- Prevents the widget renderer initialization code from attempting to load the Rust library
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| touchpoints.yml | Added SKIP_WIDGET_RENDERER environment variable to production manifest |
| touchpoints-staging.yml | Added SKIP_WIDGET_RENDERER environment variable to staging manifest |
| touchpoints-demo.yml | Added SKIP_WIDGET_RENDERER environment variable to demo manifest |
After thoroughly reviewing the changes, I found no issues with the implementation. The changes are:
- Consistent across all three manifest files
- Properly formatted with correct indentation matching each file's style
- Using the appropriate string value format (
"true") - Logically placed within the
envsections - Aligned with the existing code in
config/initializers/widget_renderer.rbwhich checks for this environment variable
The PR successfully addresses the stated problem without introducing any bugs, naming issues, or other concerns.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
SKIP_WIDGET_RENDERER: "true"to all manifest files (production, staging, demo)