Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOTNET-5871 Upgrade operator to .NET 8 #230

Merged
merged 9 commits into from
Jan 10, 2025
Merged

Conversation

gamingrobot
Copy link
Contributor

@gamingrobot gamingrobot commented Nov 18, 2024

  • Upgrade dotnet-operator-sdk to 9.1.5
  • Upgrade misc packages
  • Rework resource watching
  • Remove controllers (watched entities are now registered in OperatorBuilderExtensions.RegisterEntities)
  • Update webhook url for new operator-sdk version
  • Update JsonDiff to new System.Text.Json diff library
  • Update k8s client calls for official k8s client
  • Update rbac/crds with new manifest generator
  • Update default url for agents
  • Remove submodules

- Upgrade dotnet-operator-sdk to 9.1.5
- Upgrade misc packages
- Rework resource watching
- Remove controllers (watched entities are now registered in OperatorBuilderExtensions.RegisterEntities)
- Update webhook url for new operator-sdk version
- Update JsonDiff to new System.Text.Json diff library
- Update k8s client calls for official k8s client
@gamingrobot gamingrobot force-pushed the DOTNET-5871-net8-upgrade branch from 8279891 to 534f423 Compare January 8, 2025 21:54
@gamingrobot gamingrobot marked this pull request as ready for review January 9, 2025 20:56
@gamingrobot gamingrobot requested a review from a team as a code owner January 9, 2025 20:56
Copy link
Collaborator

@m0nkey653 m0nkey653 left a comment

Choose a reason for hiding this comment

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

👍

@gamingrobot gamingrobot merged commit 756a300 into master Jan 10, 2025
38 checks passed
@gamingrobot gamingrobot deleted the DOTNET-5871-net8-upgrade branch January 10, 2025 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants