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

Ajust a create_host.yaml and create_host.py #59

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

lucasfontini
Copy link

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

@lucasfontini lucasfontini requested review from namachieli and a team as code owners October 12, 2023 00:52
@lucasfontini lucasfontini requested review from m4dcoder and removed request for a team October 12, 2023 00:53
@namachieli
Copy link
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.

@namachieli
Copy link
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: [ "{{ ctx.var_name }}" ]

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.

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.

2 participants