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.
Added missing 'forest_delete' route

https://bugzilla.redhat.com/show_bug.cgi?id=1163382
https://bugzilla.redhat.com/show_bug.cgi?id=1163384
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.
Merge pull request #1046 from h-kataria/added_missing_route_forest_delete

Added missing 'forest_delete' route
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
add `database_version` to the connection object

This method was implemented in the Rails fork, but I think it's safe to
move over to the application as a monkey patch.  We'll extract it to a
gem later.

This implementation differs from the original in that it doesn't cache
the value of the database version.  I think this is OK for two reasons:

1. It's not a hotspot (that I can tell) so there shouldn't be a perf
impact.

2. The cached version could be wrong if we disconnect then reconnect.
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.