Bug 733474 - Sync fails silently when repo created with API, with blank URL
Summary: Sync fails silently when repo created with API, with blank URL
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Katello
Classification: Retired
Component: WebUI
Version: 1.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: ---
Assignee: Ivan Necas
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-08-25 19:04 UTC by Jeff Weiss
Modified: 2014-11-09 22:51 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
pulp-0.0.227-1.fc14.noarch
Last Closed: 2012-07-17 13:09:30 UTC
Embargoed:


Attachments (Terms of Use)

Description Jeff Weiss 2011-08-25 19:04:10 UTC
Description of problem:
The UI checks for valid urls but the API does not.  The UI does not handle this condition when syncing.

Version-Release number of selected component (if applicable):
katello-0.1.70-1.git.288.120a626.fc14.noarch

How reproducible:


Steps to Reproduce:
1. Create a product
2. In that product, use the API to create a repo with blank URL (nil).
3. In the GUI, sync the product
  
Actual results:
Fails silently

Expected results:
Fails with error message in the UI about what's wrong with that repo

Additional info:
Logs:

RestClient::NotAcceptable (Pulp::Repository: 406 Not Acceptable "Repo [1314294222681-testrepo-1314294224019-ACME_Corporation] is not setup for sync. Please add packages using upload." (POST /pulp/api/repositories/1314294222681-testrepo-1314294224019-ACME_Corporation/sync/)):
  lib/http_resource.rb:94:in `post'
  lib/resources/pulp.rb:205:in `sync'
  app/controllers/sync_management_controller.rb:173:in `sync_repos'
  app/controllers/sync_management_controller.rb:170:in `each'
  app/controllers/sync_management_controller.rb:170:in `inject'
  app/controllers/sync_management_controller.rb:170:in `sync_repos'
  app/controllers/sync_management_controller.rb:77:in `sync'
  lib/util/threadsession.rb:77:in `thread_locals'

Comment 1 Mike McCune 2011-09-02 19:24:10 UTC
The API needs to validate the URL like the UI does.  Sending to BK

Comment 2 Ivan Necas 2012-03-30 13:16:45 UTC
Fixed in commit 051d093b6cd5199a1569a23ac5f1cbf39349460c

Comment 3 Bryan Kearney 2012-07-17 13:09:30 UTC
Moving all community bugs with fixes to closed.


Note You need to log in before you can comment on or make changes to this bug.