This repository has been archived by the owner on Aug 27, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 62
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* add troubleshooting guide * review fixes
- Loading branch information
1 parent
3077485
commit 3a1a992
Showing
5 changed files
with
56 additions
and
0 deletions.
There are no files selected for viewing
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,55 @@ | ||
# Troubleshooting Zappr | ||
|
||
This page tries to explain common sources of error and give instructions how to fix them. | ||
|
||
Terminology: | ||
|
||
* Zapprfile: A file named `.zappr.yaml` located in the root directory of your repository. | ||
|
||
## I can't enable or disable checks for my repository | ||
|
||
![A Github 404 API error is visible](img/troubleshooting/no-enable-disable.png) | ||
|
||
You don't have admin access for this repository. Either become an admin yourself or ask one to toggle this check. | ||
|
||
## I updated my Zapprfile in a PR, but it's not picked up. | ||
|
||
Zappr reads your Zapprfile from the repository's default branch (it's `master` by default). Changes on a feature branch are ignored. | ||
|
||
## I enabled Zappr, but approvals don't seem to count | ||
|
||
If you have the option `approvals.from` configured, which can look like this in your Zapprfile: | ||
|
||
~~~ yaml | ||
approvals: | ||
from: | ||
orgs: | ||
- zalando | ||
~~~ | ||
|
||
And Zappr's status looks like this: | ||
|
||
![Zappr reports pending status with zero approvals](img/troubleshooting/no-approval-counted.png) | ||
|
||
Then you need to make sure that | ||
|
||
* This organization exists in the GitHub instance (github.com or your GHE) | ||
* Everybody who is supposed to approve is a **public** member of this organization | ||
|
||
## I enabled Zappr, but it hangs with status "pending" | ||
|
||
![GitHub says "Waiting for status to be reported"](img/troubleshooting/no-success-status.png) | ||
|
||
Until now there were two reasons why this happened: | ||
|
||
### The 99% case | ||
|
||
The person who enabled the Zappr check does not have access to the repository anymore, for whatever reason. Since Zappr stores a personal access token for every check (to communicate with GitHub during check execution), it's a problem if the permissions of this token change. | ||
|
||
**Solution**: Disable and enable the Zappr check to replace the stored token. | ||
|
||
### The case that happened only once | ||
|
||
There is a limit of 1000 status updates per combination of context (like `zappr`, `zappr/pr/specification`...) and commit ID. If there are A LOT of comments in a PR, this limit might be exceeded. | ||
|
||
**Solution**: Add an additional commit to the PR. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters