-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update docs #32
Update docs #32
Conversation
this is a workaround to also support virtualenv
the recommended way to python apps is uwsgi, not gunicorn
this implements #24 |
@@ -24,10 +25,26 @@ Monkeying around with virtual machines and pxe configs. | |||
|
|||
## Setup | |||
|
|||
Create `marmoset.conf` before using marmoset! See `Configuration` for details. | |||
Clone the repo, for example into /opt. Than copy the service file into the systemd directory and reload systemd to recognize the file: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Clone the repository into a directory of your choice, /opt for example" , "then"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
this should be merged after #31 because it adds readme for this PR |
Copy the `marmoset.conf.example` to `marmoset.conf` and adjust the settings to your needs. | ||
Checkout the Comments in the file our our [Configuration](#configuration) section. | ||
|
||
Now we need to setup a virtualenv and install the requirement python packages (remove libvirt from the requirements.txt and `pkg-config libvirt gcc` from the list of packages to install if you don't want to manage VMs with marmoset): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"and install the required python packages"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@@ -34,9 +35,13 @@ Network: | |||
# storage name is given. | |||
Storage: | |||
|
|||
# this gets used for storing PXE options |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"this will be used for"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
in my opinion this is ready for merge now. but we should wait until we merge #31 |
(used ´´´json instead of ```json)
f455ae1
to
5b45136
Compare
👍 from @sebastianrakel on IRC |
this adds new infos to the docs about the DHCP endpoint