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
Fix: prevent rerun app while upgrade due to old apprev lack app workflow #4852
Fix: prevent rerun app while upgrade due to old apprev lack app workflow #4852
Conversation
76b7803
to
3e26ad6
Compare
Codecov ReportBase: 60.76% // Head: 60.82% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #4852 +/- ##
==========================================
+ Coverage 60.76% 60.82% +0.05%
==========================================
Files 300 300
Lines 31051 31060 +9
==========================================
+ Hits 18869 18893 +24
+ Misses 9687 9674 -13
+ Partials 2495 2493 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
… app workflow Signed-off-by: Somefive <yd219913@alibaba-inc.com>
3e26ad6
to
40af370
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Backport failed for Please cherry-pick the changes locally. git fetch origin release-1.5
git worktree add -d .worktree/backport-4852-to-release-1.5 origin/release-1.5
cd .worktree/backport-4852-to-release-1.5
git checkout -b backport-4852-to-release-1.5
ancref=$(git merge-base 913c740a87508d72eedd1b097c71dba6b26da54c 40af3704cc97e72eaf22dde2c58e6769db853001)
git cherry-pick -x $ancref..40af3704cc97e72eaf22dde2c58e6769db853001 |
Signed-off-by: Somefive yd219913@alibaba-inc.com
Description of your changes
In version before v1.6.0-alpha.4, applications with revision under v1.5.7 (not released yet) and revision under v1.6.0-alpha.4 (released), application revision (without PublishVersion) does not record workflow in Application Spec inside, which will lead to inequality with revisions created by latest KubeVela.
This PR adds additional rules for bypassing the comparing application workflow when
I have:
make reviewable
to ensure this PR is ready for review.How has this code been tested
Special notes for your reviewer