[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