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.

Sign up for GitHub

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

Jump to bottom

139 add integration tests for ci #169

Merged
merged 44 commits into from
Nov 29, 2024
Merged

139 add integration tests for ci #169

merged 44 commits into from
Nov 29, 2024

Conversation

JavierCladellas
Copy link
Collaborator

@JavierCladellas JavierCladellas commented Nov 29, 2024 •
edited
Loading

❗ IMPORTANT ❗
To be reviewed AFTER #159, #150, #119

Done:

  • Added a parallelSum test that will be used as a Hello World benchmarking program. This test should also be used as a clear example in docs
  • CI executes this test and verifies that report files where created (at the moment, performance checking with threshold is not implemented Add reference values and thresholds #154 )
  • Added a gaya_ci.json config for more general testing in gaya. (outputs are stored inside the repo's root directory.)
  • Fixed bug in sanity checks (there was no special character escaping )

@JavierCladellas
add parallelSum program #139
8a2cf58
@JavierCladellas
add global sum to outputs #139
04abe9c
@JavierCladellas
add test config #139
0fbf937
@JavierCladellas
Merge branch '126-handle-custom-website-generation' into 139-add-inte…
74ca1f5 
…gration-tests-for-ci
@JavierCladellas
fix parameter validation
30326b7
@JavierCladellas
fix use case name #139
8b152df
@JavierCladellas
fix plots #139
76090bd
@JavierCladellas
add more elts (times too low)
034036c
@JavierCladellas
try fix scaling #139
d814e69
@JavierCladellas
set exlusive access to true #139
d784618
@JavierCladellas
Merge branch '126-handle-custom-website-generation' into 139-add-inte…
5dda751 
…gration-tests-for-ci
@JavierCladellas
Merge branch '101-do-not-store-adoc-files-in-git-tree' into 139-add-i…
b8a1b26 
…ntegration-tests-for-ci
@JavierCladellas
split computation from communication
2bde0c1
@JavierCladellas
more N size
c564164
@JavierCladellas
do not plot comm time
e75e129
@JavierCladellas
format + add outputs + write success,errors
110841c
@JavierCladellas
add outputs and sanity checks
d47f668
@JavierCladellas
add default platform
09b827f
@JavierCladellas
rm platform (use default) #139
e78a580
@JavierCladellas
init testing on CI #139
0a4c31c
@JavierCladellas
add checkout #139
c814888
@JavierCladellas
comment apptainer dirs cache and tmp dirs #139
1ec9075
@JavierCladellas
Merge branch 'master' into 139-add-integration-tests-for-ci
da14d13
@JavierCladellas
fix filepath key
96f6bcf
@JavierCladellas
up command options in ci #139
f0f3381
@JavierCladellas
rm nodes
afbc15e
@JavierCladellas
to builtin
c87fc89
@JavierCladellas
Merge branch '145-handle-prog-environments' into 139-add-integration-…
2e0504d 
…tests-for-ci
@JavierCladellas
Merge branch '158-remove-launch-options-from-machine-config' into 139…
631fd71 
…-add-integration-tests-for-ci
@JavierCladellas
Merge branch '101-do-not-store-adoc-files-in-git-tree' into 139-add-i…
71d4a46 
…ntegration-tests-for-ci
@JavierCladellas JavierCladellas added enhancement A new feature or request ci-cd Continuous Integration/Continuous Deployment cb Continuous Benchmarking labels Nov 29, 2024
@JavierCladellas JavierCladellas self-assigned this Nov 29, 2024
@JavierCladellas JavierCladellas linked an issue Nov 29, 2024 that may be closed by this pull request
Add integration tests for CI #139
Closed
5 tasks
@netlify Netlify
Copy link

netlify bot commented Nov 29, 2024 •
edited
Loading

✅ Deploy Preview for benchmarking-polite-crostata-92f389 ready!

Name Link
🔨 Latest commit eea8eef
🔍 Latest deploy log https://app.netlify.com/sites/benchmarking-polite-crostata-92f389/deploys/6749c285e2d9ba0008940f3a
😎 Deploy Preview https://deploy-preview-169--benchmarking-polite-crostata-92f389.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@vincentchabannes vincentchabannes merged commit 0288884 into master Nov 29, 2024
10 checks passed
@vincentchabannes vincentchabannes deleted the 139-add-integration-tests-for-ci branch November 29, 2024 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vincentchabannes vincentchabannes vincentchabannes approved these changes

@prudhomm prudhomm Awaiting requested review from prudhomm

Assignees

@JavierCladellas JavierCladellas

Labels
cb Continuous Benchmarking ci-cd Continuous Integration/Continuous Deployment enhancement A new feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add integration tests for CI
2 participants
@JavierCladellas @vincentchabannes