Merge pull request #185 from ydb-platform/recover-panic

recover panic on trace calls

Author: asmyasnikov

.github/workflows/gtrace.yml

@@ -12,8 +12,8 @@ jobs:
   test:
     strategy:
       matrix:
-        os: [ ubuntu-latest, macos-latest, windows-latest ]
-        go: [1.14.x, 1.15.x, 1.16.x, 1.17.x]
+        os: [ ubuntu-latest ]
+        go: [1.17.x]
     runs-on: ${{ matrix.os }}
     steps:
     - name: Checkout
@@ -30,16 +30,15 @@ jobs:
 
     - name: Build
       run: |
-        go install ./cmd/gtrace
+        go install ./internal/cmd/gtrace
 
-    - name: Remove committed codegen file for recreate with gtrace on next step
+    - name: Clean and re-generate *_gtrace.go files
       run: |
-        rm ./cmd/gtrace/internal/test/*_gtrace.go
+        cd ./trace && rm -f *_gtrace.go && go generate
 
-    - name: Generate
+    - name: Check generation
       run: |
-        cd ./cmd/gtrace/internal/test/ && go generate
-
-    - name: Test
-      run: |
-        go test -v ./cmd/gtrace/internal/test/
+        if [[ $(git status --porcelain) -ne 0 ]]; then
+          echo "code-generation not compatible with gtrace output";
+          exit 1;
+        fi

CHANGELOG.md

@@ -1,3 +1,8 @@
+* Refactored gtrace tool for generate `Compose` options
+* Added panic recover on trace calls in `Compose` call step
+* Added `trace.With{Discovery,Driver,Coordination,Ratelimiter,Table,Scheme,Scripting}PanicCallback` options
+* Added `ydb.WithPanicCallback` option
+
 ## v3.16.12
 * Fixed bug with check acquire error over `ydb.IsRatelimiterAcquireError`
 * Added full changelog link to github release description