RackHD / on-tasks

81%

master: 85%

LAST BUILD ON BRANCH power_dependency

branch: power_dependency

CHANGE BRANCH

x

Reset

• power_dependency
• 3.16-bootstrap
• build_status
• coreos_install_demo
• dependentbuilds
• feature/ibms
• forcediskboot
• hound-config
• jshintrcmv
• kvm
• log_leveling
• master
• release/1.0.0
• release/1.0.0-rc
• release/1.1.0
• release/1.1.0-branch
• release/1.1.0-rc
• release/1.2.0
• release/1.2.0-branch
• release/1.2.0-rc
• release/1.3.0
• release/1.3.0-rc
• release/2.0.0
• release/2.0.0-rc
• release/2.1.0
• release/2.1.0-rc
• release/2.10.1
• release/2.11.0
• release/2.12.0
• release/2.13.0
• release/2.14.0
• release/2.15.0
• release/2.16.0
• release/2.17.0
• release/2.18.0
• release/2.19.0
• release/2.2.0
• release/2.2.0-rc
• release/2.2.0-rc2
• release/2.2.0-rc3
• release/2.20.0
• release/2.21.0
• release/2.22.0
• release/2.23.0
• release/2.24.0
• release/2.25.0
• release/2.26.0
• release/2.27.0
• release/2.28.0
• release/2.29.0
• release/2.3.0
• release/2.3.0-rc
• release/2.30.0
• release/2.31.0
• release/2.32.0
• release/2.33.0
• release/2.4.0
• release/2.5.0
• release/2.7.1
• release/2.8.0
• release/2.9.0
• revert-108-3.16-bootstrap
• revert-109-revert-108-3.16-bootstrap
• revert-317-eventNotification
• revert-367-enable_lldp
• travis-container
• ubuntu_example
• update_notes
• updated_amt_support

Build # 66

Build Type

push

travis-ci

Committed by heckj

Commit Message

removing requiredProperty power.state on base OS.install task

Working through a SKU definition example
(https://github.com/RackHD/RackHD/pull/14/files), if you have a SKU that
flows into immediately installing an OS, there's nothing in that
sequencing that sets the property for 'power.state' to 'reboot'.

Because such an example graph is run dynamically, the system didn't
flag there was a failure due to this missing property when I POST'd it
into place.

After discussing with @benbp about if it should add a "power.state"
property to my SKU default workflow process, or remove this property
as it's not entirely accurate, he suggested we just remove it - as
it's caused more confusion than value in attempting to make sure the
system was 'rebooting' prior to getting to the installation task.

This commit removes that dependency.

Run Details

3036 of 3734 relevant lines covered (81.31%)

15.99 hits per line