Skip to content

Add the sub-command 'service' of command 'dependency'#113

Merged
kezhenxu94 merged 6 commits intoapache:masterfrom
JarvisG495:master
Sep 6, 2021
Merged

Add the sub-command 'service' of command 'dependency'#113
kezhenxu94 merged 6 commits intoapache:masterfrom
JarvisG495:master

Conversation

@JarvisG495
Copy link
Contributor

swctl dependency service [service-id] [--start=start-time] [--end=end-time]

Query the dependencies of a given service and show results in JSON format.

@fgksgf fgksgf self-requested a review September 6, 2021 02:14
@fgksgf fgksgf added the feature this pull request provides new feature label Sep 6, 2021
@fgksgf fgksgf added this to the 0.8.0 milestone Sep 6, 2021
@wu-sheng
Copy link
Member

wu-sheng commented Sep 6, 2021

FYI @mrproliu

Copy link
Member

@fgksgf fgksgf left a comment

Choose a reason for hiding this comment

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

Just some nits

JarvisG495 and others added 2 commits September 6, 2021 02:47
Co-authored-by: Hoshea Jiang <fgksgf@gmail.com>
Change command shortname

Co-authored-by: Hoshea Jiang <fgksgf@gmail.com>
fgksgf
fgksgf previously approved these changes Sep 6, 2021
Copy link
Member

@fgksgf fgksgf left a comment

Choose a reason for hiding this comment

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

LGTM

kezhenxu94
kezhenxu94 previously approved these changes Sep 6, 2021
Copy link
Member

@kezhenxu94 kezhenxu94 left a comment

Choose a reason for hiding this comment

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

Thank you @JarvisG495 , maybe add the command to https://github.com/apache/skywalking-cli/blob/master/scripts/test_commands.sh as well? Just for sanity check

@JarvisG495 JarvisG495 dismissed stale reviews from kezhenxu94 and fgksgf via 3f25c76 September 6, 2021 07:21
kezhenxu94
kezhenxu94 previously approved these changes Sep 6, 2021
@JarvisG495
Copy link
Contributor Author

Thank you @JarvisG495 , maybe add the command to https://github.com/apache/skywalking-cli/blob/master/scripts/test_commands.sh as well? Just for sanity check

Sure. I have added it in my new commit.

@kezhenxu94
Copy link
Member

Well, seems you changed the file mode, it's now not executable

Screen Shot 2021-09-06 at 15 27 20

@JarvisG495
Copy link
Contributor Author

Well, seems you changed the file mode, it's now not executable

Screen Shot 2021-09-06 at 15 27 20

Sry I made a mistake in my developing environment. I made a new commit to change the file mode back.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature this pull request provides new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants