#424 Adjust fedpkg for the new default branch in dist-git: rawhide
Merged by onosek. Opened by pingou.
pingou/fedpkg default_rawhide  into  master

Download 424.patch

In some places we also reference main which will be a symlink to
the rawhide branch and we currently still reference master to be
backward compatible (even though once the rawhide branch appears
the master branch will be removed and blocked).

Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr

This is based on a quick git grep and quite un-tested. The test suite fails for me in tox as well as when running nosetests manually :(

rebased onto fb50218267327bbd09b4692fa17b7cf824e89d19

rebased onto 885043c9d2cb883f03a747c6d9a3c858a0d8864c

1 new commit added

  • More updates to 'rawhide' default branch

I added more changes, but this also needs changes to tests.

These changes to tests are sufficient to make them pass locally for me.
https://pagure.io/fork/lsedlar/fedpkg/c/6f9b0667800b8fad5fcec2afde2617f8c8e9d0fe?branch=fix-review-424

Is the tooling that handles new branches updated to handle requests for rawhide branches? This patch would change the JSON that is submitted to list rawhide instead of master.

These changes to tests are sufficient to make them pass locally for me.
https://pagure.io/fork/lsedlar/fedpkg/c/6f9b0667800b8fad5fcec2afde2617f8c8e9d0fe?branch=fix-review-424

I've enabled PRs on my fork so you can open a PR against my fork ;-)

Is the tooling that handles new branches updated to handle requests for rawhide branches? This patch would change the JSON that is submitted to list rawhide instead of master.

It's on the todo @humaton and @mohanboddu should be looking at this iirc

1 new commit added

  • Update tests for rawhide branch rename

considering the state of these two projects, I doubt we'll change their default branches

FTR I didn't mean the default branch of the request repo itself, but the content of submitted requests:
https://pagure.io/releng/fedora-scm-requests/issue/31722
With this patch, branch would switch from master to rawhide, which is probably a wanted change, and I just want to make sure tools processing the ticket can handle it.

pretty please pagure-ci rebuild

Isn't this should be rawhide as well and while we are at it, better change the test name as well.

Maybe keep this one as is as it shows the backward compatibility and add a new one?

Maybe keep this one as is as it shows the backward compatibility and add a new one?

Sounds good to me.

1 new commit added

  • Adding test_expand_rawhide test

4 new commits added

  • Adding test_expand_rawhide test
  • Update tests for rawhide branch rename
  • More updates to 'rawhide' default branch
  • Adjust fedpkg for the new default branch in dist-git: rawhide

Looks all good to me! :)

Code looks correct. Tests are also passing. Don't know why they were not exected automatically, but I run Jenkins manually. I took a patch to build fedpkg scratch build. I tried at least building functionality from dist-git branch which has old master branch. It worked.
My only concern is whether fedora-scm (already) creates rawhide branches for fedpkg requests. Will they create "symlink" branches or just a single one?
There are 4 commits in the PR. They look complicated because at least first two commits are interfering together. The best approach for me would be 2 commits for functionality and unittests. If you don't mind, I can try to transform it during the merge. I would like to preserve authors (in case of 2 authors, I would name one of you in the commit message).

OK, answering to myself :)
https://fedoraproject.org/wiki/Changes/GitRepos-master-to-main:
Along with the src.fedoraproject.org changes, we will update the branch filter list and forbid 'master' from being pushed at all. Additionally, we will establish a sym-ref between 'rawhide' and 'main' as a convenience for people who prefer to use 'main' branch for rawhide.

My only concern is whether fedora-scm (already) creates rawhide branches for fedpkg requests. Will they create "symlink" branches or just a single one?

@humaton @mohanboddu do you know the status of fedscm-admin?

My only concern is whether fedora-scm (already) creates rawhide branches for fedpkg requests. Will they create "symlink" branches or just a single one?

@humaton @mohanboddu do you know the status of fedscm-admin?

fedscm_admin creates the main "symlink".

fedscm_admin creates the main "symlink".

And can handle new package/branch requests asking for "rawhide" instead of "master"?

Commit 34a971b2 fixes this pull-request

Pull-Request has been merged by onosek

Didn't find where the fedora-scm-request execution code resides, so I hope it is working as expected.
I regrouped 4 commits - resulting in 2. And I am going to create and release a patch.
Thank you for the code, guys.

All fedpkg updates are listed here:
https://bodhi.fedoraproject.org/users/onosek

Metadata