Skip to content

Commit

Permalink
fix: [PAYMCLOUD-109] poc GitHub runner on dev ops lab (#362)
Browse files Browse the repository at this point in the history
* Add Key Vault secret retrieval for GitHub PAT

This commit integrates the retrieval of GitHub Personal Access Token (PAT) from Azure Key Vault in the Terraform configuration. It removes the hardcoded PAT from the environment variables and injects it securely from the Key Vault secret. This enhances security by avoiding hardcoding sensitive information in the codebase.

* Add GitHub PAT to environment variables

Introduced GitHub PAT in the container environment variables to securely access repositories. Updated `main.tf` to correctly reference the existing Key Vault secret for the GitHub PAT. Removed redundant secret configuration for cleaner code.

* Refactor GitHub PAT configuration for enhanced security

Moved GitHub PAT from plain text in locals.tf to secure secret management in main.tf. This change improves security by using the secret_name attribute for handling sensitive credentials.

* Format Markdown table to use consistent line breaks

Updated the Markdown table in the README to use consistent line breaks with `<br/>`. This change ensures better readability and maintainability of the table entries.

* Fix Markdown formatting in multiple README files

Replaced all instances of "/>" with ">" in code blocks for consistency. This change improves readability and maintains a uniform coding style across the documentation.

* Align variable and table formatting

Adjusted spacing to align variables in `variables.tf` and `main.tf` for better readability. Reformatted the README.md input table to ensure consistency and clarity.
  • Loading branch information
ffppa authored Oct 15, 2024
1 parent 15bbe5e commit e557087
Show file tree
Hide file tree
Showing 44 changed files with 139 additions and 132 deletions.
20 changes: 10 additions & 10 deletions api_management/README.md

Large diffs are not rendered by default.

6 changes: 3 additions & 3 deletions api_management_api/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -83,23 +83,23 @@ No modules.
| Name | Description | Type | Default | Required |
|------|-------------|------|---------|:--------:|
| <a name="input_api_management_name"></a> [api\_management\_name](#input\_api\_management\_name) | n/a | `string` | n/a | yes |
| <a name="input_api_operation_policies"></a> [api\_operation\_policies](#input\_api\_operation\_policies) | List of api policy for given operation. | <pre>list(object({<br/> operation_id = string<br/> xml_content = string<br/> }<br/> ))</pre> | `[]` | no |
| <a name="input_api_operation_policies"></a> [api\_operation\_policies](#input\_api\_operation\_policies) | List of api policy for given operation. | <pre>list(object({<br> operation_id = string<br> xml_content = string<br> }<br> ))</pre> | `[]` | no |
| <a name="input_api_type"></a> [api\_type](#input\_api\_type) | (Optional) Type of API. Possible values are graphql, http, soap, and websocket. Defaults to http. | `string` | `"http"` | no |
| <a name="input_api_version"></a> [api\_version](#input\_api\_version) | The Version number of this API, if this API is versioned. | `string` | `null` | no |
| <a name="input_content_format"></a> [content\_format](#input\_content\_format) | The format of the content from which the API Definition should be imported. | `string` | `"swagger-json"` | no |
| <a name="input_content_value"></a> [content\_value](#input\_content\_value) | The Content from which the API Definition should be imported. | `string` | n/a | yes |
| <a name="input_description"></a> [description](#input\_description) | n/a | `string` | n/a | yes |
| <a name="input_display_name"></a> [display\_name](#input\_display\_name) | n/a | `string` | n/a | yes |
| <a name="input_name"></a> [name](#input\_name) | n/a | `string` | n/a | yes |
| <a name="input_oauth2_authorization"></a> [oauth2\_authorization](#input\_oauth2\_authorization) | n/a | <pre>object({<br/> authorization_server_name = string<br/> }<br/> )</pre> | <pre>{<br/> "authorization_server_name": null<br/>}</pre> | no |
| <a name="input_oauth2_authorization"></a> [oauth2\_authorization](#input\_oauth2\_authorization) | n/a | <pre>object({<br> authorization_server_name = string<br> }<br> )</pre> | <pre>{<br> "authorization_server_name": null<br>}</pre> | no |
| <a name="input_path"></a> [path](#input\_path) | n/a | `string` | n/a | yes |
| <a name="input_product_ids"></a> [product\_ids](#input\_product\_ids) | n/a | `list(string)` | `[]` | no |
| <a name="input_protocols"></a> [protocols](#input\_protocols) | n/a | `list(string)` | n/a | yes |
| <a name="input_resource_group_name"></a> [resource\_group\_name](#input\_resource\_group\_name) | n/a | `string` | n/a | yes |
| <a name="input_revision"></a> [revision](#input\_revision) | n/a | `string` | `"1"` | no |
| <a name="input_revision_description"></a> [revision\_description](#input\_revision\_description) | n/a | `string` | `null` | no |
| <a name="input_service_url"></a> [service\_url](#input\_service\_url) | n/a | `string` | n/a | yes |
| <a name="input_subscription_key_names"></a> [subscription\_key\_names](#input\_subscription\_key\_names) | Override the default name of the header and query string containing the subscription key header | <pre>object({<br/> header = string<br/> query = string<br/> })</pre> | `null` | no |
| <a name="input_subscription_key_names"></a> [subscription\_key\_names](#input\_subscription\_key\_names) | Override the default name of the header and query string containing the subscription key header | <pre>object({<br> header = string<br> query = string<br> })</pre> | `null` | no |
| <a name="input_subscription_required"></a> [subscription\_required](#input\_subscription\_required) | Should this API require a subscription key? | `bool` | `false` | no |
| <a name="input_version_set_id"></a> [version\_set\_id](#input\_version\_set\_id) | The ID of the Version Set which this API is associated with. | `string` | `null` | no |
| <a name="input_xml_content"></a> [xml\_content](#input\_xml\_content) | The XML Content for this Policy as a string | `string` | `null` | no |
Expand Down
22 changes: 11 additions & 11 deletions app_gateway/README.md

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion app_service/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ No modules.
| <a name="input_app_command_line"></a> [app\_command\_line](#input\_app\_command\_line) | (Optional) App command line to launch, e.g. /sbin/myserver -b 0.0.0.0. | `string` | `null` | no |
| <a name="input_app_settings"></a> [app\_settings](#input\_app\_settings) | n/a | `map(string)` | `{}` | no |
| <a name="input_auto_heal_enabled"></a> [auto\_heal\_enabled](#input\_auto\_heal\_enabled) | (Optional) True to enable the auto heal on the app service | `bool` | `false` | no |
| <a name="input_auto_heal_settings"></a> [auto\_heal\_settings](#input\_auto\_heal\_settings) | (Optional) Auto heal settings | <pre>object({<br/> startup_time = string<br/> slow_requests_count = number<br/> slow_requests_interval = string<br/> slow_requests_time = string<br/> })</pre> | `null` | no |
| <a name="input_auto_heal_settings"></a> [auto\_heal\_settings](#input\_auto\_heal\_settings) | (Optional) Auto heal settings | <pre>object({<br> startup_time = string<br> slow_requests_count = number<br> slow_requests_interval = string<br> slow_requests_time = string<br> })</pre> | `null` | no |
| <a name="input_client_affinity_enabled"></a> [client\_affinity\_enabled](#input\_client\_affinity\_enabled) | (Optional) Should the App Service send session affinity cookies, which route client requests in the same session to the same instance? Defaults to false. | `bool` | `false` | no |
| <a name="input_client_cert_enabled"></a> [client\_cert\_enabled](#input\_client\_cert\_enabled) | (Optional) Does the App Service require client certificates for incoming requests? Defaults to false. | `bool` | `false` | no |
| <a name="input_docker_image"></a> [docker\_image](#input\_docker\_image) | Framework choice | `string` | `null` | no |
Expand Down
2 changes: 1 addition & 1 deletion app_service_slot/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ No modules.
| <a name="input_app_service_name"></a> [app\_service\_name](#input\_app\_service\_name) | (Required) The name of the App Service within which to create the App Service Slot. Changing this forces a new resource to be created. | `string` | n/a | yes |
| <a name="input_app_settings"></a> [app\_settings](#input\_app\_settings) | n/a | `map(string)` | `{}` | no |
| <a name="input_auto_heal_enabled"></a> [auto\_heal\_enabled](#input\_auto\_heal\_enabled) | (Optional) True to enable the auto heal on the app service | `bool` | `false` | no |
| <a name="input_auto_heal_settings"></a> [auto\_heal\_settings](#input\_auto\_heal\_settings) | (Optional) Auto heal settings | <pre>object({<br/> startup_time = string<br/> slow_requests_count = number<br/> slow_requests_interval = string<br/> slow_requests_time = string<br/> })</pre> | `null` | no |
| <a name="input_auto_heal_settings"></a> [auto\_heal\_settings](#input\_auto\_heal\_settings) | (Optional) Auto heal settings | <pre>object({<br> startup_time = string<br> slow_requests_count = number<br> slow_requests_interval = string<br> slow_requests_time = string<br> })</pre> | `null` | no |
| <a name="input_client_affinity_enabled"></a> [client\_affinity\_enabled](#input\_client\_affinity\_enabled) | (Optional) Should the App Service send session affinity cookies, which route client requests in the same session to the same instance? Defaults to false. | `bool` | `false` | no |
| <a name="input_client_certificate_enabled"></a> [client\_certificate\_enabled](#input\_client\_certificate\_enabled) | Should the function app use Client Certificates | `bool` | `false` | no |
| <a name="input_docker_image"></a> [docker\_image](#input\_docker\_image) | Framework choice | `string` | `null` | no |
Expand Down
2 changes: 1 addition & 1 deletion application_insights_web_test_preview/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ No modules.

| Name | Description | Type | Default | Required |
|------|-------------|------|---------|:--------:|
| <a name="input_actions"></a> [actions](#input\_actions) | n/a | <pre>list(object({<br/> action_group_id = string<br/> }))</pre> | n/a | yes |
| <a name="input_actions"></a> [actions](#input\_actions) | n/a | <pre>list(object({<br> action_group_id = string<br> }))</pre> | n/a | yes |
| <a name="input_alert_description"></a> [alert\_description](#input\_alert\_description) | Web Availability Alert description | `string` | `"Web availability check alert triggered when it fails."` | no |
| <a name="input_application_insight_id"></a> [application\_insight\_id](#input\_application\_insight\_id) | Application insight id. | `string` | n/a | yes |
| <a name="input_application_insight_name"></a> [application\_insight\_name](#input\_application\_insight\_name) | Application insight instance name. | `string` | n/a | yes |
Expand Down
2 changes: 1 addition & 1 deletion azure_devops_agent/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ No modules.
| <a name="input_admin_password"></a> [admin\_password](#input\_admin\_password) | (Optional) The Password which should be used for the local-administrator on this Virtual Machine. Changing this forces a new resource to be created. will be stored in the raw state as plain-text | `string` | `null` | no |
| <a name="input_authentication_type"></a> [authentication\_type](#input\_authentication\_type) | (Required) Type of authentication to use with the VM. Defaults to password for Windows and SSH public key for Linux. all enables both ssh and password authentication. | `string` | `"SSH"` | no |
| <a name="input_encryption_set_id"></a> [encryption\_set\_id](#input\_encryption\_set\_id) | (Optional) An existing encryption set | `string` | `null` | no |
| <a name="input_image_reference"></a> [image\_reference](#input\_image\_reference) | (Optional) A source\_image\_reference block as defined below. | <pre>object({<br/> publisher = string<br/> offer = string<br/> sku = string<br/> version = string<br/> })</pre> | <pre>{<br/> "offer": "0001-com-ubuntu-server-jammy",<br/> "publisher": "Canonical",<br/> "sku": "22_04-lts-gen2",<br/> "version": "latest"<br/>}</pre> | no |
| <a name="input_image_reference"></a> [image\_reference](#input\_image\_reference) | (Optional) A source\_image\_reference block as defined below. | <pre>object({<br> publisher = string<br> offer = string<br> sku = string<br> version = string<br> })</pre> | <pre>{<br> "offer": "0001-com-ubuntu-server-jammy",<br> "publisher": "Canonical",<br> "sku": "22_04-lts-gen2",<br> "version": "latest"<br>}</pre> | no |
| <a name="input_image_resource_group_name"></a> [image\_resource\_group\_name](#input\_image\_resource\_group\_name) | (Optional) Resource group name where to find the vm image used for azdo vms. If not defined, 'resource\_group\_name' will be used | `string` | `null` | no |
| <a name="input_image_type"></a> [image\_type](#input\_image\_type) | (Required) Defines the source image to be used, whether 'custom' or 'standard'. `custom` requires `source_image_name` to be defined, `standard` requires `image_reference` | `string` | `"custom"` | no |
| <a name="input_location"></a> [location](#input\_location) | (Optional) Specifies the supported Azure location where the resource exists. Changing this forces a new resource to be created. | `string` | `"westeurope"` | no |
Expand Down
Loading

0 comments on commit e557087

Please sign in to comment.