Skip to content
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

Add the Process metadata query command #137

Merged
merged 2 commits into from
Mar 10, 2022

Conversation

mrproliu
Copy link
Contributor

Resolve apache/skywalking#8650

command-line screenshot:
image

@mrproliu mrproliu added enhancement New feature or request feature this pull request provides new feature labels Mar 10, 2022
@mrproliu mrproliu added this to the 0.10.0 milestone Mar 10, 2022
@mrproliu mrproliu requested review from wu-sheng and kezhenxu94 March 10, 2022 10:39
@codecov-commenter
Copy link

Codecov Report

Merging #137 (102588e) into master (ac30896) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #137   +/-   ##
=======================================
  Coverage   27.47%   27.47%           
=======================================
  Files          20       20           
  Lines         404      404           
=======================================
  Hits          111      111           
  Misses        286      286           
  Partials        7        7           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ac30896...102588e. Read the comment docs.

Comment on lines 48 to 49
4. List all processes by instance id "cHJvdmlkZXI=.1_cHJvdmlkZXIx":
$ swctl process ls --service-id cHJvdmlkZXI=.1_cHJvdmlkZXIx`,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the description by instance id doesn't match the command ls --service-id

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

update the command line argument name.

@wu-sheng wu-sheng merged commit db980b5 into apache:master Mar 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature this pull request provides new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CLI] Support the process metadata query
4 participants