Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): Add node-gyp version control across different distros MONGOSH-1891 #2230

Merged
merged 17 commits into from
Oct 23, 2024

Conversation

gagik
Copy link
Contributor

@gagik gagik commented Oct 21, 2024

Recent CI update likely led to Python being bumped to 3.12+ which means our current node-gyp library can no longer support it without some more manual setup. We need to bump node-gyp and also handle some older dependencies (such as RHEL which should work with 3.6).

@gagik gagik marked this pull request as draft October 21, 2024 15:33
package.json Outdated Show resolved Hide resolved
@gagik gagik marked this pull request as ready for review October 22, 2024 17:55
@gagik gagik changed the title WIP: Add node-gyp version control across different distros chore(deps): Add node-gyp version control across different distros MONGOSH-1891 Oct 22, 2024
@gagik gagik changed the title chore(deps): Add node-gyp version control across different distros MONGOSH-1891 WIP chore(deps): Add node-gyp version control across different distros MONGOSH-1891 Oct 22, 2024
@gagik gagik requested a review from addaleax October 23, 2024 08:38
@gagik
Copy link
Contributor Author

gagik commented Oct 23, 2024

I lost and re-found the issue. e2e_tests_linux_x64_60x Setup Failed still happens so probably needs manual handling. Hopefully this last thing works 🤞

@gagik gagik changed the title WIP chore(deps): Add node-gyp version control across different distros MONGOSH-1891 chore(deps): Add node-gyp version control across different distros MONGOSH-1891 Oct 23, 2024
@gagik
Copy link
Contributor Author

gagik commented Oct 23, 2024

Rest of the failing tests seem historically flaky and don't seem to be related to node-gyp so going ahead.

@gagik gagik merged commit 6f46d3a into main Oct 23, 2024
62 of 66 checks passed
@gagik gagik deleted the gagik/chore-bump-gyp branch October 23, 2024 11:51
nirinchev added a commit that referenced this pull request Oct 25, 2024
* main:
  chore: update auto-generated files (#2237)
  feat(shell-api): add shardedDataDistribution to sh.status() MONGOSH-1326 (#2214)
  chore: update auto-generated files (#2235)
  feat(tests): Add individual evergreen test results with XUnit (#2227)
  chore: update auto-generated files (#2234)
  chore: rename service-provider-server and `CliServiceProvider` (#2232)
  chore: update auto-generated files (#2233)
  chore(deps): Add node-gyp version control across different distros MONGOSH-1891 (#2230)
  chore: update auto-generated files (#2231)
  fix(shell-api): Align database and collection aggregate functions MONGOSH-1868 (#2229)
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