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

fix nginx reload flags '-c' #704

Merged
merged 1 commit into from
May 17, 2017
Merged

Conversation

ckeyer
Copy link

@ckeyer ckeyer commented May 11, 2017

add nginx reload flags -c ,
then , will allow yourself to compile the nginx version

go fmt this file

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 11, 2017
@k8s-reviewable
Copy link

This change is Reviewable

@coveralls
Copy link

Coverage Status

Coverage remained the same at 46.983% when pulling dc9664ca503b92339b87097826bb799bcb2e2bcb on ckeyer:fix_nginx_reload into c1cf8ff on kubernetes:master.

@ckeyer
Copy link
Author

ckeyer commented May 11, 2017

@aledbf PTAL

@ckeyer ckeyer force-pushed the fix_nginx_reload branch from dc9664c to 9f32b74 Compare May 17, 2017 10:30
@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) to 46.939% when pulling 9f32b74 on ckeyer:fix_nginx_reload into 8ca5fbe on kubernetes:master.

@aledbf aledbf self-assigned this May 17, 2017
@aledbf
Copy link
Member

aledbf commented May 17, 2017

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 17, 2017
@aledbf
Copy link
Member

aledbf commented May 17, 2017

@ckeyer thanks!

@aledbf aledbf merged commit 49e490e into kubernetes:master May 17, 2017
@ckeyer ckeyer deleted the fix_nginx_reload branch May 21, 2017 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants