Speed up the proto conversion workflow by using a pre-built openapi-generator jar

Signed-off-by: xil <fridalu66@gmail.com>

Author: lucy66hw

.github/workflows/backward-compatible-report.yml

@@ -102,20 +102,9 @@ jobs:
           OPENSEARCH_VERSION: ${{ steps.get_opensearch_version.outputs.version }}
         run: npm ci && npm run preprocessing -- --opensearch-version "$OPENSEARCH_VERSION"
 
-      - name: Clone Protobuf Generator Repository
-        run: |
-          git clone https://github.com/OpenAPITools/openapi-generator cloned-repo
-          cd cloned-repo
-          git checkout 6699ecd9d2f4e0868f23bb36566ea03cd1230e6a
-
-      - name: Build Protobuf Generator Tool
-        run: |
-          cd cloned-repo
-          ./mvnw clean package
-
       - name: Convert protobuf
         run: |
-          java -jar cloned-repo/modules/openapi-generator-cli/target/openapi-generator-cli.jar generate -c tools/proto-convert/src/config/protobuf-generator-config.yaml
+          java -jar tools/bin/openapi-generator-cli.jar generate -c tools/proto-convert/src/config/protobuf-generator-config.yaml
 
       - name: Check for auto-pr-branch and use its protos if exists
         run: |

.github/workflows/convert-proto.yml

@@ -13,7 +13,7 @@ on:
 jobs:
   auto-proto-convert:
     runs-on: ubuntu-latest
-    if: github.repository == 'opensearch-project/opensearch-protobufs'
+    #if: github.repository == 'opensearch-project/opensearch-protobufs'
     steps:
       - name: Checkout Repository
         uses: actions/checkout@v4
@@ -104,20 +104,9 @@ jobs:
           OPENSEARCH_VERSION: ${{ steps.get_opensearch_version.outputs.version }}
         run: npm ci && npm run preprocessing -- --opensearch-version "$OPENSEARCH_VERSION"
 
-      - name: Clone Protobuf Generator Repository
-        run: |
-          git clone https://github.com/OpenAPITools/openapi-generator cloned-repo
-          cd cloned-repo
-          git checkout 6699ecd9d2f4e0868f23bb36566ea03cd1230e6a
-
-      - name: Build Protobuf Generator Tool
-        run: |
-          cd cloned-repo
-          ./mvnw clean package
-
       - name: Convert protobuf
         run: |
-          java -jar cloned-repo/modules/openapi-generator-cli/target/openapi-generator-cli.jar generate -c tools/proto-convert/src/config/protobuf-generator-config.yaml
+          java -jar tools/bin/openapi-generator-cli.jar generate -c tools/proto-convert/src/config/protobuf-generator-config.yaml
 
       - name: Post Process Protobuf
         id: merge_report

.gitignore

@@ -58,6 +58,7 @@ maven_output/
 # Gradle
 .gradle/
 !gradle-wrapper.jar
+!tools/bin/openapi-generator-cli.jar
 
 # Logs
 *.log

CHANGELOG.md

@@ -12,6 +12,8 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 - Fix enum value annotations not being preserved ([#353](https://github.com/opensearch-project/opensearch-protobufs/pull/353))
 - Preserve comments on versioned fields and Rename "REMOVED" to "DEPRECATED" in reports ([#355](https://github.com/opensearch-project/opensearch-protobufs/pull/355))
 - Updates the backward compatibility report workflow to compare against the latest proto files ([#356](https://github.com/opensearch-project/opensearch-protobufs/pull/356))
+- Speed up the proto conversion workflow by using a pre-built openapi-generator-cli.jar ([#357](https://github.com/opensearch-project/opensearch-protobufs/pull/357))
+
 ### Removed
 
 ### Fixed

tools/bin/openapi-generator-cli.jar

@@ -588,10 +588,6 @@ export class SchemaModifier {
                     const prop = schema.properties[propName] as any;
                     if (prop && typeof prop === 'object') {
                         prop['x-oneof-property'] = true;
-
-                        if ('$ref' in prop) {
-                            this.markReferencedSchemaAsOneof(prop.$ref);
-                        }
                     }
                 }
             }