Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ci-signal-reporter, github report bug, type miss match #2473

Closed
leonardpahlke opened this issue Mar 23, 2022 · 1 comment · Fixed by #2474
Closed

ci-signal-reporter, github report bug, type miss match #2473

leonardpahlke opened this issue Mar 23, 2022 · 1 comment · Fixed by #2474
Assignees
@RobertKielty
Labels
area/release-eng Issues or PRs related to the Release Engineering subproject kind/bug Categorizes issue or PR as related to a bug. needs-priority sig/release Categorizes an issue or PR as relevant to SIG Release.

Comments

@leonardpahlke
Copy link
Member

What happened:

After the most recent change to the CI Signal Report CLI which uses the new CI Signal project board via a GitHub GraphQL libary now does not run when a GH report generated.
From the error message it seems to be a type missmatch / convertion bug.

FATA[0000] getting GitHub report data: json: cannot unmarshal number into Go struct field GitHubProjectBoardFieldSettings.width of type string 
exit status 1

How to reproduce it (as minimally and precisely as possible):

Run the report and you should see the error message above

/assign @RobertKielty
@leonardpahlke leonardpahlke added area/release-eng Issues or PRs related to the Release Engineering subproject kind/bug Categorizes issue or PR as related to a bug. sig/release Categorizes an issue or PR as relevant to SIG Release. labels Mar 23, 2022
@leonardpahlke leonardpahlke changed the title ci-signal-reporter, automatically generate weekly ci signal report kind/feature needs-priority ci-signal-reporter, github report bug, type miss match Mar 23, 2022
@RobertKielty
Copy link
Member

RobertKielty commented Mar 23, 2022 •
edited
Loading

Reproduced locally on go version go1.17.6 linux/amd64

Flipping the type from string to int on the referenced field fixes the report, PR to follow.

@RobertKielty RobertKielty mentioned this issue Mar 23, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RobertKielty RobertKielty

Labels
area/release-eng Issues or PRs related to the Release Engineering subproject kind/bug Categorizes issue or PR as related to a bug. needs-priority sig/release Categorizes an issue or PR as relevant to SIG Release.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ci-reporter: fixes broken report
3 participants
@RobertKielty @k8s-ci-robot @leonardpahlke