Do not mix chef calls and proposal saves

[dirkmueller]

Using chef calls while the propsal is still being modified can cause delays that would cause the transaction to be delayed so long that other workers get a busy exception. Shorten the lifetime of the critical section.

[vuntz]

I don't understand.

Does doing something like Proposal.where(barclamp: "crowbar", name: inst).first lock the database?

[vuntz]

So code looks correct; not sure if it will help us, but I'm fine trying.

[tboerger]

Using chef calls while the propsal is still being modified can cause delays that would cause the transaction to be delayed so long that other workers get a busy exception. Shorten the lifetime of the critical section.

I really don't understand how find_by should be related...

[vuntz]

@dirkmueller this has conflicts now. But I'm not sure we still need this...

[vuntz]

Do not merge for M11.