Skip to content

Fix configs are mounted under /<id>#11232

Merged
ndeloof merged 1 commit intodocker:mainfrom
ndeloof:config_target
Nov 29, 2023
Merged

Fix configs are mounted under /<id>#11232
ndeloof merged 1 commit intodocker:mainfrom
ndeloof:config_target

Conversation

@ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Nov 29, 2023

What I did
Fix inlined and environment-defined configs to be mounted under /<id> until an explicit target is set
includes a test case to avoid such a regression to appear again

Related issue
see compose-spec/compose-spec#346 (comment)

(not mandatory) A picture of a cute animal, if possible in relation to what you did

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
@ndeloof ndeloof requested review from a team, glours and milas and removed request for a team November 29, 2023 13:23
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.

👍

@codecov
Copy link

codecov bot commented Nov 29, 2023

Codecov Report

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

Comparison is base (16c4241) 56.87% compared to head (4f7fdd0) 56.85%.
Report is 6 commits behind head on main.

Files Patch % Lines
pkg/compose/logs.go 57.14% 4 Missing and 2 partials ⚠️
pkg/compose/secrets.go 72.22% 3 Missing and 2 partials ⚠️
cmd/compose/logs.go 57.14% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11232      +/-   ##
==========================================
- Coverage   56.87%   56.85%   -0.02%     
==========================================
  Files         133      133              
  Lines       11503    11521      +18     
==========================================
+ Hits         6542     6550       +8     
- Misses       4325     4333       +8     
- Partials      636      638       +2     

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

@ndeloof ndeloof merged commit 59f11ec into docker:main Nov 29, 2023
@ndeloof ndeloof deleted the config_target branch November 29, 2023 15:58
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