Skip to content

Add new filter based on jmespath for 5.3 version. #44

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.

Sign up for GitHub

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

Jump to bottom
Merged
rohityadavcloud merged 5 commits into from
Mar 5, 2019
Merged

Add new filter based on jmespath for 5.3 version. #44

rohityadavcloud merged 5 commits into from
Mar 5, 2019

Conversation

@snifbr
Copy link

@snifbr snifbr commented Feb 27, 2019

This add to cloudmonkey CLI the abilities to make a JMESPATH filter with following changes in version 5.3:

  • Add new argument 'query'.
  • Add new display type 'jmespath'.
  • Import jmespath module.

Some examples:
note-2019-02-27-111634

note-2019-02-27-111306

@rohityadavcloud
Copy link
Member

rohityadavcloud commented Mar 5, 2019

LGTM, the python based branch 5.3 is no longer maintained by me. It would be interesting to get this for the go based master/6.+ versions, however a lot of what was implemented could be piped to jq to get the same result. Perhaps having this output mode will remove dependency of jq. /cc @PaulAngus @DagSonsteboSB @andrijapanicsb

@rohityadavcloud rohityadavcloud merged commit db74764 into apache:5.3 Mar 5, 2019
@snifbr
Copy link
Author

snifbr commented Mar 13, 2019

LGTM, the python based branch 5.3 is no longer maintained by me. It would be interesting to get this for the go based master/6.+ versions, however a lot of what was implemented could be piped to jq to get the same result. Perhaps having this output mode will remove dependency of jq. /cc @PaulAngus @DagSonsteboSB @andrijapanicsb

Thanks @rhtyd for merged the code.

About the new version go based, I can try something for it, when I have a time.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@snifbr @rohityadavcloud