-
Notifications
You must be signed in to change notification settings - Fork 205
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
qat: improve qat_dpdk_app, openssl-qat-engine #1685
Conversation
hj-johannes-lee
commented
Mar 8, 2024
•
edited
Loading
edited
- drop generic from qat_dpdk_app
- add compress-perf to e2e
- make openssl-qat-engine use ubuntu 24.04 and drop all custom builds / cherrypicked in pr Fix repo for opencl-icd, workaround build issues and cherry-pick openssl-qat-engine changes #1728
- drop e2e tests for qat generic
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.
Looks OK but need to get e2e-qat4 green
deployments/qat_dpdk_app/base/crypto-perf-dpdk-pod-requesting-qat.yaml
Outdated
Show resolved
Hide resolved
deployments/qat_dpdk_app/base/crypto-perf-dpdk-pod-requesting-qat.yaml
Outdated
Show resolved
Hide resolved
sorry for not changing to draft before you already made comments. |
I guess because of the last commit of 3 commits cd/validate does not pass. |
You're right that the name is a bit misleading but we cannot rename it for now (getting a better name does not justify the amount of work/changes needed outside of this repository). |
deployments/qat_dpdk_app/crypto-perf/crypto-perf-dpdk-pod-requesting-qat-cy.yaml
Outdated
Show resolved
Hide resolved
I understood, and I somewhat guessed it would not be possible to change. Let me drop the last commit then. |
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.
@hj-johannes-lee looking good to me, I added some minor comments. I'd prefer to wait with this until #1699 is available so that the DPDK tests can be tested and enabled too
This isn't fixing all of it. I'm noticing I did not have the issue tasks defined very clearly. I think it would make more sense to rework the commits so that you get to run the tests on |
I am now confused. |
I proposed to make it so that it's possible to keep using e2e-qat (and DPDK tests running) for now but with all the cleanup done. This PR drops generic but it also stops running the tests |
I'd still suggest we get the big chunk of changes tested + merged on the old setup. There's a lot of unrelated things blocked by |
Signed-off-by: Hyeongju Johannes Lee <[email protected]>
still maintain ci/cd skip test for compress-perf Signed-off-by: Hyeongju Johannes Lee <[email protected]>
Signed-off-by: Hyeongju Johannes Lee <[email protected]>
Signed-off-by: Hyeongju Johannes Lee <[email protected]>
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.
We should update the SPR kernel soon-ish so we can enable compress-perf on it.