Skip to content

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

Merged
rene-dekker merged 2 commits intotigera:masterfrom
rene-dekker:ev-6451
Mar 3, 2026
Merged

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

Conversation

@rene-dekker
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

@rene-dekker rene-dekker merged commit cf205b3 into tigera:master Mar 3, 2026
6 checks passed
@rene-dekker rene-dekker deleted the ev-6451 branch March 3, 2026 17:01
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