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

Investigate current-bench removing statuses of other checks on master #440

Open
punchagan opened this issue May 3, 2023 · 4 comments
Open

Comments

@punchagan
Copy link
Contributor

Reported here by @kit-ty-kate

@punchagan
Copy link
Contributor Author

Looks like this is the offending commit.

@punchagan
Copy link
Contributor Author

The logs for the set-status job look like this:

2023-04-25 17:53.03: New job: Set ocaml/opam refs/heads/master
                     e66648f0/ocaml-benchmarks to
                     {"state":"pending","target_url":"https://autumn.ocamllabs.io/ocaml/opam"}
2023-04-25 17:53.03: Waiting for resource in pool github
2023-04-25 17:53.07: Got resource from pool github
2023-04-25 17:53.07: POST https://api.github.com/repos/ocaml/opam/statuses/e66648f07b950e5ba144a477432f437bfd8059ec:
                       {
                         "context": "ocaml-benchmarks",
                         "state": "pending",
                         "target_url": "https://autumn.ocamllabs.io/ocaml/opam"
                       }
2023-04-25 17:53.08: Job succeeded

I've looked at the code, and tried to look at the API docs and it doesn't seem like something like this would "delete" other statuses.

@kit-ty-kate
Copy link

The same thing happened for already opened PRs on ocaml/opam

@punchagan
Copy link
Contributor Author

Okay, thanks for the additional information.

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

No branches or pull requests

2 participants