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.
Description
The Docker file in the omag-server-platform distribution had stopped working. The problem was that the file permissions for egeria's /deployment directory was read-only. The Dockerfile needed a --CHMOD option on the COPY command.
The servers within the container were also unable to call one another via https://localhost:9443. In addition, the Egeria servers inside the containers needed access to Apache Kafka running outside the container.
The host name
localhost
inside the docker container needs to behost.docker.internal
. Since we only want one version of the sample configuration files, they have been changed to use placeholder variables for the platform URL root in the sample servers' configuration documents.in the application.properties file for bare metal running, the placeholder properties are set to
And in the docker container, they are:
The additional advantage of this change is that it is easy to change the port and host name of egeria for more complex deployments.
Related Issue(s)
None
Testing
Running newly created docker containers after the changes.
Release Notes & Documentation
Currenlty the documentation for docker containers is in the readmes associated with the distribution and they thave been updated. Longer term, the planning guide needs details of running with containers.
Additional notes
This change was discovered while converting the release process to use the omag-server-platform distribution rather than the obsolete open-metadata-assemblies distribution.