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.

Sign up for GitHub

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

Jump to bottom

Remove duplicated volumes section #6185

Merged
merged 1 commit into from
Sep 7, 2023
Merged

Remove duplicated volumes section #6185

merged 1 commit into from
Sep 7, 2023

Conversation

stnguyen90
Copy link
Contributor

@stnguyen90 stnguyen90 commented Sep 7, 2023 •
edited
Loading

What does this PR do?

Multiple changes caused the volumes to be added twice. This PR removes one.

Test Plan

Built the image locally and ran the install:

image

docker compose up -d works:

image

No duplicate volumes:

image

Related PRs and Issues

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

@stnguyen90 stnguyen90 linked an issue Sep 7, 2023 that may be closed by this pull request
🐛 Bug Report: Two repeated volumes are evident for: appwrite-worker-builds #6184
Closed
2 tasks
@christyjacob4 christyjacob4 merged commit 30a67c6 into 1.4.x Sep 7, 2023
2 of 3 checks passed
@stnguyen90 stnguyen90 deleted the fix-6184-worker-builds-volumes branch September 7, 2023 23:46
This was referenced Oct 1, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@christyjacob4 christyjacob4 christyjacob4 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

🐛 Bug Report: Two repeated volumes are evident for: appwrite-worker-builds
2 participants
@stnguyen90 @christyjacob4