-
Notifications
You must be signed in to change notification settings - Fork 356
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
docs: add release notes for 0.33.0 #9444
Conversation
Docsite preview being generated for this PR. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## release-0.33.0 #9444 +/- ##
===============================================
Coverage 48.57% 48.57%
===============================================
Files 1234 1234
Lines 158883 158883
Branches 2779 2779
===============================================
+ Hits 77171 77173 +2
+ Misses 81537 81535 -2
Partials 175 175
Flags with carried forward coverage won't be shown. Click here to find out more. |
Docsite preview being generated for this PR. |
|
||
Configuration: Introduce a DCGM Helm chart and Prometheus configuration to the | ||
``tools/observability`` directory. Additionally, two new dashboards, "API Monitoring" and "Resource | ||
Utilization", have been added to improve observability and operational insight. |
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.
Have been added to...where? Grafana right?
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.
Yeah after looking at the code change, they've been added to Grafana. I updated the wording to explicitly call that out.
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.
Oh unless you mean added to the tools/observability directory
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.
If so, I'm good, apologies I didn't read it correctly
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.
Yeah, I think that's what it means
Docsite preview being generated for this PR. |
- This does not affect logging in with an existing user who already has an empty or | ||
non-compliant password, but we recommend setting good passwords for such users as soon as | ||
possible. | ||
|
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.
-
WebUI: Enforce password requirements for all new non-remote users. See :ref:
password-requirements
for details.- Applies to users created using the Add User button in the WebUI for admins.
- Admins can change the passwords of other users using the same interface.
- Does not affect existing users with empty or non-compliant passwords, but setting strong passwords for these users is recommended.
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.
Updated
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.
suggested edits
Docsite preview being generated for this PR. |
(cherry picked from commit e165541)
Add release notes for 0.33.0.