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

resource-recorder: update the docker compose file #1092

Conversation

iulianbarbu
Copy link
Contributor

@iulianbarbu iulianbarbu commented Jul 17, 2023

Description of change

Added a docker-compose service for resource-recorder and Makefile tag env variable.

How has this been tested? (if applicable)

Started Shuttle locally and it worked. Tested on unstable too.

@iulianbarbu iulianbarbu changed the title Feature/eng 735 update the docker compose files for resource recorder resource-recorder: update the docker compose file Jul 17, 2023
Copy link
Member

@chesedo chesedo left a comment

Choose a reason for hiding this comment

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

I think we can use the same minimal prepare.sh file as gateway and auth 😄

resource-recorder/prepare.sh Outdated Show resolved Hide resolved
@iulianbarbu iulianbarbu force-pushed the feature/eng-735-update-the-docker-compose-files-for-resource-recorder branch from c7ddd5a to ade7e07 Compare July 18, 2023 18:29
Copy link
Member

@chesedo chesedo left a comment

Choose a reason for hiding this comment

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

Just have to undo comments

.circleci/config.yml Outdated Show resolved Hide resolved
@iulianbarbu iulianbarbu force-pushed the feature/eng-735-update-the-docker-compose-files-for-resource-recorder branch from ade7e07 to b348a09 Compare July 19, 2023 07:35
@iulianbarbu iulianbarbu self-assigned this Jul 19, 2023
...of the custom directory from the examples to custom-serivce.
This made our init for `None` example fail.
@iulianbarbu iulianbarbu force-pushed the feature/eng-735-update-the-docker-compose-files-for-resource-recorder branch from 8a33c78 to e4bf12f Compare July 19, 2023 11:31
Copy link
Member

@chesedo chesedo left a comment

Choose a reason for hiding this comment

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

PR just has an unexpected change

cargo-shuttle/src/args.rs Show resolved Hide resolved
@iulianbarbu iulianbarbu merged commit 129ad74 into main Jul 20, 2023
31 checks passed
AlphaKeks pushed a commit to AlphaKeks/shuttle that referenced this pull request Jul 21, 2023
* resource-recorder: added docker compose service

* resource-recorder: added prepare.sh

* docker-compose: fix provisioner/resource-recorder deps

* docker-compose: fixed resource-recorder args

* review/resource-recorder: addressed P review and fixed db_path

* cargo-shuttle: custom resources changed the name...

...of the custom directory from the examples to custom-serivce.
This made our init for `None` example fail.
@jonaro00 jonaro00 deleted the feature/eng-735-update-the-docker-compose-files-for-resource-recorder branch August 25, 2023 21:37
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.

None yet

2 participants