Skip to content

feat(ci): e2e flux#1219

Merged
IvoGoman merged 31 commits intomainfrom
e2e-flux
Oct 16, 2025
Merged

feat(ci): e2e flux#1219
IvoGoman merged 31 commits intomainfrom
e2e-flux

Conversation

@trouaux
Copy link
Contributor

@trouaux trouaux commented Oct 15, 2025

enhance helm-lint-test in favor of e2e flux helm git action to detect templating issues specifc to flux and helm

nit renovate git action

@trouaux trouaux marked this pull request as ready for review October 15, 2025 10:30
@trouaux trouaux requested review from a team as code owners October 15, 2025 10:30
@abhijith-darshan
Copy link
Contributor

Just to give more context to the suggestion behind an e2e folder to maintain test resources declaratively.

In future you can have more test cases and you can use assertion tools like kuttl that come with test harness.

It will ease up e2e testing and you don't have to do wait for condition and print debugging logs.
It is also easily reproducible locally and won't be action specific.

@trouaux trouaux requested a review from a team as a code owner October 16, 2025 07:05
Copy link
Contributor

@ibakshay ibakshay left a comment

Choose a reason for hiding this comment

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

🚀

@IvoGoman IvoGoman merged commit 90c3701 into main Oct 16, 2025
7 checks passed
@IvoGoman IvoGoman deleted the e2e-flux branch October 16, 2025 10:45
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.

5 participants