Skip to content

Commit

Permalink
Ignore some tests for search aggregation for security
Browse files Browse the repository at this point in the history
Signed-off-by: Vacha Shah <vachshah@amazon.com>
  • Loading branch information
VachaShah committed Aug 18, 2022
1 parent caae55c commit 5de559f
Show file tree
Hide file tree
Showing 2 changed files with 52 additions and 20 deletions.
17 changes: 7 additions & 10 deletions .github/workflows/test-integrations.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,14 +13,11 @@ jobs:
fail-fast: false
matrix:
entry:
- { version: '1.3.4', secured: 'true' }
- { version: '1.3.4', secured: 'false' }
- { version: '1.2.4', secured: 'true' }
- { version: '1.2.4', secured: 'false' }
- { version: '1.1.0', secured: 'true' }
- { version: '1.1.0', secured: 'false' }
- { version: '2.2.0', secured: 'true' }
- { version: '2.2.0', secured: 'false' }
- { version: '1.1.0' }
- { version: '1.2.4' }
- { version: '1.3.4' }
- { version: '2.2.0' }
secured: ["true", "false"]

steps:
- name: Checkout
Expand All @@ -37,6 +34,6 @@ jobs:
toolchain: nightly
override: true
components: rustfmt, clippy
- name: Integ OpenSearch ${{ matrix.entry.version }} secured=${{ matrix.entry.secured }}
run: "./.ci/run-tests opensearch ${{ matrix.entry.version }} ${{ matrix.entry.secured }}"
- name: Integ OpenSearch ${{ matrix.entry.version }} secured=${{ matrix.secured }}
run: "./.ci/run-tests opensearch ${{ matrix.entry.version }} ${{ matrix.secured }}"

55 changes: 45 additions & 10 deletions yaml_test_runner/skip_with_security.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,7 @@ tests:

free/indices.shard_stores/10_basic.yml:
# uses number as a key into object. serde_json::Value expects a string key
- "basic index test"
- "multiple indices test"
- "*"

free/indices.stats/12_level.yml:
# uses number as a key into object. serde_json::Value expects a string key
Expand Down Expand Up @@ -98,6 +97,7 @@ tests:

free/cat.allocation/10_basic.yml:
- "Node ID"
- "Bytes"

free/cat.count/10_basic.yml:
- "Test cat count output"
Expand All @@ -118,6 +118,9 @@ tests:
free/cluster.allocation_explain/10_basic.yml:
- "cluster shard allocation explanation test with empty request"

free/cluster.health/20_request_timeout.yml:
- "cluster health request timeout waiting for active shards"

free/indices.refresh/10_basic.yml:
- "Indices refresh test _all"
- "Indices refresh test empty array"
Expand All @@ -132,6 +135,9 @@ tests:
- "Basic multi-search"
- "Least impact smoke test"

free/range/10_basic.yml:
- "Integer range"

free/search/10_source_filtering.yml:
- "*"

Expand All @@ -151,42 +157,71 @@ tests:
- "pre_filter_shard_size with shards that have no hit"

free/search.aggregation/100_avg_metric.yml:
- "Aggregating wrong datatype test"
- "*"

free/search.aggregation/10_histogram.yml:
- "Basic test"
- "Format test"

free/search.aggregation/110_max_metric.yml:
- "Aggregating wrong datatype test"
- "*"

free/search.aggregation/120_min_metric.yml:
- "Aggregating wrong datatype test"
- "*"

free/search.aggregation/130_sum_metric.yml:
- "Aggregating wrong datatype test"
- "*"

free/search.aggregation/140_value_count_metric.yml:
- "*"

free/search.aggregation/150_stats_metric.yml:
- "*"

free/search.aggregation/160_extended_stats_metric.yml:
- "*"

free/search.aggregation/170_cardinality_metric.yml:
- "*"

free/search.aggregation/180_percentiles_tdigest_metric.yml:
- "Invalid params test"
- "*"

free/search.aggregation/190_percentiles_hdr_metric.yml:
- "Basic test"
- "Invalid params test"
- "*"

free/search.aggregation/20_terms.yml:
- "*"

free/search.aggregation/220_filters_bucket.yml:
- "*"

free/search.aggregation/260_weighted_avg.yml:
- "Basic test"

free/search.aggregation/270_median_absolute_deviation_metric.yml:
- "*"

free/search.aggregation/280_rare_terms.yml:
- "*"

free/search.aggregation/280_geohash_grid.yml:
- "Basic test"

free/search.aggregation/290_geotile_grid.yml:
- "Basic test"

free/search.aggregation/320_missing.yml:
- "*"

free/search.aggregation/340_geo_distance.yml:
- "basic"

free/search.aggregation/40_range.yml:
- "*"

free/search.aggregation/50_filter.yml:
- "As a child of terms"
- "*"

free/snapshot.create/10_basic.yml:
- "Create a snapshot"
Expand Down

0 comments on commit 5de559f

Please sign in to comment.