That worked applying it to the ge-0/1/0 - thanks. I didnt quite understand why the example applied it to the loopback but was trying since it didnt work previously when I applied it to the other interface. I may have had something else wrong but it is working now.
Thanks for your help.
John