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.

Sign up for GitHub

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

Jump to bottom

feat(AwsVpcPeering): remote route table update strategy #1007

Merged
merged 8 commits into from
Feb 12, 2025
Merged

feat(AwsVpcPeering): remote route table update strategy #1007

merged 8 commits into from
Feb 12, 2025

Conversation

vandjelk
Copy link
Contributor

@vandjelk vandjelk commented Feb 4, 2025

Description

Changes proposed in this pull request:

  • Extends AwsVpcPeering API with RemoteRouteTableUpdateStrategy

@kyma-bot kyma-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cla: yes Indicates the PR's author has signed the CLA. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Feb 4, 2025
@vandjelk vandjelk force-pushed the peering/aws/route-table-update-strategy branch 3 times, most recently from 20de372 to 2ad286e Compare February 4, 2025 18:32
@vandjelk vandjelk marked this pull request as ready for review February 5, 2025 08:08
@vandjelk vandjelk requested a review from a team as a code owner February 5, 2025 08:08
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 5, 2025
@vandjelk vandjelk force-pushed the peering/aws/route-table-update-strategy branch 2 times, most recently from 20e57c6 to 5855ba0 Compare February 6, 2025 12:20
@vandjelk vandjelk marked this pull request as draft February 7, 2025 11:01
@kyma-bot kyma-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 7, 2025
@vandjelk vandjelk force-pushed the peering/aws/route-table-update-strategy branch 3 times, most recently from 38f4e58 to 25af65e Compare February 12, 2025 07:51
@vandjelk vandjelk marked this pull request as ready for review February 12, 2025 08:24
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 12, 2025
@vandjelk vandjelk force-pushed the peering/aws/route-table-update-strategy branch from 4d73c64 to 6974be2 Compare February 12, 2025 08:46
@vandjelk vandjelk requested a review from tmilos77 February 12, 2025 08:48
@kyma-bot kyma-bot added the lgtm Looks good to me! label Feb 12, 2025
@vandjelk vandjelk enabled auto-merge (squash) February 12, 2025 11:07
@vandjelk vandjelk merged commit 6b85af7 into kyma-project:main Feb 12, 2025
11 checks passed
@vandjelk vandjelk deleted the peering/aws/route-table-update-strategy branch February 12, 2025 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tmilos77 tmilos77 tmilos77 approved these changes

Assignees

@tmilos77 tmilos77

Labels
cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vandjelk @tmilos77 @kyma-bot