Project

Profile

Help

Issue #4518

Updated by bherring over 5 years ago

h2. Concern 

 The behavior in 2.18.1 is incorrect for modular errata copy which could affect Satellite. 

 The correct behavior is now seen, recently, in 2-master. 

 Verified incorrect behavior with Tanya. 

 h2. Chronology 

 A regression in test_modular_copy.py [0] was noted in nightly [1] once libsolv [2] was resolved by pcreech (thanks!) on 3/3/2019 in #4405. 

 Upon looking through the newly delivered modular documentation [3], it appears that that current behavior in 2-master is correct and the behavior test_modular_copy.py and resulting behavior in 2.18.1-stable is incorrect (and has been since Sept 2018 [0]).  

 The test was originally written and approved without the understanding of the *modulemd* copies needed as part of the errata. 

 Please note the CLI examples below for comparisons between 2.18.1 and latest 2-master for all permutations of *--recursive* and *--recursive-conservative* through the CLI. 

 h3. Test Update 

 The test with the correct modular errata copy behavior check [4]    is already submitted and awaiting development designation of problem scope before implementing into test repositories.  

 Until then, we have a known, failing test in nightly. 

 h2. 2.18.1 Behaviors (since Sept 21 2018 - 2.18.1-GA) [incorrect] 

 h3. --recursive 

 <pre> 
 [root@rhel76-218 ~]# pulp-admin rpm repo copy errata --str-eq="errata_id=RHEA-2012:0059" --from-repo-id=test --to-repo-id=new --recursive 
 This command may be exited via ctrl+c without affecting the request. 


 [\] 
 Running... 

 Copied: 
  erratum: 
   RHEA-2012:0059 
  rpm: 
   duck-0.7-1-noarch 
   kangaroo-0.3-1-noarch 


 </pre> 

 h3. --recursive-conservative 

 <pre> 
 [root@rhel76-218 ~]# pulp-admin rpm repo copy errata --str-eq="errata_id=RHEA-2012:0059" --from-repo-id=test --to-repo-id=new --recursive-conservative 
 This command may be exited via ctrl+c without affecting the request. 


 [\] 
 Running... 

 Copied: 
  erratum: 
   RHEA-2012:0059 
  rpm: 
   duck-0.7-1-noarch 
   kangaroo-0.3-1-noarch 

 </pre> 

 h3. Both 

 <pre> 
 [root@rhel76-218 ~]# pulp-admin rpm repo copy errata --str-eq="errata_id=RHEA-2012:0059" --from-repo-id=test --to-repo-id=new --recursive-conservative --recursive 
 This command may be exited via ctrl+c without affecting the request. 


 [\] 
 Running... 

 Copied: 
  erratum: 
   RHEA-2012:0059 
  rpm: 
   duck-0.7-1-noarch 
   kangaroo-0.3-1-noarch 

 </pre> 

 h2. 2.19 Behaviors (testable since libsolv issues were resolved on ~3/1) [correct] 

 h3. --recursive 

 <pre> 
 [root@rhel76-219 ~]# pulp-admin rpm repo copy errata --str-eq="errata_id=RHEA-2012:0059" --from-repo-id=test --to-repo-id=new --recursive 
 This command may be exited via ctrl+c without affecting the request. 


 [\] 
 Running... 

 Copied: 
  erratum: 
   RHEA-2012:0059 
  modulemd: 
   duck-0-20180730233102-deadbeef-noarch 
   kangaroo-0-20180730223407-deadbeef-noarch 
  rpm: 
   duck-0.7-1-noarch 
   kangaroo-0.3-1-noarch 


 </pre> 

 h3. --recursive-conservative 

 <pre> 
 [root@rhel76-219 ~]# pulp-admin rpm repo copy errata --str-eq="errata_id=RHEA-2012:0059" --from-repo-id=test --to-repo-id=new --recursive-conservative 
 This command may be exited via ctrl+c without affecting the request. 


 [\] 
 Running... 

 Copied: 
  erratum: 
   RHEA-2012:0059 
  modulemd: 
   duck-0-20180730233102-deadbeef-noarch 
   kangaroo-0-20180730223407-deadbeef-noarch 
  rpm: 
   duck-0.7-1-noarch 
   kangaroo-0.3-1-noarch 
 </pre> 

 h3. Both 

 <pre> 
 [root@rhel76-219 ~]# pulp-admin rpm repo copy errata --str-eq="errata_id=RHEA-2012:0059" --from-repo-id=test --to-repo-id=new --recursive-conservative --recursive 
 This command may be exited via ctrl+c without affecting the request. 


 [\] 
 Running... 

 Copied: 
  erratum: 
   RHEA-2012:0059 
  modulemd: 
   duck-0-20180730233102-deadbeef-noarch 
   kangaroo-0-20180730223407-deadbeef-noarch 
  rpm: 
   duck-0.7-1-noarch 
   kangaroo-0.3-1-noarch 
 </pre> 


 h2. References 

 [0] - https://github.com/PulpQE/Pulp-2-Tests/commit/98a096f3d8defd23e3b7608249b87629b647794e#diff-7b8230102daf801887b80fe1b295e564 
 [1] - https://pulp-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/view/Pulp%202%20-%20Master/job/pulp-2-master-dev-rhel7/372/testReport/ 
 [2] - https://pulp.plan.io/issues/4405 
 [3] - https://github.com/pulp/pulp_rpm/blob/656d34eb2c4986574ffa8a95c1c61e9a1233b6cf/docs/user-guide/features.rst 
 [4] - https://github.com/PulpQE/Pulp-2-Tests/pull/168 

Back