search

regro / cf-scripts

Flagship repo for cf-regro-autotick-bot
Other
53 stars 74 forks source link

Missing compiler rebuild PR for metis #386

Closed jakirkham closed 6 years ago

jakirkham commented 6 years ago

AFAICT metis has no blocking dependencies, yet there is no compiler rebuild PR. Any ideas why?

xref: https://github.com/conda-forge/metis-feedstock/issues/17

CJ-Wright commented 6 years ago

Most likely it is waiting on cmake.

jakirkham commented 6 years ago

cmake is done. ( https://github.com/conda-forge/cmake-feedstock/pull/72 ) Other ideas?

CJ-Wright commented 6 years ago

There is a traceback

Traceback (most recent call last):
  File "/root/repo/cf-scripts/conda_forge_tick/auto_tick.xsh", line 313, in main
    hash_type=attrs.get('hash_type', 'sha256'))
  File "/root/repo/cf-scripts/conda_forge_tick/auto_tick.xsh", line 72, in run
    gh=gh)
  File "/root/repo/cf-scripts/conda_forge_tick/git_utils.xsh", line 133, in get_repo
    git pull @(upstream) master
  File "/opt/conda/lib/python3.6/site-packages/xonsh/__amalgam__.py", line 21217, in subproc_captured_hiddenobject
    return run_subproc(cmds, captured="hiddenobject")
  File "/opt/conda/lib/python3.6/site-packages/xonsh/__amalgam__.py", line 21180, in run_subproc
    command.end()
  File "/opt/conda/lib/python3.6/site-packages/xonsh/__amalgam__.py", line 13815, in end
    self._end(tee_output=tee_output)
  File "/opt/conda/lib/python3.6/site-packages/xonsh/__amalgam__.py", line 13834, in _end
    self._raise_subproc_error()
  File "/opt/conda/lib/python3.6/site-packages/xonsh/__amalgam__.py", line 13961, in _raise_subproc_error
    raise subprocess.CalledProcessError(rtn, spec.cmd, output=self.output)
subprocess.CalledProcessError: Command '['/usr/bin/git', 'pull', 'https://github.com/conda-forge/metis-feedstock.git', 'master']' returned non-zero exit status 1.

So there seems like there was a problem with git pull?

CJ-Wright commented 6 years ago

Maybe we should update the bot's master first?

jakirkham commented 6 years ago

Sounds reasonable.

CJ-Wright commented 6 years ago

@regro/auto-tick thoughts on how to do this nicely?

jakirkham commented 6 years ago

Could you help me understand this a bit? What informs the bot's master?

CJ-Wright commented 6 years ago

I don't know exactly myself. Here is the code https://github.com/regro/cf-scripts/blob/master/conda_forge_tick/git_utils.xsh#L127

jakirkham commented 6 years ago

What's the error message now (assuming there is one)?

( Tried to find the right log to look in, but wasn't able to. :( )

CJ-Wright commented 6 years ago

Its the same. I think we will need to reset hard our master on upstream. @scopatz thoughts?

SylvainCorlay commented 6 years ago

Now metis appears in awaiting-parents section, while it was not showing at all before.

jakirkham commented 6 years ago

Guess that is an improvement, but not sure why it thinks it is missing dependencies. It only needs cmake to build, which is already available.

CJ-Wright commented 6 years ago

fixed. xref: https://github.com/conda-forge/metis-feedstock/pull/19

jakirkham commented 6 years ago

Would be curious to know what the underlying problem ended up being.

CJ-Wright commented 6 years ago

We now hard reset onto upstream master for the bot, there were some lingering files which were stored in the bot's master which were causing pulls from upstream master to fail (even when we told git to use theirs as the merge strategy)