Skip to content
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

[CI] Refactoring: Move build-vars and get-test-matrix jobs to separate workflows #828

Draft
wants to merge 1 commit into
base: default
Choose a base branch
from

Conversation

zakkak
Copy link
Collaborator

@zakkak zakkak commented Mar 7, 2025

De-duplicate jobs.

The motivation behind this is to ultimately build quarkus main a single time and re-use with different mandrel builds. At the moment we build quarkus for each configuration, which is a waste of time and resources.

Furthermore, we plan to introduce macOS and linux-arm64 builds to the test matrix, and re-using workflows will make this easier.

@zakkak zakkak requested a review from Karm March 7, 2025 07:16
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Mar 7, 2025
steps:
- id: suffix
run: |
export SUFFIX=$(echo '${{ toJson(inputs) }}' | jq -j 'del(."build-stats-tag", ."mandrel-it-issue-number", ."issue-repo", ."issue-number") | to_entries[] | "-\(.value)"' | tr '":<>|*?\r\n\/' '-')
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This part needs to be enhanced to allow for generating different suffices depending on more inputs than just the inputs of this reusable workflow, currently results in

Error: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run

@zakkak zakkak marked this pull request as draft March 7, 2025 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant