Skip to content

reject compose file using secrets|configs.driver or template_driver#11185

Merged
ndeloof merged 1 commit intodocker:mainfrom
ndeloof:template_driver
Nov 13, 2023
Merged

reject compose file using secrets|configs.driver or template_driver#11185
ndeloof merged 1 commit intodocker:mainfrom
ndeloof:template_driver

Conversation

@ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Nov 13, 2023

What I did
reject compose file using secrets|configs.driver or template_driver
those are implemented by docker swarm configs/secrets engine-drivers, which have no equivalent component in Docker Compose, and we have no "extensibility" mechanism that would allow to run such a "driver" client-side (assuming we want to)

Related issue
see #11179

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Copy link
Contributor

@glours glours left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link

codecov bot commented Nov 13, 2023

Codecov Report

Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (43c3f54) 56.90% compared to head (e0bae63) 56.81%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11185      +/-   ##
==========================================
- Coverage   56.90%   56.81%   -0.09%     
==========================================
  Files         133      133              
  Lines       11370    11382      +12     
==========================================
- Hits         6470     6467       -3     
- Misses       4274     4286      +12     
- Partials      626      629       +3     
Files Coverage Δ
pkg/compose/create.go 57.70% <0.00%> (-0.73%) ⬇️

... and 3 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ndeloof ndeloof merged commit 426377a into docker:main Nov 13, 2023
@ndeloof ndeloof deleted the template_driver branch November 13, 2023 16:29
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

Successfully merging this pull request may close these issues.

2 participants