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 #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 #17817 (New): content-hosts import should honor --search and only process the matched hostshttps://projects.theforeman.org/issues/178172016-12-21T08:54:33ZThomas McKaythomasmckay@redhat.com
<p>Using --search during import should only process the matching hosts.</p>
<p>One problem, though, would be the processing of hypervisor and guests which occurs at the end of the import.</p> Bug #17716 (New): content-hosts - skip setting values for columns that are not specifiedhttps://projects.theforeman.org/issues/177162016-12-16T12:06:22ZThomas McKaythomasmckay@redhat.com
<p>For example, if no columns are present that require the setting of facts, do not try to update. If Subscriptions column not present, don't modify subscriptions.</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 #16234 (New): content hosts import does not populate host collectionshttps://projects.theforeman.org/issues/162342016-08-22T18:36:42ZThomas McKaythomasmckay@redhat.com
<p>The API for adding host collections to imported content hosts has changed and no longer works. Needs to support export Sat-6.1, import/export Sat-6.2, import/export katello latest.</p> Bug #15587 (New): convert Sat-5 export format to hammer csv input formathttps://projects.theforeman.org/issues/155872016-07-06T08:59:00ZThomas McKaythomasmckay@redhat.com
<p>The 'hammer import' command takes in csv from a Sat-5 export. The intention is to replace the import command with the appropriate 'hammer csv' commands, which will be supported both upstream and downstream.</p> Bug #14654 (New): default facts sent during content-host update will overwrite any existing rhsm ...https://projects.theforeman.org/issues/146542016-04-14T16:07:33ZThomas McKaythomasmckay@redhat.com
<p>During content host update a set of predefined facts are sent. If this is a real host that has been sub-mgr registered then it will have real facts. These real facts will be lost on an update to the system.</p>
<p>Fix: Solution is either 1) not send facts during update calls, or 2) fetch all existing facts and add required ones in.</p>