Skip to content

one off container name use configured Separator for naming#10357

Merged
ndeloof merged 1 commit intodocker:v2from
ndeloof:run_separator
Mar 15, 2023
Merged

one off container name use configured Separator for naming#10357
ndeloof merged 1 commit intodocker:v2from
ndeloof:run_separator

Conversation

@ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Mar 8, 2023

What I did
use configured name separator to define oneoff container name

Related issue
closes #10354

@ndeloof ndeloof force-pushed the run_separator branch 5 times, most recently from 52e3c9a to b926802 Compare March 14, 2023 13:41
@codecov
Copy link

codecov bot commented Mar 14, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.08 🎉

Comparison is base (fc4d2df) 53.32% compared to head (b926802) 53.41%.

❗ Current head b926802 differs from pull request most recent head f56aa95. Consider uploading reports for the commit f56aa95 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##               v2   #10357      +/-   ##
==========================================
+ Coverage   53.32%   53.41%   +0.08%     
==========================================
  Files         104      104              
  Lines        8911     8911              
==========================================
+ Hits         4752     4760       +8     
+ Misses       3640     3634       -6     
+ Partials      519      517       -2     
Impacted Files Coverage Δ
pkg/compose/run.go 53.93% <100.00%> (ø)

... and 2 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
@ndeloof ndeloof marked this pull request as ready for review March 14, 2023 13:52
@ndeloof ndeloof requested review from a team, StefanScherer, glours, laurazard, milas, nicksieger and ulyssessouza and removed request for a team March 14, 2023 13:52
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

@ndeloof ndeloof merged commit 754c068 into docker:v2 Mar 15, 2023
@ndeloof ndeloof deleted the run_separator branch March 15, 2023 08:47
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.

[BUG] docker-compose run still creates containers with an underscore in the name

2 participants