I've got a few problems with this method, but they shouldn't be hard to fix :).

First, if you're assigning something, the usual idiom in ruby is to name the method ```group=``` rather than ```group``` (which would be for a getter method).

Second, I think your method is doing too much, by mapping from name to id.
I think what we should go for instead is a usage pattern that looks like this:

```ruby
group = cloudpassage.groups.all.detect{|g|g[:name]==group_name}
server.group = group.id
```