Can't you get rid of those Ops.get_string here? Something like netcard["fcoe_enable"] || "yes" should work, right?
Same for Ops.get more above...