perf-tests icon indicating copy to clipboard operation
perf-tests copied to clipboard

Rename AKS provider to Azure

Open Jont828 opened this issue 1 year ago • 13 comments

/kind feature

What type of PR is this?

What this PR does / why we need it: Rename the AKS provider to Azure since it works with non-AKS Azure clusters (such as CAPZ) as well.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Jont828 avatar Sep 05 '24 22:09 Jont828

cc @JohnRusk

Do you have any thoughts on this change?

jackfrancis avatar Sep 10 '24 18:09 jackfrancis

I think the stated rationale makes sense.

The only risk I can think of would be if in fact this statement wasn't always correct: "since it works with non-AKS Azure clusters (such as CAPZ) as well". I.e. if in fact there are expected to be any kind of Azure cluster that it does not support - is that a problem? E.g. does it create an implied mainteance burden to modify this so that it does support them?

But I think my concern there is very minor, and unlikely to any reason to reject the PR. So, if you're happy with it as it stands, so am I @jackfrancis .

JohnRusk avatar Sep 13 '24 03:09 JohnRusk

The only risk I can think of would be if in fact this statement wasn't always correct: "since it works with non-AKS Azure clusters (such as CAPZ) as well". I.e. if in fact there are expected to be any kind of Azure cluster that it does not support - is that a problem? E.g. does it create an implied mainteance burden to modify this so that it does support them?

I think that is a valid point, and I would argue that if we discover this to be true in the future (novel Kubernetes + Azure infra use case) that the provider doesn't support, we want to support it. I'm committed to that level of support.

jackfrancis avatar Sep 16 '24 19:09 jackfrancis

Sounds good to me @jackfrancis

JohnRusk avatar Sep 17 '24 01:09 JohnRusk

lgtm, but needs a rebase.

mboersma avatar Sep 20 '24 17:09 mboersma

@Jont828 this needs to be rebased.

mboersma avatar Sep 25 '24 14:09 mboersma

@mboersma Done!

Jont828 avatar Oct 02 '24 20:10 Jont828

@mboersma Just rebased it, shouldn't have changed anything.

Jont828 avatar Oct 04 '24 20:10 Jont828

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Jont828, mboersma Once this PR has been reviewed and has the lgtm label, please assign wojtek-t for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

k8s-ci-robot avatar Oct 04 '24 21:10 k8s-ci-robot

PR needs rebase.

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-sigs/prow repository.

k8s-ci-robot avatar Nov 06 '24 19:11 k8s-ci-robot

The Kubernetes project currently lacks enough contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle stale
  • Close this PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Feb 04 '25 20:02 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle rotten
  • Close this PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle rotten

k8s-triage-robot avatar Mar 06 '25 20:03 k8s-triage-robot

/remove-lifecycle rotten

jackfrancis avatar Mar 06 '25 21:03 jackfrancis

The Kubernetes project currently lacks enough contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle stale
  • Close this PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Jun 04 '25 22:06 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle rotten
  • Close this PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle rotten

k8s-triage-robot avatar Jul 04 '25 22:07 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Reopen this PR with /reopen
  • Mark this PR as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/close

k8s-triage-robot avatar Aug 03 '25 22:08 k8s-triage-robot

@k8s-triage-robot: Closed this PR.

In response to this:

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Reopen this PR with /reopen
  • Mark this PR as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/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-sigs/prow repository.

k8s-ci-robot avatar Aug 03 '25 22:08 k8s-ci-robot