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

Support opensearch-dashboard repository to depend on test library #23

Open
Tengda-He opened this issue May 3, 2022 · 5 comments
Open
Labels
enhancement New feature or request

Comments

@Tengda-He
Copy link
Contributor

Tengda-He commented May 3, 2022

Is your feature request related to a problem?

need for a consensus approach for opensearch-dashboard repository to use the opensearch-dashboards-test-library

What solution would you like?

opensearch-dashboard team prefers to consume the test library as a published npm package, like other dependent packages it currently uses.

we need to establish a standard process for npm package publish, currently it is tribe knowledge.

What alternatives have you considered?

Instead of consume the test library as a publish npm version. opensearch-dashboard can also directly use it by branch reference, like how the opensearch-dashboards-functional-test repo imports it now: https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/main/package.json#L39

Trade-off comparison?

Publishing a version npm packages allows us to have version control. And avoid introducing breaking changes from test library to consuming repositories.

Using the branch reference approach saves us from conducting npm package publish process on test library, and package upgrade on consuming repositories. But it requires the change on test library must be backward compatible, otherwise this may lead to breaking changes or regression on client repositories. Increased development and test difficulty.

Do you have any additional context?

related issues:
opensearch-project/opensearch-dashboards-functional-test#29

@Tengda-He Tengda-He added the enhancement New feature or request label May 3, 2022
@Tengda-He
Copy link
Contributor Author

add @tianleh @ananzh @kavilla @peterzhuamazon @seraphjiang for visibility

@tianleh
Copy link
Member

tianleh commented May 4, 2022

Can we follow the similar style https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/main/package.json#L39 to use a branch or a tag? This can avoid NPM publish.

@kavilla
Copy link
Member

kavilla commented May 10, 2022

https://github.com/kavilla/anomaly-detection-dashboards-plugin/runs/6361986510?check_suite_focus=true

AD plugin is seeing issues, I attempted to mitigate it by updating the package-lock file but I'm not positive where to find that identity.

Unfortunately if anytime the main branch gets version bumped or changed, then it will cause the compilation of this package to fail if the lock file's sha does not match what is expected.

@tianleh
Copy link
Member

tianleh commented Jun 17, 2022

@kavilla @ananzh Do you know the org's official process to publish in NPM?

cc @peterzhuamazon @seraphjiang

@wbeckler
Copy link

This needs to get onboarded to the build process. @tianleh you can fill out this issue template to kick off the process of releasing an artifact: https://github.com/opensearch-project/opensearch-build/issues/new?assignees=&labels=release%2Cuntriaged&projects=&template=standalone_releases_template.yaml&title=%5Brelease%5D%3A+

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

No branches or pull requests

4 participants