Feat/huge renaming api and crds #73
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Isteb4k <[email protected]>
alexey-igrychev
force-pushed
the
feat/huge_renaming_api_and_crds
branch
from
b705372
to
05106a3
Compare
universal-itengineer
added
the
validation/skip/doc_changes
alexey-igrychev
force-pushed
the
feat/huge_renaming_api_and_crds
branch
from
05106a3
to
f95502e
Compare
- Resource Management:
- Rename resources to avoid naming clashes.
- CDI Changes:
- Add 'DVPInternal' prefix to enhance naming clarity.
- Implement fixes and patches for stability improvements.
- Introduce non-buffered writer for object watch streams to prevent rewrite.
- Patches for kubevirt-operator:
- Patch port in kubevirt webhooks, remove 'WEBHOOK_PROXY=no', add rules covering all known webhook paths.
- Kubevirt Adjustments:
- VM controller enhancements:
- Add subscription to detect VMI phase changes.
- Force 'application/json' in vm-controller for consistent data handling.
- Simplify ownerReferences renaming to reduce complexity.
- Virtualization-controller adjustments:
- Fix proxy settings in deployment.
- Prioritize proxy container, add 'default-container' annotations to streamline operations.
- Address goroutines deadlock and potential DATA RACEs in kube proxy.
- Remove debugging and development-time code for production readiness.
- Minor:
- Fix renamed field in VM status.
- Add certmanager to automatically reload certificates from updated Secrets.
- Introduce Cyrillic linter to support non-Latin scripts.
- Change contentType from Error to Warning to reduce log severity.
- Template Enhancements:
- Rewrite templates to unify configuration settings:
- Use 'KUBECONFIG' environment variable across all CDI and Kubevirt controllers.
- Consolidate configurations into a single template for no-webhook-proxy setup.
- Fix template issues to ensure stability and correctness.
Signed-off-by: Aleksei Igrychev <[email protected]>
Signed-off-by: Yaroslav Borbat <[email protected]>
Signed-off-by: Ivan Mikheykin <[email protected]>
alexey-igrychev
force-pushed
the
feat/huge_renaming_api_and_crds
branch
from
f95502e
to
9aaa05d
Compare
universal-itengineer
requested review from
universal-itengineer and
yaroslavborbat
universal-itengineer
approved these changes
May 2, 2024
This was referenced May 6, 2024
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Merge renaming PRs for testing before merging into main
Why do we need it, and what problem does it solve?
Test renamings in dev clusters
Checklist