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
Bug 1915929: CONSOLE-2503: Cypress bump to version 6 #7604
Bug 1915929: CONSOLE-2503: Cypress bump to version 6 #7604
Conversation
4b8c05c
to
070385d
Compare
aa589db
to
71a6def
Compare
71a6def
to
ed3f4ca
Compare
ed3f4ca
to
fc45b30
Compare
Removing WIP label as PR has passed CI 3+ times |
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.
Thanks @dtaylor113
@@ -97,6 +97,9 @@ describe('Kubernetes resource CRUD operations', () => { | |||
|
|||
testObjs.forEach( | |||
({ kind, namespaced = true, humanizeKind = true, testI18n = true }, resource) => { | |||
if (Cypress.env('k8TestResource') && kind !== Cypress.env('k8TestResource')) { |
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.
What is this env var for?
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.
debugging, so only one resource is run of the 28. It was helpful being able to only run one of the crud tests when re-running the entire e2e suite for testing this update.
@@ -394,7 +394,9 @@ const Graph: React.FC<GraphProps> = ({ | |||
|
|||
const GraphLink = () => | |||
query ? ( | |||
<Link to={queryBrowserURL(query, namespace)}>{t('monitoring~View in Metrics')}</Link> | |||
<Link aria-label="link to View in Metrics" to={queryBrowserURL(query, namespace)}> |
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.
aria-label
should be translated
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.
fixed
frontend/packages/topology/src/components/page/TopologyPageToolbar.tsx
Outdated
Show resolved
Hide resolved
@@ -17,6 +17,7 @@ const UtilizationAxis: React.FC<UtilizationAxisProps> = ({ timestamps = [] }) => | |||
<div ref={containerRef}> | |||
{!!timestamps.length && ( | |||
<ChartAxis | |||
containerComponent={<ChartContainer title="time axis" />} |
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.
Needs to be translated.
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 this is changing more than test code, we need to have QE review. A Bugzilla bug would be good since it has accessible fixes.
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.
7613e22
to
7ba4714
Compare
/retest Please review the full test history for this PR and help us cut down flakes. |
16 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/lgtm cancel @dtaylor113 You need to run prettier :/ |
…ria labels & titles
acb306a
to
ef7f806
Compare
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dtaylor113, spadgett 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 |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
@dtaylor113: All pull requests linked via external trackers have merged: Bugzilla bug 1915929 has been moved to the MODIFIED state. 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 kubernetes/test-infra repository. |
This PR:
CONSOLE-2503