Add default envs in ScriptRunStage: SR_TRIGGERED_COMMANDER, SR_IS_ROLLBACK #5464
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>
Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>
Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>
Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>
Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>
t-kikuc
requested review from
nghialv,
khanhtc1202,
kentakozuka,
ffjlabo and
Warashi
as code owners
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5464 +/- ##
==========================================
+ Coverage 26.10% 26.13% +0.02%
==========================================
Files 451 457 +6
Lines 48837 49111 +274
==========================================
+ Hits 12748 12833 +85
- Misses 35072 35255 +183
- Partials 1017 1023 +6 ☔ View full report in Codecov by Sentry. |
Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>
Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>
t-kikuc
commented
Dec 27, 2024
| RepositoryURL: d.GitPath.Repo.Remote, | ||
| Summary: d.Summary, | ||
| Labels: d.Labels, | ||
| IsRollback: isRollback, |
There was a problem hiding this comment.
I fixed to use isRollback flag instead of d.Status.
That's because d.Status is ALWAYS PLANNED since scheduler does not update it...
ffjlabo
approved these changes
Jan 6, 2025
Merged
github-actions bot
pushed a commit
that referenced
this pull request
Jan 21, 2025
…LBACK (#5464) * Add TriggeredCommandar Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * add isRollback Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * update docs Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * fix name->id Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * fix env name Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * fix: specify isRollback since deployment.Status is always Planned Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * fix typo Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * Add case of `pipectl sync` Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> --------- Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> Signed-off-by: pipecd-bot <pipecd.dev@gmail.com>
Warashi
added a commit
that referenced
this pull request
Jan 21, 2025
* [ECS] Support 5 missing fields in TaskDefinition (#5424) * Add missing fields in RegisterTaskDef Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * remove two restrictions Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> --------- Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> Signed-off-by: pipecd-bot <pipecd.dev@gmail.com> * [ECS] Enabled to configure/update `PropagateTags` and `EnableECSManagedTags` (#5425) * Enable updating PropagateTags and specifying other than SERVICE Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * Enable updating EnableECSManagedTags Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * Remove default PropagateTags: unnecessary Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * support propagateTags other than SERVICE Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> --------- Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> Signed-off-by: pipecd-bot <pipecd.dev@gmail.com> * Enhance logging in EventWatcher (#5443) * Add logger.Error() before returning for detailed logs Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * Enhance error logging in event watcher with additional context for push and commit failures Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> --------- Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> Signed-off-by: pipecd-bot <pipecd.dev@gmail.com> * fixed to certainly store the startTime (#5461) Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> Signed-off-by: pipecd-bot <pipecd.dev@gmail.com> * Add default envs in ScriptRunStage: SR_TRIGGERED_COMMANDER, SR_IS_ROLLBACK (#5464) * Add TriggeredCommandar Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * add isRollback Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * update docs Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * fix name->id Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * fix env name Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * fix: specify isRollback since deployment.Status is always Planned Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * fix typo Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> * Add case of `pipectl sync` Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> --------- Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> Signed-off-by: pipecd-bot <pipecd.dev@gmail.com> * Release v0.50.1 (#5498) Signed-off-by: Shinnosuke Sawada-Dazai <shin@warashi.dev> Signed-off-by: pipecd-bot <pipecd.dev@gmail.com> --------- Signed-off-by: t-kikuc <tkikuchi07f@gmail.com> Signed-off-by: pipecd-bot <pipecd.dev@gmail.com> Signed-off-by: Shinnosuke Sawada-Dazai <shin@warashi.dev> Co-authored-by: Tetsuya KIKUCHI <97105818+t-kikuc@users.noreply.github.com> Co-authored-by: Shinnosuke Sawada-Dazai <shin@warashi.dev>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does:
as title
Why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: