From eb7a9c5d8263d5e57b6e42f6a9ad303ce77a0e08 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20Lobato=20Garc=C3=ADa?= Date: Fri, 8 Apr 2016 20:53:44 +0200 Subject: [PATCH] Allow options on servers.all - as per the 'fog standard' Problem: All other fog 'Servers' object allow you to pass an argument to `all`. The fact this one doesn't, means writing generic code for fog providers that relies on `.all(opts)` isn't possible. Solution: fog's goal is to facilitate generic code for multiple compute resources, so I suggest this method takes the same number of arguments as the rest. --- lib/fog/azure/models/compute/servers.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/fog/azure/models/compute/servers.rb b/lib/fog/azure/models/compute/servers.rb index 9885dfc..56c2554 100644 --- a/lib/fog/azure/models/compute/servers.rb +++ b/lib/fog/azure/models/compute/servers.rb @@ -28,7 +28,7 @@ class Azure class Servers < Fog::Collection model Fog::Compute::Azure::Server - def all() + def all(options = {}) servers = [] service.list_virtual_machines.each do |vm| hash = {}