chore(build): update Makefile and CI

pingcap · Dec 22, 2023 · e82920b · e82920b
1 parent 1f39ee0
commit e82920b
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 116 deletions.
diff --git a/.github/workflows/manual-create-pd-pr.yaml b/.github/workflows/manual-create-pd-pr.yaml
@@ -4,7 +4,11 @@ on:
   workflow_dispatch:
     inputs:
       release_version:
-        description: "Release version, e.g. v2021.07.09.1"
+        description: "Release version, e.g. 2021.07.09.1"
+        required: true
+      pd_branchs:
+        description: 'PD branch, e.g. ["master", "release-7.6"]'
+        default: '["master"]'
         required: true
 
 jobs:
@@ -13,7 +17,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        branch: [master, release-5.4, release-6.1]
+        # https://stackoverflow.com/questions/69781005/combine-dynamic-github-workflow-matrix-with-input-values-and-predefined-values
+        branch: ${{ fromJson(github.event.inputs.pd_branchs) }} 
     name: Create PD PR - ${{ matrix.branch }}
     steps:
       - name: Check out PD code base
@@ -23,7 +28,7 @@ jobs:
           ref: ${{ matrix.branch }}
       - uses: actions/setup-go@v3
         with:
-          go-version: "1.18"
+          go-version: "1.21"
       - name: Load go module cache
         uses: actions/cache@v3
         with:
@@ -35,23 +40,17 @@ jobs:
             ${{ runner.os }}-go-pd-
       - name: Update TiDB Dashboard in PD code base
         run: |
-          go get -d "github.com/pingcap/tidb-dashboard@${{ github.event.inputs.release_version }}"
-          go mod tidy
-          make pd-server
-          go mod tidy
-          cd tests/client
-          go mod tidy
-          cd ../..
+          scripts/update-dashboard.sh ${{ github.event.inputs.release_version }}
       - name: Commit PD code base change
         id: git_commit
         run: |
           git diff
-          git config user.name "tidb-dashboard-bot"
-          git config user.email "tidb-dashboard-bot@pingcap.com"
+          git config user.name "baurine"
+          git config user.email "2008.hbl@gmail.com"
           git add .
           if git status | grep -q "Changes to be committed"
           then
-            git commit --signoff --message "Update TiDB Dashboard to ${{ github.event.inputs.release_version }}, ref #4257"
+            git commit --signoff --message "Update TiDB Dashboard to v${{ github.event.inputs.release_version }}"
             echo "::set-output name=committed::1"
           else
             echo "No changes detected, skipped"
@@ -65,19 +64,17 @@ jobs:
         with:
           token: ${{ secrets.BOT_PAT }}
           branch: update-tidb-dashboard/${{ matrix.branch }}-${{ github.event.inputs.release_version }}-${{ steps.build_id.outputs.id }}
-          title: Update TiDB Dashboard to ${{ github.event.inputs.release_version }} [${{ matrix.branch }}]
+          title: Update TiDB Dashboard to v${{ github.event.inputs.release_version }} [${{ matrix.branch }}]
           body: |
             ### What problem does this PR solve?
 
             Issue Number: ref #4257
 
-            Update TiDB Dashboard to ${{ github.event.inputs.release_version }}.
-
-            Upstream commit: https://github.com/${{ github.repository }}/commit/${{ github.sha }} .
+            Update TiDB Dashboard to [v${{ github.event.inputs.release_version }}](https://github.com/pingcap/tidb-dashboard/releases/tag/v${{ github.event.inputs.release_version }}).
 
             ### Release note
 
             ```release-note
             None
             ```
-          push-to-fork: tidb-dashboard-bot/pd
+          push-to-fork: baurine/pd
diff --git a/.github/workflows/release-check.yaml b/.github/workflows/release-check.yaml
diff --git a/.github/workflows/release.yaml b/.github/workflows/release.yaml
@@ -2,8 +2,8 @@ name: Release
 
 on:
   push:
-    branches:
-      - release
+    tags:
+      - "v*"
 
 jobs:
   release:
@@ -40,7 +40,7 @@ jobs:
       - name: Lookup release version
         id: lookup_release_version
         run: |
-          echo "::set-output name=release_version::$(grep -v '^#' ./release-version)"
+          echo "::set-output name=release_version::$(git describe --tags --dirty --always)"
       - name: Build UI
         env:
           REACT_APP_MIXPANEL_TOKEN: ${{ secrets.REACT_APP_MIXPANEL_TOKEN }}
@@ -85,78 +85,3 @@ jobs:
           asset_path: ./embedded-assets-golang.zip
           asset_name: embedded-assets-golang.zip
           asset_content_type: application/zip
-  pd_pr:
-    if: "!contains(github.event.head_commit.message, 'skip pd_pr')"
-    name: Create PD PR
-    runs-on: ubuntu-latest
-    needs: release
-    strategy:
-      fail-fast: false
-      matrix:
-        branch: [master, release-5.4, release-6.1]
-    steps:
-      - name: Check out PD code base
-        uses: actions/checkout@v3
-        with:
-          repository: tikv/pd
-          ref: ${{ matrix.branch }}
-      - uses: actions/setup-go@v3
-        with:
-          go-version: "1.18"
-      - name: Load go module cache
-        uses: actions/cache@v3
-        with:
-          path: |
-            ~/.cache/go-build
-            ~/go/pkg/mod
-          key: ${{ runner.os }}-go-pd-${{ hashFiles('**/go.sum') }}
-          restore-keys: |
-            ${{ runner.os }}-go-pd-
-      - name: Update TiDB Dashboard in PD code base
-        run: |
-          go get -d "github.com/pingcap/tidb-dashboard@${{ needs.release.outputs.release_version }}"
-          go mod tidy
-          make pd-server
-          go mod tidy
-          cd tests/client
-          go mod tidy
-          cd ../..
-      - name: Commit PD code base change
-        id: git_commit
-        run: |
-          git diff
-          git config user.name "tidb-dashboard-bot"
-          git config user.email "[email protected]"
-          git add .
-          if git status | grep -q "Changes to be committed"
-          then
-            git commit --signoff --message "Update TiDB Dashboard to ${{ needs.release.outputs.release_version }}, ref #4257"
-            echo "::set-output name=committed::1"
-          else
-            echo "No changes detected, skipped"
-          fi
-      - name: Set build ID
-        id: build_id
-        run: echo "::set-output name=id::$(date +%s)"
-      - name: Create PR based on PD code base
-        uses: peter-evans/create-pull-request@v3
-        if: steps.git_commit.outputs.committed == 1
-        with:
-          token: ${{ secrets.BOT_PAT }}
-          branch: update-tidb-dashboard/${{ matrix.branch }}-${{ needs.release.outputs.release_version }}-${{ steps.build_id.outputs.id }}
-          title: Update TiDB Dashboard to ${{ needs.release.outputs.release_version }} [${{ matrix.branch }}]
-          body: |
-            ### What problem does this PR solve?
-
-            Issue Number: ref #4257
-
-            This is an automatic updating PR for TiDB Dashboard. See #4257 for details.
-
-            This PR updates TiDB Dashboard to ${{ needs.release.outputs.release_version }} for upstream commit: https://github.com/${{ github.repository }}/commit/${{ github.sha }} .
-
-            ### Release note
-
-            ```release-note
-            None
-            ```
-          push-to-fork: tidb-dashboard-bot/pd
diff --git a/Makefile b/Makefile
@@ -14,7 +14,8 @@ E2E_SPEC ?=
 
 UI ?=
 
-RELEASE_VERSION := $(shell grep -v '^\#' ./release-version)
+# RELEASE_VERSION := $(shell grep -v '^\#' ./release-version)
+RELEASE_VERSION ?= $(shell git describe --tags --dirty --always)
 
 LDFLAGS += -X "$(DASHBOARD_PKG)/pkg/utils/version.InternalVersion=$(RELEASE_VERSION)"
 LDFLAGS += -X "$(DASHBOARD_PKG)/pkg/utils/version.Standalone=Yes"