zyn71

Results 8 comments of zyn71

hi,@reneleonhardt @zzstoatzz (we have argued before) Sorry to nudge, but I wanted to gently follow-up on this PR. It’s been open for three days, the checks (CI, pre-commit, dev env)...

Hi @desertaxle — thank you for the quick feedback! Below is a **full, single-file guide** my team follows to test the new **VolcanoWorker** locally. Everything has been validated on real...

hi again @desertaxle !Yes, Volcano must be installed in the cluster before the `VolcanoWorker` can be used. Because this PR has not been merged yet, you also need to build...

hi @desertaxle !I totally understand your point—the first thing I tried was exactly what you’re suggesting. However, the **default manifest is injected at the *work‑pool* level, not at the worker...

Hi @desertaxle! Thanks a lot for the suggestions and guidance. I’ve now added a `_get_default_volcano_job_manifest_template()` to `VolcanoWorker`, so we have anative Volcano base manifest. Following your advice, once the UI...

hi! @desertaxle,Sorry for the delayed response - I've been quite busy lately and this took longer than expected. First of all, I want to say that the refactoring work you...

Hi! Our MR hasn’t made any progress in a week. Do you have any suggestions, or is there anything else I can do to move it forward or improve it?

hi,its been a long time.can i do sth to keep the pr forward?