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

[ANCHOR-868] Fix broken docker-compose file #1583

Conversation

lijamie98
Copy link
Collaborator

@lijamie98 lijamie98 commented Nov 21, 2024

Description

  • Make KT_REFERENCE_SERVER_CONFIG optional and default to the reference_config.yaml file.
  • Add v2_stable and latest profiles to docker_compose.yaml
  • Fix Kafka platform problem for MacOS
  • Modify the README.md file

Context

  • Fix broken docker_compose.yaml file
  • Improve developer experience

Testing

  • ./gradlew test

Documentation

N/A

Known limitations

This will break typescript-wallet-sdk. The PR to fix typescript-wallet-sdk is stellar/typescript-wallet-sdk#181

.run/Test Profile_ auth-jwt-custody.run.xml Outdated Show resolved Hide resolved
```shell
docker build --build-arg BASE_IMAGE=gradle:7.6.4-jdk17 -t anchor-platform:local ./
docker compose -f service-runner/src/main/resources/docker-compose.yaml up -d
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we provide a way for developers to start the stack against a locally built image?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think this is good idea as I was thinking about it.

I will add a local profile.

.run/Reference Server_ default.run.xml Show resolved Hide resolved
Copy link
Contributor

@philipliu philipliu left a comment

Choose a reason for hiding this comment

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

Minor comment

docs/README.md Outdated
To build the Anchor Platform image locally, run the following command in the root directory of the project.
```shell
# In the root directory of the project
docker build -t stellar/anchor-platform .
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to add the :local tag here?

@lijamie98 lijamie98 merged commit 9eeedb8 into stellar:develop Nov 26, 2024
8 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.

2 participants