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

Provisioner does not update Azure specific configuration correctly #3037

Open
koala7659 opened this issue Aug 31, 2023 · 5 comments
Open

Provisioner does not update Azure specific configuration correctly #3037

koala7659 opened this issue Aug 31, 2023 · 5 comments
Labels
area/control-plane Related to all activities around Kyma Control Plane kind/bug Categorizes issue or PR as related to a bug.
Milestone

Comments

@koala7659
Copy link
Contributor

koala7659 commented Aug 31, 2023

Description

The code for Azure NAT infra change to enable NAT gateway for existing runtimes is invalid and not used, but in some circonstances may cause problems to some migrated clusters:

https://github.com/kyma-project/control-plane/blob/main/components/provisioner/internal/model/gardener_config.go#L452-L489

Expected result

Remove obsolete code.

@koala7659 koala7659 added the area/control-plane Related to all activities around Kyma Control Plane label Aug 31, 2023
@koala7659 koala7659 added this to the Future milestone Aug 31, 2023
@koala7659 koala7659 self-assigned this Aug 31, 2023
@koala7659
Copy link
Contributor Author

Quote from Slack thread where we analysed the code:

Basically there is nothing wrong with the code, it’s just for many runtimes the VNet CIDR was expanded during the HA migration and this was not reflected back in provisioner DB. This resulted in an undefined behaviour when executing that specific part of the code using stale input from DB vs changed infra config on shoot. Since the particular code was developed for NAT gateway enablement of existing SKRs which was never used and won’t be needed in the future, it is safe to remove this code to avoid future problems

@koala7659
Copy link
Contributor Author

koala7659 commented Sep 5, 2023

More information can be found at control-plane issue 4387 in some internal Github

@kyma-bot
Copy link
Contributor

kyma-bot commented Nov 4, 2023

This issue or PR has been automatically marked as stale due to the lack of recent activity.
Thank you for your contributions.

This bot triages issues and PRs according to the following rules:

  • After 60d of inactivity, lifecycle/stale is applied
  • After 7d of inactivity since lifecycle/stale was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Close this issue or PR with /close

If you think that I work incorrectly, kindly raise an issue with the problem.

/lifecycle stale

@kyma-bot kyma-bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Nov 4, 2023
@kyma-bot
Copy link
Contributor

This issue or PR has been automatically closed due to the lack of activity.
Thank you for your contributions.

This bot triages issues and PRs according to the following rules:

  • After 60d of inactivity, lifecycle/stale is applied
  • After 7d of inactivity since lifecycle/stale was applied, the issue is closed

You can:

  • Reopen this issue or PR with /reopen
  • Mark this issue or PR as fresh with /remove-lifecycle stale

If you think that I work incorrectly, kindly raise an issue with the problem.

/close

@kyma-bot
Copy link
Contributor

@kyma-bot: Closing this issue.

In response to this:

This issue or PR has been automatically closed due to the lack of activity.
Thank you for your contributions.

This bot triages issues and PRs according to the following rules:

  • After 60d of inactivity, lifecycle/stale is applied
  • After 7d of inactivity since lifecycle/stale was applied, the issue is closed

You can:

  • Reopen this issue or PR with /reopen
  • Mark this issue or PR as fresh with /remove-lifecycle stale

If you think that I work incorrectly, kindly raise an issue with the problem.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@tobiscr tobiscr added kind/bug Categorizes issue or PR as related to a bug. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Dec 28, 2023
@tobiscr tobiscr reopened this Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/control-plane Related to all activities around Kyma Control Plane kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

3 participants