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.

Sign up for GitHub

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

Jump to bottom

Update azure requirements and regions #14706

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Update azure requirements and regions #14706

wants to merge 4 commits into from
+62 −5

Conversation

jackivanov
Copy link
Collaborator

@jackivanov jackivanov commented Mar 13, 2024 •
edited
Loading

Description

  • Upgrade azure requirements
  • Add new locations

Motivation and Context

Fixes #14704

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • [] My change requires a change to the documentation.
  • [] I have updated the documentation accordingly.
  • [] I have added tests to cover my changes.
  • All new and existing tests passed.

@jackivanov jackivanov mentioned this pull request Mar 13, 2024
Open
@MHN452
Copy link

MHN452 commented Mar 13, 2024

It all works well now, thanks a lot!

@jackivanov jackivanov enabled auto-merge (squash) March 14, 2024 02:31
Majid6561
Majid6561 reviewed Apr 19, 2024
View reviewed changes
roles/cloud-azure/defaults/main.yml

This comment was marked as off-topic.

Sign in to view

@shijialee
Copy link

shijialee commented Apr 20, 2024 •
edited
Loading

I only applied the change where version changed in roles/cloud-azure/tasks/venv.yml and it is working for me now. on MacOS.
Hope this branch gets merge soon.

DeveTho pushed a commit to DeveTho/algo that referenced this pull request Dec 26, 2024
@jackivanov @DeveTho
update azure requirements and regions
1e7e713 
Taken from trailofbits#14706
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Majid6561 Majid6561 Majid6561 left review comments

@MHN452 MHN452 MHN452 approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Azure Deployment ModuleNotFoundError: No module named 'azure.mgmt.rdbms.postgresql_flexibleservers'
4 participants
@jackivanov @MHN452 @shijialee @Majid6561