Skip to content

Update flight flags and make corresponding changes #355

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

Merged
merged 2 commits into from
Mar 7, 2025

Conversation

daxian-dbw
Copy link
Member

PR Summary

Update flight flags and make corresponding changes:

  1. Add the flag pipelineorchestration per the Azure Copilot team's request.
  2. Update flag values based on the values used by Azure Portal, and remove 2 flags that are no longer used by Azure Portal.
  3. Update the topic names for AzCLI handler and AzPS handler. They changed after adopting the new flag values.
  4. Simplify the instruction prompt to save words for the input. Also include Azure PowerShell in the instruction.
  5. Use better OS description for the instruction prompt.
  6. Log the topic name for every response, for diagnostic purpose. We already send this information in telemetry.
  7. Fix a bug in response parsing in the validation of the beginning of placeholder section.
  8. Fix the status report to only report ProgressTopic status, which are progress reports only (fix Very brief large loading status message appears when Azure agent is waiting for response back #352).
  9. Special case TaskCanceledException in AzCLI metadata query for logging and telemetry collection. This exception simply means the query action timed out after 1200 ms. Given that the metadata endpoint is an Azure Function, this could easily happen when the query hits a cold start of the function app. So, we don't want to log the exception object here.

@daxian-dbw daxian-dbw merged commit eba1e7a into PowerShell:main Mar 7, 2025
4 checks passed
@daxian-dbw daxian-dbw deleted the flights branch March 7, 2025 22:21
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.

Very brief large loading status message appears when Azure agent is waiting for response back
1 participant