Use node['fqdn'] instead of node['hostname'] for mydestination.

Because node['postfix']['myhostname'] is most likely the same with  node['hostname'] because it's supposed to be used with node['postfix']['mydomain'], I think it makes more sense to have node['fqdn'] than node['hostname'] again in the list.