-
Notifications
You must be signed in to change notification settings - Fork 103
Updating rad version command to also show control plane version #9167
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
Conversation
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9167 +/- ##
==========================================
+ Coverage 55.84% 55.98% +0.13%
==========================================
Files 603 604 +1
Lines 41344 41466 +122
==========================================
+ Hits 23090 23216 +126
+ Misses 16486 16484 -2
+ Partials 1768 1766 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
pkg/cli/cmd/version/version.go
Outdated
// writeVersionInfo displays both CLI and Control Plane version information | ||
func (r *Runner) writeVersionInfo(format string) error { | ||
// Display CLI version information | ||
cliVersion := struct { |
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.
lots of dup code here. MIght be best to move this to a builder or helper method to keep thing DRY
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.
I get an error when running rad version
when not connected to a radius installation. Maybe just a message saying control plane not connected or something similar.
CLI Version Information:
RELEASE VERSION BICEP COMMIT
edge v0.45.0-rc1-11-gbaab250 0.33.93 baab250fd813b4fcdb57717a0f4acec929d1ab7a
Failed to check Radius control plane: Kubernetes cluster unreachable: Get "http://localhost:8080/version": dial tcp 127.0.0.1:8080: connect: connection refused
Control Plane Information:
VERSION DAPR VERSION
Not installed Not installed
Also this when running rad version --output json
$ rad version --output json
CLI Version Information:
{
"release": "edge",
"version": "v0.45.0-rc1-11-gbaab250",
"bicep": "0.33.93",
"commit": "baab250fd813b4fcdb57717a0f4acec929d1ab7a"
}
Failed to check Radius control plane: Kubernetes cluster unreachable: Get "http://localhost:8080/version": dial tcp 127.0.0.1:8080: connect: connection refused
Control Plane Information:
{
"version": "Not installed",
"daprVersion": "Not installed"
}
Signed-off-by: ytimocin <[email protected]>
baab250
to
324fb1d
Compare
Signed-off-by: ytimocin <[email protected]>
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.
🚀 Looks great:
$ rad version
CLI Version Information:
RELEASE VERSION BICEP COMMIT
edge v0.46.0-8-gf7b0f3a 0.35.1 f7b0f3a39769393566c450e3c151de5fdb6b53cb
Control Plane Information:
STATUS VERSION
Installed 0.46.0
$ rad version -o json
{
"release": "edge",
"version": "v0.46.0-8-gf7b0f3a",
"bicep": "0.35.1",
"commit": "f7b0f3a39769393566c450e3c151de5fdb6b53cb"
}
{
"version": "0.46.0",
"status": "Installed"
}
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.
minor nitpicks but otherwise looks good to me
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
c2b221a
to
5f8f844
Compare
The base branch was changed.
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
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.
🚢
Co-authored-by: Will <[email protected]> Signed-off-by: Yetkin Timocin <[email protected]> Signed-off-by: ytimocin <[email protected]>
5f8f844
to
ae24b04
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Description
Updating rad version command to also show control plane version:
Type of change
Fixes: #8953
Contributor checklist
Please verify that the PR meets the following requirements, where applicable: