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

Public CI metaissue #2565

Open
6 of 8 tasks
nomeata opened this issue Jun 9, 2021 · 9 comments
Open
6 of 8 tasks

Public CI metaissue #2565

nomeata opened this issue Jun 9, 2021 · 9 comments
Labels
P1 high priority, resolve before the next milestone testing Related to test suite

Comments

@nomeata
Copy link
Collaborator

nomeata commented Jun 9, 2021

This issue tracks various refinements we want to do to our CI infrastructure post-open source.

@nomeata
Copy link
Collaborator Author

nomeata commented Jun 9, 2021

Useful article about pull_request and pull_request_target, and the security implications of PRs from other repositories: https://securitylab.github.com/research/github-actions-preventing-pwn-requests/

@nomeata
Copy link
Collaborator Author

nomeata commented Jun 9, 2021

Discussing nix-based CI options on the Nix Discourse: https://discourse.nixos.org/t/github-actions-and-or-with-hercules-ci/13537

@rossberg rossberg added P1 high priority, resolve before the next milestone testing Related to test suite labels Jun 29, 2021
@osa1
Copy link
Contributor

osa1 commented Aug 18, 2021

"CI: Publish reports" this is not something we do today, right?

I'm thinking what's left to do before stopping using Hydra. I think once we port posting perf and size changes to GH CI we should be able ditch it?

@nomeata
Copy link
Collaborator Author

nomeata commented Aug 18, 2021

"CI: Publish reports" this is not something we do today, right?

Yes, we have direct links to the latest flame graph, documentation, etc. for each PR. Would be a shame to lose that, but maybe ok temporarily until we have a good idea. I should research what kind of services are there out there.

Besides that it's just the perf changes comments, which I can look into this week maybe

@osa1
Copy link
Contributor

osa1 commented Aug 18, 2021

Yes, we have direct links to the latest flame graph, documentation, etc. for each PR.

Interesting, I didn't know that. Where can I see these reports?

@nomeata
Copy link
Collaborator Author

nomeata commented Aug 18, 2021

You didn't know? Anyways, see #2562 for some of the links, which we removed from the README to keep up appearances.

@nomeata
Copy link
Collaborator Author

nomeata commented Aug 18, 2021

Flame graphs at https://hydra.dfinity.systems/job/dfinity-ci-build/motoko/tests.profiling-graphs/latest/download/1/index.html I assume (I can't check, of course). One can construct similar urls for each PR. I hope to preserve that feature, or improve it (easier discoverability in particular of course). But needs not block turning off hydra.

@nomeata
Copy link
Collaborator Author

nomeata commented Aug 21, 2021

Maybe we can sign up to coveralls.io (free for open repos) and upload coverage data there. Supports OCaml (https://github.com/aantron/bisect_ppx#Coveralls). We used to have coverage reports, but they have somewhat bitrotted.

nomeata added a commit that referenced this issue Sep 8, 2021
this fixes #2565 by removing the remaining files that are only there for
Hydra’s sake, and updates the docs to remove mention of hydra.
@nomeata
Copy link
Collaborator Author

nomeata commented Feb 11, 2022

We might very easily be able to speed up our CI system by hooking up to (and paying for) https://nixbuild.net/, with no noticable changes to our existing workflows.

Would it be possible to get signed up for this? I'll then set it up.

Oh, no Mac support it seems. Maybe not then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 high priority, resolve before the next milestone testing Related to test suite
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants