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

Connection options for Metadata Database ℹ️ #11

Merged
merged 1 commit into from
Nov 22, 2024
Merged

Connection options for Metadata Database ℹ️ #11

merged 1 commit into from
Nov 22, 2024

Conversation

alismx
Copy link
Collaborator

@alismx alismx commented Nov 21, 2024

DEVOPS PULL REQUEST

Related Issue

Changes Proposed

  • Added a new Terraform file _check.tf to ensure proper configuration of the database_data_non_integrated_viewer condition.
  • Updated _local.tf to include new local variables for database_data and modified the environment variables accordingly.
  • Introduced new variables postgres_database_data and sqlserver_database_data in _variable.tf to handle database configuration.
  • Removed the variable non_integrated_viewer from _variable.tf.
  • Updated the README.md file to include new requirements, providers, modules, resources, inputs, and outputs.
  • Added a new script tfutil.sh to format Terraform files, generate Terraform docs, and run tflint.

@alismx alismx force-pushed the alis/10 branch 3 times, most recently from 75ebe4d to 8b9d5e4 Compare November 21, 2024 23:05
@alismx alismx changed the title setup postgres and sqlserver data variable for connecting to a metada… Connection options for Metadata Database ℹ️ Nov 21, 2024
@alismx alismx marked this pull request as ready for review November 21, 2024 23:09
@alismx alismx self-assigned this Nov 21, 2024
Copy link
Collaborator

@rin-skylight rin-skylight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@alismx alismx merged commit b9dca8a into main Nov 22, 2024
2 checks passed
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

Successfully merging this pull request may close these issues.

Setup database connection options
2 participants