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
refactor(headless-client): split up the entry points for headless client and IPC service #4942
Conversation
…tor/remove-linux-client
…ne into refactor/remove-auto-ipc
…ent and IPC service
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Terraform Cloud Plan Output
|
…efactor/split-binaries
Signed-off-by: Reactor Scram <ReactorScram@users.noreply.github.com>
…e into refactor/split-binaries
@@ -131,8 +131,9 @@ jobs: | |||
timeout 600 docker compose -f rust/snownet-tests/docker-compose.${{ matrix.name }}.yml up --exit-code-from dialer --abort-on-container-exit | |||
|
|||
compatibility-tests: | |||
# Tests the older, published Client and Gateway against the new infrastructure |
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.
I could not remember which way it went
# Don't run compatibility tests when called from hotfix.yml or publish.yml on `main` because | ||
# it'll be red if there was a breaking change we're tring to publish, | ||
# it'll be red if there was a breaking change we're trying to publish, |
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.
noticed this typo while working on this
@@ -320,7 +320,6 @@ services: | |||
- type=registry,ref=us-east1-docker.pkg.dev/firezone-staging/cache/client:main | |||
args: | |||
PACKAGE: firezone-headless-client | |||
# Add "standalone" to the command here once PR $4604 merges |
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.
Changing CLI args is complicated, skipping it for now
Performance Test ResultsTCP
UDP
|
Closing in favor of #4953. The refactors are taking too long so I'll skip most of them for now and try to do IPC without the refactoring. |
Builds atop #4930 and #4943
Tasks