fix(infra): resolve CI, test, and release workflow failures#4
Open
fix(infra): resolve CI, test, and release workflow failures#4
Conversation
- CI Dialyzer: Add PLT path config to root mix.exs and change alias from `cmd mix dialyzer` (which loops in umbrella children) to run dialyxir directly from the umbrella root. Create priv/plts dir in CI. - Test AgentChannel: Update stale tests that expected direct topic join to be rejected; the channel now allows direct join with auto-auth. Restructure integration tests to use direct topic join for pre-auth state. Add try/rescue to authenticate handler to prevent silent channel crashes when bootstrap_agent fails. - Test VerifyClientCertificate: Replace openssl x509 -not_before/ -not_after flags (requires OpenSSL >= 3.2) with openssl ca -startdate/-enddate which works across all OpenSSL versions. - Release Docker: Set placeholder DATABASE_URL and SECRET_KEY_BASE during asset build stage to prevent runtime.exs from raising when mix assets.deploy triggers config evaluation in prod.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
cmdalias-startdate/-enddateinstead of-not_before/-not_after)DATABASE_URLandSECRET_KEY_BASEduring asset compilationTest plan