Fixes & improvements for servers:yaml rake task (#1056)

* use string keys over symbols in hash produced by servers:yaml task, such that YAML.safe_load consumes it correctly (with default arguments), and for better human write-/readability

* fix 'enabled' value in servers:yaml output

* fix server.enabled? in servers:yaml[verbose]

* fix logic mistake which prevented state 'cordoned' to be outputted in output of servers:yaml[verbose]

lib/server_sync.rb

@@ -171,15 +171,15 @@ def self.sync(servers, mode = 'cordon', dryrun = false)
   def self.dump(verbose)
     Server.all.to_h do |server|
       info = {
-        url: server.url,
-        secret: server.secret,
-        load_multiplier: server.load_multiplier.to_f || 1.0,
-        enabled: server.enabled,
+        "url" => server.url,
+        "secret" => server.secret,
+        "load_multiplier" => server.load_multiplier.to_f || 1.0,
+        "enabled" => server.enabled?,
       }
       if verbose
-        info[:state] = server.state.presence || server.enabled ? 'enabled' : 'disabled'
-        info[:load] = server.load.presence || -1.0
-        info[:online] = server.online
+        info["state"] = server.state.presence || (server.enabled? ? 'enabled' : 'disabled')
+        info["load"] = server.load.presence || -1.0
+        info["online"] = server.online
       end
       [server.id, info]
     end

lib/tasks/servers.rake

@@ -200,7 +200,7 @@ namespace :servers do
 
   desc 'Return a yaml compatible with servers:sync'
   task :yaml, [:verbose] => :environment do |_t, args|
-    puts({ servers: ServerSync.dump(!!args.verbose) }.to_yaml)
+    puts({ "servers" => ServerSync.dump(!!args.verbose) }.to_yaml)
   end
 
   desc('List all meetings running in specific BigBlueButton servers')