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

Implement subnet-ids parameter #90

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nirit100
Copy link

@nirit100 nirit100 commented Feb 14, 2025

...and necessarily updated a lot of the code on the way.

The workflow now accepts a list of subnet-ids to deploy the image to.
To be able to do that, I had to

  • update the library versions
  • add 'private' IP functionality

So I had to touch a lot of stuff I do not know much about. I tested it with a work related project, so I cannot share anything about that, but it worked there and the API seems to report errors in the input very nicely.

This is aiming to resolve #74

Please just let me know if there is something wrong with the PR, I am new to Azure and related Workflows.

@nirit100 nirit100 force-pushed the implement-subnet-ids branch 3 times, most recently from 34bd1a2 to c6e4dc9 Compare February 14, 2025 15:14
Also needed to update dependencies for this to work.

This is a squash of 7 commits:

first try

second try

third try

updated meta info

added libraries

removed necessity for dns label

fr this time

Old summary:
implemented subnet-ids, private IPs, updated lots of stuff
@nirit100 nirit100 force-pushed the implement-subnet-ids branch from c6e4dc9 to 727cd48 Compare February 14, 2025 15:47
@nirit100
Copy link
Author

@microsoft-github-policy-service agree

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.

Add support for Vnet integration
1 participant