Skip to content

[client] Add disable system flags#3153

Merged
lixmal merged 5 commits intomainfrom
add-disable-system-flags
Jan 7, 2025
Merged

[client] Add disable system flags#3153
lixmal merged 5 commits intomainfrom
add-disable-system-flags

Conversation

@lixmal
Copy link
Copy Markdown
Collaborator

@lixmal lixmal commented Jan 6, 2025

Describe your changes

New cli flags:

      --disable-client-routes           Disable client routes. If enabled, the client won't process client routes received from the management service.
      --disable-dns                     Disable DNS. If enabled, the client won't configure DNS settings.
      --disable-firewall                Disable firewall configuration. If enabled, the client won't modify firewall rules.
      --disable-server-routes           Disable server routes. If enabled, the client won't act as a router for server routes received from the management service.

Issue ticket number and link

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

@lixmal lixmal force-pushed the add-disable-system-flags branch from 9926126 to b35b226 Compare January 6, 2025 15:57
@lixmal lixmal force-pushed the add-disable-system-flags branch from b35b226 to f53b960 Compare January 6, 2025 16:07
@lixmal lixmal force-pushed the add-disable-system-flags branch from 4ae8080 to 20cdd49 Compare January 6, 2025 16:37
Comment thread client/cmd/up.go
Comment thread client/internal/routemanager/manager.go
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jan 7, 2025

@lixmal lixmal merged commit d9905d1 into main Jan 7, 2025
@lixmal lixmal deleted the add-disable-system-flags branch January 7, 2025 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants