dtk-client 0.5.17 → 0.5.18

data/lib/commands/common/thor/assembly_workspace.rb

@@ -25,7 +25,7 @@ module DTK::Client
         mapping = [REQ_ASSEMBLY_OR_WS_ID,:option_1]
       end
 
-      assembly_or_workspace_id, node_pattern = context_params.retrieve_arguments(mapping,method_argument_names)      
+      assembly_or_workspace_id, node_pattern = context_params.retrieve_arguments(mapping,method_argument_names)
       assembly_start(assembly_or_workspace_id, node_pattern)
     end
 
@@ -49,13 +49,24 @@ module DTK::Client
       post rest_url("assembly/cancel_task"), post_body
     end
 
-    #mode will be :create or :update
-    def promote_assembly_aux(mode,assembly_or_workspace_id,service_module_name=nil,assembly_template_name=nil)
+    # mode will be :create or :update
+    # service_module_name_x can be name or fullname (NS:MOduleName)
+    def promote_assembly_aux(mode,assembly_or_workspace_id,service_module_name_x=nil,assembly_template_name=nil)
       post_body = {
         :assembly_id => assembly_or_workspace_id,
         :mode => mode.to_s
       }
-      post_body.merge!(:service_module_name => service_module_name) if service_module_name
+      if service_module_name_x
+        service_module_name = service_module_name_x
+        namespace = nil
+        if service_module_name_x =~ /(^[^:]+):([^:]+$)/
+          namespace,service_module_name = [$1,$2]
+        end
+        post_body.merge!(:service_module_name => service_module_name) 
+        if namespace
+          post_body.merge!(:namespace => namespace)
+        end
+      end
       post_body.merge!(:assembly_template_name => assembly_template_name) if assembly_template_name
       response = post rest_url("assembly/promote_to_template"), post_body
       # when changing context send request for getting latest assembly_templates instead of getting from cache
@@ -63,9 +74,10 @@ module DTK::Client
 
       return response unless response.ok?()
       #synchronize_clone will load new assembly template into service clone on workspace (if it exists)
-      commit_sha,workspace_branch = response.data(:commit_sha,:workspace_branch)
+      commit_sha,workspace_branch,namespace = response.data(:commit_sha,:workspace_branch,:module_namespace)
       service_module_name ||= response.data(:module_name)
-      Helper(:git_repo).synchronize_clone(:service_module,service_module_name,commit_sha,:local_branch=>workspace_branch)
+      opts = {:local_branch=>workspace_branch, :namespace => namespace}
+      Helper(:git_repo).synchronize_clone(:service_module,service_module_name,commit_sha,opts)
     end
 
     def list_violations_aux(context_params)
@@ -73,7 +85,7 @@ module DTK::Client
       response = post rest_url("assembly/find_violations"),:assembly_id => assembly_or_workspace_id
       response.render_table(:violation)
     end
-     
+
     def converge_aux(context_params)
       assembly_or_workspace_id = context_params.retrieve_arguments([REQ_ASSEMBLY_OR_WS_ID],method_argument_names)
 
@@ -97,7 +109,7 @@ module DTK::Client
 
       if response.data
         confirmation_message = response.data["confirmation_message"]
-        
+
         if confirmation_message
           return unless Console.confirmation_prompt("Workspace service is stopped, do you want to start it"+'?')
           post_body.merge!(:start_assembly=>true)
@@ -122,7 +134,9 @@ module DTK::Client
       return response unless response.ok?
 
 
-      assembly_name,component_module_id,version,repo_url,branch,commit_sha = response.data(:assembly_name,:module_id,:version,:repo_url,:workspace_branch,:branch_head_sha)
+      assembly_name,component_module_id,version,repo_url,branch,commit_sha,full_module_name = response.data(:assembly_name,:module_id,:version,:repo_url,:workspace_branch,:branch_head_sha,:full_module_name)
+      component_module_name = full_module_name if full_module_name
+
       edit_opts = {
         :automatically_clone => true,
         :pull_if_needed => true,
@@ -137,6 +151,7 @@ module DTK::Client
           :commit_sha => commit_sha
         }
       }
+
       version = nil #TODO: version associated with assembly is passed in edit_opts, which is a little confusing
       edit_aux(:component_module,component_module_id,component_module_name,version,edit_opts)
     end
@@ -150,7 +165,7 @@ module DTK::Client
       }
       response = post rest_url("assembly/prepare_for_edit_module"), post_body
       return response unless response.ok?
-      assembly_name,service_module_id,service_module_name,version,repo_url,branch,branch_head_sha,edit_file = response.data(:assembly_name,:module_id,:module_name,:version,:repo_url,:workspace_branch,:branch_head_sha,:edit_file)
+      assembly_name,service_module_id,service_module_name,version,repo_url,branch,branch_head_sha,edit_file = response.data(:assembly_name,:module_id,:full_module_name,:version,:repo_url,:workspace_branch,:branch_head_sha,:edit_file)
       edit_opts = {
         :automatically_clone => true,
         :assembly_module => {
@@ -170,26 +185,23 @@ module DTK::Client
       version = nil #TODO: version associated with assembly is passed in edit_opts, which is a little confusing
       edit_aux(:service_module,service_module_id,service_module_name,version,edit_opts)
     end
-    
+
     def edit_attributes_aux(context_params)
       assembly_or_workspace_id = context_params.retrieve_arguments([REQ_ASSEMBLY_OR_WS_ID],method_argument_names)
-      
-      # if no format is set then we use 'yaml'
-      format = options.format || 'yaml'
-      context_params.forward_options( { :format => format})
-      
-      response = list_attributes_aux(context_params)
+      context_params.forward_options(:format => 'yaml')
+
+      response = list_attributes_aux(context_params,:attribute_type=>:editable)
       return response unless response.ok?
 
-      attributes_list = response.data
-      attributes_hash = attributes_editor(attributes_list, format)
-      
+      yaml_input = response.data
+      edited_yaml = attributes_editor(yaml_input)
+      # sending params in yaml format because marshalling fouls with some data types like nil and Booleans
       post_body = {
         :assembly_id => assembly_or_workspace_id,
-        :av_pairs_hash => attributes_hash
+        :settings_yaml_content => edited_yaml
       }
-      
-      response = post rest_url("assembly/set_attributes"), post_body
+
+      post rest_url("assembly/apply_attribute_settings"), post_body
     end
 
     def push_module_updates_aux(context_params)
@@ -204,14 +216,13 @@ module DTK::Client
       return response unless response.ok?
       return Response::Ok.new() unless response.data(:any_updates)
       if dsl_parsing_errors = response.data(:dsl_parsing_errors)
-        #TODO: not sure if this should be reached
         error_message = "Module '#{component_module_name}' parsing errors found:\n#{dsl_parsing_errors}\nYou can fix errors using 'edit' command from module context and invoke promote-module-updates again.\n"
-        OsUtil.print(dsl_parsed_message, :red) 
+        OsUtil.print(error_message, :red)
         return Response::Error.new()
       end
-      module_name,branch,ff_change = response.data(:module_name,:workspace_branch,:fast_forward_change)
+      module_name,namespace,branch,ff_change = response.data(:module_name,:module_namespace,:workspace_branch,:fast_forward_change)
       ff_change ||= true
-      opts = {:local_branch => branch}
+      opts = {:local_branch => branch,:namespace => namespace}
       opts.merge!(:hard_reset => true) if !ff_change
       response = Helper(:git_repo).pull_changes?(:component_module,module_name,opts)
       return response unless response.ok?()
@@ -240,7 +251,7 @@ module DTK::Client
           end
         end
       end
-     
+
       response
     end
 
@@ -257,7 +268,6 @@ module DTK::Client
     def list_nodes_aux(context_params)
       context_params.method_arguments = ["nodes"]
       list_aux(context_params)
-      # list_assemblies(context_params)
     end
 
     def list_components_aux(context_params)
@@ -272,9 +282,9 @@ module DTK::Client
       # list_assemblies(context_params)
     end
 
-    def list_attributes_aux(context_params)
+    def list_attributes_aux(context_params,opts={})
       context_params.method_arguments = ["attributes"]
-      list_aux(context_params)
+      list_aux(context_params,opts)
     end
 
     def list_tasks_aux(context_params)
@@ -323,7 +333,7 @@ module DTK::Client
     #         data_type = :task
     #       else
     #         raise_validation_error_method_usage('list')
-    #     end 
+    #     end
     #   end
 
     #   post_body = {
@@ -335,7 +345,7 @@ module DTK::Client
     #   post_body.merge!(:detail_to_include => detail_to_include) if detail_to_include
     #   rest_endpoint = "assembly/info_about"
 
-    #   if context_params.is_last_command_eql_to?(:attribute)        
+    #   if context_params.is_last_command_eql_to?(:attribute)
     #     raise DTK::Client::DtkError, "Not supported command for current context level." if attribute_id
     #     about, data_type = get_type_and_raise_error_if_invalid(about, "attributes", ["attributes"])
     #   elsif context_params.is_last_command_eql_to?(:component)
@@ -358,7 +368,7 @@ module DTK::Client
     #       data_type = :assembly
     #       post_body = { :subtype  => 'instance', :detail_level => 'nodes' }
     #       rest_endpoint = "assembly/list"
-    #     end  
+    #     end
     #   end
 
     #   post_body[:about] = about
@@ -395,7 +405,7 @@ module DTK::Client
     end
 
     def create_component_aux(context_params)
-      # If method is invoked from 'assembly/node' level retrieve node_id argument 
+      # If method is invoked from 'assembly/node' level retrieve node_id argument
       # directly from active context
       if context_params.is_there_identifier?(:node)
         mapping = [REQ_ASSEMBLY_OR_WS_ID,:node_id!,:option_1!]
@@ -405,6 +415,7 @@ module DTK::Client
       end
 
       assembly_id,node_id,component_template_id = context_params.retrieve_arguments(mapping,method_argument_names)
+      namespace, component_template_id = get_namespace_and_name_for_component(component_template_id)
 
       post_body = {
         :assembly_id => assembly_id,
@@ -412,6 +423,7 @@ module DTK::Client
         :component_template_id => component_template_id
       }
 
+      post_body.merge!(:namespace => namespace) if namespace
       post rest_url("assembly/add_component"), post_body
     end
 
@@ -433,7 +445,7 @@ module DTK::Client
       end
       post_body = {
         :assembly_id => assembly_or_workspace_id,
-        :input_component_id => dep_cmp, 
+        :input_component_id => dep_cmp,
         :output_component_id => antec_cmp
       }
       post_body.merge!(:dependency_name => dependency_name) if dependency_name
@@ -477,15 +489,15 @@ module DTK::Client
 
     def info_aux(context_params)
       assembly_or_workspace_id, node_id, component_id, attribute_id = context_params.retrieve_arguments([REQ_ASSEMBLY_OR_WS_ID, :node_id, :component_id, :attribute_id],method_argument_names)
-      is_json_return = context_params.get_forwarded_option(:json_return) || false 
- 
+      is_json_return = context_params.get_forwarded_option(:json_return) || false
+
       post_body = {
         :assembly_id => assembly_or_workspace_id,
         :node_id     => node_id,
         :component_id => component_id,
         :attribute_id => attribute_id,
         :subtype     => :instance,
-        :json_return => is_json_return      
+        :json_return => is_json_return
       }
 
       resp = post rest_url("assembly/info"), post_body
@@ -496,7 +508,7 @@ module DTK::Client
       if (component_id.nil? && !node_id.nil?)
         resp.render_workspace_node_info("node")
       elsif (component_id && node_id)
-        resp.render_workspace_node_info("component") 
+        resp.render_workspace_node_info("component")
       else
         return resp
       end
@@ -523,7 +535,7 @@ module DTK::Client
       }
 
       response = post rest_url("assembly/delete"), post_body
-         
+
       # when changing context send request for getting latest assemblies instead of getting from cache
       # @@invalidate_map << :assembly
       response
@@ -544,7 +556,7 @@ module DTK::Client
       else
         mapping = (options.unset? ? [REQ_ASSEMBLY_OR_WS_ID,:option_1!] : [REQ_ASSEMBLY_OR_WS_ID,:option_1!,:option_2!])
       end
-      
+
       assembly_or_workspace_id, pattern, value = context_params.retrieve_arguments(mapping,method_argument_names)
       post_body = {
         :assembly_id => assembly_or_workspace_id,
@@ -583,7 +595,7 @@ module DTK::Client
       end
 
       assembly_or_workspace_id, pattern, value = context_params.retrieve_arguments(mapping,method_argument_names)
-      
+
       post_body = {
         :assembly_id => assembly_or_workspace_id,
         :pattern => pattern,
@@ -639,7 +651,7 @@ module DTK::Client
 
     def delete_aux(context_params)
       if context_params.is_last_command_eql_to?(:node)
-        delete_node_aux(context_params) 
+        delete_node_aux(context_params)
       elsif context_params.is_last_command_eql_to?(:component)
         delete_component_aux(context_params)
       end
@@ -689,7 +701,7 @@ module DTK::Client
       post_body = {
         :assembly_id => assembly_or_workspace_id,
         :node_id => node_id
-      }  
+      }
 
       response = post(rest_url("assembly/initiate_get_netstats"),post_body)
       return response unless response.ok?
@@ -739,7 +751,7 @@ module DTK::Client
       post_body = {
         :assembly_id => assembly_or_workspace_id,
         :components => options["component"]
-      } 
+      }
       post_body[:node_id] = node_id unless node_id.nil?
 
       response = post(rest_url("assembly/initiate_execute_tests"),post_body)
@@ -764,7 +776,7 @@ module DTK::Client
         if count > execute_test_tries or response.data(:is_complete)
           response.data(:results).each do |res|
             if res.key?('test_error')
-              test_error = res.delete('test_error')     
+              test_error = res.delete('test_error')
               res['errors'] = { "message" => test_error, "type" => "test_error" }
             end
           end
@@ -779,7 +791,7 @@ module DTK::Client
       end
 
       if (response.data(:results).empty? && options['timeout'].nil?)
-        raise DTK::Client::DtkValidationError, "Could not finish execution of tests in default timeframe (#{execute_test_tries} seconds). Try again with passing --timeout TIMEOUT parameter" 
+        raise DTK::Client::DtkValidationError, "Could not finish execution of tests in default timeframe (#{execute_test_tries} seconds). Try again with passing --timeout TIMEOUT parameter"
       elsif (response.data(:results).empty? && !options['timeout'].nil?)
         raise DTK::Client::DtkValidationError, "Could not finish execution of tests in set timeframe (#{execute_test_tries} seconds). Try again with increasing --timeout TIMEOUT parameter"
       else
@@ -798,8 +810,8 @@ module DTK::Client
       post_body = {
         :assembly_id => assembly_or_workspace_id,
         :node_id => node_id
-      }  
-      
+      }
+
       response = post(rest_url("assembly/initiate_get_ps"),post_body)
       return response unless response.ok?
 
@@ -828,18 +840,18 @@ module DTK::Client
       end
       filtered = response.data(:results).flatten
 
-      # Amar: had to add more complex filtering in order to print node id and node name in output, 
+      # Amar: had to add more complex filtering in order to print node id and node name in output,
       #       as these two values are sent only in the first element of node's processes list
-      unless (filter_pattern.nil? || !options["filter"])    
+      unless (filter_pattern.nil? || !options["filter"])
         node_id = ""
-        node_name = ""    
+        node_name = ""
         filtered.reject! do |r|
           match = r.to_s.include?(filter_pattern)
           if r["node_id"] && r["node_id"] != node_id
             node_id = r["node_id"]
             node_name = r["node_name"]
           end
-             
+
           if match && !node_id.empty?
             r["node_id"] = node_id
             r["node_name"] = node_name
@@ -847,7 +859,7 @@ module DTK::Client
             node_name = ""
           end
           !match
-        end 
+        end
       end
 
       response.set_data(*filtered)
@@ -865,9 +877,9 @@ module DTK::Client
       else
         mapping = [REQ_ASSEMBLY_OR_WS_ID,:option_1!,:option_2!,:option_3]
       end
-      
+
       assembly_or_workspace_id,node_identifier,log_path,grep_option = context_params.retrieve_arguments(mapping,method_argument_names)
-     
+
       last_line = nil
       begin
 
@@ -929,7 +941,7 @@ module DTK::Client
 
               unless output.empty?
                 file_ready = true
-                tail_temp_file << output 
+                tail_temp_file << output
                 tail_temp_file.flush
               end
 
@@ -963,7 +975,7 @@ module DTK::Client
             t1.exit()
           end
         end
-        
+
         t1.join()
         t2.join()
       rescue Interrupt
@@ -974,7 +986,7 @@ module DTK::Client
       end
     end
 
-    def grep_aux(context_params) 
+    def grep_aux(context_params)
       if context_params.is_there_identifier?(:node)
         mapping = [REQ_ASSEMBLY_OR_WS_ID,:option_1!,:node_id!,:option_2!]
       else
@@ -982,7 +994,7 @@ module DTK::Client
       end
 
       assembly_or_workspace_id,log_path,node_pattern,grep_pattern = context_params.retrieve_arguments(mapping,method_argument_names)
-         
+
       begin
         post_body = {
           :assembly_id         => assembly_or_workspace_id,
@@ -1023,7 +1035,7 @@ module DTK::Client
         end
 
         raise DTK::Client::DtkError, "Error while logging there was no successful response after 3 tries." unless response.data(:is_complete)
-        
+
         console_width = ENV["COLUMNS"].to_i
 
         response.data(:results).each do |r|
@@ -1032,7 +1044,7 @@ module DTK::Client
           message_colorized = DTK::Client::OsUtil.colorize(r[0].inspect, :green)
 
           if r[1]["output"].empty?
-            puts "NODE-ID #{message_colorized} - Log does not contain data that matches you pattern #{grep_pattern}!" 
+            puts "NODE-ID #{message_colorized} - Log does not contain data that matches you pattern #{grep_pattern}!"
           else
             puts "\n"
             console_width.times do
@@ -1048,7 +1060,7 @@ module DTK::Client
       end
     end
 
-    def assembly_start(workspace_id, node_pattern_filter)             
+    def assembly_start(workspace_id, node_pattern_filter)
       post_body = {
         :assembly_id  => workspace_id,
         :node_pattern => node_pattern_filter
@@ -1058,7 +1070,7 @@ module DTK::Client
       response = post rest_url("assembly/start"), post_body
       return response unless response.ok?()
       raise DTK::Client::DtkValidationError, response.data(:errors).first if response.data(:errors)
-      
+
       task_id = response.data(:task_id)
       post rest_url("task/execute"), "task_id" => task_id
     end
@@ -1076,7 +1088,7 @@ module DTK::Client
       response
     end
 
-    def list_aux(context_params)
+    def list_aux(context_params,opts={})
       assembly_or_workspace_id, node_id, component_id, attribute_id, about = context_params.retrieve_arguments([[:service_id!, :workspace_id],:node_id,:component_id,:attribute_id,:option_1],method_argument_names)
       detail_to_include = nil
       format = nil
@@ -1099,7 +1111,7 @@ module DTK::Client
             data_type = :component
             if options.deps?
               detail_to_include = [:component_dependencies]
-            end  
+            end
           when "attributes"
             data_type = (options.links? ? :workspace_attribute_w_link : :workspace_attribute)
             edit_attr_format = context_params.get_forwarded_options()[:format] if context_params.get_forwarded_options()
@@ -1109,6 +1121,9 @@ module DTK::Client
             elsif options.links?
               detail_to_include = [:attribute_links]
             end
+            if opts[:attribute_type]
+              post_options.merge!(:attribute_type => opts[:attribute_type])
+            end
           when "modules"
              detail_to_include = [:version_info]
              data_type = nil #TODO: DynamicDatatype
@@ -1116,7 +1131,7 @@ module DTK::Client
             data_type = :task
           else
             raise_validation_error_method_usage('list')
-        end 
+        end
       end
 
       post_body = {
@@ -1129,7 +1144,7 @@ module DTK::Client
       post_body.merge!(:detail_to_include => detail_to_include) if detail_to_include
       rest_endpoint = "assembly/info_about"
 
-      if context_params.is_last_command_eql_to?(:attribute)        
+      if context_params.is_last_command_eql_to?(:attribute)
         raise DTK::Client::DtkError, "Not supported command for current context level." if attribute_id
         about, data_type = get_type_and_raise_error_if_invalid(about, "attributes", ["attributes"])
       elsif context_params.is_last_command_eql_to?(:component)
@@ -1147,7 +1162,7 @@ module DTK::Client
       else
         if assembly_or_workspace_id
           about, data_type = get_type_and_raise_error_if_invalid(about, "nodes", ["attributes", "components", "nodes", "modules","tasks"])
-          
+
           if data_type.to_s.eql?("component")
             data_type = nil #DynamicDatatype
           end
@@ -1159,7 +1174,7 @@ module DTK::Client
           data_type = :assembly
           post_body = { :subtype  => 'instance', :detail_level => 'nodes' }
           rest_endpoint = "assembly/list"
-        end  
+        end
       end
 
       post_body[:about] = about

data/lib/commands/common/thor/clone.rb

@@ -9,16 +9,18 @@ module DTK::Client
     #                   This will change behaviour of method
     # module_type: will be :component_module or :service_module
 
-    def clone_aux(module_type,module_id,version,internal_trigger,omit_output=false,opts={})
-      module_name,repo_url,branch,not_ok_response = workspace_branch_info(module_type,module_id,version,opts)
+    def clone_aux(module_type, module_id,version,internal_trigger,omit_output=false,opts={})
+      module_name,module_namespace,repo_url,branch,not_ok_response = workspace_branch_info(module_type,module_id,version,opts)
+      full_module_name = ModuleUtil.resolve_name(module_name, module_namespace)
+
       return not_ok_response if not_ok_response
-      response = Helper(:git_repo).create_clone_with_branch(module_type,module_name,repo_url,branch,version,opts)
+      response = Helper(:git_repo).create_clone_with_branch(module_type,module_name,repo_url,branch,version,module_namespace,opts)
 
       if response.ok?
-        puts "Module '#{module_name}' has been successfully cloned!" unless omit_output
+        puts "Module '#{full_module_name}' has been successfully cloned!" unless omit_output
         unless internal_trigger
           if Console.confirmation_prompt("Would you like to edit cloned module now?")
-            context_params_for_module = create_context_for_module(module_name, module_type.to_s.gsub!(/\_/,'-').to_sym)
+            context_params_for_module = create_context_for_module(full_module_name, module_type)
             return edit(context_params_for_module)
             # if module_type.to_s.eql?("service_module")
             #   context_params_for_module = create_context_for_module(module_name, :"service-module")
@@ -33,9 +35,9 @@ module DTK::Client
       response
     end
 
-    def create_context_for_module(module_name, module_type)
+    def create_context_for_module(full_module_name, module_type)
       context_params_for_module = DTK::Shell::ContextParams.new
-      context_params_for_module.add_context_to_params(module_name, module_type, module_name)
+      context_params_for_module.add_context_to_params(full_module_name, module_type.to_s.gsub!(/\_/,'-').to_sym, full_module_name)
       return context_params_for_module
     end
   end

data/lib/commands/common/thor/common.rb

@@ -1,19 +1,26 @@
 module DTK::Client
   module CommonMixin
    private
-    #returns module_name,repo_url,branch,not_ok_response( only if error)
+    # returns module_name,module_namespace,repo_url,branch,not_ok_response( only if error)
     def workspace_branch_info(module_type, module_id, version, opts={})
+      # shortcut if have info about workspace branch already
       if info = opts[:workspace_branch_info]
-        [info[:module_name],info[:repo_url],info[:branch]]
-      else
-        post_body = get_workspace_branch_info_post_body(module_type,module_id,version,opts)
-        response = post(rest_url("#{module_type}/get_workspace_branch_info"),post_body)
-        unless response.ok?
-          [nil,nil,nil,response]
-        else
-          response.data(:module_name,:repo_url,:workspace_branch)
+        name_or_full_module_name = info[:module_name]
+        module_namespace,module_name = ModuleUtil.full_module_name_parts?(name_or_full_module_name)
+        module_namespace ||= info[:module_namespace]
+        ret = [module_name,module_namespace,info[:repo_url],info[:branch]]
+        unless  ret.find{|r|r.nil?}
+          return ret
         end
       end
+
+      post_body = get_workspace_branch_info_post_body(module_type,module_id,version,opts)
+      response = post(rest_url("#{module_type}/get_workspace_branch_info"),post_body)
+      unless response.ok?
+        [nil,nil,nil,nil,response]
+      else
+        response.data(:module_name,:module_namespace,:repo_url,:workspace_branch)
+      end
     end
 
     def get_workspace_branch_info_post_body(module_type, module_id, version_explicit, opts={})
@@ -23,7 +30,7 @@ module DTK::Client
       }
       assembly_module = opts[:assembly_module]
       if version = version_explicit||(assembly_module && assembly_module[:version])
-        post_body.merge!(:version => version) 
+        post_body.merge!(:version => version)
       end
       if assembly_module
         post_body.merge!(:assembly_module => true,:assembly_name => assembly_module[:assembly_name])