Merge main into f/pipelines-enhancement #2418
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
…datahub-io#1349) * Enhancements to model bias screen * Display of multiple bias charts simultaneously * Multi-select component, allowing free text, or select-from-list selection of chartst to display * Ability to collapse / expand individual charts * User selectable refresh rates of chart data * Chart selection and open / closed status is persisted to session cache for life of user's browser session Display user defined threshold values on charts (opendatahub-io#1163) * Clean up of bias chart logic * Displays thresholds chosen by user, or defaults if none. * Improves domain and threshold calculation based on user values or defaults
* Fix metrics submission issue and handle errors * fix lint issue * use error handler on GET functions
…and set feature flag (opendatahub-io#1390) * Default and restrict threshold, add tooltips, default duplicate name and set feature flag * fix lint * add tooltips and dropdown descriptions * clear data when closing configuration modal * really solve deleting issue, make empty table view a common component and apply it everywhere * address comments
…o#1399) * Adds refresh interval options that match openshift observability dashboard * Show first chart from list, if none selected when user first navigates to bias tab * Use search icon instead of plus for nothing selected empty state * Fix error with calculation of 30 days constant * Deleted charts are removed from session storage
…ahub-io#1403) (opendatahub-io#1404) * Refactor prometheus queries to remove duplication * Fix graph not refreshing issue * Add code review suggestions
opendatahub-io#1413) * Add performance metrics feature flag, refactor runtime server route and solve layout issues * revert some style changes
* Adds support for TrustyAI Operator (opendatahub-io#1276) * Changes from feedback
…datahub-io#1473) * Retrieve bias metrics from thanos querier (opendatahub-io#1472) * Implement review feedback
…ue-1130 Add queries for server and model performance metrics
…hanos-port Makes thanos-querier port configurable and sets bias queries to use OAuth Port
Revert configurable thanos port
…y/configurable-thanos-port"" This reverts commit fec931d.
Fixes incorrect comments
…permission-errors Handle errors when user does not have permission to access prometheus metrics
Unit Test Case for useEventListener
Update model serving images
Fixes stack charts broken when no datapoints available for one or more lines (RHOAIENG-535)
added test for useWatchSegmentKey
add ovms-kserve-ootd template add ovms-kserve-ootd template add ovms-kserve-ootd template add ovms-kserve-ootd template
…ing and token auth is selected Signed-off-by: Mike Turley <[email protected]>
When creating ServingRuntime, only create ServiceAccount if it's missing and token auth is selected
…sockets add support for cypress websockets
…serve add ovms-kserve template
Added no-unnecessary-condition rule
Unit test case for k8s/secrets file
…a-secret [RHOAIENG-2321] Fix Elyra Secret for new DSP Server Modal
Unit test case for data connection utils file
move and rename test files to align with their target file
…erge-main-into-f/pipelines-enhancement-2
|
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-merge-bot
bot
merged commit 1522ff9
into
opendatahub-io:f/pipelines-enhancement
4 checks passed
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
Temporarily turn off the
no-unnecessary-conditionlint rule. Will bring it back after the branch merges tomain.