-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show actual version running on pods #204
Conversation
🤖 Created branch: z_pr204/vthapar/show-versions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some typo fixes!
Some of our pods produce lots of logs, especially in circumstances where we need to diagnose a Submariner setup. I do still think that logging the version number will be helpful, and subctl
should look there first to find out what version of a component is running. However it might be worth adding a --version
flag to all components, and then have subctl
exec
the appropriate binary in each pod to extract the version information if it can’t get it from the logs.
|
||
## User Impact | ||
|
||
* Users will need to modify any scripts or tooling that rely on current output of `show versions` or version log message |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should offer JSON output from subctl
;-). (Just a note, not requesting this here!)
Relevant: submariner-io/submariner#2552 |
@sridhargaddam @tpantelis Proposal has been changed since your approvals, so re-requesting review. |
Closing and re-opening to trigger CI. |
🤖 Closed branches: [z_pr204/vthapar/show-versions] |
🤖 Created branch: z_pr204/vthapar/show-versions |
Closing and re-opening to trigger CI. |
🤖 Closed branches: [z_pr204/vthapar/show-versions] |
Adds a `version` file to nettest image which can help determine version in-use for submariner-metrics-proxy pods Refer submariner-io/enhancements#204 Fixes submariner-io#1327 Signed-off-by: Vishal Thapar <[email protected]>
Adds `--version` flag to Lighthouse Agent and `--subm-version` flag to Lighthouse Coredns. CoreDNS already has `--version` which prints CoreDNS version, so we can't use the same flag for Lighthouse version. Refer submariner-io/enhancements#204 Fixes submariner-io#1333 Signed-off-by: Vishal Thapar <[email protected]>
Refer submariner-io/enhancements#204 Fixes submariner-io#2751 Signed-off-by: Vishal Thapar <[email protected]>
* Adds `--version` flag to all submariner binaries. * Modifies version log msg to also show component name Refer submariner-io/enhancements#204 Fixes #2633 Signed-off-by: Vishal Thapar <[email protected]>
Adds `--version` flag to Lighthouse Agent and `--subm-version` flag to Lighthouse Coredns. CoreDNS already has `--version` which prints CoreDNS version, so we can't use the same flag for Lighthouse version. Refer submariner-io/enhancements#204 Fixes submariner-io#1333 Signed-off-by: Vishal Thapar <[email protected]>
Adds `--version` flag to Lighthouse Agent and `--subm-version` flag to Lighthouse Coredns. CoreDNS already has `--version` which prints CoreDNS version, so we can't use the same flag for Lighthouse version. Refer submariner-io/enhancements#204 Fixes submariner-io#1333 Signed-off-by: Vishal Thapar <[email protected]>
Adds `--version` flag to Lighthouse Agent and `--subm-version` flag to Lighthouse Coredns. CoreDNS already has `--version` which prints CoreDNS version, so we can't use the same flag for Lighthouse version. Refer submariner-io/enhancements#204 Fixes submariner-io#1333 Signed-off-by: Vishal Thapar <[email protected]>
Adds `--version` flag to Lighthouse Agent and `--subm-version` flag to Lighthouse Coredns. CoreDNS already has `--version` which prints CoreDNS version, so we can't use the same flag for Lighthouse version. Removed a log entry from metrics.go which didn't add much value so we can have version as first entry in log file. Refer submariner-io/enhancements#204 Fixes submariner-io#1333 Signed-off-by: Vishal Thapar <[email protected]>
Refer submariner-io/enhancements#204 Fixes submariner-io#2751 Signed-off-by: Vishal Thapar <[email protected]>
* Adds `--version` flag to all submariner binaries. * Modifies version log msg to also show component name Refer submariner-io/enhancements#204 Fixes submariner-io#2633 Signed-off-by: Vishal Thapar <[email protected]>
Refer submariner-io/enhancements#204 Fixes submariner-io#2751 Signed-off-by: Vishal Thapar <[email protected]>
Adds `--version` flag to Lighthouse Agent and `--subm-version` flag to Lighthouse Coredns. CoreDNS already has `--version` which prints CoreDNS version, so we can't use the same flag for Lighthouse version. Removed a log entry from metrics.go which didn't add much value so we can have version as first entry in log file. Refer submariner-io/enhancements#204 Fixes submariner-io#1333 Signed-off-by: Vishal Thapar <[email protected]>
Adds `--version` flag to Lighthouse Agent and `--subm-version` flag to Lighthouse Coredns. CoreDNS already has `--version` which prints CoreDNS version, so we can't use the same flag for Lighthouse version. Removed a log entry from metrics.go which didn't add much value so we can have version as first entry in log file. Refer submariner-io/enhancements#204 Fixes submariner-io#1333 Signed-off-by: Vishal Thapar <[email protected]>
Adds `--version` flag to Lighthouse Agent and `--subm-version` flag to Lighthouse Coredns. CoreDNS already has `--version` which prints CoreDNS version, so we can't use the same flag for Lighthouse version. Removed a log entry from metrics.go which didn't add much value so we can have version as first entry in log file. Refer submariner-io/enhancements#204 Fixes submariner-io#1333 Signed-off-by: Vishal Thapar <[email protected]>
Refer submariner-io/enhancements#204 Fixes submariner-io#2751 Signed-off-by: Vishal Thapar <[email protected]>
* Adds `--version` flag to all submariner binaries. * Modifies version log msg to also show component name Refer submariner-io/enhancements#204 Fixes submariner-io#2633 Signed-off-by: Vishal Thapar <[email protected]>
Refer submariner-io/enhancements#204 Fixes submariner-io#2751 Signed-off-by: Vishal Thapar <[email protected]>
Refer submariner-io/enhancements#204 Fixes submariner-io#2751 Signed-off-by: Vishal Thapar <[email protected]>
Refer submariner-io/enhancements#204 Fixes #2751 Signed-off-by: Vishal Thapar <[email protected]>
Adds `--version` flag to Lighthouse Agent and `--subm-version` flag to Lighthouse Coredns. CoreDNS already has `--version` which prints CoreDNS version, so we can't use the same flag for Lighthouse version. Removed a log entry from metrics.go which didn't add much value so we can have version as first entry in log file. Refer submariner-io/enhancements#204 Fixes #1333 Signed-off-by: Vishal Thapar <[email protected]>
* Adds `--version` flag to all submariner binaries. * Modifies version log msg to also show component name Refer submariner-io/enhancements#204 Fixes submariner-io#2633 Signed-off-by: Vishal Thapar <[email protected]>
Adds a `version` file to nettest image which can help determine version in-use for submariner-metrics-proxy pods Refer submariner-io/enhancements#204 Fixes #1327 Signed-off-by: Vishal Thapar <[email protected]>
* Adds `--version` flag to all submariner binaries. * Modifies version log msg to also show component name Refer submariner-io/enhancements#204 Fixes #2633 Signed-off-by: Vishal Thapar <[email protected]>
* Adds `--version` flag to all submariner binaries. * Modifies version log msg to also show component name Refer submariner-io/enhancements#204 Fixes #2633 Signed-off-by: Vishal Thapar <[email protected]>
Proposal to enhance
show versions
to show actual version running on pods.Related issue #798