Skip to content

fix(tier cleanup): Deprecated tier cannot be deleted#4480

Open
rene-dekker wants to merge 2 commits intotigera:masterfrom
rene-dekker:ev-6451
Open

fix(tier cleanup): Deprecated tier cannot be deleted#4480
rene-dekker wants to merge 2 commits intotigera:masterfrom
rene-dekker:ev-6451

Conversation

@rene-dekker
Copy link
Member

The manager controller did not clean up deprecated policies, due to trying to delete them from the wrong namespace. As a result, the old tier (allow-tigera) can never be deleted.

Comment on lines 1490 to 1494
// allow-tigera Tier was renamed to calico-system
objs = append(objs,
networkpolicy.DeprecatedAllowTigeraNetworkPolicyObject("manager-access", installNS),
networkpolicy.DeprecatedAllowTigeraNetworkPolicyObject("default-deny", installNS),
)
Copy link
Contributor

@radixo radixo Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we keep this one? Did I do right using installNS?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Due to manager being moved to another namespace in the previous release, there can be a network policy for 2 namespaces in the old tier.

The manager controller did not clean up deprecated policies, due to trying to delete them from the wrong namespace. As a result, the old tier (allow-tigera) can never be deleted.
The manager controller did not clean up deprecated policies, due to trying to delete them from the wrong namespace. As a result, the old tier (allow-tigera) can never be deleted.
Copy link
Contributor

@radixo radixo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants