fixed formatting issue in catalog add/edit popup
Changes to show tag category only once for each base class.

- Changed TAG_CLASSES to be a hash and then use invert on it in order to use each model only once to add tag categories for them, having base class more than once in that array was causing duplicate entries in the pull down.
- Added spec test for model_details method to make sure values in the pull down are uniq.

https://bugzilla.redhat.com/show_bug.cgi?id=1160209
Add models for cloud orchestration
Cleanup credentials for RHN partial.
Cleanup email password partial.
Do not pass SMTP password to HTML form.
Do not pass DB password to HTML form.
Do not pass Amazon secrets and LDAP password to HTML form.
Do not pass infra provider passwords to HTML form.
Add conditional placeholders to HTML password fields.
Don't send up all_checked checkbox IDs upto server on onselect event.

- Made changes to JS method to not send up all_checked checkboxes on onselect event, this was causing JS error when there were lots of nodes in the tree, as a result of that transaction was not being sent upto server. Made changes to set/save C&U collection based upon checkbox that was changed and sent up. Removed code that was expecting all_checked checkboxes to come in with JS transaction.
- Changed code to only call Metric::Targets.perf_capture_always if any of the "Collect for All XXX" checkboxes values were changed and only for the one that's value was changed.
- Renamed set_perf_collection_object_list method to perf_capture_enabled_host_ids= changed it's caller to only send up list of host ids marked for collection for a cluster. Removed querying for hosts in controller.
- Removed code that was setting @temp[:cl_recs], it was not required to store Cluster Objects in @temp.
- Removed comment that no longer applies to the code.
- Added spec tests for perf_capture_enabled_host_ids= method to make sure it set/unsets performance capture on cluster and it's hosts.

https://bugzilla.redhat.com/show_bug.cgi?id=1153047
https://bugzilla.redhat.com/show_bug.cgi?id=1145799
We're leaving miq_ae_namespaces rows around by using :all.

We shouldn't use before(:all) unless we cleanup after ourselves and even then, it's not worth it most of the times.
Fixed saving on Input Parameters for Automate Method.

- Fixed to set/save default value for datatype of an Automate Method Input field to be 'string'.
- Fixed saving of input parameters when adding a new Method.

https://bugzilla.redhat.com/show_bug.cgi?id=1161761
https://bugzilla.redhat.com/show_bug.cgi?id=1161701
Delegate the purging of 6 month+ old metrics to the purger.

Fixes timeouts during server startup.

The existing purging works in rails console using an old migrated database:

VmdbMetric.count => 4696666
require 'workers/schedule_worker'
ScheduleWorker::Jobs.new.metric_purge_all_timer
simulate_queue_worker
VmdbMetric.count => 0

Fixes #1029
Merge pull request #1047 from jrafanie/dont_use_before_all_unless_you_cleanup_after_yourself

We're leaving miq_ae_namespaces rows around by using :all.
Remove duplicate connection logic when verifying default credentials
Merge pull request #1050 from jrafanie/let_purger_handle_purging_metrics

Delegate the purging of 6 month+ old metrics to the purger.
Avoid checking if "services" tenant is accessible

It appears that checking whether the "services" tenant is accessible in
Openstack causes a barrage of "401 Unauthorized" errors in the logs.  Based on
how we handle it in the OpenstackHandle, it does not appear to be a fatal error.

However, cleaning up this error will make diagnosing problems via the logs much
easier.

https://bugzilla.redhat.com/show_bug.cgi?id=1140174
Merge pull request #1052 from brandondunne/fix_ems_redhat_credential_validation

Remove duplicate connection logic when verifying default credentials
Merge pull request #1036 from epwinchell/automate_tree_div_styling

fixed formatting issue in catalog add/edit popup