-
Notifications
You must be signed in to change notification settings - Fork 0
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
[#IOPID-2784] Introduce OnProfileUpdate cosmosDB trigger #224
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: c39d123 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Jira Pull Request LinkThis Pull Request refers to the following Jira issue IOPID-2784 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some small changes
apps/io-profile-async/src/functions/__tests__/on-profile-update.test.ts
Outdated
Show resolved
Hide resolved
4300329
to
9f1e2ff
Compare
…e.test.ts Co-authored-by: Greta Quadrati <[email protected]>
3cbe737
to
12265bb
Compare
📖 Terraform Plan ('infra/resources/prod') - successTerraform Planmodule.function_lollipop_autoscale.data.azurerm_linux_function_app.this[0]: Still reading... [10s elapsed]
module.function_profile_async_autoscale.data.azurerm_linux_function_app.this[0]: Still reading... [10s elapsed]
module.function_lv_autoscale.data.azurerm_linux_function_app.this[0]: Still reading... [10s elapsed]
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
~ update in-place
Terraform will perform the following actions:
# module.function_profile_async.azurerm_linux_function_app.this will be updated in-place
~ resource "azurerm_linux_function_app" "this" {
~ app_settings = (sensitive value)
id = "/subscriptions/ec285037-c673-4f58-b594-d7c480da4e8b/resourceGroups/io-p-itn-auth-main-rg-01/providers/Microsoft.Web/sites/io-p-itn-auth-profas-func-02"
name = "io-p-itn-auth-profas-func-02"
tags = {
"CostCenter" = "TS310 - PAGAMENTI & SERVIZI"
"CreatedBy" = "Terraform"
"Environment" = "Prod"
"ManagementTeam" = "IO Auth&Identity"
"Owner" = "IO"
"Source" = "https://github.com/pagopa/io-auth-n-identity-domain/tree/main/infra/resources/prod"
}
# (31 unchanged attributes hidden)
~ sticky_settings {
~ app_setting_names = [
# (8 unchanged elements hidden)
"AzureWebJobs.MigrateServicePreferenceFromLegacy.Disabled",
+ "AzureWebJobs.OnProfileUpdate.Disabled",
]
# (1 unchanged attribute hidden)
}
# (2 unchanged blocks hidden)
}
# module.function_profile_async.azurerm_linux_function_app_slot.this[0] will be updated in-place
~ resource "azurerm_linux_function_app_slot" "this" {
~ app_settings = (sensitive value)
id = "/subscriptions/ec285037-c673-4f58-b594-d7c480da4e8b/resourceGroups/io-p-itn-auth-main-rg-01/providers/Microsoft.Web/sites/io-p-itn-auth-profas-func-02/slots/staging"
name = "staging"
tags = {
"CostCenter" = "TS310 - PAGAMENTI & SERVIZI"
"CreatedBy" = "Terraform"
"Environment" = "Prod"
"ManagementTeam" = "IO Auth&Identity"
"Owner" = "IO"
"Source" = "https://github.com/pagopa/io-auth-n-identity-domain/tree/main/infra/resources/prod"
}
# (29 unchanged attributes hidden)
# (2 unchanged blocks hidden)
}
Plan: 0 to add, 2 to change, 0 to destroy.
Warning: Deprecated Resource
with data.azurerm_app_service.app_backend_li,
on data.tf line 24, in data "azurerm_app_service" "app_backend_li":
24: data "azurerm_app_service" "app_backend_li" {
The `azurerm_app_service` data source has been superseded by the
`azurerm_linux_function_app` and `azurerm_windows_web_app` data sources.
Whilst this resource will continue to be available in the 2.x and 3.x
releases it is feature-frozen for compatibility purposes, will no longer
receive any updates and will be removed in a future major release of the
Azure Provider.
(and 5 more similar warnings elsewhere)
─────────────────────────────────────────────────────────────────────────────
Note: You didn't use the -out option to save this plan, so Terraform can't
guarantee to take exactly these actions if you run "terraform apply" now. |
|
List of Changes
Motivation and Context
Move away
OnProfileUpdate
function from io-profileHow Has This Been Tested?
unit tests, local integration tests
Screenshots (if appropriate):
Types of changes
Checklist: