Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Update to Task API v0.23.0 #4848

Merged
merged 2 commits into from
Nov 7, 2019
Merged

Conversation

Wiezzel
Copy link

@Wiezzel Wiezzel commented Nov 4, 2019

Task API v0.23.0 provider AbortSubtask call which is now invoked when a subtask times out. Subtasks in verification are not timed out. Updated the Blender app to v0.6.0 which is compatible with Task API v0.23.0

Copy link
Contributor

@maaktweluit maaktweluit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

One small comment, asked this q before and got this reply from @mfranciszkiewicz

golem/apps/default.py Show resolved Hide resolved
golem/task/requestedtaskmanager.py Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Nov 5, 2019

Codecov Report

Merging #4848 into b0.22 will decrease coverage by 0.15%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##            b0.22    #4848      +/-   ##
==========================================
- Coverage   90.13%   89.98%   -0.16%     
==========================================
  Files         231      231              
  Lines       21267    21277      +10     
==========================================
- Hits        19170    19147      -23     
- Misses       2097     2130      +33

@mfranciszkiewicz
Copy link
Contributor

Please change the target branch to develop

@Wiezzel Wiezzel changed the base branch from mwu/e2e-rtm to develop November 6, 2019 10:11
@Wiezzel
Copy link
Author

Wiezzel commented Nov 7, 2019

@maaktweluit Do you accept?

@Wiezzel Wiezzel changed the base branch from develop to b0.22 November 7, 2019 12:20
Task API v0.23.0 provider AbortSubtask call which is now invoked when
a subtask times out. Subtasks in verification are not timed out. Updated
the Blender app to v0.6.0 which is compatible with Task API v0.23.0

Signed-off-by: Adam Wierzbicki <awierzbicki@golem.network>
To avoid updating the test every time the app definition changes.

Signed-off-by: Adam Wierzbicki <awierzbicki@golem.network>
@mfranciszkiewicz
Copy link
Contributor

Merging since only the Task API computation flow is affected

@mfranciszkiewicz mfranciszkiewicz merged commit be40a42 into b0.22 Nov 7, 2019
@Wiezzel Wiezzel deleted the abort_subtask_on_timeout branch November 7, 2019 14:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants