Skip to content

Conversation

@Sheshagiri
Copy link
Contributor

looks like the type attribute is missing for netstat action in Linux pack because of which the action in not usable.

$ st2 run linux.netstat hosts=172.31.114.68 args="-antp"
ERROR: 'type'

closes: #4946

@pull-request-size pull-request-size bot added the size/XS PR that changes 0-9 lines. Quick fix/merge. label May 13, 2020
Copy link
Member

@nmaludy nmaludy left a comment

Choose a reason for hiding this comment

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

Good find!

Could you please add a CHANGELOG entry for this.

@nmaludy nmaludy added this to the 3.2.1 milestone May 13, 2020
@Sheshagiri
Copy link
Contributor Author

Good find!

Could you please add a CHANGELOG entry for this.

done 👍

@arm4b
Copy link
Member

arm4b commented May 13, 2020

@Sheshagiri Can you also try to fix the lint failures in https://travis-ci.org/github/StackStorm/st2/jobs/686593938#L2218-L2219. Looks like a small change, but if it's not, - you can start a new PR.

It's something new as master is green and I don't think failures are related to your change. Probably just a recent upstream update that broke it.

@pull-request-size pull-request-size bot added size/S PR that changes 10-29 lines. Very easy to review. and removed size/XS PR that changes 0-9 lines. Quick fix/merge. labels May 13, 2020
Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

👍

Also thanks for fixing the Travis CI failure in master.

@arm4b arm4b requested a review from nmaludy May 13, 2020 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S PR that changes 10-29 lines. Very easy to review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

how to run linux.netstat action with arguments

4 participants