fix: pass correct local storage prefix in case an executor spawns an inner job#3488
Conversation
📝 WalkthroughWalkthroughThe changes update the Changes
Sequence Diagram(s)sequenceDiagram
participant ArgsFactory as SpawnedJobArgsFactory
participant Executor as executor_common_settings
participant Workflow as workflow
participant Config as w2a
ArgsFactory->>Executor: Evaluate non_local_exec flag
ArgsFactory->>Workflow: Evaluate remote_exec flag
alt non_local_exec true and remote_exec false
ArgsFactory->>Config: Get "storage_settings.remote_job_local_storage_prefix"
Config-->>ArgsFactory: Return remote_storage_prefix
else
ArgsFactory->>Config: Get "storage_settings.local_storage_prefix"
Config-->>ArgsFactory: Return local_storage_prefix
end
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (34)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
Please format your code with black: |
🤖 I have created a release *beep* *boop* --- ## [9.1.3](v9.1.2...v9.1.3) (2025-03-26) ### Bug Fixes * pass correct local storage prefix in case an executor spawns an inner job ([#3488](#3488)) ([1454ca1](1454ca1)) * setup_logfile in setup logger to ensure logfile handler is added ([#3486](#3486)) ([2ed13fc](2ed13fc)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
…inner job (snakemake#3488) ### QC <!-- Make sure that you can tick the boxes below. --> * [x] The PR contains a test case for the changes or the changes are already covered by an existing test case. * [x] The documentation (`docs/`) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake). <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Refactor** - Enhanced the job submission logic by clarifying the handling of local and remote execution modes, improving the transparency and maintainability of storage settings. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
🤖 I have created a release *beep* *boop* --- ## [9.1.3](snakemake/snakemake@v9.1.2...v9.1.3) (2025-03-26) ### Bug Fixes * pass correct local storage prefix in case an executor spawns an inner job ([snakemake#3488](snakemake#3488)) ([1454ca1](snakemake@1454ca1)) * setup_logfile in setup logger to ensure logfile handler is added ([snakemake#3486](snakemake#3486)) ([2ed13fc](snakemake@2ed13fc)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
QC
docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).Summary by CodeRabbit