fix: Updated KubernetesClient to 7.2.19, fixed breaking binary change…

…s in serialization. (#121)

src/DotnetKubernetesClient/DotnetKubernetesClient.csproj

@@ -23,7 +23,7 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="KubernetesClient" Version="7.0.15" />
+        <PackageReference Include="KubernetesClient" Version="7.2.19" />
     </ItemGroup>
 
 </Project>

src/DotnetKubernetesClient/Entities/EntityList.cs

@@ -1,4 +1,5 @@
 using System.Collections.Generic;
+using System.Text.Json.Serialization;
 using k8s;
 using k8s.Models;
 
@@ -7,7 +8,9 @@ namespace DotnetKubernetesClient.Entities;
 public class EntityList<T> : KubernetesObject
     where T : IKubernetesObject<V1ObjectMeta>
 {
+    [JsonPropertyName("metadata")]
     public V1ListMeta Metadata { get; set; } = new ();
 
+    [JsonPropertyName("items")]
     public IList<T> Items { get; set; } = new List<T>();
 }

src/DotnetKubernetesClient/KubernetesClient.cs

@@ -8,14 +8,9 @@
 using System.Threading.Tasks;
 using DotnetKubernetesClient.Entities;
 using DotnetKubernetesClient.LabelSelectors;
-using DotnetKubernetesClient.Serialization;
 using k8s;
+using k8s.Autorest;
 using k8s.Models;
-using Microsoft.Rest;
-using Microsoft.Rest.Serialization;
-using Newtonsoft.Json;
-using Newtonsoft.Json.Converters;
-using Newtonsoft.Json.Linq;
 
 namespace DotnetKubernetesClient;
 
@@ -94,7 +89,7 @@ public Task<string> GetCurrentNamespace(string downwardApiEnvName = "POD_NAMESPA
 
             if (result is JsonElement element)
             {
-                return KubernetesJson.Deserialize<TResource>(element);
+                return element.Deserialize<TResource>();
             }
 
             return null;
@@ -127,7 +122,7 @@ public Task<string> GetCurrentNamespace(string downwardApiEnvName = "POD_NAMESPA
 
         if (result is JsonElement element)
         {
-            var list = KubernetesJson.Deserialize<EntityList<TResource>>(element);
+            var list = element.Deserialize<EntityList<TResource>>();
             return list?.Items ?? throw new ArgumentException("Could not parse result");
         }
 
@@ -177,7 +172,7 @@ public async Task<TResource> Create<TResource>(TResource resource)
 
         if (result is JsonElement element)
         {
-            return KubernetesJson.Deserialize<TResource>(element) ?? throw new ArgumentException("Could not parse result");
+            return element.Deserialize<TResource>() ?? throw new ArgumentException("Could not parse result");
         }
 
         throw new ArgumentException("Could not parse result");
@@ -205,7 +200,7 @@ public async Task<TResource> Update<TResource>(TResource resource)
 
         if (result is JsonElement element)
         {
-            return KubernetesJson.Deserialize<TResource>(element) ?? throw new ArgumentException("Could not parse result");
+            return element.Deserialize<TResource>() ?? throw new ArgumentException("Could not parse result");
         }
 
         throw new ArgumentException("Could not parse result");
@@ -233,7 +228,7 @@ public async Task UpdateStatus<TResource>(TResource resource)
 
         if (result is JsonElement element)
         {
-            var parsed = KubernetesJson.Deserialize<TResource>(element) ?? throw new ArgumentException("Could not parse result");
+            var parsed = element.Deserialize<TResource>() ?? throw new ArgumentException("Could not parse result");
             resource.Metadata.ResourceVersion = parsed.Metadata.ResourceVersion;
             return;
         }

tests/DotnetKubernetesClient.Test/KubernetesClient.Test.cs

@@ -108,7 +108,7 @@ public async Task Should_Delete_Some_Object()
                 {
                     Kind = V1ConfigMap.KubeKind,
                     ApiVersion = V1ConfigMap.KubeApiVersion,
-                    Metadata = new(name: "demo-config-two", namespaceProperty: "default"),
+                    Metadata = new(name: RandomName(), namespaceProperty: "default"),
                     Data = new Dictionary<string, string> { { "Hello", "World" } },
                 });
             _objects.Add(config1);