gardener / gardener-extension-provider-aws

Gardener extension controller for the AWS cloud provider (https://aws.amazon.com).
https://gardener.cloud
Apache License 2.0
18 stars 96 forks source link

Fix issues with the AWS flow after last update #1038

Closed kon-angelo closed 2 weeks ago

kon-angelo commented 2 weeks ago

How to categorize this PR?

/area control-plane /kind bug /platform aws

What this PR does / why we need it:

Which issue(s) this PR fixes: Fixes #

Special notes for your reviewer:

Release note:

Correctly register infrastructure webhook with the controllerruntime manager
Fix an issue where the infrastructure state was not properly transformed to the provider status. 
kon-angelo commented 2 weeks ago

/test

testmachinery[bot] commented 2 weeks ago

Testrun: e2e-99xrf Workflow: e2e-99xrf-wf Phase: Succeeded

+---------------------+-----------------------------+-----------+----------+
|        NAME         |            STEP             |   PHASE   | DURATION |
+---------------------+-----------------------------+-----------+----------+
| dnsrecord-test      | dnsrecord-test              | Succeeded | 5m37s    |
| bastion-test        | bastion-test                | Succeeded | 4m9s     |
| infrastructure-test | infrastructure-test-flow    | Succeeded | 18m15s   |
| infrastructure-test | infrastructure-test-migrate | Succeeded | 21m0s    |
| infrastructure-test | infrastructure-test-recover | Succeeded | 18m35s   |
| infrastructure-test | infrastructure-test-tf      | Succeeded | 25m21s   |
+---------------------+-----------------------------+-----------+----------+

kon-angelo commented 2 weeks ago

/test

testmachinery[bot] commented 2 weeks ago

Testrun: e2e-hxvth Workflow: e2e-hxvth-wf Phase: Succeeded

+---------------------+-----------------------------+-----------+----------+
|        NAME         |            STEP             |   PHASE   | DURATION |
+---------------------+-----------------------------+-----------+----------+
| bastion-test        | bastion-test                | Succeeded | 8m37s    |
| dnsrecord-test      | dnsrecord-test              | Succeeded | 5m36s    |
| infrastructure-test | infrastructure-test-tf      | Succeeded | 27m0s    |
| infrastructure-test | infrastructure-test-flow    | Succeeded | 18m32s   |
| infrastructure-test | infrastructure-test-migrate | Succeeded | 21m35s   |
| infrastructure-test | infrastructure-test-recover | Succeeded | 18m57s   |
+---------------------+-----------------------------+-----------+----------+