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.

Sign up for GitHub

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

Jump to bottom

Copy Ingress history from kubernetes/contrib #10

Merged
merged 278 commits into from
Nov 10, 2016
Merged

Copy Ingress history from kubernetes/contrib #10

merged 278 commits into from
Nov 10, 2016

Conversation

aledbf
Copy link
Member

@aledbf aledbf commented Nov 10, 2016

aledbf and others added 30 commits March 29, 2016 19:15
@aledbf
Remove examples from the main README.md
045619b
@bprashanth
Merge pull request kubernetes#642 from aledbf/restrict-nginx-status
a06f0a7 
Improve nginx-ingress-controller documentation.
@aledbf
Add ump load balancing and naxsi as WAF. Update nginx to 1.9.13
c9f8a06
@aledbf
Add configuration for retries in non-idempotent requests
7abc7a7
@aledbf
Update Ingress status information in nginx controller
2632fe5
@bprashanth
Clarify default backend creation.
83643b1
@aledbf
Avoid sync Ingress updates
60e2e5f
@aledbf
Update terminationGracePeriodSeconds in examples
2ca6c82
@aledbf
Add test to verify SSL certificate creation
0a71f49
@bprashanth
Merge pull request kubernetes#681 from aledbf/add-lb-status
f22e6d2 
Update Ingress status information in nginx controller
@bprashanth
Readme for tls example.
3464d1c
@joonathan
Updated with correction for Ingress TLS spec
b6ccc1f 
Fix for `error validating data: expected type array, for field spec.tls, got map; if you choose to ignore these errors, turn validation off with --validate=false`
@bprashanth
Simplify urlmap update logic.
5b2de6e
@bprashanth
Single sync error.
ba26bca
@bprashanth
List from cloud and resync InMemoryPool.
8d59739
@bprashanth
Retry GCE client creation.
73afef4
@bprashanth
Bump up glbc version.
2afc21c
@wfhartford
Fix arguments to match replication controller
44d7008 
The DaemonSet as it exists causes the pod to crash on startup with the following:

Error syncing pod, skipping: failed to "StartContainer" for "nginx-ingress-lb" with RunContainerError: "runContainer: API error (500): Cannot start container 7eab41694a7d817665337724b5710bd17d7846f0212408aa9d6056e3211447a4: [8] System error: exec: \"/nginx-ingress-controller-lb\": stat /nginx-ingress-controller-lb: no such file or directory\n"

Modifying the arguments to match the replication controller works.
@bprashanth
Merge pull request kubernetes#706 from joonathan/patch-1
b658ba8 
Updated with correction for Ingress TLS spec
@aledbf
Allow traffic to default server _
0c2e199
@aledbf
Wait until the pod it's running before getting information about it
cf263c1
@aledbf
Add git information during build
49c3af2
@bprashanth
Add an all-in-one rc.yaml
b7e8109
@bprashanth
Add running-in-cluster=false option.
5c8a25c
@bprashanth
Parse cmdline flags so we get --v
71845f3
@bprashanth
Add a multi-tls example.
40a9eb0
@bprashanth
Merge pull request kubernetes#720 from wfhartford/patch-1
4159a40 
Fix daemonset arguments to match replication controller
@bprashanth
Simpler firewall rules
8084341
@bprashanth
Add a use-real-cloud flag to allow the controller to run on the master
fe026b7
@bprashanth
Update certificates on secret update
fc6d9a4
Kubernetes Submit Queue and others added 4 commits November 3, 2016 09:39
Merge pull request kubernetes#1879 from r2d4/map_hash_bucket_size
bff5d0e 
Automatic merge from submit-queue

Make map_hash_bucket_size configurable

I was getting an error while trying to run the nginx controller in minikube.  This allows this nginx configuration option to be passed in through a configmap.  

The default value depends on the processor's cache line size (32 | 64 | 128), however ServerNameHashBucketSize is determined similarly, so I've set it to the same default (64).

Fixes kubernetes#1817

ref kubernetes/minikube#611

cc @bprashanth
@bprashanth
Clarify ingress docs
294ff58
@aledbf
Copy Ingress history from kubernetes/contrib
7ed61b1
@aledbf
Merge branch 'copy-history' of https://github.com/aledbf/contrib into…
55acaab 
… history
@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@aledbf
Copy link
Member Author

aledbf commented Nov 10, 2016

@bprashanth no changes here. This is to keep the history from the ingress directory located in kubernetes/contrib repository

@bprashanth
Copy link
Contributor

LGTM, I think you need to re-cla.
This is just vendors and controllers/nginx,gce right?

@bprashanth bprashanth self-assigned this Nov 10, 2016
@aledbf
Copy link
Member Author

aledbf commented Nov 10, 2016

This is just vendors and controllers/nginx,gce right?

yes. No changes from the current contrib master, just a filtered git copy

I think you need to re-cla.

good question. This failure is not related to my code it appears "aledbf authorized" and this is existing code. I'm not sure if this is a valid warning.

@bprashanth
Copy link
Contributor

Ah, well then

@bprashanth bprashanth merged commit 098bcb5 into kubernetes:master Nov 10, 2016
@aledbf aledbf deleted the history branch November 10, 2016 21:42
@ixdy ixdy mentioned this pull request Feb 2, 2017
Merged
@shashank-g172 shashank-g172 mentioned this pull request Aug 16, 2018
Closed
@kfox1111 kfox1111 mentioned this pull request Mar 13, 2019
Closed
@smg-bg smg-bg mentioned this pull request Oct 19, 2020
Closed
VincentGuoCQ pushed a commit to VincentGuoCQ/ingress-nginx that referenced this pull request Jan 4, 2021
@whalecold
[CPS-7025] fix(metrics): correct metrics service name when traffic wa…
d7b5462 
…s send to canary service (kubernetes#10)

Co-authored-by: Lisheng Zheng <zhenglisheng@caicloud.io>
hanxueluo pushed a commit to hanxueluo/ingress-nginx that referenced this pull request Jan 11, 2021
@whalecold
fix(metrics): correct metrics service name when traffic was send to c…
0ffc278 
…anary service (kubernetes#10)

Co-authored-by: Lisheng Zheng <zhenglisheng@caicloud.io>
This was referenced Jan 27, 2021
Closed
Closed
haoqing0110 referenced this pull request in stolostron/management-ingress Mar 5, 2021
@rashmi43 @qiujian16
Nginx lua bearer check (#10)
8ab45ca
haoqing0110 referenced this pull request in stolostron/management-ingress Mar 5, 2021
@clyang82
Merge pull request #10 from open-cluster-management/update-lab-lic
ee632ee 
update label and lic
@zoemac11 zoemac11 mentioned this pull request May 10, 2021
Closed
@derek-burdick derek-burdick mentioned this pull request Sep 25, 2021
Closed
Alvaro-Campesino added a commit to Alvaro-Campesino/ingress-nginx-k8s that referenced this pull request Sep 1, 2022
@Alvaro-Campesino
Only download vault certificates if they are not in the disk, avoidin…
f3de907 
…g race conditions due to vault response time (kubernetes#10)

* fix changelog

* only download cert if not in disk
Alvaro-Campesino added a commit to Alvaro-Campesino/ingress-nginx-k8s that referenced this pull request Jan 16, 2023
@Alvaro-Campesino @unai-ttxu
Only download vault certificates if they are not in the disk, avoidin…
3516622 
…g race conditions due to vault response time (kubernetes#10)

* fix changelog

* only download cert if not in disk
Alvaro-Campesino added a commit to Alvaro-Campesino/ingress-nginx-k8s that referenced this pull request Jan 24, 2023
@Alvaro-Campesino
Only download vault certificates if they are not in the disk, avoidin…
06e1eea 
…g race conditions due to vault response time (kubernetes#10)

* fix changelog

* only download cert if not in disk
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@bprashanth bprashanth

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.