Skip to content

Fixed cypress memory failures.#1396

Merged
AWSHurneyt merged 13 commits intoopensearch-project:mainfrom
AWSHurneyt:main-cypress-mem-fix
Apr 2, 2026
Merged

Fixed cypress memory failures.#1396
AWSHurneyt merged 13 commits intoopensearch-project:mainfrom
AWSHurneyt:main-cypress-mem-fix

Conversation

@AWSHurneyt
Copy link
Copy Markdown
Collaborator

@AWSHurneyt AWSHurneyt commented Mar 31, 2026

Description

  1. Some cypress test specs were failing due to memory issues. Fixing those issues.
We detected that the Chrome Renderer process just crashed.

We have failed the current spec but will continue running the next spec.

This can happen for a number of different reasons.

If you're running lots of tests on a memory intense application.
  - Try increasing the CPU/memory on the machine you're running on.
  - Try enabling experimentalMemoryManagement in your config file.
  - Try lowering numTestsKeptInMemory in your config file during 'cypress open'.

You can learn more here:

https://on.cypress.io/renderer-process-crashed
  1. Fixed some malformed test assets.
  2. Removed developer comments.
  3. Fixed some header styling for alerts dashboard, and monitor list pages.
  4. Removed duplicate buttons on monitor list page.

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
- "**"
env:
OPENSEARCH_DASHBOARDS_VERSION: 'main'
OPENSEARCH_VERSION: '3.0.0-SNAPSHOT'
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing this variable so the alerting backend can determine whichever version of OS core is needed to bring up a cluster.

run: |
cd OpenSearch-Dashboards/plugins/alerting-dashboards-plugin
yarn osd bootstrap --single-version=loose
- name: Build plugin bundles
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bundling plugins separately as it could take 10+ minutes to bundle during the yarn start step. During that time, the localhost:5601 endpoint is reachable, but not accessible; it displays an error calling out that opensearch dashboards isn't ready.

Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
],
"ui_metadata": {
"schedule": {}
"schedule": {
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixing malformed sample monitor.

],
"ui_metadata": {
"schedule": {}
"schedule": {
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixing malformed sample monitor.

Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
@AWSHurneyt AWSHurneyt mentioned this pull request Apr 1, 2026
5 tasks
Signed-off-by: Thomas Hurney <hurneyt@amazon.com>
@AWSHurneyt AWSHurneyt merged commit ea6d4ea into opensearch-project:main Apr 2, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants