Skip to content

Conversation

@yugangw-msft
Copy link
Contributor

@yugangw-msft yugangw-msft commented May 23, 2018

Fix #6318
2 notes:

  1. VM create has a service end issue to clarify; if turns out a blocker, i will pull related change out
  2. Live test is not feasible as it involve 2 tenants but our live run uses tenant wide service principal, so won't work; but i can write a record only test once the test tool gets fixed to scrub out the new token header, remove x-ms-authorization-auxiliary header azure-python-devtools#46
  • The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

  • I adhere to the Command Guidelines.

@yugangw-msft yugangw-msft requested a review from tjprescott May 23, 2018 22:45
@promptws
Copy link

View a preview at https://prompt.ws/r/Azure/azure-cli/6423
This is an experimental preview for @microsoft users.

@yugangw-msft
Copy link
Contributor Author

Closing this PR as Compute RP is not ready for such scenarios, which is revealed through failues on vm created from external images
I will open a new one for vnet-peering only

@yugangw-msft yugangw-msft deleted the crossTenant branch July 14, 2018 20: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.

2 participants