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

Uplift variables to global (2) #2952

Merged
merged 23 commits into from Nov 20, 2017

Conversation

Projects
None yet
4 participants
@jtobard
Contributor

jtobard commented Nov 20, 2017

New Workflow step that creates variables on global for the current execution.

  • Value: Value of the variable, can be a reference to another variable or a static value
  • Group: Group on global of the new variable. The variables created on the group export are going to be exported to parent jobs.
  • Name: Name of the new variable.
    The new variable can be used in the way ${group.name}

screenshot at nov 16 10-08-24

Variables created in global in a Job Reference step with the group export, are uplifted to the global on the parent execution context.

Variables created on global are added as state changes after every step for the workflow execution environment.

@gschueler

This comment has been minimized.

Show comment
Hide comment
@gschueler

gschueler Nov 20, 2017

Member

@jtobard testbuild.groovy needs to be updated

Member

gschueler commented Nov 20, 2017

@jtobard testbuild.groovy needs to be updated

@gschueler gschueler merged commit 578f854 into rundeck:master Nov 20, 2017

9 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
security/snyk No new issues
Details
security/snyk - build.gradle No new issues
Details
security/snyk - core/build.gradle No new issues
Details
security/snyk - rundeck-launcher/build.gradle No new issues
Details
security/snyk - rundeck-launcher/launcher/build.gradle No new issues
Details
security/snyk - rundeck-launcher/rundeck-jetty-server/build.gradle No new issues
Details
security/snyk - rundeck-storage/build.gradle No new issues
Details
security/snyk - rundeckapp/build.gradle No new issues
Details

@gschueler gschueler added this to the 2.10.1 milestone Nov 20, 2017

@jolbax

This comment has been minimized.

Show comment
Hide comment
@jolbax

jolbax Nov 24, 2017

It seams to be not possible to set the value without the "@nodename". Or I am missing something?
Is there any documentation about this new feature?

jolbax commented Nov 24, 2017

It seams to be not possible to set the value without the "@nodename". Or I am missing something?
Is there any documentation about this new feature?

@ev0ldave

This comment has been minimized.

Show comment
Hide comment
@ev0ldave

ev0ldave Sep 21, 2018

It seams to be not possible to set the value without the "@nodename". Or I am missing something?
Is there any documentation about this new feature?

The problem is the plugin is a workflow step, not a node step meaning it does not have any node context so we must explicitly define the node to read the data from. In the dialogue it gives an example without an identifier so maybe I am missing something.

ev0ldave commented Sep 21, 2018

It seams to be not possible to set the value without the "@nodename". Or I am missing something?
Is there any documentation about this new feature?

The problem is the plugin is a workflow step, not a node step meaning it does not have any node context so we must explicitly define the node to read the data from. In the dialogue it gives an example without an identifier so maybe I am missing something.

@gschueler

This comment has been minimized.

Show comment
Hide comment
@gschueler

gschueler Sep 21, 2018

Member

@ev0ldave use ${data.xyz*} from within a workflow step, which collates all node values for ${data.xyz} as a comma-separated string. You can also use ${data.xyz*-} for example to use - as the delimiter

Member

gschueler commented Sep 21, 2018

@ev0ldave use ${data.xyz*} from within a workflow step, which collates all node values for ${data.xyz} as a comma-separated string. You can also use ${data.xyz*-} for example to use - as the delimiter

@ev0ldave

This comment has been minimized.

Show comment
Hide comment
@ev0ldave

ev0ldave Sep 21, 2018

@gschueler Thanks, I thought I had tried that earlier but I must have had a typo. It definitely works for me now :)

ev0ldave commented Sep 21, 2018

@gschueler Thanks, I thought I had tried that earlier but I must have had a typo. It definitely works for me now :)

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