Project

Profile

Help

Story #1700

closed

Cancellation handler for sync should exit immediately and never call into the plugin's code

Added by ipanova@redhat.com about 8 years ago. Updated about 5 years ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

Recently we removed the cancel_repo_sync() from yum sync https://github.com/pulp/pulp_rpm/pull/810 and let the base class implementation call sys.exit()

We should do the same for the rest of the plugins.


Related issues

Related to Pulp - Story #1699: As a user, if I cancel the repo sync, I don't want the auto-publish to happen.CLOSED - WONTFIX

Actions
Actions #1

Updated by ipanova@redhat.com about 8 years ago

  • Related to Story #1699: As a user, if I cancel the repo sync, I don't want the auto-publish to happen. added
Actions #2

Updated by ipanova@redhat.com about 8 years ago

  • Sprint Candidate changed from No to Yes
Actions #3

Updated by bmbouter about 8 years ago

  • Tracker changed from Refactor to Story

This change will be user visible (and great) because cancellation will occur immediately. Each plugin that is changed should get a release note. Given that, I think it should be a story.

Actions #4

Updated by mhrivnak almost 8 years ago

  • Sprint Candidate changed from Yes to No
Actions #5

Updated by bmbouter about 5 years ago

  • Status changed from NEW to CLOSED - WONTFIX
Actions #6

Updated by bmbouter about 5 years ago

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

Actions #7

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF