[LTP] [COMMITTED][PATCH] ci/wiki-mirror: Fix triggering CI

xuyang2018.jy@fujitsu.com xuyang2018.jy@fujitsu.com
Tue Jan 18 02:11:02 CET 2022


Hi Petr

Now, I understand, thanks for your kindly explanation.

Best Regards
Yang Xu
> Hi Xu,
>
>>> FYI ba99b81708 ("doc: Fix typo missing s in .all_filesystems")
>>> verified it's working:
>
>>> https://github.com/linux-test-project/ltp/actions/runs/1707090366
>
>> I found this problem when doc/* patch isn't the last patch of a
>> patchset.  But here it is a single patch.
> Path check was removed in 820cfe539b ("ci/wiki-mirror: Don't check path").
>
> Paths should be working not only on last commit, IMHO it evaluates all files in
> push
> https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#onpushpull_requestpull_request_targetpathspaths-ignore
> "When using the push and pull_request events, you can configure a workflow to
> run based on what file paths are changed. Path filters are not evaluated for
> pushes of tags."
>
> =>  it should be working even If I revert 820cfe539b, because running that
> workflow does not take long.
>
>
> Verification it's working on my local repo:
>
> I pushed few commits in my local branch:
> https://github.com/pevik/ltp/commits/tmp
> the last commit was on .github/workflows/wiki-mirror.yml adding:
> +    paths:
> +      - 'doc/**'
>
> commit only in that branch (not in upstream):
> aa7ec06d22 ("Revert "ci/wiki-mirror: Don't check path"")
>
> Other commit was modifying repo to 'pevik/ltp'
>
> https://github.com/pevik/ltp/actions/runs/1699238259/workflow
> ...
> on:
>    push:
>      branches:
>        - master
>        - tmp
>      paths:
>        - 'doc/**'
>
> jobs:
>    mirror:
>      runs-on: ubuntu-latest
>      if: ${{ github.repository == 'pevik/ltp' }}
> ...
>
> and that push triggered CI:
> https://github.com/pevik/ltp/actions/runs/1699228985
> (failed because my local repo does not have rights)
>
> Kind regards,
> Petr


More information about the ltp mailing list