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

Migrate Azure APIs prior to Sep 30 2023 #23083

Closed
2 tasks done
bchess opened this issue Jan 13, 2023 · 5 comments · Fixed by #28311
Closed
2 tasks done

Migrate Azure APIs prior to Sep 30 2023 #23083

bchess opened this issue Jan 13, 2023 · 5 comments · Fixed by #28311
Assignees
Labels
good-first-issue Good starting point for new developers, which requires minimal understanding of Cilium. kind/bug This is a bug in the Cilium logic. kind/community-report This was reported by a user in the Cilium community, eg via Slack. sig/agent Cilium agent related.

Comments

@bchess
Copy link

bchess commented Jan 13, 2023

Is there an existing issue for this?

  • I have searched the existing issues

What happened?

Azure announced API versions older than 2021-08-01 will fail after 2023-09-30.

Cilium is currently using Network API 2020-11-01 and Compute API 2021-03-01, both of which will be affected. These will need to be upgraded prior to Sep 30.

Cilium Version

n/a

Kernel Version

n/a

Kubernetes Version

n/a

Sysdump

No response

Relevant log output

No response

Anything else?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@bchess bchess added kind/bug This is a bug in the Cilium logic. kind/community-report This was reported by a user in the Cilium community, eg via Slack. needs/triage This issue requires triaging to establish severity and next steps. labels Jan 13, 2023
@aanm
Copy link
Member

aanm commented Jan 16, 2023

Hey @bchess would you like to submit a PR? Thank you

@aanm aanm added good-first-issue Good starting point for new developers, which requires minimal understanding of Cilium. sig/agent Cilium agent related. and removed needs/triage This issue requires triaging to establish severity and next steps. labels Jan 16, 2023
@goverdhan07
Copy link

Hey @aanm, I would like to submit a PR

@harshitasao
Copy link

I would like to work on this.

/assign

@harshitasao
Copy link

Hey @aanm I'm new to this project and wanted to work on this issue. From what I understand, I need to follow this migration guide and do the upgrade. Please point me in the right direction if I'm wrong.

@aanm
Copy link
Member

aanm commented Sep 27, 2023

@harshitasao that is correct

christarazi added a commit to christarazi/cilium that referenced this issue Sep 27, 2023
Supersedes: cilium#23172
Fixes: cilium#23083

Signed-off-by: Goverdhan Biradar <goverdhanbiradar@gmail.com>
Signed-off-by: Chris Tarazi <chris@isovalent.com>
@aanm aanm changed the title Need to upgrade Azure APIs prior to Sep 30 2023 Migrate Azure APIs prior to Sep 30 2023 Sep 28, 2023
@aanm aanm reopened this Sep 28, 2023
christarazi added a commit that referenced this issue Sep 28, 2023
[ upstream commit 1ad3aee ]

Supersedes: #23172
Fixes: #23083

Signed-off-by: Goverdhan Biradar <goverdhanbiradar@gmail.com>
Signed-off-by: Chris Tarazi <chris@isovalent.com>
christarazi added a commit that referenced this issue Sep 28, 2023
[ upstream commit 1ad3aee ]

Supersedes: #23172
Fixes: #23083

Signed-off-by: Goverdhan Biradar <goverdhanbiradar@gmail.com>
Signed-off-by: Chris Tarazi <chris@isovalent.com>
aanm pushed a commit that referenced this issue Sep 28, 2023
[ upstream commit 1ad3aee ]

Supersedes: #23172
Fixes: #23083

Signed-off-by: Goverdhan Biradar <goverdhanbiradar@gmail.com>
Signed-off-by: Chris Tarazi <chris@isovalent.com>
aanm pushed a commit that referenced this issue Sep 28, 2023
[ upstream commit 1ad3aee ]

Supersedes: #23172
Fixes: #23083

Signed-off-by: Goverdhan Biradar <goverdhanbiradar@gmail.com>
Signed-off-by: Chris Tarazi <chris@isovalent.com>
hemanthmalla pushed a commit to DataDog/cilium that referenced this issue Sep 29, 2023
[ upstream commit 1ad3aee ]

Supersedes: cilium#23172
Fixes: cilium#23083

Signed-off-by: Goverdhan Biradar <goverdhanbiradar@gmail.com>
Signed-off-by: Chris Tarazi <chris@isovalent.com>
margamanterola pushed a commit to margamanterola/cilium that referenced this issue Sep 29, 2023
[ upstream commit 1ad3aee ]

Supersedes: cilium#23172
Fixes: cilium#23083

Signed-off-by: Goverdhan Biradar <goverdhanbiradar@gmail.com>
Signed-off-by: Chris Tarazi <chris@isovalent.com>
Signed-off-by: Marga Manterola <marga@isovalent.com>
aanm pushed a commit that referenced this issue Sep 29, 2023
[ upstream commit 1ad3aee ]

Supersedes: #23172
Fixes: #23083

Signed-off-by: Goverdhan Biradar <goverdhanbiradar@gmail.com>
Signed-off-by: Chris Tarazi <chris@isovalent.com>
Signed-off-by: Marga Manterola <marga@isovalent.com>
@giorio94 giorio94 closed this as completed Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good-first-issue Good starting point for new developers, which requires minimal understanding of Cilium. kind/bug This is a bug in the Cilium logic. kind/community-report This was reported by a user in the Cilium community, eg via Slack. sig/agent Cilium agent related.
Projects
Status: Done
4 participants