garden-io / garden

Automation for Kubernetes development and testing. Spin up production-like environments for development, testing, and CI on demand. Use the same configuration and workflows at every step of the process. Speed up your builds and test runs via shared result caching
https://garden.io
Mozilla Public License 2.0
3.29k stars 267 forks source link

0.13: [Bug]: npm ci missing from garden plugins pulumi preview command #6256

Closed ishantagarwal-x213235 closed 2 days ago

ishantagarwal-x213235 commented 6 days ago

garden plugins pulumi preview command is not installing the npm packages while it was doing it in previous 0.12 garden version.

Captured below logs with garden 0.12 version

ℹ thx-gcs-iam-package → Getting build status for v-c0e68e517e... ℹ thx-gcs-iam-package → Building version v-c0e68e517e... ℹ thx-gcs-iam-package → [npm ci]: npm warn deprecated @opentelemetry/api-metrics@0.32.0: Please use @opentelemetry/api >= 1.3.0 ℹ thx-gcs-iam-package → [npm ci]: ℹ thx-gcs-iam-package → [npm ci]: added 321 packages, and audited 322 packages in 7s ℹ thx-gcs-iam-package → [npm ci]: ℹ thx-gcs-iam-package → [npm ci]: 43 packages are looking for funding ℹ thx-gcs-iam-package → [npm ci]: run npm fund for details ℹ thx-gcs-iam-package → [npm ci]: ℹ thx-gcs-iam-package → [npm ci]: 2 moderate severity vulnerabilities ℹ thx-gcs-iam-package → [npm ci]: ℹ thx-gcs-iam-package → [npm ci]: To address all issues, run: ℹ thx-gcs-iam-package → [npm ci]: npm audit fix ℹ thx-gcs-iam-package → [npm ci]: ℹ thx-gcs-iam-package → [npm ci]: Run npm audit for details. ✔ thx-gcs-iam-package → [npm ci]: Run npm audit for details. → Done (took 7.2 sec) ℹ thx-gcs-iam-infra → Running pulumi preview

Captured below logs with garden 0.13 version

ℹ garden → Initializing... Project is configured with apiVersion: garden.io/v0, running with backwards compatibility. ⚠ garden-cloud → You are not logged in. To use Garden Cloud, log in with the garden login command. ℹ garden → Running in environment dev.default Running pulumi preview for actions thx-gcs-iam-infra

ℹ providers → Resolving providers... ℹ pulumi → Configuring provider... ✔ pulumi → Provider configured ✔ pulumi → Provider ready ✔ providers → Finished resolving providers (took 0.1 sec) ℹ providers → Resolving providers... ℹ exec → Configuring provider... ✔ exec → Provider configured ✔ exec → Provider ready ✔ providers → Finished resolving providers (took 0 sec) ℹ graph → Resolving actions and modules... ✔ graph → Finished resolving graph (took 0.4 sec) ℹ garden → Finished initializing Garden (took 1.8 sec)

ℹ deploy.thx-gcs-iam-infra → Running pulumi preview ✖ deploy.thx-gcs-iam-infra → Failed (took 4.1 sec) ✖ deploy.thx-gcs-iam-infra → Failed processing Deploy type=pulumi name=thx-gcs-iam-infra (from module thx-gcs-iam-infra) (took 4.11 sec). This is what happened:

npm-ci is completely missing from 0.13 garden version logs.

Expected behaviour garden plugins pulumi preview command should install the npm packages and then trigger the pulumi preview

Actual behavirour garden plugins pulumi preview command only trigger the pulumi preview command which got failed because of absence of npm modules.

thsig commented 6 days ago

Hi @ishantagarwal-x213235!

Could you run the command at the debug log level (garden plugins pulumi preview -l4) at both Garden versions (0.12 and 0.13) and share the full log output?

You could also share these via a support email to us in case there's anything sensitive there.

In the meantime, I'll be taking a closer look at some of our test projects to see if I can detect any problems with the Pulumi plugin in 0.13.

ishantagarwal-x213235 commented 6 days ago

Thanks @thsig for response. I sent an email with detail logs for both version. Pl take a look and let me know in case of any more info required.

ishantagarwal-x213235 commented 2 days ago

Kindly update the progress @thsig

thsig commented 2 days ago

@ishantagarwal-x213235 I think the PR for this should be good to go now (once the tests are passing in CI). After it's been merged, it will be included in the next release.

ishantagarwal-x213235 commented 2 days ago

Thanks @thsig

vvagaytsev commented 1 day ago

@ishantagarwal-x213235 we released the fix in 0.13.34.