Sandro Bonazzola has posted comments on this change.

Change subject: Add generic code to check repoclosure
......................................................................


Patch Set 8:

(2 comments)

http://gerrit.ovirt.org/#/c/24802/8/jobs/packaging/repo_closure_check.sh
File jobs/packaging/repo_closure_check.sh:

Line 83:                                
--repofrompath=check-extras,http://centos.mirror.constant.com/"${DISTRIBUTION_VERSION}"/extras/x86_64/
 \
Line 84:                                
--repofrompath=check-epel,http://linux.mirrors.es.net/fedora-epel/"${DISTRIBUTION_VERSION}"/x86_64/
 \
Line 85:                                
--repofrompath=check-glusterfs-epel,http://download.gluster.org/pub/gluster/glusterfs/LATEST/EPEL.repo/epel-6/x86_64/
 \
Line 86:                                
--repofrompath=check-glusterfs-epel-noarch,http://download.gluster.org/pub/gluster/glusterfs/LATEST/EPEL.repo/epel-6.4/noarch
 \
Line 87:                                
--repofrompath=check-jpackage-rhel5,ftp://jpackage.hmdc.harvard.edu/JPackage/6.0/redhat-el-5.0/free
 \
why rhel5?
Line 88:                                
--repofrompath=check-jpackage-generic,ftp://jpackage.hmdc.harvard.edu/JPackage/6.0/generic/free
 \
Line 89:                                -l check-updates \
Line 90:                                -l check-extras \
Line 91:                                -l check-epel \


Line 101:                       
custom_url="${BASE_URL}/${REPO_NAME}/rpm/${repo}"
Line 102:                       repoclosure \
Line 103:                               -t \
Line 104:                               
--repofrompath=check-custom,"${custom_url}" \
Line 105:                               
--repofrompath=check-fedora,http://mirrors.kernel.org/fedora/releases/"${DISTRIBUTION_VERSION}"/Everything/x86_64/os/
 \
why different mirror here?
Line 106:                               
--repofrompath=check-updates,http://mirrors.servercentral.net/fedora/updates/"${DISTRIBUTION_VERSION}"/x86_64/
 \
Line 107:                               
--repofrompath=check-updates-testing,http://mirrors.servercentral.net/fedora/updates/testing/"${DISTRIBUTION_VERSION}"/x86_64/
 \
Line 108:                               -l check-fedora \
Line 109:                               -l check-updates \


-- 
To view, visit http://gerrit.ovirt.org/24802
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I762abf97c9020301a388f7d83c7d1e41bdf1f062
Gerrit-PatchSet: 8
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Kiril Nesenko <knese...@redhat.com>
Gerrit-Reviewer: David Caro <dcaro...@redhat.com>
Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ew...@kohlvanwijngaarden.nl>
Gerrit-Reviewer: Eyal Edri <ee...@redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knese...@redhat.com>
Gerrit-Reviewer: Ohad Basan <oba...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to