foreman_abrt 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2a1e0fe4d7a14fcc470200bc41dbd3ccd7209e01
-  data.tar.gz: 656ef1589530a037fd2669cd6ea7014d29bbef31
+  metadata.gz: 9b48790120660d859c2e437373bc4805aa206ac3
+  data.tar.gz: c44d479890517cb1b565a282d97a4d07c22bfc74
 SHA512:
-  metadata.gz: 12800b0a258c8d3f290c0e05b62a170b5f1de76834b1fd8488b7e879a833911250636fbbea47b48f421e1d69d60105bfd3a29b3b4c67883afabc5836b8b32c8b
-  data.tar.gz: 8e426bc88cf4bd925db6c213da9a2206665c22ecfbad1b4f057ab6d0cf1f2caf38b356754168fb3a489342309fc9e447ab4e06682f4451e860fd4fa5194fa4c0
+  metadata.gz: f0a1b5e461594c02f2620eb722f8f87a3d1631bd3325495aa2a83fe60f048b2e27e34bdb210bcaf9082753688d7b4e31ca077838185d4c9fe084bf2accfe3dc1
+  data.tar.gz: a9984b9f7dbbaacffee2c24deddaf09e97e4731d73ac02ba7b313c10af93b55a13357be21062d7d28f478281fd42996b331ebccdd179f765098a3649c3197cda

data/README.md

@@ -22,20 +22,7 @@ server.
    responds with additional information about the report, such as links to bug
    trackers or suggested solutions, it is displayed alongside the report.
 
-```
-
- +--------------+  ureport   +-------------+  aggregated ureports   +---------+
- | Managed host | ---------> | Smart proxy | ---------------------> | Foreman |
- +--------------+            +-------------+                        +---------+
-                                    :                                  :  ^
-                                    :                          ureport :  :
-                                    :                                  :  : server response
-                                    :                                  V  :
-                                    :          ureport            +-------------+
-                                    + - - - - - - - - - - - - - ->| ABRT server |
-                                                                  +-------------+
-
-```
+![foreman abrt workflow](/extra/foreman_abrt_workflow.png?raw=true)
 
 ## Installation
 
@@ -43,19 +30,37 @@ To be able to see ABRT bug reports in your Foreman instance, you need to
 install the plugin itself, install ABRT plugin for your smart proxies and
 configure your hosts to send the bug reports to their smart proxy.
 
-Both plugins are available as RPMs in [Foremay YUM repositories](http://yum.theforeman.org/).
+Both plugins are available as RPMs in [Foreman YUM repositories](http://yum.theforeman.org/).
+
+### Prerequisites
 
 The plugins require both Foreman and smart-proxy to be version 1.6 or later.
 
+The plugins have been tested on Fedora 19, RHEL 6 and RHEL 7. Due to old
+version of `rubygem-ffi` the smart-proxy plugin does not work on RHEL 6 unless
+you set the option `aggregate_reports` to `false` in
+`/etc/foreman-proxy/settings.d/abrt.yml`.
+
+To have hosts automatically send ureports to Foreman, you need to have ABRT
+2.1.11 or higher installed on them. RHEL 7 and Fedora 19 and higher satisfy
+this. ABRT in RHEL6 does not have ureport support, however you can use an
+[unofficial build](https://copr.fedoraproject.org/coprs/jfilak/abrt/) if you
+wish to test this feature.
+
 ### Installing the Foreman plugin
 
 To install the Foreman plugin, follow the [plugin installation
 instructions](http://projects.theforeman.org/projects/foreman/wiki/How_to_Install_a_Plugin).
 
+You need to install the `rubygem-foreman_abrt` package (or
+`ruby193-rubygem-foreman_abrt` on RHEL/CentOS).
+
 ### Setting up smart proxies
 
 Follow the [smart-proxy plugin installation
 instructions](http://projects.theforeman.org/projects/foreman/wiki/How_to_Install_a_Smart-Proxy_Plugin).
+You need to install the `rubygem-smart_proxy_abrt` package.
+
 The plugin needs some configuration in order to work correctly.
 
 - Edit `/etc/foreman-proxy/settings.yml` to configure the main Foreman host,
@@ -66,6 +71,9 @@ The plugin needs some configuration in order to work correctly.
   # URL of your foreman instance
   :foreman_url: https://f19-foreman.tld
   ```
+  Please note that the `:foreman_url:` setting may be entirely missing in the
+  file. In that case just add the line to the end of
+  `/etc/foreman-proxy/settings.yml`.
 
 - Ensure that `/etc/foreman-proxy/settings.d/abrt.yml` contains the following line:
   ```
@@ -92,15 +100,23 @@ The plugin needs some configuration in order to work correctly.
   ```
   # URL of your foreman-proxy, with /abrt path.
   URL = https://f19-smartproxy.tld:8443/abrt
-  # Do not verify server certificate.
-  SSLVerify = no
+  # Verify the server certificate.
+  SSLVerify = yes
   # This asks puppet config for the path to the ceritificates. you can
   # explicitly provide path by using /path/to/cert:/path/to/key on the
   # right hand side.
   SSLClientAuth = puppet
   ```
 
-- Enable autoreporting by running the following command:
+- Add the Puppet CA to the list of trusted certificate authorities. This is
+  needed for verifying the validity of smart-proxy's certificate:
+
+  ```
+  ~# cp /var/lib/puppet/ssl/certs/ca.pem /etc/pki/ca-trust/source/anchors/
+  ~# update-ca-trust
+  ```
+
+- Enable auto-reporting by running the following command:
 
   ```
   ~# abrt-auto-reporting enabled
@@ -119,12 +135,20 @@ Segmentation fault (core dumped)
 ```
 
 After a couple of seconds, a new file should appear in
-`/var/spool/foreman-proxy/abrt-send` on the smart-proxy host. The reports from
+`/var/spool/foreman-proxy-abrt` on the smart-proxy host. The reports from
 the smart-proxy are sent to the Foreman in batches every half an hour (by
 default). This means that within half an hour you should be able to see the bug
 report in the Foreman web interface. You can send the reports to Foreman
 manually by running the `smart-proxy-abrt-send` command.
 
+If the will-crash package is not available, you can try the following. Please
+make sure not to actually report this to `sleep` maintainers, though:
+
+```
+~$ sleep 1d &
+~$ kill -SEGV $!
+```
+
 ## Usage
 
 The list of received bug reports can be accessed by clicking on *Bug reports*
@@ -147,12 +171,7 @@ configuration screen (*Administer*->*Settings*).
 
 ## TODO
 
-- Forwarding reports on the proxy - drop it altogether, or forward the server
-  response to the client?
 - Use puppet to configure managed hosts to send ureports to Foreman.
-- Figure out how to import the Puppet CA cert on managed hosts to the system
-  certificates so that the reporter-ureport doesn't have to skip server
-  certificate validation.
 
 ## Copyright
 

data/app/controllers/api/v2/abrt_reports_controller.rb

@@ -13,12 +13,12 @@ module Api
         rescue => e
           logger.error "Failed to import ABRT report: #{e.message}"
           logger.debug e.backtrace.join("\n")
-          render :json => { "message" => e.message }, :status => :unprocessable_entity
+          render :json => { 'message' => e.message }, :status => :unprocessable_entity
           return
         end
 
         if abrt_reports.count == 0
-          render :json => { "Failed to import any report" => e.message }, :status => :unprocessable_entity
+          render :json => { 'message' => 'Failed to import any report'}, :status => :unprocessable_entity
           return
         end
 
@@ -35,7 +35,7 @@ module Api
 
         # Do not report forwarding error to the proxy, we can manually resend
         # it later and the proxy probably can't do anything about it anyway.
-        render :json => { "message" => "OK" }
+        render :json => { 'message' => 'OK' }
       end
 
     end

data/app/helpers/abrt_reports_helper.rb

@@ -1,39 +1,49 @@
 module AbrtReportsHelper
-  def simple_format_if_multiline str
-    if str.include? "\n"
+  def simple_format_if_multiline(str)
+    if str and str.include? "\n"
       simple_format str
     else
       str
     end
   end
 
-  def count_abrt_reports abrt_reports
+  def count_abrt_reports(abrt_reports)
     range_days = 14
-    data = []
-    now = Time.now.utc
-    start = now - range_days.days
-    by_day = abrt_reports.where(:reported_at => start..now).
-                          group('DATE(reported_at)').
-                          sum(:count)
+    data       = []
+    now        = Time.now.utc
+    start      = now - range_days.days
+    by_day     = abrt_reports.where(:reported_at => start..now).
+      group('DATE(reported_at)').
+      sum(:count)
 
     range_days.downto(1) do |days_back|
-      date = (now - (days_back-1).days).strftime('%Y-%m-%d')
-      crashes = by_day[date] or 0
+      date    = (now - (days_back-1).days).strftime('%Y-%m-%d')
+      crashes = (by_day[date] || 0)
       data << [days_back, crashes]
     end
     data
   end
 
-  def render_abrt_graph abrt_reports, options = {}
+  def render_abrt_graph(abrt_reports, options = {})
     data = count_abrt_reports abrt_reports
     flot_bar_chart 'abrt_graph', _('Days Ago'), _('Number of crashes'), data, options
   end
 
-  def send_to_abrt_server abrt_report
+  class StringIOWithPath < StringIO
+    def initialize(string, path, content_type)
+      super(string)
+      @path         = path
+      @content_type = content_type
+    end
+
+    attr_reader :path, :content_type
+  end
+
+  def send_to_abrt_server(abrt_report)
     request_params = {
-      :timeout => 60,
+      :timeout      => 60,
       :open_timeout => 10,
-      :verify_ssl => Setting[:abrt_server_verify_ssl] ? OpenSSL::SSL::VERIFY_PEER : OpenSSL::SSL::VERIFY_NONE
+      :verify_ssl   => Setting[:abrt_server_verify_ssl] ? OpenSSL::SSL::VERIFY_PEER : OpenSSL::SSL::VERIFY_NONE
     }
 
     if Setting[:abrt_server_ssl_ca_file] && !Setting[:abrt_server_ssl_ca_file].empty?
@@ -46,20 +56,21 @@ module AbrtReportsHelper
       request_params[:ssl_client_key]  = OpenSSL::PKey::RSA.new(File.read(Setting[:abrt_server_ssl_priv_key]))
     end
 
-    resource = RestClient::Resource.new(Setting[:abrt_server_url], request_params)
-    response = resource['reports/new/'].post({:file => abrt_report.json, :multipart => true}, :content_type => :json, :accept => :json)
+    resource    = RestClient::Resource.new(Setting[:abrt_server_url], request_params)
+    report_file = StringIOWithPath.new(abrt_report.json, '*buffer*', 'application/json')
+    response    = resource['reports/new/'].post({ :file => report_file, :multipart => true }, :content_type => :json, :accept => :json)
 
     if response.code != 202
       logger.error "Failed to forward bug report: #{response.code}: #{response.to_str}"
-      raise ::Foreman::Exception.new(N_("Failed to forward bug report: %s: %s", response.code, response.to_str))
+      raise ::Foreman::Exception.new(N_('Failed to forward bug report: %s: %s', response.code, response.to_str))
     end
 
     JSON.parse(response.body)
   end
 
-  def format_reason reason
+  def format_reason(reason)
     if reason.nil? or reason.empty?
-      _("Unknown")
+      _('Unknown')
     else
       reason
     end

data/app/models/abrt_report.rb

@@ -40,13 +40,33 @@ class AbrtReport < ActiveRecord::Base
   scoped_search :in => :abrt_report_response_solutions, :on => :url,   :complete_value => true, :rename => :solution_url
 
   def self.import(json)
-    host = Host.find_by_name(json[:host])
     reports = []
+    host    = Host.find_by_name(json[:host])
+
+    # If the host lookup failed, it is possible that we are using subscription management certificates which
+    # have UUID in their CN instead of host's FQDN. Try asking Katello if it knows host with such UUID.
+    if host.nil? && defined?(::Katello::System) && content_host = ::Katello::System.find_by_uuid(json[:host])
+        host = content_host.foreman_host
+    end
+
+    # The subscription management certificate also has (what is likely) FQDN in the subjectAltName certificate field.
+    # Try finding such host as a last resort.
+    if host.nil? && !json[:althosts].nil?
+      json[:althosts].each do |hostname|
+        host = Host.find_by_name(hostname)
+        break unless host.nil?
+      end
+    end
+
+    if host.nil?
+      logger.error "Unable to find host #{json[:host]}"
+      return []
+    end
 
     json[:reports].each do |report|
       begin
-        reports << AbrtReport.create!(:host => host, :count => report[:count], :json => report[:full].to_json,
-                                      :duphash => report[:duphash], :reason => report[:full][:reason],
+        reports << AbrtReport.create!(:host        => host, :count => report[:count], :json => report[:full].to_json,
+                                      :duphash     => report[:duphash], :reason => report[:full][:reason],
                                       :reported_at => report[:reported_at])
       rescue => e
         logger.error "Failed to import ABRT report from #{host}: #{e.class}:#{e.message}"
@@ -61,17 +81,17 @@ class AbrtReport < ActiveRecord::Base
       abrt_report_response_solutions.clear
       abrt_report_response_destinations.clear
 
-      self.forwarded_at = Time.now
-      self.response_known = response['result']
+      self.forwarded_at     = Time.now
+      self.response_known   = response['result']
       self.response_message = response['message']
-      self.response_bthash = response['bthash']
+      self.response_bthash  = response['bthash']
 
       if response['solutions']
         response['solutions'].each do |solution|
           abrt_report_response_solutions.create!(
             :cause => solution['cause'],
-            :note => solution['note'],
-            :url => solution['url']
+            :note  => solution['note'],
+            :url   => solution['url']
           )
         end
       end
@@ -80,7 +100,7 @@ class AbrtReport < ActiveRecord::Base
         response['reported_to'].each do |destination|
           abrt_report_response_destinations.create!(
             :desttype => destination['type'],
-            :value => destination['value'],
+            :value    => destination['value'],
             :reporter => destination['reporter']
           )
         end

data/app/models/concerns/foreman_abrt/host_extensions.rb

@@ -2,7 +2,15 @@ module ForemanAbrt::HostExtensions
   extend ActiveSupport::Concern
 
   included do
-    has_many :abrt_reports, :dependent => :destroy, :foreign_key => "host_id"
+    has_many :abrt_reports, :dependent => :destroy, :foreign_key => 'host_id'
+
+    scoped_search :in => :abrt_reports, :on => :reason,           :complete_value => true, :rename => :abrt_report_reason
+    scoped_search :in => :abrt_reports, :on => :count,            :complete_value => true, :rename => :abrt_report_count,     :only_explicit => true
+    scoped_search :in => :abrt_reports, :on => :reported_at,      :complete_value => true, :rename => :abrt_report_reported,  :only_explicit => true
+
+    scoped_search :in => :abrt_reports, :on => :forwarded_at,     :complete_value => true, :rename => :abrt_report_forwarded, :only_explicit => true
+    scoped_search :in => :abrt_reports, :on => :response_known,   :complete_value => true, :rename => :abrt_report_known,     :only_explicit => true
+    scoped_search :in => :abrt_reports, :on => :response_message, :complete_value => true, :rename => :abrt_report_response
   end
 
   def recent_abrt_reports

data/app/models/setting/abrt.rb

@@ -3,13 +3,13 @@ class Setting::Abrt < ::Setting
   def self.load_defaults
     return unless super
 
-    fqdn = Facter.value(:fqdn) || SETTINGS[:fqdn]
-    lower_fqdn = fqdn.downcase
+    fqdn         = Facter.value(:fqdn) || SETTINGS[:fqdn]
+    lower_fqdn   = fqdn.downcase
 
     # Try taking the provisioning SSL setup for default
-    ssl_cert     = Setting[:ssl_certificate] or "#{SETTINGS[:puppetvardir]}/ssl/certs/#{lower_fqdn}.pem"
-    ssl_ca_file  = Setting[:ssl_ca_file]     or "#{SETTINGS[:puppetvardir]}/ssl/certs/ca.pem"
-    ssl_priv_key = Setting[:ssl_priv_key]    or "#{SETTINGS[:puppetvardir]}/ssl/private_keys/#{lower_fqdn}.pem"
+    ssl_cert     = Setting[:ssl_certificate] || "#{SETTINGS[:puppetvardir]}/ssl/certs/#{lower_fqdn}.pem"
+    ssl_ca_file  = Setting[:ssl_ca_file]     || "#{SETTINGS[:puppetvardir]}/ssl/certs/ca.pem"
+    ssl_priv_key = Setting[:ssl_priv_key]    || "#{SETTINGS[:puppetvardir]}/ssl/private_keys/#{lower_fqdn}.pem"
 
     Setting.transaction do
       [
@@ -19,7 +19,7 @@ class Setting::Abrt < ::Setting
         self.set('abrt_server_ssl_priv_key', N_('SSL private key path that Foreman would use to communicate with ABRT server'), ssl_priv_key),
         self.set('abrt_server_ssl_ca_file', N_('SSL CA file that Foreman will use to communicate with ABRT server'), ssl_ca_file),
         self.set('abrt_automatically_forward', N_('Automatically forward every report to ABRT server?'), false),
-      ].compact.each { |s| self.create s.update(:category => "Setting::Abrt") }
+      ].compact.each { |s| self.create s.update(:category => 'Setting::Abrt') }
     end
 
     true

data/app/overrides/add_host_bug_report_tab.rb

@@ -1,11 +1,11 @@
-Deface::Override.new(:virtual_path => "hosts/show",
-                     :name => "add_bug_report_tab",
-                     :insert_bottom => "ul.nav-tabs",
-                     :partial => "abrt_reports/host_tab"
+Deface::Override.new(:virtual_path  => 'hosts/show',
+                     :name          => 'add_bug_report_tab',
+                     :insert_bottom => 'ul.nav-tabs',
+                     :partial       => 'abrt_reports/host_tab'
 )
 
-Deface::Override.new(:virtual_path => "hosts/show",
-                     :name => "add_bug_report_tab_pane",
-                     :insert_bottom => "div.tab-content",
-                     :partial => "abrt_reports/host_tab_pane"
+Deface::Override.new(:virtual_path  => 'hosts/show',
+                     :name          => 'add_bug_report_tab_pane',
+                     :insert_bottom => 'div.tab-content',
+                     :partial       => 'abrt_reports/host_tab_pane'
 )

data/app/views/abrt_reports/_host_tab_pane.html.erb

@@ -1,21 +1,21 @@
 <div class="tab-pane" id="abrt">
-<% if @host.recent_abrt_reports.any? %>
-  <table class="table table-bordered table-striped">
-    <tr>
-      <th>Reported</th>
-      <th>#</th>
-      <th>Reason</th>
-    </tr>
-    <% @host.recent_abrt_reports.each do |abrt_report| %>
-    <tr>
-      <td> <%= link_to (_("%s ago") % time_ago_in_words(abrt_report.reported_at.getlocal)), abrt_report %></td>
-      <td> <%= abrt_report.count %> </td>
-      <td> <%= format_reason abrt_report.reason %> </td>
-    </tr>
-    <% end %>
-  </table>
-  <%= link_to _("All bug reports for this host"), host_abrt_reports_path(@host), :class => "btn btn-default" %>
-<% else %>
-  <div class="alert alert-success"> <%= _("No bug reports for this host in the last month") %> </div>
-<% end %>
+  <% if @host.recent_abrt_reports.any? %>
+    <table class="table table-bordered table-striped">
+      <tr>
+        <th>Reported</th>
+        <th>#</th>
+        <th>Reason</th>
+      </tr>
+      <% @host.recent_abrt_reports.each do |abrt_report| %>
+        <tr>
+          <td> <%= link_to (_('%s ago') % time_ago_in_words(abrt_report.reported_at.getlocal)), abrt_report %></td>
+          <td> <%= abrt_report.count %> </td>
+          <td> <%= format_reason abrt_report.reason %> </td>
+        </tr>
+      <% end %>
+    </table>
+    <%= link_to _('All bug reports for this host'), host_abrt_reports_path(@host), :class => 'btn btn-default' %>
+  <% else %>
+    <div class="alert alert-success"> <%= _('No bug reports for this host in the last month') %> </div>
+  <% end %>
 </div>

data/app/views/abrt_reports/_list.html.erb

@@ -3,28 +3,28 @@
     <% unless params[:host_id] %>
       <th><%= sort :host %></th>
     <% end %>
-    <th><%= sort :reported, :as => _("Reported") %></th>
-    <th class="col-md-1"><%= sort :reason, :as => _("Reason") %></th>
-    <th class="col-md-1"><%= _("Repeated") %></th>
-    <th class="col-md-1"><%= _("Forwarded") %></th>
+    <th><%= sort :reported, :as => _('Reported') %></th>
+    <th class="col-md-1"><%= sort :reason, :as => _('Reason') %></th>
+    <th class="col-md-1"><%= _('Repeated') %></th>
+    <th class="col-md-1"><%= _('Forwarded') %></th>
     <th class="col-md-1"></th>
   </tr>
-  <% for abrt_report in @abrt_reports %>
+  <% @abrt_reports.each do |abrt_report| %>
     <tr>
       <% if params[:host_id].nil? %>
-        <% if abrt_report.host.nil? %>
-            <td></td>
-        <% else %>
-            <td><%= link_to abrt_report.host, host_abrt_reports_path(abrt_report.host) %></td>
-        <% end %>
+        <td>
+          <% if abrt_report.host %>
+            <%= link_to abrt_report.host, host_abrt_reports_path(abrt_report.host) %>
+          <% end %>
+        </td>
       <% end %>
-      <td> <%= link_to(_("%s ago") % time_ago_in_words(abrt_report.reported_at.getlocal), abrt_report_path(abrt_report)) %> </td>
+      <td> <%= link_to(_('%s ago') % time_ago_in_words(abrt_report.reported_at.getlocal), abrt_report_path(abrt_report)) %> </td>
       <td> <%= format_reason abrt_report.reason %> </td>
       <td> <%= abrt_report.count %> </td>
       <td> <%= checked_icon !abrt_report.forwarded_at.nil? %> </td>
       <td align="right">
         <%= display_delete_if_authorized hash_for_abrt_report_path(:id => abrt_report.id).merge(:auth_object => abrt_report, :authorizer => authorizer),
-                                         :confirm => _("Delete bug report for %s?") % abrt_report.host.try(:name) %>
+                                         :confirm => _('Delete bug report for %s?') % abrt_report.host.try(:name) %>
       </td>
     </tr>
   <% end %>

data/app/views/abrt_reports/_show_response.html.erb

@@ -7,9 +7,14 @@
 <div class="row">
   <div class="col-md-12">
     <table class="table table-bordered table-striped">
-      <tr><th><%= _('Known problem') %></th><td><%= @abrt_report.response_known %></td></tr>
-      <tr><th><%= _('Server hash') %></th><td><%= @abrt_report.response_bthash %></td></tr>
-      <tr><th><%= _('Message') %></th><td><%= simple_format_if_multiline @abrt_report.response_message %></td></tr>
+      <tr>
+        <th><%= _('Known problem') %></th>
+        <td><%= @abrt_report.response_known %></td>
+      </tr>
+      <tr>
+        <th><%= _('Message') %></th>
+        <td><%= simple_format_if_multiline @abrt_report.response_message %></td>
+      </tr>
     </table>
   </div>
 </div>
@@ -19,7 +24,7 @@
     <h3>Reported to</h3>
     <% if !@abrt_report.abrt_report_response_destinations.empty? %>
       <table class="table table-bordered table-striped">
-        <% for destination in @abrt_report.abrt_report_response_destinations %>
+        <% @abrt_report.abrt_report_response_destinations.each do |destination| %>
           <tr>
             <% if destination.desttype == 'url' %>
               <td><a href="<%= destination.value %>"><%= destination.reporter %></a></td>
@@ -39,7 +44,7 @@
     <h3>Solutions</h3>
     <% if !@abrt_report.abrt_report_response_solutions.empty? %>
       <table class="table table-bordered table-striped">
-        <% for solution in @abrt_report.abrt_report_response_solutions %>
+        <% @abrt_report.abrt_report_response_solutions.each do |solution| %>
           <tr>
             <% if solution.url %>
               <td><a href="<%= solution.url %>"><%= solution.cause %></a></td>

data/app/views/abrt_reports/show.html.erb

@@ -1,22 +1,30 @@
 <% title @abrt_report.host.to_s %>
 
-<%= title_actions link_to(_("Host details"), @abrt_report.host),
-  link_to(_("Other reports for this host"), host_abrt_reports_path(@abrt_report.host)),
-  link_to(_("Forward report"),
-          forward_abrt_report_path(@abrt_report),
-          :class => "btn btn-success",
-          :method => :post,
-          :confirm => @abrt_report.forwarded_at ? _("The report has already been sent to the ABRT server. Send again and overwrite the previous response?") : nil),
-  display_delete_if_authorized(hash_for_abrt_report_path(:id => @abrt_report), :class => "btn btn-danger")
+<%= title_actions link_to(_('Host details'), @abrt_report.host),
+                  link_to(_('Other reports for this host'), host_abrt_reports_path(@abrt_report.host)),
+                  link_to(_('Forward report'),
+                          forward_abrt_report_path(@abrt_report),
+                          :class   => 'btn btn-success',
+                          :method  => :post,
+                          :confirm => @abrt_report.forwarded_at ? _('The report has already been sent to the ABRT server. Send again and overwrite the previous response?') : nil),
+                  display_delete_if_authorized(hash_for_abrt_report_path(:id => @abrt_report), :class => 'btn btn-danger')
 %>
 
 <div class="row">
   <div class="col-md-12">
     <table class="table table-bordered table-striped">
-      <tr><th><%= _('Reason') %></th><td><%= format_reason @abrt_report.reason %></td></tr>
-      <tr><th><%= _('First reported') %></th><td><%= @abrt_report.reported_at.getlocal %></td></tr>
-      <tr><th><%= _('Repeated') %></th><td><%= @abrt_report.count %></td></tr>
-      <tr><th><%= _('Duplication hash') %></th><td><%= @abrt_report.duphash %></td></tr>
+      <tr>
+        <th><%= _('Reason') %></th>
+        <td><%= format_reason @abrt_report.reason %></td>
+      </tr>
+      <tr>
+        <th><%= _('First reported') %></th>
+        <td><%= @abrt_report.reported_at.getlocal %></td>
+      </tr>
+      <tr>
+        <th><%= _('Repeated') %></th>
+        <td><%= @abrt_report.count %></td>
+      </tr>
     </table>
   </div>
 </div>

data/lib/foreman_abrt/engine.rb

@@ -14,7 +14,7 @@ module ForemanAbrt
     end
 
     # Add any db migrations
-    initializer "foreman_abrt.load_app_instance_data" do |app|
+    initializer 'foreman_abrt.load_app_instance_data' do |app|
       app.config.paths['db/migrate'] += ForemanAbrt::Engine.paths['db/migrate'].existent
     end
 
@@ -31,7 +31,7 @@ module ForemanAbrt
         end
 
         # Add a new role
-        role "ABRT reports handling", [:view_abrt_reports, :destroy_abrt_reports, :upload_abrt_reports, :forward_abrt_reports]
+        role 'ABRT reports handling', [:view_abrt_reports, :destroy_abrt_reports, :upload_abrt_reports, :forward_abrt_reports]
 
         #add menu entry
         menu :top_menu, :template,

data/lib/foreman_abrt/version.rb

@@ -1,3 +1,3 @@
 module ForemanAbrt
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_abrt
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Martin Milata
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-30 00:00:00.000000000 Z
+date: 2014-11-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: deface
@@ -32,44 +32,45 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- app/controllers/concerns/foreman_abrt/dashboard_controller_extensions.rb
-- app/controllers/api/v2/abrt_reports_controller.rb
-- app/controllers/abrt_reports_controller.rb
-- app/assets/javascripts/abrt_reports.js
-- app/assets/stylesheets/abrt_reports.css
-- app/models/concerns/foreman_abrt/host_extensions.rb
-- app/models/abrt_report.rb
-- app/models/abrt_report_response_destination.rb
-- app/models/abrt_report_response_solution.rb
-- app/models/setting/abrt.rb
-- app/helpers/abrt_reports_helper.rb
 - app/overrides/add_host_bug_report_tab.rb
+- app/assets/stylesheets/abrt_reports.css
+- app/assets/javascripts/abrt_reports.js
 - app/views/abrt_reports/_host_tab_pane.html.erb
 - app/views/abrt_reports/_host_tab.html.erb
-- app/views/abrt_reports/_show_response.html.erb
 - app/views/abrt_reports/index.html.erb
+- app/views/abrt_reports/_show_response.html.erb
 - app/views/abrt_reports/_list.html.erb
 - app/views/abrt_reports/show.html.erb
 - app/views/dashboard/_foreman_abrt_widget.html.erb
+- app/helpers/abrt_reports_helper.rb
+- app/controllers/concerns/foreman_abrt/dashboard_controller_extensions.rb
+- app/controllers/api/v2/abrt_reports_controller.rb
+- app/controllers/abrt_reports_controller.rb
+- app/models/setting/abrt.rb
+- app/models/concerns/foreman_abrt/host_extensions.rb
+- app/models/abrt_report.rb
+- app/models/abrt_report_response_destination.rb
+- app/models/abrt_report_response_solution.rb
 - config/routes.rb
 - db/migrate/20140414095631_create_abrt_reports.rb
 - db/migrate/20140909170102_abrt_reports_drop_timestamps.rb
-- lib/foreman_abrt/version.rb
 - lib/foreman_abrt/engine.rb
+- lib/foreman_abrt/version.rb
 - lib/foreman_abrt.rb
 - lib/tasks/foreman_abrt_tasks.rake
 - LICENSE
 - Rakefile
 - README.md
-- test/fixtures/abrt_reports.yml
-- test/test_plugin_helper.rb
-- test/functional/abrt_reports_controller_test.rb
 - test/unit/abrt_report_test.rb
-- test/unit/foreman_abrt_test.rb
 - test/unit/helpers/abrt_reports_helper_test.rb
+- test/unit/foreman_abrt_test.rb
+- test/functional/abrt_reports_controller_test.rb
+- test/fixtures/abrt_reports.yml
+- test/test_plugin_helper.rb
 - test/factories/foreman_abrt_factories.rb
 homepage: http://github.com/abrt/foreman_abrt
-licenses: []
+licenses:
+- GPL-3.0
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -92,10 +93,10 @@ signing_key:
 specification_version: 4
 summary: Display reports from Automatic Bug Reporting Tool
 test_files:
-- test/fixtures/abrt_reports.yml
-- test/test_plugin_helper.rb
-- test/functional/abrt_reports_controller_test.rb
 - test/unit/abrt_report_test.rb
-- test/unit/foreman_abrt_test.rb
 - test/unit/helpers/abrt_reports_helper_test.rb
+- test/unit/foreman_abrt_test.rb
+- test/functional/abrt_reports_controller_test.rb
+- test/fixtures/abrt_reports.yml
+- test/test_plugin_helper.rb
 - test/factories/foreman_abrt_factories.rb