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

Integration with Docker #1

Merged
merged 7 commits into from
Apr 21, 2017
Merged

Integration with Docker #1

merged 7 commits into from
Apr 21, 2017

Conversation

sevein
Copy link
Collaborator

@sevein sevein commented Apr 2, 2017

https://jiscdev.atlassian.net/browse/RDSSARK-18

See commits for more details.

It's preferably to send the events to `stderr` and push the responsibility of
log collection to another layer (e.g. systemd, upstart, docker, awslogs...).
This commit includes some changes under the static/ folder to make sure that
the CSS files stop referencing to images that do not exist.
Copy link

@fractos fractos left a comment

Choose a reason for hiding this comment

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

The storage service logs are emitted only to console now?

@sevein
Copy link
Collaborator Author

sevein commented Apr 20, 2017

The storage service logs are emitted only to console now?

@fractos yes, that worked pretty well for Docker Compose which aggregates the standard streams of all the services. It seems that it could also work in ECS but we can also setup new handlers if needed. Would that work?

@fractos
Copy link

fractos commented Apr 20, 2017

Hi @sevein - Yes, I believe so. It looks like the container console can be routed through to a CloudWatch log - http://docs.aws.amazon.com/AmazonECS/latest/developerguide/using_awslogs.html

@sevein
Copy link
Collaborator Author

sevein commented Apr 20, 2017

Cool, I really like that idea!

Copy link

@helenst helenst left a comment

Choose a reason for hiding this comment

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

This is great! 👍 Using rdss-archivematica repo, I was able to get up and running with this (and get the shibboleth branch based onto it) very quickly. The code looks good to me.

@lower29
Copy link

lower29 commented Apr 21, 2017

I have nothing further to add, it's working for me too.

Added inotify which is still useful as a reload engine even though at the
moment it does not work with gevent.
Using dj-djatabase-url to easily setup the database backend from a single
environment variable (`SS_DB_URL`). If not declared, it fallbacks to the old
variables (`SS_DB_HOST`, etc...).

Added Python packages for MySQL and PostgreSQL support. `jsonfield` updated to
solve issues with new backends.
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.

None yet

5 participants