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

GuestInfo: Add build step for exposing GuestInfo #22

Merged
merged 2 commits into from
Feb 3, 2015

Conversation

damienbiggs
Copy link
Collaborator

Variables IP, Hostname, ToolsStatus, ToolsRunningStatus, ToolsVersion,
ToolsVersionStatus, GuestState, GuestId, GuestFamily, GuestFullName,
AppHeartbeatStatus, GuestOperationsReady, InteractiveGuestOperationsReady are exposed.
This is done via reflection of the GuestInfo class.

Let me know if logging every variable that is added and skipped is too much logging.
I figured it could be useful debugging for someone who is wondering why an environment variable hasn't been added.

Variables IP, Hostname, ToolsStatus, ToolsRunningStatus, ToolsVersion,
ToolsVersionStatus,  GuestState, GuestId, GuestFamily, GuestFullName,
AppHeartbeatStatus, GuestOperationsReady, InteractiveGuestOperationsReady are exposed.
These are exposed via reflection.
All primitive types for GuestInfo are added as environmental variables.
@damienbiggs damienbiggs force-pushed the topic/exposeGuestInfo branch from ad50250 to e9b5ed1 Compare January 28, 2015 19:00
We didn't know about this variable until looking at the code.
It's quite useful to know about it.
@damienbiggs damienbiggs force-pushed the topic/exposeGuestInfo branch from 76953df to b189fa3 Compare January 30, 2015 04:11
jswager added a commit that referenced this pull request Feb 3, 2015
GuestInfo: Add build step for exposing GuestInfo
@jswager jswager merged commit 6f39193 into jenkinsci:master Feb 3, 2015
@adamzovits
Copy link

This feature seems as if it wasn't functioning: https://issues.jenkins-ci.org/browse/JENKINS-37008 Are we missing something like an extra step or a special way of invocation maybe?

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