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

CI: Disabling dev docs push from azure-pipelines while we fix it #26602

Closed
wants to merge 1 commit into from
Closed

CI: Disabling dev docs push from azure-pipelines while we fix it #26602

wants to merge 1 commit into from

Conversation

datapythonista
Copy link
Member

master builds are failing, this temporary fixes them while we figure out why the ssh key is not working.

See: #26591

CC: @jreback

@datapythonista datapythonista added the CI Continuous Integration label Jun 1, 2019
@jreback
Copy link
Contributor

jreback commented Jun 1, 2019

@jreback jreback added this to the 0.25.0 milestone Jun 1, 2019
@datapythonista
Copy link
Member Author

@vtbassmatt @davidstaheli @infin8x we're getting random errors like the one that happened here since we started using the task InstallSSHKey@0. The file mentioned in the error does exist, and there is nothing wrong in the configuration here. The only change is that in one of the steps we stop making a git push.

I saw the same error before, because the InstallSSHKey@0 task couldn't find the key, even if it was not running because the specified condition was not met.

Can you please assist?

@codecov
Copy link

codecov bot commented Jun 1, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@2c6d005). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #26602   +/-   ##
=========================================
  Coverage          ?   91.84%           
=========================================
  Files             ?      174           
  Lines             ?    50668           
  Branches          ?        0           
=========================================
  Hits              ?    46535           
  Misses            ?     4133           
  Partials          ?        0
Flag Coverage Δ
#multiple 90.38% <ø> (?)
#single 41.7% <ø> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2c6d005...7abae60. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Jun 1, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@2c6d005). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #26602   +/-   ##
=========================================
  Coverage          ?   91.84%           
=========================================
  Files             ?      174           
  Lines             ?    50668           
  Branches          ?        0           
=========================================
  Hits              ?    46535           
  Misses            ?     4133           
  Partials          ?        0
Flag Coverage Δ
#multiple 90.38% <ø> (?)
#single 41.7% <ø> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2c6d005...7abae60. Read the comment docs.

@datapythonista
Copy link
Member Author

We're also receiving another weird error here: https://dev.azure.com/pandas-dev/pandas/_build/results?buildId=12100

This build has the same exact configuration than this one that failed with the Host key verification failed (which I think it's a correct error): https://dev.azure.com/pandas-dev/pandas/_build/results?buildId=12093

And one more weird error here with a correct configuration: https://dev.azure.com/pandas-dev/pandas/_build/results?buildId=12106&view=results

It's quite worrying to see azure pipelines giving random errors like this, that don't inform which is the actual problem, and that happen in unrelated changes that are correct. Let's see if those can be fixed by azure, otherwise I'll leave the doc build in travis, and hope that pipeline becomes stable again.

CC: @pandas-dev/pandas-core

@davidstaheli
Copy link

Hi @datapythonista . For the first error about the file "not found," we're working with GitHub on the problem. It's caused by GitHub sending Azure Pipelines a "pull_request" webhook event, but when Azure Pipelines tries to fetch the files to build, the commit hasn't replicated to all of GitHub's databases yet, so GitHub returns 404 - Not Found. In addition to GitHub working on this, I'm hopeful Azure Pipelines will have better wait+retry logic within a week as a workaround.

On the 2nd weird error, I don't know the full history, but it started when a dependency was updated. It was fixed on May 29, 2019 here. I'll find out how soon we can get that fix deployed to production and push for ASAP.

@datapythonista
Copy link
Member Author

Thanks for the update @davidstaheli

In case it helps, I'm not aware of seeing those errors since we added the InstallSSHKey@0 task to our jobs. Which didn't work anyway. If you can ping us when all this is fixed, we can give another try on building the docs and pushing them to a gihub repo from azure, which we had to revert.

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

Successfully merging this pull request may close these issues.

None yet

3 participants