ci: update ingest fixtures from gh workflow #702
ryannikolaidis
merged
65 commits into
main
from
ryan/ci-update-ingest-test-fixtures
add generate workflow
8f9a70a4
on push for now
d72a76da
test bump and generate fix
27f18388
push
6a5d5095
overwrite
51617f28
push origin
00a6c726
on workflow dispatch
48c75266
use GH_CREATE_PR_TOKEN
98807933
fix counter for branch name
440a476c
not quiet
c0acc739
count on remote
73cd6924
don't increment
8ac9b559
use correct token
005a3d24
fix pr action
6533c565
no commit
dc39cf26
fix?
1c683740
forget suffix
38d0d7dc
missing quote
f1e7e3ae
use v5
ba0fa173
don't recurse
85dbc02f
Merge branch 'main' into ryan/ci-update-ingest-test-fixtures
c679860e
test
90014dae
quotes?
70b4db71
push again
637aaac0
if
4a84071e
refactor and filter on name of commit
7ec87e6a
ingest-test-fixtures-update
f2d993c1
ingest-test-fixtures-update
4a556ac8
ingest-test-fixtures-update
f362bd7f
ingest-test-fixtures-update
cd4e57f8
ingest-test-fixtures-update
d3f80af3
update name
16c5d3e5
better details on the PR branch
de8f00b5
ingest-test-fixtures-update
4f90c62c
try getting pr name
65ea5b6e
ingest-test-fixtures-update
d7513582
ingest-test-fixtures-update
163dea21
Merge branch 'main' into ryan/ci-update-ingest-test-fixtures
96c3e12f
exact match
5f96c3b5
ingest-test-fixtures-update
642d9b25
ingest-test-fixtures-update
c112e2f3
temporarily disable if
0d097dd6
ingest-test-fixtures-update
f7d5cdc2
re-enable if
eca22f93
ingest-test-fixtures-update
f9da61a1
print commit message
41bbd6bc
ingest-test-fixtures-update
29f89f17
disable if
41d896d1
ingest-test-fixtures-update
4250ed32
no more output (deprecated)
c6166bca
ingest-test-fixtures-update
91efc26f
ingest-test-fixtures-update
0d75d409
disable condition
b7802991
ingest-test-fixtures-update
2303feca
exact match on push
59a1f718
ingest-test-fixtures-update
273da966
don't check message on workflow_dispatch
d18b63b2
ingest-test-fixtures-update
71093377
branch and pr name to env
a9371aa5
ingest-test-fixtures-update
98540120
ingest-test-fixtures-update
7b80e76c
fix unintentional dropped code
54c8aa1b
ingest-test-fixtures-update
ef0868e6
ryannikolaidis
changed the title
undo azure test change
bdb3806e
pr title tweak
628261d1
cragwolfe
commented
on 2023-06-13
.github/workflows/ingest-test-fixtures-update-pr.yml
| 1 | name: Ingest Test Fixtures Update PR | ||
| 2 | |||
| 3 | on: | ||
| 4 | push: |
cragwolfe2 years agodoes this mean it gets triggered for every push?
ryannikolaidis2 years agoit does, but it will get cancelled if the commit message doesn't contain "ingest-test-fixtures-update". I couldn't find a better way to do trigger this on commit, but totally open if you have other thoughts?
cragwolfe2 years ago👍 1ah, i see. works for now!
cragwolfe
approved these changes
on 2023-06-13
Login to write a write a comment.
Adds a ingest-test-fixtures-update-pr workflow. It will generate a PR that targets the triggering branch, but includes any changes as a result of running the test ingest fixtures script (with OVERWRITE=true). It can be triggered on workflow_dispatch (which, once merged to main, should add a dispatch button to the workflow in the Actions menu). It can also be triggered with any commit containing the message: "ingest-test-fixtures-update". So one can also trigger with:
git commit -m "ingest-test-fixtures-update" --allow-empty.The created PR includes the original author as assignee and as reviewer. It is authored by Unstructured-DevOps.
Note: because we trigger on commit push and workflow dispatch we have to guess at the original PR name by looking all PRs from the target branch (hopefully each branch only ever has one PR, but theoretically a known edge case).
Testing
I made a change to the azure metadata-exclude (to trigger ingest expected result changes) and pushed a commit with the name
ingest-test-fixtures-updateto trigger creation of this PR: #731