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

aws-cli-2/2.23.4 package update #40281

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 Jan 22, 2025

Signed-off-by: wolfi-bot <121097084+wolfi-bot@users.noreply.github.com>
@octo-sts octo-sts bot added request-version-update request for a newer version of a package automated pr labels Jan 22, 2025
Copy link
Contributor Author

octo-sts bot commented Jan 22, 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: "Failed to open usr/bin/aws: open usr/bin/aws: no such file or directory"

• Error Category: Configuration

• Failure Point: Package scanning step during melange build

• Root Cause Analysis: The symlinks created for aws and aws_completer executables are not being properly included in the final package. This happens because the files are not in the expected location during the package scanning phase.

• Suggested Fix: Modify the symlink creation step in the pipeline to use the correct target directory:

  - runs: |
      rm ${{targets.destdir}}/usr/bin/aws
      rm ${{targets.destdir}}/usr/bin/aws_completer

      ln -s /usr/lib/aws-cli/aws ${{targets.destdir}}/usr/bin/aws
      ln -s /usr/lib/aws-cli/aws_completer ${{targets.destdir}}/usr/bin/aws_completer

• Explanation: The current symlink creation is using ${{targets.contextdir}} which points to a temporary build directory. We should use ${{targets.destdir}} instead, which points to the final package installation directory where melange expects to find the files during package scanning.

• Additional Notes:

  • The error occurs during the package scanning phase when melange tries to verify dependencies and shebang requirements
  • The executables themselves are installed correctly in /usr/lib/aws-cli/, but the symlinks in /usr/bin/ are missing
  • Using targets.destdir ensures files are placed in the correct location for package generation

• References:

@octo-sts octo-sts bot added the ai/skip-comment Stop AI from commenting on PR label Jan 22, 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