adjustments various

Author: cordoval

Aws/Ec2Command.php

@@ -40,12 +40,12 @@ protected function getClient()
     {
         $credentials = $this->getCredentials();
 
-        $client = Ec2Client::factory(array(
-            'key'    => $credentials['aws_api_key'],
-            'secret' => $credentials['aws_api_secret'],
-            'region' => $credentials['aws_region'],
-        ));
-
-        return $client;
+        return Ec2Client::factory(
+            array(
+                'key'    => $credentials['aws_api_key'],
+                'secret' => $credentials['aws_api_secret'],
+                'region' => $credentials['aws_region'],
+            )
+        );
     }
 }

Command/Aws/Ec2/CopyImageCommand.php

@@ -50,7 +50,7 @@ protected function configure()
             ->addOption('Description', 'description', InputOption::VALUE_OPTIONAL, 'A description for the new AMI in the destination region')
             ->addOption('ClientToken', 'clienttoken', InputOption::VALUE_OPTIONAL, 'Unique, case-sensitive identifier you provide to ensure the idempotency of the request')
             ->addOption('DryRun', 'dryrun', InputOption::VALUE_NONE, null)
-            ->addOption('AmiName', 'aminame', InputOption::VALUE_NONE, 'Use AMI name instead if ID')
+            ->addOption('AmiName', 'aminame', InputOption::VALUE_NONE, 'Use AMI name instead of ID')
         ;
     }
 
@@ -65,12 +65,29 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
         if ($options['AmiName']) {
             $name = $options['SourceImageId'];
-            $image = $client->describeImages(["Filters" => [["Name" => "name", "Values" => [$name]]]]); # TODO if more than one instance is returned, warn the user
-            $sourceImageId = $image['Images'][0]['ImageId'];
-            $options['SourceImageId'] = $sourceImageId;
+            $image = $client->describeImages(
+                [
+                    "Filters" => [
+                        [
+                            "Name" => "name",
+                            "Values" => [$name]
+                        ]
+                    ]
+                ]
+            );
+            $imageCollection = $image['Images'];
+            if (count($imageCollection) > 1) {
+                $output->writeln('<error>Know that the AMI name provided matched more than one image. Please be more specific to avoid copying the wrong image.</error>');
+
+                return self::COMMAND_FAILURE;
+            }
+
+            $options['SourceImageId'] = $imageCollection[0]['ImageId'];
         }
 
         $result = $client->copyImage($options);
-        $output->writeln($result->get('ImageId'));
+        $output->writeln(sprintf('<info>Successfully copied image %s.</info>', $result->get('ImageId')));
+
+        return self::COMMAND_SUCCESS;
     }
 }

Command/Aws/Ec2/CreateImageCommand.php

@@ -47,9 +47,10 @@ protected function configure()
             ->addArgument('Name', InputArgument::REQUIRED, 'A name for the new image.')
             ->addArgument('InstanceId', InputArgument::REQUIRED, 'The ID of the instance')
             ->addArgument('Description', InputArgument::OPTIONAL, 'A description for the new image.')
-            ->addOption('BlockDeviceMappings', 'mappings', InputOption::VALUE_OPTIONAL, 'Information about one or more block device mappings. Takes JSON')
+            ->addOption('BlockDeviceMappings', 'mappings', InputOption::VALUE_OPTIONAL, 'Information about one or more block device mappings. Takes JSON.')
             ->addOption('NoReboot', 'noreboot', InputOption::VALUE_NONE, 'Amazon EC2 will not shut down the instance before creating the image. Filesystem integrity is not guaranteed.')
-            ->addOption('DryRun', 'dryrun', InputOption::VALUE_NONE, null);
+            ->addOption('DryRun', 'dryrun', InputOption::VALUE_NONE, null)
+        ;
     }
 
     /**
@@ -62,8 +63,9 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $options['BlockDeviceMappings'] = json_decode($options['BlockDeviceMappings']);
 
         $client = $this->getClient();
-
         $result = $client->createImage($options);
-        $output->writeln($result->get('ImageId'));
+        $output->writeln('AMI Image created with id '.$result->get('ImageId'));
+
+        return self::COMMAND_SUCCESS;
     }
 }

Command/Aws/Ec2/DeregisterImageCommand.php

@@ -70,13 +70,13 @@ protected function execute(InputInterface $input, OutputInterface $output)
                 ]
             );
             $imageCollection = $image['Images'];
-            $imageId = $imageCollection[0]['ImageId'];
             if (count($imageCollection) > 1) {
                 $output->writeln('<error>Know that the AMI name provided matched more than one image. Please be more specific to avoid deregistering the wrong image.</error>');
 
                 return self::COMMAND_FAILURE;
             }
-            $options['ImageId'] = $imageId;
+
+            $options['ImageId'] = $imageCollection[0]['ImageId'];
         }
 
         $client->deregisterImage($options);