Skip to content

[DRAFT] Replace runtime.RawUnstructured with internal ArgoCD type to avoid #15126#17725

Closed
PaulSonOfLars wants to merge 27 commits intoargoproj:masterfrom
PaulSonOfLars:paul/remove-values-rawextension
Closed

[DRAFT] Replace runtime.RawUnstructured with internal ArgoCD type to avoid #15126#17725
PaulSonOfLars wants to merge 27 commits intoargoproj:masterfrom
PaulSonOfLars:paul/remove-values-rawextension

Conversation

@PaulSonOfLars
Copy link
Contributor

NOTE: This is a draft PR to see if CI has better luck with building; having issues on my local env.

Checklist:

  • Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this does not need to be in the release notes.
  • The title of the PR states what changed and the related issues number (used for the release note).
  • The title of the PR conforms to the Toolchain Guide
  • I've included "Closes [ISSUE #]" or "Fixes [ISSUE #]" in the description to automatically close the associated issue.
  • I've updated both the CLI and UI to expose my feature, or I plan to submit a second PR with them.
  • Does this PR require documentation updates?
  • I've updated documentation as required by this PR.
  • I have signed off all my commits as required by DCO
  • I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
  • My build is green (troubleshooting builds).
  • My new feature complies with the feature status guidelines.
  • I have added a brief description of why this PR is necessary and/or what this PR solves.
  • Optional. My organization is added to USERS.md.
  • Optional. For bug fixes, I've indicated what older releases this fix should be cherry-picked into (this may or may not happen depending on risk/complexity).

vladfr and others added 25 commits December 13, 2023 22:37
…j#15126)

Signed-off-by: Vlad Fratila <vlad.fratila@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
when applying a patch, the rawextenstion object used for valuesObject
needs to be diffed with jsonpatch instead of the default strategic merge

Signed-off-by: Vlad Fratila <vlad.fratila@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Vlad Fratila <vlad.fratila@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Vlad Fratila <vlad.fratila@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
…ts for both app.Source and app.Sources fields

Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
…-jsonpatching

Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
…-jsonpatching

Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
…-jsonpatching

Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
…internal type

Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
@PaulSonOfLars PaulSonOfLars force-pushed the paul/remove-values-rawextension branch from f509f1b to b98f51f Compare April 4, 2024 20:55
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
Signed-off-by: Paul Larsen <pnvlarsen@gmail.com>
@PaulSonOfLars PaulSonOfLars force-pushed the paul/remove-values-rawextension branch from a942d21 to 0d67d50 Compare April 5, 2024 14:08
@PaulSonOfLars
Copy link
Contributor Author

Closing at this test didn't work out as planned

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