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

Added check before running husky install during post install #1000

Merged
merged 1 commit into from
May 2, 2024

Conversation

amsiglan
Copy link
Collaborator

@amsiglan amsiglan commented May 2, 2024

Description

During bootstrap, postinstall script in OSD triggers the postinstall script of all the children repos due to which the path from the security analytics' script gets called is not setup correctly. This PR adds a check to ensure the husky install only runs when running from within the SA repo during first time setup for development.

Issues Resolved

#987

Check List

  • 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: Amardeepsingh Siglani <amardeep7194@gmail.com>
@amsiglan
Copy link
Collaborator Author

amsiglan commented May 2, 2024

Test failure for Windows unrelated to the change in this PR

@amsiglan amsiglan merged commit d3aced1 into opensearch-project:main May 2, 2024
9 of 12 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request May 2, 2024
Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
(cherry picked from commit d3aced1)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request May 2, 2024
Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
(cherry picked from commit d3aced1)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
amsiglan pushed a commit that referenced this pull request May 2, 2024
…1002)

(cherry picked from commit d3aced1)

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
amsiglan pushed a commit that referenced this pull request May 2, 2024
…1001)

(cherry picked from commit d3aced1)

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@amsiglan amsiglan added the backport 2.13 Triggers backport to 2.13 branch label May 9, 2024
opensearch-trigger-bot bot pushed a commit that referenced this pull request May 9, 2024
Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
(cherry picked from commit d3aced1)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
amsiglan pushed a commit that referenced this pull request May 9, 2024
…1021)

(cherry picked from commit d3aced1)

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x backport 2.13 Triggers backport to 2.13 branch backport 2.14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants