Skip to content
This repository has been archived by the owner on Jan 12, 2022. It is now read-only.

bump: nginx-ingress-controller 0.41.2 #84

Merged
merged 3 commits into from
Dec 2, 2020
Merged

bump: nginx-ingress-controller 0.41.2 #84

merged 3 commits into from
Dec 2, 2020

Conversation

jgehrcke
Copy link
Contributor

@jgehrcke jgehrcke commented Dec 2, 2020

No description provided.

jgehrcke and others added 3 commits December 2, 2020 20:09
@jgehrcke
bump: nginx-ingress-controller 0.41.2
4c2aca6 
Signed-off-by: Dr. Jan-Philip Gehrcke <jp@opstrace.com>
@sreis
chore: bump nginx-controller image to v0.41.2
b5c3806 
https://github.com/kubernetes/ingress-nginx/blob/master/Changelog.md#0412

Signed-off-by: Simão Reis <sreis@opstrace.com>
@sreis
fix: nginx deployment
215d19b 
kubernetes/ingress-nginx#4061 (comment)

Signed-off-by: Simão Reis <sreis@opstrace.com>
jgehrcke
jgehrcke commented Dec 2, 2020
View reviewed changes
packages/controller/src/resources/ingress/nginxIngress.ts
@@ -163,7 +163,7 @@ export function NginxIngressResources(
drop: ["ALL"],
add: ["NET_BIND_SERVICE"]
},
runAsUser: 33
runAsUser: 101
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @sreis!

for the record: kubernetes/ingress-nginx#4061

@jgehrcke
Copy link
Contributor Author

jgehrcke commented Dec 2, 2020 •
edited

test-remote passed, except for the new test (see #72):

[2020-12-02T20:22:39Z] 2020-12-02T20:22:39.810Z info: HTTP resp to GET(https://bk-3120-215-a.opstrace.io/):
[2020-12-02T20:22:39Z]   status: 404
[2020-12-02T20:22:39Z]   body[:500]: <html>
[2020-12-02T20:22:39Z] <head><title>404 Not Found</title></head>
[2020-12-02T20:22:39Z] <body>
[2020-12-02T20:22:39Z] <center><h1>404 Not Found</h1></center>
[2020-12-02T20:22:39Z] <hr><center>nginx</center>
[2020-12-02T20:22:39Z] </body>
[2020-12-02T20:22:39Z] </html>
[2020-12-02T20:22:39Z]
[2020-12-02T20:22:39Z]   headers: {"date":"Wed, 02 Dec 2020 20:22:39 GMT","content-type":"text/html","content-length":"146","connection":"close","strict-transport-security":"max-age=15724800; includeSubDomains"}
[2020-12-02T20:22:39Z]   totalTime: 0.046 s
[2020-12-02T20:22:39Z]   dnsDone->TCPconnectDone: 0.001 s
[2020-12-02T20:22:39Z]   connectDone->reqSent 0 s
[2020-12-02T20:22:39Z]   reqSent->firstResponseByte: 0.005 s
[2020-12-02T20:22:39Z]
[2020-12-02T20:22:39Z]     1) GET / and follow redirects and inspect HTML

...


[2020-12-02T20:23:20Z]   17 passing (41s)
[2020-12-02T20:23:20Z]   1 failing
[2020-12-02T20:23:20Z]
[2020-12-02T20:23:20Z]   1) basic cluster properties
[2020-12-02T20:23:20Z]        GET / and follow redirects and inspect HTML:
[2020-12-02T20:23:20Z]      Error: didn't see expected HTML
[2020-12-02T20:23:20Z]       at Context.<anonymous> (test_cluster_properties.ts:57:11)
[2020-12-02T20:23:20Z]       at processTicksAndRejections (internal/process/task_queues.js:93:5)

@jgehrcke jgehrcke merged commit 339bef0 into main Dec 2, 2020
@jgehrcke jgehrcke deleted the jp/nginx-ingress-041 branch December 2, 2020 20:28
@jgehrcke jgehrcke mentioned this pull request Dec 2, 2020
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jgehrcke @sreis