Conversation
jpinsonneau
force-pushed
the
cli-ui
branch
2 times, most recently
from
82afaca to
d9bdafe
Compare
|
🙈 The PR is closed and the preview is expired. |
jpinsonneau
force-pushed
the
cli-ui
branch
4 times, most recently
from
8cb93b6 to
f6418bd
Compare
jpinsonneau
requested review from
Amoghrd,
OlivierCazade,
jotak,
leandroberetta,
luisjira,
memodi and
stleerh
memodi
left a comment
memodi
left a comment
There was a problem hiding this comment.
thanks @jpinsonneau. New UI is great, ability to click right on terminal to cycle through options is much convenient indeed.
|
|
||
| Metrics visualization takes a leap forward with real-time graphs rendered directly in the CLI. | ||
| <br> | ||
| Figure 6: Metrics graphs |
There was a problem hiding this comment.
how do I get graphs on terminal? I tried oc netobserv metrics command but it just pointed me to OCP dashboard?
That's also the general comment I have - to have a command as precursor to the images of the views that's on the blog.
There was a problem hiding this comment.
You command is right, however, you need that PR for metrics: netobserv/network-observability-cli#333
The editors team is suggesting to do a video instead of a blog ... Or at least put videos instead of screens so I will probably change everything here 😿
| <br> | ||
| Figure 4: Live filters suggestions | ||
|
|
||
| - *Packet Preview*: When capturing packets, simply click a row to inspect the pcap content directly.<br><br> |
There was a problem hiding this comment.
I don't see the use of displaying pcap content on terminal, it's in encoded format anyway.
There was a problem hiding this comment.
Just click on a row and you will see the content. If you try to read http packets you can read clear text 😸
|
|
||
| # UI Gallery | ||
|
|
||
| A visual tour of the new CLI experience: |
There was a problem hiding this comment.
do you think gifs would be nice as opposed to static images?
| <br> | ||
| Figure 3: Column selection popup | ||
|
|
||
| - *Smart Filtering*: Live filters now include auto-suggestions, making it easier to select the right keys and values.<br><br> |
There was a problem hiding this comment.
It's a small detail, but the bullet is aligned left and the text seems to be aligned to the center. I think would be nicer for the text to be right after the bullet.
There was a problem hiding this comment.
That's related to the blog itself, not this article. I can take a look at the stylings in parallel 😉
Thanks for the feedback !
|
Holding this PR since I'm revisiting it as a video recording... |
|
@jpinsonneau for the community blog I think you can share both as blog/images and with the recording (personally I prefer the blog/text format :-) ) LGTM! |
We can link the video there when it will be ready. |
4 participants
Quick article to showcase CLI new UI