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

adding azure gaming: gdvmss #120

Merged
merged 51 commits into from
Aug 11, 2022
Merged

adding azure gaming: gdvmss #120

merged 51 commits into from
Aug 11, 2022

Conversation

dciborow
Copy link
Contributor

@dciborow dciborow commented Jul 26, 2022

Description

Closes #121
If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check. Once you are done and ready to submit your PR, edit the PR description and run through the relevant checklist below.

Adding a new module

  • A proposal has been submitted and approved.
  • I have included "Closes #{module_proposal_issue_number}" in the PR description.
  • I have run brm validate locally to verify the module files.
  • I have run deployment tests locally to ensure the module is deployable.

Updating an existing module

  • This is a bug fix:
    • Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
    • The bug was found by the module author, and no one has opened an issue to report it yet.
  • I have run brm validate locally to verify the module files.
  • I have run deployment tests locally to ensure the module is deployable.
  • I have read the Updating an existing module section in the contributing guide and updated the version.json file properly:
    • The PR contains backwards compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json.
    • The PR contains backwards compatible feature updates, and I have bumped the MINOR version in version.json.
    • The PR contains breaking changes, and I have bumped the MAJOR version in version.json.

@dciborow
Copy link
Contributor Author

#121

@dciborow
Copy link
Contributor Author

Closes #121

@dciborow
Copy link
Contributor Author

@bmoore-msft , in order to run the tests I need to have the VM Offer approved.

publisher='microsoft-azure-gaming' offer = 'game-dev-vm', sku = 'win10_unreal_4_27_2'

@shenglol
Copy link
Contributor

@dciborow Give me some time to accept the legal terms of the image on the subscription, which should fix the test failure.

@shenglol shenglol closed this Jul 26, 2022
@shenglol shenglol reopened this Jul 26, 2022
@shenglol
Copy link
Contributor

@dciborow Do you mind taking a look at #121 (comment)?

@dciborow dciborow closed this Aug 2, 2022
@dciborow dciborow reopened this Aug 2, 2022
@dciborow
Copy link
Contributor Author

dciborow commented Aug 2, 2022

@shenglol , this is ready for review

modules/azure-gaming/game-dev-vmss/main.bicep Show resolved Hide resolved
modules/azure-gaming/game-dev-vmss/main.bicep Outdated Show resolved Hide resolved
modules/azure-gaming/game-dev-vmss/main.bicep Outdated Show resolved Hide resolved
modules/azure-gaming/game-dev-vmss/main.bicep Outdated Show resolved Hide resolved
modules/azure-gaming/game-dev-vmss/main.bicep Outdated Show resolved Hide resolved
modules/azure-gaming/game-dev-vmss/main.bicep Show resolved Hide resolved
@shenglol
Copy link
Contributor

shenglol commented Aug 2, 2022

@dciborow Posting the question that Alex asked in the module proposal here in case you missed it:
image

@dciborow dciborow closed this Aug 4, 2022
@dciborow dciborow reopened this Aug 4, 2022
@dciborow dciborow requested a review from alex-frankel August 4, 2022 19:07
@shenglol
Copy link
Contributor

shenglol commented Aug 5, 2022

@alex-frankel Are we good with the changes? If yes, I can start the MCR manifest onboarding process.

Copy link

@alex-frankel alex-frankel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have a small change, but it is not blocking and can happen later.

Can we take an action item to further debug why the vnet module was not working? Maybe myself, @dciborow, @MrMCake can all sit down together and see if we can figure it out? I'd really like to move back to the vnet module in the registry at some point.

@dciborow
Copy link
Contributor Author

@alex-frankel , we fixed it. this now uses the module, (but still within my own virtualNetwork.bicep)
image

@alex-frankel
Copy link

Thanks! @shenglol this is ready to merge from my perspective

@shenglol shenglol merged commit d0036aa into Azure:main Aug 11, 2022
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.

[Module Proposal]: Azure Gaming: GameDev VMSS
4 participants