Skip to content
This repository has been archived by the owner on May 3, 2022. It is now read-only.

Installation controller fails to create when a namespace is terminating #396

Merged
merged 3 commits into from
May 12, 2021

Conversation

hihilla
Copy link
Contributor

@hihilla hihilla commented May 10, 2021

This change makes the kubeclient error with cause Namespace Terminating, or a forbidden (404) error with kind namespaces (namespace not found error) retryable

@hihilla hihilla changed the title WIP: Installation controller fails to create when a namespace is terminating WIP: Installation controller fails to create anchor ConfigMap when a namespace is terminating May 10, 2021
…ng or deleted

This change makes the kubeclient error with cause `Namespace Terminating`, or a forbidden (404) error with kind `namespaces` (namespace not found error) retryable
@hihilla hihilla changed the title WIP: Installation controller fails to create anchor ConfigMap when a namespace is terminating Installation controller fails to create when a namespace is terminating May 12, 2021
@hihilla hihilla marked this pull request as ready for review May 12, 2021 10:14
ArchangelX360
ArchangelX360 previously approved these changes May 12, 2021
Namespace manager
*****************
By design, Shipper does not create namespaces in the application cluster.
In case the namespace does not exist in the application cluster, and this application cluster is selected for a *Release*,
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we specify here just for completeness that the expected namespace in the application cluster has to be of the same name?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hmm you're right

nikoloup
nikoloup previously approved these changes May 12, 2021
@hihilla hihilla merged commit 4c66fd4 into master May 12, 2021
@hihilla hihilla deleted the hilla/CRUNTIME-692 branch May 12, 2021 11:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants