Re: Cleaning up the test-* namespaces

[Igor Gnatenko]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

On Wed, 2017-08-23 at 20:43 +0200, Pierre-Yves Chibon wrote:
>  Good Morning Everyone,
>  
>  A while ago we had the question about including tests in dist-git. We
>  considered
>  two options:
>  - Have the tests in the same git repos as the files to generate the
>  artifact
>  - Have the tests in a separate namespace test-<namespace>
>  
>  While we weren't quite sure what was the best approach, we put in
>  place a
>  mechanism to automatically create the project in the test-* namespace
>  when it
>  was created in the regular namespace and using the same ACLs.
>  
>  To be more concrete, we had the mechanism that was creating for every
>  repository
>  in the `rpms` namespace a corresponding repository in the `test-rpms` 
>  namespace
>  with the same maintainers and ACLs.
>  
>  The recent work going on around CI [1] has led to the decision that
>  tests stored
>  in dist-git will be held in the same git repository as the spec files
>  (for rpms).
>  
>  Now some numbers:
>  * We have currently 43663 repositories in dist-git
>  grep 'repo ' gitolite.conf |wc -l
>  43663
>  * 20843 of them correspond to the test-* namespaces
>  grep 'repo test-' gitolite.conf |wc -l
>  20843
>  * Leaving 22820 repositories not in the test-* namespaces
>  grep 'repo ' gitolite.conf |grep -v 'repo test-' |wc -l
>  22820
>  (I double checked, 20843+22820=43663 so we're good there)
>  
>  I would like to get ride of the test-* namespaces. Tim Flink for
>  which we
>  originally added this mechanism agrees with it. We also have backups
>  in case there
>  is something needed from there sometime in the future.
>  
>  Before I do so, I would like to know if anyone is opposed to this
>  idea of
>  removing the test-* namespaces from dist-git? 
I only support this decision! 😉

IMO all tests should live either in rpms/$pkg or module/$module repos,
it doesn't make any sense to keep tests out. This also complicates life
when you update package and you want to change test at the same time.
>  
>  
>  Thanks in advance for your help,
>  Pierre
>  
>  [1] http://fedoraproject.org/wiki/CI
>  _______________________________________________
>  devel-announce mailing list -- devel-announce(a)lists.fedoraproject.org
>  To unsubscribe send an email to devel-announce-leave(a)lists.fedoraproj
>  ect.org
>  _______________________________________________
>  devel mailing list -- devel(a)lists.fedoraproject.org
>  To unsubscribe send an email to devel-leave(a)lists.fedoraproject.org 
- -- 
- -Igor Gnatenko
> -----BEGIN PGP SIGNATURE-----
>
> iQIzBAEBCAAdFiEEhLFO09aHZVqO+CM6aVcUvRu8X0wFAlmedcsACgkQaVcUvRu8
> X0zOURAApOEsrxpY6fM8E9zn8pwayLgGhdYp8YIGOLqcb+whxJbHPazsuX7m2108
> YHFC3akTCloMGA8NkgaGYs/2m2XqbPYt5JXZqxV5O75NVKyWORqHac4c8oUDRJ17
> PRlq+NqQY3Z6XbY9p76MZnYvyGTmUxWahBbMmLqU8/vVrdVlWJ1k5YcC099OR6GX
> We1FEI/f/tPEJD5X0tNuO8lBZBzpZDnV30L2hDws/R4m1G6vNc1LF29qREZSzL25
> WnGXbHn5FHsF4Iy4+dFV4U2PZ47n9XSTprquAqEsKTw7JwP3Re1+kuE986NTe//a
> eoJ2dCelSkXwIzyav2alNjznoSsXeTHF8fUiQwK9UlDqVOZZ0zkOUNVEQu/ZyJ+d
> xXiRV3II95NTplki/Z1iAahzlte/eFmRXC5RTO23mrMMk5oOuVaO+/k3Gl4Pg5Vl
> pMoUaBuItwOPu/d0daaByhI2IKBsAU2rXfpteP4qRxJWmvFZvIf1IU1882oHOvy9
> oPYDSm22omshwSn8NbEtfwUeLBpSCAJlRJBgvIMR3Fnw5Ux+5j8c+vZwqzQRnt6Q
> IAvXlSQ6nA3SrDtXrke5aVN/tfsfhRJsO6mpe0qGn4iiVK88Ow1DOnwBpbzzhLgm
> ev//qCjq/8zFX0kzuBmkI/HPGJAq+m+GQKjmqVouJDytF0EMycE=
> =CqNk
> -----END PGP SIGNATURE-----