Skip to content

Commit

Permalink
fix: Use TypeString for parent ids (#2136)
Browse files Browse the repository at this point in the history
Azure ids are not guaranteed to be UUIDs
  • Loading branch information
erezrokah committed Sep 29, 2022
1 parent 09d63d9 commit a62f1a0
Show file tree
Hide file tree
Showing 103 changed files with 103 additions and 103 deletions.
2 changes: 1 addition & 1 deletion plugins/source/azure/codegen/recipes/base.go
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,7 @@ func initColumns(table *codegen.TableDefinition, definition resourceDefinition)
if definition.parent != "" {
columns = append(columns, codegen.ColumnDefinition{
Name: definition.parent,
Type: schema.TypeUUID,
Type: schema.TypeString,
Resolver: `schema.ParentResourceFieldResolver("id")`,
})
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|cdn_endpoint_id|UUID|
|cdn_endpoint_id|String|
|host_name|String|
|resource_state|String|
|custom_https_provisioning_state|String|
Expand Down
2 changes: 1 addition & 1 deletion plugins/source/azure/docs/tables/azure_cdn_endpoints.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ The following tables depend on `azure_cdn_endpoints`:
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|cdn_profile_id|UUID|
|cdn_profile_id|String|
|host_name|String|
|origins|JSON|
|origin_groups|JSON|
Expand Down
2 changes: 1 addition & 1 deletion plugins/source/azure/docs/tables/azure_cdn_routes.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|cdn_endpoint_id|UUID|
|cdn_endpoint_id|String|
|custom_domains|JSON|
|origin_group|JSON|
|origin_path|String|
Expand Down
2 changes: 1 addition & 1 deletion plugins/source/azure/docs/tables/azure_cdn_rule_sets.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ The following tables depend on `azure_cdn_rule_sets`:
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|cdn_profile_id|UUID|
|cdn_profile_id|String|
|provisioning_state|String|
|deployment_status|String|
|id (PK)|String|
Expand Down
2 changes: 1 addition & 1 deletion plugins/source/azure/docs/tables/azure_cdn_rules.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|cdn_rule_set_id|UUID|
|cdn_rule_set_id|String|
|order|Int|
|conditions|JSON|
|actions|JSON|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_cdn_profiles`](azure_cdn_profiles.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|cdn_profile_id|UUID|
|cdn_profile_id|String|
|provisioning_state|String|
|deployment_status|String|
|id (PK)|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_compute_virtual_machines`](azure_compute_virtual_m
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|compute_virtual_machine_id|UUID|
|compute_virtual_machine_id|String|
|platform_update_domain|Int|
|platform_fault_domain|Int|
|computer_name|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_compute_virtual_machines`](azure_compute_virtual_m
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|compute_virtual_machine_id|UUID|
|compute_virtual_machine_id|String|
|force_update_tag|String|
|publisher|String|
|type_handler_version|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_container_registries`](azure_container_registries.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|container_registry_id|UUID|
|container_registry_id|String|
|provisioning_state|String|
|status|JSON|
|id (PK)|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_cosmosdb_accounts`](azure_cosmosdb_accounts.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|cosmosdb_account_id|UUID|
|cosmosdb_account_id|String|
|resource|JSON|
|options|JSON|
|id (PK)|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_cosmosdb_accounts`](azure_cosmosdb_accounts.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|cosmosdb_account_id|UUID|
|cosmosdb_account_id|String|
|resource|JSON|
|options|JSON|
|id (PK)|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_eventhub_namespaces`](azure_eventhub_namespaces.md
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|eventhub_namespace_id|UUID|
|eventhub_namespace_id|String|
|trusted_service_access_enabled|Bool|
|default_action|String|
|virtual_network_rules|JSON|
Expand Down
2 changes: 1 addition & 1 deletion plugins/source/azure/docs/tables/azure_keyvault_keys.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_keyvault_vaults`](azure_keyvault_vaults.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|keyvault_vault_id|UUID|
|keyvault_vault_id|String|
|kid (PK)|String|
|attributes|JSON|
|tags|JSON|
Expand Down
2 changes: 1 addition & 1 deletion plugins/source/azure/docs/tables/azure_keyvault_secrets.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_keyvault_vaults`](azure_keyvault_vaults.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|keyvault_vault_id|UUID|
|keyvault_vault_id|String|
|id (PK)|String|
|attributes|JSON|
|tags|JSON|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_logic_workflows`](azure_logic_workflows.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|logic_workflow_id|UUID|
|logic_workflow_id|String|
|storage_account_id|String|
|service_bus_rule_id|String|
|event_hub_authorization_rule_id|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_mariadb_servers`](azure_mariadb_servers.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|mariadb_server_id|UUID|
|mariadb_server_id|String|
|value|String|
|description|String|
|default_value|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_monitor_resources`](azure_monitor_resources.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|monitor_resource_id|UUID|
|monitor_resource_id|String|
|storage_account_id|String|
|service_bus_rule_id|String|
|event_hub_authorization_rule_id|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_mysql_servers`](azure_mysql_servers.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|mysql_server_id|UUID|
|mysql_server_id|String|
|value|String|
|description|String|
|default_value|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_network_watchers`](azure_network_watchers.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|network_watcher_id|UUID|
|network_watcher_id|String|
|target_resource_id|String|
|target_resource_guid|String|
|storage_id|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|network_virtual_network_gateway_id|UUID|
|network_virtual_network_gateway_id|String|
|authorization_key|String|
|virtual_network_gateway_1|JSON|
|virtual_network_gateway_2|JSON|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ The following tables depend on `azure_network_virtual_network_gateways`:
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|network_virtual_network_id|UUID|
|network_virtual_network_id|String|
|extended_location|JSON|
|ip_configurations|JSON|
|gateway_type|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_postgresql_servers`](azure_postgresql_servers.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|postgresql_server_id|UUID|
|postgresql_server_id|String|
|value|String|
|description|String|
|default_value|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_postgresql_servers`](azure_postgresql_servers.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|postgresql_server_id|UUID|
|postgresql_server_id|String|
|start_ip_address|String|
|end_ip_address|String|
|id (PK)|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **_cq_id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|servicebus_authorization_rule_id|UUID|
|servicebus_authorization_rule_id|String|
|primary_connection_string|String|
|secondary_connection_string|String|
|alias_primary_connection_string|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ The following tables depend on `azure_servicebus_authorization_rules`:
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|servicebus_topic_id|UUID|
|servicebus_topic_id|String|
|rights|StringArray|
|system_data|JSON|
|id (PK)|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ The following tables depend on `azure_servicebus_topics`:
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|servicebus_namespace_id|UUID|
|servicebus_namespace_id|String|
|size_in_bytes|Int|
|created_at|Timestamp|
|updated_at|Timestamp|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_database_id|UUID|
|sql_database_id|String|
|weekly_retention|String|
|monthly_retention|String|
|yearly_retention|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_database_id|UUID|
|sql_database_id|String|
|kind|String|
|state|String|
|storage_endpoint|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_database_id|UUID|
|sql_database_id|String|
|location|String|
|kind|String|
|state|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_database_id|UUID|
|sql_database_id|String|
|scan_id|String|
|trigger_type|String|
|state|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_database_id|UUID|
|sql_database_id|String|
|storage_container_path|String|
|storage_container_sas_key|String|
|storage_account_access_key|String|
Expand Down
2 changes: 1 addition & 1 deletion plugins/source/azure/docs/tables/azure_sql_databases.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ The following tables depend on `azure_sql_databases`:
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_server_id|UUID|
|sql_server_id|String|
|sku|JSON|
|kind|String|
|managed_by|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_sql_servers`](azure_sql_servers.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_server_id|UUID|
|sql_server_id|String|
|kind|String|
|location|String|
|subregion|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_sql_servers`](azure_sql_servers.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_server_id|UUID|
|sql_server_id|String|
|kind|String|
|location|String|
|start_ip_address|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_managed_database_id|UUID|
|sql_managed_database_id|String|
|scan_id|String|
|trigger_type|String|
|state|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The primary key for this table is **id**.
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_managed_database_id|UUID|
|sql_managed_database_id|String|
|storage_container_path|String|
|storage_container_sas_key|String|
|storage_account_access_key|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ The following tables depend on `azure_sql_managed_databases`:
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_managed_instance_id|UUID|
|sql_managed_instance_id|String|
|collation|String|
|status|String|
|creation_date|Timestamp|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_sql_managed_instances`](azure_sql_managed_instance
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_managed_instance_id|UUID|
|sql_managed_instance_id|String|
|kind|String|
|server_key_name|String|
|server_key_type|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_sql_managed_instances`](azure_sql_managed_instance
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_managed_instance_id|UUID|
|sql_managed_instance_id|String|
|storage_container_path|String|
|storage_container_sas_key|String|
|storage_account_access_key|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_sql_servers`](azure_sql_servers.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_server_id|UUID|
|sql_server_id|String|
|administrator_type|String|
|login|String|
|sid|UUID|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_sql_servers`](azure_sql_servers.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_server_id|UUID|
|sql_server_id|String|
|state|String|
|storage_endpoint|String|
|storage_account_access_key|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_sql_servers`](azure_sql_servers.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_server_id|UUID|
|sql_server_id|String|
|system_data|JSON|
|is_azure_monitor_target_enabled|Bool|
|state|String|
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This table depends on [`azure_sql_servers`](azure_sql_servers.md).
| Name | Type |
| ------------- | ------------- |
|subscription_id|String|
|sql_server_id|UUID|
|sql_server_id|String|
|state|String|
|disabled_alerts|StringArray|
|email_addresses|StringArray|
Expand Down

0 comments on commit a62f1a0

Please sign in to comment.