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

[Petr Vorel <pvorel@suse.cz>]

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

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