Skip to content

Conversation

@fargito
Copy link
Contributor

@fargito fargito commented Jan 7, 2026

Hello, CodSpeed team member here.

New performance tests were added in #11080. However, contrary to 'Tests - Performance' job, OIDC is not used to authenticate the new job with CodSpeed. We don't currently gracefully handle the case where one job is using OIDC and the other one is not.

More details on how CodSpeed uses OIDC to authenticate workflows here.

The OIDC permissions allow CodSpeed to authenticate the workflow.
They are the same as the 'Tests - Performance' workflow.
@netlify
Copy link

netlify bot commented Jan 7, 2026

Deploy Preview for inventree-web-pui-preview canceled.

Name Link
🔨 Latest commit b860f2d
🔍 Latest deploy log https://app.netlify.com/projects/inventree-web-pui-preview/deploys/695e4810d095030008739851

@matmair matmair added the CI CI / unit testing ecosystem label Jan 7, 2026
@matmair matmair added this to the 1.2.0 milestone Jan 7, 2026
@matmair
Copy link
Contributor

matmair commented Jan 7, 2026

Thank you for the fix @fargito, the error was not really clear on what the issue was.
@SchrodingersGat IMO this does not need a full CI run as it only changes permission values and should solve #11073 (comment)

@SchrodingersGat SchrodingersGat merged commit 5448e11 into inventree:master Jan 7, 2026
29 checks passed
@fargito fargito deleted the ci/add-missing-oidc-permissions branch January 7, 2026 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI CI / unit testing ecosystem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants