Skip to content

I257 pvt network#282

Open
kayiwa wants to merge 5 commits intomainfrom
i257_pvt_network
Open

I257 pvt network#282
kayiwa wants to merge 5 commits intomainfrom
i257_pvt_network

Conversation

@kayiwa
Copy link
Member

@kayiwa kayiwa commented Jan 20, 2026

related to migration to private network. We keep DNS

https://gitlab.lib.princeton.edu/ops/team-handbook/-/issues/257

@coveralls
Copy link

coveralls commented Jan 21, 2026

Coverage Status

coverage: 100.0%. remained the same
when pulling ef7db68 on i257_pvt_network
into e1b6b9c on main.

@jrgriffiniii
Copy link
Contributor

Thank you very much for this, and I apologize for the delay in reviewing this. I am running the DevBox commands on macOS, and noticing the following warnings:
Info: Ensuring packages are installed. Info: Installing the following packages to the nix store: gcc, libxml2, nodejs@22.13.1, libffi, zlib, readline, libpq, gnumake, openssl, libxslt, ruby@3.2.3, postgresql@16, pkg-config, libyaml, yarn@1.22.10 ✓ Computed the Devbox environment. Starting a devbox shell... warning: unhandled Platform key FamilyDisplayName error: tool 'git' not found

This obviously should not break the build or block merging this PR, and is very likely related to a local macOS issue, but I wanted to please note this.

@jrgriffiniii
Copy link
Contributor

devbox run setup seemed to hang for several minutes on

Fetching mini_mime 1.1.5
Installing mini_mime 1.1.5
Fetching date 3.4.1                                                                                                                                                                                                 Installing date 3.4.1 with native extensions

...and then proceeded to fail with:

Please ensure that your Gemfiles and .gemspecs are suitably restrictive
to avoid an unexpected breakage when 3.0 is released (e.g. ~> 2.3.0).
See https://github.com/rubyzip/rubyzip for details. The Changelog also
lists other enhancements and bugfixes that have been implemented since
version 2.3.0.
Post-install message from vite_ruby:
Thanks for installing Vite Ruby!

If you upgraded the gem manually, please run:
        bundle exec vite upgrade
yarn install v1.22.10
warning package.json: "dependencies" has dependency "esbuild" with range "^0.25.0" that collides with a dependency in "devDependencies" of the same name with version "^0.25.1"
warning ../../../../package.json: No license field
warning "dependencies" has dependency "esbuild" with range "^0.25.0" that collides with a dependency in "devDependencies" of the same name with version "^0.25.1"
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
warning Pattern ["strip-ansi@^6.0.1"] is trying to unpack in the same destination "/Users/griffinj/Library/Caches/Yarn/v6/npm-strip-ansi-cjs-6.0.1-9e26c63d30f53443e9489495b2105d37b67a85d9-integrity/node_modules/strip-ansi-cjs" as pattern ["strip-ansi-cjs@npm:strip-ansi@^6.0.1"]. This could result in non-deterministic behavior, skipping.
warning Pattern ["string-width@^4.1.0"] is trying to unpack in the same destination "/Users/griffinj/Library/Caches/Yarn/v6/npm-string-width-cjs-4.2.3-269c7117d27b05ad2e536830a8ec895ef9c6d010-integrity/node_modules/string-width-cjs" as pattern ["string-width-cjs@npm:string-width@^4.2.0"]. This could result in non-deterministic behavior, skipping.
warning Pattern ["strip-ansi@^6.0.0"] is trying to unpack in the same destination "/Users/griffinj/Library/Caches/Yarn/v6/npm-strip-ansi-cjs-6.0.1-9e26c63d30f53443e9489495b2105d37b67a85d9-integrity/node_modules/strip-ansi-cjs" as pattern ["strip-ansi-cjs@npm:strip-ansi@^6.0.1"]. This could result in non-deterministic behavior, skipping.
error cookie@1.0.1: The engine "node" is incompatible with this module. Expected version ">=18". Got "14.17.4"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
yarn install v1.22.10

Again, I suspect that this is a macOS environment issue, and I am now going to test this on a local Ubuntu server. Thank you for your patience.

@jrgriffiniii
Copy link
Contributor

I am sorry, I needed to introduce the following to fix yarn install within devbox when running devbox run setup: 0ad20da

Copy link
Contributor

@jrgriffiniii jrgriffiniii left a comment

Choose a reason for hiding this comment

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

I apologize, please see #282 (comment)

@jrgriffiniii
Copy link
Contributor

I am sorry, please know that I also needed to introduce e1ee90a to ensure that postgresql was running within devbox

@jrgriffiniii
Copy link
Contributor

Please also note that I needed to introduce c4b6d1a to ensure that Vite builds properly within certain macOS environments.

@kayiwa
Copy link
Member Author

kayiwa commented Feb 3, 2026

heya @jrgriffiniii thanks for the prod to test this on non macOS. I have added that to my suite. Please let me know if my changes work for you.

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