boxgrinder-build 0.10.1 → 0.10.2

data/lib/boxgrinder-build/plugins/delivery/s3/s3-plugin.rb

@@ -19,20 +19,17 @@
 require 'rubygems'
 require 'aws-sdk'
 require 'boxgrinder-build/plugins/base-plugin'
+require 'boxgrinder-build/plugins/delivery/s3/messages'
 require 'boxgrinder-build/helpers/package-helper'
 require 'boxgrinder-build/helpers/s3-helper'
 require 'boxgrinder-build/helpers/ec2-helper'
+require 'boxgrinder-build/helpers/banner-helper'
 
 module BoxGrinder
   class S3Plugin < BasePlugin
     plugin :type => :delivery, :name => :s3, :full_name => "Amazon Simple Storage Service (Amazon S3)", :types => [:s3, :cloudfront, :ami]
 
     def after_init
-      register_supported_os("fedora", ['13', '14', '15', '16'])
-      register_supported_os("centos", ['5', '6'])
-      register_supported_os("rhel", ['5', '6'])
-      register_supported_os("sl", ['5', '6'])
-
       @ami_build_dir = "#{@dir.base}/ami"
       @ami_manifest = "#{@ami_build_dir}/#{@appliance_config.name}.ec2.manifest.xml"
     end
@@ -43,6 +40,11 @@ module BoxGrinder
       set_default_config_value('ramdisk', false)
       set_default_config_value('path', '/')
       set_default_config_value('region', 'us-east-1')
+
+      set_default_config_value('block_device_mappings', {}) do |k, m, v|
+        EC2Helper::block_device_mappings_validator(k, m, v)
+      end
+      
       validate_plugin_config(['bucket', 'access_key', 'secret_access_key'], 'http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#S3_Delivery_Plugin')
 
       subtype(:ami) do
@@ -89,6 +91,8 @@ module BoxGrinder
         when :cloudfront
           upload_to_bucket(@previous_deliverables, :public_read)
         when :ami
+          @log.info Banner.message(S3::Messages::EPHEMERAL_MESSAGE)
+    
           ami_dir = ami_key(@appliance_config.name, @plugin_config['path'])
           ami_manifest_key = @s3helper.stub_s3obj(@bucket, "#{ami_dir}/#{@appliance_config.name}.ec2.manifest.xml")
 
@@ -191,7 +195,15 @@ module BoxGrinder
       if ami = ami_by_manifest_key(ami_manifest_key)
         @log.info "Image for #{@appliance_config.name} is already registered under id: #{ami.id} (region: #{@plugin_config['region']})."
       else
-        ami = @ec2.images.create(:image_location =>  "#{@plugin_config['bucket']}/#{ami_manifest_key.key}")
+        optmap = { :image_location =>  "#{@plugin_config['bucket']}/#{ami_manifest_key.key}" }
+        
+        unless @plugin_config['block_device_mappings'].empty?
+          optmap.merge!(:block_device_mappings => @plugin_config['block_device_mappings']) 
+        end
+
+        @log.debug("Options map: #{optmap.inspect}")
+
+        ami = @ec2.images.create(optmap)
         @ec2helper.wait_for_image_state(:available, ami)
         @log.info "Image for #{@appliance_config.name} successfully registered under id: #{ami.id} (region: #{@plugin_config['region']})."
       end

data/lib/boxgrinder-build/plugins/os/fedora/fedora-plugin.rb

@@ -20,11 +20,17 @@ require 'boxgrinder-build/plugins/os/rpm-based/rpm-based-os-plugin'
 
 module BoxGrinder
   class FedoraPlugin < RPMBasedOSPlugin
-    plugin :type => :os, :name => :fedora, :full_name => "Fedora", :versions => ["13", "14", "15", "16", "rawhide"], :require_root => true
+    SUPPORTED_VERSIONS = ["13", "14", "15", "16", "17", "rawhide"]
+
+    plugin(:type => :os, 
+           :name => :fedora, 
+           :full_name => "Fedora", 
+           :versions => SUPPORTED_VERSIONS, 
+           :require_root => true)
 
     def after_init
       super
-      register_supported_os('fedora', ["13", "14", "15", "16", "rawhide"])
+      register_supported_os('fedora', SUPPORTED_VERSIONS)
       set_default_config_value('PAE', true)
     end
 
@@ -48,7 +54,7 @@ module BoxGrinder
         if @appliance_config.os.version >= "15"
           disable_biosdevname(guestfs)
           change_runlevel(guestfs)
-          disable_netfs(guestfs)
+          disable_netfs(guestfs) if @appliance_config.os.version < "17"
           link_mtab(guestfs)
         end
       end

data/lib/boxgrinder-build/plugins/platform/ec2/ec2-plugin.rb

@@ -27,7 +27,7 @@ module BoxGrinder
     def after_init
       register_deliverable(:disk => "#{@appliance_config.name}.ec2")
 
-      register_supported_os('fedora', ['13', '14', '15', '16'])
+      register_supported_os('fedora', ['13', '14', '15', '16', '17'])
       register_supported_os('centos', ['5', '6'])
       register_supported_os('sl', ['5', '6'])
       register_supported_os('rhel', ['5', '6'])
@@ -42,6 +42,11 @@ module BoxGrinder
 
       @image_helper.customize([@previous_deliverables.disk, @deliverables.disk], :automount => false) do |guestfs, guestfs_helper|
         @image_helper.sync_filesystem(guestfs, guestfs_helper)
+        
+        # TODO is this really needed?
+        @log.debug "Uploading '/etc/resolv.conf'..."
+        guestfs.upload("/etc/resolv.conf", "/etc/resolv.conf")
+        @log.debug "'/etc/resolv.conf' uploaded."
 
         if (@appliance_config.os.name == 'rhel' or @appliance_config.os.name == 'centos') and @appliance_config.os.version == '5'
           # Remove normal kernel
@@ -52,11 +57,6 @@ module BoxGrinder
           @linux_helper.recreate_kernel_image(guestfs, ['xenblk', 'xennet'])
         end
 
-        # TODO is this really needed?
-        @log.debug "Uploading '/etc/resolv.conf'..."
-        guestfs.upload("/etc/resolv.conf", "/etc/resolv.conf")
-        @log.debug "'/etc/resolv.conf' uploaded."
-
         create_devices(guestfs)
 
         guestfs.mkdir("/data") if @appliance_config.is64bit?
@@ -136,7 +136,9 @@ module BoxGrinder
       menu_lst << menu_lst_data
       menu_lst.flush
 
-      guestfs.upload(menu_lst.path, "/boot/grub/menu.lst")
+      menu_d = '/boot/grub'
+      guestfs.mkdir_p(menu_d) if guestfs.exists(menu_d) == 0
+      guestfs.upload(menu_lst.path, '/boot/grub/menu.lst')
 
       menu_lst.close
       @log.debug "'/boot/grub/menu.lst' file uploaded."

data/lib/boxgrinder-build/plugins/platform/ec2/src/fstab_32bit

@@ -1,6 +1,4 @@
 LABEL=79d3d2d4    /         #FILESYSTEM_TYPE#    defaults         1 1
-/dev/#DISK_DEVICE_PREFIX#da2  /mnt      ext3    defaults         1 2
-/dev/#DISK_DEVICE_PREFIX#da3  swap      swap    defaults         0 0
 none       /dev/pts  devpts  gid=5,mode=620   0 0
 none       /dev/shm  tmpfs   defaults         0 0
 none       /proc     proc    defaults         0 0

data/lib/boxgrinder-build/plugins/platform/ec2/src/fstab_64bit

@@ -1,6 +1,4 @@
 LABEL=79d3d2d4    /         #FILESYSTEM_TYPE#    defaults         1 1
-/dev/#DISK_DEVICE_PREFIX#db   /mnt      ext3    defaults         0 0
-/dev/#DISK_DEVICE_PREFIX#dc   /data     ext3    defaults         0 0
 none       /dev/pts  devpts  gid=5,mode=620   0 0
 none       /dev/shm  tmpfs   defaults         0 0
 none       /proc     proc    defaults         0 0

data/rubygem-boxgrinder-build.spec

@@ -1,31 +1,42 @@
-%global gemdir %(ruby -rubygems -e 'puts Gem::dir' 2>/dev/null)
-%global gemname boxgrinder-build
-%global geminstdir %{gemdir}/gems/%{gemname}-%{version}
+%global gem_name boxgrinder-build
+
+%{!?gem_dir: %global gem_dir %(ruby -rubygems -e 'puts Gem::dir' 2>/dev/null)}
+%{!?gem_instdir: %global gem_instdir %{gem_dir}/gems/%{gem_name}-%{version}}
+
+%if 0%{?fedora} >= 17
+%global rubyabi 1.9.1
+%else
 %global rubyabi 1.8
+%endif
 
-Summary: A tool for creating appliances from simple plain text files
-Name: rubygem-%{gemname}
-Version: 0.10.1
-Release: 1%{?dist}
-Group: Development/Languages
-License: LGPLv3+
-URL: http://boxgrinder.org/
-Source0: http://rubygems.org/gems/%{gemname}-%{version}.gem
+Summary:     A tool for creating appliances from simple plain text files
+Name:        rubygem-%{gem_name}
+Version:     0.10.2
+Release:     1%{?dist}
+Group:       Development/Languages
+License:     LGPLv3+
+URL:         http://boxgrinder.org/
+Source0:     http://rubygems.org/gems/%{gem_name}-%{version}.gem
 
 Requires: ruby(abi) = %{rubyabi}
-Requires: rubygem(boxgrinder-core) >= 0.3.11
+Requires: rubygem(boxgrinder-core) >= 0.3.12
 Requires: rubygem(boxgrinder-core) < 0.4.0
 Requires: ruby-libguestfs
 
 BuildArch: noarch
 
+BuildRequires: rubygems-devel
 BuildRequires: rubygem(rake)
-BuildRequires: rubygem(boxgrinder-core) >= 0.3.11
+BuildRequires: rubygem(boxgrinder-core) >= 0.3.12
 BuildRequires: rubygem(boxgrinder-core) < 0.4.0
 BuildRequires: rubygem(echoe)
 BuildRequires: ruby-libguestfs
-# Use rspec-core until rspec are migrated to RSpec 2.x
+
+%if 0%{?fedora} >= 17
+BuildRequires: rubygem(rspec)
+%else
 BuildRequires: rubygem(rspec-core)
+%endif
 
 # AWS
 Requires: euca2ools >= 1.3.1-4
@@ -52,14 +63,15 @@ BuildRequires: rubygem(nokogiri)
 BuildRequires: rubygem(builder)
 
 # RPM-BASED
-Requires: appliance-tools
+Requires: appliance-tools >= 006.1-1
 Requires: yum-utils
 
-#Elastichosts
-BuildRequires: rubygem(rest-client)
+# ElasticHosts
 Requires: rubygem(rest-client)
 
-Provides: rubygem(%{gemname}) = %{version}
+BuildRequires: rubygem(rest-client)
+
+Provides: rubygem(%{gem_name}) = %{version}
 
 Obsoletes: rubygem(boxgrinder-build-ebs-delivery-plugin) < 0.0.4-2
 Obsoletes: rubygem(boxgrinder-build-s3-delivery-plugin) < 0.0.6-1
@@ -96,53 +108,57 @@ Documentation for %{name}
 %build
 
 %install
-rm -rf %{buildroot}
-rm -rf %{_builddir}%{gemdir}
+rm -rf %{_builddir}%{gem_dir}
 
-mkdir -p %{_builddir}%{gemdir}
+mkdir -p %{_builddir}%{gem_dir}
 mkdir -p %{buildroot}/%{_bindir}
-mkdir -p %{buildroot}/%{gemdir}
+mkdir -p %{buildroot}/%{gem_dir}
 
-/usr/bin/gem install --local --install-dir %{_builddir}%{gemdir} \
+/usr/bin/gem install --local --install-dir %{_builddir}%{gem_dir} \
             --force --rdoc %{SOURCE0}
-mv %{_builddir}%{gemdir}/bin/* %{buildroot}/%{_bindir}
-find %{_builddir}%{geminstdir}/bin -type f | xargs chmod a+x
-rm -rf %{_builddir}/%{geminstdir}/integ/packages/*.rpm
-cp -r %{_builddir}%{gemdir}/* %{buildroot}/%{gemdir}
+mv %{_builddir}%{gem_dir}/bin/* %{buildroot}/%{_bindir}
+find %{_builddir}%{gem_instdir}/bin -type f | xargs chmod a+x
+rm -rf %{_builddir}/%{gem_instdir}/integ/packages/*.rpm
+cp -r %{_builddir}%{gem_dir}/* %{buildroot}/%{gem_dir}
 
 install -d -m 755 %{buildroot}/%{_sysconfdir}/bash_completion.d
-mv %{buildroot}/%{geminstdir}/bash_completion %{buildroot}/%{_sysconfdir}/bash_completion.d/%{name}
+mv %{buildroot}/%{gem_instdir}/bash_completion %{buildroot}/%{_sysconfdir}/bash_completion.d/%{name}
+
+chmod +x %{buildroot}/%{gem_instdir}/lib/boxgrinder-build/helpers/qemu.wrapper
 
 %check
-pushd %{_builddir}/%{geminstdir}
-rake spec
+pushd %{_builddir}/%{gem_instdir}
+rspec -r spec_helper -r boxgrinder-core -I. -P 'spec/**/*-spec.rb'
 popd
 
 %files
-%defattr(-, root, root, -)
 %{_bindir}/boxgrinder-build
 %{_sysconfdir}/bash_completion.d/%{name}
-%dir %{geminstdir}
-%{geminstdir}/bin
-%{geminstdir}/lib
-%doc %{geminstdir}/CHANGELOG
-%doc %{geminstdir}/LICENSE
-%doc %{geminstdir}/README.md
-%doc %{geminstdir}/Manifest
-%attr(755, root, root) %{geminstdir}/lib/boxgrinder-build/helpers/qemu.wrapper
-%{gemdir}/cache/%{gemname}-%{version}.gem
-%{gemdir}/specifications/%{gemname}-%{version}.gemspec
+%dir %{gem_instdir}
+%{gem_instdir}/bin
+%{gem_libdir}
+%doc %{gem_instdir}/CHANGELOG
+%doc %{gem_instdir}/LICENSE
+%doc %{gem_instdir}/README.md
+%doc %{gem_instdir}/Manifest
+%{gem_cache}
+%{gem_spec}
 
 %files doc
-%defattr(-, root, root, -)
-%{geminstdir}/spec
-%{geminstdir}/integ
-%{geminstdir}/Rakefile
-%{geminstdir}/rubygem-%{gemname}.spec
-%{geminstdir}/%{gemname}.gemspec
-%{gemdir}/doc/%{gemname}-%{version}
+%{gem_instdir}/spec
+%{gem_instdir}/integ
+%{gem_instdir}/Rakefile
+%{gem_instdir}/rubygem-%{gem_name}.spec
+%{gem_instdir}/%{gem_name}.gemspec
+%{gem_docdir}
 
 %changelog
+* Thu May 24 2012 Marc Savy <msavy@redhat.com> - 0.10.2
+- Upstream release: 0.10.2
+- [BGBUILD-347] Add support for Fedora 17. Remove unnecessary OS restrictions
+- [BGBUILD-353] Remove all default attaching (EBS) and mounting (EBS, S3)
+- [BGBUILD-361] Yum runs before having a proper /etc/resolv.conf (EC2 & CentOS 5)
+
 * Wed Feb 29 2012 Marc Savy <msavy@redhat.com> - 0.10.1-1
 - Upstream release: 0.10.1
 - [BGBUILD-332] Add support for bash completion

data/spec/plugins/delivery/ebs/ebs-plugin-spec.rb

@@ -65,20 +65,6 @@ module BoxGrinder
       @plugin.instance_variable_set(:@plugin_config, @plugin_config)
     end
 
-    it "should register all operating systems with specific versions" do
-      prepare_plugin do |plugin|
-        plugin.instance_variable_set(:@current_availability_zone, 'us-east-1a')
-      end
-
-      supported_oses = @plugin.instance_variable_get(:@supported_oses)
-
-      supported_oses.size.should == 3
-      Set.new(supported_oses.keys).should == Set.new(['fedora', 'rhel', 'centos'])
-      supported_oses['rhel'].should == ['6']
-      supported_oses['fedora'].should == ['13', '14', '15', '16']
-      supported_oses['centos'].should == ['5', '6']
-    end
-
     it "should adjust fstab" do
       prepare_plugin { |plugin| plugin.stub!(:after_init) }
 

data/spec/plugins/delivery/s3/s3-plugin-spec.rb

@@ -71,17 +71,6 @@ module BoxGrinder
       @dir = @plugin.instance_variable_get(:@dir)
     end
 
-    it "should register all operating systems with specific versions" do
-      supportes_oses = @plugin.instance_variable_get(:@supported_oses)
-
-      supportes_oses.size.should == 4
-      Set.new(supportes_oses.keys).should == Set.new(['centos', 'fedora', 'rhel', 'sl'])
-      supportes_oses['centos'].should == ['5', '6']
-      supportes_oses['rhel'].should == ['5', '6']
-      supportes_oses['sl'].should == ['5', '6']
-      supportes_oses['fedora'].should == ['13', '14', '15', '16']
-    end
-
     describe ".ami_key" do
 
       before(:each) do
@@ -247,17 +236,12 @@ module BoxGrinder
 
     describe ".validate" do
       before(:each) do
-        @plugin.stub!(:asset_bucket).and_return(nil)
-        @plugin.stub!(:asset_bucket).and_return(@bucket)
+        @plugin.stub(:asset_bucket).and_return(@bucket)
       end
 
       context "bucket is not already in existence" do
         before(:each) do
-          @plugin.should_receive(:set_default_config_value).with('overwrite', false)
-          @plugin.should_receive(:set_default_config_value).with('path', '/')
-          @plugin.should_receive(:set_default_config_value).with('region', 'us-east-1')
-          @plugin.should_receive(:set_default_config_value).with('ramdisk', false)
-          @plugin.should_receive(:set_default_config_value).with('kernel', false)
+          @plugin.stub(:set_default_config_value).and_return(true)
         end
 
         it "should validate only basic params" do
@@ -270,13 +254,20 @@ module BoxGrinder
         it "should validate basic and additional ami params" do
           @plugin.instance_variable_set(:@type, :ami)
 
+          @plugin.should_receive(:set_default_config_value).with('overwrite', false)
+          @plugin.should_receive(:set_default_config_value).with('path', '/')
+          @plugin.should_receive(:set_default_config_value).with('block_device_mappings', {})
+          @plugin.should_receive(:set_default_config_value).with('region', 'us-east-1')
+          @plugin.should_receive(:set_default_config_value).with('ramdisk', false)
+          @plugin.should_receive(:set_default_config_value).with('kernel', false)
+
           @plugin.should_receive(:set_default_config_value).with('snapshot', false)
 
           @plugin.should_receive(:validate_plugin_config).with(['bucket', 'access_key', 'secret_access_key'], 'http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#S3_Delivery_Plugin')
           @plugin.should_receive(:validate_plugin_config).with(["cert_file", "key_file", "account_number"], "http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#S3_Delivery_Plugin")
 
-          File.stub!(:expand_path)
-          File.stub!(:exists?)
+          File.stub(:expand_path)
+          File.stub(:exists?)
           File.should_receive(:expand_path).with('/path/to/cert/file').and_return('path1')
           File.should_receive(:exists?).with('path1').and_return(true)
           File.should_receive(:expand_path).with('/path/to/key/file').and_return('path2')
@@ -296,8 +287,8 @@ module BoxGrinder
           @plugin.should_receive(:validate_plugin_config).with(['bucket', 'access_key', 'secret_access_key'], 'http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#S3_Delivery_Plugin')
           @plugin.should_receive(:validate_plugin_config).with(["cert_file", "key_file", "account_number"], "http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#S3_Delivery_Plugin")
 
-          File.stub!(:expand_path)
-          File.stub!(:exists?)
+          File.stub(:expand_path)
+          File.stub(:exists?)
           File.should_receive(:expand_path).with('/path/to/cert/file').and_return('path1')
           File.should_receive(:exists?).with('path1').and_return(true)
           File.should_receive(:expand_path).with('/path/to/key/file').and_return('path2')
@@ -314,8 +305,8 @@ module BoxGrinder
           @plugin.should_receive(:validate_plugin_config).with(['bucket', 'access_key', 'secret_access_key'], 'http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#S3_Delivery_Plugin')
           @plugin.should_receive(:validate_plugin_config).with(["cert_file", "key_file", "account_number"], "http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#S3_Delivery_Plugin")
 
-          File.stub!(:expand_path)
-          File.stub!(:exists?)
+          File.stub(:expand_path)
+          File.stub(:exists?)
           File.should_receive(:expand_path).with('/path/to/cert/file').and_return('path1')
           File.should_receive(:exists?).with('path1').and_return(false)
 
@@ -332,8 +323,8 @@ module BoxGrinder
           @plugin.should_receive(:validate_plugin_config).with(['bucket', 'access_key', 'secret_access_key'], 'http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#S3_Delivery_Plugin')
           @plugin.should_receive(:validate_plugin_config).with(["cert_file", "key_file", "account_number"], "http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#S3_Delivery_Plugin")
 
-          File.stub!(:expand_path)
-          File.stub!(:exists?)
+          File.stub(:expand_path)
+          File.stub(:exists?)
           File.should_receive(:expand_path).with('/path/to/cert/file').and_return('path1')
           File.should_receive(:exists?).with('path1').and_return(true)
           File.should_receive(:expand_path).with('/path/to/key/file').and_return('path2')
@@ -347,10 +338,10 @@ module BoxGrinder
 
         before(:each) do
           @plugin.instance_variable_set(:@type, :ami)
-          @plugin.stub!(:set_default_config_value)
+          @plugin.stub(:set_default_config_value)
 
-          File.stub!(:expand_path)
-          File.stub!(:exists?)
+          File.stub(:expand_path)
+          File.stub(:exists?)
           File.should_receive(:expand_path).with('/path/to/cert/file').and_return('path1')
           File.should_receive(:exists?).with('path1').and_return(true)
           File.should_receive(:expand_path).with('/path/to/key/file').and_return('path2')
@@ -379,7 +370,7 @@ module BoxGrinder
 
     describe ".bucket" do
       before(:each) do
-        @plugin.unstub!(:asset_bucket)
+        @plugin.unstub(:asset_bucket)
       end
 
       it "should create the asset bucket by default" do
@@ -418,18 +409,18 @@ module BoxGrinder
     describe ".register_image" do
       before(:each) do
         @ami = mock(AWS::EC2::Image)
-        @ami.stub!(:id).and_return('ami-1234')
+        @ami.stub(:id).and_return('ami-1234')
 
         @manifest_key = mock(AWS::S3::S3Object)
-        @manifest_key.stub!(:key).and_return('ami/manifest/key')
+        @manifest_key.stub(:key).and_return('ami/manifest/key')
 
-        @ec2.stub!(:images)
-        @ec2helper.stub!(:wait_for_image_state)
+        @ec2.stub(:images)
+        @ec2helper.stub(:wait_for_image_state)
       end
 
       context "when the AMI has not been registered" do
         before(:each) do
-          @plugin.stub!(:ami_by_manifest_key).and_return(nil)
+          @plugin.stub(:ami_by_manifest_key).and_return(nil)
         end
 
         it "should register the AMI" do
@@ -442,7 +433,7 @@ module BoxGrinder
 
       context "when the AMI has been registered" do
         before(:each) do
-          @plugin.stub!(:ami_by_manifest_key).and_return(@ami)
+          @plugin.stub(:ami_by_manifest_key).and_return(@ami)
         end
 
         it "should not register the AMI" do
@@ -457,13 +448,13 @@ module BoxGrinder
      describe ".deregister_image" do
        before(:each) do
          @ami = mock(AWS::EC2::Image)
-         @plugin.stub!(:ami_by_manifest_key).and_return(@ami)
+         @plugin.stub(:ami_by_manifest_key).and_return(@ami)
 
          @manifest_key = mock(AWS::S3::S3Object)
-         @ec2helper.stub!(:wait_for_image_death)
+         @ec2helper.stub(:wait_for_image_death)
 
-         @ami.stub!(:id)
-         @ami.stub!(:location)
+         @ami.stub(:id)
+         @ami.stub(:location)
        end
 
        it "should deregister the AMI" do