#120 changing anongiturl to use src.fp.o instead of pkgs.fp.o. fixes #119
Closed 6 years ago by cqi. Opened 6 years ago by tflink.

@@ -3,7 +3,7 @@ 

  lookasidehash = sha512

  lookaside_cgi = https://src.stg.fedoraproject.org/repo/pkgs/upload.cgi

  gitbaseurl = ssh://%(user)s@pkgs.stg.fedoraproject.org/%(module)s

- anongiturl = git://pkgs.stg.fedoraproject.org/%(module)s

+ anongiturl = https://src.stg.fedoraproject.org/git/%(module)s

  branchre = f\d$|f\d\d$|el\d$|olpc\d$|master$

  kojiconfig = /etc/koji.conf.d/stg.conf

  build_client = koji

file modified
+1 -1
@@ -3,7 +3,7 @@ 

  lookasidehash = sha512

  lookaside_cgi = https://src.fedoraproject.org/repo/pkgs/upload.cgi

  gitbaseurl = ssh://%(user)s@pkgs.fedoraproject.org/%(module)s

- anongiturl = git://pkgs.fedoraproject.org/%(module)s

+ anongiturl = https://src.fedoraproject.org/git/%(module)s

  branchre = f\d$|f\d\d$|el\d$|olpc\d$|master$

  kojiconfig = /etc/koji.conf

  build_client = koji

Signed-off-by: Tim Flink tflink@fedoraproject.org

Changes anongiturl to use https and src.fedoraproject.org to fit changes to how dist-git is hosted from late last year

Does Koji work with https://src.stg.fedoraproject.org/git/%(module)s now?

This https anonymous link was reverted from this change before, please refer to https://pagure.io/fedpkg/pull-request/106

Koji needs the url to be git+https://… and then it can access it (can be tested with a scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=19122540). However with such config an anonymous clone will fail.

Could you also explain what problem you are trying to solve with this?

The problem is that fedpkg clone -a now fails when used from inside Fedora Infra:
https://phab.qa.fedoraproject.org/T944
That involves all tasks executed by Taskotron.

My understanding is that @tflink discussed this with @puiterwijk and he was told that the anonymous clone URL was incorrect and should be changed anyway. But seeing that Koji is unable to work with the new URL, maybe we can discuss this Fedora Infra again and come up with a temporary solution until Koji is fixed.

@puiterwijk solved the infra problem, so this change is no longer critical for our use case.

@lsedlar I just tried a scratch build with same prefix git+https://, but it failed. Anything changed?

[1] https://koji.fedoraproject.org/koji/taskinfo?taskID=22080126

I think this commit disabled that particular form of URL.

I cherry pick this commit and make a new PR with related change for fedpkg together. I'm going to close this PR. Thank you very much.

UPDATE:

That new PR is https://pagure.io/fedpkg/pull-request/147

Pull-Request has been closed by cqi

6 years ago