-
Notifications
You must be signed in to change notification settings - Fork 2k
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
VReplication: Improve workflow cancel/delete #15977
Draft
mattlord
wants to merge
16
commits into
vitessio:main
Choose a base branch
from
planetscale:vtctldclient_wf_delete
base: main
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
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
mattlord
added
Type: Bug
Component: VReplication
Backport to: release-18.0
Needs to be back ported to release-18.0
Backport to: release-19.0
Needs to be back ported to release-19.0
labels
May 20, 2024
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
Tests
Documentation
New flags
If a workflow is added or modified:
Backward compatibility
|
vitess-bot
bot
added
NeedsBackportReason
If backport labels have been applied to a PR, a justification is required
NeedsDescriptionUpdate
The description is not clear or comprehensive enough, and needs work
NeedsIssue
A linked issue is missing for this Pull Request
NeedsWebsiteDocsUpdate
What it says
labels
May 20, 2024
mattlord
removed
NeedsWebsiteDocsUpdate
What it says
NeedsIssue
A linked issue is missing for this Pull Request
labels
May 20, 2024
mattlord
changed the title
VReplication: remove workflow before artifacts in vtctldclient Workflow Delete
VReplication: remove workflow before artifacts after removing workflow in vtctldclient Workflow Delete
May 20, 2024
mattlord
changed the title
VReplication: remove workflow before artifacts after removing workflow in vtctldclient Workflow Delete
VReplication: remove workflow artifacts after removing workflow in vtctldclient Workflow Delete
May 20, 2024
mattlord
changed the title
VReplication: remove workflow artifacts after removing workflow in vtctldclient Workflow Delete
VReplication: remove workflow artifacts after removing workflow in Workflow Delete
May 20, 2024
mattlord
changed the title
VReplication: remove workflow artifacts after removing workflow in Workflow Delete
VReplication: Remove workflow artifacts after removing workflow in Workflow Delete
May 20, 2024
mattlord
force-pushed
the
vtctldclient_wf_delete
branch
from
May 20, 2024 22:27
9fd6793
to
56bcebb
Compare
Otherwise the cancel/delete and cleanup work can fail. Signed-off-by: Matt Lord <mattalord@gmail.com>
mattlord
force-pushed
the
vtctldclient_wf_delete
branch
from
May 20, 2024 22:38
56bcebb
to
0b12d82
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
mattlord
changed the title
VReplication: Remove workflow artifacts after removing workflow in Workflow Delete
VReplication: Improve workflow cancel/delete
May 22, 2024
mattlord
force-pushed
the
vtctldclient_wf_delete
branch
from
May 22, 2024 12:37
b2d3777
to
d2d5efe
Compare
mattlord
removed
the
Backport to: release-18.0
Needs to be back ported to release-18.0
label
May 22, 2024
mattlord
force-pushed
the
vtctldclient_wf_delete
branch
from
May 22, 2024 12:59
d2d5efe
to
d7b39d7
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
mattlord
force-pushed
the
vtctldclient_wf_delete
branch
from
May 22, 2024 13:05
d7b39d7
to
82005fb
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
mattlord
force-pushed
the
vtctldclient_wf_delete
branch
from
May 22, 2024 13:11
1ddd859
to
7dd69c2
Compare
mattlord
force-pushed
the
vtctldclient_wf_delete
branch
3 times, most recently
from
May 22, 2024 13:38
00eb275
to
06d07f8
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
mattlord
force-pushed
the
vtctldclient_wf_delete
branch
from
May 22, 2024 13:43
06d07f8
to
371c90c
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
mattlord
force-pushed
the
vtctldclient_wf_delete
branch
from
May 22, 2024 22:38
504695a
to
4b381f5
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #15977 +/- ##
=======================================
Coverage 68.24% 68.24%
=======================================
Files 1562 1541 -21
Lines 197171 197157 -14
=======================================
- Hits 134550 134545 -5
+ Misses 62621 62612 -9 ☔ View full report in Codecov by Sentry. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Backport to: release-19.0
Needs to be back ported to release-19.0
Component: VReplication
NeedsBackportReason
If backport labels have been applied to a PR, a justification is required
NeedsDescriptionUpdate
The description is not clear or comprehensive enough, and needs work
Type: Bug
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.
Description
This work makes two changes:
ReverseTraffic
call, however, then this could result in the workflow being gone/deleted when we failed to cleanup the artifacts for any reason- Related to this, we now put those in place on the target during workflow creation rather than waiting until the first traffic switch (previously those would only be put in place on the target after
SwitchTraffic
->ReverseTraffic
)- More generally, we now do not fail the delete/cancel if the thing we went to cleanup is already gone
Related Issue(s)
Checklist