Skip to content

feat: remote deployment (full)#343

Open
akafazov wants to merge 92 commits intomainfrom
feat/remote-deployment-#2-argocd
Open

feat: remote deployment (full)#343
akafazov wants to merge 92 commits intomainfrom
feat/remote-deployment-#2-argocd

Conversation

@akafazov
Copy link
Copy Markdown
Contributor

@akafazov akafazov commented Feb 1, 2026 •
edited
Loading

Summary

This PR adds remote deployment capabilities and ArgoCD support to the platform-mesh-operator.

Major Changes

  • Remote Deployment Architecture - Supports deploying to separate infra/runtime clusters via kubeconfig
  • ArgoCD Integration - Alternative to FluxCD using ArgoCD Applications and AppProjects
  • Profile-based Configuration - Unified ConfigMap-based profiles replacing hardcoded chart references
  • Server-Side Apply (SSA) - Migration from Create/Update to Patch with SSA for better field ownership
  • Go Templates - New gotemplates/ directory for rendering infrastructure and component manifests
  • Enhanced Wait Logic - StatusFieldPath support for checking ArgoCD-style nested status fields

refers to platform-mesh/helm-charts#594

@akafazov
WIP
06b80db 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
feat: enhance configuration documentation with detailed environment v…
cf29510 
…ariables and subroutines

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
sub(deployment): implement remote deployment
318ee8a 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(deployment): replace clientDeploy with clientPlatformMesh for KCP…
288c33b 
… operations

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
feat(kcp): always connect to kcp on external endpoint
03ba251 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
Update .gitignore
34e2f30 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(e2e): improve kubeconfig handling in Kind cluster setup
4bd15af 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(test): remove unused variable in createKindCluster function
a6c4047 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(config): add exposure settings for baseDomain and protocol in pla…
d997d9f 
…tform-mesh.yaml

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(task): standardize test timeout using TEST_TIMEOUT variable in Ta…
bd0dce3 
…skfile.yml

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(providersecret): enhance KCP host handling for external and inter…
780a1e5 
…nal connections

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(lint): subroutine mock
70c326e 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(tests): refactor Test_updateOciRepo_CreateOrUpdateError to use cl…
78b4d87 
…ientMock directly

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(tests): refactor tests to use clientMock directly in subroutine t…
342b9ff 
…ests

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
feat: refactor and improve configuration
fea7319 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(tests): subroutines
f759850 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix: pass Kubernetes client to PlatformMesh reconciler
6f80a32 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
Refactor operator to replace pm-components and pm-infra-components ch…
05dad44 
…arts with gotemplates and infra and components profiles.

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
refactor(config): update struct definitions for subroutines and impro…
b6c0d4e 
…ve test cases for provider secret handling

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
refactor(subroutines): streamline Helm release update logic and enhan…
c2da616 
…ce test structure

- Removed unnecessary client Get call in updateHelmReleaseWithImageTag function.
- Improved test case structure by directly using clientMock in subroutine tests.
- Updated config struct definition for clarity.

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
feat(docker): add gotemplates to Dockerfile and update client handlin…
64206e0 
…g in operator

- Added gotemplates directory to the Dockerfile for inclusion in the build.
- Refactored operator to replace FluxCD client with a more generic Kubernetes client for improved flexibility.
- Updated merge logic in merge.go to enhance object merging behavior and logging.

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(ocm): change in upstream repo
e5dfb1d 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(operator): handle Infra client creation error
dc88d8b 
- Added error handling for Infra client creation in RunController function to log the error and exit if the client cannot be created.
- Updated merge logic comments in merge.go for clarity.
- Adjusted test case formatting in subroutine_test.go for consistency.

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
feat(profile): introduce unified profile configuration for PlatformMesh
52e0c1d 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
update CRD
9cf2cee 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
code refactoring
64145ac 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(e2e-test): add profile configuration to test
d6038e1 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
refactor(defaults): update HelmRelease configurations and improve tem…
7d48eef 
…plate rendering

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(wait): configure to wait for helmreleases using label
05babb9 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix profile and value precendence when applying manifests
f692a91 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
akafazov added 11 commits March 15, 2026 19:37
@akafazov
fix: unify template walk loops, fix SSA patches, add test coverage, a…
889194b 
…nd clean up lint issues

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
Merge remote-tracking branch 'origin/main' into feat/remote-deploymen…
791a448 
…t-#2-argocd
@akafazov
chore: remove redundant logs
fdabfda 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix typo
695325a 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix checkClusterExists
2c70c26 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
use OCM chart for deployment of the toolkit
c933d8f 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
add controller tests
6c309d3 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix(ci): split test and coverage steps so threshold failures show rep…
c7f4aa1 
…ort output

On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
pin go-test-coverage version
d037bf3 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
ping v2.17.1
8af4635 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
pin version v2.18.3 in workflow too
8e67c81 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
OlegErshov
OlegErshov reviewed Mar 23, 2026
View reviewed changes
Comment thread internal/controller/platformmesh_controller.go
Comment thread pkg/subroutines/deployment.go Outdated
Comment thread pkg/subroutines/deployment.go
Comment thread pkg/subroutines/deployment.go Outdated
Comment thread pkg/subroutines/deployment.go
Comment thread pkg/subroutines/subroutine_helpers.go
Comment thread pkg/subroutines/deployment.go
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 24, 2026 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

akafazov added 7 commits March 25, 2026 13:40
@akafazov
fix admin cert-based provider secrets
8718b26 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
Merge remote-tracking branch 'origin/main' into feat/remote-deploymen…
f12077f 
…t-#2-argocd
@akafazov
address pr comments
97d2d36 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
test gha fix
d11dca5 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix gha issue
baa2142 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix: tests and update OCM version
37c8d59 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov
fix coverage
27c2a9c 
On-behalf-of: @SAP angel.kafazov@sap.com
Signed-off-by: Angel Kafazov <akafazov@cst-bg.net>
@akafazov akafazov marked this pull request as ready for review March 26, 2026 09:07
@akafazov akafazov requested review from a team as code owners March 26, 2026 09:07
OlegErshov
OlegErshov approved these changes Mar 26, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@OlegErshov OlegErshov left a comment

Choose a reason for hiding this comment

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

Looks nice!

@mirzakopic mirzakopic mentioned this pull request Apr 23, 2026
Open
@akafazov akafazov self-assigned this May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] requested changes

@simontesar simontesar simontesar approved these changes

@OlegErshov OlegErshov OlegErshov approved these changes

Requested changes must be addressed to merge this pull request.

Assignees

@akafazov akafazov

Labels

feature fix

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@akafazov @CLAassistant @simontesar @OlegErshov @hyperlingg