Skip to content

[Github][CI] Upload artifacts directory for premerge workflow #135538

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

boomanaiden154
Copy link
Contributor

The premerge pipeline currently creates an artifacts directory with some
statistics that gets uploaded on the buildkite side for later
inspection. This patch adds support for this on the Github side by using
the upload artifacts action.

Created using spr 1.3.4
@llvmbot
Copy link
Member

llvmbot commented Apr 13, 2025

@llvm/pr-subscribers-github-workflow

@llvm/pr-subscribers-bolt

Author: Aiden Grossman (boomanaiden154)

Changes

The premerge pipeline currently creates an artifacts directory with some
statistics that gets uploaded on the buildkite side for later
inspection. This patch adds support for this on the Github side by using
the upload artifacts action.


Full diff: https://github.com/llvm/llvm-project/pull/135538.diff

2 Files Affected:

  • (modified) .github/workflows/premerge.yaml (+7)
  • (modified) bolt/CMakeLists.txt (+1)
diff --git a/.github/workflows/premerge.yaml b/.github/workflows/premerge.yaml
index b7d2c7154762e..51333d9c4c397 100644
--- a/.github/workflows/premerge.yaml
+++ b/.github/workflows/premerge.yaml
@@ -65,6 +65,13 @@ jobs:
           export CXX=/opt/llvm/bin/clang++
 
           ./.ci/monolithic-linux.sh "${projects_to_build}" "${project_check_targets}" "${runtimes_to_build}" "${runtimes_check_targets}"
+            - name: "Upload artifact"
+      - name: Upload Artifacts
+        uses: actions/upload-artifact@65c4c4a1ddee5b72f698fdd19549f0f0fb45cf08 # v4.6.0
+        with:
+          name: Premerge Artifacts
+          path: artifacts/
+          retention-days: 5
 
   premerge-checks-windows:
     name: Windows Premerge Checks (Test Only - Please Ignore Results)
diff --git a/bolt/CMakeLists.txt b/bolt/CMakeLists.txt
index f5ffa81227064..b5e202156fc60 100644
--- a/bolt/CMakeLists.txt
+++ b/bolt/CMakeLists.txt
@@ -1,3 +1,4 @@
+# testing
 cmake_minimum_required(VERSION 3.20.0)
 
 set(LLVM_SUBPROJECT_TITLE "BOLT")

Created using spr 1.3.4
@boomanaiden154
Copy link
Contributor Author

(Sorry for pinging all the BOLT maintainers)

boomanaiden154 added a commit to boomanaiden154/llvm-project that referenced this pull request Apr 13, 2025
The premerge pipeline currently creates an artifacts directory with some
statistics that gets uploaded on the buildkite side for later
inspection. This patch adds support for this on the Github side by using
the upload artifacts action.

Pull Request: llvm#135538
Created using spr 1.3.4
Created using spr 1.3.4
boomanaiden154 added a commit to boomanaiden154/llvm-project that referenced this pull request Apr 13, 2025
The premerge pipeline currently creates an artifacts directory with some
statistics that gets uploaded on the buildkite side for later
inspection. This patch adds support for this on the Github side by using
the upload artifacts action.

Pull Request: llvm#135538
@boomanaiden154 boomanaiden154 merged commit 97bc913 into main Apr 14, 2025
9 checks passed
@boomanaiden154 boomanaiden154 deleted the users/boomanaiden154/githubci-upload-artifacts-directory-for-premerge-workflow branch April 14, 2025 07:37
llvm-sync bot pushed a commit to arm/arm-toolchain that referenced this pull request Apr 14, 2025
The premerge pipeline currently creates an artifacts directory with some
statistics that gets uploaded on the buildkite side for later
inspection. This patch adds support for this on the Github side by using
the upload artifacts action.

Reviewers: Keenuts, lnihlen, mizvekov, tstellar, Endilll

Reviewed By: mizvekov

Pull Request: llvm/llvm-project#135538
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants