Skip to content

Commit 0c1eb75

Browse files
authored
Merge branch 'main' into frank.spano-kube-actions-par
2 parents 6d75363 + 00b74a9 commit 0c1eb75

3 files changed

Lines changed: 37 additions & 17 deletions

File tree

.github/CODEOWNERS

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -272,6 +272,7 @@
272272
/cmd/agent/dist/conf.d/oracle.d/ @DataDog/database-monitoring
273273
/cmd/agent/dist/conf.d/oracle-dbm.d/ @DataDog/database-monitoring
274274
/cmd/agent/dist/conf.d/network_path.d/ @DataDog/network-path
275+
/cmd/agent/dist/conf.d/network_config_management.d/ @DataDog/ndm-integrations
275276
/cmd/agent/dist/conf.d/orchestrator_ecs.d/ @DataDog/ecs-experiences
276277
/cmd/agent/dist/conf.d/sbom.d/ @DataDog/agent-security @DataDog/container-integrations
277278
/cmd/agent/dist/conf.d/snmp.d/ @DataDog/network-device-monitoring-core

pkg/fleet/installer/packages/datadog_agent_linux.go

Lines changed: 26 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,7 @@ func installFilesystem(ctx HookContext) (err error) {
219219

220220
// 7. Stop and remove legacy procmgr unit names so only datadog-agent-procmgr.service runs dd-procmgrd
221221
if err = retireLegacyProcmgrUnits(ctx); err != nil {
222-
return fmt.Errorf("failed to retire legacy procmgr units: %w", err)
222+
log.Warnf("failed to retire legacy procmgr units: %v", err)
223223
}
224224
return nil
225225
}
@@ -228,29 +228,48 @@ func installFilesystem(ctx HookContext) (err error) {
228228
// A host that upgraded from datadog-agent-procmgrd.service could otherwise run two dd-procmgrd
229229
// instances (socket and processes.d conflicts).
230230
func retireLegacyProcmgrUnits(ctx HookContext) error {
231-
switch service.GetServiceManagerType() {
231+
serviceManager := service.GetServiceManagerType()
232+
log.Debugf("retire legacy procmgr units: service manager %q", serviceManager)
233+
switch serviceManager {
232234
case service.SystemdType:
233235
default:
236+
log.Debugf("retire legacy procmgr units: skipping, systemd not in use")
234237
return nil
235238
}
236239
running, err := systemd.IsRunning()
237240
if err != nil {
238-
return err
241+
log.Debugf("retire legacy procmgr units: check systemd running failed: %v", err)
242+
return fmt.Errorf("check systemd running: %w", err)
239243
}
244+
log.Debugf("retire legacy procmgr units: systemd running=%t", running)
240245
if running {
246+
log.Debugf("retire legacy procmgr units: stopping units %v", legacyProcmgrUnitNames)
241247
if err := systemd.StopUnits(ctx, legacyProcmgrUnitNames...); err != nil {
242-
return err
248+
log.Debugf("retire legacy procmgr units: stop failed: %v", err)
249+
return fmt.Errorf("stop legacy procmgr units: %w", err)
243250
}
251+
log.Debugf("retire legacy procmgr units: disabling units %v", legacyProcmgrUnitNames)
244252
if err := systemd.DisableUnits(ctx, legacyProcmgrUnitNames...); err != nil {
245-
return err
253+
log.Debugf("retire legacy procmgr units: disable failed: %v", err)
254+
return fmt.Errorf("disable legacy procmgr units: %w", err)
246255
}
256+
} else {
257+
log.Debugf("retire legacy procmgr units: skipping stop/disable, systemd not running")
247258
}
248259
for _, unitsPath := range systemdUnitInstallPaths {
260+
log.Debugf("retire legacy procmgr units: removing unit files from %q", unitsPath)
249261
if err := legacyProcmgrUnitPaths.EnsureAbsent(ctx, unitsPath); err != nil {
250-
return err
262+
log.Debugf("retire legacy procmgr units: remove unit files from %q failed: %v", unitsPath, err)
263+
return fmt.Errorf("remove legacy procmgr unit files from %s: %w", unitsPath, err)
251264
}
252265
}
253-
return systemd.Reload(ctx)
266+
log.Debugf("retire legacy procmgr units: reloading systemd")
267+
if err := systemd.Reload(ctx); err != nil {
268+
log.Debugf("retire legacy procmgr units: reload failed: %v", err)
269+
return fmt.Errorf("reload systemd after retiring legacy procmgr units: %w", err)
270+
}
271+
log.Debugf("retire legacy procmgr units: done")
272+
return nil
254273
}
255274

256275
const parExecutorProcmgrConfigName = "datadog-agent-action-executor.yaml"

test/static/static_quality_gates.yml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
static_quality_gate_agent_deb_amd64:
2-
max_on_disk_size: 759.01 MiB
2+
max_on_disk_size: 759.22 MiB
33
max_on_wire_size: 181.38 MiB
44
static_quality_gate_agent_deb_amd64_fips:
55
max_on_disk_size: 710.52 MiB
@@ -8,43 +8,43 @@ static_quality_gate_agent_heroku_amd64:
88
max_on_disk_size: 315.23 MiB
99
max_on_wire_size: 80.56 MiB
1010
static_quality_gate_agent_msi:
11-
max_on_disk_size: 657.0 MiB
11+
max_on_disk_size: 657.29 MiB
1212
max_on_wire_size: 157.89 MiB
1313
static_quality_gate_agent_rpm_amd64:
14-
max_on_disk_size: 758.98 MiB
14+
max_on_disk_size: 759.19 MiB
1515
max_on_wire_size: 184.16 MiB
1616
static_quality_gate_agent_rpm_amd64_fips:
1717
max_on_disk_size: 710.52 MiB
1818
max_on_wire_size: 175.6 MiB
1919
static_quality_gate_agent_rpm_arm64:
20-
max_on_disk_size: 730.38 MiB
20+
max_on_disk_size: 730.63 MiB
2121
max_on_wire_size: 165.03 MiB
2222
static_quality_gate_agent_rpm_arm64_fips:
2323
max_on_disk_size: 688.91 MiB
2424
max_on_wire_size: 158.68 MiB
2525
static_quality_gate_agent_suse_amd64:
26-
max_on_disk_size: 758.98 MiB
26+
max_on_disk_size: 759.19 MiB
2727
max_on_wire_size: 184.13 MiB
2828
static_quality_gate_agent_suse_amd64_fips:
2929
max_on_disk_size: 710.52 MiB
3030
max_on_wire_size: 175.74 MiB
3131
static_quality_gate_agent_suse_arm64:
32-
max_on_disk_size: 730.38 MiB
32+
max_on_disk_size: 730.63 MiB
3333
max_on_wire_size: 164.99 MiB
3434
static_quality_gate_agent_suse_arm64_fips:
3535
max_on_disk_size: 688.91 MiB
3636
max_on_wire_size: 158.67 MiB
3737
static_quality_gate_docker_agent_amd64:
38-
max_on_disk_size: 814.6 MiB
38+
max_on_disk_size: 814.81 MiB
3939
max_on_wire_size: 275.7 MiB
4040
static_quality_gate_docker_agent_arm64:
41-
max_on_disk_size: 815.75 MiB
41+
max_on_disk_size: 815.96 MiB
4242
max_on_wire_size: 263.67 MiB
4343
static_quality_gate_docker_agent_jmx_amd64:
44-
max_on_disk_size: 1005.36 MiB
44+
max_on_disk_size: 1005.57 MiB
4545
max_on_wire_size: 344.32 MiB
4646
static_quality_gate_docker_agent_jmx_arm64:
47-
max_on_disk_size: 995.43 MiB
47+
max_on_disk_size: 995.64 MiB
4848
max_on_wire_size: 328.27 MiB
4949
static_quality_gate_docker_cluster_agent_amd64:
5050
max_on_disk_size: 211.25 MiB

0 commit comments

Comments
 (0)