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

Fixed the scripts and added a read me #54

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

FFranza
Copy link

@FFranza FFranza commented Dec 6, 2023

Hey Professor Jared!

It's Frederick, and I'm not sure if you've been informed by Professor Matt, but me and Sebastian have fully setup the range in our server. There were some issues though in regards to 2 scripts in particularly, you should be able to see it in the commits. I also documented and created a read me for future usage towards other environments. It should be fine in regards to a general environment, if anything needs to be changed please do reach out!

Hoping to hear from you soon!

Copy link

Choose a reason for hiding this comment

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

The changes on lines 17-19 apply to the pre-built OVA image only. The original did not work for us, but we also see that the script is written this way for users who are building it.

These are the changes we had to make when using the OVA image.

Copy link

Choose a reason for hiding this comment

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

Made changes to this document and sent them to you. Cheers!

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