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

Limit concurrency of CI workflows #760

Merged
merged 1 commit into from
Mar 6, 2023
Merged

Conversation

ch1bo
Copy link
Member

@ch1bo ch1bo commented Mar 3, 2023

We only want to have one job per PR per workflow running at the same time. This will make all other jobs get cancelled (e.g. when restarting jobs after rebasing)


  • CHANGELOG update not needed
  • Documentation update not needed
  • Added and/or updated haddocks not needed
  • No new TODOs introduced or explained herafter

@github-actions
Copy link

github-actions bot commented Mar 3, 2023

Transactions Costs

Sizes and execution budgets for Hydra protocol transactions. Note that unlisted parameters are currently using arbitrary values and results are not fully deterministic and comparable to previous runs.

Metadata
Generated at 2023-03-06 11:33:25.766363141 UTC
Max. memory units 14000000
Max. CPU units 10000000000
Max. tx size (kB) 16384

Script summary

Name Hash Size (Bytes)
μHead N/A 4458
νInitial 9492414f8f96e8483a0b8ee268fc06a954641cb2cbaa9a8b093c2c9b 4621
νCommit 5d3f107aaa56d06188cf231941cf8163e777236a9cfdc48fd4bbfa23 2422
νHead 82f16b51e2d81c6f4d42dd7398b4713a445464902f63dfd86ffe754e 8954

Cost of Init Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 5058 14.37 5.67 0.53
2 5264 15.39 6.01 0.55
3 5462 20.66 8.11 0.62
5 5875 22.91 8.87 0.66
10 6897 36.61 14.10 0.85
36 12236 97.62 37.12 1.74

Cost of Commit Transaction

Currently only one UTxO per commit allowed (this is about to change soon)

UTxO Tx size % max Mem % max CPU Min fee ₳
1 603 15.83 6.22 0.35

Cost of CollectCom Transaction

Parties UTxO (bytes) Tx size % max Mem % max CPU Min fee ₳
1 57 816 24.42 9.92 0.45
2 114 1140 40.84 16.70 0.65
3 171 1463 59.15 24.34 0.87
4 227 1785 80.68 33.34 1.12

Cost of Close Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 672 17.83 7.80 0.38
2 813 18.39 8.07 0.39
3 970 19.78 8.84 0.42
5 1333 23.31 10.85 0.48
10 2192 30.99 15.14 0.61
50 8839 88.66 47.24 1.60

Cost of Contest Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 677 22.21 9.29 0.43
2 833 23.92 10.18 0.45
3 1032 26.21 11.46 0.49
5 1369 29.17 13.04 0.54
10 2219 38.40 17.88 0.69
44 7842 97.61 48.36 1.63

Cost of Abort Transaction

Some variation because of random mixture of still initial and already committed outputs.

Parties Tx size % max Mem % max CPU Min fee ₳
1 5170 29.53 12.62 0.71
2 5493 48.00 20.73 0.93
3 5816 70.22 30.52 1.19
4 6283 98.41 43.06 1.53

Cost of FanOut Transaction

Involves spending head output and burning head tokens. Uses ada-only UTxO for better comparability.

Parties UTxO UTxO (bytes) Tx size % max Mem % max CPU Min fee ₳
5 0 0 5074 10.59 4.43 0.49
5 1 57 5115 12.12 5.31 0.51
5 5 285 5259 18.97 9.14 0.60
5 10 569 5432 26.35 13.44 0.70
5 20 1137 5799 41.36 22.15 0.90
5 30 1707 6156 56.37 30.86 1.10
5 40 2280 6519 71.89 39.78 1.30
5 50 2849 6876 87.27 48.65 1.51
5 58 3303 7164 99.39 55.67 1.67

@github-actions
Copy link

github-actions bot commented Mar 3, 2023

Test Results

281 tests   - 11   275 ✔️  - 11   18m 7s ⏱️ -36s
  94 suites  -   4       6 💤 ±  0 
    4 files    -   1       0 ±  0 

Results for commit 78a5ebd. ± Comparison against base commit feaea6a.

This pull request removes 11 tests.
Hydra.TUI.Options ‑ parses --cardano-signing-key option
Hydra.TUI.Options ‑ parses --connect option
Hydra.TUI.Options ‑ parses --network-id option
Hydra.TUI.Options ‑ parses --node-socket option
Hydra.TUI/end-to-end smoke tests ‑ display feedback long enough
Hydra.TUI/end-to-end smoke tests ‑ doesn't allow multiple initializations
Hydra.TUI/end-to-end smoke tests ‑ starts & renders
Hydra.TUI/end-to-end smoke tests ‑ supports the full Head life cycle
Hydra.TUI/end-to-end smoke tests ‑ supports the init & abort Head life cycle
Hydra.TUI/text rendering errors ‑ should show not enough fuel message and suggestion
…

We only want to have one job per PR per workflow running at the same
time. This will make all other jobs get cancelled (e.g. when restarting
jobs after rebasing)
@ch1bo ch1bo force-pushed the ch1bo/limit-ci-concurrency branch from 78a5ebd to 94d807d Compare March 6, 2023 11:23
@ch1bo ch1bo merged commit 502432c into master Mar 6, 2023
@ch1bo ch1bo deleted the ch1bo/limit-ci-concurrency branch March 6, 2023 11:24
@pgrange pgrange added this to the 0.10.0 milestone May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants