-
Notifications
You must be signed in to change notification settings - Fork 71
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
feat: delegated identity support #244
base: master
Are you sure you want to change the base?
feat: delegated identity support #244
Conversation
This reverts commit e979e99.
T ysalazar/mi for image pull
…salazar/azure-aci into t-ysalazar/mi-for-image-pull
T ysalazar/mi for image pull
dynamic values in mi-image-pull podspecs
Codecov Report
@@ Coverage Diff @@
## master #244 +/- ##
==========================================
+ Coverage 42.92% 43.30% +0.37%
==========================================
Files 35 35
Lines 3471 3621 +150
==========================================
+ Hits 1490 1568 +78
- Misses 1800 1866 +66
- Partials 181 187 +6
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@@ -69,3 +68,69 @@ func (c *Client) ListContainerGroups(ctx context.Context, resourceGroup string) | |||
|
|||
return &list, nil | |||
} | |||
|
|||
func (c *Client) ListAKSClusters(ctx context.Context, resourceGroup string) (*AKSClusterListResult, error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After migrating to use nodeUtils, you should find an easier way to get k8s configs using
k8sClient, err := nodeutil.ClientsetFromEnv(kubeConfigPath)
Delegated Identity support along with MI for image pull
Request sent to ACI for pod with delegated identity