Skip to content

The first version operator And tests the release workflow #1

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

Merged
merged 22 commits into from
Dec 9, 2024
Merged

Conversation

0x5457
Copy link
Collaborator

@0x5457 0x5457 commented Dec 9, 2024

No description provided.

0x5457 added 21 commits December 3, 2024 05:43
- Add finalizer handling in TensorFusionConnection controller
- Implement Release method in NaiveScheduler for resource cleanup

This change ensures proper cleanup of GPU resources when a
TensorFusionConnection is deleted, preventing resource leaks.
- Add webhook manifests for pod mutation
- Remove conversion webhook config from PROJECT file
- Implement webhook test suite with ginkgo framework
- Update pod webhook tests to focus on defaulting instead of conversion
- Add CA injection verification in e2e tests
…king

- Update Release method in naive scheduler to properly track and restore resources
- Add requeue duration for pending connections
- Fix status updates in TensorFusionConnection controller
- Update tests to verify partial and full resource releases
- Make QosClass optional in TensorFusionConnection status
- Add direct URL response for running connections
- Enable gin logging middleware for better request tracking
- Improve connection status handling
…sts/Limits to align with Kubernetes resource naming conventions.
@0x5457 0x5457 requested a review from Code2Life December 9, 2024 08:40
Copy link
Contributor

@Code2Life Code2Life left a comment

Choose a reason for hiding this comment

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

Awesome, LGTM for v0.1 !

@0x5457 0x5457 requested a review from Code2Life December 9, 2024 09:20
@Code2Life Code2Life merged commit 19478c5 into main Dec 9, 2024
4 checks passed
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