Ajust a create_host.yaml and create_host.py#59
Open
lucasfontini wants to merge 7 commits intoStackStorm-Exchange:masterfrom
Open
Ajust a create_host.yaml and create_host.py#59lucasfontini wants to merge 7 commits intoStackStorm-Exchange:masterfrom
lucasfontini wants to merge 7 commits intoStackStorm-Exchange:masterfrom
Conversation
Collaborator
|
Hi, Thank you for the PR. I will take a look at this when I have some time. It looks like checks are failing so I also need to review that as well. |
Collaborator
|
My apologies for the delay on this. I took a look and I think that it does need to remain as an array, since multiple IPs can be passed. I think the correct course of action to fix your workflow is to wrap the singular IP into a list: Please give this a try and see if it resolves your issue. Changing this from an array to a string will break many other's workflows that do utilize an array of IPs. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
When we try to create a workflow and insert this action, if you need to put a variable like <% ctx.var_name %> or {{ ctx.var_name }} an error occured to interpret this var, thats because of a type of field ipadress expect to receive an array. after change i was able to run my workflow property