Chris Doherty comments

Results 8 comments of


                                            Chris Doherty

Need support for AWS tests

Having `be_idempotent` run the code instead of `expect` is super goofy, for the record.

Bug using machine option `chef_server` on a `machine_batch`

@afiune Did you get this sorted out? Okay to close?

Docs around expect_recipe() are probably wrong as of Chef 12.4

The [same specs](https://github.com/chef/cheffish/blob/master/spec%2Fintegration%2Frspec%2Fconverge_spec.rb#L73) for `expect_recipe` pass for 12.3 and 12.4, so we'll have to look at that and add stuff if there's really a discrepancy.

Expose new :from_machine attribute on machine_image

In this case, `machine_image` will converge by checking for the named AMI before trying to create it.

Resizing instances

@gmiranda23 That's my understanding of the situation, yes. If it were implemented conservatively enough, I don't think we'd object on principle.

AWS_* env var overrides

To consolidate this, here's the comment from the Fog file: ``` # Order of operations: # compute_options[:aws_access_key_id] / compute_options[:aws_secret_access_key] / compute_options[:aws_security_token] / compute_options[:region] # compute_options[:aws_profile] # ENV['AWS_ACCESS_KEY_ID'] / ENV['AWS_SECRET_ACCESS_KEY'] /...

Extending support for aws_auto_scaling_group

No one is working on it that we know of, so please feel free!

Ugly output message during machine resource convergence

The fix for this should probably be ported to `chef-provisioning-fog`, which has the same error message [here](https://github.com/chef/chef-provisioning-fog/blob/master/lib%2Fchef%2Fprovisioning%2Ffog_driver%2Fdriver.rb#L677).