Set a dependency on collector

Its better to restart collector after the ceilo-upgrade is done
so that the dispatchers are loaded and communicate with other
services like gnocchi and panko and dispatch data. Otherwise
user has to retsrat collector sometimes.

Closes-bug: #1680126

Change-Id: Ib23913d698199ca87658e28e99acb5d56cebd858

elements/puppet-stack-config/puppet-stack-config.pp

@@ -204,8 +204,10 @@ if str2bool(hiera('enable_telemetry', true)) {
   }
 
 # Ensure all endpoint exists and only then run the upgrade.
+# ensure we restart ceilometer collector as well
   Keystone::Resource::Service_identity<||> ->
-  Openstacklib::Service_validation['gnocchi-status'] -> Exec['ceilo-gnocchi-upgrade']
+  Openstacklib::Service_validation['gnocchi-status'] ->
+  Exec['ceilo-gnocchi-upgrade'] ~> Service['ceilometer-collector']
 
   Cron <| title == 'ceilometer-expirer' |> { command =>
     "sleep $((\$(od -A n -t d -N 3 /dev/urandom) % 86400)) && ${::ceilometer::params::expirer_command}" }

releasenotes/notes/restart-collector-b043489fcdf1e9c7.yaml

@@ -0,0 +1,4 @@
+---
+fixes:
+  - Add a dependency to restart collector after other services are
+    up and ceilometer upgrade is complete.