search

open-services-group / community

This repository handles a few common things, it is mainly used by our bots...
GNU General Public License v3.0
8 stars 24 forks source link

[SIG Data Science][SP Metrics] Add subproject repo urls to sigs.yaml #92

Closed chauhankaranraj closed 2 years ago

chauhankaranraj commented 2 years ago

As a part of the Metrics subproject, we want to collect the github data corresponding to all the subprojects under various sigs. For this, we need to have a list of repo's corresponding to each subproject. However, repo url's are not currently listed in sigs.yaml. Could we update it and add them?

Alternatively, we could also parse the repo slugs from the owners file entries in sigs.yaml. Here, however, we noticed that the urls are inconsistent across subprojects - for example see this and this. Should we also update these to be uniform across the board?

/cc @hemajv @Shreyanand

Shreyanand commented 2 years ago

/cc @schwesig

schwesig commented 2 years ago

@Shreyanand thanks for cc, I will take care of this

/cc @codificat

schwesig commented 2 years ago

see @codificat link to #94, we not always have a specified repo for all groups. kind of a catch 22 and we need maybe first a decision, if we first need to clean the "repo structure" or collecting the "connections" not fully and manually added?

codificat commented 2 years ago

I believe we should properly identify the repos and update sigs.yaml accordingly.

Besides that though, should we also consider SIGs in related projects (e.g. Thoth) or only the reference from this repo?

durandom commented 2 years ago

can't we just use the OWNERS link as a proxy? No OWNERS no reporting. That would trickle down do all repos maintaining an OWNERS file too

schwesig commented 2 years ago

can't we just use the OWNERS link as a proxy?

what do you mean by "proxy" ?

That would trickle down do all repos maintaining an OWNERS file too

and yes, that was in the plan. But I didn't want to wait on this to long and blocking the metrics, MVP.

chauhankaranraj commented 2 years ago

can't we just use the OWNERS link as a proxy?

Yep! That's what I meant when I said this:

Alternatively, we could also parse the repo slugs from the owners file entries in sigs.yaml.

Just wanted to point out that in order to do so, we'd have to make sure the URLs in the owners field all follow the same format. Right now, some look like https://raw.githubusercontent.com/org/repo/master/OWNERS while others look like https://github.com/org/repo/blob/main/OWNERS.

I can send a PR and fix it if that helps :)