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

DataFactory API returns resource group name in lower-case #5788

Closed
mbfrahry opened this issue Apr 29, 2019 · 4 comments
Closed

DataFactory API returns resource group name in lower-case #5788

mbfrahry opened this issue Apr 29, 2019 · 4 comments
Labels
Data Factory Service Attention Workflow: This issue is responsible by Azure service team.

Comments

@mbfrahry
Copy link
Contributor

Hello! This is following up on #3964. It looks like the DataFactory API based off 2018-06-01 inside v26.7.0 of the azure sdk also returns the resource group name in lower case.

Terraform is throwing a ForceNew when a capitalized resource group name is passed into any DataFactory resource. For example:

------- Stdout: -------
=== RUN   TestAccAzureRMDataFactoryDatasetPostgreSQL_update
=== PAUSE TestAccAzureRMDataFactoryDatasetPostgreSQL_update
=== CONT  TestAccAzureRMDataFactoryDatasetPostgreSQL_update
--- FAIL: TestAccAzureRMDataFactoryDatasetPostgreSQL_update (83.73s)
    testing.go:568: Step 0 error: After applying this step, the plan was not empty:
        
        DIFF:
        
        DESTROY/CREATE: azurerm_data_factory.test
          github_configuration.#: "0" => "0"
          id:                     "/subscriptions/8708baf2-0a54-4bb4-905b-78d21ac150da/resourceGroups/acctestrg-1154640959761039184/providers/Microsoft.DataFactory/factories/acctestdf1154640959761039184" => "<computed>"
          identity:               "" => "<computed>"
          identity.#:             "0" => ""
          location:               "westeurope" => "westeurope"
          name:                   "acctestdf1154640959761039184" => "acctestdf1154640959761039184"
          resource_group_name:    "acctestrg-1154640959761039184" => "acctestRG-1154640959761039184" (forces new resource)
          tags:                   "" => "<computed>"
          vsts_configuration.#:   "0" => "0"

The following tests are failing as well;

 TestAccAzureRMDataFactoryDatasetMySQL_basic     
 TestAccAzureRMDataFactoryDatasetMySQL_update     
 TestAccAzureRMDataFactoryDatasetPostgreSQL_basic     
 TestAccAzureRMDataFactoryDatasetPostgreSQL_update     
 TestAccAzureRMDataFactoryDatasetSQLServerTable_basic     
 TestAccAzureRMDataFactoryDatasetSQLServerTable_update     
 TestAccAzureRMDataFactoryLinkedServiceMySQL_basic     
 TestAccAzureRMDataFactoryLinkedServiceMySQL_update     
 TestAccAzureRMDataFactoryLinkedServicePostgreSQL_basic     
 TestAccAzureRMDataFactoryLinkedServicePostgreSQL_update     
 TestAccAzureRMDataFactoryLinkedServiceSQLServer_basic     
 TestAccAzureRMDataFactoryPipeline_basic     
 TestAccAzureRMDataFactoryPipeline_update     
 TestAccAzureRMDataFactory_basic     
 TestAccAzureRMDataFactory_github     
 TestAccAzureRMDataFactory_identity     
 TestAccAzureRMDataFactory_tags 

Thanks!

@ghost
Copy link

ghost commented May 27, 2020

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @shawnxzq, @lmy269.

@lmy269
Copy link

lmy269 commented May 29, 2020

@shawnxza to take a look

@shawnxzq
Copy link
Member

@LijuanZ Could you please have a look at this?

@LijuanZ
Copy link
Contributor

LijuanZ commented Jul 27, 2020

This is by design that Data Factory stores only lowercase resource group name

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Data Factory Service Attention Workflow: This issue is responsible by Azure service team.
Projects
None yet
Development

No branches or pull requests

6 participants