Skip to content

Fix all actions which use client.query() method#4

Closed
Kami wants to merge 3 commits intomasterfrom
fix_query_actions
Closed

Fix all actions which use client.query() method#4
Kami wants to merge 3 commits intomasterfrom
fix_query_actions

Conversation

@Kami
Copy link
Copy Markdown
Contributor

@Kami Kami commented Jul 25, 2017

Fixes actions which use st2client query() method so they work again after a breaking change in StackStorm/st2#3514 which has been included in v2.3.1.

In the future we need to be more careful when we do a breaking client changes like that one. We some how missed that breaking change and I believe we also didn't document it so we need to go back and explicitly document this in upgrade notes of st2 v2.3.

Resolves StackStorm/st2#3606

Kami added 2 commits July 25, 2017 16:44
result.

In StackStorm v2.3.0, breaking client change has been introduced for
query method as part of StackStorm/st2#3514.
@LindsayHill
Copy link
Copy Markdown
Contributor

Hmmm. Problem with this one is that existing users need to manually run st2 pack install st2. Doesn't get done automatically on upgrade.

@lakshmi-kannan
Copy link
Copy Markdown

@LindsayHill I think the breaking change is now replaced with backward compatible API StackStorm/st2#3616. I think we can close this PR? cc: @humblearner @Kami

@Kami
Copy link
Copy Markdown
Contributor Author

Kami commented Aug 1, 2017

Yep, closing 👍

@Kami Kami closed this Aug 1, 2017
@Kami Kami deleted the fix_query_actions branch August 1, 2017 13:53
nmaludy pushed a commit that referenced this pull request Aug 25, 2020
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