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

OCPCLOUD-2462: Add Image Credential Provider flags for Kubelet on Azure #4222

Conversation

theobarberbany
Copy link
Contributor

- What I did

Adds configuration for the ACR credential provider to kubelet on Azure
Unit tests show new flags being set correctly.

Builds on #4103

Depends on the acr-credential-provider RPM shipping in RHCOS.

- How to verify it

Kubelet should be able to pull an image from a private ACR registry.

- Description for the changelog

Adds support for azure acr credential provider

@theobarberbany theobarberbany changed the title Add Image Credential Provider flags for Kubelet on Azure OCPCLOUD-2462: Add Image Credential Provider flags for Kubelet on Azure Feb 28, 2024
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Feb 28, 2024
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Feb 28, 2024

@theobarberbany: This pull request references OCPCLOUD-2462 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set.

In response to this:

- What I did

Adds configuration for the ACR credential provider to kubelet on Azure
Unit tests show new flags being set correctly.

Builds on #4103

Depends on the acr-credential-provider RPM shipping in RHCOS.

- How to verify it

Kubelet should be able to pull an image from a private ACR registry.

- Description for the changelog

Adds support for azure acr credential provider

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 openshift-eng/jira-lifecycle-plugin repository.

@theobarberbany
Copy link
Contributor Author

/jira refresh

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Feb 28, 2024

@theobarberbany: This pull request references OCPCLOUD-2462 which is a valid jira issue.

In response to this:

/jira refresh

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 openshift-eng/jira-lifecycle-plugin repository.

@JoelSpeed
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Feb 28, 2024
@theobarberbany
Copy link
Contributor Author

/retest-required

@JoelSpeed
Copy link
Contributor

@theobarberbany Can you identify if there are any Azure E2Es that cover pulling from ACR and run those here using the /payload-job command if possible, else perhaps we can get some QE pre-merge sign off that this is in fact working

@theobarberbany
Copy link
Contributor Author

I can't find any E2Es that are obviously pulling from ACR after a quick dig.

Let's ask QE, as discussed :)

@theobarberbany theobarberbany force-pushed the tb/add-acr-credential-provider-plugin branch from f644a09 to 2f2a62f Compare March 4, 2024 10:30
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Mar 4, 2024
@theobarberbany theobarberbany force-pushed the tb/add-acr-credential-provider-plugin branch 3 times, most recently from 286623c to 7c96b3b Compare March 4, 2024 15:43
@sunzhaohua2
Copy link

/label qe-approved
Tested before pr merge, pull image works well.

az acr login --name zhsunregistry
docker pull openshift/hello-openshift
docker tag openshift/hello-openshift:latest zhsunregistry.azurecr.io/hello-acr:latest
docker push zhsunregistry.azurecr.io/hello-acr:latest
oc new-project hello-acr
oc new-app --name hello-acr --allow-missing-images \
  --image zhsunregistry.azurecr.io/hello-acr:latest
$ oc get po                                                                                                                      
NAME                         READY   STATUS    RESTARTS   AGE
hello-acr-79d8cb6997-q85hv   1/1     Running   0          6s

@openshift-ci openshift-ci bot added the qe-approved Signifies that QE has signed off on this PR label Mar 5, 2024
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Mar 5, 2024

@theobarberbany: This pull request references OCPCLOUD-2462 which is a valid jira issue.

In response to this:

- What I did

Adds configuration for the ACR credential provider to kubelet on Azure
Unit tests show new flags being set correctly.

Builds on #4103

Depends on the acr-credential-provider RPM shipping in RHCOS.

- How to verify it

Kubelet should be able to pull an image from a private ACR registry.

- Description for the changelog

Adds support for azure acr credential provider

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 openshift-eng/jira-lifecycle-plugin repository.

@theobarberbany theobarberbany force-pushed the tb/add-acr-credential-provider-plugin branch from 7c96b3b to 7701e25 Compare March 6, 2024 17:04
@cheesesashimi
Copy link
Member

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 6, 2024
Copy link
Contributor

openshift-ci bot commented Mar 6, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cheesesashimi, JoelSpeed, theobarberbany

The full list of commands accepted by this bot can be found here.

The pull request process is described 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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 6, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit 53bbe70 into openshift:master Mar 6, 2024
15 of 17 checks passed
Copy link
Contributor

openshift-ci bot commented Mar 6, 2024

@theobarberbany: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/okd-scos-e2e-aws-ovn 7701e25 link false /test okd-scos-e2e-aws-ovn

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@openshift-bot
Copy link
Contributor

[ART PR BUILD NOTIFIER]

This PR has been included in build ose-machine-config-operator-container-v4.16.0-202403070215.p0.g53bbe70.assembly.stream.el8 for distgit ose-machine-config-operator.
All builds following this will include this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. qe-approved Signifies that QE has signed off on this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants