-
Notifications
You must be signed in to change notification settings - Fork 375
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
fix(scheduler): compose deployment with imagepullsecrets #2456
Conversation
Codecov Report
@@ Coverage Diff @@
## release/1.3 #2456 +/- ##
===============================================
+ Coverage 15.95% 16.08% +0.13%
===============================================
Files 1211 1211
Lines 122014 122062 +48
===============================================
+ Hits 19467 19636 +169
+ Misses 100531 100348 -183
- Partials 2016 2078 +62
|
@@ -455,6 +457,11 @@ func (k *Kubernetes) newDeployment(service *apistructs.Service, sg *apistructs.S | |||
Labels: make(map[string]string), | |||
}, | |||
Spec: apiv1.PodSpec{ | |||
ImagePullSecrets: []apiv1.LocalObjectReference{ | |||
{ | |||
Name: AliyunRegistry, |
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.
ImagePullSecrets can be used as an environment variable
if err == nil { | ||
deployment.Spec.Template.Spec.ImagePullSecrets = []apiv1.LocalObjectReference{ | ||
{ | ||
Name: AliyunRegistry, |
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.
+1
/approve |
/cherry-pick master |
/cherry-pick release/1.4 |
What type of this PR
Add one of the following kinds:
/kind bugfix
What this PR does / why we need it:
Which issue(s) this PR fixes:
Specified Reviewers:
/assign @luobily
ChangeLog
Need cherry-pick to release versions?
Add comment like
/cherry-pick release/1.0
when this PR is merged.