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

NGF unnecessary reloads NGINX when it reconciles Secrets #1112

Closed
pleshakov opened this issue Oct 5, 2023 · 2 comments · Fixed by #2047
Closed

NGF unnecessary reloads NGINX when it reconciles Secrets #1112

pleshakov opened this issue Oct 5, 2023 · 2 comments · Fixed by #2047
Assignees
Labels
area/performance Performance related enhancement New feature or request good first issue Good for newcomers refined Requirements are refined and the issue is ready to be implemented. size/medium Estimated to be completed within a week
Milestone

Comments

@pleshakov
Copy link
Contributor

pleshakov commented Oct 5, 2023

Describe the bug

During longevity test, NGF for some reason periodically reconciled secrets
image

While the reason is unclear why NGF reconciled the Secret, the contents of the Secret (cert and key) wasn't changed. So in any case, NGF should not have regenerated the config and reloaded NGINX.

To Reproduce

Update the Secret without changing it. For example, add an annotation.
See that NFG would reload NGINX.

Expected behavior
no reload of NGINX

Your environment

NGF:

commit: "07d76315931501d878f3ed079142aa1899be1bd3"
date: "2023-09-28T16:49:51Z"
level: "info"
msg: "Starting NGINX Gateway Fabric in static mode"
ts: "2023-09-28T22:35:21Z"
version: "edge"

Kubernetes:

Server Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.3-gke.100", GitCommit:"6466b51b762a5c49ae3fb6c2c7233ffe1c96e48c", GitTreeState:"clean", BuildDate:"2023-06-23T09:27:28Z", GoVersion:"go1.20.5 X:boringcrypto", Compiler:"gc", Platform:"linux/amd64"}

Additional Context

This might be due to the controller-runtime forcing a reconciliation after a certain amount of time.

@pleshakov pleshakov changed the title NGF unnecessary reloads NGINX when it reconcile Secrets NGF unnecessary reloads NGINX when it reconciles Secrets Oct 6, 2023
@kate-osborn kate-osborn added enhancement New feature or request area/performance Performance related labels Oct 9, 2023
@kate-osborn kate-osborn added this to the v1.2.0 milestone Oct 9, 2023
@kate-osborn kate-osborn added refined Requirements are refined and the issue is ready to be implemented. size/medium Estimated to be completed within a week labels Oct 9, 2023
@mpstefan mpstefan modified the milestones: v1.2.0, v1.3.0 Dec 11, 2023
@kate-osborn kate-osborn added temp good first issue Good for newcomers and removed temp labels Dec 18, 2023
@bumarcell
Copy link

Yeah, what's up with the irrelevant secret reconciliation..? 🤔

@bumarcell
Copy link

Found the answer here

@ciarams87 ciarams87 self-assigned this May 28, 2024
@ciarams87 ciarams87 moved this from 🆕 New to 🏗 In Progress in NGINX Gateway Fabric May 28, 2024
@ciarams87 ciarams87 moved this from 🏗 In Progress to 👀 In Review in NGINX Gateway Fabric May 29, 2024
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in NGINX Gateway Fabric May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/performance Performance related enhancement New feature or request good first issue Good for newcomers refined Requirements are refined and the issue is ready to be implemented. size/medium Estimated to be completed within a week
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants