Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow resuming OSPd-based OpenVAS tasks (9.0) #869

Merged
merged 11 commits into from
Dec 3, 2019

Conversation

timopollmeier
Copy link
Member

@timopollmeier timopollmeier commented Nov 15, 2019

The new scanner type previously always started a new scan and report.
With this PR the last stopped or interrupted report is reused and only unfinished hosts are rescanned, as it was done before switching to OSP.

Checklist:

The new scanner type previously always started a new scan and report.
This commit will currently just clear the existing report and reuse it.

Scanning only unfinished hosts again or recovering scans from a still
running scanner will be handled later.
If the OSP scanner is still running or finished gvmd can still get the
results without starting a new scan.
When resuming a task where the task was stopped start the OSP scan only
for the unfinished hosts instead of rescanning everything.
The new version of the gvm-libs is required to be able to use the new
element `finished_hosts` in the `start_scan` OSP command.
While it's possible to simply continue getting the results from the
scanner if it's finished or still runnig, gvmd may have crashed while
receiving or storing the results, so some may be missing.
@timopollmeier timopollmeier marked this pull request as ready for review November 15, 2019 12:38
src/alert_methods/SCP/alert.orig Outdated Show resolved Hide resolved
This file was created and added accidentally.
@mattmundell mattmundell merged commit 51d7aa2 into greenbone:gvmd-9.0 Dec 3, 2019
@timopollmeier timopollmeier deleted the osp-resume_task-9.0 branch June 9, 2020 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants