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

Vendor gardener@1.81.6 and implement ForceDelete #59

Merged
merged 5 commits into from
Nov 3, 2023

Conversation

acumino
Copy link
Member

@acumino acumino commented Oct 6, 2023

How to categorize this PR?

/area delivery open-source
/kind enhancement

What this PR does / why we need it:

  • Vendor gardener@1.81.0-dev
  • Implement ForceDelete method for extension
  • Add e2e test for force-delete case

Which issue(s) this PR fixes:
Part of gardener/gardener#3110

Special notes for your reviewer:

/hold
- Need to vendor gardener@1.81 when it's released.

Release note:

`registry-cache` extension now supports [Shoot Force Deletion](https://github.com/gardener/gardener/blob/v1.82.0/docs/usage/shoot_operations.md#force-deletion).

@gardener-prow gardener-prow bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. area/delivery Delivery related area/open-source Open Source (community, enablement, contributions, conferences, CNCF, etc.) related kind/enhancement Enhancement, improvement, extension cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. labels Oct 6, 2023
@gardener-prow gardener-prow bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Oct 6, 2023
@acumino acumino changed the title Vendor gardener@1.81.0-dev and implement ForceDelete Vendor gardener@1.81.0 and implement ForceDelete Oct 6, 2023
@acumino
Copy link
Member Author

acumino commented Oct 6, 2023

/unhold

@gardener-prow gardener-prow bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 6, 2023
@acumino acumino force-pushed the vendor/gardener@1.81 branch 2 times, most recently from b001823 to 6305e5b Compare October 10, 2023 09:08
Copy link
Contributor

@dimitar-kostadinov dimitar-kostadinov left a comment

Choose a reason for hiding this comment

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

Some minor nits, otherwise looks good

test/e2e/create_enable_force_delete_test.go Outdated Show resolved Hide resolved
test/e2e/create_enable_force_delete_test.go Show resolved Hide resolved
@ialidzhikov
Copy link
Member

/assign

pkg/controller/extension/actuator.go Outdated Show resolved Hide resolved
test/e2e/common.go Outdated Show resolved Hide resolved
test/e2e/create_enable_force_delete_test.go Show resolved Hide resolved
@gardener-prow gardener-prow bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 16, 2023
@gardener-prow gardener-prow bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 27, 2023
@acumino
Copy link
Member Author

acumino commented Oct 27, 2023

/hold for testing

@gardener-prow gardener-prow bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 27, 2023
@acumino acumino force-pushed the vendor/gardener@1.81 branch 2 times, most recently from 1aa2a7f to 00a7465 Compare October 27, 2023 09:49
@gardener-prow gardener-prow bot removed the cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. label Oct 27, 2023
@gardener-prow gardener-prow bot added cla: no Indicates the PR's author has not signed the cla-assistant.io CLA. cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. and removed cla: no Indicates the PR's author has not signed the cla-assistant.io CLA. labels Oct 27, 2023
@acumino
Copy link
Member Author

acumino commented Oct 27, 2023

/unhold

@gardener-prow gardener-prow bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 27, 2023
Copy link
Member

@ialidzhikov ialidzhikov left a comment

Choose a reason for hiding this comment

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

Final comments, otherwise lgtm

test/e2e/create_enable_force_delete_test.go Show resolved Hide resolved
pkg/controller/extension/actuator.go Show resolved Hide resolved
Copy link
Member

@ialidzhikov ialidzhikov left a comment

Choose a reason for hiding this comment

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

/lgtm

@gardener-prow gardener-prow bot added the lgtm Indicates that a PR is ready to be merged. label Nov 3, 2023
Copy link
Contributor

gardener-prow bot commented Nov 3, 2023

LGTM label has been added.

Git tree hash: 4f41e98416d7438e39f7701af09e84eb6495239b

Copy link
Contributor

gardener-prow bot commented Nov 3, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ialidzhikov

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gardener-prow gardener-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 3, 2023
@ialidzhikov ialidzhikov changed the title Vendor gardener@1.81.0 and implement ForceDelete Vendor gardener@1.81.6 and implement ForceDelete Nov 3, 2023
@gardener-prow gardener-prow bot merged commit 36d3ac3 into gardener:main Nov 3, 2023
6 checks passed
ialidzhikov added a commit to ialidzhikov/gardener-extension-registry-cache that referenced this pull request Nov 3, 2023
@acumino acumino deleted the vendor/gardener@1.81 branch November 3, 2023 16:29
ialidzhikov added a commit to ialidzhikov/gardener-extension-registry-cache that referenced this pull request Nov 10, 2023
gardener-prow bot pushed a commit that referenced this pull request Nov 10, 2023
…83)

* Do not deploy the `registry-configuration-cleaner` on Shoot deletion

If the Shoot is in deletion, then there is no need to clean up the registry configuration from Nodes.
The Shoot deletion flows ensures that the Worker is deleted before the Extension deletion.
Hence, there are no Nodes, no need to clean up registry configuration.

* Nit: Add empty line for better readability

A leftover from #59

* Update developer docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/delivery Delivery related area/open-source Open Source (community, enablement, contributions, conferences, CNCF, etc.) related cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. kind/enhancement Enhancement, improvement, extension lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants