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

CLI: add descirbe workflow execution #636

Merged
merged 3 commits into from
Mar 27, 2018
Merged

CLI: add descirbe workflow execution #636

merged 3 commits into from
Mar 27, 2018

Conversation

vancexu
Copy link
Contributor

@vancexu vancexu commented Mar 27, 2018

image
image

@coveralls
Copy link

coveralls commented Mar 27, 2018

Coverage Status

Coverage decreased (-0.2%) to 63.219% when pulling 13f5d15 on cli into c444cac on master.

@mfateev
Copy link
Contributor

mfateev commented Mar 27, 2018

Could you show a screenshot of a result that includes activities with a progress? The progress part of activities is the most important part of the result as it cannot be extracted from the history.

./cadence workflow descibe -w 3ea6b242-b23c-4279-bb13-f215661b4717 -r 866ae14c-88cf-4f1e-980f-571e031d71b0
# a shortcut of this is (without -w -r flag)
./cadence workflow descibeid 3ea6b242-b23c-4279-bb13-f215661b4717 866ae14c-88cf-4f1e-980f-571e031d71b0
# both workflow_id and run_id are required for show workflow execution
Copy link
Contributor

Choose a reason for hiding this comment

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

According to Samar runId shouldn't be required

@vancexu
Copy link
Contributor Author

vancexu commented Mar 27, 2018

image

@mfateev
Copy link
Contributor

mfateev commented Mar 27, 2018

Nice!

@vancexu vancexu merged commit 3a65572 into master Mar 27, 2018
@vancexu vancexu deleted the cli branch March 27, 2018 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants