Skip to content

Conversation

altendky
Copy link
Contributor

@altendky altendky commented Jul 11, 2025

Purpose:

undoing a rogue change to the lock file in dac837e

Current Behavior:

New Behavior:

Testing Notes:

@altendky altendky requested a review from a team as a code owner July 11, 2025 15:00
@altendky altendky added Fixed Required label for PR that categorizes merge commit message as "Fixed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes labels Jul 11, 2025
@Starttoaster
Copy link
Contributor

The failed test here seems unrelated, but also doesn't appear to be a flake. Has someone investigated that miniupnpc test issue already? I'm guessing missing C library, but not sure why it would be specific to that job offhand.

@altendky
Copy link
Contributor Author

https://github.com/Chia-Network/chia-blockchain/actions/runs/16223153715/job/45935307869?pr=19806#step:16:405

FAILED .venv/lib/python3.9/site-packages/chia/_tests/core/server/test_upnp.py::test_miniupnpc_imports_successfully - ImportError: dlopen(/Users/runner/work/chia-blockchain/chia-blockchain/.venv/lib/python3.9/site-packages/miniupnpc.cpython-39-darwin.so, 0x0002): symbol not found in flat namespace '_FreeUPNPUrls'

nightlies are finding this too, but are too flaky for people to spend time checking them since they fail every day.

@altendky
Copy link
Contributor Author

altendky commented Jul 15, 2025

the error is being addressed in #19814

@altendky altendky closed this Jul 15, 2025
@altendky altendky reopened this Jul 15, 2025
@Starttoaster Starttoaster merged commit 590317c into release/2.5.5 Jul 15, 2025
6096 of 6112 checks passed
@Starttoaster Starttoaster deleted the recover_chiabip_tgz branch July 15, 2025 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes Fixed Required label for PR that categorizes merge commit message as "Fixed" for changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants