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
nixos/tests/matrix-synapse: fix test #45596
Conversation
Since matrix-synapse 0.33.0 underscores in server names are rejected by server name validation, causing the test to fail: valueError: Server name 'server_sqlite' contains invalid characters Relevant upstream change: matrix-org/synapse@546bc9e
@GrahamcOfBorg test matrix-synapse |
Timed out, unknown build status on x86_64-linux (full log) Attempted: tests.matrix-synapse Partial log (click to expand)
|
Timeout because of a kernel update, let's wait and try again later. |
Success on aarch64-linux (full log) Attempted: tests.matrix-synapse Partial log (click to expand)
|
@GrahamcOfBorg test matrix-synapse |
Success on aarch64-linux (full log) Attempted: tests.matrix-synapse Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: tests.matrix-synapse Partial log (click to expand)
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Runs fine here as well, looks good :)
@GrahamcOfBorg test matrix-synapse |
Success on aarch64-linux (full log) Attempted: tests.matrix-synapse Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: tests.matrix-synapse Partial log (click to expand)
|
Motivation for this change
Since
matrix-synapse
was upgraded to 0.33.0, underscores in server names are rejected by server name validation, causing the test to fail on hydra:valueError: Server name 'server_sqlite' contains invalid characters
Relevant upstream change: matrix-org/synapse@546bc9e
Not sure if this is an upstream bug or a deliberate decision, filed matrix-org/synapse#3759 to check.
Anyway, we can just fix the tests by using server names without underscores.
Things done