Skip to content

Remove resource limits for shield; do not need to verify legacy nginx…#781

Merged
awhdesmond merged 1 commit into
mainfrom
awhdesmond/nginx-shield-limits
Jul 6, 2026
Merged

Remove resource limits for shield; do not need to verify legacy nginx…#781
awhdesmond merged 1 commit into
mainfrom
awhdesmond/nginx-shield-limits

Conversation

@awhdesmond

@awhdesmond awhdesmond commented Jul 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Remove resource limits for shield (worker process model don't play nice with CFS) ;
  • do not need to verify legacy nginx as we only have 1 upstream

@awhdesmond awhdesmond merged commit 55c5497 into main Jul 6, 2026
7 checks passed
@awhdesmond awhdesmond deleted the awhdesmond/nginx-shield-limits branch July 6, 2026 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ensonic ensonic ensonic approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@awhdesmond @ensonic