Skip to content

introduce compose logs --index to select a replica container #11231

Merged
ndeloof merged 6 commits intodocker:mainfrom
ndeloof:10130_cli_logs_index
Nov 29, 2023
Merged

introduce compose logs --index to select a replica container #11231
ndeloof merged 6 commits intodocker:mainfrom
ndeloof:10130_cli_logs_index

Conversation

@ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Nov 29, 2023

What I did
introduce compose logs --index to select a replica container

Related issue
closes #10130

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

Signed-off-by: Amit Saha <asaha@atlassian.com>
Signed-off-by: Amit Saha <asaha@atlassian.com>
Signed-off-by: Amit Saha <asaha@atlassian.com>
Signed-off-by: Amit Saha <asaha@atlassian.com>
Signed-off-by: Amit Saha <asaha@atlassian.com>
Signed-off-by: Amit Saha <asaha@atlassian.com>
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
@ndeloof ndeloof force-pushed the 10130_cli_logs_index branch from ddbe2af to cc34eb4 Compare November 29, 2023 08:53
@ndeloof ndeloof requested review from a team, glours, laurazard, milas, nicksieger and ulyssessouza and removed request for a team November 29, 2023 08:57
@codecov
Copy link

codecov bot commented Nov 29, 2023

Codecov Report

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

Comparison is base (f5e3ff0) 56.81% compared to head (cc34eb4) 56.79%.

Files Patch % Lines
pkg/compose/logs.go 57.14% 4 Missing and 2 partials ⚠️
cmd/compose/logs.go 57.14% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11231      +/-   ##
==========================================
- Coverage   56.81%   56.79%   -0.03%     
==========================================
  Files         133      133              
  Lines       11500    11516      +16     
==========================================
+ Hits         6534     6540       +6     
- Misses       4329     4337       +8     
- Partials      637      639       +2     

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

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

Copy link
Member

@laurazard laurazard 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 750553c into docker:main Nov 29, 2023
@ndeloof ndeloof deleted the 10130_cli_logs_index branch November 29, 2023 12:56
@asaha123
Copy link
Contributor

Hey @ndeloof thank you for getting this in! Did you have to do much or was the existing PR close to complete?

@ndeloof
Copy link
Contributor Author

ndeloof commented Nov 30, 2023

@asaha123 yes this was close enough I just had to fix merge conflicts and move flag compatibility check to the command

@asaha123
Copy link
Contributor

thanks!

matt9ucci added a commit to matt9ucci/DockerCompletion that referenced this pull request Apr 8, 2024
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.

docker-compose logs for a specific instance of a service

4 participants