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

mssql_sql_login imports have many null fields which are required settings on plan/apply. #122

Open
rocky4570 opened this issue Mar 20, 2023 · 1 comment

Comments

@rocky4570
Copy link

Am using this module with AWS RDS SQL Server and so far seems to work well.

I have setup a map of imported users and plan to manage that when users are added or removed. This would run as a pipeline rather than click-ops.

I now have a problem in that the resource definition mssql_sql_login expects password as required. Importing existing logins does not import all the details, the extract of TF state file after importing example: default language, password, default_database_id are all null.

Is there a mechanism to override the required fields or import the passwords in from the server (unlikely i guess). The import does seem to miss the default_database yet I have seen that information come out of data.mssql_sql_users method.

{
"version": 4,
"terraform_version": "1.4.2",
"serial": 17,
"lineage": "",
"outputs": {},
"resources": [
{
"mode": "managed",
"type": "mssql_sql_login",
"name": "logins",
"provider": "provider["registry.terraform.io/pgssoft/mssql"]",
"instances": [
{
"index_key": "",
"schema_version": 0,
"attributes": {
"check_password_expiration": null,
"check_password_policy": null,
"default_database_id": null,
"default_language": null,
"id": "0x",
"must_change_password": null,
"name": "",
"password": null,
"principal_id": "123"
},
"sensitive_attributes": []
}
]
}
],
"check_results": null
}

@l33tCod-er
Copy link

Is there any news on this?

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

No branches or pull requests

2 participants