Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 18, 2025

Add .NET 10 Support - Rebased on Master ✅

Successfully added .NET 10 target framework support to the kubernetes-client/csharp repository and rebased on the latest master branch.

Changes Implemented:

  • Update main source projects to target net10.0
    • src/KubernetesClient/KubernetesClient.csproj (net8.0;net9.0 → net8.0;net9.0;net10.0)
    • src/KubernetesClient.Aot/KubernetesClient.Aot.csproj (net8.0;net9.0 → net8.0;net9.0;net10.0)
    • src/KubernetesClient.Kubectl/KubernetesClient.Kubectl.csproj (net8.0;net9.0 → net8.0;net9.0;net10.0)
  • Update test projects to target net10.0
    • tests/KubernetesClient.Tests/KubernetesClient.Tests.csproj (net8.0;net9.0 → net8.0;net9.0;net10.0)
    • tests/E2E.Tests/E2E.Tests.csproj (net8.0;net9.0 → net8.0;net9.0;net10.0)
    • tests/Kubectl.Tests/Kubectl.Tests.csproj (net8.0;net9.0 → net8.0;net9.0;net10.0)
    • tests/KubernetesClient.Classic.Tests/KubernetesClient.Classic.Tests.csproj (net8.0;net9.0;net10.0)
  • Update GitHub Actions workflows to install .NET 10 SDK
    • .github/workflows/buildtest.yaml (added 10.0.x to all dotnet setup steps)
    • .github/workflows/draft.yaml (added 10.0.x)
    • .github/workflows/nuget.yaml (added 10.0.x)
    • .github/workflows/docfx.yaml (added 10.0.x)
    • .github/workflows/codeql-analysis.yml (added 10.0.x)
  • Update README.md version compatibility table to reflect .NET 10 support
  • Rebased on master (commit 9b6afe9)
  • Build and test all changes
    • Build: ✅ Successful for all frameworks including net10.0
    • Tests: ✅ 190/192 tests passing on net10.0 (2 failures are pre-existing WireMock IPv6 dual mode socket issue, unrelated to .NET 10 support)
  • Run security scan with CodeQL - ✅ No security vulnerabilities found

Test Results:

  • All projects successfully build for net10.0 target
  • Tests run successfully on net10.0 with expected pass rate
  • No new issues introduced by .NET 10 support

Security Summary:

✅ CodeQL security scan completed with 0 alerts. No security vulnerabilities found in the changes.

The implementation is minimal and surgical, only adding net10.0 to the existing target frameworks without modifying any code logic. Rebased on master to incorporate latest changes.

Original prompt

.net10 is released today, please add support net10 target


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 18, 2025
Co-authored-by: tg123 <170430+tg123@users.noreply.github.com>
@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 18, 2025
Copilot AI changed the title [WIP] Add support for .net10 target Add .NET 10 target framework support Nov 18, 2025
Copilot AI requested a review from tg123 November 18, 2025 02:42
Copilot finished work on behalf of tg123 November 18, 2025 02:42
@tg123
Copy link
Member

tg123 commented Nov 18, 2025

@copilot could you please rebase with master

Copilot AI and others added 2 commits November 18, 2025 17:21
…oseconds (#1687)

* Initial plan

* Fix datetime serialization to always output 6 decimal places for fractional seconds

Co-authored-by: tg123 <170430+tg123@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: tg123 <170430+tg123@users.noreply.github.com>
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 18, 2025
Copy link
Contributor Author

Copilot AI commented Nov 18, 2025

@copilot could you please rebase with master

Rebased on master successfully. The branch now includes the latest changes from master (commit a0c9b67 - datetime serialization fix) and all commits have been updated accordingly.

Copilot finished work on behalf of tg123 November 18, 2025 17:24
@tg123 tg123 marked this pull request as ready for review November 19, 2025 02:46
Copilot AI review requested due to automatic review settings November 19, 2025 02:46
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 19, 2025
Copy link
Member

@tg123 tg123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/LGTM

Thanks @copilot you are awesome

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 19, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Copilot, tg123

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 19, 2025
@k8s-ci-robot k8s-ci-robot merged commit 068251c into master Nov 19, 2025
16 of 18 checks passed
Copilot finished reviewing on behalf of tg123 November 19, 2025 02:48
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds .NET 10 target framework support to the Kubernetes C# client library. The changes are minimal and surgical, adding net10.0 to existing multi-targeting configurations without modifying core functionality. Additionally, the PR includes a bug fix for DateTime serialization to ensure RFC3339Micro format compliance with the Kubernetes API.

Key Changes

  • Added net10.0 target framework to all source and test projects
  • Updated CI/CD workflows to install and use .NET 10 SDK
  • Fixed DateTime serialization to always output exactly 6 decimal places for fractional seconds (RFC3339Micro format requirement)

Reviewed Changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/KubernetesClient/KubernetesClient.csproj Added net10.0 to target frameworks
src/KubernetesClient.Aot/KubernetesClient.Aot.csproj Added net10.0 to target frameworks
src/KubernetesClient.Kubectl/KubernetesClient.Kubectl.csproj Added net10.0 to target frameworks
src/KubernetesClient/KubernetesJson.cs Fixed DateTime serialization to ensure 6-digit fractional seconds for RFC3339Micro format
tests/KubernetesClient.Tests/KubernetesClient.Tests.csproj Added net10.0 to target frameworks
tests/KubernetesClient.Classic.Tests/KubernetesClient.Classic.Tests.csproj Added net10.0 to target frameworks for both Windows and non-Windows builds
tests/Kubectl.Tests/Kubectl.Tests.csproj Added net10.0 to target frameworks
tests/E2E.Tests/E2E.Tests.csproj Added net10.0 to target frameworks
tests/KubernetesClient.Tests/KubernetesJsonTests.cs Added test to validate 6-digit fractional seconds requirement
.github/workflows/buildtest.yaml Added 10.0.x SDK to build, test, and E2E jobs; updated MSBuild codegen job to use 10.0.x
.github/workflows/nuget.yaml Added 10.0.x SDK to NuGet packaging workflow
.github/workflows/draft.yaml Added 10.0.x SDK to draft release workflow
.github/workflows/docfx.yaml Added 10.0.x SDK to documentation generation workflow
.github/workflows/codeql-analysis.yml Added 10.0.x SDK to CodeQL security analysis workflow
README.md Updated version compatibility table to include net10.0 for SDK version 18.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI added a commit that referenced this pull request Nov 19, 2025
* Initial plan

* Add .NET 10 support to all projects and workflows

Co-authored-by: tg123 <170430+tg123@users.noreply.github.com>

* Fix datetime serialization to always output 6 decimal places for microseconds (#1687)

* Initial plan

* Fix datetime serialization to always output 6 decimal places for fractional seconds

Co-authored-by: tg123 <170430+tg123@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: tg123 <170430+tg123@users.noreply.github.com>

* Initial plan

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: tg123 <170430+tg123@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants