-
Notifications
You must be signed in to change notification settings - Fork 881
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
feat(framework:skip) Update Docker Compose distributed
files for 1.13.0
#4414
Conversation
Signed-off-by: Robert Steiner <robert@flower.ai>
distributed
files for 1.13.0
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.
Thanks @Robert-Steiner. Similar to #4413, let's set the image tags to nightly
.
@@ -2,6 +2,10 @@ services: | |||
superlink: | |||
image: flwr/superlink:${FLWR_VERSION:-1.12.0} |
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.
image: flwr/superlink:${FLWR_VERSION:-1.12.0} | |
image: flwr/superlink:${FLWR_VERSION:-1.12.0} |
Same as this comment. This should be nightly
for now, until 1.13
is released.
build: | ||
context: ${PROJECT_DIR:-.} | ||
dockerfile_inline: | | ||
FROM flwr/superexec:${FLWR_VERSION:-1.12.0} | ||
FROM flwr/serverapp:${FLWR_VERSION:-1.12.0} |
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.
Same comment as above.
Co-authored-by: Chong Shen Ng <chong.shen@flower.ai>
Signed-off-by: Robert Steiner <robert@flower.ai>
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!
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!
Issue
Description
Updates the Docker Compose complete setup to incorporate the upcoming 1.13.0 changes.
Related issues/PRs
Proposal
Explanation
Checklist
#contributions
)Any other comments?