Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Revisit Pod-Template in Keycloak CR #10503

@vmuzikar

Description

@vmuzikar

This is a follow-up from #10098.

The current approach has several severe downsides:

  • It is done by explicitly merging every field that is very error-prone and hard to validate (a field could be easily missed and/or wrongly merged).
  • The test coverage is incomplete, doesn't cover all fields.

To consider this issue resolved:

  • Increase test coverage to include every field.
  • If possible, merge the fields in a generic way (probably by using reflections).
  • Consider removing Pod-Templates completely if not required anymore.

Metadata

Metadata

Assignees

No one assigned

    Labels

    area/operatorKeycloak.X Operatorkind/enhancementCategorizes a PR related to an enhancement

    Type

    No type

    Projects

    Status

    Done ✅

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions