Bug 1226014 - Options -u and -a must be exclusive for foreman-debug
Summary: Options -u and -a must be exclusive for foreman-debug
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Hammer
Version: Unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: Unspecified
Assignee: Lukas Zapletal
QA Contact: Katello QA List
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks: 1134109
TreeView+ depends on / blocked
 
Reported: 2015-05-28 18:37 UTC by Corey Welton
Modified: 2017-02-23 19:58 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-08-12 14:01:52 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 10699 0 None None None 2016-04-22 16:04:02 UTC

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.


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