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

Make leap-dev download target configurable. #217

Merged
merged 2 commits into from
Sep 6, 2023

Conversation

oschwaldp-oci
Copy link
Contributor

@oschwaldp-oci oschwaldp-oci commented Sep 5, 2023

Change Description

Make leap-dev download target configurable.

Use defaults.json to specify default location/target to attempt to download the leap-dev package from. Allow dynamic configuration through workflow_dispatch input in github.

Resolves: #216

For example:

The run here: Manual Build & Test Run was manually dispatched from cdt branch cicd-override-leap-dev-target with the Override leap-dev target input set to hotstuff_integration. This can be seen in the Build & Test (ubuntu20) output in the step Download leap-dev.deb (Ubuntu20 only):

Run AntelopeIO/asset-artifact-download-action@v3
  with:
    owner: AntelopeIO
    repo: leap
    file: leap-dev.*(x86_64|amd64).deb
    target: hotstuff_integration
    prereleases: false
    artifact-name: leap-dev-ubuntu20-amd64
    container-package: experimental-binaries
    token: ***
    fail-on-missing-target: true

Use defaults.json to specify default location/target to attempt to downlaod the leap-dev package from. Allow dynamic configuration through workflow_dispatch input in github.
@oschwaldp-oci oschwaldp-oci changed the title **WIP** Make leap-dev download target configurable. Make leap-dev download target configurable. Sep 5, 2023
@oschwaldp-oci oschwaldp-oci added the OCI Work exclusive to OCI team label Sep 5, 2023
with:
owner: AntelopeIO
repo: leap
file: 'leap-dev.*(x86_64|amd64).deb'
target: main
target: '${{needs.versions.outputs.leap-dev-target}}'
prereleases: ${{fromJSON(needs.versions.outputs.leap-dev-prerelease)}}
artifact-name: leap-dev-ubuntu20-amd64
container-package: experimental-binaries
token: ${{github.token}}
Copy link
Member

Choose a reason for hiding this comment

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

with v3 you can get rid of this line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the reminder. I keep forgetting that is no longer required with v3. Removed.

@oschwaldp-oci oschwaldp-oci merged commit 4673b45 into main Sep 6, 2023
@oschwaldp-oci oschwaldp-oci deleted the cicd-override-leap-dev-target branch September 6, 2023 00:01
@ericpassmore ericpassmore added this to the CDT 4.1.0-rc1 milestone Dec 21, 2023
@ericpassmore
Copy link
Contributor

Note:start
group: RELEASE
category: CHORE
summary: Support configurable leap target for CICD.
Note:end

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCI Work exclusive to OCI team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add possibility of specifying different target for integration tests
4 participants