fix interpolation of pid
Merge pull request #104 from aspiers/pid-interpolation-fix

fix interpolation of pid
Remove check for CHEF_ONLINE

The CHEF_ONLINE constant was removed from crowbar sometime ago already.
See: https://github.com/crowbar/barclamp-crowbar/pull/921 for details.
fix Travis breakage

Travis started failing with:

  uninitialized constant RSpec::Matchers::BuiltIn::RaiseError::MatchAliases (NameError)

due to bundler installing rspec-expectations-2.99 which is a
pre-release:

  https://github.com/rubygems/rubygems/issues/853
Merge pull request #114 from aspiers/fix-travis

fix Travis breakage
Merge pull request #113 from rhafer/no_chef_online

Remove check for CHEF_ONLINE
Add optional crowbar-applied boolean attribute to schema

We add a crowbar applied attribute to mark the proposal as applied after
the chef run. This allows the UI to show that the current form of the
proposal has been 'applied' successfully (i.e. chef-client completed the
run w/ success) and that the proposal was not modified after that.

Update the proposal_schema_revision, so we can add the attribute to
schema for ('3rd party') barclamps that currently do not have it.

Refs: crowbar/crowbar#2044 (which adds this attribute to barclamps which
lack it), crowbar/barclamp-crowbar#1068 (which marks the proposals as
applied) and https://bugzilla.novell.com/show_bug.cgi?id=877486
Merge branch 'release/roxy/master' into merge-roxy
Merge pull request #117 from vuntz/merge-roxy

Merge roxy branch into master
Merge pull request #116 from bkutil/add_applied_attribute

Add optional crowbar-applied boolean attribute to schema
Fixed length calls (bnc#879657)

It is possible that crowbar bails out while apply a pacemaker proposal
with an exception that length is not a method of NilClass. With this fix
it should be fine now.
Use more descriptive msg on sbd init check failure

Stonith + SBD deployment can fail due to SBD not being created on the
shared disk. The message does not really tell what went wrong and what
should be done to fix it. Use a more descriptive one.

The SBD creation is a destructive op and a typo could wipe the user
data, so this is the reason that sbd create is not run by default.

An alternative would be to run the sdb create tonly after user types
the volume name into the input field in the UI twice, for confirmation.
Merge pull request #120 from bkutil/sbd_message

Use more descriptive msg on sbd init check failure
Merge pull request #118 from tboerger/feature/fixed-length

Fixed length calls (bnc#879657)
Fixed length calls (bnc#879657)

It is possible that crowbar bails out while apply a pacemaker proposal
with an exception that length is not a method of NilClass. With this fix
it should be fine now.

(cherry picked from commit 7c2fede9fc355db2b108e2fc720218e5f2c597c3)
Merge pull request #119 from tboerger/feature/roxy-fixed-length

Fixed length calls (bnc#879657)
Fix conversion of per node stonith parameters

This converts the per node stonith parameters from Chef::Node::Attribute to a
"normal" ruby hash when stripping the domain from the node names. It seems
assigning a hash of Chef::Node::Attribute objects to the node attributes
doesn't work. When accessing the values later on (in pacemaker::stonith) they
just evaluated to nil.

https://bugzilla.novell.com/show_bug.cgi?id=887244
Fix extraction of stonith parameter from parameter string

Pacemaker::Resource.extract_hash() is written to extract things from a complete
CIB object definition. Therefore it rightfully expects whitespaces before the
data_type ("params" in this case). So we just add those here to make the string
parsable by extract_hash.

https://bugzilla.novell.com/show_bug.cgi?id=887244
Merge pull request #121 from rhafer/per_node_stonith

Per node stonith config
Fix javascript after barclamp-crowbar change

https://github.com/crowbar/barclamp-crowbar/pull/1091 exposes a bug in
the javascript where we don't specify the default value of a json
attribute we try to read and that doesn't initially exist.