Project

General

Profile

Bug #6651

Content: Should not be able to try and sync a repo with no repourl

Added by Partha Aji over 5 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web UI
Target version:
Difficulty:
trivial
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1094986
Description of problem:
User can create a product repo that contains no repourl. When user attempts to save this, save is successful, and subsequent attempts to sync produce an error. We should either disallow creation of repo w/o url, or disable sync on such repos.

Version-Release number of selected component (if applicable):

Satellite-6.0.3-RHEL-6-20140430.4
How reproducible:

Steps to Reproduce:
1. Create a product, "testproduct"
2. Attempt to create a repo within "testproduct", called "testrepo" do NOT provide a repourl
3. Attempt to sync.

Actual results:
Error in UI

Expected results:
We shouldn't let users save incomplete repos OR
We should disable sync button on repos that are obviously incomplete.

Additional info:

Associated revisions

Revision a06bb953 (diff)
Added by Partha Aji over 5 years ago

Fixes #6651,bz1094986 - Disable Sync without repo url

Added code not allow syncing on repositories without a URL.

Revision e5132143
Added by Partha Aji over 5 years ago

Merge pull request #4442 from parthaa/disable-sync

Fixes #6651,bz1094986 - Disable Sync without repo url

History

#1 Updated by The Foreman Bot over 5 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 49
  • Pull request https://github.com/Katello/katello/pull/4442 added
  • Pull request deleted ()

#2 Updated by Eric Helms over 5 years ago

  • Difficulty set to trivial
  • Triaged changed from No to Yes

#3 Updated by Partha Aji over 5 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#4 Updated by Eric Helms about 5 years ago

  • Legacy Backlogs Release (now unused) set to 13

Also available in: Atom PDF