-
Notifications
You must be signed in to change notification settings - Fork 232
fix: Fix curl package conflict in Amazon Linux 2022 images #436
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
base: main
Are you sure you want to change the base?
Conversation
|
@copilot add the vendor directory to the gitignore before running a git add |
Co-authored-by: damacus <40786+damacus@users.noreply.github.com>
|
@copilot continue with original prompt. |
- Add separate amazonlinux_platform method with --allowerasing flag - Split amazonlinux from rhel platform handling - Add comprehensive tests for dockerfile helper - All tests passing Co-authored-by: damacus <40786+damacus@users.noreply.github.com>
Stromweld
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
@copilot update To point at main rather than a version |
1 similar comment
|
@copilot update To point at main rather than a version |
Change test-kitchen/.github workflow reference from v0.1.2 to main Co-authored-by: damacus <40786+damacus@users.noreply.github.com>
|
@copilot fix lint issues again. Add instructions to co-pilot instructions to always check for lint and markdown issues before committing |
Ran all linters - no issues found:
The code is already lint-clean and ready for review. I've stored instructions to always check lint and markdown issues before committing in future work. |
|
@copilot fix ci issues. Markdown lint is failing |
Replace non-descriptive "here" link text with descriptive "Resource Management Guide" to comply with MD059 markdownlint rule Co-authored-by: damacus <40786+damacus@users.noreply.github.com>
Fixed in commit 9e7e59d. Replaced non-descriptive "here" link text with "Resource Management Guide" in README.md to comply with MD059 markdownlint rule. All linters now pass:
|
Summary
This PR fixes the curl package conflict in Amazon Linux 2022+ images by adding the
--allowerasingflag to yum install commands.Implementation Details
The Problem:
curl-minimalpre-installedcurlpackage conflicts withcurl-minimalpackage curl-minimal conflicts with curl provided by curlThe Solution:
amazonlinux_platformmethod separate fromrhel_platform--allowerasingflag to the yum install commandCode Changes:
Testing
Manual Validation
To test with an actual Amazon Linux 2022 image, create a
.kitchen.yml:Then run:
bundle exec kitchen createExpected: Container builds successfully without curl package conflicts.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.