Enabled yml/sort-sequence-values.

Signed-off-by: dblock <[email protected]>
opensearch-project · Jul 12, 2024 · 0201262 · 0201262
1 parent feb199e
commit 0201262
Show file tree

Hide file tree

Showing 65 changed files with 485 additions and 473 deletions.
diff --git a/.github/opensearch-cluster/docker-compose.yml b/.github/opensearch-cluster/docker-compose.yml
@@ -7,6 +7,6 @@ services:
       - '9200:9200'
       - '9600:9600'
     environment:
-      - discovery.type=single-node
       - 'OPENSEARCH_INITIAL_ADMIN_PASSWORD=${OPENSEARCH_PASSWORD:-myStrongPassword123!}'
+      - discovery.type=single-node
       - path.repo=/tmp/opensearch/repo
diff --git a/.github/workflows/changelog.yml b/.github/workflows/changelog.yml
@@ -1,7 +1,7 @@
 name: Verify Changelog
 on:
   pull_request:
-    types: [opened, edited, review_requested, synchronize, reopened, ready_for_review, labeled, unlabeled]
+    types: [edited, labeled, opened, ready_for_review, reopened, review_requested, synchronize, unlabeled]
 
 jobs:
   verify-changelog:

diff --git a/.github/workflows/check-links.yml b/.github/workflows/check-links.yml
@@ -1,6 +1,6 @@
 name: Check Links
 
-on: [push, pull_request]
+on: [pull_request,push]
 
 jobs:
   check:

diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -4,13 +4,13 @@ on:
   push:
     branches: ['**']
     paths:
-      - '**/*.yaml'
       - '**/*.ts'
+      - '**/*.yaml'
   pull_request:
     branches: ['**']
     paths:
-      - '**/*.yaml'
       - '**/*.ts'
+      - '**/*.yaml'
 
 jobs:
   lint:

diff --git a/.github/workflows/test-spec.yml b/.github/workflows/test-spec.yml
@@ -5,16 +5,16 @@ on:
     paths:
       - .github/workflows/test-spec.yml
       - package*.json
-      - tsconfig.json
-      - tools/src/tester/**
       - spec/**
+      - tools/src/tester/**
+      - tsconfig.json
   pull_request:
     paths:
       - .github/workflows/test-spec.yml
       - package*.json
-      - tsconfig.json
-      - tools/src/tester/**
       - spec/**
+      - tools/src/tester/**
+      - tsconfig.json
 
 jobs:
   test-opensearch-spec:

diff --git a/.github/workflows/test-tools-integ.yml b/.github/workflows/test-tools-integ.yml
@@ -4,21 +4,21 @@ on:
   push:
     branches: ['**']
     paths:
-      - 'package*.json'
+      - '.github/workflows/test-tools*.yml'
       - eslint.config.mjs
       - jest.config.js
-      - tsconfig.json
+      - 'package*.json'
       - 'tools/**'
-      - '.github/workflows/test-tools*.yml'
+      - tsconfig.json
   pull_request:
     branches: ['**']
     paths:
-      - 'package*.json'
+      - '.github/workflows/test-tools*.yml'
       - eslint.config.mjs
       - jest.config.js
-      - tsconfig.json
+      - 'package*.json'
       - 'tools/**'
-      - '.github/workflows/test-tools*.yml'
+      - tsconfig.json
 
 jobs:
   test:

diff --git a/.github/workflows/test-tools-unit.yml b/.github/workflows/test-tools-unit.yml
@@ -4,21 +4,21 @@ on:
   push:
     branches: ['**']
     paths:
-      - 'package*.json'
+      - '.github/workflows/test-tools*.yml'
       - eslint.config.mjs
       - jest.config.js
-      - tsconfig.json
+      - 'package*.json'
       - 'tools/**'
-      - '.github/workflows/test-tools*.yml'
+      - tsconfig.json
   pull_request:
     branches: ['**']
     paths:
-      - 'package*.json'
+      - '.github/workflows/test-tools*.yml'
       - eslint.config.mjs
       - jest.config.js
-      - tsconfig.json
+      - 'package*.json'
       - 'tools/**'
-      - '.github/workflows/test-tools*.yml'
+      - tsconfig.json
 
 jobs:
   test:

diff --git a/_config.yml b/_config.yml
@@ -1,8 +1,8 @@
 source: .
 destination: ./_site
 include:
-  - index.html
   - favicon.ico
+  - index.html
 exclude:
   - '*'
 keep_files:

diff --git a/eslint.config.mjs b/eslint.config.mjs
@@ -118,7 +118,8 @@ export default [
       'yml/plain-scalar': ['error', 'always'],
       'yml/no-trailing-zeros': 'error',
       'yml/no-multiple-empty-lines': 'error',
-      '@cspell/spellchecker': ['error', { customWordListFile: '.cspell', autoFix: true }]
+      '@cspell/spellchecker': ['error', { customWordListFile: '.cspell', autoFix: true }],
+      'yml/sort-sequence-values': ['error', { pathPattern: '.*', order: { type: 'asc' } }]
     }
   }
 ]
diff --git a/json_schemas/_superseded_operations.schema.yaml b/json_schemas/_superseded_operations.schema.yaml
@@ -15,7 +15,8 @@ patternProperties:
         type: array
         items:
           type: string
+          # eslint-disable-next-line yml/sort-sequence-values
           enum: [GET, POST, PUT, DELETE, HEAD, OPTIONS, PATCH]
-    required: [superseded_by, operations]
+    required: [operations, superseded_by]
     additionalProperties: false
 additionalProperties: false
diff --git a/json_schemas/test_story.schema.yaml b/json_schemas/test_story.schema.yaml
@@ -20,7 +20,7 @@ properties:
       $ref: '#/definitions/Chapter'
   version:
     $ref: '#/definitions/Version'
-required: [description, chapters]
+required: [chapters,description]
 additionalProperties: false
 
 definitions:
@@ -69,6 +69,7 @@ definitions:
         type: string
       method:
         type: string
+        # eslint-disable-next-line yml/sort-sequence-values
         enum: [GET, PUT, POST, DELETE, PATCH, HEAD, OPTIONS]
       parameters:
         type: object
@@ -80,7 +81,7 @@ definitions:
         $ref: '#/definitions/Output'
       version:
         $ref: '#/definitions/Version'
-    required: [path, method]
+    required: [method, path]
 
   Output:
     description: |
@@ -140,7 +141,7 @@ definitions:
       error:
         type: object
         description: Error object.
-    required: [status, content_type, payload]
+    required: [content_type, payload, status]
     additionalProperties: false
 
   Payload:

diff --git a/spec/_superseded_operations.yaml b/spec/_superseded_operations.yaml
@@ -1,3 +1,5 @@
+# eslint-disable yml/sort-sequence-values
+
 $schema: ./json_schemas/_superseded_operations.schema.yaml
 
 /_opendistro/_alerting/destinations:

diff --git a/spec/namespaces/_core.yaml b/spec/namespaces/_core.yaml
@@ -2649,8 +2649,8 @@ components:
               num_freed:
                 type: number
             required:
-              - succeeded
               - num_freed
+              - succeeded
     count@200:
       description: ''
       content:
@@ -2663,8 +2663,8 @@ components:
               _shards:
                 $ref: '../schemas/_common.yaml#/components/schemas/ShardStatistics'
             required:
-              - count
               - _shards
+              - count
     create@200:
       description: ''
       content:
@@ -2792,8 +2792,8 @@ components:
               get:
                 $ref: '../schemas/_common.yaml#/components/schemas/InlineGet'
             required:
-              - _index
               - _id
+              - _index
               - matched
     field_caps@200:
       description: ''
@@ -2811,8 +2811,8 @@ components:
                   additionalProperties:
                     $ref: '../schemas/_core.field_caps.yaml#/components/schemas/FieldCapability'
             required:
-              - indices
               - fields
+              - indices
     get@200:
       description: ''
       content:
@@ -2986,9 +2986,9 @@ components:
                 additionalProperties:
                   type: object
             required:
-              - metric_score
               - details
               - failures
+              - metric_score
     reindex@200:
       description: ''
       content:
@@ -3101,9 +3101,9 @@ components:
                 additionalProperties:
                   $ref: '../schemas/_core.search_shards.yaml#/components/schemas/ShardStoreIndex'
             required:
+              - indices
               - nodes
               - shards
-              - indices
     search_template@200:
       description: ''
       content:
@@ -3148,10 +3148,10 @@ components:
               terminated_early:
                 type: boolean
             required:
-              - took
-              - timed_out
               - _shards
               - hits
+              - timed_out
+              - took
     termvectors@200:
       description: ''
       content:
@@ -3174,11 +3174,11 @@ components:
               _version:
                 $ref: '../schemas/_common.yaml#/components/schemas/VersionNumber'
             required:
-              - found
               - _id
               - _index
-              - took
               - _version
+              - found
+              - took
     update@200:
       description: ''
       content:

diff --git a/spec/namespaces/indices.yaml b/spec/namespaces/indices.yaml
@@ -2134,8 +2134,8 @@ components:
                   $ref: '../schemas/indices.add_block.yaml#/components/schemas/IndicesBlockStatus'
             required:
               - acknowledged
-              - shards_acknowledged
               - indices
+              - shards_acknowledged
     indices.analyze@200:
       description: ''
       content:
@@ -2205,9 +2205,9 @@ components:
               acknowledged:
                 type: boolean
             required:
+              - acknowledged
               - index
               - shards_acknowledged
-              - acknowledged
     indices.create_data_stream@200:
       description: ''
       content:
@@ -2476,9 +2476,9 @@ components:
                 items:
                   $ref: '../schemas/indices.resolve_index.yaml#/components/schemas/ResolveIndexDataStreamsItem'
             required:
-              - indices
               - aliases
               - data_streams
+              - indices
     indices.rollover@200:
       description: ''
       content:
@@ -2524,8 +2524,8 @@ components:
               _shards:
                 $ref: '../schemas/_common.yaml#/components/schemas/ShardStatistics'
             required:
-              - indices
               - _shards
+              - indices
     indices.shard_stores@200:
       description: ''
       content:
@@ -2554,8 +2554,8 @@ components:
                 $ref: '../schemas/_common.yaml#/components/schemas/IndexName'
             required:
               - acknowledged
-              - shards_acknowledged
               - index
+              - shards_acknowledged
     indices.simulate_index_template@200:
       description: ''
       content:
@@ -2592,8 +2592,8 @@ components:
                 $ref: '../schemas/_common.yaml#/components/schemas/IndexName'
             required:
               - acknowledged
-              - shards_acknowledged
               - index
+              - shards_acknowledged
     indices.stats@200:
       description: ''
       content:
@@ -2610,8 +2610,8 @@ components:
               _all:
                 $ref: '../schemas/indices.stats.yaml#/components/schemas/IndicesStats'
             required:
-              - _shards
               - _all
+              - _shards
     indices.update_aliases@200:
       description: ''
       content: