Bug 1698383

Summary: [RFE] Unify Web UI distro url import behavior with CLI distro-import
Product: [Retired] Beaker Reporter: Tomas Klohna 🔧 <tklohna>
Component: generalAssignee: Matej Dujava <mdujava>
Status: CLOSED NEXTRELEASE QA Contact: tools-bugs <tools-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 26CC: bpeck, cbeer, cbouchar, dowang, mastyk, tklohna
Target Milestone: 27.0Keywords: FutureFeature, Patch, Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-10-22 13:54:13 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Tomas Klohna 🔧 2019-04-10 09:38:23 UTC
Import Distro Tree URL in Web behaves differently and has separate code from the distro-import CLI. Some of this behavior can be unified so they both behave the same.

For instance:
* Importing duplicate URL through CLI will lead to distro-import skipping over that URL but doing this in Web UI will result in error (500, should be fixed as part of this bug.
* Importing URL with same scheme that is already present in said distro tree will lead to distro-import replacing the old URL with the new one. Doing this in Web UI will result in multiple URLs with the same scheme (that should never happen)
* There is non-functional part in the distro-import code that's supposed to be checking if URL is not absolute. Right now it's not working but works correctly in the Web UI. Should be fixed and unified as well.

Comment 1 Tomas Klohna 🔧 2019-04-10 09:39:24 UTC
*** Bug 1198948 has been marked as a duplicate of this bug. ***