One additional comment w/ this patch, is the following addition used anywhere? (I couldn't find where it was referenced)
-Mo
+ def build_credentials + doc = Nokogiri::XML('') + doc.root = Nokogiri::XML::Node.new('provider_credentials', doc) + root = doc.root.at_xpath('/provider_credentials') + + credential_node_name = provider.provider_type.deltacloud_driver + '_credentials' + credential_node = Nokogiri::XML::Node.new(credential_node_name, doc) + root<< credential_node + + creds_label_hash.each do |h| + element = Nokogiri::XML::Node.new(h[:label], doc) + element.content = h[:value] + credential_node<< element + end + doc + end +