Issue with picking 2 random players?

So my pickrandom() function is having issues when finding player 2 I put in a repeat to make sure it doesn't pick the same player that has already been picked but it's not working and it's causing issues with teleporting so I'm pretty sure it's one of the variables causing problems..

if somebody could help me clean up the function and explain what went wrong I'd appreciate it.

line 50 - 65

local replicatedStorage = game.ReplicatedStorage
local status = replicatedStorage:WaitForChild('InfoValue')
local mapstorage = workspace.Mapstorage
local mapinfo = replicatedStorage:WaitForChild("Map")
local plr1 = replicatedStorage:WaitForChild("plr1")
local plr2 = replicatedStorage:WaitForChild("plr2")
local plr1died = replicatedStorage:WaitForChild("Player1Died")
local plr2died = replicatedStorage:WaitForChild("Player2Died")

local player1
local player2

_G. playing = {}

local intermissionTimer = 15

while true do
	while game.Players.NumPlayers < 2 do
		status.Value = "The game requires a minimum of 2 players please invite a friend or patiently wait.."
		repeat wait(2) until game.Players.NumPlayers >= 2
	end

	game.Players.Changed:connect(function()
		if game.Players.NumPlayers >= 2 then
			intermissionTimer = 30
		elseif game.Players.NumPlayers > 2 and game.Players.NumPlayers <= 8 then
			intermissionTimer = 20
		elseif game.Players.NumPlayers > 8 and game.Players.NumPlayers <= 15 then
		end
	end)
	for i = intermissionTimer,0,-1 do
		status.Value = "Intermission : "..i
		wait(1)
	end

	function giveplayer1skin()
			local sword = replicatedStorage.Sword
			local newsword = sword:Clone()
			newsword.Parent = game.Players:FindFirstChild(player1).Backpack
			game.Players:FindFirstChild(player1).leaderstats.Cash.Value = game.Players:FindFirstChild(player1).leaderstats.Cash.Value + 25
	end
	
	function giveplayer2skin()
			local sword = replicatedStorage.Sword
			local newsword = sword:Clone()
			newsword.Parent = game.Players:FindFirstChild(player2).Backpack
			game.Players:FindFirstChild(player2).leaderstats.Cash.Value = game.Players:FindFirstChild(player2).leaderstats.Cash.Value + 25
	end
		
	function pickrandom()
		local plrs = game.Players:GetPlayers()	
		local pl1 = plrs[math.random(1, #plrs)]
		player1 = pl1.Name
		print(player1)
		plr1.Value = player1
		
		local pl2
		
		repeat
			pl2 = plrs[math.random(1, #plrs)]
			player2 = pl2.Name
			plr2.Value = player1
			print(player2)
		until pl2 and pl2 ~= pl1
	end
	
	local mapinserverstorage = game:GetService('ServerStorage'):GetChildren()
	local chosenmap = mapinserverstorage[math.random(1, #mapinserverstorage)]
	local spawn1 = chosenmap:FindFirstChild("Spawns")['Spawn1']
	local spawn2 = chosenmap:FindFirstChild("Spawns")['Spawn2']
	
		mapinfo.Value = chosenmap.Name
		status.Value = "Loading Map : ".. chosenmap.Name
		chosenmap:Clone().Parent = mapstorage
		wait(5)
		status.Value = "Get ready to be teleported!"
		pickrandom()
		wait(1)
					plr1died.Value = false
					plr2died.Value = false
					wait(1)
					game.Players:FindFirstChild(player1).Character.Torso.CFrame = CFrame.new(spawn1.Position + Vector3.new(0,2,0))
					game.Players:FindFirstChild(player2).Character.Torso.CFrame = CFrame.new(spawn2.Position + Vector3.new(0,2,0))	
					wait(1)
					giveplayer1skin()
					giveplayer2skin()
					
		
		for i = 60, 0, -1 do
			if i == 0 then
				status.Value = "Time up!"
				-- kill players left in ring
				game.Players:FindFirstChild(player1).Character:BreakJoints()
				print('killing player1')
				game.Players:FindFirstChild(player2).Character:BreakJoints()
				print('killing player2')
				
				plr1died.Value = false
				plr2died.Value = false
				break
			end
			
			wait(1)
			if plr1died.Value == true then
						game.Players:FindFirstChild(player2):WaitForChild("leaderstats").Cash.Value = game.Players:FindFirstChild(player2):WaitForChild("leaderstats").Cash.Value + 250
						game.Players:FindFirstChild(player2)['leaderstats'].Wins.Value = game.Players:FindFirstChild(player2)['leaderstats'].Wins.Value + 1
						wait(2)
						game.Players:FindFirstChild(player2).Character:BreakJoints()
						print('killing winner')
						wait(1)
						status.Value = player2.. " Defeated ".. player1 -- victory text
						plr1.Value = ""
						plr1died.Value = false
						plr2.Value = ""
						player1 = ""
						player2 = ""
						break
			elseif plr2died.Value == true then
						game.Players:FindFirstChild(player1):WaitForChild("leaderstats").Cash.Value = game.Players:FindFirstChild(player1):WaitForChild("leaderstats").Cash.Value + 250
						game.Players:FindFirstChild(player1)['leaderstats'].Wins.Value = game.Players:FindFirstChild(player1)['leaderstats'].Wins.Value + 1
						wait(2)						
						game.Players:FindFirstChild(player1).Character:BreakJoints()
						print('killing winner')
						wait(1)
						status.Value = player1.. " Defeated ".. player2 -- victory text
						plr1.Value = ""
						plr2.Value = ""
						plr2died.Value = false
						player1 = ""
						player2 = ""
						break	
				end
				status.Value = i.. " seconds remaining!"
		end
	mapstorage:ClearAllChildren()
wait(5)
end