Skip to content

Document opting out of Fedora CI#1114

Merged
centosinfra-prod-github-app[bot] merged 1 commit intopackit:mainfrom
m-blaha:fedora-ci-opt-out
Mar 24, 2026
Merged

Document opting out of Fedora CI#1114
centosinfra-prod-github-app[bot] merged 1 commit intopackit:mainfrom
m-blaha:fedora-ci-opt-out

Conversation

@m-blaha
Copy link
Member

@m-blaha m-blaha commented Mar 18, 2026

@centosinfra-prod-github-app
Copy link
Contributor

Copy link
Member

@lbarcziova lbarcziova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

otherwise LGTM!

Comment on lines +28 to +39
## Opting out of Log Detective

[Log Detective](https://log-detective.com/) analyzes failed Koji build logs to help identify the root cause.
To opt out of this analysis for your project, open a [pull request](https://github.com/packit/deployment/pulls)
adding your project URL to `fedora_ci.disabled_projects_for_logdetective` in the
[deployment configuration](https://github.com/packit/deployment/blob/main/secrets/packit/prod/packit-service.yaml.j2):

```yaml
fedora_ci:
disabled_projects_for_logdetective:
- https://src.fedoraproject.org/rpms/your-package
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's omit this before the feature is available in prod and add the docs once we transition

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@m-blaha, did you have a chance to look into this? Now that we have this in prod, it would be good to update the docs.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was thinking about postponing merging of this PR until the LD is available. But you are right, let's document the part that is already in prod.

@lbarcziova lbarcziova moved this from New to In review in Packit pull requests Mar 23, 2026
@m-blaha m-blaha force-pushed the fedora-ci-opt-out branch from d21e056 to 0373c73 Compare March 24, 2026 08:24
@m-blaha
Copy link
Member Author

m-blaha commented Mar 24, 2026

I removed LD part (and will file it as a separate PR that can be merged after LD is enabled).

@centosinfra-prod-github-app
Copy link
Contributor

Copy link
Member

@lbarcziova lbarcziova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@m-blaha m-blaha added the mergeit Merge via Zuul label Mar 24, 2026
@centosinfra-prod-github-app
Copy link
Contributor

@centosinfra-prod-github-app centosinfra-prod-github-app bot merged commit e1a858c into packit:main Mar 24, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Packit pull requests Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mergeit Merge via Zuul

Projects

Development

Successfully merging this pull request may close these issues.

3 participants