Skip to content
This repository has been archived by the owner on Nov 29, 2021. It is now read-only.

Fix resume task. (backport #464) #470

Merged
merged 2 commits into from
Oct 13, 2021
Merged

Fix resume task. (backport #464) #470

merged 2 commits into from
Oct 13, 2021

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Oct 12, 2021

This is an automatic backport of pull request #464 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.io/

In some cases, the task is stopped and quickly resumed. In this cases there was race condition
in which the scan id is present in the scan table but there is no target information.
Although the quick resume can lead first into a stopped and scan followed by an interrupted scan,
finally the scan can be resumed.

(cherry picked from commit d4bf472)
@mergify mergify bot requested a review from a team as a code owner October 12, 2021 12:11
@codecov
Copy link

codecov bot commented Oct 12, 2021

Codecov Report

Merging #470 (37337a9) into master (9ac74d3) will decrease coverage by 0.02%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #470      +/-   ##
==========================================
- Coverage   75.28%   75.25%   -0.03%     
==========================================
  Files          23       23              
  Lines        2670     2675       +5     
==========================================
+ Hits         2010     2013       +3     
- Misses        660      662       +2     
Impacted Files Coverage Δ
ospd/scan.py 92.45% <66.66%> (-0.63%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9ac74d3...37337a9. Read the comment docs.

@y0urself y0urself merged commit 1df8875 into master Oct 13, 2021
@y0urself y0urself deleted the mergify/bp/master/pr-464 branch October 13, 2021 06:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants