Skip to content

Add glibc to supported libc architectures in .yarnrc.yml#2378

Merged
grolu merged 2 commits intogardener:masterfrom
Lappihuan:fix-supported-architecture
Mar 31, 2025
Merged

Add glibc to supported libc architectures in .yarnrc.yml#2378
grolu merged 2 commits intogardener:masterfrom
Lappihuan:fix-supported-architecture

Conversation

@Lappihuan
Copy link
Copy Markdown
Contributor

What this PR does / why we need it:
On yarn install i kept getting:

➤ YN0077: │ @parcel/watcher-linux-x64-glibc@npm:2.5.1: Your current architecture (linux-x64) is supported by this package, but is missing from the supportedArchitectures setting
➤ YN0077: │ @rollup/rollup-linux-x64-gnu@npm:4.34.6: Your current architecture (linux-x64) is supported by this package, but is missing from the supportedArchitectures setting

which resulted in this error when i tried to yarn serve in the frontend:

yarn serve 
/home/ode/vscode-workspace/dashboard/.yarn/cache/rollup-npm-4.34.6-b6bc3b6a7d-0d55e43754.zip/node_modules/rollup/dist/native.js:64
                throw new Error(
                      ^

Error: Cannot find module @rollup/rollup-linux-x64-gnu. npm has a bug related to optional dependencies (https://github.com/npm/cli/issues/4828). Please try `npm i` again after removing both package-lock.json and node_modules directory
...

i worked arround it with a local install of the package until now, but got annoying.

adding glibc to the .yarnrc.yml and yarn cache clean then yarn fixes it.

Which issue(s) this PR fixes:
N/A

Special notes for your reviewer:
N/A

Release note:

NONE

@gardener-robot
Copy link
Copy Markdown

@Lappihuan Thank you for your contribution.

@gardener-robot gardener-robot added needs/review Needs review size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 28, 2025
@gardener-robot-ci-2
Copy link
Copy Markdown
Contributor

Thank you @Lappihuan for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below.

@gardener-robot
Copy link
Copy Markdown

@grolu, @holgerkoser You have pull request review open invite, please check

Copy link
Copy Markdown
Member

@petersutter petersutter left a comment

Choose a reason for hiding this comment

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

/lgtm

@gardener-robot gardener-robot added reviewed/lgtm Has approval for merging and removed needs/review Needs review labels Mar 31, 2025
@ghost ghost added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Mar 31, 2025
@gardener-robot-ci-1 gardener-robot-ci-1 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Mar 31, 2025
Copy link
Copy Markdown
Member

@grolu grolu left a comment

Choose a reason for hiding this comment

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

/lgtm

@grolu grolu merged commit 9be1349 into gardener:master Mar 31, 2025
7 checks passed
@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Mar 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) reviewed/lgtm Has approval for merging size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. status/closed Issue is closed (either delivered or triaged)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants