OU-962: Merge main into release-coo-0.5 #614
Merged
+19,589
−8,860
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
OU-833: fix 2 bugs for incidents page
The no-session login did not run the validation code, which lead to not closing the openshift guide. This commit wraps the validation login into a cypress command and runs it also with the no-session login.
…ession-login-fix NO-JIRA: Login validation for all session flows
RHINENG-20863: improved fetching logic for incidents
OBSINTA-789: Fix alerts chart empty state
OU-1005: upgrade perses plugins
NO-JIRA: Improve incidents timeline chart domain and loading UX
…rios - Implement Prometheus query_range API interception for mocking the incidents page data - Add complete YAML-based incident fixture system with JSON Schema validation - Support complex incident timelines with severity changes and resolved/ongoing states - Include 5 predefined incident scenario fixtures - Add custom Cypress commands: cy.mockIncidents() and cy.mockIncidentFixture() - Support both YAML and JSON fixture formats - Add timezone configuration via CYPRESS_TIMEZONE environment variable (necessary for correct timeline mocking) - Implement schema validation with CLI tool for fixture validation - Add Cursor commands to generate the scenarios in .cursor/commands/ - Add example test demonstrating different mocking approaches - Integrate mocking system into existing incidents test (01.incidents.cy.ts) - Add dependencies: ajv, js-yaml, @types/ajv, @types/js-yaml - Fix Cypress fixtures folder path from 'fixtures' to 'cypress/fixtures'
…s testing - Add data-test IDs for incidents page components and charts - Refactor Cypress page objects with improved element selectors - Update test cases to use new data-test attributes - Enhance incident search and traversal functionality
OU-989: Kubevirt with fixes after namespace level and session handler
NO-JIRA: fix: immer dependency anr patternfly icons import
…d-data-test-selectors OBSINTA-795: Data Test Selectors for Incidents Page
NO-JIRA: Update translations for incidents
OU-1015: fix: incidents dropdown stays open fix
cluster:health:components:map was changed to cluster_health_components_map. This commit updates the mocking to return the new metric and supports running tests in environments with old backend and new frontend. This workaround is by default disabled and will be fully removed after the COO 1.3.0 release
cluster:health:components:map was changed to cluster_health_components_map. This commit updates the mocking to return the new metric and supports running tests in environments with old backend and new frontend. This workaround is by default disabled and will be fully removed after the COO 1.3.0 release
cluster:health:components:map was changed to cluster_health_components_map. This commit updates the mocking to return the new metric and supports running tests in environments with old backend and new frontend. This workaround is by default disabled and will be fully removed after the COO 1.3.0 release
The UI changes introduced few quirks that break the traversal of incidents, but also solved some that had existing workarounds in the testsuite. This commit removes the retry for the not loading alerts chart, as it is loading reliably now, but because stale state is loaded first, it introduces hard coded wait time. Additionally, it fixes the failing traversal when no incidents exits in the UI and adds test case that verifies that on mocked data.
getIncidentsTimeRanges() used incidentsLastRefreshTime from useSelector immediately after dispatching an update. Since Redux state updates asynchronously, it always received the stale/old value instead of the current time. - Changed calls to use the local currentTime variable instead of Redux state - Simplified getIncidentsTimeRanges() signature: removed dispatch parameter and null handling
OU-1030: revert most of my previous changes & avoid flickering when chang…
OCPBUGS-63056, OCPBUGS-63061: Improve alignment with console SDK
OU-1032: resolve stale Redux state bug in incidents time range calculation
OU-1033: close the first dropdown when deselecting incident
- Add getSelectedIncidentAlerts() to retrieve structured alert data - Add generic table row selectors for flexible table interactions - Fix timing in incident traversal with deselection wait Enables more sophisticated test scenarios requiring alert inspection.
…infrastructure-improvement OBSINTA-858: Improve Cypress Incidents Testing Infrastructure
- Add test for initial incidents loading without filter manipulation (OU-1002) - Add parameterized tests for dropdown closure on deselection (OU-1033) * Tests 3 dropdown types (filter type, severity, days) * Tests 2 deselection methods (bar click, chip removal) - Add test for incident ID filter preservation (OU-1030)
OBSINTA-777: Tests for Incidents Redux Related Regressions
openshift-ci-robot
added
the
jira/valid-reference
|
@PeterYurkovich: This pull request references OU-962 which is a valid jira issue. In response to this: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
openshift-ci
bot
added
the
approved
|
@PeterYurkovich: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
/label ux-approved |
openshift-ci
bot
added
ux-approved
|
@PeterYurkovich: This pull request references OU-962 which is a valid jira issue. In response to this: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: PeterYurkovich, zhuje 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 5517668
into
openshift:release-coo-0.5
5 checks passed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
11 participants
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.