David Caro has posted comments on this change.

Change subject: Added standard mock runner
......................................................................


Patch Set 4: Code-Review-1 Workflow+1

(3 comments)

https://gerrit.ovirt.org/#/c/40393/4/mock_configs/mock_runner.sh
File mock_configs/mock_runner.sh:

Line 37: 
Line 38:     Where the ID is the name used for it on the requirements file 
(usually in
Line 39:     the style fc19, el6 or similar) and MOCK_CONF is the mock 
configuration
Line 40:     name as passed to the mock -r option (usually, the configuration 
name
Line 41:     without extension, for example fedora-22-x86_64)
also explain the short chroot matching
Line 42: 
Line 43:     Options:
Line 44:         -h|--help
Line 45:             Show this help


Line 89:     To run only the build artifacts script on fedoras 21 and 22
Line 90:     > $0 --build-only fc21:fedora-21-x86_64 fc22:fedora-22-x86_64
Line 91: 
Line 92:     To open a shell to debug the check-merged script on el7
Line 93:     > $0 --merge-only --shell el7:epel-7-x86_64
add the short chroot specification
Line 94: 
Line 95: EOH
Line 96:     return 0
Line 97: }


Line 245:         fi
Line 246:     done
Line 247:     return 1
Line 248: }
Line 249: 
newline
Line 250: run_shell() {
Line 251:     local mock_env="${1?}"
Line 252:     local script="${2?}"
Line 253:     local rebuild="${3:-false}"


-- 
To view, visit https://gerrit.ovirt.org/40393
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbe80f0001affb0d16cb1db680f8f02283824cec
Gerrit-PatchSet: 4
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: David Caro <[email protected]>
Gerrit-Reviewer: Barak Korren <[email protected]>
Gerrit-Reviewer: David Caro <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Eyal Edri <[email protected]>
Gerrit-Reviewer: Max Kovgan <[email protected]>
Gerrit-Reviewer: Paz Dangur <[email protected]>
Gerrit-Reviewer: Sagi Shnaidman <[email protected]>
Gerrit-Reviewer: Sharon Naftaly <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to