Skip to content

fix: chart render error when enable initcontainer#1146

Merged
drivebyer merged 1 commit intoOT-CONTAINER-KIT:masterfrom
drivebyer:charts
Dec 2, 2024
Merged

fix: chart render error when enable initcontainer#1146
drivebyer merged 1 commit intoOT-CONTAINER-KIT:masterfrom
drivebyer:charts

Conversation

@drivebyer
Copy link
Copy Markdown
Collaborator

Description

Fixes #1145

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

  • Tests have been added/modified and all tests pass.
  • Functionality/bugs have been confirmed to be unchanged or fixed.
  • I have performed a self-review of my own code.
  • Documentation has been updated or added where necessary.

Additional Context

@drivebyer drivebyer added the bug Something isn't working label Dec 2, 2024
@drivebyer drivebyer force-pushed the charts branch 2 times, most recently from 9a4a92b to 9c9d4ca Compare December 2, 2024 09:27
- Updated version and appVersion for redis, redis-cluster, redis-replication, and redis-sentinel charts.
- Modified initContainer and sidecar property references in templates to include the current context.

This update ensures consistency across the Redis charts and improves template rendering.

Signed-off-by: drivebyer <wuyangmuc@gmail.com>
@drivebyer drivebyer merged commit b9034e8 into OT-CONTAINER-KIT:master Dec 2, 2024
@drivebyer drivebyer deleted the charts branch December 2, 2024 09:36
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 29.95%. Comparing base (d121d86) to head (9a9529c).
Report is 141 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1146      +/-   ##
==========================================
- Coverage   35.20%   29.95%   -5.25%     
==========================================
  Files          19       54      +35     
  Lines        3213     6243    +3030     
==========================================
+ Hits         1131     1870     +739     
- Misses       2015     4182    +2167     
- Partials       67      191     +124     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Error in redis-standalone.yaml for include function"

1 participant