-
Notifications
You must be signed in to change notification settings - Fork 80
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
Adding documentation for running workloads #394
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,6 +7,7 @@ This document will walk you through on what's needed to start contributing code | |
- [Prerequisites](#prerequisites) | ||
- [Setup](#setup) | ||
- [Importing the project into an IDE](#importing-the-project-into-an-ide) | ||
- [Setting Up a Local OpenSearch Cluster For OSB Development (Optional)](#setting-up-a-local-opensearch-cluster-for-osb-development-optional) | ||
- [Executing tests](#executing-tests) | ||
- [Unit tests](#unit-tests) | ||
- [Integration tests](#integration-tests) | ||
|
@@ -24,6 +25,13 @@ This document will walk you through on what's needed to start contributing code | |
- **Pyenv** : Install `pyenv` and follow the instructions in the output of `pyenv init` to set up your shell and restart it before proceeding. | ||
For more details please refer to the [PyEnv installation instructions](https://github.com/pyenv/pyenv#installation). | ||
|
||
**Optional Step:** For Debian-based systems, install the following modules to continue with the next steps: | ||
``` | ||
sudo apt-get install -y make build-essential libssl-dev zlib1g-dev libbz2-dev \ | ||
libreadline-dev libsqlite3-dev wget curl llvm libncurses5-dev libncursesw5-dev \ | ||
xz-utils tk-dev libffi-dev liblzma-dev git | ||
``` | ||
|
||
- **JDK**: Although OSB is a Python application, it optionally builds and provisions OpenSearch clusters. JDK version 17 is used to build the current version of OpenSearch. Please refer to the [build setup requirements](https://github.com/opensearch-project/OpenSearch/blob/ca564fd04f5059cf9e3ce8aba442575afb3d99f1/DEVELOPER_GUIDE.md#install-prerequisites). | ||
Note that the `javadoc` executable should be available in the JDK installation. An earlier version of the JDK can be used, but not all the integration tests will pass. | ||
|
||
|
@@ -38,7 +46,9 @@ This document will walk you through on what's needed to start contributing code | |
|
||
### Setup | ||
|
||
To develop OSB properly, it is recommended that you fork the official OpenSearch Benchmark repository. | ||
To develop OSB properly, it is recommended that you fork the official OpenSearch Benchmark repository. | ||
|
||
For those working on WSL2, it is recommended to clone the repository and set up the working environment within the Linux subsystem. Refer to the guide for setting up WSL2 on [Visual Studio Code](https://code.visualstudio.com/docs/remote/wsl) or [PyCharm](https://www.jetbrains.com/help/pycharm/using-wsl-as-a-remote-interpreter.html#create-wsl-interpreter). | ||
|
||
After you git cloned the forked copy of OpenSearch Benchmark, use the following command-line instructions to set up OpenSearch Benchmark for development: | ||
``` | ||
|
@@ -74,6 +84,74 @@ This is typically created in PyCharm IDE by visiting the `Python Interpreter`, s | |
` | ||
In order to run tests within the PyCharm IDE, ensure the `Python Integrated Tools` / `Testing` / `Default Test Runner` is set to `pytest`. | ||
|
||
## Setting Up a Local OpenSearch Cluster For OSB Development (Optional) | ||
|
||
### OpenSearch Installation | ||
|
||
Download the latest release of OpenSearch from https://opensearch.org/downloads.html. If you are using WSL, make sure to download it into your `/home/<user>` directory instead of `/mnt/c`. | ||
``` | ||
wget https://artifacts.opensearch.org/releases/bundle/opensearch/<x.x.x>/opensearch-<x.x.x>-linux-x64.tar.gz | ||
tar -xf opensearch-x.x.x-linux-x64.tar.gz | ||
cd opensearch-x.x.x | ||
``` | ||
Comment on lines
+87
to
+96
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Some of this material is being updated in the doc repo https://github.com/opensearch-project/documentation-website. It would be good to consolidate the two and direct readers there. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I just saw your PR. Shall I remove this section and just redirect to that page once the PR is merged? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @AkshathRaghav Yes, let's remove this section and redirect it to the page once that PR is merged in. We will have the other PR merged in shortly. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @AkshathRaghav It's published now. You can redirect the readers to this link: https://opensearch.org/docs/latest/benchmark/quickstart/#set-up-an-opensearch-cluster |
||
NOTE: Have Docker running in the background for the next steps. Refer to the installation instructions [here](https://docs.docker.com/compose/install/). | ||
|
||
### OpenSearch Cluster setup | ||
|
||
Add the following settings to the `opensearch.yml` file under the config directory | ||
``` | ||
vim config/opensearch.yml | ||
``` | ||
``` | ||
# | ||
discovery.type: single-node | ||
plugins.security.disabled: true | ||
Comment on lines
+107
to
+108
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should not be necessary, if certificate checking is eliminated. See opensearch-project/documentation-website#5177. |
||
# | ||
``` | ||
Run the opensearch-tar-install.sh script to install and setup a cluster for our use. | ||
``` | ||
bash opensearch-tar-install.sh | ||
``` | ||
Check the output of `curl.exe "http://localhost:9200/_cluster/health?pretty"`. Output should be similar to this: | ||
``` | ||
{ | ||
"cluster_name" : "<name>", | ||
"status" : "green", | ||
"timed_out" : false, | ||
"number_of_nodes" : 1, | ||
"number_of_data_nodes" : 1, | ||
"discovered_master" : true, | ||
"discovered_cluster_manager" : true, | ||
"active_primary_shards" : 3, | ||
"active_shards" : 3, | ||
"relocating_shards" : 0, | ||
"initializing_shards" : 0, | ||
"unassigned_shards" : 0, | ||
"delayed_unassigned_shards" : 0, | ||
"number_of_pending_tasks" : 0, | ||
"number_of_in_flight_fetch" : 0, | ||
"task_max_waiting_in_queue_millis" : 0, | ||
"active_shards_percent_as_number" : 100.0 | ||
} | ||
``` | ||
Now, you have a local cluster running! You can connect to this and run the workload for the next step. | ||
|
||
### Running Workloads on a locally installed Cluster | ||
|
||
Here's a sample executation of the geonames benchmark which can be found from the [workloads](https://github.com/opensearch-project/opensearch-benchmark-workloads) repo. | ||
``` | ||
opensearch-benchmark execute-test --pipeline=benchmark-only --workload=geonames --target-host=127.0.0.1:9200 --test-mode --workload-params '{"number_of_shards":"1","number_of_replicas":"0"}' | ||
``` | ||
|
||
And we're done! You should be seeing the performance metrics soon enough! | ||
|
||
### Debugging | ||
|
||
**If you are not seeing any results, it should be an indicator that there is an issue with your cluster setup or the way the manager is accessing it**. Use the command below to view the logs. | ||
``` | ||
tail -f ~/.bencmark/logs/bechmark.log | ||
``` | ||
|
||
## Executing tests | ||
|
||
Once setup is complete, you may run the unit and integration tests. | ||
|
@@ -87,10 +165,10 @@ make test | |
|
||
### Integration Tests | ||
|
||
Integration tests can be run on the following operating systems: | ||
Integration tests are expected to run for approximately **20-30 mins** and can be run on the following operating systems: | ||
* RedHat | ||
* CentOS | ||
* Ubuntu | ||
* Ubuntu (and WSL) | ||
* Amazon Linux 2 | ||
* MacOS | ||
|
||
|
@@ -100,7 +178,6 @@ Invoke integration tests by running the following command within the root direct | |
make it | ||
``` | ||
|
||
Integration tests are expected to run for approximately 20-30 mins. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why was this line removed? Is the duration different for WSL, perhaps? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I placed it in the line above and bolded the timeframe there. I did this because I had not seen it the first time and I was confused as to why it was taking so long. |
||
|
||
## Submitting your changes for a pull request | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be good to indicate that WSL is a supported platform for development, if that is the case. Some documentation on whether it is WSL1 or WSL2 (or both) would be helpful.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I have tested, it is possible to work on both. Shall I just replace WSL2 with WSL in the documentation, or explicitly say that both are ok to work with.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recommend explicitly saying that both WSL and WSL2 work with OSB