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

[bug] Add soveregin cloud endpoints and suffixes for modules (part 2) #183

Closed
davesee opened this issue May 20, 2021 · 0 comments · Fixed by #184 or #187
Closed

[bug] Add soveregin cloud endpoints and suffixes for modules (part 2) #183

davesee opened this issue May 20, 2021 · 0 comments · Fixed by #184 or #187
Labels
bug Something isn't working

Comments

@davesee
Copy link
Contributor

davesee commented May 20, 2021

Describe the bug
The MS SQL database db_permissions execution fails in non Azure Public clouds. This is because the URI suffix is hard coded into the module and is different for each cloud.

By passing the cloud information automatically down as a global setting, module authors are able to customize the behavior of their module for individual clouds when needed by using the proper variable instead of a string value.

The goal of this story is enhance sovereign cloud support by passing this value to CAF landing zones and modules as global settings. This allows modules and add-ons to conditionally load different configurations for different clouds.

The update consists of 3 repo updates:

  1. Rover retrieval of cloud variables and setting them to environment variables
  2. CAF_Solution landing zone creating a new variable namespace for each variable retrieved in step 1
  3. Module integration of cloud variables from step 2

To Reproduce
Steps to reproduce the behavior:
0. Login to a non Azure Public cloud

  1. Deploy CAF solution with mssql_database db_permissons
  2. Notice error:
Sqlcmd: Error: Microsoft ODBC Driver 17 for SQL Server : A network-related or instance-specific error has occurred while establishing a connection to SQL Server. Server is not found or not accessible. Check if instance name is correct and if SQL Server is configured to allow remote connections. For more information see SQL Server Books Online..
Error on or near line 286: Error running terraform apply; exiting with status 2001\

Expected behavior
The terraform deployment and db permission scripts should have the proper suffix and therefore connect and apply the permissions as required.

Configuration (please complete the following information):

  • Rover devcontainer latest
  • Landing zones latest
  • Azurerm modules latest

Additional context
Values can be retrieved by running 'az cloud show' in the respective cloud. This is run as part of rover and the values are extracted so they do not have to be set by the user.

@davesee davesee added the bug Something isn't working label May 20, 2021
@davesee davesee changed the title [bug] Add soveregin cloud endpoints and suffixes for modules (step 2) [bug] Add soveregin cloud endpoints and suffixes for modules (part 2) May 20, 2021
@arnaudlh arnaudlh linked a pull request May 27, 2021 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
1 participant