Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

Tested running Cortx on OVA #895

Merged
merged 4 commits into from
Apr 15, 2021
Merged

Conversation

stx-hseow
Copy link
Contributor

Signed-off-by: Harrison Seow harrison.seow@seagate.com

Describe your changes in brief

Changes

  • Why is this change required? What problem does it solve?
  • If proposing a new change then please raise an issue first

Tested running cortx-va-1.0.3.ova on Windows 10, VMWare Workstation 16 Player.
Confirmed sanity test not failing without S3 account (issue).

How Has This Been Tested? (Optional)

  • Please describe in detail how you tested your changes.
  • Include details of your testing environment, and the tests you ran to
  • How your change affects other areas of the code, etc.

Screenshots (if appropriate)

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

Signed-off-by: Harrison Seow <harrison.seow@seagate.com>
Copy link
Contributor

@mukul-seagate11 mukul-seagate11 left a comment

Choose a reason for hiding this comment

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

looks good

@mukul-seagate11 mukul-seagate11 mentioned this pull request Apr 6, 2021
9 tasks
@stale
Copy link

stale bot commented Apr 11, 2021

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @mukul-seagate11 for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

@stale stale bot removed the needs-attention label Apr 15, 2021
@johnbent johnbent merged commit 31d9f90 into Seagate:main Apr 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants