Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add finalizers #409

Closed

Conversation

KyriosGN0
Copy link

i have added finalizers as a medata element (used the same implementation that was in the application resource)
closes #391

Signed-off-by: AvivGuiser <[email protected]>
Copy link

This pr is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Jul 30, 2024
@KyriosGN0
Copy link
Author

not stale

@onematchfox
Copy link
Collaborator

Slowly trying to make my way through open PRs. Thanks for your patience.

@github-actions github-actions bot removed the Stale label Jul 31, 2024
@onematchfox
Copy link
Collaborator

@KyriosGN0, quick update - I'm afraid implementing this isn't as simple as it may seem. There is a bunch of discussion/history contained in #224 that you may want to read through. In essence though, the finalizers field is something that could well be mutated by other resources within the cluster and so tracking this state within Terraform could well lead to Terraform overwriting/deleting things it shouldn't. This is also an issue with annotations for example (see #173). At the moment I'm not sure what the correct way forward here. At the very least we would need to ensure that the provider only own and track the finalizer(s) on the given Application (or Project, Cluster, etc) in order not to overwrite/delete any finalizers that haven't been explicitly set. However, doing this may be is tricky - particularly in the case of the removal of a finalizer (that was originally added via the terraform resource).

Copy link

This pr is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Aug 23, 2024
@onematchfox
Copy link
Collaborator

Keep for now. Could be worth exploring how the Kubernetes provider handles annotations.

@github-actions github-actions bot removed the Stale label Aug 25, 2024
@KyriosGN0
Copy link
Author

@onematchfox i've read a bit the docs you sent, i'll give them another look cuz i really want to understand the issue

Copy link

github-actions bot commented Sep 9, 2024

This pr is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Sep 9, 2024
@github-actions github-actions bot closed this Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add support for finalizers in metadata.
2 participants