Project

Profile

Help

Issue #1440

closed

Puppet repository doesn't sync second and consequent repositories

Added by placko about 8 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Master
Platform Release:
2.8.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

Steps to reproduce bug:

  1. Create puppet repository repo1 with feed containing pulp manifest (auto_publish=True)

  2. Sync repo1

  3. Create puppet repository repo2 with feed containing pulp manifest (auto_publish=True)

  4. Sync repo2
    repo1 now contains 2 units as expected, while repo2 is empty. This happens only on development version of pulp, not in 2.7 or 2.6.

    $ pulp-admin puppet repo create --repo-id=repo1 --feed=https://repos.fedorapeople.org/pulp/pulp/demo_repos/puppet_manifest/modules/
    Successfully created repository [repo1]
    
    [root@ec2-54-216-139-81 ~]# pulp-admin puppet repo sync run --repo-id repo1
    +----------------------------------------------------------------------+
                        Synchronizing Repository [repo1]
    +----------------------------------------------------------------------+
    
    This command may be exited via ctrl+c without affecting the request.
    
    Downloading metadata...
    [==================================================] 100%
    Metadata Query: 1/1 items
    ... completed
    
    Downloading new modules...
    [==================================================] 100%
    Module: 2/2 items
    ... completed
    
    Task Succeeded
    
    Publishing modules...
    [==================================================] 100%
    Module: 2/2 items
    ... completed
    
    Generating repository metadata...
    [-]
    ... completed
    
    Publishing repository over HTTP...
    ... completed
    
    Publishing repository over HTTPS...
    ... skipped
    
    Task Succeeded
    
    $ pulp-admin puppet repo list
    +----------------------------------------------------------------------+
                              Puppet Repositories
    +----------------------------------------------------------------------+
    
    Id:                  repo1
    Display Name:        None
    Description:         None
    Content Unit Counts:
      Puppet Module: 2
    
    $ pulp-admin puppet repo create --repo-id=repo2 --feed=https://repos.fedorapeople.org/pulp/p
    ulp/demo_repos/puppet_manifest/modules/
    Successfully created repository [repo2]
    
    $ pulp-admin puppet repo sync run --repo-id repo2
    +----------------------------------------------------------------------+
                        Synchronizing Repository [repo2]
    +----------------------------------------------------------------------+
    
    This command may be exited via ctrl+c without affecting the request.
    
    Downloading metadata...
    [==================================================] 100%
    Metadata Query: 1/1 items
    ... completed
    
    Downloading new modules...
    [==================================================] 100%
    Module: 0/0 items
    ... completed
    
    Task Succeeded
    
    Publishing modules...
    [==================================================] 100%
    Module: 0/0 items
    ... completed
    
    Generating repository metadata...
    [-]
    ... completed
    
    Publishing repository over HTTP...
    ... completed
    
    Publishing repository over HTTPS...
    ... skipped
    
    Task Succeeded
    
    $ pulp-admin puppet repo list
    +----------------------------------------------------------------------+
                              Puppet Repositories
    +----------------------------------------------------------------------+
    
    Id:                  repo1
    Display Name:        None
    Description:         None
    Content Unit Counts:
      Puppet Module: 2
    
    Id:                  repo2
    Display Name:        None
    Description:         None
    Content Unit Counts:
    
    $
    
Actions #1

Updated by mhrivnak about 8 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com
  • Priority changed from Normal to High
  • Platform Release set to 2.8.0
  • Triaged changed from No to Yes
Actions #2

Updated by dkliban@redhat.com about 8 years ago

I was able to reproduce this on 2.8.0 nightly build.

Actions #3

Updated by dkliban@redhat.com about 8 years ago

Just tested on 2.7.1 and did not see the same behavior. This is definitely a regression.

This seems to only occur when the modules in the second repository have previously been imported into Pulp (via a different repo).

Actions #4

Updated by rbarlow about 8 years ago

  • Project changed from Pulp to Puppet Support
  • Category deleted (14)
Actions #5

Updated by dkliban@redhat.com about 8 years ago

  • Status changed from ASSIGNED to POST

Added by dkliban@redhat.com about 8 years ago

Revision 91bc0c58 | View on GitHub

Adds use of GetLocalUnitsStep to track which units already exist in Pulp

During conversion to mongoengine the check for already existing units broke. This fixes that problem.

https://pulp.plan.io/issues/1440 closes #1440

Actions #6

Updated by dkliban@redhat.com about 8 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #7

Updated by dkliban@redhat.com about 8 years ago

  • Status changed from MODIFIED to 5
Actions #8

Updated by dkliban@redhat.com almost 8 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
Actions #9

Updated by bmbouter almost 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF