-
Notifications
You must be signed in to change notification settings - Fork 104
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
devstats,repo_groups: generate repo_groups.sql #332
base: main
Are you sure you want to change the base?
Conversation
Skipping CI for Draft Pull Request. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
0b86fd4
to
67fd05c
Compare
67fd05c
to
a7e1bac
Compare
Updated PR to generate like this, from comment to cncf/devstats#77, where that PR was initially created from a previous version which contained SQL syntax errors, i.e. comma missing etc. |
@xpivarc @jean-edouard may I ask for a review? |
Signed-off-by: Daniel Hiller <[email protected]>
a7e1bac
to
09dcd18
Compare
Tested in job https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/logs/postsubmit-community-update-kubevirt-devstats-repo-sql/1848991035489783808 |
@dhiller if there a way to see the end result? how is this going to look like? |
What this PR does / why we need it:
CNCF devstats project provides dashboards for KubeVirt that show GitHub data 2. This PR is a required part to keep the repository groups of the user statistics by repository 3 in sync with the content of
sigs.yaml
. PR kubevirt/project-infra#3701 will change the job to use this tool.This PR adds a go tool that generates SQL statements to update repository groups for CNCF devstats 1.
See #330 for more details.
Tested in job https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/logs/postsubmit-community-update-kubevirt-devstats-repo-sql/1848991035489783808
which produced changes reflected from PR #334 in a PR against repo_groups.sql for KubeVirt: https://github.com/cncf/devstats/pull/80/files
Background
KubeVirt repository groups are updated with an SQL script 4 - we want these repository groups to be updated automatically whenever there's changes in our sigs.yaml.
A while ago we had decided to directly align our repository groups to the ownership references inside the sig definitions - in effect this maps any change in a repository to one or more SIGs.
When calling the tool, i.e.
it generates SQL code that is targeted to update the kubevirt repo_groups
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer: