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

feat(cvi): apply new controller design #111

Merged
merged 2 commits into from
Jun 25, 2024
Merged

Conversation

Isteb4k
Copy link
Contributor

@Isteb4k Isteb4k commented May 28, 2024

Description

Apply new CVI controller design

! The VD controller is disabled

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

@Isteb4k Isteb4k force-pushed the feat/cvi/apply-new-design branch 22 times, most recently from 1ade4cf to 916fb74 Compare June 3, 2024 11:01
@Isteb4k Isteb4k marked this pull request as ready for review June 3, 2024 11:03
@Isteb4k Isteb4k force-pushed the feat/cvi/apply-new-design branch 6 times, most recently from 051c186 to f6b7ed5 Compare June 3, 2024 13:15
@Isteb4k Isteb4k force-pushed the feat/cvi/apply-new-design branch from f6b7ed5 to ed78d5f Compare June 9, 2024 19:01
@Isteb4k Isteb4k force-pushed the feat/cvi/apply-new-design branch 3 times, most recently from b4ae20c to 77933a0 Compare June 20, 2024 15:38
Copy link
Member

@diafour diafour left a comment

Choose a reason for hiding this comment

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

Some comments

@Isteb4k Isteb4k requested a review from diafour June 21, 2024 07:38
@Isteb4k Isteb4k force-pushed the feat/cvi/apply-new-design branch from 9b7c0b7 to 7fabb70 Compare June 21, 2024 07:41
@Isteb4k Isteb4k requested a review from yaroslavborbat June 24, 2024 12:22
@Isteb4k Isteb4k force-pushed the feat/cvi/apply-new-design branch from 7fabb70 to 62f1ed6 Compare June 25, 2024 11:38
Copy link
Member

@diafour diafour left a comment

Choose a reason for hiding this comment

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

Let's go!

@Isteb4k Isteb4k merged commit 9e21de8 into main Jun 25, 2024
1 check passed
@Isteb4k Isteb4k deleted the feat/cvi/apply-new-design branch June 25, 2024 11:57
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.

3 participants