Project

Profile

Help

Issue #2577

closed

last_unit_added is not updated when units are copied over from another repo

Added by bizhang about 7 years ago. Updated about 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
3. High
Version:
Platform Release:
2.12.1
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Sprint 15
Quarter:

Description

As a part of #1847[0] last_unit_added is updated for upload and sync. But as pointed out by @wiad[1] we're not updating this when content units are being copied.

Steps to reproduce:

pulp-admin rpm repo create --repo-id=test

pulp-admin rpm repo copy rpm --from-repo-id=zoo --to-repo-id=test

pulp-admin rpm repo search --str-eq "id=test"

Note that last_unit_added is not updated.

[0] https://pulp.plan.io/issues/1847
[1] https://github.com/pulp/pulp/pull/2905#issuecomment-278295468

Actions #1

Updated by mhrivnak about 7 years ago

Let's get a smash issue filed for this also.

Actions #3

Updated by bizhang about 7 years ago

  • Priority changed from Normal to High
  • Sprint/Milestone set to 33
  • Severity changed from 2. Medium to 3. High
  • Triaged changed from No to Yes

Added by bizhang about 7 years ago

Revision 0005b137 | View on GitHub

update last_unit_added when units are copied

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

Added by bizhang about 7 years ago

Revision 0005b137 | View on GitHub

update last_unit_added when units are copied

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

Actions #4

Updated by bizhang about 7 years ago

  • Status changed from NEW to POST
  • Assignee set to bizhang
Actions #5

Updated by bizhang about 7 years ago

  • Status changed from POST to MODIFIED
Actions #6

Updated by bizhang about 7 years ago

  • Platform Release set to 2.12.1
Actions #7

Updated by bizhang about 7 years ago

  • Status changed from MODIFIED to 5
Actions #8

Updated by bizhang about 7 years ago

  • Version set to 2.12.1
Actions #9

Updated by pthomas@redhat.com about 7 years ago

[root@mgmt5 ~]# pulp-admin rpm repo copy rpm --from-repo-id=zoo --to-repo-id=copy-me
This command may be exited via ctrl+c without affecting the request.

[\]
Running...

Copied:
  bear-4.1-1-noarch
  camel-0.1-1-noarch
  cat-1.0-1-noarch
  cheetah-1.25.3-5-noarch
  chimpanzee-0.21-1-noarch
  cockateel-3.1-1-noarch
  cow-2.2-3-noarch
  crow-0.8-1-noarch
  dog-4.23-1-noarch
  dolphin-3.10.232-1-noarch
  duck-0.6-1-noarch
  elephant-8.3-1-noarch
  fox-1.1-2-noarch
  frog-0.1-1-noarch
  giraffe-0.67-2-noarch
  gorilla-0.62-1-noarch
  horse-0.22-2-noarch
  kangaroo-0.2-1-noarch
  lion-0.4-1-noarch
  mouse-0.1.12-1-noarch
  penguin-0.9.1-1-noarch
  pike-2.2-1-noarch
  shark-0.1-1-noarch
  squirrel-0.1-1-noarch
  stork-0.12-2-noarch
  tiger-1.0-4-noarch
  trout-0.12-1-noarch
  walrus-0.71-1-noarch
  walrus-5.21-1-noarch
  whale-0.2-1-noarch
  wolf-9.4-2-noarch
  zebra-0.1-2-noarch

[root@mgmt5 ~]# pulp-admin rpm repo search --str-eq "id=copy-me"
+----------------------------------------------------------------------+
                              Repositories
+----------------------------------------------------------------------+

Id:                  copy-me
Display Name:        None
Description:         None
Content Unit Counts: 
  Rpm: 32
Last Unit Added:     2017-02-17T15:26:02Z
Last Unit Removed:   None
Notes:               
Scratchpad:          
Actions #10

Updated by bizhang about 7 years ago

  • Version deleted (2.12.1)
Actions #11

Updated by bizhang about 7 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

Added by bizhang about 7 years ago

Revision dda4b23e | View on GitHub

update last_unit_added when units are copied

re #2577 https://pulp.plan.io/issues/2577

(cherry picked from commit 0005b1375f02d9cbe29bf23b892fa5ec7af343b8)

The cherry-picked commit was accidentally reverted on 2.12-dev in commit c0ea5ee4bf8898d80d72c481fbc116fc548b8eeb. The original PR containing this change was https://github.com/pulp/pulp/pull/2949

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

Added by bizhang about 7 years ago

Revision dda4b23e | View on GitHub

update last_unit_added when units are copied

re #2577 https://pulp.plan.io/issues/2577

(cherry picked from commit 0005b1375f02d9cbe29bf23b892fa5ec7af343b8)

The cherry-picked commit was accidentally reverted on 2.12-dev in commit c0ea5ee4bf8898d80d72c481fbc116fc548b8eeb. The original PR containing this change was https://github.com/pulp/pulp/pull/2949

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

Added by bizhang about 7 years ago

Revision f52b538f | View on GitHub

update last_unit_added when units are copied

re #2577 https://pulp.plan.io/issues/2577

(cherry picked from commit 0005b1375f02d9cbe29bf23b892fa5ec7af343b8)

The cherry-picked commit was accidentally reverted on 2.12-dev in commit c0ea5ee4bf8898d80d72c481fbc116fc548b8eeb. The original PR containing this change was https://github.com/pulp/pulp/pull/2949

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

(cherry picked from commit dda4b23eb8334fc53a160b104063047c6492493f)

^ And this second cherry-pick happened because I opened the PR against master, not 2.12-dev. A bug so nice, we fixed it thrice and picked it twice.

Added by bizhang about 7 years ago

Revision f52b538f | View on GitHub

update last_unit_added when units are copied

re #2577 https://pulp.plan.io/issues/2577

(cherry picked from commit 0005b1375f02d9cbe29bf23b892fa5ec7af343b8)

The cherry-picked commit was accidentally reverted on 2.12-dev in commit c0ea5ee4bf8898d80d72c481fbc116fc548b8eeb. The original PR containing this change was https://github.com/pulp/pulp/pull/2949

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

(cherry picked from commit dda4b23eb8334fc53a160b104063047c6492493f)

^ And this second cherry-pick happened because I opened the PR against master, not 2.12-dev. A bug so nice, we fixed it thrice and picked it twice.

Actions #12

Updated by bmbouter about 6 years ago

  • Sprint set to Sprint 15
Actions #13

Updated by bmbouter about 6 years ago

  • Sprint/Milestone deleted (33)
Actions #14

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF