Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

No log output #1314

Closed
mYmNeo opened this issue Nov 2, 2020 · 5 comments · Fixed by #1315
Closed

No log output #1314

mYmNeo opened this issue Nov 2, 2020 · 5 comments · Fixed by #1315
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@mYmNeo
Copy link

mYmNeo commented Nov 2, 2020

What happened:
add --v=X, X is log level. There is no log output for any level.

What you expected to happen:
option --v=X works well

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:

Environment:

  • Kubernetes version (use kubectl version)
  • KubeFed version
    master
  • Scope of installation (namespaced or cluster)
  • Others

/kind bug

@k8s-ci-robot k8s-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Nov 2, 2020
@makkes
Copy link
Contributor

makkes commented Nov 5, 2020

Hi @mYmNeo, would you mind providing some more details on where (which component, kubefedctl, controller,...) and how you provided the flag, please? Thanks!

@mYmNeo
Copy link
Author

mYmNeo commented Nov 5, 2020

Hi @mYmNeo, would you mind providing some more details on where (which component, kubefedctl, controller,...) and how you provided the flag, please? Thanks!

I've tried controller and hyperkube, I think kubefedctl is the same situation. Since a commit about upgrade to kubernetes 1.19 dependency, this problem occurred. And v0.4.1 is not affected.

The flags is very simple, just run any of these binary with --v=4, For example, controller should print all flags result because of the print level greater than 1 , but in fact none of flags printed.

@hectorj2f
Copy link
Contributor

@mYmNeo You are right, I can see the same behavior. It seems it ignores the verbose level in v0.5.0. For v0.4.0 and v0.4.1, it works as expected. Perhaps we have some hidden bugs when upgrading the new dependencies.

@hectorj2f
Copy link
Contributor

@mYmNeo I pushed a bugfix if you have the chance to have a look at it #1315

@mYmNeo
Copy link
Author

mYmNeo commented Nov 10, 2020

@mYmNeo I pushed a bugfix if you have the chance to have a look at it #1315

@hectorj2f I've tried this patch, it works. 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants