Azure / azure-linux-extensions

Linux Virtual Machine Extensions for Azure
Apache License 2.0
304 stars 252 forks source link

Ensure 1P syslog config is always present in case of 1P/3P combined mode #1840

Closed simathih closed 10 months ago

simathih commented 10 months ago

when we added syslog watcher thread in 1.26 (https://github.com/Azure/azure-linux-extensions/commit/d6a32c812eb4edd25d29590d44881dc5fbfb583e), if there is mixed mode in next generation config and customer has all the following:

  1. genevaConfiguration is enabled in the next generation schema
  2. azureMonitorConfiguration is enabled in the next generation schema
  3. 3P DCR has no syslog configured and hence no marker file is generated

then, in enable(), we first deploy syslog config but they are wiped in the watcher thread because syslog is not enabled as per 3P DCR even though 1P mode needs it