Fix "Build main project" job failure by updating to C++17 and optimizing CI workflow

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>

Author: mmorel-35

.github/workflows/main.yml

@@ -11,6 +11,10 @@ on:
 permissions:
   contents: read
 
+env:
+  GRPC_VERSION: v1.74.1
+  SKYWALKING_PROTOCOL_REF: a9ee9b1b2b9b3e74a152ebb8b61aac50934ac9ed
+
 jobs:
   format:
     runs-on: ubuntu-22.04
@@ -26,11 +30,10 @@ jobs:
       - name: Run clang-format
         run: find ./ -iname "*.h" -o -iname "*.cc" | xargs ./run-clang-format/run-clang-format.py
 
-  unit-test:
+  bazel-test:
     runs-on: ubuntu-22.04
     steps:
-      - run: |
-          echo "/opt/llvm/bin" >> $GITHUB_PATH
+      - run: echo "/opt/llvm/bin" >> $GITHUB_PATH
 
       - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
 
@@ -53,62 +56,88 @@ jobs:
             bazel-build-cache-${{ runner.os }}-${{ github.base_ref }}
             bazel-build-cache-${{ runner.os }}-
 
-      - name: Run bazel test with GCC c++17
-        run: |
-          bazel test --config=ci --test_output=all //...
+      - name: Run bazel test with GCC
+        run: bazel test --config=ci --test_output=all //...
 
-      - name: Run bazel test with CLANG c++17
+      - name: Run bazel test with CLANG
+        run: bazel test --config=ci --test_output=all -c opt --config=clang //...
+
+      - name: Install lcov and genhtml and link llvm
+        run: |
+          sudo apt update
+          sudo apt -y install lcov
+          sudo ln -s /usr/bin/llvm-profdata-11 /usr/bin/llvm-profdata
+          sudo ln -s /usr/bin/llvm-cov-11 /usr/bin/llvm-cov
+
+      - name: Run coverage test
+        run: |
+          ./coverage.sh
+
+  cmake-build:
+    runs-on: ubuntu-22.04
+    steps:
+      - run: echo "/opt/llvm/bin" >> $GITHUB_PATH
+
+      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+
+      - name: Generate dependencies hash
+        id: deps-hash
         run: |
-          bazel test --config=ci --test_output=all -c opt --config=clang //...
+          # Create a hash from dependency names and versions
+          DEPS_STRING="grpc:${{ env.GRPC_VERSION }}|skywalking:${{ env.SKYWALKING_PROTOCOL_REF }}"
+          DEPS_HASH=$(echo -n "$DEPS_STRING" | sha256sum | cut -c1-8)
+          echo "hash=$DEPS_HASH" >> $GITHUB_OUTPUT
+          echo "Dependencies hash (first 8 chars): $DEPS_HASH"
 
       - name: Checkout skywalking-data-collect-protocol
         uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
         with:
           repository: apache/skywalking-data-collect-protocol
-          ref: a9ee9b1b2b9b3e74a152ebb8b61aac50934ac9ed
+          ref: ${{ env.SKYWALKING_PROTOCOL_REF }}
           path: 3rdparty/skywalking-data-collect-protocol
 
+      # Cache gRPC build
+      - name: Cache gRPC build
+        uses: actions/cache@v3
+        with:
+          path: |
+            grpc/build
+            /usr/local/lib/libgrpc*
+            /usr/local/lib/libprotobuf*
+            /usr/local/lib/cmake/grpc
+            /usr/local/lib/cmake/protobuf
+            /usr/local/include/grpc*
+            /usr/local/include/google/protobuf
+            /usr/local/bin/grpc_*
+            /usr/local/bin/protoc
+          key: grpc-cmake-${{ runner.os }}-${{ steps.deps-hash.outputs.hash }}-${{ hashFiles('CMakeLists.txt', 'cmake/grpc.cmake') }}
+          restore-keys: |
+            grpc-cmake-${{ runner.os }}-${{ steps.deps-hash.outputs.hash }}-
+            grpc-cmake-${{ runner.os }}-
+
       - name: Checkout grpc
         uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
         with:
           repository: grpc/grpc
-          ref: v1.74.1
+          ref: ${{ env.GRPC_VERSION }}
           path: grpc
           submodules: true
 
-      - name: Install cmake
+      - name: Install cmake dependencies and run cmake compile
         run: |
           sudo apt update && sudo apt -y install cmake build-essential
-          
-      - name: Build grpc
-        run: |
           sudo cmake -S ./grpc -B ./grpc/build
           sudo cmake --build ./grpc/build --parallel 8 --target install
-          
-      - name: Build main project
-        run: |
           sudo cmake -S . -B ./build
           sudo cmake --build ./build
 
-      - name: Install lcov and genhtml and link llvm
-        run: |
-          sudo apt update
-          sudo apt -y install lcov
-          sudo ln -s /usr/bin/llvm-profdata-11 /usr/bin/llvm-profdata
-          sudo ln -s /usr/bin/llvm-cov-11 /usr/bin/llvm-cov
-
-      - name: Run coverage test
-        run: |
-          ./coverage.sh
-
   e2e-test:
     runs-on: ubuntu-22.04
     steps:
       - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
 
       - name: Prepare service container
-        run: |
-          docker compose -f test/e2e/docker/docker-compose.yml up -d
+        run: docker compose -f test/e2e/docker/docker-compose.yml up -d
 
       - name: Run e2e
         run: |

CMakeLists.txt

@@ -19,7 +19,7 @@ option(GENERATE_CPP2SKY_PKGCONFIG  "Generate and install pkg-config files for UN
 
 
 if(OVERRIDE_CXX_STANDARD_FLAGS)
-    set(CMAKE_CXX_STANDARD 11)
+    set(CMAKE_CXX_STANDARD 17)
     set(CMAKE_CXX_EXTENSIONS OFF)
 endif()