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

Test failures on aarch64 after merging PR 8851 #9682

Open
niedbalski opened this issue Dec 3, 2024 · 0 comments
Open

Test failures on aarch64 after merging PR 8851 #9682

niedbalski opened this issue Dec 3, 2024 · 0 comments

Comments

@niedbalski
Copy link
Collaborator

Description

After merging PR [#8851](#8851), the following tests were re-enabled for the aarch64 actuated job:

  • -DFLB_WITHOUT_flb-it-utils=1 -DFLB_WITHOUT_flb-it-pack=1
  • Without any omit option.

However, these tests are now failing on the master branch.

Affected Tests and Errors

  1. Proxy URL Test Failure

    Test test_write_str_buffer_overrun...           [ OK ]
    Error: oxy_url_split...                         [2024/12/03 09:26:23] [error] only HTTP proxy is supported.
    [ OK ]
    

    Error Details:
    The oxy_url_split test is emitting an error:

    [2024/12/03 09:26:23] [error] only HTTP proxy is supported.
    
  2. Quoted String Splitting Test Failures

    Test test_flb_utils_split...                    [ OK ]
    Test test_flb_utils_split_quoted...             [ OK ]
    Error: st_flb_utils_split_quoted_errors...      [2024/12/03 09:26:23] [error] Parsing failed: aa "unbalanced quotes should fail
    Error: [2024/12/03 09:26:23] [error] Parsing failed: aa 'unbalanced quotes should fail
    

    Error Details:

    • st_flb_utils_split_quoted_errors fails when processing invalid quoted strings.
    • The errors indicate unbalanced quotes should cause a failure but instead generate parsing error logs.

Steps to Reproduce

  1. Checkout the master branch post-merge of PR [#8851](cmake: fix UNICODE-escaped characters on aarch64 #8851).
  2. Enable the tests by omitting the -DFLB_WITHOUT_flb-it-utils=1 -DFLB_WITHOUT_flb-it-pack=1 options.
  3. Run the tests on an aarch64 environment.
  4. Observe failures in the oxy_url_split and st_flb_utils_split_quoted_errors tests.

Expected Behavior

The tests should pass on the aarch64 platform without errors when re-enabled.

Environment

@niedbalski niedbalski changed the title Test Failures on aarch64 After Merging PR [[#8851](https://github.com/fluent/fluent-bit/pull/8851)](https://github.com/fluent/fluent-bit/pull/8851) Test Failures on aarch64 After Merging PR 8851 Dec 3, 2024
@niedbalski niedbalski changed the title Test Failures on aarch64 After Merging PR 8851 Test failures on aarch64 after merging PR 8851 Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant