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

fix(sequencer): rerun steps only if explicitly requested #727

Merged
merged 11 commits into from
May 31, 2024

Conversation

cmatsuoka
Copy link
Contributor

@cmatsuoka cmatsuoka commented May 15, 2024

Instead of implicitly cleaning and re-executing steps when a part name
is specified, only do it when explicitly requested by the user.
Applications must set the rerun argument accordingly when planning
the lifecycle execution.

Fixes canonical/snapcraft#4806

  • Have you signed the CLA?

@cmatsuoka cmatsuoka requested review from lengau and mr-cal May 15, 2024 13:50
@lengau lengau added this to the 2.0 milestone May 15, 2024
Copy link
Collaborator

@mr-cal mr-cal left a comment

Choose a reason for hiding this comment

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

Looks good!

Can you add/extend an integration test in tests/integration/sequencer/test_sequencer.py?

@lengau lengau modified the milestones: 2.0, 3.0 May 17, 2024
@cmatsuoka cmatsuoka changed the base branch from main to feature/2.0 May 17, 2024 12:21
@cmatsuoka cmatsuoka requested a review from mr-cal May 18, 2024 10:50
Instead of implicitly cleaning and re-executing steps when a part name
is specified, only do it when explicitly requested by the user.
Applications must set the `rerun` argument accordingly when planning
the lifecycle execution.

Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
@cmatsuoka
Copy link
Contributor Author

Rebased on feature/2.0

Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
@cmatsuoka cmatsuoka requested a review from tigarmo May 28, 2024 21:50
@lengau lengau merged commit 9182585 into feature/2.0 May 31, 2024
10 checks passed
@lengau lengau deleted the work/only-explicit-reruns branch May 31, 2024 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants