Remove kubernetes-deploy* (#622)

* Remove kubernetes-deploy*
* Rename class
* Remove template_dir and only_filenames from RenderTask
* Remove template_dir from DeployTask
* Remove allow_globals from DeployTask
* Move unsafe parallel tests to the serial suite
* Remove duplicated resources from pruning whitelis

1.0-Upgrade.md

@@ -15,7 +15,14 @@ custom resources. See [blacklist](https://github.com/Shopify/kubernetes-deploy/b
 
 ## Public API changes
 
-The only breaking change in the public API (so far) is the renaming of the `KubernetesDeploy` namespace to `Krane`. Otherwise, the APIs of the major public classes (`DeployTask`, `RenderTask`, `RunnerTask`, and `RestartTask`) have not changed between 0.31.1 and 1.0.0. If you're curious about this API, check the comment-based docs on these classes or the [rendered documentation at RubyGems.org](https://www.rubydoc.info/gems/kubernetes-deploy/1.0.0/KubernetesDeploy/DeployTask).
+Important changes:
+- Removed `KubernetesDeploy#*`: use the appropriate `Krane::*` class instead.
+- Removed `template_dir` from `DeployTask#initialize`: use `template_paths` instead.
+- Removed `allow_protected_ns` from `DeployTask#run`: use `protected_namespaces` instead.
+- Removed `template_dir` from `RenderTask#initialize` and `only_filenames` from `RenderTask#run`: use `template_paths` instead.
+- Removed `allow_globals` from `DeployTask`: use `GlobalDeployTask` instead.
+
+If you want to see the current state of this API, check the comment-based docs on these classes or the [rendered documentation at RubyGems.org](https://www.rubydoc.info/gems/kubernetes-deploy/1.0.0/KubernetesDeploy/DeployTask).
 
 ## Command-line interface changes
 

lib/krane/cli/deploy_command.rb

@@ -63,8 +63,7 @@ def self.from_options(namespace, context, options)
           raise Thor::RequiredArgumentMissingError, 'At least one of --filenames or --stdin must be set'
         end
 
-        ::Krane::OptionsHelper.with_processed_template_paths(filenames,
-          require_explicit_path: true) do |paths|
+        ::Krane::OptionsHelper.with_processed_template_paths(filenames) do |paths|
           deploy = ::Krane::DeployTask.new(
             namespace: namespace,
             context: context,
@@ -79,7 +78,6 @@ def self.from_options(namespace, context, options)
 
           deploy.run!(
             verify_result: options["verify-result"],
-            allow_protected_ns: !protected_namespaces.empty?,
             prune: options[:prune]
           )
         end

lib/krane/cli/global_deploy_command.rb

@@ -35,8 +35,7 @@ def self.from_options(context, options)
           raise Thor::RequiredArgumentMissingError, 'At least one of --filenames or --stdin must be set'
         end
 
-        ::Krane::OptionsHelper.with_processed_template_paths(filenames,
-          require_explicit_path: true) do |paths|
+        ::Krane::OptionsHelper.with_processed_template_paths(filenames) do |paths|
           deploy = ::Krane::GlobalDeployTask.new(
             context: context,
             filenames: paths,

lib/krane/cluster_resource_discovery.rb

@@ -20,7 +20,7 @@ def prunable_resources(namespaced:)
       black_list = %w(Namespace Node ControllerRevision)
       api_versions = fetch_api_versions
 
-      fetch_resources(namespaced: namespaced).map do |resource|
+      fetch_resources(namespaced: namespaced).uniq { |r| r['kind'] }.map do |resource|
         next unless resource['verbs'].one? { |v| v == "delete" }
         next if black_list.include?(resource['kind'])
         group_versions = api_versions[resource['apigroup'].to_s]