Skip to content

improvement(CI): switch to proper rust caching#2303

Merged
onur-ozkan merged 5 commits intodevfrom
improve-caching
Dec 26, 2024
Merged

improvement(CI): switch to proper rust caching#2303
onur-ozkan merged 5 commits intodevfrom
improve-caching

Conversation

@onur-ozkan
Copy link
Copy Markdown

@onur-ozkan onur-ozkan commented Dec 23, 2024

self-explanatory (see commit by commit if it helps)

Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
@borngraced
Copy link
Copy Markdown

Can you try pushing multiple commits at different intervals and see if it will trigger cache issue again.. cc @onur-ozkan

@onur-ozkan
Copy link
Copy Markdown
Author

I think it will never be enough to know if this fixes the problem entirely. There is one way of finding it out, which is merging it to dev.

Consider this as a general improvement on caching, as you can see build times are reduced by couple minutes.

@onur-ozkan onur-ozkan merged commit 7c4bc27 into dev Dec 26, 2024
@onur-ozkan onur-ozkan deleted the improve-caching branch December 26, 2024 06:58
dimxy pushed a commit to dimxy/komodo-defi-framework that referenced this pull request Jan 17, 2025
* dev: (35 commits)
  fix(crypto): allow non bip39 mnemonics storage (GLEECBTC#2312)
  fix(legacy_swap): check for existing maker/taker payment before timeout (GLEECBTC#2283)
  feat(tendermint): validators RPC (GLEECBTC#2310)
  chore(CI): validate Cargo lock file (GLEECBTC#2309)
  test(P2P): add test for peer time sync validation (GLEECBTC#2304)
  fix mm2_p2p dev build (GLEECBTC#2311)
  update Cargo.lock (GLEECBTC#2308)
  chore(CI): unlock wasm-pack version (GLEECBTC#2307)
  add `wasm` feature on WASM for timed-map (GLEECBTC#2306)
  replace broken rpc link (GLEECBTC#2305)
  chore(eth-websocket): remove some unnecessary wrappers (GLEECBTC#2291)
  improvement(CI): switch to proper rust caching (GLEECBTC#2303)
  fix(wasm): add test-ext-api feature to mm2_main and mm2_bin_lib tomls (GLEECBTC#2295)
  chore(ci): Update docker build for wasm (GLEECBTC#2294)
  chore(p2p): follow-up nits (GLEECBTC#2302)
  feat(p2p): ensure time synchronization in the network (GLEECBTC#2255)
  bump libp2p (GLEECBTC#2296)
  chore(adex-cli): use "Komodo DeFi Framework" name in adex_cli  (GLEECBTC#2290)
  chore(ctx): replace gstuff constructible with oncelock (GLEECBTC#2267)
  don't rely on core (GLEECBTC#2289)
  ...
shamardy pushed a commit that referenced this pull request Jan 28, 2025
* switch to proper rust caching

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* fix linting

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* rename cargo-cache to build-cache

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* try caching

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* switch to rust-cache

Signed-off-by: onur-ozkan <work@onurozkan.dev>

---------

Signed-off-by: onur-ozkan <work@onurozkan.dev>
shamardy pushed a commit that referenced this pull request Jan 30, 2025
* switch to proper rust caching

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* fix linting

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* rename cargo-cache to build-cache

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* try caching

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* switch to rust-cache

Signed-off-by: onur-ozkan <work@onurozkan.dev>

---------

Signed-off-by: onur-ozkan <work@onurozkan.dev>
shamardy pushed a commit that referenced this pull request Jan 30, 2025
* switch to proper rust caching

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* fix linting

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* rename cargo-cache to build-cache

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* try caching

Signed-off-by: onur-ozkan <work@onurozkan.dev>

* switch to rust-cache

Signed-off-by: onur-ozkan <work@onurozkan.dev>

---------

Signed-off-by: onur-ozkan <work@onurozkan.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants