I find it confusing to have a local variable which has exactly the same name as a fact. I also don't understand why you chose to not use the $::lsbmajdistrelease fact directly, as it was done before ?