bernardodemarco commented on PR #10790:
URL: https://github.com/apache/cloudstack/pull/10790#issuecomment-3473608551
@DaanHoogland, @weizhouapache, @rosi-shapeblue, done (see
2d7e0c2a9610b783e16b719a903b002e645e87f9).
With this new approach, no UI/integration tests changes are required and we
do not need to worry about backward compatibility. Based on some new test cases
that I've just performed, everything seems to be working great, both via UI and
CloudMonkey:
```
(lab) 🐱 > update configuration name="alert.email.addresses" value="e1,e2,e3"
{
"configuration": {
"category": "Alert",
"component": "management-server",
"description": "Comma separated list of email addresses which are going
to receive alert emails.",
"displaytext": "Alert email addresses",
"group": "Management Server",
"isdynamic": false,
"name": "alert.email.addresses",
"subgroup": "Alerts",
"type": "CSV",
"value": "e1,e2,e3"
}
}
(lab) 🐱 > update configuration name="alert.email.addresses" value=
{
"configuration": {
"category": "Alert",
"component": "management-server",
"description": "Comma separated list of email addresses which are going
to receive alert emails.",
"displaytext": "Alert email addresses",
"group": "Management Server",
"isdynamic": false,
"name": "alert.email.addresses",
"subgroup": "Alerts",
"type": "CSV",
"value": ""
}
}
(lab) 🐱 > update configuration name="alert.email.addresses"
🙈 Error: (HTTP 431, error code 4350) A value for the [alert.email.addresses]
configuration must be informed.
```
Can we rerun the integration tests?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
