UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Resolve "Fix Kustomize for the Package"

Merged Jeffrey Victor requested to merge 2245-fix-kustomize-for-the-package into master
All threads resolved!

Package Merge Request

Package Changes

Remove the logic that doesn't invoke when kustomize is being used as its required for the kustomize.

Package MR

Issue was pointed out by a slack community member https://bigbanguniver-ft39451.slack.com/archives/C051A2BPS0K/p1721678936848139 The following values were used

packages:  
  crossplane:
    enabled: true
    kustomize: true

with no changes you receive the following error Error: 1 error occurred: * Secret in version "v1" cannot be handled as a Secret: json: cannot unmarshal object into Go struct field Secret.stringData of type string but with the changes it actually goes through, as per the fluxcd documentation you should be passing in the secret as a stringData. Not sure if previous fluxcd versions did not support it that way, but now it works as intended.

image

image

For Issue

Closes (link to issue here)

Upgrade Notices

N/A

Closes #2245 (closed)

Edited by Jeffrey Victor

Merge request reports

Merge request pipeline #3493654 passed

Merge request pipeline passed for 4dec74fc

Approval is optional

Merged by Michael MartinMichael Martin 8 months ago (Jul 23, 2024 8:09pm UTC)

Merge details

Pipeline #3494088 passed

Pipeline passed for 3956f2db 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
  • Jeffrey Victor resolved all threads

    resolved all threads

  • added 1 commit

    Compare with previous version

  • Jeffrey Victor changed the description

    changed the description

  • Jeffrey Victor changed the description

    changed the description

  • Jeffrey Victor changed milestone to %2.33.0

    changed milestone to %2.33.0

  • Michael Martin approved this merge request

    approved this merge request

  • Michael Martin mentioned in commit 3956f2db

    mentioned in commit 3956f2db

  • Please register or sign in to reply
    Loading