Drop dry-* dependencies (#147)

* Step 1, Unaltered config spec passes

* Step 2: slightly altered specs pass without dry

* Update yardocs

* Backport Hash#transform_keys for 2.4

* Fix transport config + specs

* Yardoc fixes

* Call to_h on Config.new param

* Remove the .to_sym / .tr trickery in Resource

* Normalize to_hash to to_h

Author: Kimmo Lehto

README.md

@@ -55,7 +55,7 @@ require 'k8s/json_parser/yajl'
 ### Overview
 The top-level `K8s::Client` provides access to separate `APIClient` instances for each Kubernetes API Group (`v1`, `apps/v1`, etc.), which in turns provides access to separate `ResourceClient` instances for each API resource type (`nodes`, `pods`, `deployments`, etc.).
 
-Individual resources are returned as `K8s::Resource` instances, which are `RecursiveOpenStruct` instances providing attribute access (`resource.metadata.name`). The resource instances are returned by methods such as `client.api('v1').resource('nodes').get('foo')`, and passed as arguments for `client.api('v1').resource('nodes').create_resource(res)`. Resources can also be loaded from disk using `K8s::Resource.from_files(path)`, and passed to the top-level methods such as `client.create_resource(res)`, which lookup the correct API/Resource client from the resource `apiVersion` and `kind`.
+Individual resources are returned as `K8s::Resource` instances, which provide attribute access (`resource.metadata.name`). The resource instances are returned by methods such as `client.api('v1').resource('nodes').get('foo')`, and passed as arguments for `client.api('v1').resource('nodes').create_resource(res)`. Resources can also be loaded from disk using `K8s::Resource.from_files(path)`, and passed to the top-level methods such as `client.create_resource(res)`, which lookup the correct API/Resource client from the resource `apiVersion` and `kind`.
 
 The different `K8s::Error::API` subclasses represent different HTTP response codes, such as `K8s::Error::NotFound` or `K8s::Error::Conflict`.
 
@@ -134,9 +134,9 @@ K8s::Transport.verbose!
 
 ```
 I, [2018-08-09T14:19:50.404739 #1]  INFO -- K8s::Transport: Using config with server=https://167.99.39.233:6443
-I, [2018-08-09T14:19:50.629521 #1]  INFO -- K8s::Transport<https://167.99.39.233:6443>: GET /version => HTTP 200: <K8s::API::Version> in 0.224s
-I, [2018-08-09T14:19:50.681367 #1]  INFO -- K8s::Transport<https://167.99.39.233:6443>: GET /api/v1 => HTTP 200: <K8s::API::MetaV1::APIResourceList> in 0.046s
-I, [2018-08-09T14:19:51.018740 #1]  INFO -- K8s::Transport<https://167.99.39.233:6443>: GET /api/v1/pods => HTTP 200: <K8s::API::MetaV1::List> in 0.316s
+I, [2018-08-09T14:19:50.629521 #1]  INFO -- K8s::Transport<https://167.99.39.233:6443>: GET /version => HTTP 200: <K8s::Resource> in 0.224s
+I, [2018-08-09T14:19:50.681367 #1]  INFO -- K8s::Transport<https://167.99.39.233:6443>: GET /api/v1 => HTTP 200: <K8s::Resource> in 0.046s
+I, [2018-08-09T14:19:51.018740 #1]  INFO -- K8s::Transport<https://167.99.39.233:6443>: GET /api/v1/pods => HTTP 200: <K8s::Resource> in 0.316s
 ```
 
 Using `K8s::Transport.debug!` will also log request/response bodies. The `EXCON_DEBUG=true` env will log all request/response attributes, including headers.

k8s-client.gemspec

@@ -24,8 +24,6 @@ Gem::Specification.new do |spec|
   spec.required_ruby_version = '~> 2.4'
 
   spec.add_runtime_dependency "excon", "~> 0.66"
-  spec.add_runtime_dependency "dry-struct", "~> 0.5.0"
-  spec.add_runtime_dependency "dry-types", "~> 0.13.0"
   spec.add_runtime_dependency "recursive-open-struct", "~> 1.1.0"
   spec.add_runtime_dependency 'hashdiff', '~> 1.0.0'
   spec.add_runtime_dependency 'jsonpath', '~> 0.9.5'

lib/k8s/api.rb

@@ -36,27 +36,26 @@ def api_resources?
       !!@api_resources
     end
 
-    # @param api_resources [Array<K8s::API::MetaV1::APIResource>]
+    # @param api_resources [Array<K8s::Resource>]
     attr_writer :api_resources
 
     # Force-update APIResources
     #
-    # @return [Array<K8s::API::MetaV1::APIResource>]
+    # @return [Array<K8s::Resource>]
     def api_resources!
-      @api_resources = @transport.get(path,
-                                      response_class: K8s::API::MetaV1::APIResourceList).resources
+      @api_resources = @transport.get(path).resources
     end
 
     # Cached APIResources
     #
-    # @return [Array<K8s::API::MetaV1::APIResource>]
+    # @return [Array<K8s::Resource>]
     def api_resources
       @api_resources || api_resources!
     end
 
     # @param resource_name [String]
     # @raise [K8s::Error::UndefinedResource]
-    # @return [K8s::API::MetaV1::APIResource]
+    # @return [K8s::Resource]
     def find_api_resource(resource_name)
       found_resource = api_resources.find{ |api_resource| api_resource.name == resource_name }
       found_resource ||= api_resources!.find{ |api_resource| api_resource.name == resource_name }

lib/k8s/api/metav1.rb

@@ -9,8 +9,6 @@
 require 'k8s/json_parser'
 require 'k8s/util'
 
-require 'k8s/api/metav1'
-require 'k8s/api/version'
 require 'k8s/config'
 require 'k8s/logging'
 require 'k8s/api_client'
@@ -115,7 +113,7 @@ def initialize(transport, namespace: nil)
     end
 
     # @raise [K8s::Error]
-    # @return [K8s::API::Version]
+    # @return [K8s::Resource]
     def version
       @version ||= @transport.version
     end
@@ -134,7 +132,7 @@ def api_groups!
       synchronize do
         @api_groups = @transport.get(
           '/apis',
-          response_class: K8s::API::MetaV1::APIGroupList
+          response_class: K8s::Resource
         ).groups.flat_map{ |api_group| api_group.versions.map(&:groupVersion) }
 
         @api_clients.clear
@@ -165,7 +163,7 @@ def apis(api_versions = nil, prefetch_resources: false, skip_missing: false)
 
         # load into APIClient.api_resources=
         begin
-          @transport.gets(*api_paths, response_class: K8s::API::MetaV1::APIResourceList, skip_missing: skip_missing).each do |api_resource_list|
+          @transport.gets(*api_paths, skip_missing: skip_missing).each do |api_resource_list|
             api(api_resource_list.groupVersion).api_resources = api_resource_list.resources if api_resource_list
           end
         rescue K8s::Error::NotFound, K8s::Error::ServiceUnavailable # rubocop:disable Lint/HandleExceptions