Skip to content
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

Fix an error in GitHub action tests #184

Open
wants to merge 35 commits into
base: 4.12.0
Choose a base branch
from

Conversation

JuanGarriuz
Copy link
Member

@JuanGarriuz JuanGarriuz commented Feb 7, 2025

Description

The Ddest flag in the mvn repository has been deprecated, This commit shows how the GitHub actions are changed:
opensearch-project@90b2e77

Additionally, test .github/workflows/cypress-test-tenancy-disabled.yml has a mismatched test file. The commit changes the patch to fix it.

Issues Resolved

Testing

  • The GitHub actions Download security plugin and create setup scripts must pass.
  • The GitHub actions Configure and Run OpenSearch Dashboards with Cypress Test Cases must pass.

Check List

  • New functionality includes testing
  • New functionality has been documented
  • Commits are signed per the DCO using --signoff

@JuanGarriuz JuanGarriuz self-assigned this Feb 7, 2025
@JuanGarriuz JuanGarriuz linked an issue Feb 7, 2025 that may be closed by this pull request
@JuanGarriuz JuanGarriuz reopened this Feb 12, 2025
@JuanGarriuz
Copy link
Member Author

@JuanGarriuz
Copy link
Member Author

JuanGarriuz commented Feb 12, 2025

Research

First evidence of port fail:

CypressError: Timed out retrying after 60000ms: The command was expected to run against origin http://localhost:5601` but the application is at origin http://localhost:7000.`

image
image

This error is not related.

image

@JuanGarriuz
Copy link
Member Author

JuanGarriuz commented Feb 12, 2025

@JuanGarriuz
Copy link
Member Author

Research

Since the latest available version of Cypress is installed, the tests are incompatible. However, using an earlier version, such as 13.5.1, makes them compatible with the repository's tests.

@JuanGarriuz
Copy link
Member Author

Research

These two tests are compared by looking for any substantial change that could make a difference, leading to a specific change that causes the multi-auth not to work.

image
image

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.

opensearch-security.zip (No such file or directory)
1 participant