Skip to content

Standardize ACK controller logs - FieldExport format cleanup #2412

@a-hilaly

Description

@a-hilaly

While reviewing log formats in kro#462, we noticed that ACK logs are already quite close to being standardized — using structured JSON logs with useful fields like controller, controllerGroup, and controllerKind.

However, there are some minor inconsistencies, particularly with FieldExport log entries, where values like the controller name may include embedded version/kind info (e.g "controller": "field-export.ec2.services.k8s.aws/v1alpha1, Kind=Subnet"). These could be cleaned up for better parsing and consistency across controllers

AI: we should normalize FieldExport logs to use the same structure as other controllers and avoid embedding version/kind info in the controller field.

I had a quick chat with Matthew from Azure ASO team, and we agreed that all of our controllers (ACK and ASO) should follow the same recommended format - like we're doing in KRO 🚀

Metadata

Metadata

Assignees

No one assigned

    Labels

    area/runtimeIssues or PRs as related to controller runtime, common reconciliation logic, etckind/featureCategorizes issue or PR as related to a new feature.lifecycle/rottenDenotes an issue or PR that has aged beyond stale and will be auto-closed.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions