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

[BUG] FATAL Error: Cannot find module '../../../../packages/osd-std' #1487

Closed
ruanyl opened this issue Jun 26, 2023 · 8 comments · Fixed by #1498
Closed

[BUG] FATAL Error: Cannot find module '../../../../packages/osd-std' #1487

ruanyl opened this issue Jun 26, 2023 · 8 comments · Fixed by #1498
Assignees
Labels
bug Something isn't working triaged v2.9.0 Label description

Comments

@ruanyl
Copy link
Member

ruanyl commented Jun 26, 2023

What is the bug?
Found this on main branch, runtime error which says FATAL Error: Cannot find module '../../../../packages/osd-std'

How can one reproduce the bug?
Steps to reproduce the behavior:

  1. Build security-dashboards-plugin
  2. Install security-dashboards-plugin tarball
  3. Start OSD server
  4. Open OSD, you should see error

What is the expected behavior?
No error

What is your host/environment?
Not applicable

Do you have any screenshots?
Not applicable

Do you have any additional context?
The reason is the path of imported osd-std package, it should not be a relative path '../../../../packages/osd-std', but instead from @osd/std

@ruanyl ruanyl added bug Something isn't working untriaged labels Jun 26, 2023
@DarshitChanpura
Copy link
Member

[Triage] Thanks for filing this issue. @cwperks Please add your thoughts on this.

@AMoo-Miki
Copy link
Contributor

This will be a 2.9 release blocker.

@wbeckler
Copy link

How did this get past CI?

@davidlago davidlago added the v2.9.0 Label description label Jun 30, 2023
@davidlago
Copy link

Good question @wbeckler... I see the PR that introduced those imports with the relative paths passed all CI checks. @cwperks I see you are assigned to it, please look into how this snuck past CI to make sure we catch future ones. Thanks!

peternied added a commit to peternied/security-dashboards-plugin that referenced this issue Jun 30, 2023
- Resolves opensearch-project#1487

Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
@peternied
Copy link
Member

Thanks for providing a fix in the description @ruanyl. Would you be able to reproduce your build scenario from a fresh repository(ies), and then copy the full transcript of your log so we can inspect it?

@peternied
Copy link
Member

peternied commented Jun 30, 2023

In my PR, I didn't look into how this snuck past CI, I suspect it has to do with path differences from how our CI workflow is set up compared to how ruanyl performed their build. The code that was called out is clearly wrong, but there might be more issues even past what was resolved in #1498.

@cwperks I'll look to your guidance on what makes sense around merging #1498.

peternied added a commit that referenced this issue Jun 30, 2023
* Fix a bad import path

- Resolves #1487

Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
Co-authored-by: Ryan Liang <109499885+RyanL1997@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this issue Jun 30, 2023
* Fix a bad import path

- Resolves #1487

Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
Co-authored-by: Ryan Liang <109499885+RyanL1997@users.noreply.github.com>
(cherry picked from commit 32c02c8)
peternied added a commit that referenced this issue Jun 30, 2023
* Fix a bad import path

- Resolves #1487

Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
Co-authored-by: Ryan Liang <109499885+RyanL1997@users.noreply.github.com>
(cherry picked from commit 32c02c8)

Co-authored-by: Peter Nied <petern@amazon.com>
@peternied
Copy link
Member

Reopening to make sure the investigation into the root cause isn't lost

@stephen-crawford
Copy link
Contributor

[Triage] Hi @ruanyl, thank you for filing this issue. Looks like this is being addressed by work done by @cwperks.

opensearch-trigger-bot bot pushed a commit that referenced this issue Aug 7, 2023
* Fix a bad import path

- Resolves #1487

Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
Co-authored-by: Ryan Liang <109499885+RyanL1997@users.noreply.github.com>
(cherry picked from commit 32c02c8)
peternied added a commit that referenced this issue Aug 7, 2023
* Fix a bad import path

- Resolves #1487

Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
Co-authored-by: Ryan Liang <109499885+RyanL1997@users.noreply.github.com>
(cherry picked from commit 32c02c8)

Co-authored-by: Peter Nied <petern@amazon.com>
samuelcostae pushed a commit to samuelcostae/security-dashboards-plugin that referenced this issue Aug 10, 2023
* Fix a bad import path

- Resolves opensearch-project#1487

Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
Co-authored-by: Ryan Liang <109499885+RyanL1997@users.noreply.github.com>
Signed-off-by: Sam <samuel.costa@eliatra.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triaged v2.9.0 Label description
Projects
None yet
8 participants