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.

Sign up for GitHub

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

Jump to bottom

bug: CI fails when deploying on newer kernels #268

Open
darcywong00 opened this issue Jan 29, 2025 · 2 comments · May be fixed by #270
Open

bug: CI fails when deploying on newer kernels #268

darcywong00 opened this issue Jan 29, 2025 · 2 comments · May be fixed by #270
Assignees
@darcywong00
Milestone
B18S3

Comments

@darcywong00
Copy link
Contributor

Recapping notes as @tim-eves was attempting to migrate our staging sites:

The current version of keyman-staging is running fine on the existing cluster, there has been no down time.
While migrating keyman-staging to the new cluster, we ran into this bug: microsoft/mssql-docker#881
The new cluster is running newer kernels that expose the bug in mssql.
versions 2022-CU13-ubuntu-22.04 and 2022-latest are reported to work, though you container is locked to an earlier commit of 2022-latest

We can try to revert #217 so they newer images get deployed to the staging images on the old and new clusters

Going to 2022-latest does run into a PDOException in

        INSERT t_keyboard_language (
          keyboard_id,
          bcp47,
          language_id,
          region_id,
          script_id,
          description
        ) VALUES
              (N'cantonese_telex',
              N'yue-latn-001',
              N'yue',
              N'001',
              N'latn',
              N'Chinese, Yue');
Array
(
    [0] => 42000
    [1] => 2628
    [2] => [Microsoft][ODBC Driver 18 for SQL Server][SQL Server]String or binary data would be truncated in table 'keyboards.k0.t_keyboard_language', column 'region_id'. Truncated value: '00'.
    [3] => 01000
    [4] => 3621
    [5] => [Microsoft][ODBC Driver 18 for SQL Server][SQL Server]The statement has been terminated.
)
Failure: PDOException: SQLSTATE[42000]: [Microsoft][ODBC Driver 18 for SQL Server][SQL Server]String or binary data would be truncated in table 'keyboards.k0.t_keyboard_language', column 'region_id'. Truncated value: '00'. in /var/www/html/tools/db/build/build.inc.php:153

Seems yue-latn-001 region is getting truncated to 00.
@darcywong00 darcywong00 added this to the A18S20 milestone Jan 29, 2025
@darcywong00 darcywong00 self-assigned this Jan 29, 2025
@darcywong00 darcywong00 mentioned this issue Jan 29, 2025
Merged
@darcywong00
Copy link
Contributor Author

Fixed by #267

@darcywong00
Copy link
Contributor Author

darcywong00 commented Jan 30, 2025 •
edited
Loading

Re-opening. Tim reports CI is still halting on the same crash.

api.keyman-staging.com is working with this fix.
Keeping the issue open till we merge #270 to master.

@darcywong00 darcywong00 reopened this Jan 30, 2025
darcywong00 added a commit that referenced this issue Jan 30, 2025
@darcywong00
fix: Update mssql.Dockerfile server:2022-latest to newer pin
608ee30 
Cherrypick of #267 to master. Fixes #268
@darcywong00 darcywong00 linked a pull request Jan 30, 2025 that will close this issue
Open
@darcywong00 darcywong00 modified the milestones: A18S20, B18S3 Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@darcywong00 darcywong00

Labels
None yet
Projects
Keyman
Archived in project
Milestone
B18S3
Development

Successfully merging a pull request may close this issue.

fix: Update mssql.Dockerfile server:2022-latest to newer pin 🍒 keymanapp/api.keyman.com
1 participant
@darcywong00