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

Set build variables on the node instead of setup node #10469

Merged
merged 1 commit into from
Aug 27, 2020

Conversation

vsebe
Copy link
Contributor

@vsebe vsebe commented Aug 27, 2020

Set freemarkers, boot_jdk and git cache on the node for correct
interpolation and update commands accordingly.
Check if locations exist.

[ci skip]

Signed-off-by: Violeta Sebe vsebe@ca.ibm.com

@vsebe
Copy link
Contributor Author

vsebe commented Aug 27, 2020

Jenkins compile plinux jdk11

@vsebe
Copy link
Contributor Author

vsebe commented Aug 27, 2020

@jdekonin please review, thank you!

@vsebe vsebe changed the title Set build variables on the node instead of setup node WIP: Set build variables on the node instead of setup node Aug 27, 2020
Set freemarkers, boot_jdk and git cache on the node for correct
interpolation and update commands accordingly.
Check if locations exist.

[ci skip]

Signed-off-by: Violeta Sebe <vsebe@ca.ibm.com>
Copy link
Contributor

@jdekonin jdekonin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@vsebe vsebe changed the title WIP: Set build variables on the node instead of setup node Set build variables on the node instead of setup node Aug 27, 2020
@vsebe
Copy link
Contributor Author

vsebe commented Aug 27, 2020

@pshipton please review, thank you!

@pshipton
Copy link
Member

Jenkins compile plinux jdk11

@pshipton pshipton merged commit 68888be into eclipse-openj9:master Aug 27, 2020
@vsebe vsebe deleted the ibm-jdk11-jcldev.zos_vars branch September 25, 2020 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants