Remove feature flag for VNet, unhide tsh vnet (#42304)

gravitational · Jun 3, 2024 · dc2c649 · dc2c649
1 parent c59bc5a
commit dc2c649
Show file tree

Hide file tree

Showing 6 changed files with 4 additions and 11 deletions.
diff --git a/tool/tsh/common/vnet_darwin.go b/tool/tsh/common/vnet_darwin.go
@@ -33,8 +33,7 @@ type vnetCommand struct {
 
 func newVnetCommand(app *kingpin.Application) *vnetCommand {
 	cmd := &vnetCommand{
-		// TODO(nklaassen): unhide this when ready to ship.
-		CmdClause: app.Command("vnet", "Start Teleport VNet, a virtual network for TCP application access.").Hidden(),
+		CmdClause: app.Command("vnet", "Start Teleport VNet, a virtual network for TCP application access."),
 	}
 	return cmd
 }

diff --git a/web/packages/teleterm/src/mainProcess/fixtures/mocks.ts b/web/packages/teleterm/src/mainProcess/fixtures/mocks.ts
@@ -33,7 +33,6 @@ export class MockMainProcessClient implements MainProcessClient {
       jsonSchemaFile: createMockFileStorage(),
       platform: this.getRuntimeSettings().platform,
     });
-    this.configService.set('feature.vnet', true);
   }
 
   subscribeToNativeThemeUpdate() {

diff --git a/web/packages/teleterm/src/services/config/appConfigSchema.ts b/web/packages/teleterm/src/services/config/appConfigSchema.ts
@@ -122,7 +122,6 @@ export const createAppConfigSchema = (platform: Platform) => {
       .boolean()
       .default(false)
       .describe('Disables SSH connection resumption.'),
-    'feature.vnet': z.boolean().default(false).describe('Shows UI for VNet.'),
   });
 };
 

diff --git a/web/packages/teleterm/src/ui/DocumentCluster/ActionButtons.story.tsx b/web/packages/teleterm/src/ui/DocumentCluster/ActionButtons.story.tsx
@@ -44,7 +44,6 @@ export default {
 
 export function ActionButtons() {
   const appContext = new MockAppContext();
-  appContext.configService.set('feature.vnet', true);
   prepareAppContext(appContext);
 
   return (

diff --git a/web/packages/teleterm/src/ui/TopBar/Connections/Connections.story.tsx b/web/packages/teleterm/src/ui/TopBar/Connections/Connections.story.tsx
@@ -265,7 +265,6 @@ const prepareAppContext = (appContext: MockAppContext) => {
   appContext.connectionTracker.disconnectItem = async () => {};
   appContext.connectionTracker.removeItem = async () => {};
   appContext.connectionTracker.useState = () => null;
-  appContext.configService.set('feature.vnet', true);
 };
 
 const useOpenConnections = () => {

diff --git a/web/packages/teleterm/src/ui/Vnet/vnetContext.tsx b/web/packages/teleterm/src/ui/Vnet/vnetContext.tsx
@@ -55,7 +55,7 @@ export const VnetContext = createContext<VnetContext>(null);
 
 export const VnetContextProvider: FC<PropsWithChildren> = props => {
   const [status, setStatus] = useState<VnetStatus>('stopped');
-  const { vnet, mainProcessClient, configService } = useAppContext();
+  const { vnet, mainProcessClient } = useAppContext();
   const isWorkspaceStateInitialized = useStoreSelector(
     'workspacesService',
     useCallback(state => state.isInitialized, [])
@@ -65,10 +65,8 @@ export const VnetContextProvider: FC<PropsWithChildren> = props => {
   });
 
   const isSupported = useMemo(
-    () =>
-      mainProcessClient.getRuntimeSettings().platform === 'darwin' &&
-      configService.get('feature.vnet').value,
-    [mainProcessClient, configService]
+    () => mainProcessClient.getRuntimeSettings().platform === 'darwin',
+    [mainProcessClient]
   );
 
   const [startAttempt, start] = useAsync(