Skip to content
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

Deployer code improvements #692

Open
BMurri opened this issue Aug 25, 2023 · 0 comments
Open

Deployer code improvements #692

BMurri opened this issue Aug 25, 2023 · 0 comments
Assignees
Labels
Bug Something isn't working enhancement New feature or request
Milestone

Comments

@BMurri
Copy link
Collaborator

BMurri commented Aug 25, 2023

Describe the bug
There are multiple inconsistencies in "List" API usages (leading to inconsistent ability for the deployer to find the results it's looking for.
There are also multiple places where deployment cancellation due to failures can stall out, delaying the error reporting as well as permitting more parallel deployment activities to continue.

These two issues are found in both CoA and TES deployers.

Steps to Reproduce
Use an account with access to a lot of subscriptions with a lot of resources already deployed.

Expected behavior
Existing resources are found rather than not found, resulting in either duplicate resource creation (possibly leading to disconnects or failures later in connectivity) or failures in the deployment due to not finding the existing specified resource.

Code dependencies
Will this require code changes in:

  • CoA: yes (deployer code only)
  • TES: yes (deployer, CommonUtilities, TesApi.Web, TesApi.Test)
  • Build pipeline: no
  • Integration tests: no

Additional context
Code was written and is in the TES service that ameliorates the issues with list apis not always returning all the needed data using a consistent pattern that can be checked for and it eliminates having to learn all the different API styles for paged lists in the azure sdks. It should be made available to be used in both repositories (ideally by placing it in the CommonUtilities project).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant