Bug 1226014

Summary:	Options -u and -a must be exclusive for foreman-debug
Product:	Red Hat Satellite	Reporter:	Corey Welton <cwelton>
Component:	Hammer	Assignee:	Lukas Zapletal <lzap>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Katello QA List <katello-qa-list>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	Unspecified	CC:	bbuckingham, bkearney
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
URL:	http://projects.theforeman.org/issues/10699
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-08-12 14:01:52 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:
Bug Depends On:
Bug Blocks:	1134109

Description Corey Welton 2015-05-28 18:37:22 UTC

Description of problem:
When user tries to zip up the debug file, it fails.

Version-Release number of selected component (if applicable):
Compose 6  RHEL7.1


How reproducible:


Steps to Reproduce:
1.  foreman-debug -g -v -a -d /tmp/test
2. View results

Actual results:
A debug directory has been created: /tmp/test

You may want to upload the tarball to our public server via rsync. There is a
write only directory (readable only by Foreman core developers) for that. Note
the rsync transmission is UNENCRYPTED:

  rsync /tmp/test.tar.xz rsync://theforeman.org/debug-incoming

Do you want to do this now? [y/N] y
Uploading...

rsync: link_stat "/tmp/test.tar.xz" failed: No such file or directory (2)


Expected results:
creation of tgz is valid.

Additional info:

Comment 2 Mike McCune 2015-05-29 15:14:48 UTC

we need to remove the upload function from foreman-debug for the downstream builds, this is an upstream only feature.

Comment 3 Lukas Zapletal 2015-06-04 08:07:46 UTC

Yeah, there is one additional patch that removes upload interactivity and adds -u flag to get that behavior. I will backport this plus add extra patch that will check flags -u and -d (and error out if both are provided).

Comment 5 Bryan Kearney 2015-06-04 18:06:00 UTC

Moving to POST since upstream bug http://projects.theforeman.org/issues/10699 has been closed
-------------
Lukas Zapletal
Applied in changeset commit:a94b02d8c7c10c2a7ab55a05ee1df9c2083d5488.

Comment 8 Corey Welton 2015-06-10 21:05:07 UTC

QE Verified in Snap 8

Comment 9 Bryan Kearney 2015-08-11 13:27:36 UTC

This bug is slated to be released with Satellite 6.1.

Comment 10 Bryan Kearney 2015-08-12 14:01:52 UTC

This bug was fixed in version 6.1.1 of Satellite which was released on 12 August, 2015.