Skip to content

Feature/eks#1225

Draft
s2oBCN wants to merge 45 commits intomasterfrom
feature/eks
Draft

Feature/eks#1225
s2oBCN wants to merge 45 commits intomasterfrom
feature/eks

Conversation

@s2oBCN
Copy link
Member

@s2oBCN s2oBCN commented Aug 4, 2025

EKS deployment
Created vars/odsComponentStageRolloutEKSDeployment.groovy
this calls the RolloutEKSDeploymentStage that calls the
src/org/ods/component/HelmDeploymentStrategy.groovy

The main difference with EKS, is that it doesn't have imageStreams.

So In order to change the code as less as possible now there's an interface: IImageRepository and 2 implementations: ImageRepositoryEKS and ImageRepositoryOpenshift, the rest of the changes are to make a call to this classe

@tbugfinder
Copy link
Contributor

I couldn't find a specific issue for EKS support.
Would it be possible to elaborate a bit why "this" approach is picked and wether additional tooling like argoc&fluxcd could overall improve compliance, speed and facilitate gitops mindset?

@metmajer
Copy link
Member

metmajer commented Feb 14, 2026

Hi @s2oBCN, it's important to keep this PR in sync with master to avoid deviations. Note that there were several PRs in the recent weeks that may conflict with the one or the other change you have made. Aligning your PR with these recent changes to master and with the EDP Platform team will be critical for getting it into mainline.

FYI @jafarre-bi @hrcornejo

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.

3 participants