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

pulumi/3.153.1 package update #44209

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

octo-sts[bot]
Copy link
Contributor

@octo-sts octo-sts bot commented Mar 1, 2025

@octo-sts octo-sts bot added request-version-update request for a newer version of a package automated pr labels Mar 1, 2025
Copy link
Contributor Author

octo-sts bot commented Mar 1, 2025

⚠️ EXPERIMENTAL

Please use 👍 or 👎 on this comment to indicate if you agree or disagree with the recommendation.

To provide more detailed feedback please comment on the recommendation prefixed with /ai-verify:

e.g. /ai-verify partially helpful but I also added bash to the build environment

Gen AI suggestions to solve the build error:

• Detected Error:

sort: unrecognized option '--key=3'
./generate.sh: line 21: docker: command not found
make: *** [Makefile:50: build_proto] Error 1

• Error Category: Build Configuration

• Failure Point:

  • Initial failure on sort command with unsupported option
  • Secondary failure due to missing Docker dependency for protobuf generation

• Root Cause Analysis:

  1. BusyBox sort doesn't support GNU sort's '--key' long option format
  2. The build process requires Docker for protobuf generation
  3. The build system is using BusyBox tools rather than GNU coreutils

• Suggested Fix:

  1. Add coreutils to the environment packages:
environment:
  contents:
    packages:
      - build-base
      - busybox
      - ca-certificates-bundle
      - coreutils    # Add this
      - docker       # Add this
      - go
      - nodejs
      - patch
      - python-3.11
      - python-3.11-dev
      - uv
      - yarn
  1. Alternative approach if Docker cannot be used:
    Add a pre-build step to generate protobuf files locally:
pipeline:
  - runs: |
      # Generate protobuf files without Docker
      cd pkg/resource/provider/proto
      protoc --go_out=. --go_opt=paths=source_relative \
             --go-grpc_out=. --go-grpc_opt=paths=source_relative \
             provider.proto

• Explanation:

  • coreutils provides GNU sort with full feature support
  • The build requires proper sort functionality for dependency management
  • Either Docker access or local protobuf generation is needed for the build

• Additional Notes:

  • If Docker cannot be added to the build environment, you'll need to ensure all protobuf files are pre-generated
  • Consider adding protoc and protoc-gen-go to the build dependencies if taking the local generation approach

• References:

@octo-sts octo-sts bot added the ai/skip-comment Stop AI from commenting on PR label Mar 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ai/skip-comment Stop AI from commenting on PR automated pr request-version-update request for a newer version of a package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant