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

implement Docker API v1.41.1 #142

Closed
mavogel opened this issue Mar 19, 2021 · 11 comments
Closed

implement Docker API v1.41.1 #142

mavogel opened this issue Mar 19, 2021 · 11 comments
Labels
enhancement New feature or request pinned refactoring Indicates it does not add/remove functionality rather simplifies the code
Milestone

Comments

@mavogel
Copy link
Contributor

mavogel commented Mar 19, 2021

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

New or Affected Resource(s)

  • docker_*

References

@mavogel mavogel added the enhancement New feature or request label Mar 19, 2021
@mavogel mavogel added this to the v3.0.0 milestone Mar 19, 2021
@mavogel mavogel added the refactoring Indicates it does not add/remove functionality rather simplifies the code label Mar 19, 2021
@suzuki-shunsuke
Copy link
Collaborator

Please let me confirm what to do.

  • make all properties consistent (description etc.)

Does this mean to update field description in document and schema description in code according to the API document?

Description: "List of string values to customize labels for MLS systems, such as SELinux. See https://docs.docker.com/engine/reference/run/#security-configuration",

  • be consistent with API v1.41.1

Does this mean to upgrade Go Module of Docker API or to support additional resources?

@mavogel
Copy link
Contributor Author

mavogel commented Mar 29, 2021

Does this mean to update field description in document and schema description in code according to the API document?

Yes and also. add missing properties of the API, which are not reflected yet in the schema.

Does this mean to upgrade Go Module of Docker API or to support additional resources?

IMHO we're already using the latest version since #119

@mavogel
Copy link
Contributor Author

mavogel commented May 13, 2021

Related to #185

@tomalok
Copy link

tomalok commented Jun 6, 2021

I was thinking about this recently -- perhaps it's possible to generate the terraform schema directly from the official docker API definition?

@mavogel
Copy link
Contributor Author

mavogel commented Jun 14, 2021

That's a very good point. Any idea/tools in mind @tomalok how to achieve this?

@tomalok
Copy link

tomalok commented Jun 15, 2021

I did a cursory search and found https://github.com/dikhan/terraform-provider-openapi which might be a good starting point / reference -- there may be others out there.

Obviously Docker's swagger is a lot bigger than the examples...

@github-actions
Copy link

This issue is stale because it has been open 60 days with no activity.
Remove stale label or comment or this will be closed in 7 days.
If you don't want this issue to be closed, please set the label pinned.

@github-actions github-actions bot added the stale label Aug 15, 2021
@mavogel mavogel added pinned and removed stale labels Aug 24, 2021
@matifali
Copy link

are runtime arguments supported now?

@YurNaybor
Copy link

Sorry for digging out an old ticket, but is it now possible to configure capabilities for a service? I can't find it in the docs and trying to add

      capabilities {
        drop = ["ALL"]
        add = ["CAP_CHOWN", "CAP_FOWNER"]
      }

to docker_service -> task_spec -> container_spec yields an error in provider version 3.0.2 "Blocks of type "capabilities" are not expected here."

@dhilgarth
Copy link

@mavogel : Has this ever been implemented?

@chrisclayson
Copy link

Am exceptionally interested in this as require the cap_add functionality for Swarm services (#34) ... whilst this issue was closed as completed, the requirement from #34 doesn't seem to have been included ... is there any roadmap for this? Currently, finding myself needing manually provision a service that requires specific capabilities 😞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pinned refactoring Indicates it does not add/remove functionality rather simplifies the code
Projects
None yet
Development

No branches or pull requests

7 participants