-
Notifications
You must be signed in to change notification settings - Fork 592
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
chore(agw): remove docker-compose.dev.file #14050
Conversation
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
Did you test the case of a production setup? I guess the variable is not set and then it just works and it's also not set in the containers? Edit: Discussed in person that it works if the variable is not set. |
@@ -161,6 +174,8 @@ services: | |||
retries: 3 | |||
cap_add: | |||
- NET_RAW | |||
environment: | |||
MAGMA_DEV_MODE: ${MAGMA_DEV_MODE} | |||
command: /usr/bin/env python3 -m magma.monitord.main | |||
|
|||
redirectd: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason it isn't needed for redirectd?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its probably not needed for a lot of other service as well. But I only removed it from service where I know from the top of my head it wasn't required.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, sure. Maybe you could open an issue when this gets merged to remove any other unnecessary ones?
894a1c1
to
c1d4ebe
Compare
Signed-off-by: Christian Krämer <christian.kraemer@tngtech.com>
…ntainerized integ tests Signed-off-by: Christian Krämer <christian.kraemer@tngtech.com>
Signed-off-by: Christian Krämer <christian.kraemer@tngtech.com>
9bdb216
to
211bc8e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Fritz Lehnert <13189449+Neudrino@users.noreply.github.com>
magma#14139) otherwise hanging containers take up to 20s to restart and tests run into timeouts. Signed-off-by: Marco Pfirrmann <marco.pfirrmann@tngtech.com> Co-authored-by: Marco Pfirrmann <christian.kraemer@tngtech.com> Addressed review comment Signed-off-by: Rashmi <rashmi.sarwad@radisys.com> feat(agw): time before killing containers during restart is reduced as (magma#14139) otherwise hanging containers take up to 20s to restart and tests run into timeouts. Signed-off-by: Marco Pfirrmann <marco.pfirrmann@tngtech.com> Co-authored-by: Marco Pfirrmann <christian.kraemer@tngtech.com> Addressed review comment Signed-off-by: Rashmi <rashmi.sarwad@radisys.com> feat(agw): time before killing containers during restart is reduced as (magma#14139) otherwise hanging containers take up to 20s to restart and tests run into timeouts. Signed-off-by: Marco Pfirrmann <marco.pfirrmann@tngtech.com> Co-authored-by: Marco Pfirrmann <christian.kraemer@tngtech.com> fix(ci): there was a bit too much deleted in magma#14050 (magma#14143) Signed-off-by: Fritz Lehnert <13189449+Neudrino@users.noreply.github.com>
Summary
The docker-compose.dev.file is redundant. This pr removes it.
Test Plan