Revision b58bff5ed942c84aef56015d236c573ea024270f authored by Joe Stringer on 18 May 2018, 18:27:13 UTC, committed by Romain Lenglet on 18 May 2018, 23:01:08 UTC
When adding an L7 rule to an L4 policy, the recent endpoint logic changes did not properly propagate the need to recompile the BPF program, so entirely skipped it, leading to #4181. Fix it by tracking whether the L4 policy changes in regeneratePolicy() and including this in the final "needToRegenerateBPF" result. Fixes: #4181 Signed-off-by: Joe Stringer <joe@covalent.io>
1 parent ae39ed9
pull_request_template.md
<!-- Thanks for contributing to Cilium!
If this is your first time contributing, then please see the following
guidelines for detailed instructions on how to contribute:
https://github.com/cilium/cilium/blob/master/CONTRIBUTING.md for detailed instructions
-->
**Summary of changes**:
Fixes: #issue-number
```release-note
<!-- Enter the release note text here if needed -->
```
**How to test (optional)**:
e.g. Run `make tests`, `make runtime-tests` and/or some more specific way to
exercise the new behavior(s).
Note: [The runtime testsuite will modify the host it runs on.](http://cilium.readthedocs.io/en/latest/contributing/#testsuite)
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...