opensearch-project / OpenSearch-Dashboards

📊 Open source visualization dashboards for OpenSearch.
https://opensearch.org/docs/latest/dashboards/index/
Apache License 2.0
1.65k stars 857 forks source link

[BUG] @opensearch-dashboards-test/opensearch-dashboards-test-library is causing bootstrap errors for dashboards-maps #5953

Open derek-ho opened 6 months ago

derek-ho commented 6 months ago

Describe the bug

Adding a dependency on @opensearch-dashboards-test/opensearch-dashboards-test-library is causing bootstrap issues for dashboards maps plugin. It seems that this dependency was introduced in this PR: https://github.com/opensearch-project/OpenSearch-Dashboards/pull/5725.

When bootstrapping the dashboards, the following error shows up:

ERROR [single_version_dependencies] Multiple version ranges for the same dependency
      were found declared across different package.json files. Please consolidate
      those to match across all package.json files. Different versions for the
      same dependency is not supported.
      If you have questions about this please reach out to the operations team.
      The conflicting dependencies are:
        @opensearch-dashboards-test/opensearch-dashboards-test-library
          https://github.com/opensearch-project/opensearch-dashboards-test-library/archive/refs/tags/1.0.6.tar.gz => opensearch-dashboards
          git+https://github.com/opensearch-project/opensearch-dashboards-test-library.git#main => customImportMap

Even with the --single-version=loose flag. I am wondering if this has anything to do with the way it is being introduced, as a archive tar file.

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

OpenSearch Version Please list the version of OpenSearch being used.

Dashboards Version Please list the version of OpenSearch Dashboards being used.

Plugins

Please list all plugins currently enabled.

Screenshots

If applicable, add screenshots to help explain your problem.

Host/Environment (please complete the following information):

Additional context

Add any other context about the problem here.

derek-ho commented 6 months ago

@manasvinibs can you take a look here?

manasvinibs commented 6 months ago

We have follow-up task to remove dependency on this test library which was introduced as part of Cypress tests configuration. Will be addressed as part of this https://github.com/opensearch-project/OpenSearch-Dashboards/issues/5892