Gerrit Code Review - Revision history

Peter at 18:18, 29 June 2013

2013-06-29T18:18:16Z

← Older revision		Revision as of 20:18, 29 June 2013
Line 8:		Line 8:

	Gerrit fyller främst två funktioner:		Gerrit fyller främst två funktioner:
	* Administrera repo/projekt rättigheter		* Administrera repo- & projekt-rättigheter
	* Support för granskning av commits innan dom mergas in i repot		* Support för granskning av commits innan dom mergas in i repot

Peter: /* Komma igång */

2013-05-18T08:17:05Z

Komma igång

← Older revision		Revision as of 10:17, 18 May 2013
Line 30:		Line 30:

	{{Cmd\|ssh -p 29418 <host>}}		{{Cmd\|ssh -p 29418 <host>}}

			{{Note\|Om ditt användarnamn på datorn och Gerrit är olika, så måste du alltid ange <user>@<host>.}}

	Om allt går bra ska du se en hälsningsfras liknande:		Om allt går bra ska du se en hälsningsfras liknande:
	{{Bc\|		{{Bc\|

Peter: /* Scenarion */

2013-05-12T06:38:51Z

Scenarion

← Older revision		Revision as of 08:38, 12 May 2013
Line 69:		Line 69:
	= Scenarion =		= Scenarion =

	Detta kapitel kommer beskriva några ~~användarfall~~ med Gerrit där ett projekt har följande inställningar:		Detta kapitel kommer beskriva några scenarion med Gerrit där ett projekt har följande inställningar:
	* Fast-forward only		* Fast-forward only
	* Change-Id krävs i varje commit		* Change-Id krävs i varje commit

Peter at 20:50, 11 May 2013

2013-05-11T20:50:27Z

← Older revision		Revision as of 22:50, 11 May 2013
Line 119:		Line 119:
	[[File:gerrit-review-helloworld-3.png]]		[[File:gerrit-review-helloworld-3.png]]

	== Scenario: Rätta underkänd granskning ==		== Scenario: Rätta underkänd verifiering eller granskning ==

	Anta att du har push:at dina ändringar till for/master i Gerrit, men koden godkänns inte utan måste rättas till. I detta exempel har vi gjort en förändring på en branch som heter hello-msg. Vi har ändrat vårt "Hello, world!" meddelande till "Hello, gerrit!" och push:at till Gerrit. Tyvärr så kommer denna förändring inte igenom granskningen då någon underkänner våra förändringar:		Anta att du har push:at dina ändringar till for/master i Gerrit, men koden godkänns inte utan måste rättas till. I detta exempel har vi gjort en förändring på en branch som heter hello-msg. Vi har ändrat vårt "Hello, world!" meddelande till "Hello, gerrit!" och push:at till Gerrit. Tyvärr så kommer denna förändring inte igenom granskningen då någon underkänner våra förändringar:

Peter: /* Introduktion */

2013-05-11T19:29:36Z

Introduktion

← Older revision		Revision as of 21:29, 11 May 2013
Line 11:		Line 11:
	* Support för granskning av commits innan dom mergas in i repot		* Support för granskning av commits innan dom mergas in i repot

	Istället för att pusha commits direkt till t.ex. master, så pushar man till en fiktiv granskningsbranch som heter for/master. Eftersom en commit kan underkännas och kräver justering innan acceptering, så använder Gerrit ett Change-Id i commit meddelandet för att koppla ihop dessa till samma ändring. Nya commits med samma Change-Id blir nya s.k. patch sets i Gerrit. Commits som har olika Change-Ids behandlas som helt olika förändringar. Det går även att ställa in så man har rättighet att pusha direkt till master (eller annan) branch. Man går då förbi granskningsdelen i Gerrit.		Istället för att pusha commits direkt till t.ex. master, så pushar man till en fiktiv granskningsbranch som heter for/master. Eftersom en commit kan underkännas och kräver justering innan acceptering, så använder Gerrit ett Change-Id i commit meddelandet för att koppla ihop dessa till samma ändring. Nya commits med samma Change-Id blir nya s.k. patch sets i Gerrit. Commits som har olika Change-Ids behandlas som helt olika förändringar. Det går även att ställa in så man har rättighet att pusha direkt till master (eller annan) branch. Man går då förbi granskningsdelen i Gerrit. Om man behåller granskningsfunktionen i Gerrit, måste varje commit granskas och verifieras innan den kan kan submittas till t.ex. master i repositoryt. Gerri använder ett poängsystem där verifiering +1 betyder ok, och -1 betyder att committen underkändes. För granskning kan -2 till +2 poäng sättas. -1 till +1 betyder rekommendationer. -2 betyder att committen underkänds och +2 betyder att den kan accepteras.

	En commit ~~som måste~~ granskas och verifieras innan den kan kan submittas till ~~repot~~. Gerri använder ett poängsystem där verifiering +1 betyder ok, och -1 betyder att committen underkändes. För granskning kan -2 till +2 poäng sättas. -1 till +1 betyder rekommendationer. -2 betyder att committen underkänds och +2 betyder att den kan accepteras.

	Gerrit stödjer också olika policies när en commit får submittas till t.ex. master branchen. Gerrit kan tillåta att den måste göra triviala mergningar, men man kan också vara sträng och kräva att committen baseras på den senaste committen som redan finns i repot (s.k. fast-forward). Följande policies existerar:		Gerrit stödjer också olika policies när en commit får submittas till t.ex. master branchen. Gerrit kan tillåta att den måste göra triviala mergningar, men man kan också vara sträng och kräva att committen baseras på den senaste committen som redan finns i repot (s.k. fast-forward). Följande policies existerar:

Peter at 18:49, 11 May 2013

2013-05-11T18:49:50Z

← Older revision		Revision as of 20:49, 11 May 2013
Line 173:		Line 173:

	[[File:gerrit-review-helloworld-8.png]]		[[File:gerrit-review-helloworld-8.png]]

	~~= (Gammalt) Klona ett repository, förändra och pusha tillbaka till repositoryt för granskning =~~

	~~Antag att vi vill jobba mot ett Gerrit repository som heter quadcopter, för att klona repositoryt kör:~~
	~~{{Cmd\|git clone ssh://<username>@kerwien.homeip.net:29418/quadcopter.git}}~~
	~~Man kan utesluta <username> i alla kommandon om ens username på datorn är samma som det man har på Gerrit servern. Gå ner i repositoryt:~~
	~~{{Cmd\|cd quadcopter}}~~

	~~Kopiera commit-hooken från Gert:~~
	~~{{Cmd\|scp -p -P 29418 <username>@kerwien.homeip.net:hooks/commit-msg .git/hooks/}}~~

	~~Skapa en lokal branch, samt byt till denna, som heter first:~~
	~~{{Cmd\|git checkout -b first}}~~
	~~Man bör alltid byta till en branch och inte jobba direkt på master branchen. Genomför editering i någon fil.~~

	~~Kommitta ändringen till ditt lokala respository med kommentaren "first commit" (texten är valfri):~~
	~~{{Cmd\|git commit -am "first commit"}}~~

	~~Pusha upp ändringen till Gerrit för granskning och sen merge till master branchen:~~
	~~{{Cmd\|git push origin HEAD:refs/for/master}}~~

	~~Logga in på Gerrit webben och kolla att din kommitt kommit upp för granskning och test. Du bör hitta den under t.ex. My => Changes.~~

	[[Category:Guide]]		[[Category:Guide]]

Peter at 18:49, 11 May 2013

2013-05-11T18:49:14Z

← Older revision		Revision as of 20:49, 11 May 2013
Line 157:		Line 157:
	[[File:gerrit-review-helloworld-6.png]]		[[File:gerrit-review-helloworld-6.png]]

	== Scenario: Rebase:a innan submit till master ==		== Scenario: Rebase innan submit till master ==

	Då vårt projekt kräver fast-forward, så måste varje patch set baseras på senaste commit:en på master innan de kan accepteras. Anta att vi gjort en ändring på en branch som heter opt-build som vi har pushat till Gerrit för granskning. Men innan vår commit godkänns, så hinner en annan commit komma in på master. Vi kommer då se följande när vi försöker submit:a i Gerrit:		Då vårt projekt kräver fast-forward, så måste varje patch set baseras på senaste commit:en på master innan de kan accepteras. Anta att vi gjort en ändring på en branch som heter opt-build som vi har pushat till Gerrit för granskning. Men innan vår commit godkänns, så hinner en annan commit komma in på master. Vi kommer då se följande när vi försöker submit:a i Gerrit:

Peter at 18:48, 11 May 2013

2013-05-11T18:48:27Z

← Older revision		Revision as of 20:48, 11 May 2013
Line 157:		Line 157:
	[[File:gerrit-review-helloworld-6.png]]		[[File:gerrit-review-helloworld-6.png]]

	== Scenario: ~~Måste rebase~~:a innan submit till master ==		== Scenario: Rebase:a innan submit till master ==

	Då vårt projekt kräver fast-forward, så måste varje patch set baseras på senaste commit:en på master innan de kan accepteras. Anta att vi gjort en ändring på en branch som heter opt-build som vi har pushat till Gerrit för granskning. Men innan vår commit godkänns, så hinner en annan commit komma in på master. Vi kommer då se följande när vi försöker submit:a i Gerrit:		Då vårt projekt kräver fast-forward, så måste varje patch set baseras på senaste commit:en på master innan de kan accepteras. Anta att vi gjort en ändring på en branch som heter opt-build som vi har pushat till Gerrit för granskning. Men innan vår commit godkänns, så hinner en annan commit komma in på master. Vi kommer då se följande när vi försöker submit:a i Gerrit:

Peter: /* Scenario: Måste rebase:a innan submit till master */

2013-05-11T18:47:49Z

Scenario: Måste rebase:a innan submit till master

← Older revision		Revision as of 20:47, 11 May 2013
Line 169:		Line 169:
	Vår commit har nu förändrats, men dess commit-meddelande och därmed också Change-Id är intakt. Vi kan därför direkt push:a till Gerrit och skapa ett patch set 2:		Vår commit har nu förändrats, men dess commit-meddelande och därmed också Change-Id är intakt. Vi kan därför direkt push:a till Gerrit och skapa ett patch set 2:
	{{Cmd\|git push origin HEAD:refs/for/master}}		{{Cmd\|git push origin HEAD:refs/for/master}}

			Nu kan våra ändringar godkännas och submit:as till master:

			[[File:gerrit-review-helloworld-8.png]]

	= (Gammalt) Klona ett repository, förändra och pusha tillbaka till repositoryt för granskning =		= (Gammalt) Klona ett repository, förändra och pusha tillbaka till repositoryt för granskning =

Peter: /* Scenario: Måste rebase:a innan submit till master */

2013-05-11T18:46:22Z

Scenario: Måste rebase:a innan submit till master

← Older revision		Revision as of 20:46, 11 May 2013
Line 162:		Line 162:

	[[File:gerrit-review-helloworld-7.png]]		[[File:gerrit-review-helloworld-7.png]]

			Gå tillbaka till repositoryt och börja med att uppdatera din master:
			{{Cmd\|git checkout master\|git pull}}
			Byt sedan tillbaka till opt-build och rebase:a:
			{{Cmd\|git checkout opt-build\|git rebase master}}
			Vår commit har nu förändrats, men dess commit-meddelande och därmed också Change-Id är intakt. Vi kan därför direkt push:a till Gerrit och skapa ett patch set 2:
			{{Cmd\|git push origin HEAD:refs/for/master}}

	= (Gammalt) Klona ett repository, förändra och pusha tillbaka till repositoryt för granskning =		= (Gammalt) Klona ett repository, förändra och pusha tillbaka till repositoryt för granskning =