ok, I'm thinking about upgrading my system from:-
cpu - amd fx8350
ram - 16gb corsair vengeance 1866
mobo - gigabyte GA-990FXA-UD5
gpu - asus 1070 gtx

to:-

cpu - amd ryzen 2700x
ram - 16gb corsair vengeance 3200 or 3333 depends on price at the time
mobo - asus x470 prime-pro
gpu - asus 1070 gtx (no changing)

still going to be running at resolution at 1080p

any suggestion/changes etc

Yup I would, it'll be a nice upgrade and you'll see it in gaming.

that 990fx board will support the 220w CPU's (9370FX) i may be interested if you do upgrade Mr. T

Looks good to me, ryzen loves fast ram so at least 3200, try to get CL14 ram also.

G.SKILL TridentZ Series 16GB (2 x 8GB) 288-Pin DDR4 SDRAM DDR4 3200 (PC4 25600) Intel Z170 Platform Desktop Memory Model F4-3200C14D-16GTZKW is what i would get. And I will be getting lol. It works with ryzen well despite saying Intel Z170.

Hey three,
When i was researching this, i found a video about ram usage with ryzen gen 2 and they tested vengeance vs geil and another cannot remember which but the vengeance was the only one to reach 100% well 99.9% of its capability the others rangeed from 77$ to 99.6% in the various test.

i think it was something to do with the chips they used and secondly the vengeance is £50 cheaper than the g.skill tridentz for me

I run the trident's DDR 4 and love them.

Its the Type of chip on the sticks, You want something that has Samsung B die from what I've seen.  I know those Gskill ones have them, I think some corsair ones do but I believe they need to be Cas Latency 14 like on the Trident ones there.  The compatibility is getting a lot better now so you should be ok with those corsairs.
these corsair ones are on amd's list

Corsair Vengeance LPX 16GB (2x8GB) DDR4 DRAM 3200MHz C16 Desktop Memory Kit - Black (CMK16GX4M2B3200C16)