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

Antonios exercise #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Antonios exercise #1

wants to merge 1 commit into from

Conversation

antoniogomezalvarado
Copy link
Owner

@antoniogomezalvarado antoniogomezalvarado commented Oct 21, 2017

Hi! Nice to e-meet you :)

I've added three roles:

  • img-panda -> role that contains all the relevant files associated with the img-panda application
  • smart-panda -> role that contains all the relevant files associated with the smart-panda application
  • python -> to include python dependency for both services above

A couple of pointers:

  • Access the services using localhost:8080 for img-panda and localhost:8888 for smart-panda
  • Obviously there can always be some enhancements - such as adding a rollback task in case verification fails. Wasn't straight forward as Ansible is new to me.
  • Flow is -> copy app from assuming verified and tested env to production host, stop current version of app running, start new version and verify the service is available
  • EDIT: I've edited this a couple of times :) - regarding the services crashing - reading out the serve_forever() method is a built in loop ( according to the python documentation https://docs.python.org/2/library/socketserver.html ) so this should be pretty robust or am I missing something out ? Crash in case the host goes down ? Which could be solved by adding the service to /etc/init.d to make sure its up on boot time

Thanks!

I'd be glad to get any inputs for future improvement :)
Antonio

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