Bump php-wasm and php-cgi-wasm to 0.0.9-alpha-21

Created on 23 October 2024, 3 months ago

Problem/Motivation

This updates SQLite to a Drupal 11 supported version. And then we can add vrnzo for HTTP support.

πŸ“Œ Task
Status

Active

Component

Track: Trial Experience

Created by

πŸ‡ΊπŸ‡ΈUnited States mglaman WI, USA

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @mglaman
  • Merge request !148Bump php-wasm to 0.0.9-alpha-21 β†’ (Open) created by mglaman
  • Pipeline finished with Canceled
    3 months ago
    Total: 222s
    #319752
  • πŸ‡ΊπŸ‡ΈUnited States mglaman WI, USA
  • πŸ‡ΊπŸ‡ΈUnited States mglaman WI, USA

    still need to patch for https://github.com/seanmorris/php-wasm/issues/62 I believe

  • Pipeline finished with Failed
    3 months ago
    Total: 553s
    #319756
  • Pipeline finished with Failed
    3 months ago
    Total: 587s
    #319754
  • Pipeline finished with Failed
    3 months ago
    Total: 953s
    #319761
  • Pipeline finished with Failed
    3 months ago
    Total: 903s
    #319762
  • Merge request !149Bump php-wasm-sqlite β†’ (Merged) created by mglaman
  • πŸ‡ΊπŸ‡ΈUnited States mglaman WI, USA

    Since we hit a bug in vrzno on streams with HTTP method configured, I opened an MR which just bumps sqlite https://git.drupalcode.org/project/drupal_cms/-/merge_requests/149

  • Pipeline finished with Failed
    3 months ago
    Total: 691s
    #319840
  • Pipeline finished with Failed
    3 months ago
    Total: 688s
    #319841
  • Pipeline finished with Success
    3 months ago
    Total: 687s
    #319849
  • Pipeline finished with Success
    3 months ago
    Total: 661s
    #319850
  • Pipeline finished with Failed
    3 months ago
    Total: 662s
    #320047
  • Pipeline finished with Failed
    3 months ago
    Total: 698s
    #320046
  • πŸ‡ΊπŸ‡ΈUnited States mglaman WI, USA

    tried alpha23, failure: https://git.drupalcode.org/issue/drupal_cms-3483001/-/jobs/3154364#L82

    ⎯⎯⎯⎯ Unhandled Rejection ⎯⎯⎯⎯⎯
    TypeError: Headers.append: "" is an invalid header name.
     ❯ node:internal/deps/undici/undici:12618:11
    

    I think it's this HTTP/2 bug in undici, maybe a node 18 issue? https://github.com/nodejs/undici/pull/2332 I'll have to run it locally on my node 22 to see

  • πŸ‡ΊπŸ‡ΈUnited States mglaman WI, USA

    quirk in vrnzo, seanmorris found out and is working on a fix

  • πŸ‡ΊπŸ‡ΈUnited States mglaman WI, USA

    alpha24 is out and needs a try

  • First commit to issue fork.
  • Merge request !153Resolve #3483001 "24" β†’ (Merged) created by eiriksm
  • Pipeline finished with Canceled
    3 months ago
    Total: 306s
    #320650
  • Pipeline finished with Failed
    3 months ago
    Total: 721s
    #320660
  • Pipeline finished with Failed
    3 months ago
    Total: 761s
    #320659
  • Pipeline finished with Success
    3 months ago
    Total: 1016s
    #320658
  • Pipeline finished with Failed
    3 months ago
    Total: 382s
    #320673
  • Pipeline finished with Failed
    3 months ago
    Total: 386s
    #320672
  • Pipeline finished with Failed
    3 months ago
    Total: 959s
    #320681
  • Pipeline finished with Failed
    3 months ago
    Total: 966s
    #320680
  • Pipeline finished with Failed
    3 months ago
    Total: 434s
    #320804
  • Pipeline finished with Failed
    3 months ago
    Total: 443s
    #320805
  • Pipeline finished with Failed
    3 months ago
    Total: 804s
    #320811
  • Pipeline finished with Failed
    3 months ago
    Total: 753s
    #320814
  • πŸ‡ΊπŸ‡ΈUnited States mglaman WI, USA

    Okay we're close. It looks like setting env for CGI isn't working in the latest alpha

  • Pipeline finished with Failed
    3 months ago
    Total: 783s
    #320822
  • Pipeline finished with Failed
    3 months ago
    Total: 802s
    #320823
  • πŸ‡ΊπŸ‡ΈUnited States mglaman WI, USA

    The env value is respected. It's just that the form submission isn't working the same as it used to for some reason. The default value is sqlite but form validation has gone sideways.

  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    To unblock D11 support, I'll merge !149 now, then work on the other component can continue.

  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts
  • Pipeline finished with Skipped
    3 months ago
    #324409
  • Pipeline finished with Skipped
    3 months ago
    #324411
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    Back to the ol' drawing board!

  • Pipeline finished with Failed
    3 months ago
    Total: 610s
    #324434
  • πŸ‡ΊπŸ‡ΈUnited States mglaman WI, USA

    We found out the "bug": php-wasm added pdo_postgres support but it is compiled statically, not as a shared library like the other pdo extensions. So Drupal thinks Postgres is available and that's why the interactive installer test broke. We just need to workaround this in the trial installer.

  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    Debugged MR !153 with some guidance from @mglaman and discovered that our logic around submitting the database settings form was flawed - we need to assertively send it our default value when there is more than one choice available. Previously this worked by accident because SQLite was the only game in town, but the updated runtime also supports PostgreSQL, which meant that we were using SQLite but validating against Postgres' settings because of how the database form works.

    This is now fixed.

  • Pipeline finished with Failed
    3 months ago
    #324452
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts
  • Pipeline finished with Failed
    3 months ago
    Total: 486s
    #324479
  • Pipeline finished with Failed
    3 months ago
    Total: 670s
    #324492
  • Pipeline finished with Failed
    3 months ago
    Total: 686s
    #324491
  • Pipeline finished with Failed
    3 months ago
    Total: 513s
    #324516
  • Pipeline finished with Canceled
    3 months ago
    Total: 621s
    #324517
  • Pipeline finished with Skipped
    3 months ago
    #324530
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    What a slog this was to get done, but we got it done. And VRZNO is part of this update, so we get HTTP support now too - amazing!

  • Pipeline finished with Failed
    3 months ago
    Total: 601s
    #324523
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024