Skip to content

Cleaner: Pod UID based identity check should be used when handling any LCM events #23

@Levovar

Description

@Levovar

In #5 a correction was made to the periodic cleanup handler to avoid accidentally freeing valid allocations belonging to StatefulSets.
This is all nice and dandy, but the handling of the events triggered by Pod state changes, such as Pod Delete: https://github.com/nokia/danm-utils/blob/master/pkg/cleaner/cleaner.go#L163 is effectively still Pod name + namespace based.

The code shouldb e adapted to use custom keys, including Pod UID; and use UID when validating the identities of Pods.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions