Skip to content

Add initial module #1

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
Open
wants to merge 74 commits into
base: main
Choose a base branch
from
Open

Add initial module #1

wants to merge 74 commits into from
+2,038 −91

Conversation

rswrz
Copy link
Member

@rswrz rswrz commented May 9, 2025

This pull request introduces the initial implementation of the Terraform module, including the core logic, input/output definitions, and required structure to support usage as a reusable module. This sets the foundation for further enhancements, testing, and documentation.

rswrz added 30 commits February 4, 2025 12:59
@rswrz
Add initial Terraform code from Microsoft Learn article
8d40605 
source: https://learn.microsoft.com/en-us/azure/ai-studio/how-to/create-hub-terraform

Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Adjust Terraform tests
70e58a0 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Run pre-commit hooks
0d9200a 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Add lost random_string resource
fc7bec4 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Adjust body value type to make it compatible azureapi provider >=2.0
48a9590 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Rename Terraform resource to this
7eb702b 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Move data source to dedicated file
ce44858 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Use required provider azurerm v4
4484a9d 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Fix drift after initial deployment
bd7a10c 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Replace azapi AI Service resource with azurerm one
16f2c3f 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Fix Terraform drift on output/tags
fc0b20e 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Refactor resource naming
e686eef 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Update usage example
e9e50bc 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Add missing mock to example test
086c9e6 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Adjust local default test
814d45d 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Build resource group id in local instead of getting it via data
da28069 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Adjust remote test
0fe021c 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Update CI configuration files
5ea6ce5 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Fix TFLint issues
cf11d68 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Add search service resource
3546e08 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Add connection to search service
d9b2517 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Add models input variable
6c69249 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Move Azure AI Foundry resources to a submodule
8bd1fc3 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Enable identity auth mode and prepare network restrictions
6ff533b 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Move Azure AI Service resources to a submodule
8e1e73e 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Bump API version
abf3325 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Make Key Vault and Storage Account private
1940479 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Rename resource group name in example
f439cba 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Improve sorting logic for better readability and efficiency
ca7ce13 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Add helpful outputs
549f6bc 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
rswrz added 5 commits March 11, 2025 22:05
@rswrz
Change variable name to basename
aae31da 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Refactor RBAC creation
3297161
@rswrz
Update config files
aed0a1c 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Fix terraform-docs whitespaces on tables
4e1de61 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Consider tflint configs for examples and tests in submodules
34a114a 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Fix Terraform test failures
5a507cb 
- Add required mocks to example-based tests
- Refactor name variable usage to use basename in local and remote test cases
- Set azurerm provider version constraint to >= 4.14 to ensure compatibility

Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz rswrz added the feature label May 9, 2025
rswrz added 2 commits May 9, 2025 08:59
@rswrz
Enable AzureAD authentication for storage account in remote test
e914dda 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Add comment about create_rbac variable in usage example
dad4545 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
lixhunter
lixhunter requested changes May 9, 2025
View reviewed changes
@rswrz
Fix typo in pre-commit config
58488a3 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
lixhunter
lixhunter previously approved these changes May 9, 2025
View reviewed changes
rswrz added 4 commits May 23, 2025 09:25
@rswrz
Add new outputs for hub, project, and key vault ID and principal ID
7d7d375 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Implement model deployment_name property
95d2f22 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Change variable description
742d664 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Fix drift on empty description
56d2703 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
rswrz added 7 commits June 12, 2025 21:25
@rswrz
Add search service outputs
165c84e 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Update model versions
4753fab 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Add text-embedding-ada-002 model defaults
a58e9da 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Adjust azapi version constraint
a172b25 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Fix resource ID reference in AI services connection for search service
1dfca5d 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Use standardized names for search and ai service
a416858 
Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
@rswrz
Add explicit dependencies to prevent intermittent Azure API errors fo…
0494018 
…r AI services and search service connections

Signed-off-by: Roman Schwarz <rs@cloudeteer.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lixhunter lixhunter lixhunter approved these changes

@Phil-Thoennissen Phil-Thoennissen Awaiting requested review from Phil-Thoennissen Phil-Thoennissen is a code owner

@neonwhiskers neonwhiskers Awaiting requested review from neonwhiskers neonwhiskers is a code owner

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

Successfully merging this pull request may close these issues.
2 participants
@rswrz @lixhunter