Re: [LTP] [PATCH v6 4/4] ci: apply patchwork series in ci-docker-build workflow

[Cyril Hrubis <chrubis@suse.cz>]

Hi!
> +    - name: Apply Patchwork series
> +      if: inputs.SERIES_ID != '' && inputs.SERIES_MBOX != ''
> +      env:
> +        PATCHWORK_TOKEN: ${{ secrets.PATCHWORK_TOKEN }}
> +      run: |
> +        git config --global user.name 'GitHub CI'
> +        git config --global user.email 'github@example.com'
> +        git config --global --add safe.directory "$GITHUB_WORKSPACE"
> +
> +        git checkout -b review_patch_series_"${{ inputs.SERIES_ID }}"

So we create a branch for each CI run but I do not see any place where
we get rid of it. Shouldn't we remove it after the CI run?

> +        curl -k "${{ inputs.SERIES_MBOX }}" | git am
> +
> +        ./ci/tools/patchwork.sh state "${{ inputs.SERIES_ID }}" "needs-review-ack"
> +
>      - name: ver_linux
>        run: ./ver_linux
>  
> @@ -158,3 +184,14 @@ jobs:
>        run: |
>          if [ "$MAKE_INSTALL" = 1 ]; then INSTALL_OPT="-i"; fi
>          ./build.sh -r install -o ${TREE:-in} $INSTALL_OPT
> +
> +    - name: Send results to Patchwork
> +      if: always() && inputs.SERIES_ID != '' && inputs.SERIES_MBOX != ''
> +      env:
> +        PATCHWORK_TOKEN: ${{ secrets.PATCHWORK_TOKEN }}
> +      run: |
> +        ./ci/tools/patchwork.sh check \
> +          "${{ inputs.SERIES_ID }}" \
> +          "${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}" \
> +          "${{ matrix.container }}" \
> +          "${{ job.status }}"

The rest looks good, once the branch deletion is clarified or fixed:

Reviewed-by: Cyril Hrubis <chrubis@suse.cz>

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp