Skip to content

fix: override pull(Async) in RemoteTemplateStorage#1692

Merged
derklaro merged 1 commit intonightlyfrom
pull-template-storage
Jul 20, 2025
Merged

fix: override pull(Async) in RemoteTemplateStorage#1692
derklaro merged 1 commit intonightlyfrom
pull-template-storage

Conversation

@derklaro
Copy link
Member

Motivation

RemoteTemplateStorage does currently not override pull and pullAsync leading to the methods being executed on the node. This leads to an issue where the template is not pulled into the a relative path to the current wrapper directory, but relative to the node directory.

Modification

Override pull and pullAsync to let them use the chunked data transfer infrastructure and extract the zipped template into the (possible relative to the service directory) path passed to the respective pull method.

Result

RemoteTemplateStorage has a correct implementation of pull and pullAsync.

Other context

Fixes #1303

@derklaro derklaro added this to the 4.0.0-RC14 milestone Jul 17, 2025
@derklaro derklaro requested a review from 0utplay July 17, 2025 19:33
@derklaro derklaro self-assigned this Jul 17, 2025
@derklaro derklaro added v: 4.X This pull should be included in the 4.0 release in: driver An issue/pull request releated to the driver module code t: fix A pull request introducing a fix for a bug. labels Jul 17, 2025
@github-actions
Copy link

Test Results

 56 files  ±0   56 suites  ±0   2m 46s ⏱️ ±0s
570 tests ±0  570 ✅ ±0  0 💤 ±0  0 ❌ ±0 
935 runs  ±0  935 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 9923731. ± Comparison against base commit a7046d1.

This pull request removes 34 and adds 34 tests. Note that renamed tests count towards both.
eu.cloudnetservice.driver.impl.document.DocumentSerialisationTest ‑ [4] {"b":1,"s":2,"i":3,"l":4,"f":5.0,"d":6.0,"c":"/","string":"Hello, World!","bol":true,"cloud":["Ben?","Yes","No","HoHoHoHo"],"world":{"insane":"!","hello":"world","this":"is"}}, PRETTY
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [23] 08:28:51.343299863
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [28] 08:28:51.343409378Z
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [29] 08:28:51.343429646Z
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [30] 08:28:51.343450194+05:00
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [31] 08:28:51.343465763-03:00
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [34] 2025-07-17T08:28:51.343592149
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [39] 2025-07-17T08:28:51.343720339Z
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [4] 2025-07-17T08:28:51.337132534Z
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [5] 2025-07-17T08:20:31.337136231Z
…
eu.cloudnetservice.driver.impl.document.DocumentSerialisationTest ‑ [4] {"b":1,"s":2,"i":3,"l":4,"f":5.0,"d":6.0,"c":"/","string":"Hello, World!","bol":true,"cloud":["Ben?","Yes","No","HoHoHoHo"],"world":{"insane":"!","this":"is","hello":"world"}}, PRETTY
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [23] 19:35:23.220810389
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [28] 19:35:23.220911999Z
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [29] 19:35:23.220932878Z
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [30] 19:35:23.220956713+05:00
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [31] 19:35:23.220974977-03:00
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [34] 2025-07-17T19:35:23.221091966
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [39] 2025-07-17T19:35:23.221221208Z
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [4] 2025-07-17T19:35:23.211295327Z
eu.cloudnetservice.driver.impl.document.gson.JavaTimeSerializerTest ‑ [5] 2025-07-17T19:27:03.211299194Z
…

@derklaro derklaro merged commit 5f5a71d into nightly Jul 20, 2025
6 checks passed
@derklaro derklaro deleted the pull-template-storage branch July 20, 2025 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: driver An issue/pull request releated to the driver module code t: fix A pull request introducing a fix for a bug. v: 4.X This pull should be included in the 4.0 release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TemplateStorage#pull does not work in wrapper with relative paths

2 participants