Skip to content
This repository has been archived by the owner on Sep 12, 2024. It is now read-only.

fix(docker): fixes #853 added restart policy #854

Merged
merged 5 commits into from
Apr 6, 2022
Merged

fix(docker): fixes #853 added restart policy #854

merged 5 commits into from
Apr 6, 2022

Conversation

varunp2k
Copy link
Member

@varunp2k varunp2k commented Mar 29, 2022 •
edited
Loading

closes #853

@varunp2k varunp2k requested a review from Samyak2 March 29, 2022 10:34
@netlify
Copy link

netlify bot commented Mar 29, 2022 •
edited
Loading

✅ Deploy Preview for frontend-sb canceled.

Name Link
🔨 Latest commit 0cf4bee
🔍 Latest deploy log https://app.netlify.com/sites/frontend-sb/deploys/624dc30e9ed93500075e855d

@Samyak2 Samyak2 added this to the v0.6.0 milestone Mar 29, 2022
Samyak2
Samyak2 suggested changes Mar 29, 2022
View reviewed changes
@Samyak2 Samyak2 added the 🏗️ infrastructure infrastructure, devops, setup, etc. label Mar 29, 2022
@varunp2k
Copy link
Member Author

@Samyak2 Please check

Samyak2
Samyak2 suggested changes Mar 29, 2022
View reviewed changes
Copy link
Contributor

@Samyak2 Samyak2 left a comment

Choose a reason for hiding this comment

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

Also, we should update the stable docker-compose.yml and docker-compose.thirdparty.yml too.

@varunp2k @Samyak2
Update docker-compose.latest.yml
207f4e5 
Co-authored-by: Samyak Sarnayak <[email protected]>
@varunp2k varunp2k requested a review from Samyak2 April 4, 2022 08:36
Samyak2
Samyak2 approved these changes Apr 6, 2022
View reviewed changes
Copy link
Contributor

@Samyak2 Samyak2 left a comment

Choose a reason for hiding this comment

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

LGTM

@Samyak2 Samyak2 merged commit 5c2e026 into develop Apr 6, 2022
@Samyak2 Samyak2 deleted the docker-restart-policy branch April 6, 2022 16:44
@Samyak2 Samyak2 linked an issue Apr 8, 2022 that may be closed by this pull request
Add a restart policy to docker compose services #853
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Samyak2 Samyak2 Samyak2 approved these changes

Assignees
No one assigned
Labels
🏗️ infrastructure infrastructure, devops, setup, etc.
Projects
None yet
Milestone
v0.6.0
Development

Successfully merging this pull request may close these issues.

Add a restart policy to docker compose services
2 participants
@varunp2k @Samyak2