-
Notifications
You must be signed in to change notification settings - Fork 212
Prepare 0.34.1 #2178
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
Prepare 0.34.1 #2178
Conversation
This file contains hidden or 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
The QAT out-of-tree driver does not provide sysfs based cfg_services interface to read (and write) services configurations so QAT debugfs is used as the fallback. bad65eb changed how the services string is parsed and trimmed but missed the debugfs path. Because of this, the resource registration errors due so invalid resource names (e.g., "sym;dc", which should be "sym-dc"). Fix by handling the debugfs path the same way as the cfg_services sysfs path. Signed-off-by: Mikko Ylinen <mikko.ylinen@intel.com>
And remove the temperature values from health Signed-off-by: Tuomas Katila <tuomas.katila@intel.com>
Signed-off-by: Tuomas Katila <tuomas.katila@intel.com>
Use the existing "temp-limit" as the global limit, and introduce GPU and memory thresholds. Signed-off-by: Tuomas Katila <tuomas.katila@intel.com>
Fix uninitialized variable that caused random behaviour. Signed-off-by: Tuomas Katila <tuomas.katila@intel.com>
By re-downloading the components, we save on the overall container size. While the build time increases slightly, the container size drops by around 100M (520->420). Signed-off-by: Tuomas Katila <tuomas.katila@intel.com>
default behaviour stays the same. Signed-off-by: Tuomas Katila <tuomas.katila@intel.com>
Signed-off-by: Tuomas Katila <tuomas.katila@intel.com> Co-authored-by: Eero Tamminen <eero.t.tamminen@intel.com>
DCAP repository was renamed and this changes the download checksum. The result is that the image build fails and blocks CI. Update the DCAP repository url, download checksum, and the release version. Also, move the base image(s) from jammy to noble. Signed-off-by: Mikko Ylinen <mikko.ylinen@intel.com>
Signed-off-by: Tuomas Katila <tuomas.katila@intel.com>
b40a1c8 to
cbc8c39
Compare
mythi
approved these changes
Dec 4, 2025
Contributor
mythi
left a comment
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
eero-t
reviewed
Dec 4, 2025
Signed-off-by: Tuomas Katila <tuomas.katila@intel.com>
cbc8c39 to
a1fc47d
Compare
eero-t
approved these changes
Dec 5, 2025
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.
No description provided.