Uh oh!
There was an error while loading. Please reload this page.
CI Rehash - #3146
Conversation
tseaver
commented
Mar 15, 2017
@lukesneeringer I tagged this one "do not merge" because you are still iterating rapidly. |
lukesneeringer
commented
Mar 15, 2017
Thank you! |
lukesneeringer
commented
Mar 15, 2017
This should be almost ready. Because this commit moves literally everything, it triggers all the system tests now, so they need to pass before it is. Once it is ready, I will rebase into a smaller number of commits so it is reviewable on a per-commit basis. Once approved, we will still squash. The rebase will be for review's sake. |
lukesneeringer
commented
Mar 16, 2017
Hey @tseaver -- Do you happen to know off hand what this failure is about? Is it that the test (or backend) is flaky, or is there something else I should be investigating? No need to do a ton of research; this is mostly a lazy if-you-know-off-the-top-of-your-head thing. |
tseaver
commented
Mar 16, 2017
@lukesneeringer I'm not seeing the error detail show up on Circle CI today: just spinning arrows. I do have a pending Spaner flaky-unit-test fix (#3147), which I'm about to merge. |
@tseaver This one is actually a system test; here is the failure: |
lukesneeringer
commented
Mar 17, 2017
I now have CircleCI's build passing...sometimes. We have some system tests that are still flaky and we need to weed out. That said, this is probably close to ready, and still an improvement over where we were. I will rebase tomorrow so that the PR is parse-able and reviewable. |
Also, the total serial test run-time is down to 40 minutes. (I think I could implement some CircleCI caching of the virtualenvs to cut that in half.) |
Depends on nox PR #13 being accepted.
dhermes
commented
Mar 23, 2017
@lukesneeringer Why the merge without review? |
lukesneeringer
commented
Mar 23, 2017
It has already gone through a non-trivial amount of review, by everyone here. The only "unreviewed" work was the AppVeyor thrashing. |
csilvers
commented
Jul 6, 2017
It looks like CONTRIBUTING.rst still talks about running tox. Possibly other documentation does too. Can you fix it to show how to run tests in the new world? I can't figure out how to even install nox, much less run it. |
csilvers
commented
Jul 6, 2017
Ah, I see, this is still open in #3205 |
Restore `index.yml` file removed in PR #3146.
Restore `index.yml` file removed in PR #3146.
This is a continuation of #3135. It is moved to
google-cloud-python:ci(from my personal branch) because system tests do not run from personal branches.This is an attempt to fix the CI situation on this repository. In particular, this will hit the following goals:
Move to parallel builds on a per-API basis.Superceded by not running unnecessary APIs.In doing this, I made several changes:
This is not done. In particular, several things need to be put back into place: