From 46861d8c8d261b6560b75d65bafb5e16f96215c8 Mon Sep 17 00:00:00 2001 From: Miki Date: Mon, 8 Nov 2021 15:11:06 -0800 Subject: [PATCH 1/7] Add DCO workflow Signed-off-by: Miki --- .github/workflows/dco.yml | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) create mode 100644 .github/workflows/dco.yml diff --git a/.github/workflows/dco.yml b/.github/workflows/dco.yml new file mode 100644 index 00000000..d4c9ff2c --- /dev/null +++ b/.github/workflows/dco.yml @@ -0,0 +1,18 @@ +name: Developer Certificate of Origin Check + +on: [pull_request] + +jobs: + check: + runs-on: ubuntu-latest + + steps: + - name: Get PR Commits + id: 'get-pr-commits' + uses: tim-actions/get-pr-commits@v1.1.0 + with: + token: ${{ secrets.GITHUB_TOKEN }} + - name: DCO Check + uses: tim-actions/dco@v1.1.0 + with: + commits: ${{ steps.get-pr-commits.outputs.commits }} \ No newline at end of file From 2882c841cd2784e5c2ee97b0558c99fdf94a17c3 Mon Sep 17 00:00:00 2001 From: Keith Chan <12404772+keithhc2@users.noreply.github.com> Date: Thu, 11 Nov 2021 14:52:40 -0800 Subject: [PATCH 2/7] Escaped quotes Signed-off by: Keith Chan --- _search-plugins/ppl/commands.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/_search-plugins/ppl/commands.md b/_search-plugins/ppl/commands.md index fa472d06..6ab816ab 100644 --- a/_search-plugins/ppl/commands.md +++ b/_search-plugins/ppl/commands.md @@ -498,7 +498,7 @@ Field | Description | Required To get all documents from the `accounts` index where `account_number` is 1 or gender is `F`: ```sql -search source=accounts | where account_number=1 or gender="F" | fields account_number, gender; +search source=accounts | where account_number=1 or gender=\"F\" | fields account_number, gender; ``` | account_number | gender From a11d931a992b957de86d210f66a14fc2d7534c2c Mon Sep 17 00:00:00 2001 From: Keith Chan <12404772+keithhc2@users.noreply.github.com> Date: Thu, 11 Nov 2021 15:09:19 -0800 Subject: [PATCH 3/7] Escaped more characters Signed-off by: Keith Chan --- _search-plugins/ppl/commands.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/_search-plugins/ppl/commands.md b/_search-plugins/ppl/commands.md index 6ab816ab..b84f26d0 100644 --- a/_search-plugins/ppl/commands.md +++ b/_search-plugins/ppl/commands.md @@ -57,7 +57,7 @@ search source=accounts; To get all documents from the `accounts` index that have either `account_number` equal to 1 or have `gender` as `F`: ```sql -search source=accounts account_number=1 or gender="F"; +search source=accounts account_number=1 or gender=\"F\"; ``` | account_number | firstname | address | balance | gender | city | employer | state | age | email | lastname | From c128c2937403e5377b10922e3c2097fbbd8d7aef Mon Sep 17 00:00:00 2001 From: Keith Chan <12404772+keithhc2@users.noreply.github.com> Date: Thu, 11 Nov 2021 15:23:12 -0800 Subject: [PATCH 4/7] Fixed typos Signed off by: Signed-off-by: Keith Chan --- _search-plugins/ppl/commands.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/_search-plugins/ppl/commands.md b/_search-plugins/ppl/commands.md index b84f26d0..727b3955 100644 --- a/_search-plugins/ppl/commands.md +++ b/_search-plugins/ppl/commands.md @@ -79,7 +79,7 @@ Field | Description | Type | Required | Default :--- | :--- |:--- |:--- |:--- `int` | Retain the specified number of duplicate events for each combination. The number must be greater than 0. If you do not specify a number, only the first occurring event is kept and all other duplicates are removed from the results. | `string` | No | 1 `keepempty` | If true, keep the document if any field in the field list has a null value or a field missing. | `nested list of objects` | No | False -`consecutive` | If true, remove only consecutive events with duplicate combinations of values. | No | False | - +`consecutive` | If true, remove only consecutive events with duplicate combinations of values. | `Boolean` | No | False `field-list` | Specify a comma-delimited field list. At least one field is required. | Yes | - | - *Example 1*: Dedup by one field @@ -213,12 +213,12 @@ search source=accounts | eval doubleAge = age * 2, ddAge = doubleAge * 2 | field ## fields -Use the `field` command to keep or remove fields from a search result. +Use the `fields` command to keep or remove fields from a search result. ### Syntax ```sql -field [+|-] +fields [+|-] ``` Field | Description | Required | Default From 52fac96ad2fb4388cc7b440fac7cf51cf61fecca Mon Sep 17 00:00:00 2001 From: Keith Chan <12404772+keithhc2@users.noreply.github.com> Date: Thu, 11 Nov 2021 15:35:40 -0800 Subject: [PATCH 5/7] Fixed `field-list` in `dedup` Signed-off-by: Keith Chan --- _search-plugins/ppl/commands.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/_search-plugins/ppl/commands.md b/_search-plugins/ppl/commands.md index 727b3955..7f180cc4 100644 --- a/_search-plugins/ppl/commands.md +++ b/_search-plugins/ppl/commands.md @@ -80,7 +80,7 @@ Field | Description | Type | Required | Default `int` | Retain the specified number of duplicate events for each combination. The number must be greater than 0. If you do not specify a number, only the first occurring event is kept and all other duplicates are removed from the results. | `string` | No | 1 `keepempty` | If true, keep the document if any field in the field list has a null value or a field missing. | `nested list of objects` | No | False `consecutive` | If true, remove only consecutive events with duplicate combinations of values. | `Boolean` | No | False -`field-list` | Specify a comma-delimited field list. At least one field is required. | Yes | - | - +`field-list` | Specify a comma-delimited field list. At least one field is required. | `string` or comma-separated list of strings | Yes | - *Example 1*: Dedup by one field From 8332843c5acff32d034e341048d8fc1b72fce045 Mon Sep 17 00:00:00 2001 From: Liz Snyder <31932630+lizsnyder@users.noreply.github.com> Date: Fri, 12 Nov 2021 05:39:43 -0800 Subject: [PATCH 6/7] Fix link --- _opensearch/install/helm.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/_opensearch/install/helm.md b/_opensearch/install/helm.md index ccbd4502..e66bf3a8 100644 --- a/_opensearch/install/helm.md +++ b/_opensearch/install/helm.md @@ -20,7 +20,7 @@ Resource | Description The specification in the default Helm chart supports many standard use cases and setups. You can modify the default chart to configure your desired specifications and set Transport Layer Security (TLS) and role-based access control (RBAC). For information about the default configuration, steps to configure security, and configurable parameters, see the -[README](https://github.com/opensearch-project/helm-charts/tree/main/charts). +[README](https://github.com/opensearch-project/helm-charts/blob/main/README.md). The instructions here assume you have a Kubernetes cluster with Helm preinstalled. See the [Kubernetes documentation](https://kubernetes.io/docs/setup/) for steps to configure a Kubernetes cluster and the [Helm documentation](https://helm.sh/docs/intro/install/) to install Helm. {: .note } From b9c7520ebdc4f075d6597b0fddcf76e7e04028a9 Mon Sep 17 00:00:00 2001 From: Ryan Bogan Date: Fri, 12 Nov 2021 20:23:23 +0000 Subject: [PATCH 7/7] Add support for codeowners to repo Signed-off-by: Ryan Bogan --- .github/CODEOWNERS | 2 ++ 1 file changed, 2 insertions(+) create mode 100644 .github/CODEOWNERS diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS new file mode 100644 index 00000000..ee246a32 --- /dev/null +++ b/.github/CODEOWNERS @@ -0,0 +1,2 @@ +# This should match the owning team set up in https://github.com/orgs/opensearch-project/teams +* @opensearch-project/docs \ No newline at end of file