Skip to content

Make leap-dev download target configurable.#217

Merged
oschwaldp-oci merged 2 commits intomainfrom
cicd-override-leap-dev-target
Sep 6, 2023
Merged

Make leap-dev download target configurable.#217
oschwaldp-oci merged 2 commits intomainfrom
cicd-override-leap-dev-target

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
prereleases: ${{fromJSON(needs.versions.outputs.leap-dev-prerelease)}}
artifact-name: leap-dev-ubuntu20-amd64
container-package: experimental-binaries
token: ${{github.token}}
Copy link
Contributor

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