CI session
==========
- We should *only* merge code where the CI is not failing.
- We should consider having the test-network-all tests running as well as stem
tests.
- Sometimes consensus might fail because of the way we fire off timers eagerly.
- Cascading failure modes for consensus, we might happen for the public
network, but it's more unlikely.
- Let's start by having it as allow to fail tests on Travis while we test it.
- We should put high priority to always fixing CI failures.
- We should ensure CI passes before we merge more stuff.
- Should we have asn/dgoulet create PR's as part of the review ticket triage:
yes, we do, but only for a test period where they note down how much time
they spend.
- CI/coverity role creates a ticket when there is CI failure and tries to
ensure that it gets assigned to the right people (subsystem maintainer).
- Create ticket when a test has been disabled due to CI failures. Review the
list of tickets at the upcoming network-team meeting.
- AppVeyor is currently broken.
- If we don't watch AppVeyor we should watch jenkins for Windows' failures.
- For volunteers either put their code in a PR and/or put the ticket into needs_info
- Rationales for CI:
- When CI is broken on master, it's a lot of work to figure out if it's
master or your own branch that is the problem.
Comments
Please register or sign in to add a comment.