Task #2989: Use Publication.created instead of Publisher.last_published and RepositoryVersion.created instead of Importer.last_sync. - Pulp

Send by e-mail

Task #2989

Use Publication.created instead of Publisher.last_published and RepositoryVersion.created instead of Importer.last_sync.

Added by jortel@redhat.com about 2 years ago. Updated 6 months ago.

Status:

MODIFIED

Priority:

Normal

Assignee:

daviddavis

Category:

Sprint/Milestone:

3.0

Start date:

Due date:

% Done:

100%

Platform Release:

Blocks Release:

Backwards Incompatible:

Groomed:

Sprint Candidate:

Tags:

QA Contact:

Complexity:

Smash Test:

Verified:

Verification Required:

Sprint:

Description

Use the Publication.created instead of Publisher.last_published. The Publisher.last_published may be removed if no longer needed.

Hide closed

Checklist

Remove Importer.last_sync
Remove Publisher.last_published

Associated revisions

Revision 353e019e View on GitHub
Added by daviddavis 11 months ago

Removing last_published and last_synced

fixes #2989
https://pulp.plan.io/issues/2989

Revision 353e019e View on GitHub
Added by daviddavis 11 months ago

Removing last_published and last_synced

fixes #2989
https://pulp.plan.io/issues/2989

History

#1 Updated by jortel@redhat.com about 2 years ago

Description updated (diff)

#2 Updated by mhrivnak about 2 years ago

Great observation. The "last_published" date in Pulp 2 was only needed because we didn't track the publications themselves. It also assumes that the last publication still exists, but that cannot be guaranteed.

#3 Updated by jortel@redhat.com over 1 year ago

Subject changed from Use Publication.created instead of Publisher.last_published. to Use Publication.created instead of Publisher.last_published and RepositoryVersion.created instead of Importer.last_sync.

Use Publication.created instead of Publisher.last_published and RepositoryVersion.created instead of Importer.last_sync. Let's remove the fields from the Importer and Publisher and stop managing the redundant information.