Skip to content

fix: dynamic activation server dns, profile, dnssuffix#1167

Open
nbmaiti wants to merge 1 commit intomainfrom
rpc_go_launch
Open

fix: dynamic activation server dns, profile, dnssuffix#1167
nbmaiti wants to merge 1 commit intomainfrom
rpc_go_launch

Conversation

@nbmaiti
Copy link

@nbmaiti nbmaiti commented Feb 13, 2026

update launch for dynamic profile name, dnssuffix
and activation server names

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the VS Code launch configuration to support dynamic input prompts for activation server addresses, profile names, and DNS suffixes. The changes replace hardcoded placeholder values with interactive prompts, making it easier for developers to test different configurations without manually editing the launch.json file.

Changes:

  • Added an "inputs" section with three promptString inputs for serverAddress, profileName, and dnsSuffix
  • Replaced hardcoded placeholder values ([server], [profile]) with dynamic variable references (${input:serverAddress}, ${input:profileName})
  • Added "buildFlags": ["-race"] to all Go debug configurations to enable race detection
  • Removed the "asRoot": true property from all debug configurations

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

    update launch for dynamic profile name, dnssuffix
    and activation server names

Signed-off-by: Nabendu Maiti <nabendu.bikash.maiti@intel.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

1 participant