Skip to content

Pull Redis image from Bitnami legacy repo instead#146

Closed
norrisng-bc wants to merge 2 commits into
masterfrom
chore/redis-bitnami-legacy
Closed

Pull Redis image from Bitnami legacy repo instead#146
norrisng-bc wants to merge 2 commits into
masterfrom
chore/redis-bitnami-legacy

Conversation

@norrisng-bc

@norrisng-bc norrisng-bc commented Dec 11, 2025

Copy link
Copy Markdown
Member

Description

Pull the Redis image from the Bitnami legacy repo instead. This is because Bitnami has since yanked their free charts.

Separately, the Codeclimate GitHub Action has also been bumped up to v9. This was already updated in our previous repos (e.g. https://github.com/bcgov/document-generation-showcase/pull/95/files#diff-6e608e02c595d53ab6b70822a2bf19abcfc6ddcc976c2f536ad5bfca20f0443f); without this update, it will fail to run and break the pipeline.

https://apps.nrs.gov.bc.ca/int/jira/browse/SHOWCASE-4078

Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the OpenAPI 3.0 v*.api-spec.yaml documentation (if appropriate)
  • I have added necessary documentation (if appropriate)

Further comments

Bitnami yanked their public Helm chart catalog >:(
@github-actions

Copy link
Copy Markdown

Coverage Report

Totals Coverage
Statements: 96.61% ( 769 / 796 )
Methods: 98.08% ( 102 / 104 )
Lines: 97.95% ( 429 / 438 )
Branches: 93.7% ( 238 / 254 )

@norrisng-bc norrisng-bc force-pushed the chore/redis-bitnami-legacy branch from 3de8187 to 30bfff3 Compare December 11, 2025 19:20
Side note: codeclimate-action was bumped to v9 in our other repos before Code Climate shut down their API
@norrisng-bc norrisng-bc force-pushed the chore/redis-bitnami-legacy branch from 30bfff3 to d2c1463 Compare December 11, 2025 21:59
@TimCsaky TimCsaky force-pushed the master branch 13 times, most recently from 08e7aa2 to 68b57eb Compare December 13, 2025 00:02
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