Skip to content

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

Merged
lijamie98 merged 7 commits intostellar:developfrom
lijamie98:feature/anchor-868-servicerunner-run-test-profile
Nov 26, 2024
Merged

[ANCHOR-868] Fix broken docker-compose file#1583
lijamie98 merged 7 commits intostellar:developfrom
lijamie98:feature/anchor-868-servicerunner-run-test-profile

Conversation

@lijamie98
Copy link
Contributor

@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

### For version 2.x.x stable release
```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
Contributor 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.

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
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