Description of problem: As the mapping of test name (e.g. /CoreOS/openssh/Multihost/gssapi) to GIT repository is not easy to do properly (i.e. hard to guess) and this mapping does not seem to be anywhere, I am proposing to add this information to rpm metadata which can be then found easily from the test name: # TEST="/CoreOS/openssh/Multihost/gssapi" # repoquery --info \*${TEST//\//-} | grep URL URL : git://pkgs.devel.redhat.com/tests/openssh I understand that to be able to rely on this all tests would need to be resubmitted, but anyway, even without that this makes more sense then the current default FAKEURL value :) I am attaching patch to make this work. Version-Release number of selected component (if applicable): rhts-4.70-1
Created attachment 1164372 [details] Patch for the RFE against master/HEAD of rhts git repository
Dear Miroslav, does this bug need a NEEDINFO from Dan? Maybe you wanted to set the review flag?
Regardless... Yes this sounds like a great idea. The patch looks okay, I would suggest it might be simpler to just make rhts-mk-build-package figure out the SCM URL instead of doing it in the Makefile. I will have a go at adjusting the patch and post it to Gerrit.
Well, it does not, that is just my way how to get attention :) Sorry for the confusion, we do not use review flag in my team, but I will try to remember that when attaching patches next time. (In reply to Roman Joost from comment #2) > Dear Miroslav, > > does this bug need a NEEDINFO from Dan? Maybe you wanted to set the review > flag?
(In reply to Dan Callaghan from comment #3) > Regardless... Yes this sounds like a great idea. > > The patch looks okay, I would suggest it might be simpler to just make > rhts-mk-build-package figure out the SCM URL instead of doing it in the > Makefile. I will have a go at adjusting the patch and post it to Gerrit. Yes, sure, I was thinking about that too, but this seemed to me as a more generic solution. Anyway, I am ok with doing this in python right away. And BTW, getting the URL can be done simply by: git config --get remote.origin.url Which works in RHEL6 and RHEL7 just fine.
I wonder if we should also include the subdirectory in the # anchor, as in: git://git.beaker-project.org/beaker-core-tasks#virt/install so that you can distinguish between multiple tasks living in subdirectories under a single git repository. The # anchor syntax matches what restraint accepts.
http://gerrit.beaker-project.org/4966 I added in svn and cvs support too although I think everyone is keeping their tasks in git these days.
(In reply to Dan Callaghan from comment #6) > I wonder if we should also include the subdirectory in the # anchor, as in: > > git://git.beaker-project.org/beaker-core-tasks#virt/install > > so that you can distinguish between multiple tasks living in subdirectories > under a single git repository. The # anchor syntax matches what restraint > accepts. +1 ... that sounds like a neat idea.
Tagged as rhts 4.71.
rhts 4.71 has been released.