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

ci: fix learna not found issue by forcefully installing lerna in tasks #1419

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

abhilash-sivan
Copy link
Contributor

@abhilash-sivan abhilash-sivan commented Oct 28, 2024

refs INSTA-17150

We are getting lerna not found issue in the prerelease pipeline only. All the test suits are scheduled to run after install npm dependencies step. However, this issue arises sometimes but it's not reproducible for the moment.

@@ -91,6 +91,11 @@ spec:
GIT_COMMIT_DESC=$(git log --format=%B -n 1 $(params.revision))
fi

if ! command -v lerna &> /dev/null; then
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a check for lerna and if not installed, then installing it globally

@abhilash-sivan
Copy link
Contributor Author

The issue reappeared in this branch even after the fix code, node modules were not installed completely. The step shows completed (Green tick) in tekton, but in actuality node-gyp step failed and the node modules are not properly installed. This could be an issue with the prerelease pipeline and the dependencies. Even though a rerun resolves this issue, the root cause needs to be investigated further.

pipeline URL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant