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.

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

Add step that generate issues based on JUnit test results #38270

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

ArthurSens
Copy link
Member

Description

Now that we're uploading files correctly (#38236), this PR extends the current unit test workflow to download the artifacts and calling issuegenerator to create the issues.

Link to tracking issue

Related to #36761

@ArthurSens
Copy link
Member Author

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
@ArthurSens
Copy link
Member Author

@mx-psi, I just wanted to let you know that I believe I'll need to patch issuegenerator to make this work.

As seen in this workflow run, issuegenerator is failing with this error:

2025-03-11T22:06:17.144Z	FATAL	issuegenerator@v0.20.0/main.go:244	Failed to search GitHub Issues	{"error": "GET https://api.github.com/repos/open-telemetry/open-telemetry/opentelemetry-collector-contrib/issues?state=open: 404 Not Found []"}

It is writing open-telemetry twice in the URL. See the difference by clicking:

The reason behind this is that the ENV variable GITHUB_REPOSITORY doesn't return only the repository but a string like <owner>/<repository>, as shown in the docs.

Issuegenerator is building this API call by concatenating the ENV variables, GITHUB_REPOSITORY_OWNER and GITHUB_REPOSITORY, and since the latter already contains the owners, we're duplicating that information. See code references [1], [2] and [3].

@ArthurSens
Copy link
Member Author

Opened open-telemetry/opentelemetry-go-build-tools#719 to address the problem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant