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

Compare and contrast azure-py-vdc and azureng-py-vdc #806

Merged
merged 19 commits into from
Oct 8, 2020
Merged

Compare and contrast azure-py-vdc and azureng-py-vdc #806

merged 19 commits into from
Oct 8, 2020

Conversation

jamesianberry
Copy link
Contributor

This includes both an update to azure-py-virtual-data-center and a new azureng-py-virtual-data-center.

The two examples are interoperable - you can peer a stack in one with a stack in the other. Both implement the latest Azure Firewall features including forced tunnelling - although the current azure provider can't have additional properties.

Code has been refactored so that both examples are directly comparable - examine vdc.py to see the differences between providers. For example, some parameter names have changed e.g. public_ip_address is now publicIPAddress. Differences in other files are to do with the lack of auto-naming in azure-nextgen.

Code has been optimised to overcome contention issues that show up in the activity log, by moving things around and adding depends_on statements.

These examples demonstrate custom routing that avoids a common pitfall, where traffic already headed to one member of the Azure Firewall VM Scale Set is redirected to the private IP address - causing looping and dropped traffic.

@mikhailshilkov
Copy link
Member

This looks great @jamesianberry, amazing job!

One high-level question: what do you think about using argument classes instead of untyped dictionaries? (see this example) They are a relatively new addition but I find them very valuable in terms of safety and IDE support.

@mikhailshilkov mikhailshilkov merged commit e942d61 into pulumi:master Oct 8, 2020
@jamesianberry jamesianberry deleted the jamesianberry/azure-nextgen-py-vdc branch October 8, 2020 16:41
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.

None yet

2 participants