Skip to content

fixes for roundtrip tests in Premium SKU#80

Open
EMaher wants to merge 3 commits into
mainfrom
copilot/verify-integration-test-premium-sku
Open

fixes for roundtrip tests in Premium SKU#80
EMaher wants to merge 3 commits into
mainfrom
copilot/verify-integration-test-premium-sku

Conversation

@EMaher
Copy link
Copy Markdown
Contributor

@EMaher EMaher commented May 22, 2026

This pull request introduces several improvements and fixes across the workflow configuration, resource type modeling, dependency graph, and publishing logic, with a focus on supporting workspace resources, improving API revision handling, and enhancing logging and masking practices. The most important changes are grouped below:

Workflow and Logging Enhancements:

  • Added a log_level input to .github/workflows/integration-test.yml to control PowerShell log verbosity, defaulting to Verbose, and validated allowed values. The default Azure region was changed from eastus2 to centralus. [1] [2] [3]
  • Added guidance and technical recommendations for masking sensitive data in logs, especially around PowerShell transcript logging, ARM async-operation URLs, and error stream handling. [1] [2]

Workspace Resource Support:

  • Introduced a new Workspace resource type, updated RESOURCE_TYPE_METADATA, and ensured workspace containers are parsed and published correctly before their children. [1] [2] [3] [4]
  • Updated the dependency graph to include Workspace as a Tier 1 resource and added relevant dependency edges. [1] [2] [3] [4]

API Revision Publishing Logic:

  • Changed the API publishing flow to ensure the root API is only aligned as the active revision when the source explicitly marks it as current (isCurrent: true in apiInformation.json). Refactored revision publishing to return a count, and added logic to re-apply root metadata for alignment when necessary. [1] [2] [3] [4] [5] [6] [7] [8]

Resource Path and Association Handling:

  • Refactored the handling of association resource types to clarify which are parent-published and to ensure workspace container descriptors are discovered and published correctly. [1] [2]

Other Improvements and Fixes:

  • Improved error handling and logging in workspace extraction, and removed unnecessary try/catch blocks. [1] [2]
  • Updated documentation and research history to reflect new findings on resource grammar and masking practices.

These changes collectively improve the robustness, security, and maintainability of the codebase, with special attention to workspace resource support and secure logging practices.

Agent-Logs-Url: https://github.com/Azure/apiops-cli/sessions/08964b6c-4c8a-4dda-bcb9-17322e0e16bb

Fixes #58.
Fixes #23

Agent-Logs-Url: https://github.com/Azure/apiops-cli/sessions/08964b6c-4c8a-4dda-bcb9-17322e0e16bb

Co-authored-by: EMaher <9244742+EMaher@users.noreply.github.com>
@EMaher EMaher force-pushed the copilot/verify-integration-test-premium-sku branch from 56669da to 7146ca8 Compare May 22, 2026 07:05
@EMaher EMaher requested a review from petehauge May 22, 2026 07:13
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.

Verify all-types integration test with Premium SKU Remove unused validation rules from "expected-structure.json".

2 participants