Closed Bug 915341 Opened 8 years ago Closed 8 years ago
make slaveapi packageable/installable
It'll be much easier to deploy properly if it's installable through pip. Some notes about this patch: * I've removed the vendor library because PuppetAgain handles both compiled and pure python dependencies without issue -- there's no need to embed it in the repository anymore. This also means we don't need the requirements/ directory anymore, as those have moved into setup.py. * Existing content has been moved from slaveapi/__init__.py to global_state.py so that setup.py can import the version without requiring things like gevent be installed already. * Pidfile is now defined in the .ini and the sample file uses absolute paths. This is mostly a UX thing because after pip installing slaveapi, slaveapi-server.py will be in $PATH. Given that, I didn't think it made sense to assume that the pid, log, and credentials.json are in a particular place.
Attachment #803225 - Flags: review?(jhopkins)
Attachment #803225 - Flags: review?(jhopkins) → review+
Comment on attachment 803225 [details] [diff] [review] make slaveapi installable Merged and pushed.
Attachment #803225 - Flags: checked-in+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.