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

resolves: 181 handle report deployment #183

Merged
merged 65 commits into from
Dec 13, 2024
Merged

resolves: 181 handle report deployment #183

merged 65 commits into from
Dec 13, 2024

Conversation

JavierCladellas
Copy link
Collaborator

@JavierCladellas JavierCladellas commented Dec 6, 2024 •
edited
Loading

❗ IMPORTANT ❗
Merge AFTER #182

The workflow is the following :

  1. After a new benchmark is executed (through workflow or repo dispatch) , benchmarks are uploaded to a girder staging/ folder. A new PR is created (tagged with new-benchmark)
  2. When a PR tagged with new-benchmark is created, a deploy preview is available for ONLY the reports under the staging directory.
  3. IF the PR is accepted and merged, reports will be copied to a production/ folder, then moved to merged/, and the website config will be updated and reuploaded.
  4. IF the PR is closed without merging, staging reports will be moved to a denied/ folder. No changes will be made to the production website config file.

@JavierCladellas JavierCladellas linked an issue Dec 6, 2024 that may be closed by this pull request
Handle report deployment #181
Closed
4 tasks
@JavierCladellas JavierCladellas self-assigned this Dec 6, 2024
@netlify Netlify
Copy link

netlify bot commented Dec 6, 2024 •
edited
Loading

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

Name Link
🔨 Latest commit 236d194
🔍 Latest deploy log https://app.netlify.com/sites/benchmarking-polite-crostata-92f389/deploys/675bfe7eb77cbd000882f64f
😎 Deploy Preview https://deploy-preview-183--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.

@JavierCladellas JavierCladellas added enhancement A new feature or request step: deploy deployement step github_actions Pull requests that update GitHub Actions code ci-cd Continuous Integration/Continuous Deployment and removed new-benchmark labels Dec 9, 2024
@JavierCladellas JavierCladellas marked this pull request as ready for review December 10, 2024 11:38
JavierCladellas and others added 12 commits December 10, 2024 12:59
@JavierCladellas
rm tmp files from git tree... #181
3b62c8a 
[ci skip]
(Oopsie)
@JavierCladellas
Merge branch 'master' into 181-handle-report-deployment
82b4dd5
@JavierCladellas @github-actions
[create-pull-request] automated change
80bc4a6
@JavierCladellas
trigger netlify
7b71257
@JavierCladellas
fix merge-json-config args #181
d61ca70
@JavierCladellas
Merge branch '181-handle-report-deployment' into new-benchmark-feelpp…
de63fb1 
…_toolbox_heat
@JavierCladellas
try fix label recognition [ci skip]
0e91697
@JavierCladellas
Merge branch '181-handle-report-deployment' into new-benchmark-feelpp…
cc61ece 
…_toolbox_heat
@JavierCladellas
forgot girder api key #181 :p [ci skip]
9b82dc2
@JavierCladellas
Merge branch '181-handle-report-deployment' into new-benchmark-feelpp…
1e2a603 
…_toolbox_heat
@JavierCladellas
Merge pull request #187 from feelpp/new-benchmark-feelpp_toolbox_heat
6b4247a 
Add benchmark for feelpp_toolbox_heat - ThermalBridgesENISO10211
@JavierCladellas
Merge branch 'master' into 181-handle-report-deployment
236d194
@vincentchabannes vincentchabannes merged commit 282233a into master Dec 13, 2024
10 checks passed
@vincentchabannes vincentchabannes deleted the 181-handle-report-deployment branch December 13, 2024 10:21
@JavierCladellas JavierCladellas mentioned this pull request Dec 13, 2024
Closed
1 task
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 github_actions Pull requests that update GitHub Actions code step: deploy deployement step
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Handle report deployment Remove website config from git repo
2 participants
@JavierCladellas @vincentchabannes