Skip to content

container logs do not invoke logs cmd#1115

Closed
fahedouch wants to merge 3 commits intocontainerd:mainfrom
fahedouch:container-logging-without-invoke-logs-cmd
Closed

container logs do not invoke logs cmd#1115
fahedouch wants to merge 3 commits intocontainerd:mainfrom
fahedouch:container-logging-without-invoke-logs-cmd

Conversation

@fahedouch
Copy link
Member

@fahedouch fahedouch commented Jun 9, 2022

PR fixes :

  • nerdctl compose logs no longer rely on nerdctl logs
  • nerdctl compose up log output no longer rely on nerdctl logs
  • refacto nerdctl logs

And
fixing #1071
Signed-off-by: Fahed DORGAA [email protected]

@fahedouch fahedouch marked this pull request as draft June 9, 2022 19:43
@AkihiroSuda AkihiroSuda added this to the v0.20.1 milestone Jun 10, 2022
@fahedouch fahedouch changed the title container logs do not invoke logs cmd [WIP] container logs do not invoke logs cmd Jun 14, 2022
@fahedouch fahedouch marked this pull request as ready for review June 14, 2022 17:23
@fahedouch fahedouch force-pushed the container-logging-without-invoke-logs-cmd branch 2 times, most recently from a6afce6 to 8031641 Compare June 14, 2022 17:25
@fahedouch fahedouch changed the title [WIP] container logs do not invoke logs cmd container logs do not invoke logs cmd Jun 16, 2022
@fahedouch fahedouch force-pushed the container-logging-without-invoke-logs-cmd branch from 9c8c3d0 to 91c543f Compare June 16, 2022 17:44
@fahedouch fahedouch force-pushed the container-logging-without-invoke-logs-cmd branch from 91c543f to 49e5431 Compare June 16, 2022 17:45
@fahedouch fahedouch requested a review from AkihiroSuda June 16, 2022 17:45
@AkihiroSuda
Copy link
Member

Thanks, but this PR seems too huge for fixing #1071 , and the tests are failing.

Let me merge #1132 instead

@AkihiroSuda AkihiroSuda removed this from the v0.20.1 milestone Jun 17, 2022
Signed-off-by: Fahed DORGAA <[email protected]>
Signed-off-by: Fahed DORGAA <[email protected]>
@apostasie
Copy link
Contributor

@fahedouch #1071 was fixed, but I am wondering if there is still value in your work here.
Would you rebase it if so?

Cheers.

@fahedouch fahedouch closed this Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants