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

Documentation updates for data directory deletion/backup on installing a new version #1541

Merged
merged 3 commits into from
Oct 21, 2021

Conversation

shreyamalviya
Copy link
Contributor

What does this PR do?

Related to #1114

PR Checklist

  • Have you added an explanation of what your changes do and why you'd like to include them?
  • Is the TravisCI build passing?
  • Was the CHANGELOG.md updated to reflect the changes?
  • Was the documentation framework updated to reflect the changes?

Testing Checklist

  • Added relevant unit tests?
  • Have you successfully tested your changes locally? Elaborate:

    Tested by running Hugo

  • If applicable, add screenshots or log transcripts of the feature working

image

@codecov
Copy link

codecov bot commented Oct 21, 2021

Codecov Report

Merging #1541 (edcc41e) into develop (19cce1f) will not change coverage.
The diff coverage is n/a.

❗ Current head edcc41e differs from pull request most recent head a43c1b0. Consider uploading reports for the commit a43c1b0 to get more accurate results
Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1541   +/-   ##
========================================
  Coverage    43.45%   43.45%           
========================================
  Files          480      480           
  Lines        14285    14285           
========================================
  Hits          6208     6208           
  Misses        8077     8077           

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 19cce1f...a43c1b0. Read the comment docs.

Comment on lines 40 to 57
## I updated to a new version of the Infection Monkey and I'm being asked to delete my existing data directory. Why?

The [data directory]({{< ref "/reference/data_directory" >}})contains the
Infection Monkey's database and other internal
data. For the new version of Infection Monkey to work flawlessly, a data
directory with a compatible structure needs to be set up.

If you would like to save the data gathered from the Monkey's previous runs,
you can make a backup of your [existing data directory]({{< ref
"/reference/data_directory" >}}) before deleting it.

## How can I use an old data directory?

To use the data stored in a data directory from an older version, reinstall the
version of the Monkey Island which matches your data directory's version. Then,
copy the backup of your old data directory to the [appropriate location]({{<
ref "/reference/data_directory" >}}).

Copy link
Contributor

Choose a reason for hiding this comment

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

I feel like this info should be in setup. In setup pages of appimage and MSI we should display a link: "If you are installing an Infection Monkey update you might be prompted to delete data directory". This doesn't concern docker, right?

Copy link
Contributor

Choose a reason for hiding this comment

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

FAQ is already big, we should reserve it for frequent questions only. Let's move it to FAQ is we actually get asked about this

Copy link
Collaborator

Choose a reason for hiding this comment

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

Info boxes referencing these FAQs have been added to the Linux and Windows setup pages.

@mssalvatore mssalvatore merged commit e5b1302 into develop Oct 21, 2021
@mssalvatore mssalvatore deleted the 1530-docs-use-backup-data-dir branch October 21, 2021 12:54
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.

3 participants