fix: linting changes

STARTcloud · Jan 9, 2024 · b74c980 · b74c980
1 parent eeaf8eb
commit b74c980
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 17 deletions.
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -12,7 +12,6 @@ Metrics/ClassLength:
   Enabled: true
   Exclude:
     - lib/vagrant-zones/driver*
-    - lib/vagrant-zones/action/package*
 
 Naming/MethodName:
   EnforcedStyle: snake_case

diff --git a/lib/vagrant-zones/action/package.rb b/lib/vagrant-zones/action/package.rb
@@ -30,20 +30,14 @@ def execute(...)
         def call(env)
           @machine = env[:machine]
           @driver = @machine.provider.driver
-          name = @machine.name
           boxname = env['package.output']
           mpc = @machine.provider_config
-          brand = mpc.brand
-          kernel = mpc.kernel
-          vcc = mpc.vagrant_cloud_creator
-          boot = mpc.boot
-          boxshortname = mpc.boxshortname
           files = {}
           raise "#{boxname}: Already exists" if File.exist?(boxname)
 
           ## Create Snapshot
           FileUtils.mkdir_p("#{Dir.pwd}/_tmp_package")
-          datasetpath = "#{mpc.boot['array']}/#{mpc.boot['dataset']}/#{name}"
+          datasetpath = "#{mpc.boot['array']}/#{mpc.boot['dataset']}/#{@machine.name}"
           datetime = Time.new.strftime('%Y-%m-%d-%H:%M:%S')
           snapshot_create(datasetpath, datetime, env[:ui], mpc)
           snapshot_send(datasetpath, "#{Dir.pwd}/_tmp_package/box.zss", datetime, env[:ui], mpc)
@@ -89,21 +83,21 @@ def call(env)
           files[env['package.vagrantfile']] = '_Vagrantfile' if env['package.vagrantfile']
 
           info_content_hash = {
-            'boxname' => "#{boxshortname}",
-            'Author' => "#{vcc}",
-            'Vagrant-Zones' => "This box was built with Vagrant-Zones: https://github.com/STARTcloud/vagrant-zones"
+            'boxname' => boxname,
+            'Author' => mpc.vagrant_cloud_creator,
+            'Vagrant-Zones' => 'This box was built with Vagrant-Zones: https://github.com/STARTcloud/vagrant-zones'
           }
           File.write("#{Dir.pwd}/_tmp_package/info.json", info_content_hash.to_json)
 
           metadata_content_hash = {
             'provider' => 'zone',
             'architecture' => 'amd64',
-            'brand' => brand,
+            'brand' => mpc.brand,
             'format' => 'zss',
-            'url' => "https://app.vagrantup.com/#{vcc}/boxes/#{boxshortname}"
+            'url' => "https://app.vagrantup.com/#{mpc.vagrant_cloud_creator}/boxes/#{mpc.boxshortname}"
           }
 
-          metadata_content_hash['kernel'] = kernel if !kernel.nil? && kernel != false
+          metadata_content_hash['kernel'] = mpc.kernel if !mpc.kernel.nil? && mpc.kernel != false
           File.write("#{Dir.pwd}/_tmp_package/metadata.json", metadata_content_hash.to_json)
 
           ## Create the Box file
@@ -134,9 +128,9 @@ def snapshot_send(datasetpath, destination, datetime, uii, config)
         def assemble_box(boxname, tmp_dir)
           is_linux = `bash -c '[[ "$(uname -a)" =~ "Linux" ]]'`
           Dir.chdir(tmp_dir) do
-          files = Dir.glob(File.join('.', '*'))
-          tar_command = is_linux ? "tar -cvzf ../#{boxname}" : "tar -cvzEf ../#{boxname}"
-          `#{tar_command} #{files.join(' ')}`
+            files = Dir.glob(File.join('.', '*'))
+            tar_command = is_linux ? "tar -cvzf" : "tar -cvzEf"
+            `#{tar_command} ../#{boxname} #{files.join(' ')}`
           end
         end
       end