Enable verbose Git logging in push-changes workflow step

Add `GIT_TRACE=2` and `GIT_CURL_VERBOSE=1` to enhance Git command logging during the push-changes step. This provides detailed trace and curl debugging information for troubleshooting CI/CD issues.
CI: adjust allow 1 retry for first time contributor and fork and skip sync lfs on fork PRs (#647 )
2026-11-03 17:03:43 +08:00 · 2025-03-06 15:34:26 +01:00 · 2025-03-06 15:02:29 +01:00 · 2025-03-06 10:24:14 +01:00 · 2025-03-06 09:38:58 +01:00 · 2025-03-06 09:34:33 +01:00
4 changed files with 7 additions and 2 deletions
@@ -0,0 +1,3 @@
+*       @sunnypilot/dev-internal
+/.github/ @devtekve @sunnyhaibin
+/release/ci/ @devtekve @sunnyhaibin
@@ -23,7 +23,7 @@ jobs:
  sync:
    runs-on: ubuntu-latest
    # Skip if PR is in draft mode
-    if: github.event_name != 'pull_request' || (github.event_name == 'pull_request' && github.event.pull_request.draft == false)
+    if: (github.event_name != 'pull_request' || (github.event_name == 'pull_request' && github.event.pull_request.draft == false)) && !github.event.pull_request.head.repo.fork
    steps:
      - name: Checkout Repository
        uses: actions/checkout@v4
@@ -19,7 +19,7 @@ runs:
    - shell: bash
      name: No retries!
      run: |
-        if [ "${{ github.run_attempt }}" -gt 1 ]; then
+        if [ "${{ github.run_attempt }}" -gt ${{ github.event.pull_request.head.repo.fork && github.event.pull_request.author_association == 'NONE' && 2 || 1}} ]; then
          echo -e "\033[0;31m##################################################"
          echo -e "\033[0;31m    Retries not allowed! Fix the flaky test!      "
          echo -e "\033[0;31m##################################################\033[0m"
@@ -116,6 +116,8 @@ jobs:
      - name: Push changes if there are diffs
        id: push-changes  # Add an id so we can reference this step
        run: |
+          GIT_TRACE=2 
+          GIT_CURL_VERBOSE=1 
          TARGET_BRANCH="${{ inputs.target_branch || env.DEFAULT_TARGET_BRANCH }}"

          # Fetch the latest from remote
Author	SHA1	Message	Date
DevTekVE	a233c3949c	Enable verbose Git logging in push-changes workflow step Add `GIT_TRACE=2` and `GIT_CURL_VERBOSE=1` to enhance Git command logging during the push-changes step. This provides detailed trace and curl debugging information for troubleshooting CI/CD issues.	2025-03-06 15:34:26 +01:00
DevTekVE	7ddaf52b2b	CI: adjust allow 1 retry for first time contributor and fork and skip sync lfs on fork PRs (#647 ) * Update workflow condition to skip forked PRs during sync This change ensures the LFS maintenance workflow does not run for pull requests originating from forked repositories. It enhances efficiency by avoiding unnecessary sync actions in these cases. * Modified retry conditions in GitHub workflow file Adjusted the retry conditions in the GitHub workflow setup file. The updated condition allows for a second attempt if the pull request originator is a 'FIRST_TIME_CONTRIBUTOR' from a forked repository. This modification lends more flexibility to first-time contributors by providing them with an additional chance to pass the tests. * test * test2 * test3 * Fix conditional logic for retry check in workflow script The condition now correctly checks if the pull request is not from a fork using 'false' instead of 'true'. This ensures proper validation of retry attempts based on the contributor and repository source. * Fix placement of event JSON logging in workflow script Moved the `toJSON(github.event)` logging statement to ensure it runs after retry logic validation. This improves the logical flow of the script and maintains appropriate debugging output. * Add debug logs for PR metadata and run attempt values This change introduces debug logs to output key PR metadata, such as fork status, author association, and the GitHub run attempt value. These logs will aid in debugging and provide better visibility into workflow execution. * Add debug log for PR fork and author association check This change adds a debug log to display the evaluation of the fork and author association condition in the workflow. It helps in troubleshooting PR handling logic and ensures better visibility into the condition being assessed. * Fix logical negation for fork checks in workflow script Correct the evaluation of fork status using proper negation syntax. This ensures accurate logic in pull request conditions, improving the reliability of the workflow execution. * Add log for first-time contributor check in workflow This adds a debug statement to log whether the PR author is a first-time contributor. It helps improve observability and makes it easier to debug workflows involving author associations. * Simplify log output in GitHub action script Removed unnecessary debug echo statements to streamline the script and improve readability. Retained essential logic to handle retries and enforce restrictions on test flakiness. * Simplify retry logic for pull request contributors. Updated the condition to correctly identify and handle retries for first-time contributors and forked repositories. This ensures a clearer and more accurate retry policy in the workflow. * Refine workflow conditions for PR checks and retries Adjusted conditional logic in `lfs-maintenance.yaml` to improve readability and ensure proper handling of draft PRs. Updated retry logic in `setup/action.yaml` to reflect correct author association for stricter validations.	2025-03-06 15:02:29 +01:00
DevTekVE	a51ed062ca	CI: allow 1 retry for first time contributor and fork and skip sync lfs on fork PRs (#646 ) * Update workflow condition to skip forked PRs during sync This change ensures the LFS maintenance workflow does not run for pull requests originating from forked repositories. It enhances efficiency by avoiding unnecessary sync actions in these cases. * Modified retry conditions in GitHub workflow file Adjusted the retry conditions in the GitHub workflow setup file. The updated condition allows for a second attempt if the pull request originator is a 'FIRST_TIME_CONTRIBUTOR' from a forked repository. This modification lends more flexibility to first-time contributors by providing them with an additional chance to pass the tests.	2025-03-06 10:24:14 +01:00
DevTekVE	17d292c773	CI: Protecting /release/ci with CODEOWNERS	2025-03-06 09:38:58 +01:00
DevTekVE	8c3f8064b2	Uploading codeowners (#645 )	2025-03-06 09:34:33 +01:00