cluster-api-provider-azure icon indicating copy to clipboard operation
cluster-api-provider-azure copied to clipboard

provide e2e test for ACR image pulling

Open andyzhangx opened this issue 3 years ago • 6 comments

/kind feature

Describe the solution you'd like [A clear and concise description of what you want to happen.]

We already have in-tree credential provider and will have out of tree credential provider, so we should have e2e test for both scenarios.

Anything else you would like to add: [Miscellaneous information that will assist in solving the issue.]

/kind test

Environment:

  • cluster-api-provider-azure version:
  • Kubernetes version: (use kubectl version):
  • OS (e.g. from /etc/os-release):

andyzhangx avatar Jun 21 '22 01:06 andyzhangx

@andyzhangx: The label(s) kind/test cannot be applied, because the repository doesn't have them.

In response to this:

/kind feature

Describe the solution you'd like [A clear and concise description of what you want to happen.]

We already have in-tree credential provider and will have out of tree credential provider, so we should have e2e test for both scenarios.

Anything else you would like to add: [Miscellaneous information that will assist in solving the issue.]

/kind test

Environment:

  • cluster-api-provider-azure version:
  • Kubernetes version: (use kubectl version):
  • OS (e.g. from /etc/os-release):

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.

k8s-ci-robot avatar Jun 21 '22 01:06 k8s-ci-robot

cc @lzhecheng

andyzhangx avatar Jun 21 '22 01:06 andyzhangx

/assing @lzhecheng

andyzhangx avatar Jun 21 '22 01:06 andyzhangx

/assign @lzhecheng

andyzhangx avatar Jun 21 '22 01:06 andyzhangx

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

This bot triages issues and 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 issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Mark this issue or PR as rotten with /lifecycle rotten
  • Close this issue or 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 Sep 19 '22 02:09 k8s-triage-robot

/remove-lifecycle stale

lzhecheng avatar Sep 19 '22 13:09 lzhecheng

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

This bot triages issues and 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 issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Mark this issue or PR as rotten with /lifecycle rotten
  • Close this issue or 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 Dec 18 '22 13:12 k8s-triage-robot

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

This bot triages issues and 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 issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle rotten
  • Close this issue or 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 Jan 17 '23 14:01 k8s-triage-robot

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

This bot triages issues 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 issue is closed

You can:

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

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

/close not-planned

k8s-triage-robot avatar Feb 16 '23 15:02 k8s-triage-robot

@k8s-triage-robot: Closing this issue, marking it as "Not Planned".

In response to this:

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

This bot triages issues 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 issue is closed

You can:

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

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

/close not-planned

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.

k8s-ci-robot avatar Feb 16 '23 15:02 k8s-ci-robot

/reopen

andyzhangx avatar Apr 19 '23 09:04 andyzhangx

@andyzhangx: Reopened this issue.

In response to this:

/reopen

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.

k8s-ci-robot avatar Apr 19 '23 09:04 k8s-ci-robot

/remove-lifecycle rotten

andyzhangx avatar Apr 19 '23 09:04 andyzhangx

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

This bot triages un-triaged issues 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 issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue 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 Jul 18 '23 09:07 k8s-triage-robot

/remove-lifecycle stale

lzhecheng avatar Aug 17 '23 05:08 lzhecheng

There will be 3 steps to complete the task:

  • [ ] Create a capz cluster with oot kubelet credential provider (linux, windows)
  • [ ] Deploy such cluster with a test-infra job
  • [ ] Verify it works with e2e test code

For the first step, I have verified locally that the cluster template will work for linux worker node. It curls credential provider binary and credential-provider-config.yaml file in kubeadm script. Also it puts kubelet args like feature-gates into kubeletExtraArgs. I can put the template in cloud-provider-azure repo for now but I believe it will be moved to capz repo since the oot is in the roadmap.

For the second step, I am working on capz script. The new script will build kubelet credential provider, just like CCM and CNM images, push them to az storage blob. The behavior will be controlled by an environment variable: TEST_ACR_CREDENTIAL_PROVIDER.

For the third step, I will create a cloudproviderazure ACR (anonymous pull disabled) to hold a hello-world image. The e2e test will simply apply a DaemonSet to ensure the image can be pulled on all worked node (linux and windows).

lzhecheng avatar Aug 17 '23 06:08 lzhecheng

/close Pipeline here: https://testgrid.k8s.io/provider-azure-cloud-provider-azure#cloud-provider-azure-master-credential-provider-capz

lzhecheng avatar Sep 01 '23 02:09 lzhecheng

@lzhecheng: Closing this issue.

In response to this:

/close Pipeline here: https://testgrid.k8s.io/provider-azure-cloud-provider-azure#cloud-provider-azure-master-credential-provider-capz

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.

k8s-ci-robot avatar Sep 01 '23 02:09 k8s-ci-robot