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

Fix docker container #8227

Merged
merged 2 commits into from
Jun 11, 2024
Merged

Fix docker container #8227

merged 2 commits into from
Jun 11, 2024

Conversation

mandy-chessell
Copy link
Contributor

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 be host.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

platform.placeholder.variables=\
  {\
    "kafkaEndpoint" : "localhost:9092",\
    "egeriaEndpoint" : "https://localhost:9443"\
  }

And in the docker container, they are:

platform.placeholder.variables=\
  {\
    "kafkaEndpoint" : "host.docker.internal:9092",\
    "egeriaEndpoint" : "https://host.docker.internal:9443"\
  }

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.

Signed-off-by: Mandy Chessell <[email protected]>
@mandy-chessell mandy-chessell requested a review from planetf1 as a code owner June 11, 2024 16:50
@mandy-chessell mandy-chessell enabled auto-merge June 11, 2024 16:51
@mandy-chessell mandy-chessell merged commit 11fa693 into odpi:main Jun 11, 2024
3 of 4 checks passed
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.

1 participant