Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KEP 98: Promotion Phase for GitOps use cases #107

Merged
merged 4 commits into from
Feb 12, 2024
Merged

KEP 98: Promotion Phase for GitOps use cases #107

merged 4 commits into from
Feb 12, 2024

Conversation

mowies
Copy link
Member

@mowies mowies commented Feb 1, 2024

No description provided.

Signed-off-by: Moritz Wiesinger <[email protected]>
@mowies mowies requested review from a team as code owners February 1, 2024 09:35
text/0098-promotion-phase.md Outdated Show resolved Hide resolved
text/0098-promotion-phase.md Outdated Show resolved Hide resolved
mowies and others added 2 commits February 1, 2024 12:34
Co-authored-by: Giovanni Liva <[email protected]>
Signed-off-by: Moritz Wiesinger <[email protected]>
@mowies
Copy link
Member Author

mowies commented Feb 5, 2024

@thisthat pls re-review :)

Signed-off-by: Moritz Wiesinger <[email protected]>
@thschue
Copy link
Contributor

thschue commented Feb 6, 2024

This EP makes much sense and enables many workflows (especially regarding promotion tasks). Could it be that this partially deprecates this one: #98 (on-success tasks but implemented in a new phase)?

@mowies
Copy link
Member Author

mowies commented Feb 6, 2024

@thschue it's connected to the other KEP a bit, yes. But only in part of course. The thing is that you can't reference tasks here. So you basically have an unordered list of tasks that are run as a result of the success of the post-deployment phase.
But there's no way of referencing specific tasks when a specific post-deployment task succeeded for example.

Copy link
Contributor

@thschue thschue left a comment

Choose a reason for hiding this comment

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

lgtm

@thisthat
Copy link
Member

thisthat commented Feb 6, 2024

I will merge this next Monday (2024-02-12) if there will be no more additional feedback :)

@heckelmann
Copy link

Two questions: Will this phase being executed as well when the status of a post deployment task or evaluation is failing? If so, do I get the results in my promotion task as well?

@mowies
Copy link
Member Author

mowies commented Feb 7, 2024

No, it will only execute if the phases before it succeeded.

@thisthat thisthat merged commit 23a2f10 into main Feb 12, 2024
1 check passed
@mowies mowies deleted the kep/98 branch February 12, 2024 07:21
@mowies
Copy link
Member Author

mowies commented Feb 13, 2024

epic ticket for this: keptn/lifecycle-toolkit#2907

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

5 participants