Project

General

Profile

Bug #33044

Katello saves publication as a repo's version_href at sync time if Pulp auto-creates publications

Added by Ian Ballou over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Repositories
Target version:

Description

SaveVersions in Katello does this:

elsif input[:tasks].present?
version_href = input[:tasks].last[:created_resources].first

The sync task also creates publications, so SaveVersions is randomly saving it. We need SaveVersions to look for a repo version to save.

Related sync task output:

---
pulp_tasks:
- pulp_href: "/pulp/api/v3/tasks/56431225-cff1-47af-917e-7fc47864c118/"
pulp_created: '2021-07-15T15:24:48.524+00:00'
state: completed
name: pulp_rpm.app.tasks.synchronizing.synchronize
logging_cid: a154fc23-bf09-4175-b259-33544b05f5fc
started_at: '2021-07-15T15:24:48.579+00:00'
finished_at: '2021-07-15T15:25:55.461+00:00'
worker: "/pulp/api/v3/workers/86c776dd-6335-448c-8fb2-a5b5915fe5ec/"
child_tasks: []
progress_reports:
- message: Downloading Metadata Files
code: sync.downloading.metadata
state: completed
done: 14
- message: Parsed Modulemd
code: sync.parsing.modulemds
state: completed
total: 110
done: 110
- message: Parsed Modulemd-defaults
code: sync.parsing.modulemd_defaults
state: completed
total: 45
done: 45
- message: Parsed Packages
code: sync.parsing.packages
state: completed
total: 6407
done: 6407
- message: Downloading Artifacts
code: sync.downloading.artifacts
state: completed
done: 0
- message: Parsed Comps
code: sync.parsing.comps
state: completed
total: 67
done: 67
- message: Associating Content
code: associating.content
state: completed
done: 6629
- message: Downloading Metadata Files
code: sync.downloading.metadata
state: completed
done: 13
- message: Downloading Artifacts
code: sync.downloading.artifacts
state: completed
done: 1
- message: Associating Content
code: associating.content
state: completed
done: 1923
- message: Un-Associating Content
code: unassociating.content
state: completed
done: 0
- message: Parsed Packages
code: sync.parsing.packages
state: completed
total: 1880
done: 1880
- message: Parsed Comps
code: sync.parsing.comps
state: completed
total: 42
done: 42
created_resources:
- "/pulp/api/v3/publications/rpm/rpm/ec641ff4-bf4d-46f4-be45-4fb2fb9eefdd/"
- "/pulp/api/v3/repositories/rpm/rpm/a6e4b32c-10fb-4e16-b8fb-bf99b9a9182b/versions/1/"
- "/pulp/api/v3/publications/rpm/rpm/1d0ebf7d-1eb2-499f-bb3f-49f8f9704fb5/"
reserved_resources_record:
- "/pulp/api/v3/repositories/rpm/rpm/a6e4b32c-10fb-4e16-b8fb-bf99b9a9182b/"
- "/pulp/api/v3/remotes/rpm/rpm/25f9965d-7d6e-4081-afbb-3c1a18c343e7/"
create_version: true
task_groups: []
poll_attempts:
total: 24
failed: 0


Related issues

Has duplicate Katello - Bug #33136: CentOS 8 repository sync errorsDuplicate

Associated revisions

Revision 8b9fbc41 (diff)
Added by Justin Sherrill over 1 year ago

Fixes #33044 - re-use existing publication on sync

As part of pulp 3.14 publications can now be created during sync time
in this case, if a publication appears, we should re-use it for
distribution instead of regenerating it.

Revision 11ad37c1 (diff)
Added by Justin Sherrill over 1 year ago

Refs #33044 - properly check publication_href on SP sync

History

#1 Updated by Ian Ballou over 1 year ago

Pulp is now creating publications automatically when mirror=true is set.

#2 Updated by Ian Ballou over 1 year ago

  • Target version set to Katello 4.1.2

Targeted to 4.1 since this relates to the pulpcore 3.14 upgrade

#3 Updated by Ian Ballou over 1 year ago

  • Category set to Repositories

#4 Updated by Ian Ballou over 1 year ago

  • Bugzilla link set to 1983143

#5 Updated by The Foreman Bot over 1 year ago

  • Assignee set to Justin Sherrill
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/9476 added

#6 Updated by The Foreman Bot over 1 year ago

  • Fixed in Releases Katello 4.3.0 added

#7 Updated by Justin Sherrill over 1 year ago

  • Status changed from Ready For Testing to Closed

#8 Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/Katello/katello/pull/9483 added

#9 Updated by James Jeffers over 1 year ago

  • Triaged changed from No to Yes

#10 Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/Katello/katello/pull/9485 added

#11 Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/Katello/katello/pull/9486 added

#12 Updated by Ian Ballou over 1 year ago

  • Has duplicate Bug #33136: CentOS 8 repository sync errors added

Also available in: Atom PDF