These look correct, the only difference I see between this and my old sample is using snmp v2 in the community. But that is really a trap version so it does not seem like that would matter.
set snmp community "ReadOnly" Read-Write Trap-on traffic version v2c
set snmp host "ReadOnly" 1.2.3.4/32 src-interface bgroup0 trap v2