Skip to content

docs: Fix bpfMasquerade option#12302

Merged
aanm merged 1 commit intocilium:masterfrom
sayboras:bugfix/fix-bpfMasquerade-option
Jun 26, 2020
Merged

docs: Fix bpfMasquerade option#12302
aanm merged 1 commit intocilium:masterfrom
sayboras:bugfix/fix-bpfMasquerade-option

Conversation

@sayboras
Copy link
Copy Markdown
Member

@sayboras sayboras commented Jun 26, 2020

The value of bpfMasquerade should be coming from config chart only.
However, few places are still referring to global.

Relates 8ec15e2

docs: Fix bpfMasquerade option

Fixes #12219

The value of bpfMasquerade should be coming from config chart only.
However, few places are still referring to global.

Relates 8ec15e2

Signed-off-by: Tam Mach <sayboras@yahoo.com>
@sayboras sayboras requested review from a team as code owners June 26, 2020 12:12
@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@aanm aanm added the release-note/misc This PR makes changes that have no direct user impact. label Jun 26, 2020
@aanm
Copy link
Copy Markdown
Member

aanm commented Jun 26, 2020

test-me-please

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 36.921% when pulling c6e027e on sayboras:bugfix/fix-bpfMasquerade-option into b99c7b8 on cilium:master.

@nebril
Copy link
Copy Markdown
Member

nebril commented Jun 26, 2020

retest-4.9

@aanm aanm merged commit 0c856f7 into cilium:master Jun 26, 2020
@sayboras sayboras deleted the bugfix/fix-bpfMasquerade-option branch October 17, 2020 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: BPFmasquarade usage for helm option is wrong

7 participants