-
Notifications
You must be signed in to change notification settings - Fork 73
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
[PH] Create workflow to test backward compatibility of performance harness #1356
Merged
Merged
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
fe05332
Create workflow to test backward compatibility of performance harness…
oschwaldp-oci c42f25b
Merge branch 'main' into GH-1156-ph-cicd-nodeos-versions
oschwaldp-oci f848de5
Try to placate different version artifact naming conventions.
oschwaldp-oci 3a72042
missed the extract step
oschwaldp-oci 2e90e39
Do not run tests in parallel.
oschwaldp-oci cd69265
Use low-tier for longer run time
oschwaldp-oci a2aa513
Split up tests into individual runs.
oschwaldp-oci 258fefc
Tests not found - move change dir build into first test.
oschwaldp-oci 99d2316
Add regex flag to ctest.
oschwaldp-oci 91cf4db
Give tests a little longer to complete.
oschwaldp-oci d42f0e7
Split test scenarios into separate jobs.
oschwaldp-oci 58b0f32
Revert "Split test scenarios into separate jobs."
oschwaldp-oci 5499dd0
Always run test scenarios.
oschwaldp-oci ea37b69
make sure to change to the build dir.
oschwaldp-oci 3a0960e
Use full nodeosVersion.
oschwaldp-oci 441f5d3
Read only transaction feature support was added in 4.0, don't test in…
oschwaldp-oci e6ea54b
Read only transaction feature support was added in 4.0, remove unsupp…
oschwaldp-oci 8cb771a
Fix multiple if statements in steps.
oschwaldp-oci ff1a837
Shouldn't run read only tests in if < v4.0
oschwaldp-oci 800a37e
Collapse perf tests back into regular expressions to simplifly and re…
oschwaldp-oci c63042b
Try reducing steps using regex for deb name matching across versions.
oschwaldp-oci cb447c5
Try removing workaround, as it shouldn't be necessary here.
oschwaldp-oci b7dea9b
Install the leap deb package to bring along dependencies and then use…
oschwaldp-oci bc46133
Working to fix install of leap deb pkg.
oschwaldp-oci b2b22ab
Cleanup.
oschwaldp-oci 265ee66
Sanity check nodeos version.
oschwaldp-oci 1a3ef7d
Merge branch 'main' into GH-1156-ph-cicd-nodeos-versions
oschwaldp-oci 4f158de
Try breaking out build steps into reusable workflow.
oschwaldp-oci 8044aa1
Needs to use this branch's reusable workflow for now, not main
oschwaldp-oci de40f8a
Inherit secrets.
oschwaldp-oci 7df3af7
Provide packages write permission for Build Platforms job.
oschwaldp-oci cf7ae29
Give packages write permissions.
oschwaldp-oci 7be77cd
Convert build.yaml to use reusable build-base workflow.
oschwaldp-oci d46a16a
Merge branch 'main' into GH-1156-ph-cicd-nodeos-versions
oschwaldp-oci 8eb51fc
Merge branch 'main' into GH-1156-ph-cicd-nodeos-versions
oschwaldp-oci 477f62c
Remove hardcoded org and repo and ref.
oschwaldp-oci e0b3caa
Merge branch 'main' into GH-1156-ph-cicd-nodeos-versions
oschwaldp-oci 495d6ea
Try to rework permissions to leave top level read, but pass write to …
oschwaldp-oci 0da3e7b
Fix path to workflow.
oschwaldp-oci c3db754
Merge branch 'main' into GH-1156-ph-cicd-nodeos-versions
oschwaldp-oci 4e19e36
Somehow put the wrong permission here.
oschwaldp-oci 56c0027
fix permission.
oschwaldp-oci b59fede
Merge branch 'main' into GH-1156-ph-cicd-nodeos-versions
oschwaldp-oci bfa1685
Add names to jobs for better UI experience.
oschwaldp-oci File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,87 @@ | ||
name: "Build leap" | ||
|
||
on: | ||
workflow_dispatch: | ||
workflow_call: | ||
outputs: | ||
p: | ||
description: "Discovered Build Platforms" | ||
value: ${{ jobs.d.outputs.p }} | ||
|
||
permissions: | ||
packages: read | ||
contents: read | ||
|
||
defaults: | ||
run: | ||
shell: bash | ||
|
||
jobs: | ||
d: | ||
name: Discover Platforms | ||
runs-on: ubuntu-latest | ||
outputs: | ||
missing-platforms: ${{steps.discover.outputs.missing-platforms}} | ||
p: ${{steps.discover.outputs.platforms}} | ||
steps: | ||
- name: Discover Platforms | ||
id: discover | ||
uses: AntelopeIO/discover-platforms-action@v1 | ||
with: | ||
platform-file: .cicd/platforms.json | ||
password: ${{secrets.GITHUB_TOKEN}} | ||
package-name: builders | ||
|
||
build-platforms: | ||
name: Build Platforms | ||
needs: d | ||
if: needs.d.outputs.missing-platforms != '[]' | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
platform: ${{fromJSON(needs.d.outputs.missing-platforms)}} | ||
runs-on: ["self-hosted", "enf-x86-beefy"] | ||
permissions: | ||
packages: write | ||
contents: read | ||
steps: | ||
- name: Login to Container Registry | ||
uses: docker/login-action@v2 | ||
with: | ||
registry: ghcr.io | ||
username: ${{github.repository_owner}} | ||
password: ${{secrets.GITHUB_TOKEN}} | ||
- name: Build and push | ||
uses: docker/build-push-action@v3 | ||
with: | ||
push: true | ||
tags: ${{fromJSON(needs.d.outputs.p)[matrix.platform].image}} | ||
file: ${{fromJSON(needs.d.outputs.p)[matrix.platform].dockerfile}} | ||
|
||
Build: | ||
name: Build leap | ||
needs: [d, build-platforms] | ||
if: always() && needs.d.result == 'success' && (needs.build-platforms.result == 'success' || needs.build-platforms.result == 'skipped') | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
platform: [ubuntu20, ubuntu22] | ||
runs-on: ["self-hosted", "enf-x86-beefy"] | ||
container: ${{fromJSON(needs.d.outputs.p)[matrix.platform].image}} | ||
steps: | ||
- uses: actions/checkout@v3 | ||
with: | ||
submodules: recursive | ||
- name: Build | ||
id: build | ||
run: | | ||
# https://github.com/actions/runner/issues/2033 | ||
chown -R $(id -u):$(id -g) $PWD | ||
cmake -B build -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/usr -GNinja | ||
cmake --build build | ||
tar -pc --exclude "*.o" build | zstd --long -T0 -9 > build.tar.zst | ||
- name: Upload builddir | ||
uses: AntelopeIO/upload-artifact-large-chunks-action@v1 | ||
with: | ||
name: ${{matrix.platform}}-build | ||
path: build.tar.zst |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this job (that uses the reusable workflow) doesn't have a name it seems to show up as simply
build-base
in the GUI in various places. I'm a little surprises it doesn't inherit the callee workflow name. Maybe we could consider giving this a name (even justBuild
) to show up nicer.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated with name


Run Build Workflow
: