This repository has been archived by the owner on Dec 16, 2019. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Separate the OS (packages + python install) from actual source code in order to streamline build based on an idea by @silverdaz, an effort to improve: #348
Part of this PR:
7.5.1804
ega-os
,ega-base
,ega-openssh
andega-inbox
ega-os
andega-openssh
should be built only if theDockerfile
has changed, we useextras/travis.sh
for this.ega-base
the LocalEGA app, probably the most frequently changed.ega-inbox
makes use of of multi stage builds and usesega-base
andega-openssh
images. Should be built as theega-base
changes.No. Image names and configuration stayed the same.
As it stands check the
extras/travis.sh
and after merge we should do two additional PRs one with aimages/os/Dockerfile
change and one without.Yes indeed, but there was a bug #225 and we fixed the version. In the meantime rabbitmq 3.7 changed their configuration formatting: http://www.rabbitmq.com/blog/2018/02/22/new-configuration-format-in-rabbitmq-3-7/ . If enough interest and depending on CEGA version we might change it. Mostly because of security updates.
We should increase travis
sleep
, run again that job or be smart about it and modify the integration tests and make it wait for services to be up.