Hammer CLI CSV: Issueshttps://projects.theforeman.org/https://projects.theforeman.org/favicon.ico?16970236342018-01-19T07:50:29ZForeman
Redmine Bug #22321 (New): hammer csv content-hosts create empty csv filehttps://projects.theforeman.org/issues/223212018-01-19T07:50:29ZDjebran Lezzoumldjebran@gmail.com
<p>hammer csv content-hosts create empty csv file<br />The automation test case cover customer bug <a class="external" href="https://bugzilla.redhat.com/show_bug.cgi?id=1296978">https://bugzilla.redhat.com/show_bug.cgi?id=1296978</a><br />The test in question seems randomly passing and failing, but when debugging that command we realize that command lead to other operations, where one of them is retrieving organizations, noticed that when the organization is not in that list (the list seems to be sorted by org name with limit 20) the csv file is empty, there is no error logged</p> Bug #21843 (New): Extract the strings for the foreman-admin pluginhttps://projects.theforeman.org/issues/218432017-12-01T21:27:28ZBryan Kearneybryan.kearney@gmail.com
<p>The strings are old, and not connected to transifex.</p> Bug #19945 (New): Consider matching records on import using label instead of namehttps://projects.theforeman.org/issues/199452017-06-06T14:55:12ZDavid Davisdaviddavis@redhat.com
<p>Consider the following scenario:</p>
<p>1. I export a content view to csv<br />2. I import that record into another instance<br />3. I change the content view name in the original instance<br />4. I do another import into my other instance.</p>
<p>This'll create a new record in my second instance instead of updating the existing one. With labels being immutable, the above scenario would work if we were using labels during import.</p>
<p>Not sure if this is a feature or a bug.</p> Bug #19915 (New): Support importing/exporting package group filtershttps://projects.theforeman.org/issues/199152017-06-04T15:59:02ZDavid Davisdaviddavis@redhat.com
<p>Support importing/exporting of package group filters. It looks like currently, we just export the name of the package group that the filter rule is acting on but this is insufficient since the name could exist in two different repositories. You'll note for example that you can see package group AND repo name when you go to create a package filter rule.</p>
<p>Also, we need to support importing of package filters. It looks like the code that exists is incomplete.</p> Bug #19724 (New): Support importing/exporting erratum filtershttps://projects.theforeman.org/issues/197242017-05-31T15:01:30ZDavid Davisdaviddavis@redhat.com
<p>Create an erratum filter and export it via csv. You'll get something like:</p>
<pre>
Name,Content View,Organization,Type,Description,Repositories,Rules
ERRATFILTER,Test,Default Organization,Exclude unknown,,"","start = 2012-01-01,end = 2013-01-01,""types = bugfix,enhancement,security"""
PKGFILTER,Test,Default Organization,Exclude Packages,,zoolander,test|=|3.0
</pre>
<p>Notice the type for my ERRATFILTER is unknown.</p>
<p>Relevant code: <a class="external" href="https://git.io/vHBhC">https://git.io/vHBhC</a></p> Bug #19722 (New): Content view filter export/import does not support archhttps://projects.theforeman.org/issues/197222017-05-31T10:36:36ZDavid Davisdaviddavis@redhat.com
<p>The architecture field seems to be missing.</p> Feature #18097 (New): As a user, I want to name :export: config sections for convenient use on th...https://projects.theforeman.org/issues/180972017-01-16T12:34:43ZThomas McKaythomasmckay@redhat.com
<p>For example, I have a custom export defined for content-hosts</p>
<pre>
:csv:
:enable_module: true
:columns:
:content-hosts:
:define:
- :name: Subscription Status
:json:
- subscription_status_label
- :name: Last Checkin
:json:
- subscription_facet_attributes
- last_checkin
:export:
- Name
- Organization
- Subscription Status
- Last Checkin
</pre>
<p>Currently the above config means that --export will always export just those four columns.</p>
<p>Instead, let me add a :name: to the :export: section that may be referred to. Something like this if I had included ":name: substatus" in the conf</p>
<p>hammer csv content-hosts --export --export-name substatus</p> Bug #17693 (New): content-hosts --help missing description for SUBS_GUEST columnhttps://projects.theforeman.org/issues/176932016-12-15T14:39:25ZThomas McKaythomasmckay@redhat.com
<p>The --help is missing description of SUBS_GUEST. Another case where a more robust column definition would be better; too many manual places in the code.</p> Bug #17645 (New): add a --import just because it's natural to type it after using --exporthttps://projects.theforeman.org/issues/176452016-12-12T16:09:50ZThomas McKaythomasmckay@redhat.com
<p>The default is to import the specified --file, however it is natural for user to think that --import is an option to use. Add that option (make it conflict with --export) but no need to actually check it since code should always "if !--export, then import"</p> Bug #17551 (New): change content-hosts columns --help output to use new help extensionshttps://projects.theforeman.org/issues/175512016-12-01T10:51:10ZThomas McKaythomasmckay@redhat.com
<p><a class="external" href="https://github.com/theforeman/hammer-cli-foreman/pull/263/files#diff-d97db2a6b7f5e45c90b2311d60668dc9R5">https://github.com/theforeman/hammer-cli-foreman/pull/263/files#diff-d97db2a6b7f5e45c90b2311d60668dc9R5</a></p>
<p>The ability to add extended command info was added and the column help for content-hosts should leverage that rather than trying to just output more strings.</p>
<p>May also be worth changing the way columns are defined to be more structured so that the help framework can be shared between commands. At the moment with the columns simply being variables rather than a list or hash or something the help is very manual process.</p> Feature #16675 (New): hammer csv subscription --in-portal should prompt for portal-password rathe...https://projects.theforeman.org/issues/166752016-09-23T13:33:41ZAndrew Kofinkakofink@redhat.com
<p>Passing `--portal-password superSecretP@assw0rd` via the command line is insecure for multiple reasons such as cli history logging, etc. It would be much better if the user could pass a blank `--portal-password` and hammer prompt the user for the password instead.</p> Bug #16672 (New): column to disable console passwordshttps://projects.theforeman.org/issues/166722016-09-23T10:19:26ZThomas McKaythomasmckay@redhat.com
<p>Need a column to set console passwords toggle</p> Bug #15937 (New): add auto-attach column to content-hostshttps://projects.theforeman.org/issues/159372016-08-01T13:22:12ZThomas McKaythomasmckay@redhat.comBug #15747 (New): add column for download policyhttps://projects.theforeman.org/issues/157472016-07-19T14:44:08ZThomas McKaythomasmckay@redhat.com
<p>The download policy should be reflected in the export/import of repositories.</p> Bug #15106 (New): arch and release not calculated correctly on subscriptions exporthttps://projects.theforeman.org/issues/151062016-05-20T04:45:19ZThomas McKaythomasmckay@redhat.com
<p>Red Hat OpenShift Enterprise 3.1 (RPMs)<br />Red Hat Enterprise Linux 7 Server (RPMs)</p>
<p>Perhaps there is a field in json to use instead of trying to parse the content set name</p>