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

Set correct state when creating task with no funds #5137

Merged
merged 4 commits into from
Mar 19, 2020

Conversation

kmazurek
Copy link
Contributor

Resolves: https://github.com/golemfactory/golem/issues/5012

When task creation fails due to insufficient funds, the task's state is
now set to errorCreating. This commit also changes that state to be
considered final.

When task creation fails due to insufficient funds, the task's state is
now set to errorCreating. This commit also changes that state to be
considered final.
@kmazurek kmazurek requested review from jiivan and shadeofblue March 17, 2020 20:52
@kmazurek kmazurek requested a review from maaktweluit as a code owner March 17, 2020 20:52
@kmazurek kmazurek self-assigned this Mar 17, 2020
Copy link
Contributor

@jiivan jiivan left a comment

Choose a reason for hiding this comment

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

lgtm

@codecov
Copy link

codecov bot commented Mar 18, 2020

Codecov Report

Merging #5137 into b0.23 will increase coverage by 0.03%.
The diff coverage is 90.9%.

@@            Coverage Diff            @@
##           b0.23    #5137      +/-   ##
=========================================
+ Coverage   89.9%   89.94%   +0.03%     
=========================================
  Files        238      238              
  Lines      22438    22443       +5     
=========================================
+ Hits       20174    20186      +12     
+ Misses      2264     2257       -7

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!

@kmazurek kmazurek merged commit b738487 into b0.23 Mar 19, 2020
@kmazurek kmazurek deleted the task-creation-missing-funds branch March 19, 2020 13:50
maaktweluit pushed a commit that referenced this pull request Mar 25, 2020
When task creation fails due to insufficient funds, the task's state is
now set to errorCreating. This commit also changes that state to be
considered final.
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants