fixes #6919 / BZ 1113588 - host - show validation error if no puppet environment

This commit contains a small change so that if the user doesn't provide
a puppet environment, they'll see the validation error sent by the server.

E.g. Puppet Environment: [] can't be blank

The code change is to ensure that the html rendered includes
'for="environment_id"', which is used to support displaying
the error associated with that attribute.
fixes #6330 - using pulp & pulp_node features of smart proxy

* to determine the default smart proxy
* to determine whether to show the lifecycle env selection screen
* to determine which proxies to who under the content source selection
Fixes #6967: Add the correct location of mongo, and collect all log files
Fixes #6944/bz1106434 - Fixed Repodiscovery product validation

Added code in the repo discovery page to validate new product.
Previously product errors like blank name invalid label etc would not
show up on the screen. This commit tries to address that.
Refs #6867,BZ1038323 - Show katello-agent status for systems
Fixes #6993: Allow repo-discovery through a proxy.

Note that the Anemone gem allows the passing of proxy_host and proxy_port
by default but does not currently support proxy user and password.
In order to get around this, a monkey patch is applied to anemone
to allow passing user and port.
fixes #6992 - ensure rhel 5 repos can be used by el5 clients

We can not publish the repos prior to syncing as this will cause
pulp to set the checksum on the repos to sha256.  See https://bugzilla.redhat.com/show_bug.cgi?id=1127793
We also need to pull the checksum after sync and add it to the repo
to ensure that clones use the same checksum
Fixes #6828/BZ1104636: show correct version of puppet module. …

The latest version of puppet modules was being incorrectly
calculated, based on an incorrect assumption that the last
item in the array was the latest version.  This commit
explicitly sorts the puppet modules by the version such that
the latest version is correct.

http://projects.theforeman.org/issues/6828
https://bugzilla.redhat.com/show_bug.cgi?id=1104636
fixes #7005 / BZ 1122736 - content host - change registration behavior with foreman host

This commit alters the behavior of registering a content host
(with or without an activation key) that is to be associated
with a foreman host to do the following:

1. identify the foreman host that this content host will be
   associated with by both name and mac address (previously
   it was only by name)
2. if the foreman host identified is already associated with a
   content host, delete it and then create/associate a new one.
   (This will avoid there becoming duplicate content hosts with
   the same name.)
fixes #6970 - content host bulk actions - fix 2 regressions

Stumbled across a couple of issues that need fixing.

1. unable to perform bulk actions (via systems_bulk_actions_controller)
   due to permission denied errors
2. unable to retrieve applicable errata
fixes #7019 / BZ 1128473 - content search - change to eliminate errors with ambiguous 'id'

This commit is to update a few queries that use 'pluck' to be explicit
on the table that the attribute (id) is being plucked from.  Without them
the user can experience errors like:

Processing by Katello::ContentSearchController#packages as */*
  Parameters: {"repos"=>{"autocomplete"=>[{"id"=>"13", "name"=>"samrepo"}]}, "packages"=>{"search"=>""}, "content_type"=>"packages", "mode"=>"all", "environments"=>[2], "content_search"=>{"repos"=>{"autocomplete"=>[{"id"=>"13", "name"=>"samrepo"}]}, "packages"=>{"search"=>""}, "content_type"=>"packages", "mode"=>"all", "environments"=>[2]}}
PGError: ERROR:  column reference "id" is ambiguous
LINE 1: SELECT id FROM "katello_repositories" INNER JOIN "katello_co...
               ^
: SELECT id FROM "katello_repositories" INNER JOIN "katello_content_view_versions" ON "katello_content_view_versions"."id" = "katello_repositories"."content_view_version_id" WHERE "katello_content_view_versions"."content_view_id" IN (1, 2, 3, 4, 6) ORDER BY katello_repositories.name ASC (ActiveRecord::StatementInvalid)
Fixes #6175 - Handle new status icon that was added

In the following commit we added a new status icon but didn't check to make
sure that it was properly refreshed so #6175 was reopened to deal with this.

https://github.com/Katello/katello/commit/7e86b53e#diff-8d1fe504fb4c8c0ff058edda520089f9R17
Fixes #7024 - fixes delete cv from environment

In the cv api, this fixes the guard that checks that cv is in environment prior to removal.
fixes #5190 - adding annotations to deprecated routes, BZ 1102264
Fixes #7025 - replace default_environment in sys

In System api, default_environment is still being retrieved from the
User (which is no longer being defined there). This fix obtains the
environment from the user's default_organization.
Fixes #7026/BZ1128471: fix error on updating errata date range filter.

There was an error when updating an errata data range filter.  This
commit fixes the error as well as displays error messages on the form
in the event of a legitimate error.

http://projects.theforeman.org/issues/7026
https://bugzilla.redhat.com/show_bug.cgi?id=1128471
Merge pull request #4527 from bbuckingham/issue-6919

fixes #6919 / BZ 1113588 - host - show validation error if no puppet environment
Merge pull request #4560 from dustint-rh/typo_in_delete_cv_from_env

Fixes #7024 - fixes delete cv from environment
Merge pull request #4484 from komidore64/rmi5190-deprecated-systems-calls

Systems/Hosts: The Katello Systems API should exist but be documented as unsupported.
Merge pull request #4562 from waldenraines/7026

Fixes #7026/BZ1128471: fix error on updating errata date range filter.