Skip to content

Conversation

@parkan
Copy link
Collaborator

@parkan parkan commented Oct 27, 2025

omnibus PR to update our many, many badly outdated dependencies

major changes:

  • upgrade rclone to v1.68.0, this pulls in V2 AWS SDK instead of the long-deprecated V1
  • upgrade gorm and use lighter touch gorm features to implement Fix/cascade #583 instead of manually running sql
  • vendor my fork of lassie (see Update dependencies and test code filecoin-project/lassie#512 (comment)) without bitswap, this allows us to use most recent boxo -- I am happy with this functionality reduction as lassie is only used for the retrieval API and bitswap use for SPs is quite uncommon, but can revert this if desired
  • upgrade boxo, libp2p, etc to much fresher versions

if [ ! -f "/home/vscode/.local/share/pg/pgdata/PG_VERSION" ]; then
echo "Initializing Postgres..."
/usr/lib/postgresql/16/bin/initdb -D /home/vscode/.local/share/pg/pgdata --auth trust --auth-local trust --auth-host trust
/usr/lib/postgresql/16/bin/initdb -D /home/vscode/.local/share/pg/pgdata --auth trust --auth-local trust --auth-host trust --encoding=UTF8 --locale=C.UTF-8
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this got clobbered again by some hook magic I didn't fully remove, sneaking in here b/c it was breaking my tests for validating the upgrades

Arkadiy Kukarkin added 2 commits October 27, 2025 13:01
- need rclone >= v1.68.0 to upgrade away from deprecated v1 aws sdk
- need go-libp2p >= v0.44.0 for go-quic version compatibility
- rclone dropped fs.NewConfig(), need to hardcode more defaults :(
- azureblob retired years ago
@parkan parkan marked this pull request as ready for review October 28, 2025 12:28
@parkan
Copy link
Collaborator Author

parkan commented Oct 29, 2025

ok we now have a nicely formatted test output:

image

Copy link
Collaborator

@lanzafame lanzafame left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

excellent clean up

@parkan
Copy link
Collaborator Author

parkan commented Oct 30, 2025

hmmm we are still skipping some tests that I would expect to run, I think there's an issue with the dialector for sqlite bailing and skipping the other tests; I'll look into that to validate the FK/cascade fix properly but otherwise we should be good

@parkan
Copy link
Collaborator Author

parkan commented Oct 31, 2025

ok I think the skips are reasonable dialector mismatch (postgres specific test should not run with mysql etc) so we should be good

@parkan parkan merged commit 48630ba into main Oct 31, 2025
4 checks passed
@parkan parkan deleted the chore/update branch October 31, 2025 11:57
@parkan
Copy link
Collaborator Author

parkan commented Oct 31, 2025

interesting, the glif-dependent tests fail on main while passing on the PR, looking

@parkan
Copy link
Collaborator Author

parkan commented Nov 3, 2025

breakage fixed in #591

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants