-
-
Notifications
You must be signed in to change notification settings - Fork 245
fix: Update HAProxy versions: 2.8.5 → 2.8.13 and add 3.2.7 LTS tests #595
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
Co-authored-by: damacus <[email protected]>
|
@copilot add tests for the latest LTS, 3.2.7 |
Co-authored-by: damacus <[email protected]>
Head branch was pushed to by a user without write access
damacus
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.
@copilot Add the 3.2 tst to the CI matrix
Co-authored-by: damacus <[email protected]>
😰 RSpec failure27 examples, 11 failures
|
1 similar comment
😰 RSpec failure27 examples, 11 failures
|
|
@copilot fix rspec issues |
Co-authored-by: damacus <[email protected]>
Slowest examplesTop 10 slowest examples (1.53 seconds, 54.16% of total time)
|
Slowest examplesTop 10 slowest examples (1.21 seconds, 49.58% of total time)
|
|
@copilot address rspec error |
All RSpec tests are passing locally (16 examples, 0 failures) including the updated install_spec.rb with the 2.8.13 version. Could you provide more details about which specific RSpec error you're seeing? This would help me address it more effectively. |
|
@copilot check CI run for failures |
* Add PCRE2 support for CentOS Stream 10 and enable it in CI Co-authored-by: damacus <[email protected]> * Add validation and improve documentation for PCRE2 support Co-authored-by: damacus <[email protected]> * fix: Platform tests Signed-off-by: Dan Webb <[email protected]> * fix: Reverse PCRE version logic per maintainer feedback Changed PCRE2 to be used for RHEL < 10 and PCRE for RHEL >= 10. Updated tests and documentation accordingly. Co-authored-by: damacus <[email protected]> * fix: consolidate PCRE detection logic and simplify make command construction - Extract pcre_package_name helper to eliminate duplication - Use consistent platform_family? check across all RHEL derivatives - Simplify make command conditional with pcre_make_flag helper - Fix test for use_pcre false to verify make flags - Add test coverage for Amazon Linux and Fedora - Improve documentation clarity for PCRE platform behavior - Remove unnecessary Rubocop disable comment Signed-off-by: Dan Webb <[email protected]> * Update .windsurf/rules/definition-of-done.md Co-authored-by: Copilot <[email protected]> * fix: Fix rspec test for PCRE disabled scenario - Changed stub_command to return false so bash resource runs - Rewrote test to directly check bash_resource.code instead of using .with(code:) matcher - All 14 examples now pass with 0 failures Co-authored-by: ramereth <[email protected]> * Cookstyle fix Signed-off-by: Lance Albertson <[email protected]> * fix: Revert PCRE logic to use PCRE2 for RHEL >= 10 The maintainer's earlier feedback to reverse the logic was incorrect. CentOS Stream 10 requires pcre2-devel as pcre-devel is not available. Reverted to original logic: RHEL >= 10 uses PCRE2, RHEL < 10 uses PCRE. - Updated helpers.rb: RHEL >= 10 uses pcre2-devel - Updated install.rb comment to match - Fixed unit tests for AlmaLinux 9/10 - Updated test recipe comments - Updated documentation Co-authored-by: ramereth <[email protected]> --------- Signed-off-by: Dan Webb <[email protected]> Signed-off-by: Lance Albertson <[email protected]> Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: damacus <[email protected]> Co-authored-by: Dan Webb <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: ramereth <[email protected]> Co-authored-by: Lance Albertson <[email protected]>
… support (#551) * Add RPATH support for custom OpenSSL libraries to fix RHEL linking issue Co-authored-by: damacus <[email protected]> * fix: Correct stub_command in RSpec test to allow bash resource execution Co-authored-by: damacus <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: damacus <[email protected]> Co-authored-by: Dan Webb <[email protected]>
Co-authored-by: damacus <[email protected]>
Found and fixed the CI RSpec failures in commit 73f9f5e. Root cause: The main branch was updated with expanded test coverage for multiple platforms (AlmaLinux 9/10, Amazon Linux, Fedora, PCRE disabled, custom OpenSSL) after this PR was created. The branch only had stub_command updated for Ubuntu context. Fix: Rebased on latest main and updated all 7 Verification: All 29 RSpec examples now pass locally (0 failures). CI should pass on next run. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
HAProxy Version Update - Complete
This PR addresses issue #501: Update HAProxy versions from 2.8.5 and 2.9.3 to latest patch releases.
Completed
resources/install.rb- source_version 2.8.5 → 2.8.13test/cookbooks/test/recipes/source_28.rb- version 2.8.5 → 2.8.13spec/unit/recipes/install_spec.rb- ALL test stub expectations for 2.8.13 across all platformsChanges Made
HAProxy 2.8.5 → 2.8.13:
source_versioninresources/install.rbsource_checksumto:13dc06a65b7705b94c843bda8b845edbb621bf45e8a9dc7db590d40ab920a9cesource_28.rbwith new version and checksumspec/unit/recipes/install_spec.rbfor ALL platform contexts:HAProxy 3.2.7 LTS Tests Added:
test/cookbooks/test/recipes/source_32.rb1f0ae9dfb0b319e2d5cb6e4cdf931a0877ad88e0090c46cf16faf008fbf54278source_3.2suite tokitchen.ymlfor integration testingsource-32to CI workflow matrices in.github/workflows/ci.ymltest/integration/source_3.2/controls/source_spec.rbTesting Status
integrationandintegration-amazonlinuxjob matricesFix Applied
The RSpec failures were caused by the main branch having expanded test coverage for multiple platforms (AlmaLinux, Amazon Linux, Fedora) that were added after the initial branch was created. After rebasing on main, all
stub_commandcalls for version checking were updated from 2.8.5 to 2.8.13 across all test contexts.Remaining Work
HAProxy 2.9.x Update: Optional - requires maintainer decision on target version
test/cookbooks/test/recipes/source_openssl.rbOriginal 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.