Actions Artifacts v4 backend

[ChristopherHX]

Fixes #28853

Needs both https://gitea.com/gitea/act_runner/pulls/473 and https://gitea.com/gitea/act_runner/pulls/471 on the runner side and patched actions/upload-artifact@v4 / actions/download-artifact@v4, like christopherhx/gitea-upload-artifact@v4 and christopherhx/gitea-download-artifact@v4, to not return errors due to GHES not beeing supported yet.

---

Known issues

• official action v4 doesn't allow replace, this is not checked
  • to check this, we need to know if the artifact has been created in the current run_attempt of the job
• added fixture uses some invalid task token data, which are not used by the test and required by constraints
  • I'm not shure how to generate these fields for a fixture

Open Questions

• Can we Append Blockstorage?
  • merging chunks like now is inefficient
  • use the url of the backend to upload via a signed url?
  • GitHub returns the azure Blockstorage url for direct upload without any proxy like used in this PR

Workflow used for testing purposes

on:
  push:
jobs:
  _5:
    runs-on: self-hosted
    steps: 
    - run: env
    - run: |
        github:
        ${{ tojson(github) }}
        inputs:
        ${{ tojson(inputs) }}
        matrix:
        ${{ tojson(matrix) }}
        needs:
        ${{ tojson(needs) }}
        strategy:
        ${{ tojson(strategy) }}            
      shell: cp {0} context.txt
    - uses: christopherhx/gitea-upload-artifact@v4
      with:
        name: test
        path: context.txt

    - uses: christopherhx/gitea-download-artifact@v4
      with:
        name: test
        path: out
    - run: cat out/context.txt

[ChristopherHX]

I have noticed that they have added a delete Artifacts api after my initial implementation of the internal ip.

• Add methods to delete artifacts actions/toolkit#1626

My plan is to update this PR to include that endpoint and add more tests

Uploading a tar archive to the blobstorage as single is possible on GitHub Actions, however the content disposition header in the UI always names it <name>.zip

See here the summary of the changes in this backend following GitHub Actions

• https://github.com/actions/toolkit/blob/415c42d27ca2a24f3801dd9406344aaea00b7866/packages/artifact/README.md#improvements

[ChristopherHX]

I have noticed in my previous PR that the
--- is not respected to mark content that should not be part of the commit message during PR merge. So this additional content has been manually removed from my side.

[wxiaoguang]

I have noticed in my previous PR that the --- is not respected to mark content that should not be part of the commit message during PR merge. So this additional content has been manually removed from my side.

I have called for "please edit the commit message manually before merge" in the member's discord channel.

But I guess such initiative is seldom followed. (At least, when I merge, I always edit).

[ChristopherHX]

Ah I didn't know that stripping the commit message is a manual task. I rather thought that some kind of automation for this would exist that broke. Features like "merge queue / auto merge" doesn't play nicely with it. I will just copy the original pr body back after merge.