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

bump azure sdk #3916

Merged
merged 1 commit into from May 29, 2023
Merged

bump azure sdk #3916

merged 1 commit into from May 29, 2023

Conversation

flavianmissi
Copy link
Contributor

v1.3.0 of azidentity introduces support to workload identity.


Hi folks! I'm the team lead for OpenShift's internal image registry, and we're adding automagic support for azure workload identities.
The necessary work in the azure sdk was just release under v1.3.0.
I'd like us to upgrade to that version so that distribution can transparently support it.

According to the docs for DefaultAzureCredential, things should just work (given some environment variables are set).

Thanks!

@wy65701436 wy65701436 closed this May 15, 2023
@wy65701436 wy65701436 reopened this May 15, 2023
@codecov-commenter
Copy link

codecov-commenter commented May 15, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.01 🎉

Comparison is base (8e29e87) 56.88% compared to head (7caf058) 56.89%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3916      +/-   ##
==========================================
+ Coverage   56.88%   56.89%   +0.01%     
==========================================
  Files         106      106              
  Lines       10703    10684      -19     
==========================================
- Hits         6088     6079       -9     
+ Misses       3942     3933       -9     
+ Partials      673      672       -1     

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@milosgajdos
Copy link
Member

I'm going to close this and reopen because I think the recent GH outage made DCO check stuck

@milosgajdos milosgajdos reopened this May 19, 2023
@milosgajdos
Copy link
Member

@flavianmissi you need to sign your commit

v1.3.0 of azidentity introduces support to workload identity.

Signed-off-by: Flavian Missi <fmissi@redhat.com>
@flavianmissi
Copy link
Contributor Author

@wy65701436 are you able to help with review?

@milosgajdos milosgajdos merged commit 3f6bf09 into distribution:main May 29, 2023
12 checks passed
@flavianmissi flavianmissi deleted the bump-azure-sdk branch May 30, 2023 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants