Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Deadline: more detailed temp file name for environment json #2824

Merged
merged 1 commit into from
Mar 3, 2022
Merged

Deadline: more detailed temp file name for environment json #2824

merged 1 commit into from
Mar 3, 2022

Conversation

kalisp
Copy link
Member

@kalisp kalisp commented Mar 1, 2022

Brief description

DL with enabled concurrent tasks fails with an error of nonexistent json. It probably gets deleted by different task.

Testing notes:

  1. enable concurrent tasks in DL
  2. publish something that triggers multiple tasks in DL

@kalisp
OL-2799 - more detailed temp file name for environment json for Deadline
6a463bf 
Previous implementation probably wasn't detailed enough.
@kalisp kalisp added type: bug Something isn't working module: Deadline AWS Deadline related features labels Mar 1, 2022
@kalisp kalisp requested a review from 64qam March 1, 2022 13:38
@kalisp kalisp self-assigned this Mar 1, 2022
@mkolar
Copy link
Member

mkolar commented Mar 1, 2022

Task linked: OP-2799 DL inject_environment fails with concurrent tasks

@kalisp kalisp marked this pull request as ready for review March 2, 2022 09:43
@kalisp kalisp merged commit dec5f14 into develop Mar 3, 2022
@kalisp kalisp deleted the bugfix/OP-2799_DL-inject_environment-fails-with-concurrent-tasks branch March 3, 2022 09:51
@mkolar mkolar added this to the 3.9.0 milestone Mar 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@64qam 64qam 64qam approved these changes

Assignees

@kalisp kalisp

Labels
module: Deadline AWS Deadline related features type: bug Something isn't working
Projects
None yet
Milestone
3.9.0
Development

Successfully merging this pull request may close these issues.
3 participants
@kalisp @mkolar @64qam