-
Notifications
You must be signed in to change notification settings - Fork 211
Remove deprecated and unused functionality: GPU RM, QAT Kerneldrv, xpumanager-sidecar #2100
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
Conversation
In addition to plugin, also remove support for RM from the operator. This has a domino effect where parts of the code in the operator becomes unused/not needed. Signed-off-by: Tuomas Katila <[email protected]>
Signed-off-by: Tuomas Katila <[email protected]>
As GAS was deprecated, xpu-manager sidecar also becomes not needed. Its functionality was to provide info to GAS for scheduling. Signed-off-by: Tuomas Katila <[email protected]>
Signed-off-by: Tuomas Katila <[email protected]>
60f41aa to
27831bb
Compare
|
Other than qat_plugin README update it looks OK to me. |
27831bb to
8161105
Compare
Signed-off-by: Tuomas Katila <[email protected]>
8161105 to
a33d04a
Compare
|
@uniemimu your goodbyes as the codeowner are needed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
@tkatila This missed the image in the GPU plugin readme, it still mentions resource management: https://github.com/intel/intel-device-plugins-for-kubernetes/blob/main/cmd/gpu_plugin/README.md#operation-modes-for-different-workload-types |
|
Summary:
Fixes #1948
Fixes #1765