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

ci: config logging for integration tests #4751

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

jmayclin
Copy link
Contributor

@jmayclin jmayclin commented Sep 4, 2024

Description of changes:

The integ v2 jobs rely on the following environmental variables

        variables:
          S2N_BUILD_PRESET:
            - awslc_gcc9
            - awslc-fips_gcc4-8
            - awslc-fips-2022_gcc6
            - openssl-1.0.2
            - openssl-1.0.2-fips
            - openssl-1.1.1_gcc9
            - openssl-3.0
          INTEGV2_TEST:
            - "test_happy_path"
            - "test_cross_compatibility test_client_authentication"
            - "test_early_data test_sslyze test_well_known_endpoints test_hello_retry_requests test_sni_match test_pq_handshake test_fragmentation test_key_update"
            - "test_session_resumption test_renegotiate_apache test_buffered_send"
            - "test_dynamic_record_sizes test_npn test_signature_algorithms"
            - "test_version_negotiation test_external_psk test_ocsp test_renegotiate test_serialization"

This test will log them to ideally make it a bit easier to debug, and hopefully identify trends over time.

Testing:

How is this change tested (unit tests, fuzz tests, etc.)? Are there any testing steps to be verified by the reviewer?

Is this a refactor change? If so, how have you proved that the intended behavior hasn't changed?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@github-actions github-actions bot added the s2n-core team label Sep 4, 2024
@jmayclin jmayclin marked this pull request as ready for review September 18, 2024 18:24
@jmayclin jmayclin requested a review from dougch as a code owner September 18, 2024 18:24
@jmayclin jmayclin requested review from jouho and goatgoose September 18, 2024 18:24
@jmayclin jmayclin enabled auto-merge December 12, 2024 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants