You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@yunikorn.apache.org by GitBox <gi...@apache.org> on 2020/10/09 14:19:07 UTC

[GitHub] [incubator-yunikorn-core] adamantal commented on pull request #211: [YUNIKORN-429] Set default logging level to INFO

adamantal commented on pull request #211:
URL: https://github.com/apache/incubator-yunikorn-core/pull/211#issuecomment-706202211


   Hi @yangwwei,
   
   Used docker-desktop to validate the messages locally. I uploaded the INFO level logs of running some sleep jobs (a few of them were pending due to insufficient resources).
   You can check too, that 1-3, 5-6 has traces in the logs.
   
   One missing piece that I found is the reservation. There were pending pods in my scenario, but the reservation did not get triggered, so I could not see the related logs. Also I added an extra log in the reservation and the unreservation phase that may be helpful in the future.
   
   I hope that this make sense. If you have any concerns please let me know.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org