Skip to content
Snippets Groups Projects

build: cern-magnum is packaged as two seperate helm charts

Merged Jack Charlie Munday requested to merge package-crds into master
All threads resolved!

From discussion on !231 (closed), we have decided to package cern-magnum as two separate charts (one with only and another excluding the crds).

This is required as for when multiple dependencies with very large crds are enabled in cern-magnum installation breaks owning to us exceeding the 2MB secret limit for our helm release.

Hiding this change purley in the CI from a package / installation perspective provides the least intrusive approach (as it does not require restructuring or maintaining two `values.yaml` [one for crds and another for the umbrella chart], nor does it require complicated if gating around templates or crds). This is intended as a temporary solution until plans to migrate the management of cern-magnum as an applicationset in argo are planned for and completed later on in the year.

Edited by Jack Charlie Munday

Merge request reports

Pipeline #7968882 passed

Pipeline passed for 58be5d88 on package-crds

Approval is optional

Merged by Jack Charlie MundayJack Charlie Munday 9 months ago (Aug 20, 2024 1:45pm UTC)

Merge details

  • Changes merged into master with 116f22af.
  • Deleted the source branch.

Pipeline #7968982 passed

Pipeline passed for 116f22af on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Jack Charlie Munday marked this merge request as ready

    marked this merge request as ready

  • Jack Charlie Munday changed title from Draft: cern-magnum is packaged as two seperate helm charts to build: cern-magnum is packaged as two seperate helm charts

    changed title from Draft: cern-magnum is packaged as two seperate helm charts to build: cern-magnum is packaged as two seperate helm charts

  • mentioned in issue #93 (closed)

  • added 1 commit

    • e5493f52 - build: package cern-magnum as two seperate charts

    Compare with previous version

  • added 1 commit

    • 4aa6b4ee - build: package cern-magnum as two seperate charts

    Compare with previous version

  • Ricardo Rocha approved this merge request

    approved this merge request

  • Jack Charlie Munday resolved all threads

    resolved all threads

  • added 1 commit

    • 30c327b3 - update build_chart stage to run package.sh

    Compare with previous version

  • Jack Charlie Munday reset approvals from @rbritoda by pushing to the branch

    reset approvals from @rbritoda by pushing to the branch

  • added 1 commit

    • 58be5d88 - update build_chart stage to run package.sh

    Compare with previous version

  • Ricardo Rocha approved this merge request

    approved this merge request

  • mentioned in commit 116f22af

  • Please register or sign in to reply
    Loading