Skip to content

Conversation

a1994sc
Copy link
Contributor

@a1994sc a1994sc commented May 26, 2025

Description

Updates gitea chart to 12.0.0:

  • The upstream chart replaces redis with valkey, so I disabled the valkey cluster.
  • Changed url to use OCI chart
  • Changed gitea_image to use ghcr.io endpoint to avoid rate-limiting
    • Might want to consider changing registry_image_* to use ghcr.io endpoints

Related Issue

Fixes #N/A

Checklist before merging

@a1994sc a1994sc requested review from a team as code owners May 26, 2025 19:30
Copy link

netlify bot commented May 26, 2025

Deploy Preview for zarf-docs ready!

Name Link
🔨 Latest commit 6e3362d
🔍 Latest deploy log https://app.netlify.com/projects/zarf-docs/deploys/688d60d0eab752000814c796
😎 Deploy Preview https://deploy-preview-3859--zarf-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

- the upstream chart replaces `redis` with `valkey`

Signed-off-by: Allen Conlon <[email protected]>
@a1994sc a1994sc force-pushed the feat/update-cots-gitea branch from ef8094c to 06dc9c6 Compare May 26, 2025 19:31
Copy link

codecov bot commented May 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@brandtkeller
Copy link
Member

Hey @a1994sc , would you mind rebasing this when you have a moment? I'd like to see where this lands for review currently.

@a1994sc
Copy link
Contributor Author

a1994sc commented Jul 25, 2025

Oh rebase, not merge, yea I can update that later today

Copy link
Member

@brandtkeller brandtkeller left a comment

Choose a reason for hiding this comment

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

E2E without cluster workflows are failing with a hardcoded gitea image

I'd be fine with updating this line again unless you see a more dynamic option available.

@a1994sc
Copy link
Contributor Author

a1994sc commented Jul 25, 2025

ok, I think I saw that recent too, so I can update that.

@a1994sc a1994sc changed the title chore: update gitea chart to 12.0.0 chore: update gitea chart to 12.1.2 Aug 2, 2025
@a1994sc a1994sc requested a review from brandtkeller August 4, 2025 11:54
@brandtkeller
Copy link
Member

reviewing the delta of updates between the indicated application version as well as the helm chart versions. No action required by the PR.

Appreciative of catching the update on this.

@a1994sc
Copy link
Contributor Author

a1994sc commented Aug 8, 2025

Not that is a very exhaustive change, but I have been rolling the latest version of the helm chart/docker image internally without too much issue. However I understand my needs aren't going to fit for the whole zarf user base

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