Sandro Bonazzola has posted comments on this change. Change subject: vdsm: gluster: provide 3.7 nightly builds ......................................................................
Patch Set 2: (4 comments) https://gerrit.ovirt.org/#/c/39139/2/jobs/packaging/repo_closure_check.sh File jobs/packaging/repo_closure_check.sh: Line 137: --lookaside check-glusterfs-epel-"${distid}"-nightly \ Line 138: --lookaside check-glusterfs-epel-"${distid}" \ Line 139: --lookaside check-glusterfs-noarch-epel-"${distid}" \ Line 140: --lookaside check-base-"${distid}" \ Line 141: --lookaside check-base-i386-$ditsid \ > typo? typo, exposing the fact that the repo wasn't needed. Dropping. Line 142: --lookaside check-patternfly-"${distid}" \ Line 143: --repoid check-custom-$distid Line 144: else Line 145: repoclosure \ Line 149: --repofrompath=check-base-i386-"${distid}","${CENTOS_MIRROR}/${DISTRIBUTION_VERSION}"/os/i386/ \ Line 150: --repofrompath=check-updates-"${distid}","${CENTOS_MIRROR}/${DISTRIBUTION_VERSION}"/updates/x86_64/ \ Line 151: --repofrompath=check-extras-"${distid}","${CENTOS_MIRROR}/${DISTRIBUTION_VERSION}"/extras/x86_64/ \ Line 152: --repofrompath=check-epel-"${distid}","${EPEL_MIRROR}"/epel/"${DISTRIBUTION_VERSION}"/x86_64/ \ Line 153: --repofrompath=check-glusterfs-epel-$distid-nightly,"${GLUSTER_MIRROR}"/pub/gluster/glusterfs/nightly/glusterfs/epel-"${DISTRIBUTION_VERSION}"-x86_64/ \ > missed quotes here Done Line 154: --repofrompath=check-glusterfs-epel-"${distid}","${GLUSTER_MIRROR}"/pub/gluster/glusterfs/LATEST/EPEL.repo/epel-"${DISTRIBUTION_VERSION}"/x86_64/ \ Line 155: --repofrompath=check-glusterfs-epel-noarch-"${distid}","${GLUSTER_MIRROR}"/pub/gluster/glusterfs/LATEST/EPEL.repo/epel-"${DISTRIBUTION_VERSION}"/noarch \ Line 156: --repofrompath=check-jpackage-generic-"${distid}","${JPACKAGE_MIRROR}"/jpackage/6.0/generic/free \ Line 157: --repofrompath=check-patternfly-"${distid}","${COPR}/patternfly/patternfly1/epel-6-x86_64" \ Line 157: --repofrompath=check-patternfly-"${distid}","${COPR}/patternfly/patternfly1/epel-6-x86_64" \ Line 158: --lookaside check-updates-"${distid}" \ Line 159: --lookaside check-extras-"${distid}" \ Line 160: --lookaside check-epel-"${distid}" \ Line 161: --lookaside check-glusterfs-epel-$distid-nightly \ > missed quotes here Done Line 162: --lookaside check-glusterfs-epel-"${distid}" \ Line 163: --lookaside check-glusterfs-noarch-epel-"${distid}" \ Line 164: --lookaside check-base-"${distid}" \ Line 165: --lookaside check-base-i386-$ditsid \ Line 161: --lookaside check-glusterfs-epel-$distid-nightly \ Line 162: --lookaside check-glusterfs-epel-"${distid}" \ Line 163: --lookaside check-glusterfs-noarch-epel-"${distid}" \ Line 164: --lookaside check-base-"${distid}" \ Line 165: --lookaside check-base-i386-$ditsid \ > typo? as before, dropped since it's not useful Line 166: --lookaside check-jpackage-rhel5-"${distid}" \ Line 167: --lookaside check-jpackage-generic-"${distid}" \ Line 168: --lookaside check-patternfly-"${distid}" \ Line 169: --repoid check-custom-$distid -- To view, visit https://gerrit.ovirt.org/39139 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0c56659eb2e768695c6f22f92f44f19587921fae Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Darshan N <dnara...@redhat.com> Gerrit-Reviewer: David Caro <dcaro...@redhat.com> Gerrit-Reviewer: Eyal Edri <ee...@redhat.com> Gerrit-Reviewer: Max Kovgan <m...@redhat.com> Gerrit-Reviewer: Sahina Bose <sab...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
