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

deploy section and old cpus option #10075

Closed
nexcode opened this issue Dec 13, 2022 · 6 comments
Closed

deploy section and old cpus option #10075

nexcode opened this issue Dec 13, 2022 · 6 comments

Comments

@nexcode
Copy link

nexcode commented Dec 13, 2022

  1. deploy options don't map 1-to-1 to container configurations
  2. v2 is not technically older (v2.3 was introduced around the same time as v3.4)
  3. The two versions serve different purposes. We recommend v3 in our documentation because it is better suited to deploying distributed applications.

This is just circling back to the same points again, so I'll lock this thread. If you want to keep discussing v2/v3 disparity and Compose format design, please use the forums.

Originally posted by @shin- in #4513 (comment)

@nexcode
Copy link
Author

nexcode commented Dec 13, 2022

Initial discussion: #10073

@nexcode
Copy link
Author

nexcode commented Dec 13, 2022

This is from the official website of the docker itself:
Снимок экрана 2022-12-13 в 16 34 56

@nexcode
Copy link
Author

nexcode commented Dec 13, 2022

There are many threads on the topic that the deploy section really only works in swarm mode.
Please, explain how it really works :)

@ndeloof
Copy link
Contributor

ndeloof commented Dec 19, 2022

compose specification (2 years old) made v2 vs v3 debate obsolete. You can now use the "legacy" attributes from v2 file format in your compose file, and should omit the version attribute.

@ndeloof
Copy link
Contributor

ndeloof commented Dec 19, 2022

also have a look at https://github.com/compose-spec/compose-spec/blob/master/spec.md#cpus for this specific attribute

@ndeloof
Copy link
Contributor

ndeloof commented Dec 19, 2022

I'm re-opening #10073, as I can confirm an issue with cpu reservation

@ndeloof ndeloof closed this as completed Dec 19, 2022
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

No branches or pull requests

2 participants