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

[gce] Ensure ressources are freed #17949

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

clinique
Copy link
Contributor

@clinique clinique commented Dec 21, 2024

Solves issue #17940
Global code review

A bit of code revamp

Signed-off-by: clinique <[email protected]>
@clinique clinique added bug An unexpected problem or unintended behavior of an add-on work in progress A PR that is not yet ready to be merged labels Dec 21, 2024
@clinique clinique self-assigned this Dec 21, 2024
Signed-off-by: Gaël L'hopital <[email protected]>
Signed-off-by: clinique <[email protected]>
Signed-off-by: clinique <[email protected]>
Signed-off-by: clinique <[email protected]>
@clinique clinique added additional testing preferred The change works for the pull request author. A test from someone else is preferred though. and removed work in progress A PR that is not yet ready to be merged labels Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
additional testing preferred The change works for the pull request author. A test from someone else is preferred though. bug An unexpected problem or unintended behavior of an add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant