agent icon indicating copy to clipboard operation
agent copied to clipboard

Fix panic when updating config concurrently

Open bejjrajesh opened this issue 11 months ago • 2 comments

Proposed changes

This MR will fix the panic error happen when updating the nginx config concurrently. image

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • [x] I have read the CONTRIBUTING document
  • [ ] I have run make install-tools and have attached any dependency changes to this pull request
  • [ ] If applicable, I have added tests that prove my fix is effective or that my feature works
  • [ ] If applicable, I have checked that any relevant tests pass after adding my changes
  • [ ] If applicable, I have updated any relevant documentation (README.md)
  • [ ] If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

bejjrajesh avatar Mar 19 '25 14:03 bejjrajesh

✅ All required contributors have signed the F5 CLA for this PR. Thank you!
Posted by the CLA Assistant Lite bot.

github-actions[bot] avatar Mar 19 '25 14:03 github-actions[bot]

I have hereby read the F5 CLA and agree to its terms

bejjrajesh avatar Mar 19 '25 14:03 bejjrajesh

@bejjrajesh Can you run the following make commands:

make install-tools
make deps

These will setup some tools, install git hooks and update the vendor folders.

dhurley avatar Apr 30 '25 13:04 dhurley

Closing this PR as further changes are required. This issue will be resolved by this new PR https://github.com/nginx/agent/pull/1088

dhurley avatar May 23 '25 15:05 dhurley