Bug #19250
closedRepo sync: several parameters not passed to pulp
Description
Observed that performing an 'Advanced Sync' -> 'Validate Content Sync' is not passing the following parameters from the dynflow action input to pulp via the REST API:
download_policy: on_demand
force_full: true
auto_publish: false
E.g. REST API observed:
2017-04-11 21:24:27 [katello/pulp_rest] [D] RestClient.post "https://centos7-katello-nightly.example.com/pulp/api/v2/repositories/94387fd8-35ac-41e2-b696-387cf333220a/actions/sync/", "{\"override_config\":{\"num_threads\":4,\"validate\":true}}", "Accept"=>"*/*; q=0.5, application/xml", "Accept-Encoding"=>"gzip, deflate", "Content-Length"=>"53", "accept"=>"application/json", "content_type"=>"application/json" | \n# => 202 Accepted | application/json 172 bytes
As a result of this, if an RPM is corrupted on the filesystem, it will not be corrected during the sync operation as expected.
Scenario:
- create a product (zoo)
- create a repo (zoo) with immediate download policy
- sync repo
- after sync completes, update repo to 'on demand' policy
- perform an Advanced Sync > Validate Content Sync observe in the log (e.g. development.log) that the sync isn't including the additional parameters (listed above).
- the expected behavior is that all parameters would be included, similar to the following (observe: download_policy, force_full, & auto_publish):
2017-04-11 21:29:57 [katello/pulp_rest] [D] RestClient.post "https://centos7-katello-nightly.example.com/pulp/api/v2/repositories/94387fd8-35ac-41e2-b696-387cf333220a/actions/sync/", "{\"override_config\":{\"download_policy\":\"on_demand\",\"force_full\":true,\"auto_publish\":false,\"num_threads\":4,\"validate\":true}}", "Accept"=>"*/*; q=0.5, application/xml", "Accept-Encoding"=>"gzip, deflate", "Content-Length"=>"122", "accept"=>"application/json", "content_type"=>"application/json"
Updated by Brad Buckingham over 7 years ago
- Translation missing: en.field_release set to 211
- Bugzilla link set to 1223023
Note: this is related to bugzilla 1223023.
Updated by The Foreman Bot over 7 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/6747 added
Updated by Brad Buckingham over 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset katello|222d933dae12dcb70ac225de81c3382ea26f3ace.