-
Notifications
You must be signed in to change notification settings - Fork 268
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
chore(gui-client/linux): fix group name, firezone
should be firezone-client
#4889
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Terraform Cloud Plan Output
|
Performance Test ResultsTCP
UDP
|
1. Check the IP again | ||
1. Export the logs | ||
1. Expect `zipinfo` to show |
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.
Oops, forgot to finish this part. I'll get it before it 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.
lgtm
```[tasklist] ### Before merging - [x] (FAILED) Test CI deb on Ubuntu 24.04 #4883 - [x] Wait for everything else to merge: #3884 - [x] Fix #4889 - [x] Fix #4890 - [x] Test on Ubuntu 20.04 (683bddc passed) - [x] Test on Ubuntu 22.04 (683bddc passed) - [x] diff between main and 683bddc to make sure nothing in the code changed - [ ] Someone other than me should give it a once-over. `intended_behavior.md` has the manual smoke test I've been doing on it. Install script is <https://github.com/firezone/firezone/blob/a943a9dba1797317327563bb2de48e22f239175a/scripts/firezone-client-gui-install.sh> Deb package from CI is <https://github.com/firezone/firezone/actions/runs/8972824465/artifacts/1477261361> ```
.context("can't get group by name")? | ||
.context("firezone group must exist on the system")?; | ||
.context("`{FIREZONE_GROUP}` group must exist on the system")?; |
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.
@ReactorScram This string interpolation does not work, you need format!
for that :)
Also improved the manual testing checklist a little