conda-forge / status

https://conda-forge.org/status/
8 stars 13 forks source link

Migrations error in bot-bot job #145

Closed mfansler closed 1 year ago

mfansler commented 1 year ago

Monitoring R 4.3 migration, I noticed we stopped getting PRs the last couple days, despite fixing up stuff I identified as blocking. Having a look at recent bot-bot jobs showed they run ~5 mins, as opposed to the typical ~1 hr runtimes from a few days back. The run migrations section of the bot-bot logs shows the following error:

Run export START_TIME=$(date +%s)
  export START_TIME=$(date +%s)
  export TIMEOUT=7200
  export CIRCLE_BUILD_URL="[https://github.com/regro/cf-scripts/actions/runs/${RUN_ID}](https://github.com/regro/cf-scripts/actions/runs/$%7BRUN_ID%7D)"
  export CIRCLE_BUILD_NUM="actually-actions-${RUN_ID}"

  pushd cf-graph

  conda-forge-tick auto-tick

  popd
  shell: /usr/bin/bash -l {0}
  env:
    MAMBA_ROOT_PREFIX: /home/runner/micromamba
    MAMBA_EXE: /home/runner/micromamba-bin/micromamba
    CONDARC: /home/runner/micromamba-bin/.condarc
    CI_SKIP: 
    USERNAME: regro-cf-autotick-bot
    PASSWORD: ***
    RUN_ID: 5354311023
    MEMORY_LIMIT_GB: 7
    CF_TICK_GRAPH_DATA_BACKENDS: mongodb:file
    MONGODB_CONNECTION_STRING: ***
~/work/cf-scripts/cf-scripts/cf-graph ~/work/cf-scripts/cf-scripts
limit read as 7.0 GB
Setting memory limit to 6.0 GB
collapsing closed PR json
processing bot-rerun labels
Traceback (most recent call last):
  File "/home/runner/micromamba/envs/cf-scripts/bin/conda-forge-tick", line 8, in <module>
    sys.exit(main())
  File "/home/runner/work/cf-scripts/cf-scripts/cf-scripts/conda_forge_tick/cli.py", line 77, in main
    func(args)
  File "/home/runner/work/cf-scripts/cf-scripts/cf-scripts/conda_forge_tick/auto_tick.py", line 1489, in main
    _update_graph_with_pr_info()
  File "/home/runner/work/cf-scripts/cf-scripts/cf-scripts/conda_forge_tick/auto_tick.py", line 1471, in _update_graph_with_pr_info
    _update_nodes_with_bot_rerun(gx)
  File "/home/runner/work/cf-scripts/cf-scripts/cf-scripts/conda_forge_tick/auto_tick.py", line 1341, in _update_nodes_with_bot_rerun
    with node["payload"] as payload, payload["pr_info"] as pri, payload[
  File "/home/runner/work/cf-scripts/cf-scripts/cf-scripts/conda_forge_tick/lazy_json_backends.py", line 636, in __getitem__
    return self._data[item]
KeyError: 'pr_info'
~/work/cf-scripts/cf-scripts
beckermr commented 1 year ago

This should be fixed now. Let's monitor it for several hours and we can close this if things seem ok.

jakirkham commented 1 year ago

Thanks Matt! 🙏

Yeah have seen some migrator PRs opened. So think things are getting better 🙂

jakirkham commented 1 year ago

On a different note, it seems the status page isn't reflecting this issue

beckermr commented 1 year ago

Looks like it's working.

beckermr commented 1 year ago

Let's open a separate issue for the status page.

beckermr commented 1 year ago

Fixed with a python upgrade.

jakirkham commented 1 year ago

FWIW here's a recent R 4.3 migrator PR ( https://github.com/conda-forge/r-tigris-feedstock/pull/12 )

mfansler commented 1 year ago

Thank you!