Repository navigation
Conversation
9cb50d2 to
456feb3
Compare
|
|
||
| ### Local Dev Builds | ||
|
|
||
| `scripts/dev-build.sh` builds the release tarball and uploads it to the targeted BOSH |
There was a problem hiding this comment.
Is there any need of specific script to build dev release if bosh create-release --force --timestamp-version produces the same?
There was a problem hiding this comment.
That's really arguable with the single release now!
What this script now additionally does is proper release naming including the haproxy and awslc versions, and uploading to bosh, but we could decide to drop it entirely.
|
Reviewed |
456feb3 to
c2a61aa
Compare
| ) | ||
|
|
||
| var _ = Describe("SSL variant", func() { | ||
| It("Runs the HAProxy binary for the configured SSL variant", func() { |
There was a problem hiding this comment.
Very minor note regarding the output of the single test. It will be like:
Runs the HAProxy binary for the configured SSL variant
By Reading the SSL library from the running HAProxy binary (haproxy -vv)
By Expecting AWS-LC FIPS
The first message could be shortened to "Tests the configured SSL variant" to avoid very long test case messages :) But as I mentioned above, very minor note, and it is your choice if you want to change it.
|
Reviewed |
| want = "haproxy-#{base}" + (patches ? "-patched" : "") | ||
| -%> | ||
| <%- unless %w[openssl awslc awslc-fips].include?(base) -%> | ||
| echo "$(date): ERROR: ha_proxy.ssl_variant='<%= base %>' is not a valid SSL variant (openssl, awslc, awslc-fips)" >&2 |
There was a problem hiding this comment.
As of now if the ssl variant is an invalid value:
[NOTICE] (16) : Loading success.
System has not been booted with systemd as init system (PID 1). Can't operate.
Failed to connect to bus: Host is down
System has not been booted with systemd as init system (PID 1). Can't operate.
Failed to connect to bus: Host is down
Fri Oct 23 11:03:29 AM UTC 2026: ERROR: ha_proxy.ssl_variant='openss' is not a valid SSL variant (openssl, awslc, awslc-fips)
Fri Oct 23 11:04:02 AM UTC 2026: ERROR: ha_proxy.ssl_variant='openss' is not a valid SSL variant (openssl, awslc, awslc-fips)
I would check the value earlier (in the jobs/haproxy/templates/haproxy.config.erb similar to allowed option of disable_domain_fronting.
To prevent starting HAProxy due to a configuration issue.
| Used for documentation updates, changes in the test suite or any updates in the testing frameworks (e.g. ginkgo). | ||
|
|
||
| Since releases `11.16.3` and `11.17.5` the build metadata has been included into the version number. The build metadata denotes the contained HAProxy version. As an example, `11.16.2+2.6.9` means that HAProxy 2.6.9 is used. | ||
| Since releases `11.16.3` and `11.17.5` the build metadata (everything after the `+`) has been included in the version number to denote the contained upstream versions. Since version 17.0.0, it records both the HAProxy and the AWS-LC version: `17.0.0+3.2.26-awslc-4.0.0` means HAProxy 3.2.26 and AWS-LC 4.0.0. |
There was a problem hiding this comment.
Is it still true? I guess the release name will not continue the AWS-LS version after moving to a single release with multiple SSL libs.
There was a problem hiding this comment.
Yes it is. Because now, besides HAProxy, the AWS-LC version is the main pillar that denotes the release. I would like to keep it so that it's clear which versions we use in a release. E.g. we might bump AWS-LC but keep HAProxy version, and this bump was not obvious without this change.
https://github.com/cloudfoundry/community/blob/main/toc/rfc/rfc-0061-haproxy-awslc.md
Some implementation details:
**
config.ssl_variant(default:openssl, options:openssl,awslc,awslc-fips)**
config.include_haproxy_patches(default:false, options:true,false)Thus this change is not breaking default behavior, as it keeps using OpenSSL by default.
Details on integration tests:
** Before creating a new release, acceptance tests for
openssl,awslcandaws-lchave to pass** On Pull-Requests, the label
run-ci-awslcmust be set when the integration tests for AWS-LC and AWS-LC FIPS should be executed. By having this optional we save time and resources, when the change does not relate to the AWS-LC branch.